All the same checks are preserved, including the manage_start_stop check:

+       if (!sdkp->device->manage_start_stop)
+               goto error;

That function is just a simplified version of sd_start_stop_device but with 
only the start functionality used.

-----Original Message-----
From: Oliver Neukum [mailto:[email protected]] 
Sent: Saturday, August 24, 2013 1:21 AM
To: Brandt, Todd E
Cc: [email protected]; [email protected]; [email protected]; 
[email protected]; [email protected]
Subject: Re: [PATCH v3 2/2] Hard disk S3 resume time optimization

On Fri, 2013-08-23 at 14:43 -0700, Brandt, Todd E wrote:
> This is v3 of the non-blocking S3 resume patch. It's been broken into
> two pieces, this part is for the scsi subsystem. I've addressed Alan 
> Stern's comments in particular by reformatting the call to conform
> to the proper style guidelines.

Hi,

you are changing the generic resume for all sd disks.
In particular, you are dropping the check of
sdkp->device->manage_start_stop
What happens if you use this patch on a real SCSI disk?

        Regards
                Oliver


N�����r��y����b�X��ǧv�^�)޺{.n�+����{���"�{ay�ʇڙ�,j��f���h���z��w���
���j:+v���w�j�m��������zZ+�����ݢj"��!�i

Reply via email to