Messages in this thread | | | From | Andi Kleen <> | Subject | Re: [PATCH] x86_64: fix page table size | Date | Thu, 7 Feb 2008 18:28:25 +0100 |
| |
On Thursday 07 February 2008 12:54:42 Ingo Molnar wrote: > * Andi Kleen <andi@firstfloor.org> wrote: > > Yinghai Lu <Yinghai.Lu@Sun.COM> writes: > > > diff --git a/arch/x86/mm/init_64.c b/arch/x86/mm/init_64.c > > > index eb376b5..31f0e82 100644 > > > --- a/arch/x86/mm/init_64.c > > > +++ b/arch/x86/mm/init_64.c > > > @@ -370,7 +370,7 @@ static void __init find_early_table_space(unsigned > > > long end) > > > > > > puds = (end + PUD_SIZE - 1) >> PUD_SHIFT; > > > tables = round_up(puds * sizeof(pud_t), PAGE_SIZE); > > > - if (direct_gbpages) { > > > + if (!direct_gbpages) { > > > > What tree did you use? The patch I submitted had > > > > if (direct_gbpages == GBP_ON) { > > yes, the bug was introduced in your original submission of gbpages
I see yes. The original was ok I think, but it must have been a typo when I switched the boolean to a enum on Thomas request and for some reason the new breakage didn't show up on my testing.
I wonder why you didn't keep the enum even though Thomas insisted on it. Since you removed it again the safest would have been to just keep it correct as it originally was. And it was rather pointless to force me to do changes when you then afterwards half way rewrite the code anyways. To be honest that habit makes it rather unpleasant to submit patch to you recently. At least you could have indicated that in advance and safe everybody trouble.
-Andi
| |