ACPICA: Use original data_table_region pointer for accesses
authorJessica Clarke <jrtc27@jrtc27.com>
Wed, 22 Dec 2021 16:21:25 +0000 (17:21 +0100)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Mon, 27 Dec 2021 16:01:28 +0000 (17:01 +0100)
commitca25f92b72d25457653dbf2a81f322235804fb05
treef2f76e44ffbcc7e0c05e1a0adb4f04ee4b6f2a28
parentf81bdeaf816142e0729eea0cc84c395ec9673151
ACPICA: Use original data_table_region pointer for accesses

ACPICA commit d9eb82bd7515989f0b29d79deeeb758db4d6529c

Currently the pointer to the table is cast to acpi_physical_address and
later cast back to a pointer to be dereferenced. Whether or not this is
supported is implementation-defined.

On CHERI, and thus Arm's experimental Morello prototype architecture,
pointers are represented as capabilities, which are unforgeable bounded
pointers, providing always-on fine-grained spatial memory safety. This
means that any pointer cast to a plain integer will lose all its
associated metadata, and when cast back to a pointer it will give a
null-derived pointer (one that has the same metadata as null but an
address equal to the integer) that will trap on any dereference. As a
result, this is an implementation where acpi_physical_address cannot be
used as a hack to store real pointers.

Thus, add a new field to struct acpi_object_region to store the pointer for
table regions, and propagate it to acpi_ex_data_table_space_handler via the
region context, to use a more portable implementation that supports
CHERI.

Link: https://github.com/acpica/acpica/commit/d9eb82bd
Signed-off-by: Bob Moore <robert.moore@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/acpica/acevents.h
drivers/acpi/acpica/acobject.h
drivers/acpi/acpica/dsopcode.c
drivers/acpi/acpica/evhandler.c
drivers/acpi/acpica/evrgnini.c
drivers/acpi/acpica/excreate.c
drivers/acpi/acpica/exregion.c
include/acpi/actypes.h