Re: [patch 2/3] RCU move trace defines to rcupdate_defines.h

[Ingo Molnar]

* Mathieu Desnoyers <mathieu.desnoyers@xxxxxxxxxx> wrote:

> * Ingo Molnar (mingo@xxxxxxx) wrote:
> > 
> > * mathieu.desnoyers@xxxxxxxxxx <mathieu.desnoyers@xxxxxxxxxx> wrote:
> > 
> > >  include/linux/rcupdate.h         |  127 ------------------------------
> > >  include/linux/rcupdate_defines.h |  163 +++++++++++++++++++++++++++++++++++++++
> > 
> > Looks good to me (if Paul accepts it too), but please use the 
> > standard convention we use in other places for 'light' headers: 
> > rcupdate_types.h.
> > 
> > 	Ingo
> 
> Given there were not "types" declared per-se, I thought _defines 
> was more appropriate. But I don't mind, as long as we separate the 
> heavy from the "light" header. I'll repost with _types.h.

Ok - but it's really up to Paul - i'll wait for his take.

	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/