[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 1/3] scsi/utils: Rename 'illegal request' -> 'invalid requ

From: Eric Blake
Subject: Re: [PATCH v2 1/3] scsi/utils: Rename 'illegal request' -> 'invalid request'
Date: Wed, 20 Jan 2021 10:21:53 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.6.0

On 1/20/21 10:15 AM, Gerd Hoffmann wrote:
> On Wed, Jan 20, 2021 at 04:35:20PM +0100, Philippe Mathieu-Daudé wrote:
>> While these requests are invalid, they are not breaking any laws :)
>> Rename 'illegal request' as 'invalid request'.
> Given the scsi spec calls this "illegal request" I think it makes sense
> to *not* change this and stay consistent with the spec for better
> grep-ability.  Note the variables are already renamed, I expect the
> comments have been left as-is intentionally ...

Consistency with the spec and grep-ability takes precedence over any pet
peeves of mine.  I'm fine if we disregard this patch.

Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org

reply via email to

[Prev in Thread] Current Thread [Next in Thread]