On Monday 30 January 2012, Michael S. Tsirkin wrote: > Kevin Cernekee reported that recent cleanup > that replaced pci_iomap with a generic function > failed to take into account the differences > in io port handling on mips and sh architectures. > > Rather than revert the changes reintroducing the > code duplication, this patchset fixes this > by adding ability for architectures to override > ioport mapping for pci devices. > > I put this in my tree that feeds into linux-next > and intend to ask Linus to pull this fix if this > doesn't cause any issues and there are no objections. > > The patches were tested on x86 and compiled on mips and sh. > Would appreciate reviews/acks/testing reports. Looks good to me, except for the one detail I've commented on in the third patch. Acked-by: Arnd Bergmann <arnd@xxxxxxxx> I do wonder if the sh and mips implementations are robust enough however (independent of your work): It seems that an ioport number is treated differently in pci_iomap than it is using ioport_map and inb/outb, the assumption being that the port number is a local index per PCI domain. This would mean that any port access other than pci_iomap would only work on the primary PCI domain. There are two IMHO better solutions that I've seen on other architectures: 1. create a larger (e.g. 1MB) io port mapping range in virtual memory that is split into 64kb per domain, and use the domain number to find the per domain range when setting the resources. Port numbers will be larger than 65535 this way, but PCI will ignore the upper address bits for any access so it works fine. 2. split the 64kb io port range into subsections per domain (on page granularity, e.g. 2 domains with 32kb or at most 16 domains with 4kb) and map them virtually contiguous, then reassign all io port resources so that only the correct region for each domain is used. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
