[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <924EFEDD5F540B4284297C4DC59F3DEE5E88DF@orsmsx423.amr.corp.intel.com>
Date: Tue, 15 Jan 2008 18:51:25 -0800
From: "Pallipadi, Venkatesh" <venkatesh.pallipadi@...el.com>
To: "Huang, Ying" <ying.huang@...el.com>, "Ingo Molnar" <mingo@...e.hu>
Cc: <akpm@...ux-foundation.org>, "H. Peter Anvin" <hpa@...or.com>,
"Thomas Gleixner" <tglx@...utronix.de>,
"Ingo Molnar" <mingo@...hat.com>, "Andi Kleen" <ak@...e.de>,
<linux-kernel@...r.kernel.org>
Subject: RE: [PATCH -mm 0/3] i386 boot: replace boot_ioremap with enhancedbt_ioremap
>-----Original Message-----
>From: Huang, Ying
>Sent: Tuesday, January 15, 2008 1:49 AM
>To: Ingo Molnar; Pallipadi, Venkatesh
>Cc: akpm@...ux-foundation.org; H. Peter Anvin; Thomas
>Gleixner; Ingo Molnar; Andi Kleen; linux-kernel@...r.kernel.org
>Subject: Re: [PATCH -mm 0/3] i386 boot: replace boot_ioremap
>with enhancedbt_ioremap
>
>On Tue, 2008-01-15 at 09:44 +0100, Ingo Molnar wrote:
>> * Huang, Ying <ying.huang@...el.com> wrote:
>>
>> > This patchset replaces boot_ioremap with a enhanced version of
>> > bt_ioremap and renames the bt_ioremap to early_ioremap.
>This reduces
>> > 12k from .init.data segment and increases the size of
>memory that can
>> > be re-mapped before paging_init to 64k.
>>
>> in latest x86.git#mm there's an early_ioremap() introduced
>as part of
>> the PAT series - available on both 32-bit and 64-bit. Could
>you take a
>> look at it and use that if it's OK for your purposes?
>
>After checking the early_ioremap() implementation in
>arch/x86/kernel/setup_32.c, I found that it is a duplication of
>bt_ioremap() implementation in arch/x86/mm/ioremap_32.c. Both
>implementations use set_fixmap(), so they can be used only after
>paging_init().
>
>The early_ioremap implementation provided in this patchset works as
>follow:
>
>- Enhances bt_ioremap, make it usable before paging_init() via a
>dedicated PTE page.
>- Rename bt_ioremap to early_ioremap
>
>So I think maybe we should replace the early_ioremap()
>implementation in
>PAT series with that of this series.
>
Agreed. PAT can use this for early mappings. Thanks for the patches :)
-Venki
--
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