Re: [RFC] Implementation of SCSI dynamic power management

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

 



Am Dienstag, 8. Januar 2008 16:12:52 schrieb Alan Stern:
> On Tue, 8 Jan 2008, Oliver Neukum wrote:
> 
> > Am Montag, 7. Januar 2008 20:42:23 schrieb Alan Stern:
> > >  /**
> > > + *     autoresume - perform dynamic (runtime) host resume
> > > + *     @shp: host to resume
> > > + *
> > > + *     Resume (return to an operational power level) the specified host.
> > > + *     Return 0 if the resume was successful, otherwise a negative
> > > + *     error code.
> > > + *
> > > + *     Locks: struct Scsi_Host::pm_mutex held throughout the call.
> > > + *
> > > + *     Calling context: process
> > > + *
> > > + *     Notes: If the host is not currently suspended, this method does
> > > + *     need to do anything.
> > > + *
> > > + *     Optionally defined in: LLD
> > > + **/
> > > +    int autoresume(struct Scsi_Host *shp)
> > > +
> > 
> > This seems to be a bit misleading. It seems to me that you must cancel
> > any outstanding request to autosuspend from other layers.
> 
> How about something more like this:
> 
> *	Resume (return to an operational power level) the specified host,
> *	and prevent autosuspends from other software layers until the
> *	template autosuspend method has been called again.
> *	Return 0 if the resume was successful, otherwise a negative
> *	error code.

Much better.

	Regards
		Oliver


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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux