[PATCH 3/8] Remove unused pte_offset variable

Michael Ellerman michael at ellerman.id.au
Wed Feb 27 19:11:00 EST 2008


On Wed, 2008-02-27 at 19:04 +1100, Benjamin Herrenschmidt wrote:
> On Wed, 2008-02-27 at 18:28 +1100, Michael Ellerman wrote:
> > The cell IOMMU code no longer needs to save the pte_offset variable
> > separately, it is incorporated into tbl->it_offset.
> > 
> > Signed-off-by: Michael Ellerman <michael at ellerman.id.au>
> > ---
> 
> Do this work on spider ?
> 
> There is a subtle difference with spider is that the 0x80000000 you give
> to devices disappears when reaching the iommu (so isn't to be catered
> in the iommu offset).
> 
> It's a bit like the offset you have to add to the direct mapping on
> axon, in fact (the 0x8000060000000000 or so ...)

I'm pretty sure I tested it and it worked.

But I didn't actually change anything WRT the spider offset anyway, we
still add it to it_offset just like we used to, it's just we don't put
it in the window and then pull it out again.

The only thing that's changed is we now use offset (aka. base) in the
tce routines in addition to pte_offset.

cheers

-- 
Michael Ellerman
OzLabs, IBM Australia Development Lab

wwweb: http://michael.ellerman.id.au
phone: +61 2 6212 1183 (tie line 70 21183)

We do not inherit the earth from our ancestors,
we borrow it from our children. - S.M.A.R.T Person
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://lists.ozlabs.org/pipermail/linuxppc-dev/attachments/20080227/832d88e7/attachment.pgp>


More information about the Linuxppc-dev mailing list