Re: [PATCH v6 1/7] scsi: sr: support runtime pm for ODD

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

 



On 09/06/2012 11:06 PM, Alan Stern wrote:
On Thu, 6 Sep 2012, Aaron Lu wrote:

That's why we have an autosuspend delay.  Although for some reason the
SCSI subsystem doesn't use it currently...  We need to add a call to
pm_runtime_use_autosuspend() in scsi_sysfs_add_sdev().  Likewise, the
pm_schedule_suspend() call in scsi_runtime_idle() should be changed to
pm_runtime_autosuspend().  And there should be calls to
pm_runtime_set_autosuspend_delay() in the sd and sr drivers.

I tried to use autosuspend when preparing the patch, but the fact that
the devices will be polled every 2 seconds make it impossible to enter
suspend state if the autosuspend delay is larger than that.

You can always increase the polling interval.
But in the long run that wouldn't be a good solution.

Agree.

What I'd really like is a way to do the status polling without having
it reset the idle timer.

I like this, I'll try to see if this can be done.

If we idle the device immediately, we would suspend/resume the ODD every
2 seconds, which may not be a good idea.

And we can't increate the polling interval that much, like to the level
of minutes, and if we can't put the device into suspend state long
enough, it may not worth the effort.

What do you think?

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


[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux