[lkml]   [2005]   [Jun]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: patch] Real-Time Preemption, plist fixes
On Sat, 2005-06-04 at 17:53 -0700, Daniel Walker wrote:

> You mean really annoying in the plist code?

Usually I mean what I write.

> > 2. Add the proper list_head initializer in the replacement path.
> Not sure what you mean here.

list_replace_rcu(&pl->dp_node, &pl_new->dp_node);
+ INIT_LIST_HEAD(&pl->dp_node);

> > 4. Make plist_entry() work as expected by anybody who ever used
> > list_entry(). Add a plist_first_entry macro for those places where the
> > provided functionality was accidentaly correct.
> >
> I already released a patch to fix this.

Nice to know. Where ?

> > 5. Modify the comments in the header file to explain the real intention
> > of the implemenation.
> You should wait till it's stable before you finalize the documentation.

Thats plain wrong. Having explanations in place which do not match the
code is worse than no explanation at all.


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2005-06-05 10:35    [W:0.105 / U:2.780 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site