Re: [PATCH v5 14/32] efi: Add an EFI table address match function

Tom Lendacky
2017-05-16T21:53:49Z

On 5/15/2017 1:09 PM, Borislav Petkov wrote:
> On Tue, Apr 18, 2017 at 04:18:48PM -0500, Tom Lendacky wrote:
>> Add a function that will determine if a supplied physical address matches
>> the address of an EFI table.
>>
>> Signed-off-by: Tom Lendacky <thomas.lendacky@amd.com>
>> ---
>> drivers/firmware/efi/efi.c | 33 +++++++++++++++++++++++++++++++++
>> include/linux/efi.h | 7 +++++++
>> 2 files changed, 40 insertions(+)
>>
>> diff --git a/drivers/firmware/efi/efi.c b/drivers/firmware/efi/efi.c
>> index b372aad..8f606a3 100644
>> --- a/drivers/firmware/efi/efi.c
>> +++ b/drivers/firmware/efi/efi.c
>> @@ -55,6 +55,25 @@ struct efi __read_mostly efi = {
>> };
>> EXPORT_SYMBOL(efi);
>>
>> +static unsigned long *efi_tables[] = {
>> + &efi.mps,
>> + &efi.acpi,
>> + &efi.acpi20,
>> + &efi.smbios,
>> + &efi.smbios3,
>> + &efi.sal_systab,
>> + &efi.boot_info,
>> + &efi.hcdp,
>> + &efi.uga,
>> + &efi.uv_systab,
>> + &efi.fw_vendor,
>> + &efi.runtime,
>> + &efi.config_table,
>> + &efi.esrt,
>> + &efi.properties_table,
>> + &efi.mem_attr_table,
>> +};
>> +
>> static bool disable_runtime;
>> static int __init setup_noefi(char *arg)
>> {
>> @@ -854,6 +873,20 @@ int efi_status_to_err(efi_status_t status)
>> return err;
>> }
>>
>> +bool efi_table_address_match(unsigned long phys_addr)
>
> efi_is_table_address() reads easier/better in the code.

Will do.

Thanks,
Tom

>