From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f66.google.com (mail-pj1-f66.google.com [209.85.216.66]) by mx.groups.io with SMTP id smtpd.web10.13855.1598362880286344222 for ; Tue, 25 Aug 2020 06:41:20 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@linaro.org header.s=google header.b=WLhlXjQg; spf=pass (domain: linaro.org, ip: 209.85.216.66, mailfrom: tanmay.jagdale@linaro.org) Received: by mail-pj1-f66.google.com with SMTP id mt12so1265579pjb.4 for ; Tue, 25 Aug 2020 06:41:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=NnLE6fsntEwWh/sv0EpKUm8pUUgoy3EJrHcdZHXCFE8=; b=WLhlXjQgV2Qn0LPvAsW0EOzi87gWK3PfHfv+9Y934706avt3lW9HjYYKOg8SYe9jYC xmQAC9hwfsLdeg8/ha+PtOKFlC8UTafl4/M1bjGgqlQYUmj68/LxfBkFwsr1ujyMlYU+ bjWT6zZSNCXsH8gdbGjuGBSQNTo1WC8pNn5wvRCi3j7UmwtJewvUfoTEQSenPcS7PYqQ wkqbeLtAnXEmtrrAC5lRUndw8JTkisEWMTDDsTF5Bjow3vKsvxWVMZKhiZqUY2RlXpy4 JVPBGuYEY9+GRQPzxNvYaCuerRrnvMbBHkHkqI0NM9wJiww4IdWQ/p/GxsAq1KUiX1yQ rwQg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=NnLE6fsntEwWh/sv0EpKUm8pUUgoy3EJrHcdZHXCFE8=; b=THrCf0MfBhlgmXn8srGIIO3BqOnd2VrYQGd9kZ/ZU9twJZ/2Of4xMKLoTzNWk87d80 FJFjo8MeMTfenTq9d3O5IIq5JesamIj391GQvqZA33DlFSZsvlaoag/KYAPKm17MwknB DNi0WzlbBgfYHn5sKAV0EHJD4mMYIe85bmxHY8B8lWSqcPVZHcwJzLQ9h+A4kt3W5sdb hwCFHxxFWlGRTFC1bXthdu3tXY8Kj4RAFTkRLNy+v0R7jT5/M+StbVxmtY+fYuyrBBRv /Or2yORs/y+8QLkphGbYh3b8BMkwSJgNHx5dbi5XjRgxePRUzIAfGSAA7tStiwv9fa07 Gcfw== X-Gm-Message-State: AOAM532e+Dmef7/jD5YYOqZ+rcmbDvMz909aDm8lU7bO43zld6WGiSjK 9DZ81bQetbKAHfpns9NnWlQx4w== X-Google-Smtp-Source: ABdhPJwnzHlj0m3AZERIFSyLetuRlWhmzI7ztXUZEM4mjBMwuQNYSde4x6+QdiEM+9W6fmfmirTQTA== X-Received: by 2002:a17:90b:509:: with SMTP id r9mr1734452pjz.228.1598362879818; Tue, 25 Aug 2020 06:41:19 -0700 (PDT) Return-Path: Received: from bean-canyon.localdomain ([106.51.140.68]) by smtp.googlemail.com with ESMTPSA id z186sm3913768pfb.199.2020.08.25.06.41.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 25 Aug 2020 06:41:19 -0700 (PDT) From: "Tanmay Jagdale" To: leif@nuviainc.com, graeme@nuviainc.com, devel@edk2.groups.io Cc: shashi.mallela@linaro.org, Tanmay Jagdale Subject: [PATCH v3 edk2-platforms 6/8] SbsaQemu: AcpiDxe: Create SSDT table at runtime Date: Tue, 25 Aug 2020 19:09:56 +0530 Message-Id: <20200825133958.17372-7-tanmay.jagdale@linaro.org> X-Mailer: git-send-email 2.28.0 In-Reply-To: <20200825133958.17372-1-tanmay.jagdale@linaro.org> References: <20200825133958.17372-1-tanmay.jagdale@linaro.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit - Add support to create SSDT table at runtime. Since SSDT table is a data table, added a few helper macros to create the AML entries. - Also added a function to calculate the length of Packages. Signed-off-by: Tanmay Jagdale --- Silicon/Qemu/SbsaQemu/Include/IndustryStandard/SbsaQemuAcpi.h | 29 ++++ Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c | 144 ++++++++++++++++++++ 2 files changed, 173 insertions(+) diff --git a/Silicon/Qemu/SbsaQemu/Include/IndustryStandard/SbsaQemuAcpi.h b/Silicon/Qemu/SbsaQemu/Include/IndustryStandard/SbsaQemuAcpi.h index 7a9a0061675f..00c7c68256fd 100644 --- a/Silicon/Qemu/SbsaQemu/Include/IndustryStandard/SbsaQemuAcpi.h +++ b/Silicon/Qemu/SbsaQemu/Include/IndustryStandard/SbsaQemuAcpi.h @@ -43,4 +43,33 @@ #define SBSAQEMU_PCI_SEG0_BUSNUM_MIN 0x00 #define SBSAQEMU_PCI_SEG0_BUSNUM_MAX 0xFF +#define SBSAQEMU_ACPI_SCOPE_OP_MAX_LENGTH 5 + +#define SBSAQEMU_ACPI_SCOPE_NAME { '_', 'S', 'B', '_' } + +#define SBSAQEMU_ACPI_CPU_DEV_LEN 0x1C +#define SBSAQEMU_ACPI_CPU_DEV_NAME { 'C', '0', '0', '0' } + +// Macro to convert Integer to Character +#define SBSAQEMU_ACPI_ITOA(Byte) (0x30 + (Byte > 9 ? (Byte + 1) : Byte)) + +#define SBSAQEMU_ACPI_CPU_HID { \ + AML_NAME_OP, AML_NAME_CHAR__, 'H', 'I', 'D', \ + AML_STRING_PREFIX, 'A', 'C', 'P', 'I', '0', '0', '0', '7', \ + AML_ZERO_OP \ + } + +#define SBSAQEMU_ACPI_CPU_UID { \ + AML_NAME_OP, AML_NAME_CHAR__, 'U', 'I', 'D', AML_BYTE_PREFIX, \ + AML_ZERO_OP, AML_ZERO_OP \ + } + +typedef struct { + UINT8 device_header[2]; + UINT8 length; + UINT8 dev_name[4]; + UINT8 hid[15]; + UINT8 uid[8]; +} SBSAQEMU_ACPI_CPU_DEVICE; + #endif diff --git a/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c b/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c index 16cb4e904e6f..06e7a5310810 100644 --- a/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c +++ b/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c @@ -7,6 +7,7 @@ * **/ #include +#include #include #include #include @@ -198,6 +199,144 @@ AddMadtTable ( return Status; } +/* + * Function to calculate the PkgLength field in ACPI tables + */ +STATIC +UINT32 +SetPkgLength ( + IN UINT8 *TablePtr, + IN UINT32 Length +) +{ + UINT8 ByteCount; + UINT8 *PkgLeadByte = TablePtr; + + if (Length < 64) { + *TablePtr = Length; + return 1; + } + + // Set the LSB of Length in PkgLeadByte and advance Length + *PkgLeadByte = Length & 0xF; + Length = Length >> 4; + + while (Length) { + TablePtr++; + *TablePtr = (Length & 0xFF); + Length = (Length >> 8); + } + + // Calculate the number of bytes the Length field uses + // and set the ByteCount field in PkgLeadByte. + ByteCount = (TablePtr - PkgLeadByte) & 0xF; + *PkgLeadByte |= (ByteCount << 6); + + return ByteCount + 1; +} + +/* + * A function that adds SSDT ACPI table. + */ +EFI_STATUS +AddSsdtTable ( + IN EFI_ACPI_TABLE_PROTOCOL *AcpiTable + ) +{ + EFI_STATUS Status; + UINTN TableHandle; + UINT32 TableSize; + EFI_PHYSICAL_ADDRESS PageAddress; + UINT8 *New; + UINT32 CpuId; + UINT32 Offset; + UINT8 ScopeOpName[] = SBSAQEMU_ACPI_SCOPE_NAME; + UINT32 NumCores = PcdGet32 (PcdCoreCount); + + EFI_ACPI_DESCRIPTION_HEADER Header = + SBSAQEMU_ACPI_HEADER ( + EFI_ACPI_6_0_SECONDARY_SYSTEM_DESCRIPTION_TABLE_SIGNATURE, + EFI_ACPI_DESCRIPTION_HEADER, + EFI_ACPI_6_0_SECONDARY_SYSTEM_DESCRIPTION_TABLE_REVISION); + + SBSAQEMU_ACPI_CPU_DEVICE CpuDevice = { + { AML_EXT_OP, AML_EXT_DEVICE_OP }, /* Device () */ + SBSAQEMU_ACPI_CPU_DEV_LEN, /* Length */ + SBSAQEMU_ACPI_CPU_DEV_NAME, /* Device Name "C000" */ + SBSAQEMU_ACPI_CPU_HID, /* Name (HID, "ACPI0007") */ + SBSAQEMU_ACPI_CPU_UID, /* Name (UID, 0) */ + }; + + // Calculate the new table size based on the number of cores + TableSize = sizeof (EFI_ACPI_DESCRIPTION_HEADER) + + SBSAQEMU_ACPI_SCOPE_OP_MAX_LENGTH + sizeof (ScopeOpName) + + (sizeof (CpuDevice) * NumCores); + + Status = gBS->AllocatePages ( + AllocateAnyPages, + EfiACPIReclaimMemory, + EFI_SIZE_TO_PAGES (TableSize), + &PageAddress + ); + if (EFI_ERROR(Status)) { + DEBUG ((DEBUG_ERROR, "Failed to allocate pages for SSDT table\n")); + return EFI_OUT_OF_RESOURCES; + } + + New = (UINT8 *)(UINTN) PageAddress; + ZeroMem (New, TableSize); + + // Add the ACPI Description table header + CopyMem (New, &Header, sizeof (EFI_ACPI_DESCRIPTION_HEADER)); + ((EFI_ACPI_DESCRIPTION_HEADER*) New)->Length = TableSize; + New += sizeof (EFI_ACPI_DESCRIPTION_HEADER); + + // Insert the top level ScopeOp + *New = AML_SCOPE_OP; + New++; + Offset = SetPkgLength (New, + (TableSize - sizeof (EFI_ACPI_DESCRIPTION_HEADER) - 1)); + New += Offset; + CopyMem (New, &ScopeOpName, sizeof (ScopeOpName)); + New += sizeof (ScopeOpName); + + // Add new Device structures for the Cores + for (CpuId = 0; CpuId < NumCores; CpuId++) { + SBSAQEMU_ACPI_CPU_DEVICE *CpuDevicePtr; + UINT8 CpuIdByte1, CpuIdByte2, CpuIdByte3; + + CopyMem (New, &CpuDevice, sizeof (SBSAQEMU_ACPI_CPU_DEVICE)); + CpuDevicePtr = (SBSAQEMU_ACPI_CPU_DEVICE *) New; + + CpuIdByte1 = CpuId & 0xF; + CpuIdByte2 = (CpuId >> 4) & 0xF; + CpuIdByte3 = (CpuId >> 8) & 0xF; + + CpuDevicePtr->dev_name[1] = SBSAQEMU_ACPI_ITOA(CpuIdByte3); + CpuDevicePtr->dev_name[2] = SBSAQEMU_ACPI_ITOA(CpuIdByte2); + CpuDevicePtr->dev_name[3] = SBSAQEMU_ACPI_ITOA(CpuIdByte1); + + CpuDevicePtr->uid[6] = CpuIdByte1 | CpuIdByte2; + CpuDevicePtr->uid[7] = CpuIdByte3; + New += sizeof (SBSAQEMU_ACPI_CPU_DEVICE); + } + + // Perform Checksum + AcpiPlatformChecksum ((UINT8*) PageAddress, TableSize); + + Status = AcpiTable->InstallAcpiTable ( + AcpiTable, + (EFI_ACPI_COMMON_HEADER *)PageAddress, + TableSize, + &TableHandle + ); + if (EFI_ERROR(Status)) { + DEBUG ((DEBUG_ERROR, "Failed to install SSDT table\n")); + } + + return Status; +} + EFI_STATUS EFIAPI InitializeSbsaQemuAcpiDxe ( @@ -227,5 +366,10 @@ InitializeSbsaQemuAcpiDxe ( DEBUG ((DEBUG_ERROR, "Failed to add MADT table\n")); } + Status = AddSsdtTable (AcpiTable); + if (EFI_ERROR(Status)) { + DEBUG ((DEBUG_ERROR, "Failed to add SSDT table\n")); + } + return EFI_SUCCESS; } -- 2.28.0