From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Jon Smirl <jonsmirl@yahoo.com>
Cc: James Simmons <jsimmons@infradead.org>,
fb-devel <linux-fbdev-devel@lists.sourceforge.net>
Subject: Re: Re: mode setting and console locking, nvidia failure
Date: Tue, 18 May 2004 13:37:14 +1000 [thread overview]
Message-ID: <1084851433.15183.20.camel@gaston> (raw)
In-Reply-To: <20040518033811.3221.qmail@web14930.mail.yahoo.com>
> One part of this has got to be a problem with the console semaphore. That
> semaphore should not be accessed from an fbdev driver. If you need to access if
> from an fbdev driver then the fbdev design is broken. Think about the case with
> multiple fbdev's loaded. Since we only have one console all of the fbdev can't
> play with the semaphore. That semaphore needs to be controlled from fb/vgacon
> and lock only the fbdev that is displaying the console, not the others.
>
> Next experiment is to remove all reference to the console semaphore from my
> fbdev driver.
You should try to understand things rather...
You should _not_ touch the console semaphore from your driver except in
special known circumstances like the PM code path. It will make sure you
don't get called by fbcon, but it's perfectly safe to take it even when
you aren't beeing called by fbcon. However, you should not take it on
normal fbdev operations since you will be called with it already held by
the common fbmem code. The only case where you may want to bother is the
PM callback.
Ben.
-------------------------------------------------------
This SF.Net email is sponsored by: SourceForge.net Broadband
Sign-up now for SourceForge Broadband and get the fastest
6.0/768 connection for only $19.95/mo for the first 3 months!
http://ads.osdn.com/?ad_id=2562&alloc_id=6184&op=click
next prev parent reply other threads:[~2004-05-18 3:43 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-05-16 15:57 mode setting and console locking Jon Smirl
2004-05-16 22:03 ` Benjamin Herrenschmidt
2004-05-17 2:29 ` Jon Smirl
2004-05-17 3:51 ` Jon Smirl
2004-05-17 3:51 ` Benjamin Herrenschmidt
2004-05-17 4:12 ` Jon Smirl
2004-05-17 4:15 ` Benjamin Herrenschmidt
2004-05-17 4:51 ` Re: mode setting and console locking, nvidia failure Jon Smirl
2004-05-17 5:03 ` Jon Smirl
2004-05-17 22:56 ` James Simmons
2004-05-17 23:26 ` Jon Smirl
2004-05-18 3:00 ` Jon Smirl
2004-05-18 3:38 ` Jon Smirl
2004-05-18 3:37 ` Benjamin Herrenschmidt [this message]
2004-05-18 3:53 ` Jon Smirl
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1084851433.15183.20.camel@gaston \
--to=benh@kernel.crashing.org \
--cc=jonsmirl@yahoo.com \
--cc=jsimmons@infradead.org \
--cc=linux-fbdev-devel@lists.sourceforge.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).