Re: [PATCH] scsi_error: Fix language abuse.
From: linux-os (Dick Johnson)
Date: Fri Feb 08 2008 - 11:07:55 EST
The correct word should be "invalid," in spite of
the fact that the SCSI committee used invalid syntax.
Alan is right. There is nothing illegal in the kernel
and if there is, it must be removed as soon as it
is discovered!
On Fri, 8 Feb 2008, James Bottomley wrote:
>
> On Fri, 2008-02-08 at 15:32 +0000, Alan Cox wrote:
>> The word "illegal" has a precise dictionary meaning of "prohibited by
>> law". The error messages are therefore incorrect as so far nobody has
>> made SCSI violations a criminal offence.
>
> Um, I'm really reluctant to do this without an incredibly good reason.
> Illegal is a defined term under the SCSI standards, and the messages you
> are changing actually appear with the word Illegal in the actual
> ASC/ASCQ definition. If I accept this patch, you'll no longer be able
> to look the messages up in the relevant standard:
>
> http://www.t10.org/ftp/t10/drafts/spc3/spc3r23.pdf
>
> By a simple text search.
>
> I don't think the pedantry is worth the confusion ...
>
> James
>
Cheers,
Dick Johnson
Penguin : Linux version 2.6.22.1 on an i686 machine (5588.29 BogoMips).
My book : http://www.AbominableFirebug.com/
_
****************************************************************
The information transmitted in this message is confidential and may be privileged. Any review, retransmission, dissemination, or other use of this information by persons or entities other than the intended recipient is prohibited. If you are not the intended recipient, please notify Analogic Corporation immediately - by replying to this message or by sending an email to DeliveryErrors@xxxxxxxxxxxx - and destroy all copies of this information, including any attachments, without reading or disclosing them.
Thank you.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/