Re: [parisc-linux] Untested port of parisc_device to generic device

J.E.J. Bottomley (James.Bottomley@HansenPartnership.com)
Wed, 13 Nov 2002 12:23:48 -0500


> Absolutely. Boxes with multiple IOMMUs (at least ia64, sparc64,
> parisc) need to look up the correct IOMMU with which to map the
> allocated buffer. Typically this is in the pci_dev sysdata.

Actually, I think all of the DMA mapping api needs to become bus independent
and take a struct device * instead of a pci_dev. How this lookup/mapping is
done could be abstracted per architecture inside the DMA api internals.

We should also allow devices to do all the setup through bus generic
functions, but leave open the possibility that the driver may (once it knows
the bus type) obtain the pci_dev (or whatever) from the struct device if it
really, really has to muck with bus specific registers.

As far as the SCSI mid layer goes, all we really need from struct device is
the dma_mask for setting up the I/O bounce buffers.

The simplest way to do all of this is probably to add a pointer to the
dma_mask in struct device and make it point to the same thing in pci_dev. If
we find we need more than this per device, it could become a pointer to a
generic dma information structure later on.

Drivers need to advertise DMA conformance (at the moment, requires consistent
allocation, or fully writeback/invalidate compliant)

We should also adopt Adam's pointer approach to the sync/invalidate points so
we can treat a dma_alloc_consistent failure as a real failure and not clutter
the code with writeback/invalidate fallbacks.

The above changes would allow me to yank all of the pci_dev specific code out
of the scsi mid layer, and also introduce a mca_dev type, convert the 53c700
driver to using the generic dma API and *finally* get us to the point where I
don't have to use bounce buffers for highmem access on the MCA bus.

Since the 53c700 is also used by parisc (including some machines with
IOMMUs---which, unfortunately, I don't have access to), it probably makes an
ideal conversion test case.

This can probably all be wrappered so the current SCSI pci drivers don't
notice anything wrong.

James

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/