]> xenbits.xensource.com Git - qemu-xen-4.1-testing.git/commit
hw/piix4acpi: Make writes to ACPI_DBG_IO_ADDR actually work.
authorKonrad Rzeszutek Wilk <konrad@kernel.org>
Mon, 11 Nov 2013 18:42:56 +0000 (13:42 -0500)
committerIan Jackson <Ian.Jackson@eu.citrix.com>
Mon, 25 Nov 2013 13:53:49 +0000 (13:53 +0000)
commit2d4fa397ba0fbae0c62282de619e1f088a320b33
tree73df73fa21718f42524b67e3c6fa2ab5ed20ae63
parent4904cdf6166687198fc29cb30ca0fa09f316486f
hw/piix4acpi: Make writes to ACPI_DBG_IO_ADDR actually work.

The ACPI AML code has little snippets where it uses two
memory locations to stash debug information when doing PCI
hotplug, such as:

Device (S20)
{
    Name (_ADR, 0x00040000)
    Name (_SUN, 0x04)
    Method (_EJ0, 1, NotSerialized)
    {
Store (0x20, \_GPE.DPT1)
Store (0x88, \_GPE.DPT2)
Store (One, \_GPE.PH20)
    }

    Method (_STA, 0, NotSerialized)
    {
Store (0x20, \_GPE.DPT1)
Store (0x89, \_GPE.DPT2)
    }
}

and DPT1 (and DPT2) is defined as:

OperationRegion ( DG1, SystemIO, 0xb044, 0x04 )
Field ( DG1, ByteAcc, NoLock, Preserve ) {
    DPT1, 8, DPT2, 8
}

But unfortunately when we do the writes they are done
as byte writes, not as 4-byte writes (long). Hence
any debug values are never show in QEMU.

This fixes it so that we can see them in the log file.

Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
(cherry picked from commit 832f9bde25542961ea4f768be6e00e4801484e3a)
(cherry picked from commit 147f83f9b7d87a698c200c4f3eb2d36a0e4fe54b)
(cherry picked from commit 7c51fb2971cd455eb728e716f16a488c01f8e759)
hw/piix4acpi.c