linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: [RFC/PATCH] Deferred disk spinup during system resume
       [not found]     ` <AANLkTikEusD_CvkOERsr_wr7T70g1kuU=JnBJm5ur5Wg@mail.gmail.com>
@ 2011-01-19  7:05       ` Jeff Garzik
  2011-01-19 20:29         ` Maksim Rayskiy
  0 siblings, 1 reply; 3+ messages in thread
From: Jeff Garzik @ 2011-01-19  7:05 UTC (permalink / raw)
  To: Maksim Rayskiy; +Cc: Tejun Heo, linux-scsi, Linux IDE mailing list, Jens Axboe

On 01/12/2011 03:01 PM, Maksim Rayskiy wrote:
>>
>> The bottom line is that this patch simply wants to trigger an ATA command,
>> and return immediately, discarding the command results.  I'm not even sure a
>> "run this command in background, and discard results" facility requires the
>> EH.
>>
>
> This is why I was asking if using a workqueue instead of EH might be a
> better idea.
> EH looks like an overkill here.

A kernel modification might not even be needed.

Have you tried simply issuing READ VERIFY via bsg, and not caring if it 
completes?  bsg should be able to handle an app submitting a command, 
but never checking the 'done' list, right?  A simple shell app could execute

	write(bsg_fd, ... SCSI READ VERIFY command ...)
	exit(0)

to avoid waiting for READ VERIFY command completion, I would hope.

	Jeff




^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [RFC/PATCH] Deferred disk spinup during system resume
  2011-01-19  7:05       ` [RFC/PATCH] Deferred disk spinup during system resume Jeff Garzik
@ 2011-01-19 20:29         ` Maksim Rayskiy
  2011-01-20  6:01           ` Jeff Garzik
  0 siblings, 1 reply; 3+ messages in thread
From: Maksim Rayskiy @ 2011-01-19 20:29 UTC (permalink / raw)
  To: Jeff Garzik; +Cc: Tejun Heo, linux-scsi, Linux IDE mailing list, Jens Axboe

> A kernel modification might not even be needed.
>
> Have you tried simply issuing READ VERIFY via bsg, and not caring if it
> completes?  bsg should be able to handle an app submitting a command, but
> never checking the 'done' list, right?  A simple shell app could execute
>
>        write(bsg_fd, ... SCSI READ VERIFY command ...)
>        exit(0)
>
> to avoid waiting for READ VERIFY command completion, I would hope.
>
>        Jeff
>

My question was how to speed up system resume when he the spinup
request is coming from sd_resume(). For shell method to work I would
have to ignore this request entirely and do READ VERIFY when system is
fully restored.
Can it be done without kernel modification? Tejun mentioned using
manage_start_stop flag but had some reservations against it.

Maksim.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [RFC/PATCH] Deferred disk spinup during system resume
  2011-01-19 20:29         ` Maksim Rayskiy
@ 2011-01-20  6:01           ` Jeff Garzik
  0 siblings, 0 replies; 3+ messages in thread
From: Jeff Garzik @ 2011-01-20  6:01 UTC (permalink / raw)
  To: Maksim Rayskiy; +Cc: Tejun Heo, linux-scsi, Linux IDE mailing list, Jens Axboe

On 01/19/2011 03:29 PM, Maksim Rayskiy wrote:
>> A kernel modification might not even be needed.
>>
>> Have you tried simply issuing READ VERIFY via bsg, and not caring if it
>> completes?  bsg should be able to handle an app submitting a command, but
>> never checking the 'done' list, right?  A simple shell app could execute
>>
>>         write(bsg_fd, ... SCSI READ VERIFY command ...)
>>         exit(0)
>>
>> to avoid waiting for READ VERIFY command completion, I would hope.
>>
>>         Jeff
>>
>
> My question was how to speed up system resume when he the spinup
> request is coming from sd_resume(). For shell method to work I would
> have to ignore this request entirely and do READ VERIFY when system is
> fully restored.
> Can it be done without kernel modification? Tejun mentioned using
> manage_start_stop flag but had some reservations against it.

Oh, if we're talking about sd_resume(), SCSI definitely has an internal 
mechanism to fire off a request, without blocking and waiting for a 
response.  It does sound like a kernel modification, but it should be 
straightforward via existing APIs.

	Jeff




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-01-20  6:01 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <1294795457-9006-1-git-send-email-maksim.rayskiy@gmail.com>
     [not found] ` <20110112112142.GA9610@mtj.dyndns.org>
     [not found]   ` <4D2DF45B.5010101@pobox.com>
     [not found]     ` <AANLkTikEusD_CvkOERsr_wr7T70g1kuU=JnBJm5ur5Wg@mail.gmail.com>
2011-01-19  7:05       ` [RFC/PATCH] Deferred disk spinup during system resume Jeff Garzik
2011-01-19 20:29         ` Maksim Rayskiy
2011-01-20  6:01           ` Jeff Garzik

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).