On 07/21/2016 07:58 AM, Christoph Hellwig wrote:
>> +++ b/block/blk-core.c
>> @@ -2462,9 +2462,13 @@ struct request *blk_peek_request(struct request_queue 
>> *q)
>>  
>>                      rq = NULL;
>>                      break;
>> -            } else if (ret == BLKPREP_KILL || ret == BLKPREP_INVALID) {
>> +            } else if (ret == BLKPREP_KILL || ret == BLKPREP_INVALID ||
>> +                       ret == BLKPREP_DONE) {
>>                      int err = (ret == BLKPREP_INVALID) ? -EREMOTEIO : -EIO;
>>  
>> +                    if (ret == BLKPREP_DONE)
>> +                            err = 0;
>> +
> 
> Please just use a proper switch statement for the return values.
> 
Ok.

Cheers,

Hannes
-- 
Dr. Hannes Reinecke                Teamlead Storage & Networking
h...@suse.de                                   +49 911 74053 688
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton
HRB 21284 (AG Nürnberg)
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to