diff options
author | Linus Torvalds <[email protected]> | 2009-08-02 14:04:19 -0700 |
---|---|---|
committer | Linus Torvalds <[email protected]> | 2009-08-02 14:10:18 -0700 |
commit | 79896cf42f6a96d7e14f2dc3473443d68d74031d (patch) | |
tree | bf4aa83fedfbe51d9a2a60d03007ec69de5a0cd2 /scripts/basic/docproc.c | |
parent | 0dd8486b5cfe8048e0613334659d9252ecd1b08a (diff) |
Make pci_claim_resource() use request_resource() rather than insert_resource()
This function has traditionally used "insert_resource()", because before
commit cebd78a8c5 ("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 a76117dfd6 ("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 <[email protected]>
Cc: Matthew Wilcox <[email protected]>
Cc: Andrew Patterson <[email protected]>
Cc: Linux PCI <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Diffstat (limited to 'scripts/basic/docproc.c')
0 files changed, 0 insertions, 0 deletions