Re: [Qemu-devel] [PATCH] fix dma.c MemoryRegion convertion

2012-12-16 Thread Marcelo Tosatti
On Sat, Dec 15, 2012 at 01:45:38PM +, Julien Grall wrote: > On Fri, Dec 14, 2012 at 11:39 PM, Marcelo Tosatti wrote: > > > > > > The high byte of the ioport address is necessary to compute > > the register address, see "82371AB PCI ISA IDE Xcelerator (PIIX4)" > > document, eg: > > > > 4.2.1.1.

Re: [Qemu-devel] [PATCH] fix dma.c MemoryRegion convertion

2012-12-15 Thread Andreas Färber
Am 15.12.2012 14:45, schrieb Julien Grall: > On Fri, Dec 14, 2012 at 11:39 PM, Marcelo Tosatti wrote: >> >> >> The high byte of the ioport address is necessary to compute >> the register address, see "82371AB PCI ISA IDE Xcelerator (PIIX4)" >> document, eg: >> >> 4.2.1.1. DCOM—DMA Command Register

Re: [Qemu-devel] [PATCH] fix dma.c MemoryRegion convertion

2012-12-15 Thread Julien Grall
On Fri, Dec 14, 2012 at 11:39 PM, Marcelo Tosatti wrote: > > > The high byte of the ioport address is necessary to compute > the register address, see "82371AB PCI ISA IDE Xcelerator (PIIX4)" > document, eg: > > 4.2.1.1. DCOM—DMA Command Register (IO) > I/O Address: Channels 0–3—08h; Channels 4–7—

Re: [Qemu-devel] [PATCH] fix dma.c MemoryRegion convertion

2012-12-15 Thread Blue Swirl
On Fri, Dec 14, 2012 at 11:39 PM, Marcelo Tosatti wrote: > > The high byte of the ioport address is necessary to compute > the register address, see "82371AB PCI ISA IDE Xcelerator (PIIX4)" > document, eg: > > 4.2.1.1. DCOM—DMA Command Register (IO) > I/O Address: Channels 0–3—08h; Channels 4–7—0D

[Qemu-devel] [PATCH] fix dma.c MemoryRegion convertion

2012-12-14 Thread Marcelo Tosatti
The high byte of the ioport address is necessary to compute the register address, see "82371AB PCI ISA IDE Xcelerator (PIIX4)" document, eg: 4.2.1.1. DCOM—DMA Command Register (IO) I/O Address: Channels 0–3—08h; Channels 4–7—0D0h Also the size of the region is wrong. Fixes WinXP-32 installation