]> git.baikalelectronics.ru Git - kernel.git/commit
Make pci_claim_resource() use request_resource() rather than insert_resource()
authorLinus Torvalds <torvalds@linux-foundation.org>
Sun, 2 Aug 2009 21:04:19 +0000 (14:04 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Sun, 2 Aug 2009 21:10:18 +0000 (14:10 -0700)
commit0d938d4672cf45013a95fb8404b825f88fecda5e
treebf4aa83fedfbe51d9a2a60d03007ec69de5a0cd2
parent134451bac83834bc4923d489687757665595d607
Make pci_claim_resource() use request_resource() rather than insert_resource()

This function has traditionally used "insert_resource()", because before
commit 71899fdac3 ("Fix pci_claim_resource") it used to just insert the
resource into whatever root resource tree that was indicated by
"pcibios_select_root()".

So there Matthew fixed it to actually look up the proper parent
resource, which means that now it's actively wrong to then traverse the
resource tree any more: we already know exactly where the new resource
should go.

And when we then did commit 1a22e636ce ("x86: Use pci_claim_resource"),
which changed the x86 PCI code from the open-coded

pr = pci_find_parent_resource(dev, r);
if (!pr || request_resource(pr, r) < 0) {

to using

if (pci_claim_resource(dev, idx) < 0) {

that "insert_resource()" now suddenly became a problem, and causes a
regression covered by

http://bugzilla.kernel.org/show_bug.cgi?id=13891

which this fixes.

Reported-and-tested-by: Rafael J. Wysocki <rjw@sisk.pl>
Cc: Matthew Wilcox <willy@linux.intel.com>
Cc: Andrew Patterson <andrew.patterson@hp.com>
Cc: Linux PCI <linux-pci@vger.kernel.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/pci/setup-res.c