[PATCH] Add try_acquire_console_sem

Benjamin Herrenschmidt benh at kernel.crashing.org
Tue Jan 18 12:39:18 EST 2005


Hi !

The new PowerMac sleep code provides an arch hook that can be used by video drivers
on laptops to bring back the screen very early, pretty much before anything else.
This basically turns reports of the style "my laptop doesn't wakeup" to "I get this
or this oops/error/panic on wakeup", making fixing the PM related bugs possible on
a whole range of them.

However, the fbdev wakeup code triggers WARN_ON's in the VT subsystem if called
without the console semaphore when redrawing the screen (I added those warnings a
couple of kernel versions ago), and we can't call acquire_console_sem() since we
are so early in the wakeup process that we are considered as in_atomic() (we hold
irqs off too).

This patch addds a try_acquire_console_sem() function that can be used by those
video drivers that implement this early wakeup hook. If the acquire fails (which
should never happen in practice), wakeup is delayed to the normal PCI callback
which does a blocking acquire_console_sem().

Signed-off-by: Benjamin Herrenschmidt <benh at kernel.crashing.org>

Index: linux-work/include/linux/console.h
===================================================================
--- linux-work.orig/include/linux/console.h	2004-11-22 11:50:54.000000000 +1100
+++ linux-work/include/linux/console.h	2005-01-17 13:49:21.006958624 +1100
@@ -105,6 +105,7 @@
 extern int unregister_console(struct console *);
 extern struct console *console_drivers;
 extern void acquire_console_sem(void);
+extern int try_acquire_console_sem(void);
 extern void release_console_sem(void);
 extern void console_conditional_schedule(void);
 extern void console_unblank(void);
Index: linux-work/kernel/printk.c
===================================================================
--- linux-work.orig/kernel/printk.c	2005-01-14 08:17:22.000000000 +1100
+++ linux-work/kernel/printk.c	2005-01-17 13:49:21.008958320 +1100
@@ -611,6 +611,16 @@
 }
 EXPORT_SYMBOL(acquire_console_sem);
 
+int try_acquire_console_sem(void)
+{
+	if (down_trylock(&console_sem))
+		return -1;
+	console_locked = 1;
+	console_may_schedule = 0;
+	return 0;
+}
+EXPORT_SYMBOL(try_acquire_console_sem);
+
 int is_console_locked(void)
 {
 	return console_locked;





More information about the Linuxppc-dev mailing list