Re: [RFC][PATCH 2/3] PM: Make console suspending configureable

[Stefan Seyfried <seife@suse.de>]

Hi,

sorry, i am only slowly catching up after vacation.

On Wed, Aug 16, 2006 at 01:09:34PM +0200, Rafael J. Wysocki wrote:
> Change suspend_console() so that it waits for all consoles to flush the
> remaining messages and make it possible to switch the console suspending
> off with the help of a Kconfig option.
> 
> Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>

> +#ifndef CONFIG_DISABLE_CONSOLE_SUSPEND
>  /**
>   * suspend_console - suspend the console subsystem
>   *
> @@ -709,8 +710,14 @@ int __init add_preferred_console(char *n
>   */
>  void suspend_console(void)
>  {
> +	printk("Suspending console(s)\n");
>  	acquire_console_sem();
>  	console_suspended = 1;
> +	/* This is needed so that all of the messages that have already been
> +	 * written to all consoles can be actually transmitted (eg. over a
> +	 * network) before we try to suspend the consoles' devices.
> +	 */
> +	ssleep(2);

Sorry, but no. Suspend and resume is already slow enough, no need to make
both of them much slower.
If we can condition this on the netconsole being used, ok, but not for the
most common case of "console is on plain VGA".

-- 
Stefan Seyfried                  \ "I didn't want to write for pay. I
QA / R&D Team Mobile Devices      \ wanted to be paid for what I write."
SUSE LINUX Products GmbH, Nürnberg \                    -- Leonard Cohen

-- 
VGER BF report: U 0.49988