Re: PATCH: cdrecord: avoiding scsi device numbering for ide devices

Albert Cahalan
2004-08-09T22:53:56Z

Alan Cox writes:
> On Llu, 2004-08-09 at 15:12, Joerg Schilling wrote:
>> [Alan Cox writes]

>>> Linux has capabilities, ACLs and SELinux rulesets which
>>> can also be used to manage this. I can give the cd burner
>>> a role that permits it certain things.
>>
>> If you are right, why then is SuSE removing the warnings
>> in cdrecord that are there to tell the user that cdrecord
>> is running with insufficient privilleges?
>
> You'd have to ask them. Probably for the reason that most vendors
> remove a lot of the other weird warnings - it confuses end users.

Oh dear my. I'm going to, at least partially, agree with Joerg.
Pigs are cleared for take-off.

The warning about "cdrecord dev=/dev/hdc" is truly crap.
The same goes for "unsettled issues with Linux-2.5". The
other warnings are sane though, and should not be removed.

If the warnings are confusing, they need to be re-worded.
When you remove warnings, users get hurt. The users do
things that may well fail, and then have no clue why things
are failing.

Shall we rip the printk() calls out of the kernel? Many
of them are weird. They might confuse the end users.

Joerg:
"WARNING: Cannot do mlockall(2).\n"
"WARNING: This causes a high risk for buffer underruns.\n"
Fixed:
"Warning: You don't have permission to lock memory.\n"
" If the computer is not idle, the CD may be ruined.\n"

Joerg:
"WARNING: Cannot set priority class parameters priocntl(PC_SETPARMS)\n"
"WARNING: This causes a high risk for buffer underruns.\n"
Fixed:
"Warning: You don't have permission to hog the CPU.\n"
" If the computer is not idle, the CD may be ruined.\n"