Re: [Qemu-devel] [PATCH 1/2] pc: map PCI address space as catchall region for not mapped addresses

[Laszlo Ersek <lersek@redhat.com>]

On 11/12/13 14:58, Igor Mammedov wrote:
> From: "Michael S. Tsirkin" <mst@redhat.com>
> 
> With a help of negative memory region priority PCI address space
> is mapped underneath RAM regions effectively catching every access
> to addresses not mapped by any other region.
> It simplifies PCI address space mapping into system address space.
> 
> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
> Signed-off-by: Igor Mammedov <imammedo@redhat.com>
> ---
>  hw/i386/pc.c              | 20 ++++++--------------
>  hw/i386/pc_piix.c         |  2 --
>  hw/pci-host/piix.c        | 26 ++++----------------------
>  hw/pci-host/q35.c         | 27 +++++----------------------
>  include/hw/i386/pc.h      | 14 ++------------
>  include/hw/pci-host/q35.h |  2 --
>  6 files changed, 17 insertions(+), 74 deletions(-)

Tested with OVMF running from flash (and storing nvvars there).

Also tested with my pending series "i440fx-test: check firmware
visibility" (reviews welcome :)).

GTest: result: OK
GTest: run: /i440fx/firmware/bios
(MSG: qemu cmdline: -S -display none -bios /tmp/fw_blob_UAGE6W)
GTest: result: OK
GTest: run: /i440fx/firmware/pflash
(MSG: qemu cmdline: -S -display none -pflash /tmp/fw_blob_DHXN6W)
GTest: result: OK

Tested-by: Laszlo Ersek <lersek@redhat.com>

Thanks!
Laszlo