Re: [PATCH] PCI / PM: Block races between runtime PM and systemsleep

From: Alan Stern
Date: Sat Jun 25 2011 - 22:39:37 EST


On Fri, 24 Jun 2011, Rafael J. Wysocki wrote:

> > > I'm still not clear on why the error handler needs to run at this time.
> >
> > Because SATA ports are suspended with the help of the SCSI error handling
> > mechanism (which Tejun claims is the best way to do that).

> I've carried out this exercise to see how complicated it is going to be
> and it doesn't really seem to be _that_ complicated. The appended patch
> illustrates this, but it hasn't been tested, so caveat emptor.

The patch is straightforward enough. But will it be sufficient?

Suppose a SATA port is already in runtime suspend when the system sleep
starts. Will the error handler be able to do its special job? I don't
know... It may turn out to be necessary for the SATA port to be
runtime resumed somewhere along the line.

Alan Stern

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