Re: [Qemu-devel] [PATCH 4/9] acpi: build_append_nameseg(): add padding if necessary

2014-12-09 Thread Igor Mammedov
On Tue, 9 Dec 2014 12:38:12 +0200 "Michael S. Tsirkin" wrote: > On Tue, Dec 09, 2014 at 11:32:45AM +0100, Igor Mammedov wrote: > > On Mon, 8 Dec 2014 23:15:32 +0200 > > "Michael S. Tsirkin" wrote: > > > > > On Mon, Dec 08, 2014 at 04:08:03PM +, Igor Mammedov wrote: > > > > According to ACPI

Re: [Qemu-devel] [PATCH 4/9] acpi: build_append_nameseg(): add padding if necessary

2014-12-09 Thread Michael S. Tsirkin
On Tue, Dec 09, 2014 at 11:32:45AM +0100, Igor Mammedov wrote: > On Mon, 8 Dec 2014 23:15:32 +0200 > "Michael S. Tsirkin" wrote: > > > On Mon, Dec 08, 2014 at 04:08:03PM +, Igor Mammedov wrote: > > > According to ACPI spec NameSeg shorter than 4 characters > > > must be padded up to 4 charact

Re: [Qemu-devel] [PATCH 4/9] acpi: build_append_nameseg(): add padding if necessary

2014-12-09 Thread Igor Mammedov
On Mon, 8 Dec 2014 23:15:32 +0200 "Michael S. Tsirkin" wrote: > On Mon, Dec 08, 2014 at 04:08:03PM +, 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" > > > >

Re: [Qemu-devel] [PATCH 4/9] acpi: build_append_nameseg(): add padding if necessary

2014-12-08 Thread Michael S. Tsirkin
On Mon, Dec 08, 2014 at 04:08:03PM +, 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 abo

[Qemu-devel] [PATCH 4/9] acpi: build_append_nameseg(): add padding if necessary

2014-12-08 Thread Igor Mammedov
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 --- hw/i386/acpi-build.c | 18 ++