Re: [PATCH v2 08/10] arm64: dmi: Add SMBIOS/DMI support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Oct 28, 2014 at 05:18:41PM +0100, Ard Biesheuvel wrote:
> From: Yi Li <yi.li@xxxxxxxxxx>
> 
> SMBIOS is important for server hardware vendors. It implements a spec for
> providing descriptive information about the platform. Things like serial
> numbers, physical layout of the ports, build configuration data, and the like.
> 
> Signed-off-by: Yi Li <yi.li@xxxxxxxxxx>
> Tested-by: Suravee Suthikulpanit <suravee.suthikulpanit@xxxxxxx>
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>

On FVP Base AEMv8-A (UEFI), and qemu (-kernel, so non-UEFI):
Tested-by: Leif Lindholm <leif.lindholm@xxxxxxxxxx>

> ---
> A short history of this patch:
> 
> v4: Moved call to dmi_scan_machine() to separate core_initcall(), so that it is
>     called unconditionally, i.e., even if UEFI fails to initialize. Otherwise,
>     any drivers that attempt to consult DMI info for quirks handling start
>     spewing errors, as Catalin unfortunately found out after merging (and
>     subsequently reverting) this patch for the second time.
> 
> v3: Moved call to dmi_scan_machine() into arm64_enter_virtual_mode(). This is
>     necessary, because dmi_scan_machine() needs to be called before
>     dmi_id_init(), which itself is invoked using an arch_initcall(). DMI depends
>     on UEFI on arm64, so it is legal to only invoke dmi_scan_machine() when
>     building with UEFI support. However, calling it from
>     arm64_enter_virtual_mode() was a mistake, as it could result in
>     dmi_scan_machine() not being called at all.
> 
> v2: Use efi_lookup_mapped_addr() to obtain the virtual address of the SMBIOS
>     structure table instead of calling ioremap_cache(). This seemed a good idea
>     at the time, as the UEFI memory map covers those regions, so the virtual
>     mapping should be known as well. However, this is only true if the firmware
>     has requested a virtual remapping of the region by setting the
>     EFI_MEMORY_RUNTIME bit, which Tianocore/EDK2 appears to do, but violates
>     the UEFI spec. ("In general, UEFI Configuration Tables loaded at boot time
>     (e.g., SMBIOS table) can be contained in memory of type
>     EfiRuntimeServicesData (recommended and the system firmware must not request
>     a virtual mapping), [...]", section 2.3.6, UEFI spec v2.4B). This version
>     was merged into the arm64 for-next/core branch and reverted again per our
>     request.
> ---
>  arch/arm64/Kconfig           | 11 +++++++++++
>  arch/arm64/include/asm/dmi.h | 31 +++++++++++++++++++++++++++++++
>  arch/arm64/kernel/efi.c      | 13 +++++++++++++
>  3 files changed, 55 insertions(+)
>  create mode 100644 arch/arm64/include/asm/dmi.h
> 
> diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig
> index 9532f8d5857e..2c3c2ca6f8bc 100644
> --- a/arch/arm64/Kconfig
> +++ b/arch/arm64/Kconfig
> @@ -401,6 +401,17 @@ config EFI
>  	  allow the kernel to be booted as an EFI application. This
>  	  is only useful on systems that have UEFI firmware.
>  
> +config DMI
> +	bool "Enable support for SMBIOS (DMI) tables"
> +	depends on EFI
> +	default y
> +	help
> +	  This enables SMBIOS/DMI feature for systems.
> +
> +	  This option is only useful on systems that have UEFI firmware.
> +	  However, even with this option, the resultant kernel should
> +	  continue to boot on existing non-UEFI platforms.
> +
>  endmenu
>  
>  menu "Userspace binary formats"
> diff --git a/arch/arm64/include/asm/dmi.h b/arch/arm64/include/asm/dmi.h
> new file mode 100644
> index 000000000000..69d37d87b159
> --- /dev/null
> +++ b/arch/arm64/include/asm/dmi.h
> @@ -0,0 +1,31 @@
> +/*
> + * arch/arm64/include/asm/dmi.h
> + *
> + * Copyright (C) 2013 Linaro Limited.
> + * Written by: Yi Li (yi.li@xxxxxxxxxx)
> + *
> + * based on arch/ia64/include/asm/dmi.h
> + *
> + * This file is subject to the terms and conditions of the GNU General Public
> + * License.  See the file "COPYING" in the main directory of this archive
> + * for more details.
> + */
> +
> +#ifndef __ASM_DMI_H
> +#define __ASM_DMI_H
> +
> +#include <linux/io.h>
> +#include <linux/slab.h>
> +
> +/*
> + * According to section 2.3.6 of the UEFI spec, the firmware should not
> + * request a virtual mapping for configuration tables such as SMBIOS.
> + * This means we have to map them before use.
> + */
> +#define dmi_early_remap(x, l)		ioremap_cache(x, l)
> +#define dmi_early_unmap(x, l)		iounmap(x)
> +#define dmi_remap(x, l)			ioremap_cache(x, l)
> +#define dmi_unmap(x)			iounmap(x)
> +#define dmi_alloc(l)			kzalloc(l, GFP_KERNEL)
> +
> +#endif
> diff --git a/arch/arm64/kernel/efi.c b/arch/arm64/kernel/efi.c
> index 558572ef1ea3..9ae5e7918b8f 100644
> --- a/arch/arm64/kernel/efi.c
> +++ b/arch/arm64/kernel/efi.c
> @@ -11,6 +11,7 @@
>   *
>   */
>  
> +#include <linux/dmi.h>
>  #include <linux/efi.h>
>  #include <linux/export.h>
>  #include <linux/memblock.h>
> @@ -469,3 +470,15 @@ err_unmap:
>  	return -1;
>  }
>  early_initcall(arm64_enter_virtual_mode);
> +
> +static int __init arm64_dmi_init(void)
> +{
> +	/*
> +	 * On arm64, DMI depends on UEFI, and dmi_scan_machine() needs to
> +	 * be called early because dmi_id_init(), which is an arch_initcall
> +	 * itself, depends on dmi_scan_machine() having been called already.
> +	 */
> +	dmi_scan_machine();
> +	return 0;
> +}
> +core_initcall(arm64_dmi_init);
> -- 
> 1.8.3.2
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-efi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux