| Message ID | 20120709182018.18165.98339.stgit@bhelgaas.mtv.corp.google.com (mailing list archive) |
|---|---|
| State | New, archived |
| Delegated to: | Bjorn Helgaas |
| Headers | show |
On Mon, 09 Jul 2012 12:20:18 -0600 Bjorn Helgaas <bhelgaas@google.com> wrote: > After 253d2e5498, we disable MEM and IO decoding for most devices > while we size 32-bit BARs. However, we restore the original COMMAND > register before we size the upper 32 bits of 64-bit BARs, so we can > still cause a conflict. > > This patch waits to restore the original COMMAND register until we're > completely finished sizing the BAR. agreed. Thanks, Jacob -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi, On Mon, Jul 9, 2012 at 11:20 AM, Bjorn Helgaas <bhelgaas@google.com> wrote: > After 253d2e5498, we disable MEM and IO decoding for most devices while we > size 32-bit BARs. However, we restore the original COMMAND register before > we size the upper 32 bits of 64-bit BARs, so we can still cause a conflict. > > This patch waits to restore the original COMMAND register until we're > completely finished sizing the BAR. > > Reference: https://lkml.org/lkml/2007/8/25/154 > Signed-off-by: Bjorn Helgaas <bhelgaas@google.com> This patch causes boot lockup on PA Semi hardware, since it disables the bar on the UART that is used for console, and it has printks between the old and the new re-enable location. If I boot with 'debug' level for printk, I hit this. If I boot with just regular console args, I don't. I'm guessing any other platform that uses MMIO-based UART on PCI for console will have similar issues. I can verify on Chrome OS x86 hardware tomorrow if legacy powerpc isn't important enough to care about. :-) I have no proposal for a fix for this. Can you please consider reverting for 3.6 unless someone has a better idea? -Olof -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/pci/probe.c b/drivers/pci/probe.c index 658ac97..66b3a6f 100644 --- a/drivers/pci/probe.c +++ b/drivers/pci/probe.c @@ -152,9 +152,6 @@ int __pci_read_base(struct pci_dev *dev, enum pci_bar_type type, pci_read_config_dword(dev, pos, &sz); pci_write_config_dword(dev, pos, l); - if (!dev->mmio_always_on) - pci_write_config_word(dev, PCI_COMMAND, orig_cmd); - /* * All bits set in sz means the device isn't working properly. * If the BAR isn't implemented, all bits must be 0. If it's a @@ -239,6 +236,9 @@ int __pci_read_base(struct pci_dev *dev, enum pci_bar_type type, } out: + if (!dev->mmio_always_on) + pci_write_config_word(dev, PCI_COMMAND, orig_cmd); + return (res->flags & IORESOURCE_MEM_64) ? 1 : 0; fail: res->flags = 0;
After 253d2e5498, we disable MEM and IO decoding for most devices while we size 32-bit BARs. However, we restore the original COMMAND register before we size the upper 32 bits of 64-bit BARs, so we can still cause a conflict. This patch waits to restore the original COMMAND register until we're completely finished sizing the BAR. Reference: https://lkml.org/lkml/2007/8/25/154 Signed-off-by: Bjorn Helgaas <bhelgaas@google.com> --- drivers/pci/probe.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html