[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V2 4/8] acpi: build_append_nameseg(): add paddin
From: |
Claudio Fontana |
Subject: |
Re: [Qemu-devel] [PATCH V2 4/8] acpi: build_append_nameseg(): add padding if necessary |
Date: |
Fri, 19 Dec 2014 10:29:32 +0100 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; rv:24.0) Gecko/20100101 Thunderbird/24.0.1 |
On 16.12.2014 11:58, Igor Mammedov wrote:
> According to ACPI spec NameSeg shorter than 4 characters
> must be padded up to 4 characters with "_" symbol.
> ACPI 5.0: 20.2.2 "Name Objects Encoding"
>
> Do it in build_append_nameseg() so that caller shouldn't know
> or care about it.
>
> Signed-off-by: Igor Mammedov <address@hidden>
> ---
> v2:
> * simplify padding, suggested by: "Michael S. Tsirkin" <address@hidden>
> ---
> hw/i386/acpi-build.c | 15 ++++++++++-----
> 1 file changed, 10 insertions(+), 5 deletions(-)
>
> diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
> index f5ec66a..2bf9a09 100644
> --- a/hw/i386/acpi-build.c
> +++ b/hw/i386/acpi-build.c
> @@ -292,6 +292,8 @@ static inline void build_append_array(GArray *array,
> GArray *val)
> g_array_append_vals(array, val->data, val->len);
> }
>
> +#define ACPI_NAMESEG_LEN 4
> +
> static void GCC_FMT_ATTR(2, 3)
> build_append_nameseg(GArray *array, const char *format, ...)
> {
> @@ -304,8 +306,11 @@ build_append_nameseg(GArray *array, const char *format,
> ...)
> len = vsnprintf(s, sizeof s, format, args);
> va_end(args);
>
> - assert(len == 4);
> + assert(len <= ACPI_NAMESEG_LEN);
> +
> g_array_append_vals(array, s, len);
> + /* Pad up to ACPI_NAMESEG_LEN characters if necessary. */
> + g_array_append_vals(array, "____", ACPI_NAMESEG_LEN - len);
> }
>
> /* 5.4 Definition Block Encoding */
> @@ -846,7 +851,7 @@ static void build_pci_bus_end(PCIBus *bus, void
> *bus_state)
>
> if (bus->parent_dev) {
> op = 0x82; /* DeviceOp */
> - build_append_nameseg(bus_table, "S%.02X_",
> + build_append_nameseg(bus_table, "S%.02X",
> bus->parent_dev->devfn);
> build_append_byte(bus_table, 0x08); /* NameOp */
> build_append_nameseg(bus_table, "_SUN");
> @@ -966,7 +971,7 @@ static void build_pci_bus_end(PCIBus *bus, void
> *bus_state)
> build_append_int(notify, 0x1U << i);
> build_append_byte(notify, 0x00); /* NullName */
> build_append_byte(notify, 0x86); /* NotifyOp */
> - build_append_nameseg(notify, "S%.02X_", PCI_DEVFN(i, 0));
> + build_append_nameseg(notify, "S%.02X", PCI_DEVFN(i, 0));
> build_append_byte(notify, 0x69); /* Arg1Op */
>
> /* Pack it up */
> @@ -1023,7 +1028,7 @@ static void build_pci_bus_end(PCIBus *bus, void
> *bus_state)
> if (bus->parent_dev) {
> build_append_byte(parent->notify_table, '^'); /*
> ParentPrefixChar */
> build_append_byte(parent->notify_table, 0x2E); /* DualNamePrefix
> */
> - build_append_nameseg(parent->notify_table, "S%.02X_",
> + build_append_nameseg(parent->notify_table, "S%.02X",
> bus->parent_dev->devfn);
> build_append_nameseg(parent->notify_table, "PCNT");
> }
> @@ -1093,7 +1098,7 @@ build_ssdt(GArray *table_data, GArray *linker,
> GArray *sb_scope = build_alloc_array();
> uint8_t op = 0x10; /* ScopeOp */
>
> - build_append_nameseg(sb_scope, "_SB_");
> + build_append_nameseg(sb_scope, "_SB");
>
> /* build Processor object for each processor */
> for (i = 0; i < acpi_cpus; i++) {
>
Reviewed-by: Claudio Fontana <address@hidden>
--
Claudio Fontana
Server Virtualization Architect
Huawei Technologies Duesseldorf GmbH
Riesstraße 25 - 80992 München
office: +49 89 158834 4135
mobile: +49 15253060158
- [Qemu-devel] [PATCH V2 0/8] pc: acpi: various fixes and cleanups, Igor Mammedov, 2014/12/16
- [Qemu-devel] [PATCH V2 1/8] pc: acpi: fix WindowsXP BSOD when memory hotplug is enabled, Igor Mammedov, 2014/12/16
- [Qemu-devel] [PATCH V2 2/8] pc: acpi: decribe bridge device as not hotpluggable, Igor Mammedov, 2014/12/16
- [Qemu-devel] [PATCH V2 3/8] pc: acpi-build: cleanup AcpiPmInfo initialization, Igor Mammedov, 2014/12/16
- [Qemu-devel] [PATCH V2 4/8] acpi: build_append_nameseg(): add padding if necessary, Igor Mammedov, 2014/12/16
- Re: [Qemu-devel] [PATCH V2 4/8] acpi: build_append_nameseg(): add padding if necessary,
Claudio Fontana <=
- [Qemu-devel] [PATCH V2 5/8] acpi: move generic aml building helpers into dedictated file, Igor Mammedov, 2014/12/16
- [Qemu-devel] [PATCH V2 7/8] acpi: drop min-bytes in build_package(), Igor Mammedov, 2014/12/16
- [Qemu-devel] [PATCH V2 6/8] acpi: add build_append_namestring() helper, Igor Mammedov, 2014/12/16
- [Qemu-devel] [PATCH V2 8/8] pc: acpi-build: simplify PCI bus tree generation, Igor Mammedov, 2014/12/16