Hi! Pavel,
Could you do me just one tiny favor?
Please consider attempting an error recovery level path, maybe ...
Every patch I have glanced at has 'panic("blah blah");'
If you do not have enough hardware to generate an accurate path for
recovery, then please do not force the kernel into panic. Would you
consider failing the request making it jump back to S1 ? This at least
allows it crash like a power failure.
Cheers,
Andre Hedrick
LAD Storage Consulting Group
On Sun, 19 Jan 2003, Pavel Machek wrote:
> Hi!
>
> SUSPEND_RESUME phase is needed for turning off IO-APIC. [I believe
> SUSPEND_DISABLE should be so simple that errors just should not be
> there, and besides we would not know how to safely enable devices from
> such weird state, anyway]. Please apply,
>
> Pavel
>
> --- clean/drivers/acpi/sleep.c 2003-01-05 22:58:25.000000000 +0100
> +++ linux-swsusp/drivers/acpi/sleep.c 2003-01-19 21:27:00.000000000 +0100
> @@ -143,6 +143,10 @@
> return error;
> }
>
> + error = device_suspend(state, SUSPEND_DISABLE);
> + if (error)
> + panic("Sorry, devices really should know how to disable\n");
> +
> /* flush caches */
> ACPI_FLUSH_CPU_CACHE();
>
> --
> Worst form of spam? Adding advertisment signatures ala sourceforge.net.
> What goes next? Inserting advertisment *into* email?
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Thu Jan 23 2003 - 22:00:25 EST