Skip to contentSkip to conversation

shouldn't nouveau.ko depend on fbcon.ko?

PATCH Applied Dragos Delcea 3 participants 4 messages Aug 25, 2010 3 messages not in the archive

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

Hi,

I noticed that loading nouveau doesn't load fbcon automatically; and without fbcon, when loading nouveau the console turns blank. I spent a few good hours until I was able to figure out that I had to manually modprobe fbcon to get the console back after loading nouveau.
This is happening on a vanilla 2.6.35.3; shouldn't there be dependencies in place between modules?

Note that I haven't got to the xorg part just yet, I'm currently installing gentoo.

thanks,
Dragos

PS please include me in the reply as I'm not subscribed.

_______________________________________________
Nouveau mailing list [email protected]
http://lists.freedesktop.org/mailman/listinfo/nouveau

A message that is not in the archive (sent off-list or not yet received): AANLkTinqnf3qpSBhb6g7EtxVO3d+_BbcYfxXZJC9Ht9A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org

#2 lore raw
Quoted 9 lines · Dragos Delcea

Hi,

I noticed that loading nouveau doesn't load fbcon automatically; and without fbcon, when loading nouveau the console turns blank. I spent a few good hours until I was able to figure out that I had to manually modprobe fbcon to get the console back after loading nouveau.
This is happening on a vanilla 2.6.35.3; shouldn't there be dependencies in place between modules?

Note that I haven't got to the xorg part just yet, I'm currently installing gentoo.

We already do, except for kernels compiled with "Configure standard kernel features (for small systems)". Strictly speaking, nouveau doesn't depend on fbcon so that option is left open for e.g. embedded platforms that don't need a framebuffer console.

Quoted 9 lines

thanks,
Dragos

PS please include me in the reply as I'm not subscribed.

_______________________________________________
Nouveau mailing list [email protected]
http://lists.freedesktop.org/mailman/listinfo/nouveau

_______________________________________________
Nouveau mailing list [email protected]
http://lists.freedesktop.org/mailman/listinfo/nouveau

A message that is not in the archive (sent off-list or not yet received): [email protected]

#4 lore raw

PATCH 1/1 drm/nouveau: try to load fbcon

Quoted 9 lines · Francisco Jerez
Quoted 7 lines · Dragos Delcea

Hi,

I noticed that loading nouveau doesn't load fbcon automatically; and without fbcon, when loading nouveau the console turns blank. I spent a few good hours until I was able to figure out that I had to manually modprobe fbcon to get the console back after loading nouveau.
This is happening on a vanilla 2.6.35.3; shouldn't there be dependencies in place between modules?

Quoted 6 lines
Quoted 2 lines

Note that I haven't got to the xorg part just yet, I'm currently installing gentoo.

We already do, except for kernels compiled with "Configure standard kernel features (for small systems)". Strictly speaking, nouveau doesn't depend on fbcon so that option is left open for e.g. embedded platforms that don't need a framebuffer console.

But it's only a build time dependency. Let's fix it.

---
From: Marcin Slusarz <[email protected]>
Subject: [PATCH] drm/nouveau: try to load fbcon

Currently users have to load fbcon manually or build it into the kernel to have any console output. Let's make it easy for them and load fbcon automatically at init time. Nouveau already selects fbcon at build time when !EMBEDDED.

---
 drivers/gpu/drm/nouveau/nouveau_drv.c |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)
drivers/gpu/drm/nouveau/nouveau_drv.c +6 -0
index a8d3d17..df9a687 100644
@@ -434,6 +434,12 @@ static int __init nouveau_init(void)
 	if (!nouveau_modeset)
 		return 0;
 
+#if defined(CONFIG_FRAMEBUFFER_CONSOLE_MODULE)
+	request_module("fbcon");
+#elif !defined(CONFIG_FRAMEBUFFER_CONSOLE)
+	printk(KERN_INFO "CONFIG_FRAMEBUFFER_CONSOLE was not enabled. You won't get any console output.\n");
+#endif
+
 	nouveau_register_dsm_handler();
 	return drm_init(&driver);
 }
Signature · 1 line
1.7.1.1

A message that is not in the archive (sent off-list or not yet received): [email protected]

#6 lore raw

Re: [PATCH] drm/nouveau: try to load fbcon

Quoted 42 lines · Marcin Slusarz
Quoted 9 lines · Francisco Jerez
Quoted 7 lines · Dragos Delcea

Hi,

I noticed that loading nouveau doesn't load fbcon automatically; and without fbcon, when loading nouveau the console turns blank. I spent a few good hours until I was able to figure out that I had to manually modprobe fbcon to get the console back after loading nouveau.
This is happening on a vanilla 2.6.35.3; shouldn't there be dependencies in place between modules?

Quoted 6 lines
Quoted 2 lines

Note that I haven't got to the xorg part just yet, I'm currently installing gentoo.

We already do, except for kernels compiled with "Configure standard kernel features (for small systems)". Strictly speaking, nouveau doesn't depend on fbcon so that option is left open for e.g. embedded platforms that don't need a framebuffer console.

But it's only a build time dependency. Let's fix it.

---
From: Marcin Slusarz <[email protected]>
Subject: [PATCH] drm/nouveau: try to load fbcon

Currently users have to load fbcon manually or build it into the kernel to have any console output. Let's make it easy for them and load fbcon automatically at init time. Nouveau already selects fbcon at build time when !EMBEDDED.

---
 drivers/gpu/drm/nouveau/nouveau_drv.c |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)
drivers/gpu/drm/nouveau/nouveau_drv.c +0 -0
index a8d3d17..df9a687 100644
@@ -434,6 +434,12 @@ static int __init nouveau_init(void)
 	if (!nouveau_modeset)
 		return 0;

drivers/gpu/drm/nouveau/nouveau_drv.c static int __init nouveau_init(void) · line 445

Quoted 10 lines
drivers/gpu/drm/nouveau/nouveau_drv.c +6 -0
 
+#if defined(CONFIG_FRAMEBUFFER_CONSOLE_MODULE)
+	request_module("fbcon");
+#elif !defined(CONFIG_FRAMEBUFFER_CONSOLE)
+	printk(KERN_INFO "CONFIG_FRAMEBUFFER_CONSOLE was not enabled. You won't get any console output.\n");
+#endif
+
 	nouveau_register_dsm_handler();
 	return drm_init(&driver);
 }

Pushed to master, thanks.

_______________________________________________
Nouveau mailing list [email protected]
http://lists.freedesktop.org/mailman/listinfo/nouveau