All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/5] vt: Fix replacement console check when unbinding
@ 2014-06-05 14:58 ` Daniel Vetter
  0 siblings, 0 replies; 72+ messages in thread
From: Daniel Vetter @ 2014-06-05 14:58 UTC (permalink / raw)
  To: Intel Graphics Development
  Cc: DRI Development, LKML, Daniel Vetter, Greg Kroah-Hartman, Jiri Slaby

I don't fully understand the magic of the vt register/unregister
logic, but apparently everything but the inital console (as set
in the conswitchp pointer) is marked with FLAG_MODULE. Which means
if something unregistered the boot vt driver (e.g. i915.ko kicking
out vga_con) there's nothing left when trying to unbind e.g. fbcon
through sysfs.

But we always have the dummy console hanging around, so this test
is fairly dubious. What we actually want is simply a different console
than the one we want to unbind.

Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Jiri Slaby <jslaby@suse.cz>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
---
 drivers/tty/vt/vt.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/tty/vt/vt.c b/drivers/tty/vt/vt.c
index 3ad0b61e35b4..ea600f482eeb 100644
--- a/drivers/tty/vt/vt.c
+++ b/drivers/tty/vt/vt.c
@@ -3155,8 +3155,7 @@ int do_unbind_con_driver(const struct consw *csw, int first, int last, int deflt
 	for (i = 0; i < MAX_NR_CON_DRIVER; i++) {
 		con_back = &registered_con_driver[i];
 
-		if (con_back->con &&
-		    !(con_back->flag & CON_DRIVER_FLAG_MODULE)) {
+		if (con_back->con && con_back->con != csw) {
 			defcsw = con_back->con;
 			retval = 0;
 			break;
-- 
1.8.1.4


^ permalink raw reply related	[flat|nested] 72+ messages in thread

end of thread, other threads:[~2014-07-08  8:10 UTC | newest]

Thread overview: 72+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-06-05 14:58 [PATCH 1/5] vt: Fix replacement console check when unbinding Daniel Vetter
2014-06-05 14:58 ` Daniel Vetter
2014-06-05 14:58 ` [PATCH 2/5] vt: Fix up unregistration of vt drivers Daniel Vetter
2014-06-05 14:58   ` Daniel Vetter
2014-06-06  7:24   ` David Herrmann
2014-06-06  7:24     ` David Herrmann
2014-06-06  7:56     ` Daniel Vetter
2014-06-06  7:56       ` Daniel Vetter
2014-06-06  8:47       ` David Herrmann
2014-06-06  9:40         ` Daniel Vetter
2014-06-06  9:40           ` Daniel Vetter
2014-06-06 15:51           ` Greg Kroah-Hartman
2014-06-06 20:21             ` [Intel-gfx] " Daniel Vetter
2014-06-06 20:21               ` Daniel Vetter
2014-06-05 14:58 ` [PATCH 3/5] vt: Don't ignore unbind errors in vt_unbind Daniel Vetter
2014-06-05 14:58   ` Daniel Vetter
2014-06-06  7:13   ` David Herrmann
2014-06-06  7:13     ` David Herrmann
2014-06-05 14:58 ` [PATCH 4/5] drm/i915: Fixup global gtt cleanup Daniel Vetter
2014-06-05 14:58   ` Daniel Vetter
2014-06-06 13:22   ` Imre Deak
2014-06-06 13:22     ` Imre Deak
2014-06-05 14:58 ` [PATCH 5/5] drm/i915: Kick out vga console Daniel Vetter
2014-06-05 14:58   ` Daniel Vetter
2014-06-05 14:58   ` Daniel Vetter
2014-06-06  7:28   ` David Herrmann
2014-06-06  7:28     ` David Herrmann
2014-06-06  7:28     ` David Herrmann
2014-06-06  7:47     ` Daniel Vetter
2014-06-06  7:47       ` Daniel Vetter
2014-06-06  7:47       ` Daniel Vetter
2014-06-06 15:20   ` Daniel Vetter
2014-06-06 15:20     ` Daniel Vetter
2014-06-06 15:20     ` Daniel Vetter
2014-06-09 13:22     ` Tomi Valkeinen
2014-06-09 13:22       ` Tomi Valkeinen
2014-06-09 13:22       ` Tomi Valkeinen
2014-06-28 19:28   ` Ed Tomlinson
2014-06-29  3:55     ` Ed Tomlinson
2014-06-29  3:55       ` Ed Tomlinson
2014-06-30  6:59       ` Chris Wilson
2014-06-30  6:59         ` Chris Wilson
2014-06-30  6:59         ` Chris Wilson
2014-06-30  8:19         ` David Herrmann
2014-06-30  8:19           ` David Herrmann
2014-07-01 13:51         ` Ed Tomlinson
2014-07-01 13:51           ` Ed Tomlinson
2014-07-01 13:51           ` Ed Tomlinson
2014-07-07  8:48         ` [Intel-gfx] " Daniel Vetter
2014-07-07  8:48           ` Daniel Vetter
2014-07-07  8:48           ` [Intel-gfx] " Daniel Vetter
2014-07-07 10:45           ` Ed Tomlinson
2014-07-07 10:45             ` Ed Tomlinson
2014-07-07 10:45             ` [Intel-gfx] " Ed Tomlinson
2014-07-07 12:26             ` Daniel Vetter
2014-07-07 12:26               ` Daniel Vetter
2014-07-07 12:26               ` Daniel Vetter
2014-07-08  2:53               ` Ed Tomlinson
2014-07-08  2:53                 ` Ed Tomlinson
2014-07-08  2:53                 ` [Intel-gfx] " Ed Tomlinson
2014-07-08  8:10                 ` Daniel Vetter
2014-07-08  8:10                   ` Daniel Vetter
2014-07-08  8:10                   ` [Intel-gfx] " Daniel Vetter
2014-07-07 10:59           ` Ed Tomlinson
2014-07-07 10:59             ` Ed Tomlinson
2014-07-07 10:59             ` [Intel-gfx] " Ed Tomlinson
2014-06-06  7:16 ` [PATCH 1/5] vt: Fix replacement console check when unbinding David Herrmann
2014-06-06  7:16   ` David Herrmann
2014-06-06  7:49   ` Daniel Vetter
2014-06-06  7:49     ` Daniel Vetter
2014-06-06  9:43 ` [PATCH] " Daniel Vetter
2014-06-06  9:43   ` Daniel Vetter

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.