(2013/02/22 19:30), Glauber Costa wrote: > After we create a boot cache, we may allocate from it until it is bootstraped. > This will move the page from the partial list to the cpu slab list. If this > happens, the loop: > > list_for_each_entry(p, &n->partial, lru) > > that we use to scan for all partial pages will yield nothing, and the pages > will keep pointing to the boot cpu cache, which is of course, invalid. To do > that, we should flush the cache to make sure that the cpu slab is back to the > partial list. > > Although not verified in practice, I also point out that it is not safe to > scan > the full list only when debugging is on in this case. As unlikely as it is, it > is theoretically possible for the pages to be full. If they are, they will > become unreachable. Aside from scanning the full list, we also need to make > sure that the pages indeed sit in there: the easiest way to do it is to make > sure the boot caches have the SLAB_STORE_USER debug flag set. > > Signed-off-by: Glauber Costa <[email protected]> > Reported-by: Steffen Michalke <[email protected]> > Cc: Christoph Lameter <[email protected]> > Cc: Andrew Morton <[email protected]> > Cc: Tejun Heo <[email protected]> > Cc: Pekka Enberg <[email protected]> > Cc: Kamezawa Hiroyuki <[email protected]> > You're quick :) the issue is fixed in my environ.
Tested-by: KAMEZAWA Hiroyuki <[email protected],com> -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
