[PATCH] iommu fixes, round 2
John Rose
johnrose at austin.ibm.com
Thu Oct 28 09:25:47 EST 2004
On Wed, 2004-10-27 at 18:08, Paul Mackerras wrote:
> John Rose writes:
>
> > Thirdly, iommu_devnode_init() also has an iSeries implementation, so
> > it's not pSeries-specific. No need to rename it, as suggested in the
> > comment.
>
> I would rather we didn't have two functions with the same name, as we
> do for iommu_devnode_init (with iSeries and pSeries implementations),
> because that is one more obstacle to eventually making a single kernel
> binary that can run on iSeries and on other machines. That goal is
> still some distance off but we shouldn't make it harder to reach if
> possible. That's the motivation for having a function pointer in
> ppc_md for it.
Good point. To contradict my earlier statement, let's rename them :)
None of the other functions in [i,p]Series_iommu.c share names. The two
implementations are called from i and p-specific locations anyway, so
renaming won't be a problem.
Will post a patch tmw.
Thanks-
John
More information about the Linuxppc64-dev
mailing list