On Wed, Aug 09, 2006 at 10:11:59AM +0400, Evgeniy Polyakov wrote:
> On Tue, Aug 08, 2006 at 10:52:30PM -0700, David Miller ([EMAIL PROTECTED]) 
> wrote:
> > > Using LIST_POISON is a flag that kevent is in appropriate queue or not,
> > > I can add some flag into the structure, but why, if it is clear just by
> > > looking into list's pointers.
> > 
> > What is wrong with using list_empty() as this indicator?
> 
> RCU only replaces ->prev pointer, and list_empty(entry) checks for
> entry->next == head, but actually I do not see how kevent entry can
> point to itself, so I will try this.

Looking at LIST_POISON1 is still not acceptable at all.  It's an internal
implementation details that goes and comes back once in a while.  We already
had a past discussion on a use of this and it went away, maybe Andrew
can remember it.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to