From: "Michal Suchanek" <hramrach@centrum.cz>
To: "Antonino A. Daplas" <adaplas@gmail.com>
Cc: Andrew Morton <akpm@osdl.org>,
linux-fbdev-devel@lists.sourceforge.net,
Linux Kernel Development <linux-kernel@vger.kernel.org>,
Knut Petersen <Knut_Petersen@t-online.de>
Subject: Re: [PATCH 0/7] Detaching fbcon
Date: Mon, 12 Jun 2006 18:44:36 +0200 [thread overview]
Message-ID: <a5d587fb0606120944t6d064e9bvb1bfa24eb69e5ce6@mail.gmail.com> (raw)
In-Reply-To: <448D7716.5070205@gmail.com>
On 6/12/06, Antonino A. Daplas <adaplas@gmail.com> wrote:
> Michal Suchanek wrote:
> > On 6/12/06, Antonino A. Daplas <adaplas@gmail.com> wrote:
> >>> Non-experimental behaviour really should be to allow unbinding
> >>> only if the framebuffer driver allows it. The fbcon and vt layer simply
> >>> does not know if it is safe, and thus they have to interrogate the
> >>> hardware layer that knows the answer.
> >>>
> >>> Your current proposal is to allow unbinding and removal of the framebuffer
> >>> driver and the fbcon layer, no matter what the result will be. I don´t think
> >>> that this is ok. There is John User that tries to switch to text mode, and
> >>> he will complain if it does leave his hardware in an unusable state.
> >> This I disagree with you. The view that an operation should be totally
> >> done 100% within the kernel is very utopic. We already have features that
> >> require both the kernel and userspace for them to work. The nearest example
> >> is suspend/resume. It's good if you can make suspend/resume work without
> >> additional effort, but that's not the case. Majority of users still need
> >> utilities such as vbetool.
> >
> > Well, I guess that the fb driver should try to restore the original
> > mode (ie text mode on x86) on unload. And it is either considered
> > flawed if it doesnt or it should indicate somewhere that it does not
> > know how to unload itself.
>
> Well, failure to implement proper suspend and resume by a driver does not
> stop the entire suspend/resume process, does it? Even if it results
> in a blank screen. It doesn't because we have a userspace options and
> tools that complements the kernel side. So why should it be any different
> in this case?
I do not say it must be different. vgacon (or whatever is the text
console called) fails to do proper suspend/resume for me. I just use X
and think the console is flawed. However, it would be nice if I was
warned in advance that the console will die on suspend.
>
> Some drivers, i810fb and rivafb, saves the hardware state on the
> first open, and restores the hardware state on the last close. This
> is usually sufficient for them to restore the hardware to VGA text
> mode on unload. It would be nice if we can implement something like this
> for all drivers, but failure to do so should not be a reason to stop the
> entire process.
>
> > I like the possibility to change X resolution using fbset (from inside
> > X). I use it to correct problems caused by crashed X programs that
> > change resolution. But I run X with the UseFbDev option. The X server
> > would be probably quite unhappy if I removed the fb driver but since
> > it uses the fb device the driver should not unload.
>
> If X is using fbdev, then X is also holding a reference count on the
> driver. And the driver will not unload even if you try to.
yes, that's it. And if it does not it is X problem. Not fb problem.
Thanks
Michal
_______________________________________________
Linux-fbdev-devel mailing list
Linux-fbdev-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-fbdev-devel
next prev parent reply other threads:[~2006-06-12 16:44 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-06-06 11:08 [PATCH 0/7] Detaching fbcon Antonino A. Daplas
2006-06-06 16:10 ` Jon Smirl
2006-06-06 16:19 ` Jon Smirl
2006-06-06 19:45 ` Antonino A. Daplas
2006-06-06 21:00 ` Jon Smirl
2006-06-06 21:39 ` Dave Airlie
2006-06-06 21:55 ` Jon Smirl
2006-06-06 23:15 ` Antonino A. Daplas
2006-06-06 23:17 ` Antonino A. Daplas
2006-06-06 19:45 ` Antonino A. Daplas
2006-06-11 13:25 ` Knut Petersen
2006-06-11 20:57 ` Antonino A. Daplas
2006-06-12 7:49 ` Knut Petersen
2006-06-12 12:17 ` Antonino A. Daplas
2006-06-12 12:25 ` Geert Uytterhoeven
2006-06-12 13:28 ` Michal Suchanek
2006-06-12 14:15 ` [Linux-fbdev-devel] " Antonino A. Daplas
2006-06-12 16:44 ` Michal Suchanek [this message]
2006-06-12 14:20 ` Michel Dänzer
2006-06-12 16:46 ` Michal Suchanek
2006-06-12 17:20 ` Michel Dänzer
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=a5d587fb0606120944t6d064e9bvb1bfa24eb69e5ce6@mail.gmail.com \
--to=hramrach@centrum.cz \
--cc=Knut_Petersen@t-online.de \
--cc=adaplas@gmail.com \
--cc=akpm@osdl.org \
--cc=linux-fbdev-devel@lists.sourceforge.net \
--cc=linux-kernel@vger.kernel.org \
/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).