Skip to content

Commit

Permalink
ACPI battery: fix async boot oops
Browse files Browse the repository at this point in the history
> BUG: unable to handle kernel NULL pointer dereference at (null)

What happens is that the battery module's init sections are being freed
before the async callback (which was marked __init) has run. This theory
is supported by the fact that the bad RIP value is a vmalloc address.

The immediate fix is to make this a non-init call.

(A better long-term fix is of course to wait with init-section unloading
until a module's async initcalls have been run, which would allow us to
discard this function which is still only run once, after all. Perhaps a
new async_initcall() function for the async/module API, if this is needed
for other modules in the future?)

Reported-by: Arkadiusz Miskiewicz <[email protected]>
Signed-off-by: Vegard Nossum <[email protected]>
Tested-by: Alessandro Suardi <[email protected]>
Tested-by: Rafael J. Wysocki <[email protected]>
Signed-off-by: Len Brown <[email protected]>
  • Loading branch information
vegard authored and lenb committed Apr 7, 2009
1 parent a3c2705 commit 5d38258
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/acpi/battery.c
Original file line number Diff line number Diff line change
Expand Up @@ -903,7 +903,7 @@ static struct acpi_driver acpi_battery_driver = {
},
};

static void __init acpi_battery_init_async(void *unused, async_cookie_t cookie)
static void acpi_battery_init_async(void *unused, async_cookie_t cookie)
{
if (acpi_disabled)
return;
Expand Down

0 comments on commit 5d38258

Please sign in to comment.