Daniel Phillips <[EMAIL PROTECTED]> writes: > On Wednesday 19 November 2008 04:17, OGAWA Hirofumi wrote: >> Unfortunately, quick search was right. SLUB didn't warn by >> test. (probably, I should send a patch to lkml) So, I tweaked malloc() >> like the following. >> >> might_sleep() is only for CONFIG_SLUB. And we would like to use >> GFP_NOFS, not GFP_KERNEL. >> >> If it's ok, please pull again. Thanks. > > There is no real harm in harm in having an extra might_sleep for now. > Let's just leave that one the way you originally had it instead of > adding a new IFDEF. Please shout if that is not convenient (i.e., > already committed more to your local repo). I was just curious why > you saw the need and now we know: it was inadvertently left out of > SLUB.
Ok. I'll leave might_sleep as is, and will be queueing GFP_NOFS for next series later. Thanks. -- OGAWA Hirofumi <[EMAIL PROTECTED]> _______________________________________________ Tux3 mailing list [email protected] http://tux3.org/cgi-bin/mailman/listinfo/tux3
