> > There seems to be another race: running do_anonymous_page outside the
> > global lock relies on put_page/set_pte being atomic.
> Well, it doesn't, really. It does rely on set_pte() being one single
> memory operation, but it doesn't require it to actually be synchronized
> with other CPU's (it's ok if another CPU sees the old zero value).
How about a comment to this effect? This way, it's less likely that others
overlook the requirement when patching, or loose time chasing an
non-existing race.
BTW, great work, everybody!
-- Horst von Brand vonbrand@sleipnir.valparaiso.cl Casilla 9G, Viņa del Mar, Chile +56 32 672616- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.rutgers.edu Please read the FAQ at http://www.tux.org/lkml/