@@ -728,14 +728,10 @@ int pci_claim_bridge_resource(struct pci_dev
*bridge, int i)
break;
case 2:
pci_setup_bridge_mmio_pref(bridge);
- break;
default:
- return -EINVAL;
+ break;
}
- if (pci_claim_resource(bridge, i) == 0)
- return 0; /* claimed a smaller window */
-
return -EINVAL;
}
And I suggested to clear UNSET instead, and posted the patch
but I had the From : Lorenzo to keep him have to authorship
as he noticed the problem.
From: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Subject: [PATCH] PCI: Fix clipped bridge resource reserve
Commit 8505e729a2f6eb ("PCI: Add pci_claim_bridge_resource() to clip
window if necessary") introduced a new API to claim bridge resources.
pci_claim_bridge_resource() tries to claim a bridge resource, and if
the claiming fails the function tries to clip the resource to make
it fit within the parent resource window.
If the clipping succeeds the bridge apertures are set-up accordingly
and the pci_claim_bridge_resource() tries to claim the resource
again.
Commit c770cb4cb505 ("PCI: Mark invalid BARs as unassigned") added
code that sets the IORESOURCE_UNSET flag on claiming failure.
This means that the second resource claiming after window clipping will
always fail, since the resource flags contain IORESOURCE_UNSET,
previously set on failure by pci_claim_resource(), so the subsequent
pci_claim_resource() call ends up spitting a log message and return
failure with no chance whatsoever to succeed.
This patch clear the UNSET in resource flags, and it makes second
pci_claim_resource() work again.
Fixes: c770cb4cb505 ("PCI: Mark invalid BARs as unassigned")
Cc: stable@vger.kernel.org [v4.1+]
[change to clear UNSET instead, Yinghai]
Signed-off-by: Yinghai Lu <yinghai@kernel.org>
@@ -733,6 +733,7 @@ int pci_claim_bridge_resource(struct pci_dev *bridge, int i)
return -EINVAL;
}
+ bridge->resource[i].flags &= ~IORESOURCE_UNSET;
if (pci_claim_resource(bridge, i) == 0)
return 0; /* claimed a smaller window */