diff options
author | Yinghai Lu <yhlu.kernel@gmail.com> | 2009-02-07 18:39:41 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-02-09 07:35:07 -0500 |
commit | 7d97277b754d3ee098a5ec69b6aaafb00c94e2f2 (patch) | |
tree | 457ff6256a2da2f39f93500453a6a643ab29ef2b /drivers/acpi/bus.c | |
parent | 05876f88ed9a66b26af613e222795ae790616252 (diff) |
acpi/x86: introduce __apci_map_table, v4
to prevent wrongly overwriting fixmap that still want to use.
ACPI used to rely on low mappings being all linearly mapped and
grew a habit: it never really unmapped certain kinds of tables
after use.
This can cause problems - for example the hypothetical case
when some spurious access still references it.
v2: remove prev_map and prev_size in __apci_map_table
v3: let acpi_os_unmap_memory() call early_iounmap too, so remove extral calling to
early_acpi_os_unmap_memory
v4: fix typo in one acpi_get_table_with_size calling
Signed-off-by: Yinghai Lu <yhlu.kernel@gmail.com>
Acked-by: Len Brown <len.brown@intel.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'drivers/acpi/bus.c')
-rw-r--r-- | drivers/acpi/bus.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/drivers/acpi/bus.c b/drivers/acpi/bus.c index fb1be7b5dbc1..765fd1c56cd6 100644 --- a/drivers/acpi/bus.c +++ b/drivers/acpi/bus.c | |||
@@ -694,12 +694,6 @@ void __init acpi_early_init(void) | |||
694 | if (!acpi_strict) | 694 | if (!acpi_strict) |
695 | acpi_gbl_enable_interpreter_slack = TRUE; | 695 | acpi_gbl_enable_interpreter_slack = TRUE; |
696 | 696 | ||
697 | /* | ||
698 | * Doing a zero-sized mapping will clear out the previous | ||
699 | * __acpi_map_table() mapping, if any. | ||
700 | */ | ||
701 | __acpi_map_table(0, 0); | ||
702 | |||
703 | acpi_gbl_permanent_mmap = 1; | 697 | acpi_gbl_permanent_mmap = 1; |
704 | 698 | ||
705 | status = acpi_reallocate_root_table(); | 699 | status = acpi_reallocate_root_table(); |