Re: [PATCH] driver core: Add the ability to unbind drivers to devices from userspace

Oliver Neukum
2005-08-05T18:15:50Z

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.

Regards
Oliver