On Tue, May 28, 2013 at 05:34:53PM -0700, Eric Dumazet wrote: > On Tue, 2013-05-28 at 13:10 +0400, Roman Gushchin wrote: > > On 28.05.2013 04:12, Eric Dumazet wrote: > > > > Adding a barrier() is probably what we want. > > > > I agree, inserting barrier() is also a correct and working fix. > > Yeah, but I can not find a clean way to put it inside the "for (;;)" > > for (barrier();;) -> > > error: expected expression before ‘__asm__’ > > No user currently does : > > if (condition) > hlist_nulls_for_each_entry_rcu(tpos, pos, head, member) > > But who knows...
I still have my earlier question, but I suggest "({ barrier(); XXX })" to put the barrier into the for loop, either in the second or third clause, where XXX was the original second or third clause. Thanx, Paul -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/