Hi, There's currently an issue with PCI configuration space accesses on Tegra. The PCI host controller driver's ->map_bus() implementation remaps I/O memory on-demand to avoid potentially wasting 256 MiB of virtual address space. The reason why this is done is because the mapping isn't compatible with ECAM and the extended register number is encoded in the uppermost 4 bits. This means that if we want to address the configuration space for a single bus we already need to map 256 MiB of memory, even if only 1 MiB is really used. tegra_pcie_bus_alloc() is therefore used to stitch together a 1 MiB block of virtual addresses per bus made up of 16 64 KiB chunks each so that only what's really needed is mapped. That function gets called the first time a PCI configuration access is performed on a bus. The code calls functions that may sleep, and that causes problems because the PCI configuration space accessors are called with the global pci_lock held. This works in practice but it blows up when lockdep is enabled. I remember coding up a fix for this using the ARM/PCI ->add_bus() callbacks at one point and then forgetting about it. When I wanted to revive that patch a little while ago I noticed that ->add_bus() is now gone. What I'm asking myself now is how to fix this. I suppose it'd be possible to bring back ->add_bus(), though I suspect there were good reasons to remove it (portability?). Another possible fix would be to get rid of the spinlock protecting these accesses. It seems to me like it's not really necessary in the majority of cases. For drivers that do a simple readl() or writel() on some memory-mapped I/O the lock doesn't protect anything. Then again, there are a lot of pci_ops implementations in the tree, and simply removing the global lock seems like it'd have a good chance of breaking things for somebody. So short of auditing all pci_ops implementations and pushing the lock down into drivers, does anyone have any good ideas on how to fix this? Thanks, Thierry
Attachment:
signature.asc
Description: PGP signature
