(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 <glom...@parallels.com> > Reported-by: Steffen Michalke <stmicha...@web.de> > Cc: Christoph Lameter <c...@linux.com> > Cc: Andrew Morton <a...@linux-foundation.org> > Cc: Tejun Heo <t...@kernel.org> > Cc: Pekka Enberg <penb...@kernel.org> > Cc: Kamezawa Hiroyuki <kamezawa.hir...@jp.fujitsu.com> > You're quick :) the issue is fixed in my environ.
Tested-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu,com> -- 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/
