[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 3/7] AMD IOMMU emulation
From: |
Eduard - Gabriel Munteanu |
Subject: |
[Qemu-devel] Re: [PATCH 3/7] AMD IOMMU emulation |
Date: |
Sun, 29 Aug 2010 00:53:33 +0300 |
User-agent: |
Mutt/1.5.20 (2009-06-14) |
On Sat, Aug 28, 2010 at 03:58:23PM +0000, Blue Swirl wrote:
> On Sat, Aug 28, 2010 at 2:54 PM, Eduard - Gabriel Munteanu
> <address@hidden> wrote:
> > This introduces emulation for the AMD IOMMU, described in "AMD I/O
> > Virtualization Technology (IOMMU) Specification".
> >
> > Signed-off-by: Eduard - Gabriel Munteanu <address@hidden>
> > ---
[snip]
> > diff --git a/hw/amd_iommu.c b/hw/amd_iommu.c
> > new file mode 100644
> > index 0000000..43e0426
> > --- /dev/null
> > +++ b/hw/amd_iommu.c
[snip]
> > +static void amd_iommu_update_mmio(AMDIOMMUState *st,
> > + ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ??target_phys_addr_t addr)
> > +{
> > + ?? ??size_t reg = addr & ~0x07;
> > + ?? ??uint64_t *base = (uint64_t *) &st->mmio_buf[reg];
>
> This is still buggy.
>
> > + ?? ??uint64_t val = le64_to_cpu(*base);
mmio_buf is always LE, so a BE host will have *base in reversed
byteorder. But look at the next line, where I did the le64_to_cpu().
That should swap the bytes on a BE host, yielding the correct byteorder.
On a LE host, *base is right the first time and le64_to_cpu() is a nop.
In any case, I only use 'val', not '*base' directly. I suppose it could
be rewritten for clarity (i.e. ditch 'base').
Do you still think it's wrong? Or is it for another reason?
Thanks,
Eduard
- [Qemu-devel] [PATCH 0/7] AMD IOMMU emulation patchset v4, Eduard - Gabriel Munteanu, 2010/08/28
- [Qemu-devel] [PATCH 2/7] pci: memory access API and IOMMU support, Eduard - Gabriel Munteanu, 2010/08/28
- [Qemu-devel] [PATCH 4/7] ide: use the PCI memory access interface, Eduard - Gabriel Munteanu, 2010/08/28
- [Qemu-devel] [PATCH 6/7] eepro100: use the PCI memory access interface, Eduard - Gabriel Munteanu, 2010/08/28
- [Qemu-devel] [PATCH 7/7] ac97: use the PCI memory access interface, Eduard - Gabriel Munteanu, 2010/08/28
- [Qemu-devel] [PATCH 5/7] rtl8139: use the PCI memory access interface, Eduard - Gabriel Munteanu, 2010/08/28
- [Qemu-devel] [PATCH 3/7] AMD IOMMU emulation, Eduard - Gabriel Munteanu, 2010/08/28
- Re: [Qemu-devel] [PATCH 3/7] AMD IOMMU emulation, Isaku Yamahata, 2010/08/29
- [Qemu-devel] [PATCH 1/7] pci: expand tabs to spaces in pci_regs.h, Eduard - Gabriel Munteanu, 2010/08/28
- [Qemu-devel] Re: [PATCH 0/7] AMD IOMMU emulation patchset v4, Blue Swirl, 2010/08/28