diff options
author | Chen, Justin <justin.chen@hp.com> | 2006-10-10 17:07:00 -0400 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2006-10-27 02:31:16 -0400 |
commit | 2f000f5c153e984d5c166e42a9d38113de8693b3 (patch) | |
tree | bec7309d65b6ada54c86692df4cfa85a8d57137c /drivers/acpi | |
parent | 616362de2fe224512fe105aec08f19f5470afb01 (diff) |
ACPI: optimize pci_rootbridge search
acpi_get_pci_rootbridge_handle() walks the ACPI name space
searching for seg, bus and the PCI_ROOT_HID_STRING --
returning the handle as soon as if find the match.
But the current codes always parses through the whole namespace because
the user_function find_pci_rootbridge() returns status=AE_OK when it finds the match.
Make the find_pci_rootbridge() return AE_CTRL_TERMINATE when it finds the match.
This reduces the ACPI namespace walk for acpi_get_pci_rootbridge_handle().
Signed-off-by: Justin Chen <justin.chen@hp.com>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/acpi')
-rw-r--r-- | drivers/acpi/glue.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/acpi/glue.c b/drivers/acpi/glue.c index 10f160dc75b1..ba8fe83c6298 100644 --- a/drivers/acpi/glue.c +++ b/drivers/acpi/glue.c | |||
@@ -189,8 +189,12 @@ find_pci_rootbridge(acpi_handle handle, u32 lvl, void *context, void **rv) | |||
189 | bus = tmp; | 189 | bus = tmp; |
190 | 190 | ||
191 | if (seg == find->seg && bus == find->bus) | 191 | if (seg == find->seg && bus == find->bus) |
192 | { | ||
192 | find->handle = handle; | 193 | find->handle = handle; |
193 | status = AE_OK; | 194 | status = AE_CTRL_TERMINATE; |
195 | } | ||
196 | else | ||
197 | status = AE_OK; | ||
194 | exit: | 198 | exit: |
195 | kfree(buffer.pointer); | 199 | kfree(buffer.pointer); |
196 | return status; | 200 | return status; |