]> git.proxmox.com Git - mirror_qemu.git/commit
ioport: reserve the whole range of an I/O port in the AddressSpace
authorPaolo Bonzini <pbonzini@redhat.com>
Mon, 30 Mar 2015 10:35:00 +0000 (12:35 +0200)
committerPaolo Bonzini <pbonzini@redhat.com>
Mon, 27 Apr 2015 16:24:19 +0000 (18:24 +0200)
commit4080a13c11398d684668d286da27b6f8ee668e44
treeb91c9f6508185fd5f37fca36bd7ac86c2e9d2b39
parent147ed379838176d4780688157891c06f49403b19
ioport: reserve the whole range of an I/O port in the AddressSpace

When an I/O port is more than 1 byte long, ioport.c is currently
creating "short" regions, for example 0x1ce-0x1ce for the 16-bit
Bochs index port.  When I/O ports are memory mapped, and thus
accessed via a subpage_ops memory region, subpage_accepts gets
confused because it finds a hole at 0x1cf and rejects the access.

In order to fix this, modify registration of the region to cover
the whole size of the I/O port.  Attempts to access an invalid
port will be blocked by find_portio returning NULL.

This only affects the VBE DISPI regions.  For all other cases,
the MemoryRegionPortio entries for 2- or 4-byte accesses overlap
an entry for 1-byte accesses, thus the size of the memory region
is not affected.

Reported-by: Zoltan Balaton <balaton@eik.bme.hu>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
ioport.c