Re: [linux-pm] subtle pm_runtime_put_sync race and sdio functions

2010-12-18 Thread Ohad Ben-Cohen
Hi Alan, Sorry for the belated response, On Sat, Dec 11, 2010 at 4:50 PM, Alan Stern st...@rowland.harvard.edu wrote: Think of a device which you have no way to reset other than powering it down and up again. If that device is managed by runtime PM, the only way to do that is by using

Re: [linux-pm] subtle pm_runtime_put_sync race and sdio functions

2010-12-18 Thread Ohad Ben-Cohen
On Sat, Dec 18, 2010 at 5:07 PM, Rafael J. Wysocki r...@sisk.pl wrote: On Saturday, December 18, 2010, Ohad Ben-Cohen wrote: On Sat, Dec 11, 2010 at 4:50 PM, Alan Stern st...@rowland.harvard.edu wrote: Think of a device which you have no way to reset other than powering it down and up

Re: [linux-pm] subtle pm_runtime_put_sync race and sdio functions

2010-12-18 Thread Johannes Berg
On Sat, 2010-12-18 at 18:00 +0200, Ohad Ben-Cohen wrote: That's where the problem is. If there's a difference, from the driver's point of view, between suspend and some other operation, there should be a way to tell the driver what case it actually is dealing with. Yes, the problem will

Re: 2.36.1 sdhci R5C822 regression

2010-12-18 Thread Bringfried Stecklum
I just tried with the irqpoll option which prevents the kernel oops indeed (unlike without the patch) but I still get the timeout waiting for hardware interrupt message. The number of irqs seems to be exceedingly high, e.g. cat /proc/interrupts CPU0 CPU1 0: 20239

Re: [linux-pm] subtle pm_runtime_put_sync race and sdio functions

2010-12-18 Thread Alan Stern
On Sat, 18 Dec 2010, Johannes Berg wrote: Sounds to me like the difference isn't really in the driver, but the core PM subsystem. Why does it care when powering off a device whether it's during suspend, or during runtime? The two operations are quite different. One involves preparing the

Re: [linux-pm] subtle pm_runtime_put_sync race and sdio functions

2010-12-18 Thread Alan Stern
On Sat, 18 Dec 2010, Ohad Ben-Cohen wrote: Sounds to me like the difference isn't really in the driver, but the core PM subsystem. Why does it care when powering off a device whether it's during suspend, or during runtime? Agree. If we can add a dev_pm_info bit, that would allow using

Re: [linux-pm] subtle pm_runtime_put_sync race and sdio functions

2010-12-18 Thread Rafael J. Wysocki
On Saturday, December 18, 2010, Ohad Ben-Cohen wrote: On Sat, Dec 18, 2010 at 5:07 PM, Rafael J. Wysocki r...@sisk.pl wrote: On Saturday, December 18, 2010, Ohad Ben-Cohen wrote: On Sat, Dec 11, 2010 at 4:50 PM, Alan Stern st...@rowland.harvard.edu wrote: Think of a device which you

Re: [linux-pm] subtle pm_runtime_put_sync race and sdio functions

2010-12-18 Thread Ohad Ben-Cohen
On Sun, Dec 19, 2010 at 12:47 AM, Rafael J. Wysocki r...@sisk.pl wrote: On Saturday, December 18, 2010, Ohad Ben-Cohen wrote: On Sat, Dec 18, 2010 at 5:07 PM, Rafael J. Wysocki r...@sisk.pl wrote: On Saturday, December 18, 2010, Ohad Ben-Cohen wrote: On Sat, Dec 11, 2010 at 4:50 PM, Alan