Skip to content

Commit 5003ad7

Browse files
committed
ACPI: OSL: Clean up the removal of unused memory mappings
Fold acpi_os_map_cleanup_deferred() into acpi_os_map_remove() and pass the latter to INIT_RCU_WORK() in acpi_os_drop_map_ref() to make the code more straightforward. No intentional functional impact. Signed-off-by: Rafael J. Wysocki <[email protected]>
1 parent a968fba commit 5003ad7

File tree

1 file changed

+6
-8
lines changed

1 file changed

+6
-8
lines changed

drivers/acpi/osl.c

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -377,18 +377,16 @@ void *__ref acpi_os_map_memory(acpi_physical_address phys, acpi_size size)
377377
}
378378
EXPORT_SYMBOL_GPL(acpi_os_map_memory);
379379

380-
static void acpi_os_map_remove(struct acpi_ioremap *map)
380+
static void acpi_os_map_remove(struct work_struct *work)
381381
{
382+
struct acpi_ioremap *map = container_of(to_rcu_work(work),
383+
struct acpi_ioremap,
384+
track.rwork);
385+
382386
acpi_unmap(map->phys, map->virt);
383387
kfree(map);
384388
}
385389

386-
static void acpi_os_map_cleanup_deferred(struct work_struct *work)
387-
{
388-
acpi_os_map_remove(container_of(to_rcu_work(work), struct acpi_ioremap,
389-
track.rwork));
390-
}
391-
392390
/* Must be called with mutex_lock(&acpi_ioremap_lock) */
393391
static void acpi_os_drop_map_ref(struct acpi_ioremap *map)
394392
{
@@ -397,7 +395,7 @@ static void acpi_os_drop_map_ref(struct acpi_ioremap *map)
397395

398396
list_del_rcu(&map->list);
399397

400-
INIT_RCU_WORK(&map->track.rwork, acpi_os_map_cleanup_deferred);
398+
INIT_RCU_WORK(&map->track.rwork, acpi_os_map_remove);
401399
queue_rcu_work(system_wq, &map->track.rwork);
402400
}
403401

0 commit comments

Comments
 (0)