On 10/3/23 08:39, Phillip Susi wrote: > > I went to test these patches tonight and it looks like Linus already > merged them ( or mostly? ). I enabled runtime pm on the scsi target and > the ata port, and the disk spins down and the port does too. > > I noticed though, that when entering system suspend, a disk that has > already been runtime suspended is resumed only to immediately be > suspended again before the system suspend. That shouldn't happen should > it? Indeed. Will look at this. Geert also reported seeing an issue with resume (one time only, so I suspect there is still a race with libata-eh). So looks like something is still missing. -- Damien Le Moal Western Digital Research
