On 7/23/24 09:49, Martin K. Petersen wrote: > > Yohan, > >> Add SPC version verification to avoid unnecessary inquiry command > > Are you experiencing an actual error condition as a result of this > INQUIRY operation? > > Devices often adopt new protocol features prior to the spec being > finalized. Therefore we generally use INQUIRY to discover capabilities > rather than relying on the reported spec compliance. > >> + /* Support for CPR was defined in SPC-5. */ >> + if (sdev->scsi_level < SCSI_SPC_5) >> + return; >> + > > I'm OK with the change but I'll defer to Damien as to whether this is an > acceptable heuristic. If this is solving an issue with an ATA device managed by libata, we can handle that in libata. Otherwise, if this is an ATA device connected to a SAS HBA or a SAS drive, then the issue could be with the HBA. I missed this patch so I am not aware of the actual issue this is trying to solve... -- Damien Le Moal Western Digital Research
