diff options
| author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2017-04-17 02:20:48 +0300 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2017-04-19 23:57:04 +0300 |
| commit | c381fc3a1bbfc9d780048a0b94afd162e4c3815b (patch) | |
| tree | f4499d7a1cf594caed95fc43dea3b5d634e5788b /drivers | |
| parent | f5beabfe61794d9a9d9549d387cda2bffd81e504 (diff) | |
| download | linux-c381fc3a1bbfc9d780048a0b94afd162e4c3815b.tar.xz | |
ACPI / scan: Avoid enumerating devices more than once
acpi_bus_attach() does not check the visited flag for devices that
have been enumerated already and some of them may be enumerated
for multiple times as a result, because some callers of
acpi_bus_scan() don't check the visited flag either.
For this reason, modify acpi_bus_attach() to check the visited flag
and avoid enumerating devices that have already been enumerated.
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Reviewed-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Reviewed-by: Joey Lee <jlee@suse.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/acpi/scan.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/acpi/scan.c b/drivers/acpi/scan.c index f74bc0d28692..c26931067415 100644 --- a/drivers/acpi/scan.c +++ b/drivers/acpi/scan.c @@ -1840,6 +1840,8 @@ static void acpi_bus_attach(struct acpi_device *device) device->flags.power_manageable = 0; device->flags.initialized = true; + } else if (device->flags.visited) { + goto ok; } ret = acpi_scan_attach_handler(device); |