All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Carlos R. Mafra" <crmafra2@gmail.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: Soeren Sonnenburg <kernel@nn7.de>,
	"Rafael J. Wysocki" <rjw@sisk.pl>,
	linux-kernel@vger.kernel.org, tomas.winkler@intel.com,
	"John W. Linville" <linville@tuxdriver.com>,
	linux-wireless@vger.kernel.org
Subject: Re: Suspend to RAM regression in 2.6.28-rc2 (bisected)
Date: Mon, 27 Oct 2008 20:11:18 +0100	[thread overview]
Message-ID: <20081027191118.GA4058@localhost.aei.mpg.de> (raw)
In-Reply-To: <1225134195.3796.39.camel@johannes.berg>

On Mon 27.Oct'08 at 20:03:15 +0100, Johannes Berg wrote:
> On Mon, 2008-10-27 at 20:00 +0100, Carlos R. Mafra wrote:
> > On Mon 27.Oct'08 at 19:44:42 +0100, Johannes Berg wrote:
> > > On Mon, 2008-10-27 at 19:31 +0100, Johannes Berg wrote:
> > > > On Mon, 2008-10-27 at 19:07 +0100, Soeren Sonnenburg wrote:
> > > > 
> > > > > > Johannes, can you pls have a look?
> > > > 
> > > > I did, and I have no idea. Makes no sense at all.
> > > 
> > > The only thing I can remotely think of is that iwlwifi doesn't like
> > > being called back from within the call that it did to mac80211, which
> > > obviously happens here. But I have no idea, the code as it stands is
> > > correct, just the interaction with iwlwifi's resume seems to be broken.
> > > 
> > > Try this patch instead:
> > 
> > Yep, with this patch it also works!
> 
> > > --- everything.orig/drivers/net/wireless/iwlwifi/iwl-agn.c	2008-10-27 19:44:12.000000000 +0100
> > > +++ everything/drivers/net/wireless/iwlwifi/iwl-agn.c	2008-10-27 19:44:15.000000000 +0100
> > > @@ -2084,7 +2084,6 @@ static void iwl_alive_start(struct iwl_p
> > >  		iwl4965_error_recovery(priv);
> > >  
> > >  	iwl_power_update_mode(priv, 1);
> > > -	ieee80211_notify_mac(priv->hw, IEEE80211_NOTIFY_RE_ASSOC);
> 
> alright, well, if nothing else turns up soon then we should probably put
> this patch in rather than reverting the other one, imho mac80211 is
> doing the right thing there and the driver is calling into it at a point
> where either mac80211 or the driver cannot handle it.
> 
> Do you get any kernel messages output? If you do, could you put messages
> into each line of ieee80211_set_disassoc to see where it hangs?

No messages appear, just a black screen.

But I can use the SysRq keys, and when I umount the
screen shows the message that umount succeed. I also tried SysRq+t but
the messages appear to fast to read.


> 
> johannes



  reply	other threads:[~2008-10-27 19:10 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-27 16:20 Suspend to RAM regression in 2.6.28-rc2 (bisected) Carlos R. Mafra
2008-10-27 17:32 ` Rafael J. Wysocki
2008-10-27 18:07   ` Soeren Sonnenburg
2008-10-27 18:31     ` Johannes Berg
2008-10-27 18:44       ` Johannes Berg
2008-10-27 19:00         ` Carlos R. Mafra
2008-10-27 19:03           ` Johannes Berg
2008-10-27 19:11             ` Carlos R. Mafra [this message]
2008-10-27 19:13               ` Johannes Berg
2008-10-27 20:39                 ` Carlos R. Mafra
2008-10-27 20:51                   ` Rafael J. Wysocki
2008-10-27 20:52                     ` Carlos R. Mafra
2008-10-27 21:07                       ` Rafael J. Wysocki
2008-10-27 22:05                         ` Carlos R. Mafra
2008-10-27 21:07                     ` Tomas Winkler
2008-10-27 22:28                       ` Carlos R. Mafra
2008-10-27 22:40                         ` Tomas Winkler
2008-10-27 22:50                           ` Rafael J. Wysocki
2008-10-27 23:12                             ` Tomas Winkler
2008-10-27 23:23                               ` Harvey Harrison
2008-10-28 15:30                               ` Rafael J. Wysocki
2008-10-28 16:12                                 ` Tomas Winkler
2008-10-28  6:32                       ` Christian Borntraeger
2008-10-27 19:06           ` Jens Axboe
2008-10-27 19:09             ` Johannes Berg
2008-10-27 19:13               ` Jens Axboe
2008-10-27 19:16                 ` Johannes Berg
2008-10-27 19:20                   ` Jens Axboe
2008-10-27 17:39 ` Rafael J. Wysocki
2008-10-27 17:57   ` Carlos R. Mafra
2008-10-27 18:00     ` John W. Linville
2008-10-27 18:16     ` Rafael J. Wysocki
2008-10-27 18:36       ` Carlos R. Mafra
2008-10-27 18:51         ` Carlos R. Mafra
2008-10-27 20:59 ` Christian Borntraeger
2008-10-27 21:06   ` Harvey Harrison

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20081027191118.GA4058@localhost.aei.mpg.de \
    --to=crmafra2@gmail.com \
    --cc=johannes@sipsolutions.net \
    --cc=kernel@nn7.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=rjw@sisk.pl \
    --cc=tomas.winkler@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.