[PATCH 0/4] Avoid panic in case of buggy bios

From: Glauber Costa
Date: Mon Feb 18 2008 - 16:03:25 EST


Hi,

I've came across a panic today after failing acpi_processor_start()
in this condition:

if (processor_device_array[pr->id] != NULL &&
processor_device_array[pr->id] != device) {
printk(KERN_WARNING "BIOS reported wrong ACPI id "
"for the processor\n");
return -ENODEV;
}

The problem is that there are some data structures that are not initialized
if we exit that early, but the removal functions assume to be present. One
solution would be to move the checks to the bottom of this function, after
all the initializations are complete, but I personally think this is
not a good idea, from the waste point of view.

So I'm attaching a series of 4 patches that provides a criteria to decide
whether or not we got past the point of initializing that data structures,
and using it to conditionally unregister them latter.

Hope it's okay with you



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/