[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Pine.LNX.4.64.0802060428050.9030@blonde.site>
Date: Wed, 6 Feb 2008 04:49:49 +0000 (GMT)
From: Hugh Dickins <hugh@...itas.com>
To: Valdis.Kletnieks@...edu
cc: Andrew Morton <akpm@...ux-foundation.org>,
Ingo Molnar <mingo@...e.hu>,
Thomas Gleixner <tglx@...utronix.de>, Andi Kleen <ak@...e.de>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH mm] stop c_p_a corrupting the pds
On Tue, 5 Feb 2008, Valdis.Kletnieks@...edu wrote:
> On Tue, 05 Feb 2008 22:27:21 GMT, Hugh Dickins said:
> > When change_page_attr splits a large page on x86_32 (without PAE), it is
> > currently corrupting every process's page directory: fix that by removing
> > the thinko which passes down a physical instead of a virtual address -
> > this version of the patch being the hotfix for 2.6.24-mm1.
> >
> > Signed-off-by: Hugh Dickins <hugh@...itas.com>
>
> <snark>
> I *knew* there was a reason we should have had this patch series in -mm for a while.
> </snark>
>
> :)
Seriously, I do agree with you on that. It seems like the excitement
of making great changes has overtaken proper caution here.
Though I guess it was just coincidence that made it more debuggable
in my -mm kernel (which gave "bad pgd" errors after starting X), when
the -git kernel just crashed somehow in starting X. And I was lucky
to have CONFIG_VMSPLIT_2G_OPT on that machine, which placed the
corruption somewhere that soon got noticed.
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