On 2021/6/9 20:09, John Garry wrote:
On 09/06/2021 13:01, Yufen Yu wrote:
Do we also need to modify aix79xx in a similar fashion?
There is no aix79xx in directory drivers/scsi. I guess you mean
aic79xxx? But it seems not need to modify.
So if you think that this HBA does not support SATA, then it would be good to mention it in the commit log.
Maybe I didn't describe it clearly. I am not mean that aic79xxx dose not support
SATA. But it is not libsas driver, so I think we don't need to modify it here.
Some more comments:
On 09/06/2021 10:36, Yufen Yu wrote:
> We found that offline a ata device on hisi sas control and then
/s/ata/SATA/
> scanning the host can probe 255 not exist devices into system.
"can probe 255 non-existant"
> It can be reproduced easily as following:
>
> Some ata devices on hisi sas v3 control:
I don't know what this means, so please drop it.
> [root@localhost ~]# lsscsi
> [2:0:0:0] disk ATA Samsung SSD 860 2B6Q /dev/sda
> [2:0:1:0] disk ATA WDC WD2003FYYS-3 1D01 /dev/sdb
> [2:0:2:0] disk SEAGATE ST600MM0006 B001 /dev/sdc
>
> 1) echo "offline" > /sys/block/sdb/device/state
> 2) echo "- - -" > /sys/class/scsi_host/host2/scan
>
> Then, we can see another 255 not exist devices in system:
use "non-existant"
Thanks to point out these error. I will fix it in next version.
Thanks,
Yufen
