[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <yq1tvlkkr02.fsf@oracle.com>
Date: Wed, 17 Oct 2018 21:01:01 -0400
From: "Martin K. Petersen" <martin.petersen@...cle.com>
To: Finn Thain <fthain@...egraphics.com.au>
Cc: "Martin K. Petersen" <martin.petersen@...cle.com>,
"Gustavo A. R. Silva" <gustavo@...eddedor.com>,
Adaptec OEM Raid Solutions <aacraid@...rosemi.com>,
"James E.J. Bottomley" <jejb@...ux.vnet.ibm.com>,
linux-scsi@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] scsi: ips: fix missing break in switch
Finn,
>> See the case statements above for another fast exit scenario.
>>
>
> But that's an error path.
Look further down. Several other SCSI commands are completed as NOPs the
same way.
Also, I don't see how the case statement for TUR/INQUIRY would do
anything meaningful in terms of preparing a START STOP UNIT for the
firmware.
--
Martin K. Petersen Oracle Linux Engineering
Powered by blists - more mailing lists