Re: [PATCH] poll: allow f_op->poll to sleep, take#6

From: Tejun Heo
Date: Thu Nov 27 2008 - 23:44:28 EST


Davide Libenzi wrote:
> Looks OK to me, although it'd be better if some more folks eye it, in
> order to avoid painful mistakes.
> Did you test it al all in a live system?

Depends on what you call a live system. I tested the functionalities
and they worked okay but considering the failure scenarios, it's
highly unlikely to happen especially on x86 cpus with all its nice
memory ordering rules. Now that it has smp_wmb() on the wake up side
and full mb on waking up side, it's as conservative as it gets, so I
think we're in the safe.

Thanks.

--
tejun
--
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/