Re: [PATCH] sh: remove stray markers

From: Ingo Molnar
Date: Thu May 07 2009 - 03:30:56 EST



* Paul Mundt <lethal@xxxxxxxxxxxx> wrote:

> On Wed, May 06, 2009 at 01:15:49PM +0200, Ingo Molnar wrote:
> >
> > * Christoph Hellwig <hch@xxxxxx> wrote:
> >
> > > On Wed, May 06, 2009 at 01:07:35PM +0200, Ingo Molnar wrote:
> > > > Also, if there's some current use of these markers then it's OK
> > > > to not do this change right now but replace them with
> > > > tracepoints in the v2.6.31 merge window, after both the SH and
> > > > the tracing infrastructure changes are upstream.
> > >
> > > There's not user of these. [...]
> >
> > I'll defer to Paul to decide whether SH needs those markers. If he
> > added them to ease his own debugging (via whatever external
> > approach), then that's an OK goal and we can handle this all
> > politely without destroying functionality, in the next merge window.
>
> They were primarily added to aid some downstream folks that were
> just adding these on their own and generally not paying attention
> to where the markers were being placed on subsequent merges. I
> don't feel too strongly one way or the other, but as they have
> proven to be useful for some people, I would rather transition
> over to tracepoints during the merge window than kill them off
> prematurely.

fair enough!

Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/