From: Jon Smirl <jonsmirl@gmail.com>
To: Oliver Neukum <oliver@neukum.org>
Cc: Pavel Machek <pavel@ucw.cz>, Greg KH <greg@kroah.com>,
Mitchell Blank Jr <mitch@sfgoth.com>,
dtor_core@ameritech.net, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] driver core: Add the ability to unbind drivers to devices from userspace
Date: Fri, 5 Aug 2005 14:47:18 -0400 [thread overview]
Message-ID: <9e47339105080511474d89ee8a@mail.gmail.com> (raw)
In-Reply-To: <200508052020.13568.oliver@neukum.org>
On 8/5/05, Oliver Neukum <oliver@neukum.org> wrote:
> Am Freitag, 5. August 2005 20:14 schrieb Jon Smirl:
> > On 8/5/05, Oliver Neukum <oliver@neukum.org> wrote:
> > > Am Freitag, 5. August 2005 15:32 schrieb Jon Smirl:
> > > > On 1/1/02, Pavel Machek <pavel@ucw.cz> wrote:
> > > > > Hi!
> > > > >
> > > > > > > > New, simplified version of the sysfs whitespace strip patch...
> > > > > > >
> > > > > > > Could you tell me why you don't just fail the operation if malformed
> > > > > > > input is supplied?
> > > > > >
> > > > > > Leading/trailing white space should be allowed. For example echo
> > > > > > appends '\n' unless you know to use -n. It is easier to fix the kernel
> > > > > > than to teach everyone to use -n.
> > > > >
> > > > > Please, NO! echo -n is the right thing to do, and users will eventually learn.
> > > > > We are not going to add such workarounds all over the kernel...
> > > >
> > > > It is not a work around. These are text attributes meant for human
> > > > use. Humans have a hard time cleaning up things they can't see. And
> > > > the failure mode for this is awful, your attribute won't set but
> > > > everything on the screen looks fine.
> > >
> > > The average user has no place poking sysfs. Root should know when
> > > to use -n, as should shell scripts.
> >
> > So the average user never needs to change their console mode? Check
> > out /sys/class/graphics/fb/modes and mode.
>
> That is what we have helper scripts for.
If we are going back to needing helper scripts then I should just
remove the entire sysfs graphics interface and switch back to using
ioctls and a helper app. Of could no one can ever find the helper app
or remember how it works. I thought one of the main reasons behind the
sysfs interface was to eliminate these helper apps.
Without doing whitespace cleanup there is a 100% probability that this
will generate bug reports. I know this for a fact because I am already
getting them.
--
Jon Smirl
jonsmirl@gmail.com
next prev parent reply other threads:[~2005-08-05 18:49 UTC|newest]
Thread overview: 57+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-07-25 4:09 [PATCH] driver core: Add the ability to unbind drivers to devices from userspace Jon Smirl
2005-07-25 4:58 ` Dmitry Torokhov
2005-07-25 14:28 ` Jon Smirl
2005-07-25 14:48 ` Dmitry Torokhov
2005-07-25 16:30 ` Jon Smirl
2005-07-26 0:00 ` Greg KH
2005-07-26 0:28 ` Jon Smirl
2005-07-26 0:30 ` Greg KH
2005-07-26 0:56 ` Jon Smirl
2005-07-26 1:54 ` Greg KH
2005-07-26 3:15 ` Jon Smirl
2005-07-26 3:29 ` Dmitry Torokhov
2005-07-28 2:05 ` Jon Smirl
2005-07-28 3:46 ` Greg KH
2005-07-28 3:59 ` Jon Smirl
2005-07-28 4:05 ` Greg KH
2005-07-28 4:49 ` Jon Smirl
2005-07-28 5:49 ` Greg KH
2005-07-28 7:04 ` Mitchell Blank Jr
2005-07-28 12:54 ` Jon Smirl
2005-07-28 13:09 ` Oliver Neukum
2005-07-28 13:16 ` Paulo Marques
2005-07-28 18:09 ` Mitchell Blank Jr
2005-07-28 19:03 ` Greg KH
2005-07-28 19:57 ` Jon Smirl
2005-07-28 20:22 ` Mitchell Blank Jr
2005-07-28 20:27 ` Jon Smirl
2005-07-29 18:50 ` Jon Smirl
2005-08-06 0:42 ` Greg KH
2005-08-06 3:48 ` Jon Smirl
2005-08-21 22:21 ` Jon Smirl
2005-07-28 21:10 ` Oliver Neukum
2005-07-28 21:12 ` Jon Smirl
2002-01-01 7:53 ` Pavel Machek
2005-08-05 13:32 ` Jon Smirl
2005-08-05 18:01 ` Oliver Neukum
2005-08-05 18:14 ` Jon Smirl
2005-08-05 18:20 ` Oliver Neukum
2005-08-05 18:47 ` Jon Smirl [this message]
2005-08-05 20:07 ` Oliver Neukum
2005-08-05 20:33 ` Jon Smirl
2005-08-06 9:39 ` Oliver Neukum
2005-08-07 18:50 ` Pavel Machek
2005-08-07 18:47 ` Pavel Machek
2005-08-07 20:17 ` Jon Smirl
2005-08-07 21:06 ` Pavel Machek
2005-08-05 22:31 ` David Weinehall
2005-07-28 21:17 ` Oliver Neukum
2005-07-28 12:52 ` Jon Smirl
-- strict thread matches above, loose matches on Subject: below --
2005-06-30 6:04 [PATCH] driver core: add bus_find_device & driver_find_device functions Greg KH
2005-06-30 6:04 ` [PATCH] driver core: Add the ability to unbind drivers to devices from userspace Greg KH
2005-06-30 6:25 ` Dmitry Torokhov
2005-06-30 6:29 ` Greg KH
2005-06-24 5:12 [RFC] bind and unbind drivers from userspace through sysfs Greg KH
2005-06-24 5:14 ` [PATCH] driver core: Add the ability to unbind drivers to devices from userspace Greg KH
2005-06-24 15:57 ` Patrick Mochel
2005-06-25 3:27 ` Greg KH
2005-06-25 4:16 ` Dmitry Torokhov
2005-06-25 9:39 ` Michael Tokarev
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=9e47339105080511474d89ee8a@mail.gmail.com \
--to=jonsmirl@gmail.com \
--cc=dtor_core@ameritech.net \
--cc=greg@kroah.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mitch@sfgoth.com \
--cc=oliver@neukum.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