linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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

  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).