mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
synced 2024-12-29 09:12:07 +00:00
Revert "ACPI: Add memory semantics to acpi_os_map_memory()"
This reverts commit437b38c511
. The memory semantics added in commit437b38c511
causes SystemMemory Operation region, whose address range is not described in the EFI memory map to be mapped as NormalNC memory on arm64 platforms (through acpi_os_map_memory() in acpi_ex_system_memory_space_handler()). This triggers the following abort on an ARM64 Ampere eMAG machine, because presumably the physical address range area backing the Opregion does not support NormalNC memory attributes driven on the bus. Internal error: synchronous external abort: 96000410 [#1] SMP Modules linked in: CPU: 0 PID: 1 Comm: swapper/0 Not tainted 5.14.0+ #462 Hardware name: MiTAC RAPTOR EV-883832-X3-0001/RAPTOR, BIOS 0.14 02/22/2019 pstate: 60000005 (nZCv daif -PAN -UAO -TCO -DIT -SSBS BTYPE=--) [...snip...] Call trace: acpi_ex_system_memory_space_handler+0x26c/0x2c8 acpi_ev_address_space_dispatch+0x228/0x2c4 acpi_ex_access_region+0x114/0x268 acpi_ex_field_datum_io+0x128/0x1b8 acpi_ex_extract_from_field+0x14c/0x2ac acpi_ex_read_data_from_field+0x190/0x1b8 acpi_ex_resolve_node_to_value+0x1ec/0x288 acpi_ex_resolve_to_value+0x250/0x274 acpi_ds_evaluate_name_path+0xac/0x124 acpi_ds_exec_end_op+0x90/0x410 acpi_ps_parse_loop+0x4ac/0x5d8 acpi_ps_parse_aml+0xe0/0x2c8 acpi_ps_execute_method+0x19c/0x1ac acpi_ns_evaluate+0x1f8/0x26c acpi_ns_init_one_device+0x104/0x140 acpi_ns_walk_namespace+0x158/0x1d0 acpi_ns_initialize_devices+0x194/0x218 acpi_initialize_objects+0x48/0x50 acpi_init+0xe0/0x498 If the Opregion address range is not present in the EFI memory map there is no way for us to determine the memory attributes to use to map it - defaulting to NormalNC does not work (and it is not correct on a memory region that may have read side-effects) and therefore commit437b38c511
should be reverted, which means reverting back to the original behavior whereby address ranges that are mapped using acpi_os_map_memory() default to the safe devicenGnRnE attributes on ARM64 if the mapped address range is not defined in the EFI memory map. Fixes:437b38c511
("ACPI: Add memory semantics to acpi_os_map_memory()") Signed-off-by: Jia He <justin.he@arm.com> Acked-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> Acked-by: Catalin Marinas <catalin.marinas@arm.com> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
This commit is contained in:
parent
e4e737bb5c
commit
12064c1768
@ -50,9 +50,6 @@ pgprot_t __acpi_get_mem_attribute(phys_addr_t addr);
|
|||||||
void __iomem *acpi_os_ioremap(acpi_physical_address phys, acpi_size size);
|
void __iomem *acpi_os_ioremap(acpi_physical_address phys, acpi_size size);
|
||||||
#define acpi_os_ioremap acpi_os_ioremap
|
#define acpi_os_ioremap acpi_os_ioremap
|
||||||
|
|
||||||
void __iomem *acpi_os_memmap(acpi_physical_address phys, acpi_size size);
|
|
||||||
#define acpi_os_memmap acpi_os_memmap
|
|
||||||
|
|
||||||
typedef u64 phys_cpuid_t;
|
typedef u64 phys_cpuid_t;
|
||||||
#define PHYS_CPUID_INVALID INVALID_HWID
|
#define PHYS_CPUID_INVALID INVALID_HWID
|
||||||
|
|
||||||
|
@ -273,8 +273,7 @@ pgprot_t __acpi_get_mem_attribute(phys_addr_t addr)
|
|||||||
return __pgprot(PROT_DEVICE_nGnRnE);
|
return __pgprot(PROT_DEVICE_nGnRnE);
|
||||||
}
|
}
|
||||||
|
|
||||||
static void __iomem *__acpi_os_ioremap(acpi_physical_address phys,
|
void __iomem *acpi_os_ioremap(acpi_physical_address phys, acpi_size size)
|
||||||
acpi_size size, bool memory)
|
|
||||||
{
|
{
|
||||||
efi_memory_desc_t *md, *region = NULL;
|
efi_memory_desc_t *md, *region = NULL;
|
||||||
pgprot_t prot;
|
pgprot_t prot;
|
||||||
@ -300,11 +299,9 @@ static void __iomem *__acpi_os_ioremap(acpi_physical_address phys,
|
|||||||
* It is fine for AML to remap regions that are not represented in the
|
* It is fine for AML to remap regions that are not represented in the
|
||||||
* EFI memory map at all, as it only describes normal memory, and MMIO
|
* EFI memory map at all, as it only describes normal memory, and MMIO
|
||||||
* regions that require a virtual mapping to make them accessible to
|
* regions that require a virtual mapping to make them accessible to
|
||||||
* the EFI runtime services. Determine the region default
|
* the EFI runtime services.
|
||||||
* attributes by checking the requested memory semantics.
|
|
||||||
*/
|
*/
|
||||||
prot = memory ? __pgprot(PROT_NORMAL_NC) :
|
prot = __pgprot(PROT_DEVICE_nGnRnE);
|
||||||
__pgprot(PROT_DEVICE_nGnRnE);
|
|
||||||
if (region) {
|
if (region) {
|
||||||
switch (region->type) {
|
switch (region->type) {
|
||||||
case EFI_LOADER_CODE:
|
case EFI_LOADER_CODE:
|
||||||
@ -364,16 +361,6 @@ static void __iomem *__acpi_os_ioremap(acpi_physical_address phys,
|
|||||||
return __ioremap(phys, size, prot);
|
return __ioremap(phys, size, prot);
|
||||||
}
|
}
|
||||||
|
|
||||||
void __iomem *acpi_os_ioremap(acpi_physical_address phys, acpi_size size)
|
|
||||||
{
|
|
||||||
return __acpi_os_ioremap(phys, size, false);
|
|
||||||
}
|
|
||||||
|
|
||||||
void __iomem *acpi_os_memmap(acpi_physical_address phys, acpi_size size)
|
|
||||||
{
|
|
||||||
return __acpi_os_ioremap(phys, size, true);
|
|
||||||
}
|
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Claim Synchronous External Aborts as a firmware first notification.
|
* Claim Synchronous External Aborts as a firmware first notification.
|
||||||
*
|
*
|
||||||
|
@ -284,8 +284,7 @@ acpi_map_lookup_virt(void __iomem *virt, acpi_size size)
|
|||||||
#define should_use_kmap(pfn) page_is_ram(pfn)
|
#define should_use_kmap(pfn) page_is_ram(pfn)
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
static void __iomem *acpi_map(acpi_physical_address pg_off, unsigned long pg_sz,
|
static void __iomem *acpi_map(acpi_physical_address pg_off, unsigned long pg_sz)
|
||||||
bool memory)
|
|
||||||
{
|
{
|
||||||
unsigned long pfn;
|
unsigned long pfn;
|
||||||
|
|
||||||
@ -295,8 +294,7 @@ static void __iomem *acpi_map(acpi_physical_address pg_off, unsigned long pg_sz,
|
|||||||
return NULL;
|
return NULL;
|
||||||
return (void __iomem __force *)kmap(pfn_to_page(pfn));
|
return (void __iomem __force *)kmap(pfn_to_page(pfn));
|
||||||
} else
|
} else
|
||||||
return memory ? acpi_os_memmap(pg_off, pg_sz) :
|
return acpi_os_ioremap(pg_off, pg_sz);
|
||||||
acpi_os_ioremap(pg_off, pg_sz);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static void acpi_unmap(acpi_physical_address pg_off, void __iomem *vaddr)
|
static void acpi_unmap(acpi_physical_address pg_off, void __iomem *vaddr)
|
||||||
@ -311,10 +309,9 @@ static void acpi_unmap(acpi_physical_address pg_off, void __iomem *vaddr)
|
|||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* __acpi_os_map_iomem - Get a virtual address for a given physical address range.
|
* acpi_os_map_iomem - Get a virtual address for a given physical address range.
|
||||||
* @phys: Start of the physical address range to map.
|
* @phys: Start of the physical address range to map.
|
||||||
* @size: Size of the physical address range to map.
|
* @size: Size of the physical address range to map.
|
||||||
* @memory: true if remapping memory, false if IO
|
|
||||||
*
|
*
|
||||||
* Look up the given physical address range in the list of existing ACPI memory
|
* Look up the given physical address range in the list of existing ACPI memory
|
||||||
* mappings. If found, get a reference to it and return a pointer to it (its
|
* mappings. If found, get a reference to it and return a pointer to it (its
|
||||||
@ -324,8 +321,8 @@ static void acpi_unmap(acpi_physical_address pg_off, void __iomem *vaddr)
|
|||||||
* During early init (when acpi_permanent_mmap has not been set yet) this
|
* During early init (when acpi_permanent_mmap has not been set yet) this
|
||||||
* routine simply calls __acpi_map_table() to get the job done.
|
* routine simply calls __acpi_map_table() to get the job done.
|
||||||
*/
|
*/
|
||||||
static void __iomem __ref
|
void __iomem __ref
|
||||||
*__acpi_os_map_iomem(acpi_physical_address phys, acpi_size size, bool memory)
|
*acpi_os_map_iomem(acpi_physical_address phys, acpi_size size)
|
||||||
{
|
{
|
||||||
struct acpi_ioremap *map;
|
struct acpi_ioremap *map;
|
||||||
void __iomem *virt;
|
void __iomem *virt;
|
||||||
@ -356,7 +353,7 @@ static void __iomem __ref
|
|||||||
|
|
||||||
pg_off = round_down(phys, PAGE_SIZE);
|
pg_off = round_down(phys, PAGE_SIZE);
|
||||||
pg_sz = round_up(phys + size, PAGE_SIZE) - pg_off;
|
pg_sz = round_up(phys + size, PAGE_SIZE) - pg_off;
|
||||||
virt = acpi_map(phys, size, memory);
|
virt = acpi_map(phys, size);
|
||||||
if (!virt) {
|
if (!virt) {
|
||||||
mutex_unlock(&acpi_ioremap_lock);
|
mutex_unlock(&acpi_ioremap_lock);
|
||||||
kfree(map);
|
kfree(map);
|
||||||
@ -375,17 +372,11 @@ static void __iomem __ref
|
|||||||
mutex_unlock(&acpi_ioremap_lock);
|
mutex_unlock(&acpi_ioremap_lock);
|
||||||
return map->virt + (phys - map->phys);
|
return map->virt + (phys - map->phys);
|
||||||
}
|
}
|
||||||
|
|
||||||
void __iomem *__ref
|
|
||||||
acpi_os_map_iomem(acpi_physical_address phys, acpi_size size)
|
|
||||||
{
|
|
||||||
return __acpi_os_map_iomem(phys, size, false);
|
|
||||||
}
|
|
||||||
EXPORT_SYMBOL_GPL(acpi_os_map_iomem);
|
EXPORT_SYMBOL_GPL(acpi_os_map_iomem);
|
||||||
|
|
||||||
void *__ref acpi_os_map_memory(acpi_physical_address phys, acpi_size size)
|
void *__ref acpi_os_map_memory(acpi_physical_address phys, acpi_size size)
|
||||||
{
|
{
|
||||||
return (void *)__acpi_os_map_iomem(phys, size, true);
|
return (void *)acpi_os_map_iomem(phys, size);
|
||||||
}
|
}
|
||||||
EXPORT_SYMBOL_GPL(acpi_os_map_memory);
|
EXPORT_SYMBOL_GPL(acpi_os_map_memory);
|
||||||
|
|
||||||
|
@ -14,14 +14,6 @@ static inline void __iomem *acpi_os_ioremap(acpi_physical_address phys,
|
|||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
#ifndef acpi_os_memmap
|
|
||||||
static inline void __iomem *acpi_os_memmap(acpi_physical_address phys,
|
|
||||||
acpi_size size)
|
|
||||||
{
|
|
||||||
return ioremap_cache(phys, size);
|
|
||||||
}
|
|
||||||
#endif
|
|
||||||
|
|
||||||
extern bool acpi_permanent_mmap;
|
extern bool acpi_permanent_mmap;
|
||||||
|
|
||||||
void __iomem __ref
|
void __iomem __ref
|
||||||
|
Loading…
Reference in New Issue
Block a user