[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH 4/7] sparc: rename hw/iommu.c
From: |
Eduard - Gabriel Munteanu |
Subject: |
Re: [Qemu-devel] [RFC PATCH 4/7] sparc: rename hw/iommu.c |
Date: |
Tue, 30 Mar 2010 23:15:04 +0300 |
User-agent: |
Mutt/1.5.20 (2009-06-14) |
On Tue, Mar 30, 2010 at 11:00:10PM +0300, Blue Swirl wrote:
> On 3/30/10, Joerg Roedel <address@hidden> wrote:
> > On Tue, Mar 30, 2010 at 08:06:36PM +0300, Blue Swirl wrote:
> > > On 3/30/10, Eduard - Gabriel Munteanu <address@hidden> wrote:
> > > > hw/iommu.c concerns the SPARC IOMMU. However we intend to implement the
> > > > AMD IOMMU, which could lead to confusion unless we rename the former.
> > >
> > > I was also thinking of renaming the file some time ago. The correct
> > > name would be "sun4m_iommu.c". Sun4c (while still Sparc based) had a
> > > different architecture (IIRC CPU MMU doubled as IOMMU) and Sun4d had
> > > several IO-UNITs instead. All Sun4m machines had an IOMMU.
> > >
> > > But the qdev name of the device is still "iommu" and we can't change
> > > that. So I'm not so sure it's worth renaming. Can't AMD IOMMU reside
> > > in amd_iommu.c?
> >
> >
> > Keeping the plain name 'iommu' will likely cause confusion when more
> > iommu implementations are added. It is better to rename it so that the
> > name better describes what the file implements. So this change makes
> > sense for me.
>
> I see. I'm OK then with "sun4m_iommu.c".
Yes, I think it's enough to just change the filename, since multiple
such devices aren't likely to conflict, in any configuration.
Then "sun4m_iommu.c" it is. Will resubmit with the next patchset.
Eduard
- [Qemu-devel] [RFC PATCH 0/7] Beginning implementing the AMD IOMMU emulation, Eduard - Gabriel Munteanu, 2010/03/30
- [Qemu-devel] [RFC PATCH 2/7] acpi: split and rename acpi_table_add(), Eduard - Gabriel Munteanu, 2010/03/30
- [Qemu-devel] [RFC PATCH 1/7] acpi: qemu_realloc() might return a different pointer, Eduard - Gabriel Munteanu, 2010/03/30
- [Qemu-devel] [RFC PATCH 7/7] acpi: fix bug in acpi_checksum() caused by garbage in checksum field, Eduard - Gabriel Munteanu, 2010/03/30
- [Qemu-devel] [RFC PATCH 3/7] acpi: move table header definition into pc.h, Eduard - Gabriel Munteanu, 2010/03/30
- [Qemu-devel] [RFC PATCH 4/7] sparc: rename hw/iommu.c, Eduard - Gabriel Munteanu, 2010/03/30
- Re: [Qemu-devel] [RFC PATCH 4/7] sparc: rename hw/iommu.c, Gerd Hoffmann, 2010/03/31
[Qemu-devel] [RFC PATCH 5/7] x86-64: AMD IOMMU stub, Eduard - Gabriel Munteanu, 2010/03/30
[Qemu-devel] [RFC PATCH 6/7] acpi: cleanup acpi_checksum(), Eduard - Gabriel Munteanu, 2010/03/30
[Qemu-devel] Re: [RFC PATCH 0/7] Beginning implementing the AMD IOMMU emulation, Joerg Roedel, 2010/03/30