Re: does scsi_io_completion need to dump sense data for ata pass through (ck_cond = 1) ?

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

 



Krzysztof Błaszkowski wrote:
Hello Eric,

I think it is not necessary and current behavior of kernel like 2.6.31 confuses QA people. They need explanation that message like below:

sd 5:0:1:0: [sdb] Sense Key : Recovered Error [current] [descriptor]
Descriptor sense data with sense descriptors (in hex):
        72 01 00 1d 00 00 00 0e 09 0c 00 00 00 00 00 00
        00 00 00 00 00 50
sd 5:0:1:0: [sdb] Add. Sense: ATA pass through information available

stands for that sense data contains ata registers file and the command was executed properly.

Here is additional condition i used in scsi_io_completion() which suppresses this message. Is it okay ?

Close but I would like to see a filter on the additional
sense code ATA PASS-THROUGH INFORMATION AVAILABLE [0x0, 0x1d].
There is nothing in SAT that precludes the return of a RECOVERED
ERROR sense code for a read on a SATA disk that was recovered
after some extra work (for example).
With that filter in place there is no need to filter on the command.

IOWs just filter on sense code RECOVERED ERROR and additional
sense code ATA PASS-THROUGH INFORMATION AVAILABLE. SAT and SAT-2
compliant implementations should then dump less to the log.

Doug Gilbert
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux