From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vitaly Wool Subject: Re: subtle pm_runtime_put_sync race and sdio functions Date: Sat, 25 Dec 2010 22:50:52 +0100 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-pm-bounces@lists.linux-foundation.org Errors-To: linux-pm-bounces@lists.linux-foundation.org To: Ohad Ben-Cohen Cc: linux-mmc@vger.kernel.org, linux-wireless@vger.kernel.org, linux-pm@lists.linux-foundation.org, Johannes Berg , Ido Yariv List-Id: linux-pm@vger.kernel.org Hi Ohad, On Sat, Dec 25, 2010 at 9:58 PM, Ohad Ben-Cohen wrote: > On Sat, Dec 25, 2010 at 6:21 PM, Alan Stern w= rote: >> Right. =A0You may or may not realize it, but this requirement means that >> the driver _must_ bypass runtime PM sometimes. > > http://www.spinics.net/lists/linux-pm/msg22864.html > >> Now you've lost me. =A0Which of the driver's handlers are you talking >> about? > > The driver's handler, which is called by mac80211, and is responsible > to power off the device. > The _same_ handler is being called either during runtime or during a > system transition to suspend > >> What races? > > Driver thinks power is off and device is now fully reset, but it's isn't = really maybe it's worth starting off with the description of chip power states and how they are mapped to runtime PM and static PM? Most of the WLAN chips have some very low power modes, but you're talking about _complete_ shutdown as a suspended state for both runtime PM and static PM, is that correct? Thanks, Vitaly