Re: [Qemu-devel] [PATCH 1/2] acpi: Allow ACPI default OEM ID and OEM table ID fields to be set.

[Michael Tokarev <mjt@tls.msk.ru>]

03.09.2015 18:09, Laszlo Ersek wrote:
> On 09/02/15 23:32, Michael S. Tsirkin wrote:
>> On Wed, Sep 02, 2015 at 08:03:37PM +0100, Richard W.M. Jones wrote:
>>> When using qemu's internal ACPI table generation, qemu sets the fields
>>> OEM ID and OEM table ID to arbitrary default values.  OEM ID is set to
>>> "BOCHS " and OEM table ID is set to "BXPCxxxx" where "xxxx" is
>>> replaced by the ACPI table name (eg. "BXPCRSDT" for the RSDT table).
>>>
>>> This patch allows you to override these default values.
>>>
>>> Use one of these alternatives:
>>>
>>>   qemu -acpidefault oem_id=ABCD,oem_table_id=EFGH
>>>   qemu -acpidefault oem_id=ABCD,oem_table_id=EFGHIJKL
>>>
>>> In the first case, the last four types of the OEM table name field are
>>> set to the ACPI table name.
>>>
>>> This does not affect the -acpitable option (for user-defined ACPI
>>> tables), which has its own method for setting these fields.
>>>
>>> Signed-off-by: Richard W.M. Jones <rjones@redhat.com>
>>
>> I don't think the 1st option makes sense - why force
>> 4 bytes of the ID?
>>
>> And the issue with the 2nd one is that it won't work if
>> there are two tables with same ID - which might be the
>> case for SSID and UEFI tables at least.
>>
>> Maybe specify this per signature?
> 
> I agree with these remarks, but I'd propose a different idea for solving
> the OEM Table ID problem.
> 
> As can be seen from the previous discussion that Rich linked in the
> blurb, and from Michael Tokarev's patch, and from Chris Evich's earlier
> testing of Win8.1 (linked in the previous discussion), the *only* guest
> OS where this matters is Windows 7. And Windows 7 doesn't care about the
> OEM Table Id, only the OEM Id.

Actually win7 do care about oem table id.  And my patch shows it.  This
is the minimal portion of RSDT (and also FACP in UEFI mode) which should
match SLIC -- oem_id (6 bytes) and oem_table_id (8 bytes). My patch copies
10 bytes starting with oem_id.  Without that, win7 offline activation
doesn't work.

FWIW :)

Thanks,

/mjt