[SRU][Wily][Xenial][PATCH 2/2] UBUNTU: SAUCE: (no-up) scsi: storvsc: Filter out storvsc messages CD-ROM medium not present

[Joseph Salisbury]

Thanks for the feedback, Andy.  I'll wiggle the code and resubmit just a single backport.


On July 4, 2016 12:32:40 PM EDT, Andy Whitcroft <apw at canonical.com> wrote:
>On Fri, Jul 01, 2016 at 09:44:14PM -0400, Joseph Salisbury wrote:
>> From: Cathy Avery <cavery at redhat.com>
>> 
>> BugLink: http://bugs.launchpad.net/bugs/1590655
>> 
>> Signed-off-by: Joseph Salisbury <joseph.salisbury at canonical.com>
>> ---
>>  drivers/scsi/storvsc_drv.c | 4 +++-
>>  1 file changed, 3 insertions(+), 1 deletion(-)
>> 
>> diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c
>> index 95462c5..9948745 100644
>> --- a/drivers/scsi/storvsc_drv.c
>> +++ b/drivers/scsi/storvsc_drv.c
>> @@ -963,7 +963,9 @@ static void storvsc_command_completion(struct
>storvsc_cmd_request *cmd_request)
>>  	if (scmnd->result) {
>>  		if (scsi_normalize_sense(scmnd->sense_buffer,
>>  				SCSI_SENSE_BUFFERSIZE, &sense_hdr) &&
>> -		    do_logging(STORVSC_LOGGING_ERROR))
>> +		    !(sense_hdr.sense_key == NOT_READY &&
>> +				 sense_hdr.asc == 0x03A) &&
>> +		    do_logging(STORVSC_LOGGING_ERROR))
>>  			scsi_print_sense_hdr(scmnd->device, "storvsc",
>>  					     &sense_hdr);
>
>Do we really need that foundation patch.  This appears to just add a
>couple of additional clauses to this if which would apply with wiggling
>pretty directly to the original code?
>
>-apw
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20160704/35f834e5/attachment.html>