diff options
author | Paolo Bonzini <pbonzini@redhat.com> | 2015-03-30 12:35:00 +0200 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2015-04-27 18:24:19 +0200 |
commit | 4080a13c11398d684668d286da27b6f8ee668e44 (patch) | |
tree | b91c9f6508185fd5f37fca36bd7ac86c2e9d2b39 /exec.c | |
parent | 147ed379838176d4780688157891c06f49403b19 (diff) | |
download | qemu-4080a13c11398d684668d286da27b6f8ee668e44.zip |
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>
Diffstat (limited to 'exec.c')
0 files changed, 0 insertions, 0 deletions