Re: 2.6.0-test2-mm4

From: William Lee Irwin III (wli@holomorphy.com)
Date: Mon Aug 04 2003 - 09:08:14 EST


On Mon, Aug 04, 2003 at 01:30:36AM -0700, Andrew Morton wrote:
> +4g4g-pmd-fix.patch

If you're going to back out pgd preconstruction, at least back it out
all the way so list poison isn't tripped over randomly on PAE. This is
actually worse than before, since you're basically doing list_del()
on whatever value of page->lru was handed to mm/slab.c from page_alloc.c
in pgd_dtor() multiple times per-page and pounding the lock for no
reason whatsoever on PAE. It's also degrading performance on non-PAE
due to the fact no preconstruction is done there, though harmless due
to the fact the only trace of pgd preconstruction left is the AGP fix.

Someone please tell me they realize this is a backout because absolutely
zero data structure initialization is done in ->ctor() and the entire
thing is memcpy()'d and memset()'d over in the front end to the slab.

I have no idea what, if anything has been absorbed from my prior posts
on this subject. AFAICT I'm getting dead air (or worse) back from
everyone else and no one's even bothering to read the code. i.e. either
no one understands a word I'm saying or no one's listening.

-- wli
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Thu Aug 07 2003 - 22:00:23 EST