Re: [PATCH 6 of 7] x86: use early_ioremap in __acpi_map_table

From: Andi Kleen
Date: Sun Sep 07 2008 - 19:40:56 EST


On Sun, Sep 07, 2008 at 03:21:18PM -0700, Jeremy Fitzhardinge wrote:
> __acpi_map_table() effectively reimplements early_ioremap(). Rather
> than have that duplication, just implement it in terms of
> early_ioremap().
>
> However, unlike early_ioremap(), __acpi_map_table() just maintains a
> single mapping which gets replaced each call, and has no corresponding
> unmap function. Implement this by just removing the previous mapping
> each time its called. Unfortunately, this will leave a stray mapping
> at the end.

It would be better to just fix the ACPI code to unmap.

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