[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Pine.LNX.4.64.0805081951570.21297@blonde.site>
Date: Thu, 8 May 2008 19:58:47 +0100 (BST)
From: Hugh Dickins <hugh@...itas.com>
To: Dave Hansen <dave@...ux.vnet.ibm.com>
cc: Nishanth Aravamudan <nacc@...ibm.com>,
Hans Rosenfeld <hans.rosenfeld@....com>,
Ingo Molnar <mingo@...e.hu>,
Jeff Chua <jeff.chua.linux@...il.com>,
Thomas Gleixner <tglx@...utronix.de>,
"H. Peter Anvin" <hpa@...or.com>,
Gabriel C <nix.or.die@...glemail.com>,
Arjan van de Ven <arjan@...ux.intel.com>,
linux-kernel@...r.kernel.org, linux-mm@...ck.org
Subject: Re: [PATCH] x86: fix PAE pmd_bad bootup warning
On Thu, 8 May 2008, Dave Hansen wrote:
>
> But, I do think it is absolutely insane to have pmd_clear_bad() going
> after perfectly good hugetlb pmds. The way it is set up now, people are
> bound to miss the hugetlb pages because just about every single
> pagetable walk has to be specially coded to handle or avoid them. We
> obviously missed it, here, and we had two good examples in the same
> file! :)
Like it or not, the pgd/pud/pmd/pte hierarchy cannot be assumed once
you're amongst hugepages. What happens varies from architecture to
architecture. Perhaps the hugepage specialists could look at what
in fact the different architectures we know today are doing, and
come up with a better abstraction to encompass them all. But it's
simply wrong for a "generic" pagewalker to be going blindly in there.
Two good examples in the same file??
Hugh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists