From: Jon Smirl <jonsmirl@gmail.com>
To: Pavel Machek <pavel@ucw.cz>
Cc: dri-devel <dri-devel@lists.sourceforge.net>,
lkml <linux-kernel@vger.kernel.org>
Subject: Re: Design for setting video modes, ownership of sysfs attributes
Date: Tue, 21 Sep 2004 11:56:52 -0400 [thread overview]
Message-ID: <9e473391040921085669c2dcd7@mail.gmail.com> (raw)
In-Reply-To: <20040921124507.GC2383@elf.ucw.cz>
On Tue, 21 Sep 2004 14:45:07 +0200, Pavel Machek <pavel@ucw.cz> wrote:
> Hi!
>
> > 1) user owns graphics devices
> > 2) user sets mode with string (or similar) format using ioctl common to
> > all drivers.
> > 3) driver is locked to prevent multiple mode sets
> > 4) common code takes this string and does a hotplug event with it.
>
> I though this was
>
> "Driver decides to either do it itself in kernel, or call userspace
> helper if that would be too complex".
The driver almost always needs to go to user space to get the file of
mode line overrides that the user can create. But there is nothing
stopping you from building everything in the kernel.
I'd just rather not have 100K of resident code waiting around for
something that doesn't occur very often. For the radeon sample I'm
working on I have moved everything to user space except for a couple
of small helper functions that directly play with the registers. Note
that all mode setting in X occurs completely in user space so you
can't argue that it can't be done.
> > How are errors going to be communicated in this scheme? I can cat the
> > sysfs mode variable to get a status. Is there a good way to do this
> > without polling?
>
> I'd say that write() to that sysfs file can simply return error. See
> echo disk > /sys/power/state, it returns error if transition failed.
>
> Pavel
> --
> People were complaining that M$ turns users into beta-testers...
> ...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!
>
--
Jon Smirl
jonsmirl@gmail.com
next prev parent reply other threads:[~2004-09-21 15:56 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-09-18 18:43 Design for setting video modes, ownership of sysfs attributes Jon Smirl
2004-09-18 19:58 ` Mike Mestnik
2004-09-18 22:12 ` Jon Smirl
2004-09-18 22:37 ` Mike Mestnik
2004-09-18 23:33 ` Keith Packard
2004-09-19 0:54 ` Jon Smirl
2004-09-19 1:57 ` Vladimir Dergachev
2004-09-19 2:16 ` Jon Smirl
2004-09-19 2:32 ` Vladimir Dergachev
2004-09-19 10:11 ` Mike Mestnik
2004-09-19 9:55 ` Mike Mestnik
2004-09-19 4:48 ` Benjamin Herrenschmidt
2004-09-19 16:12 ` Jon Smirl
2004-09-20 0:07 ` Benjamin Herrenschmidt
2004-09-20 1:06 ` Jon Smirl
2004-09-19 4:45 ` Benjamin Herrenschmidt
2004-09-19 16:46 ` Jon Smirl
2004-09-19 17:19 ` Mike Mestnik
2004-09-19 20:40 ` Keith Packard
2004-09-20 13:02 ` Alan Cox
2004-09-19 20:44 ` Felix Kühling
2004-09-20 1:25 ` Mike Mestnik
2004-09-20 0:10 ` Benjamin Herrenschmidt
2004-09-21 12:45 ` Pavel Machek
2004-09-21 15:56 ` Jon Smirl [this message]
2004-09-21 15:42 ` Alan Cox
[not found] <2FYdH-10h-5@gated-at.bofh.it>
[not found] ` <2G6Et-6D7-31@gated-at.bofh.it>
2004-09-19 14:18 ` Pascal Schmidt
2004-09-19 15:00 ` P. Benie
2004-09-19 19:08 ` Pascal Schmidt
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=9e473391040921085669c2dcd7@mail.gmail.com \
--to=jonsmirl@gmail.com \
--cc=dri-devel@lists.sourceforge.net \
--cc=linux-kernel@vger.kernel.org \
--cc=pavel@ucw.cz \
/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