lkml.org 
[lkml]   [2011]   [Apr]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [RFC PATCH 4/5] RCU: Add TASK_RCU_OFFSET
    From
    Date
    On Mon, 2011-04-11 at 14:02 -0700, Paul E. McKenney wrote:
    > On Mon, Apr 11, 2011 at 04:31:10PM +0800, Lai Jiangshan wrote:
    > > On 04/11/2011 01:12 PM, Paul E. McKenney wrote:
    > >
    > > > -static inline struct task_struct *next_thread(const struct task_struct *p)
    > > > -{
    > > > - return list_entry_rcu(p->thread_group.next,
    > > > - struct task_struct, thread_group);
    > > > -}
    > > > +/* Avoid #include hell for inlining rcu_read_lock(). */
    > > > +#define next_thread(p) \
    > > > + list_entry_rcu((p)->thread_group.next, struct task_struct, thread_group)
    > > >

    > > I admit the work will become very much simpler if this pattern is allowed.
    >
    > The guy who maintains much of sched.h suggested it. ;-)

    While its not the nicest thing its something that's on more than one
    occasion there's an existing site in sched.h and there's on in key.h and
    iirc there's another one that's relevant.

    A nice solution would be to move a lot of things out of sched.h into
    something like process.h, a header that deals with task groups and
    processes.

    But until that time, I see no problem with using this.


    \
     
     \ /
      Last update: 2011-04-11 23:27    [W:0.045 / U:152.888 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site