Re: [PATCH v1 1/6] efi: Switch to use new generic UUID API

[Andy Shevchenko]

On Thu, 2017-07-20 at 13:18 +0100, Ard Biesheuvel wrote:
> On 19 July 2017 at 19:28, Andy Shevchenko
> <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote:
> > There are new types and helpers that are supposed to be used in new
> > code.
> > 
> > As a preparation to get rid of legacy types and API functions do
> > the conversion here.
> > 
> > Cc: Matt Fleming <matt@xxxxxxxxxxxxxxxxxxx>
> > Cc: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
> > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> 
> Acked-by: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>


Thanks!

Christoph, can we apply this one at least to move things forward?

> 
> > ---
> > Âdrivers/firmware/efi/cper.c | 10 ++---
> > Âinclude/linux/cper.hÂÂÂÂÂÂÂÂ| 94 ++++++++++++++++++++++----------
> > -------------
> > Âinclude/linux/efi.hÂÂÂÂÂÂÂÂÂ|ÂÂ4 +-
> > Â3 files changed, 54 insertions(+), 54 deletions(-)
> > 
> > diff --git a/drivers/firmware/efi/cper.c
> > b/drivers/firmware/efi/cper.c
> > index 48a8f69da42a..684e65c11dde 100644
> > --- a/drivers/firmware/efi/cper.c
> > +++ b/drivers/firmware/efi/cper.c
> > @@ -534,7 +534,7 @@ static void
> > Âcper_estatus_print_section(const char *pfx, struct
> > acpi_hest_generic_data *gdata,
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂint sec_no)
> > Â{
> > -ÂÂÂÂÂÂÂuuid_le *sec_type = (uuid_le *)gdata->section_type;
> > +ÂÂÂÂÂÂÂguid_t *sec_type = (guid_t *)gdata->section_type;
> > ÂÂÂÂÂÂÂÂ__u16 severity;
> > ÂÂÂÂÂÂÂÂchar newpfx[64];
> > 
> > @@ -545,12 +545,12 @@ cper_estatus_print_section(const char *pfx,
> > struct acpi_hest_generic_data *gdata
> > ÂÂÂÂÂÂÂÂprintk("%s""Error %d, type: %s\n", pfx, sec_no,
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂcper_severity_str(severity));
> > ÂÂÂÂÂÂÂÂif (gdata->validation_bits & CPER_SEC_VALID_FRU_ID)
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂprintk("%s""fru_id: %pUl\n", pfx, (uuid_le *)gdata-
> > >fru_id);
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂprintk("%s""fru_id: %pUl\n", pfx, gdata->fru_id);
> > ÂÂÂÂÂÂÂÂif (gdata->validation_bits & CPER_SEC_VALID_FRU_TEXT)
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂprintk("%s""fru_text: %.20s\n", pfx, gdata-
> > >fru_text);
> > 
> > ÂÂÂÂÂÂÂÂsnprintf(newpfx, sizeof(newpfx), "%s%s", pfx, INDENT_SP);
> > -ÂÂÂÂÂÂÂif (!uuid_le_cmp(*sec_type, CPER_SEC_PROC_GENERIC)) {
> > +ÂÂÂÂÂÂÂif (guid_equal(sec_type, &CPER_SEC_PROC_GENERIC)) {
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂstruct cper_sec_proc_generic *proc_err =
> > acpi_hest_get_payload(gdata);
> > 
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂprintk("%s""section_type: general processor
> > error\n", newpfx);
> > @@ -558,7 +558,7 @@ cper_estatus_print_section(const char *pfx,
> > struct acpi_hest_generic_data *gdata
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂcper_print_proc_generic(newpfx, proc_err);
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂelse
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂgoto err_section_too_small;
> > -ÂÂÂÂÂÂÂ} else if (!uuid_le_cmp(*sec_type, CPER_SEC_PLATFORM_MEM)) {
> > +ÂÂÂÂÂÂÂ} else if (guid_equal(sec_type, &CPER_SEC_PLATFORM_MEM)) {
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂstruct cper_sec_mem_err *mem_err =
> > acpi_hest_get_payload(gdata);
> > 
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂprintk("%s""section_type: memory error\n", newpfx);
> > @@ -568,7 +568,7 @@ cper_estatus_print_section(const char *pfx,
> > struct acpi_hest_generic_data *gdata
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂgdata->error_data_length);
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂelse
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂgoto err_section_too_small;
> > -ÂÂÂÂÂÂÂ} else if (!uuid_le_cmp(*sec_type, CPER_SEC_PCIE)) {
> > +ÂÂÂÂÂÂÂ} else if (guid_equal(sec_type, &CPER_SEC_PCIE)) {
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂstruct cper_sec_pcie *pcie =
> > acpi_hest_get_payload(gdata);
> > 
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂprintk("%s""section_type: PCIe error\n", newpfx);
> > diff --git a/include/linux/cper.h b/include/linux/cper.h
> > index 4c671fc2081e..723e952fde0d 100644
> > --- a/include/linux/cper.h
> > +++ b/include/linux/cper.h
> > @@ -74,36 +74,36 @@ enum {
> > Â * Corrected Machine Check
> > Â */
> > Â#define
> > CPER_NOTIFY_CMCÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ
> > ÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0x2DCE8BB1, 0xBDD7, 0x450e, 0xB9, 0xAD, 0x9C,
> > 0xF4,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xEB, 0xD4, 0xF8, 0x90)
> > +ÂÂÂÂÂÂÂGUID_INIT(0x2DCE8BB1, 0xBDD7, 0x450e, 0xB9, 0xAD, 0x9C,
> > 0xF4,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xEB, 0xD4, 0xF8, 0x90)
> > Â/* Corrected Platform Error */
> > Â#define
> > CPER_NOTIFY_CPEÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ
> > ÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0x4E292F96, 0xD843, 0x4a55, 0xA8, 0xC2, 0xD4,
> > 0x81,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xF2, 0x7E, 0xBE, 0xEE)
> > +ÂÂÂÂÂÂÂGUID_INIT(0x4E292F96, 0xD843, 0x4a55, 0xA8, 0xC2, 0xD4,
> > 0x81,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xF2, 0x7E, 0xBE, 0xEE)
> > Â/* Machine Check Exception */
> > Â#define
> > CPER_NOTIFY_MCEÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ
> > ÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0xE8F56FFE, 0x919C, 0x4cc5, 0xBA, 0x88, 0x65,
> > 0xAB,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xE1, 0x49, 0x13, 0xBB)
> > +ÂÂÂÂÂÂÂGUID_INIT(0xE8F56FFE, 0x919C, 0x4cc5, 0xBA, 0x88, 0x65,
> > 0xAB,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xE1, 0x49, 0x13, 0xBB)
> > Â/* PCI Express Error */
> > Â#define
> > CPER_NOTIFY_PCIEÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0xCF93C01F, 0x1A16, 0x4dfc, 0xB8, 0xBC, 0x9C,
> > 0x4D,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xAF, 0x67, 0xC1, 0x04)
> > +ÂÂÂÂÂÂÂGUID_INIT(0xCF93C01F, 0x1A16, 0x4dfc, 0xB8, 0xBC, 0x9C,
> > 0x4D,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xAF, 0x67, 0xC1, 0x04)
> > Â/* INIT Record (for IPF) */
> > Â#define
> > CPER_NOTIFY_INITÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0xCC5263E8, 0x9308, 0x454a, 0x89, 0xD0, 0x34,
> > 0x0B,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xD3, 0x9B, 0xC9, 0x8E)
> > +ÂÂÂÂÂÂÂGUID_INIT(0xCC5263E8, 0x9308, 0x454a, 0x89, 0xD0, 0x34,
> > 0x0B,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xD3, 0x9B, 0xC9, 0x8E)
> > Â/* Non-Maskable Interrupt */
> > Â#define
> > CPER_NOTIFY_NMIÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ
> > ÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0x5BAD89FF, 0xB7E6, 0x42c9, 0x81, 0x4A, 0xCF,
> > 0x24,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0x85, 0xD6, 0xE9, 0x8A)
> > +ÂÂÂÂÂÂÂGUID_INIT(0x5BAD89FF, 0xB7E6, 0x42c9, 0x81, 0x4A, 0xCF,
> > 0x24,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0x85, 0xD6, 0xE9, 0x8A)
> > Â/* BOOT Error Record */
> > Â#define
> > CPER_NOTIFY_BOOTÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0x3D61A466, 0xAB40, 0x409a, 0xA6, 0x98, 0xF3,
> > 0x62,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xD4, 0x64, 0xB3, 0x8F)
> > +ÂÂÂÂÂÂÂGUID_INIT(0x3D61A466, 0xAB40, 0x409a, 0xA6, 0x98, 0xF3,
> > 0x62,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xD4, 0x64, 0xB3, 0x8F)
> > Â/* DMA Remapping Error */
> > Â#define
> > CPER_NOTIFY_DMARÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0x667DD791, 0xC6B3, 0x4c27, 0x8A, 0x6B, 0x0F,
> > 0x8E,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0x72, 0x2D, 0xEB, 0x41)
> > +ÂÂÂÂÂÂÂGUID_INIT(0x667DD791, 0xC6B3, 0x4c27, 0x8A, 0x6B, 0x0F,
> > 0x8E,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0x72, 0x2D, 0xEB, 0x41)
> > 
> > Â/*
> > Â * Flags bits definitions for flags in struct cper_record_header
> > @@ -170,50 +170,50 @@ enum {
> > Â * Processor Generic
> > Â */
> > Â#define
> > CPER_SEC_PROC_GENERICÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0x9876CCAD, 0x47B4, 0x4bdb, 0xB6, 0x5E, 0x16,
> > 0xF1,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0x93, 0xC4, 0xF3, 0xDB)
> > +ÂÂÂÂÂÂÂGUID_INIT(0x9876CCAD, 0x47B4, 0x4bdb, 0xB6, 0x5E, 0x16,
> > 0xF1,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0x93, 0xC4, 0xF3, 0xDB)
> > Â/* Processor Specific: X86/X86_64 */
> > Â#define
> > CPER_SEC_PROC_IAÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0xDC3EA0B0, 0xA144, 0x4797, 0xB9, 0x5B, 0x53,
> > 0xFA,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0x24, 0x2B, 0x6E, 0x1D)
> > +ÂÂÂÂÂÂÂGUID_INIT(0xDC3EA0B0, 0xA144, 0x4797, 0xB9, 0x5B, 0x53,
> > 0xFA,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0x24, 0x2B, 0x6E, 0x1D)
> > Â/* Processor Specific: IA64 */
> > Â#define
> > CPER_SEC_PROC_IPFÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0xE429FAF1, 0x3CB7, 0x11D4, 0x0B, 0xCA, 0x07,
> > 0x00,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0x80, 0xC7, 0x3C, 0x88, 0x81)
> > +ÂÂÂÂÂÂÂGUID_INIT(0xE429FAF1, 0x3CB7, 0x11D4, 0x0B, 0xCA, 0x07,
> > 0x00,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0x80, 0xC7, 0x3C, 0x88, 0x81)
> > Â/* Processor Specific: ARM */
> > Â#define
> > CPER_SEC_PROC_ARMÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0xE19E3D16, 0xBC11, 0x11E4, 0x9C, 0xAA, 0xC2,
> > 0x05,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0x1D, 0x5D, 0x46, 0xB0)
> > +ÂÂÂÂÂÂÂGUID_INIT(0xE19E3D16, 0xBC11, 0x11E4, 0x9C, 0xAA, 0xC2,
> > 0x05,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0x1D, 0x5D, 0x46, 0xB0)
> > Â/* Platform Memory */
> > Â#define
> > CPER_SEC_PLATFORM_MEMÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0xA5BC1114, 0x6F64, 0x4EDE, 0xB8, 0x63, 0x3E,
> > 0x83,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xED, 0x7C, 0x83, 0xB1)
> > +ÂÂÂÂÂÂÂGUID_INIT(0xA5BC1114, 0x6F64, 0x4EDE, 0xB8, 0x63, 0x3E,
> > 0x83,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xED, 0x7C, 0x83, 0xB1)
> > Â#define
> > CPER_SEC_PCIEÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0xD995E954, 0xBBC1, 0x430F, 0xAD, 0x91, 0xB4,
> > 0x4D,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xCB, 0x3C, 0x6F, 0x35)
> > +ÂÂÂÂÂÂÂGUID_INIT(0xD995E954, 0xBBC1, 0x430F, 0xAD, 0x91, 0xB4,
> > 0x4D,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xCB, 0x3C, 0x6F, 0x35)
> > Â/* Firmware Error Record Reference */
> > Â#define
> > CPER_SEC_FW_ERR_REC_REFÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ
> > ÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0x81212A96, 0x09ED, 0x4996, 0x94, 0x71, 0x8D,
> > 0x72,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0x9C, 0x8E, 0x69, 0xED)
> > +ÂÂÂÂÂÂÂGUID_INIT(0x81212A96, 0x09ED, 0x4996, 0x94, 0x71, 0x8D,
> > 0x72,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0x9C, 0x8E, 0x69, 0xED)
> > Â/* PCI/PCI-X Bus */
> > Â#define
> > CPER_SEC_PCI_X_BUSÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0xC5753963, 0x3B84, 0x4095, 0xBF, 0x78, 0xED,
> > 0xDA,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xD3, 0xF9, 0xC9, 0xDD)
> > +ÂÂÂÂÂÂÂGUID_INIT(0xC5753963, 0x3B84, 0x4095, 0xBF, 0x78, 0xED,
> > 0xDA,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xD3, 0xF9, 0xC9, 0xDD)
> > Â/* PCI Component/Device */
> > Â#define
> > CPER_SEC_PCI_DEVÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0xEB5E4685, 0xCA66, 0x4769, 0xB6, 0xA2, 0x26,
> > 0x06,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0x8B, 0x00, 0x13, 0x26)
> > +ÂÂÂÂÂÂÂGUID_INIT(0xEB5E4685, 0xCA66, 0x4769, 0xB6, 0xA2, 0x26,
> > 0x06,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0x8B, 0x00, 0x13, 0x26)
> > Â#define
> > CPER_SEC_DMAR_GENERICÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0x5B51FEF7, 0xC79D, 0x4434, 0x8F, 0x1B, 0xAA,
> > 0x62,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xDE, 0x3E, 0x2C, 0x64)
> > +ÂÂÂÂÂÂÂGUID_INIT(0x5B51FEF7, 0xC79D, 0x4434, 0x8F, 0x1B, 0xAA,
> > 0x62,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xDE, 0x3E, 0x2C, 0x64)
> > Â/* Intel VT for Directed I/O specific DMAr */
> > Â#define
> > CPER_SEC_DMAR_VTÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0x71761D37, 0x32B2, 0x45cd, 0xA7, 0xD0, 0xB0,
> > 0xFE,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xDD, 0x93, 0xE8, 0xCF)
> > +ÂÂÂÂÂÂÂGUID_INIT(0x71761D37, 0x32B2, 0x45cd, 0xA7, 0xD0, 0xB0,
> > 0xFE,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xDD, 0x93, 0xE8, 0xCF)
> > Â/* IOMMU specific DMAr */
> > Â#define
> > CPER_SEC_DMAR_IOMMUÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂUUID_LE(0x036F84E1, 0x7F37, 0x428c, 0xA7, 0x9E, 0x57,
> > 0x5F,ÂÂÂÂÂ\
> > -ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xDF, 0xAA, 0x84, 0xEC)
> > +ÂÂÂÂÂÂÂGUID_INIT(0x036F84E1, 0x7F37, 0x428c, 0xA7, 0x9E, 0x57,
> > 0x5F,ÂÂÂ\
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0xDF, 0xAA, 0x84, 0xEC)
> > 
> > Â#define CPER_PROC_VALID_TYPEÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0x0001
> > Â#define CPER_PROC_VALID_ISAÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ0x0002
> > @@ -290,10 +290,10 @@ struct cper_record_header {
> > ÂÂÂÂÂÂÂÂ__u32ÂÂÂvalidation_bits;
> > ÂÂÂÂÂÂÂÂ__u32ÂÂÂrecord_length;
> > ÂÂÂÂÂÂÂÂ__u64ÂÂÂtimestamp;
> > -ÂÂÂÂÂÂÂuuid_le platform_id;
> > -ÂÂÂÂÂÂÂuuid_le partition_id;
> > -ÂÂÂÂÂÂÂuuid_le creator_id;
> > -ÂÂÂÂÂÂÂuuid_le notification_type;
> > +ÂÂÂÂÂÂÂguid_tÂÂplatform_id;
> > +ÂÂÂÂÂÂÂguid_tÂÂpartition_id;
> > +ÂÂÂÂÂÂÂguid_tÂÂcreator_id;
> > +ÂÂÂÂÂÂÂguid_tÂÂnotification_type;
> > ÂÂÂÂÂÂÂÂ__u64ÂÂÂrecord_id;
> > ÂÂÂÂÂÂÂÂ__u32ÂÂÂflags;
> > ÂÂÂÂÂÂÂÂ__u64ÂÂÂpersistence_information;
> > @@ -309,8 +309,8 @@ struct cper_section_descriptor {
> > ÂÂÂÂÂÂÂÂ__u8ÂÂÂÂvalidation_bits;
> > ÂÂÂÂÂÂÂÂ__u8ÂÂÂÂreserved;ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ/* must be zero */
> > ÂÂÂÂÂÂÂÂ__u32ÂÂÂflags;
> > -ÂÂÂÂÂÂÂuuid_le section_type;
> > -ÂÂÂÂÂÂÂuuid_le fru_id;
> > +ÂÂÂÂÂÂÂguid_tÂÂsection_type;
> > +ÂÂÂÂÂÂÂguid_tÂÂfru_id;
> > ÂÂÂÂÂÂÂÂ__u32ÂÂÂsection_severity;
> > ÂÂÂÂÂÂÂÂ__u8ÂÂÂÂfru_text[20];
> > Â};
> > @@ -343,7 +343,7 @@ struct cper_sec_proc_ia {
> > 
> > Â/* IA32/X64 Processor Error Information Structure */
> > Âstruct cper_ia_err_info {
> > -ÂÂÂÂÂÂÂuuid_le err_type;
> > +ÂÂÂÂÂÂÂguid_tÂÂerr_type;
> > ÂÂÂÂÂÂÂÂ__u64ÂÂÂvalidation_bits;
> > ÂÂÂÂÂÂÂÂ__u64ÂÂÂcheck_info;
> > ÂÂÂÂÂÂÂÂ__u64ÂÂÂtarget_id;
> > diff --git a/include/linux/efi.h b/include/linux/efi.h
> > index 8269bcb8ccf7..7a322aed979f 100644
> > --- a/include/linux/efi.h
> > +++ b/include/linux/efi.h
> > @@ -47,10 +47,10 @@ typedef u16 efi_char16_t;ÂÂÂÂÂÂÂÂÂÂÂ/* UNICODE
> > character */
> > Âtypedef u64 efi_physical_addr_t;
> > Âtypedef void *efi_handle_t;
> > 
> > -typedef uuid_le efi_guid_t;
> > +typedef guid_t efi_guid_t;
> > 
> > Â#define EFI_GUID(a,b,c,d0,d1,d2,d3,d4,d5,d6,d7) \
> > -ÂÂÂÂÂÂÂUUID_LE(a, b, c, d0, d1, d2, d3, d4, d5, d6, d7)
> > +ÂÂÂÂÂÂÂGUID_INIT(a, b, c, d0, d1, d2, d3, d4, d5, d6, d7)
> > 
> > Â/*
> > Â * Generic EFI table header
> > --
> > 2.11.0
> > 

-- 
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Intel Finland Oy