Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757433Ab1FUTII (ORCPT ); Tue, 21 Jun 2011 15:08:08 -0400 Received: from mga03.intel.com ([143.182.124.21]:3993 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757366Ab1FUTIE convert rfc822-to-8bit (ORCPT ); Tue, 21 Jun 2011 15:08:04 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.65,402,1304319600"; d="scan'208";a="16354697" From: "Winkler, Tomas" To: Marc Koschewski , "greg@kroah.com" CC: Arkadiusz Miskiewicz , "Weil, Oren jer" , "linux-kernel@vger.kernel.org" Date: Tue, 21 Jun 2011 22:07:27 +0300 Subject: RE: mei driver breaks suspend on 3.0git Thread-Topic: mei driver breaks suspend on 3.0git Thread-Index: AcwwQqSO3SZsGSleTKeAsGNDmqq6XQAAs4Xg Message-ID: <6F5C1D715B2DA5498A628E6B9C124F0401B87DC7EA@hasmsx504.ger.corp.intel.com> References: <201106121314.26394.a.miskiewicz@gmail.com> <201106122016.33399.a.miskiewicz@gmail.com> <201106122038.36623.a.miskiewicz@gmail.com> <201106122040.21950.a.miskiewicz@gmail.com> <6F5C1D715B2DA5498A628E6B9C124F0401AD318B15@hasmsx504.ger.corp.intel.com> <20110621183954.GA17528@marc.osknowledge.org> In-Reply-To: <20110621183954.GA17528@marc.osknowledge.org> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: acceptlanguage: en-US Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4547 Lines: 122 > -----Original Message----- > From: Marc Koschewski [mailto:marc@osknowledge.org] > Sent: Tuesday, June 21, 2011 9:40 PM > To: Winkler, Tomas > Cc: Arkadiusz Miskiewicz; Weil, Oren jer; linux-kernel@vger.kernel.org > Subject: Re: mei driver breaks suspend on 3.0git > > * Winkler, Tomas [2011-06-12 22:41:59 +0300]: > > I suffer the same. After booting 3.0.0-r4 with that module not being loaded it > suspends, however, it doesn't resume (screen black, hard power-off > needed). The patch was already posted https://lkml.org/lkml/2011/6/13/148 but wasn't picked up for rc4 yet. For now please apply it your by yourself. Greg, will you collect staging fixes for rc5 or shell it be picked up directly by Linus? Thanks Tomas > Regards, > Marc > > > > > > > > -----Original Message----- > > > From: Arkadiusz Miskiewicz [mailto:a.miskiewicz@gmail.com] > > > Sent: Sunday, June 12, 2011 9:40 PM > > > To: Winkler, Tomas > > > Cc: Weil, Oren jer; linux-kernel@vger.kernel.org > > > Subject: Re: mei driver breaks suspend on 3.0git > > > > > > On Sunday 12 of June 2011, Arkadiusz Miskiewicz wrote: > > > > On Sunday 12 of June 2011, Arkadiusz Miskiewicz wrote: > > > > > On Sunday 12 of June 2011, Winkler, Tomas wrote: > > > > > > > -----Original Message----- > > > > > > > From: Weil, Oren jer > > > > > > > Sent: Sunday, June 12, 2011 8:36 PM > > > > > > > To: Arkadiusz Miskiewicz > > > > > > > Cc: Winkler, Tomas; linux-kernel@vger.kernel.org > > > > > > > Subject: RE: mei driver breaks suspend on 3.0git > > > > > > > > > > > > > > >It's thinkpad t400 with > > > > > > > >00:03.0 Communication controller [0780]: Intel Corporation > > > > > > > >Mobile 4 Series Chipset MEI Controller [8086:2a44] (rev 07) > > > > > > > > > > > > > > Can you send us more details? Does it happened in stress? > > > > > > > How do I reproduce it? > > > > > > > > > > No stress. This is my daily use notebook and I simply suspend to > > > > > ram when sitting under X in kde4. > > > > > > > > > > > > Can you send us the debug prints? > > > > > > > > > > Attached. suspend starts, tries to resume devices, MEI suspend > > > > > fails causing whole suspend to be aborted. > > > > > > > > Shouldn't ret > 0 in mei_wd_stop to be "success"? > > > > > > I meant this one: > > > > > > ret = wait_event_interruptible_timeout(dev->wait_stop_wd, > > > dev->wd_stopped, 10 * HZ); > > > > > > > > > And that was possible untested solution that relies on wd being > > > stopped and not anything else. > > > > > > > > > > > diff --git a/drivers/staging/mei/wd.c b/drivers/staging/mei/wd.c > > > > index 2564b03..608245a 100644 > > > > --- a/drivers/staging/mei/wd.c > > > > +++ b/drivers/staging/mei/wd.c > > > > @@ -171,8 +171,10 @@ int mei_wd_stop(struct mei_device *dev, bool > > > preserve) > > > > mutex_lock(&dev->device_lock); > > > > if (!dev->wd_stopped) > > > > dev_dbg(&dev->pdev->dev, "stop wd failed to complete.\n"); > > > > - else > > > > + else { > > > > + ret = 0; > > > > dev_dbg(&dev->pdev->dev, "stop wd complete.\n"); > > > > + } > > > > > > > > if (preserve) > > > > dev->wd_timeout = wd_timeout; > > > > > > > Looks like your patch is correct > > wait_event_interruptible_timeout() returns >0 on success unlike > > wait_even_interruptable() > > > > I wil check all the instances we have and issue a patch. > > Thanks for your effort > > Tomas > > > > --------------------------------------------------------------------- > > Intel Israel (74) Limited > > > > This e-mail and any attachments may contain confidential material for > > the sole use of the intended recipient(s). Any review or distribution > > by others is strictly prohibited. If you are not the intended > > recipient, please contact the sender and delete all copies. > > -- > Marc Koschewski --------------------------------------------------------------------- Intel Israel (74) Limited This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/