* new radeon code and acquire/release_console_sem()
@ 2003-09-14 0:23 Jon Smirl
2003-09-14 0:29 ` Benjamin Herrenschmidt
0 siblings, 1 reply; 4+ messages in thread
From: Jon Smirl @ 2003-09-14 0:23 UTC (permalink / raw)
To: Benjamin Herrenschmidt, kronos; +Cc: fb-devel
I don't run fb console on my system so I don't have acquire_console_sem() and
release_console_sem(). Shouldn't these have an #ifdef CONFIG_FRAMEBUFFER_CONSOLE?
=====
Jon Smirl
jonsmirl@yahoo.com
__________________________________
Do you Yahoo!?
Yahoo! SiteBuilder - Free, easy-to-use web site design software
http://sitebuilder.yahoo.com
-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: new radeon code and acquire/release_console_sem()
2003-09-14 0:23 new radeon code and acquire/release_console_sem() Jon Smirl
@ 2003-09-14 0:29 ` Benjamin Herrenschmidt
2003-09-14 0:34 ` Jon Smirl
0 siblings, 1 reply; 4+ messages in thread
From: Benjamin Herrenschmidt @ 2003-09-14 0:29 UTC (permalink / raw)
To: Jon Smirl; +Cc: Kronos, fb-devel
On Sun, 2003-09-14 at 02:23, Jon Smirl wrote:
> I don't run fb console on my system so I don't have acquire_console_sem() and
> release_console_sem(). Shouldn't these have an #ifdef CONFIG_FRAMEBUFFER_CONSOLE?
Those are in kernel/printk.c, you should probably have them.
We need some locking for the fbcon itself and so far, this is the best we
have at hand though it's still broken as the various ioctls should take
it as well.
Ben.
-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: new radeon code and acquire/release_console_sem()
2003-09-14 0:29 ` Benjamin Herrenschmidt
@ 2003-09-14 0:34 ` Jon Smirl
2003-09-14 0:38 ` Benjamin Herrenschmidt
0 siblings, 1 reply; 4+ messages in thread
From: Jon Smirl @ 2003-09-14 0:34 UTC (permalink / raw)
To: Benjamin Herrenschmidt; +Cc: Kronos, fb-devel
I get this from insmod:
radeonfb: Unknown symbol release_console_sem
--- Benjamin Herrenschmidt <benh@kernel.crashing.org> wrote:
> On Sun, 2003-09-14 at 02:23, Jon Smirl wrote:
> > I don't run fb console on my system so I don't have acquire_console_sem()
> and
> > release_console_sem(). Shouldn't these have an #ifdef
> CONFIG_FRAMEBUFFER_CONSOLE?
>
> Those are in kernel/printk.c, you should probably have them.
>
> We need some locking for the fbcon itself and so far, this is the best we
> have at hand though it's still broken as the various ioctls should take
> it as well.
>
> Ben.
>
>
>
>
> -------------------------------------------------------
> This sf.net email is sponsored by:ThinkGeek
> Welcome to geek heaven.
> http://thinkgeek.com/sf
> _______________________________________________
> Linux-fbdev-devel mailing list
> Linux-fbdev-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/linux-fbdev-devel
=====
Jon Smirl
jonsmirl@yahoo.com
__________________________________
Do you Yahoo!?
Yahoo! SiteBuilder - Free, easy-to-use web site design software
http://sitebuilder.yahoo.com
-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: new radeon code and acquire/release_console_sem()
2003-09-14 0:34 ` Jon Smirl
@ 2003-09-14 0:38 ` Benjamin Herrenschmidt
0 siblings, 0 replies; 4+ messages in thread
From: Benjamin Herrenschmidt @ 2003-09-14 0:38 UTC (permalink / raw)
To: Jon Smirl; +Cc: Kronos, fb-devel
On Sun, 2003-09-14 at 02:34, Jon Smirl wrote:
> I get this from insmod:
> radeonfb: Unknown symbol release_console_sem
I think the problem is that the core kernel doesn't export it,
I'll look into this. I added the console sem stuff to all the
PM code (atyfb, aty128fb and radeonfb so far, chipsfb soon)
because I want to prevent any access while setting up the
chip for sleep
Ben.
-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2003-09-14 0:38 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2003-09-14 0:23 new radeon code and acquire/release_console_sem() Jon Smirl
2003-09-14 0:29 ` Benjamin Herrenschmidt
2003-09-14 0:34 ` Jon Smirl
2003-09-14 0:38 ` Benjamin Herrenschmidt
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).