All of lore.kernel.org
 help / color / mirror / Atom feed
From: Larry Finger <Larry.Finger@lwfinger.net>
To: Alan Jenkins <alan-jenkins@tuffmail.co.uk>
Cc: htl10@users.sourceforge.net,
	Alan Stern <stern@rowland.harvard.edu>,
	Bob Copeland <me@bobcopeland.com>,
	linux-wireless@vger.kernel.org,
	linux-pm@lists.linux-foundation.org,
	USB list <linux-usb@vger.kernel.org>,
	Herton Ronaldo Krzesinski <herton@mandriva.com.br>
Subject: Re: [linux-pm] rtl8187 usb wifi adaptor causes suspend hang
Date: Wed, 31 Dec 2008 13:33:13 -0600	[thread overview]
Message-ID: <495BC8F9.4040203@lwfinger.net> (raw)
In-Reply-To: <495B82CB.6050205@tuffmail.co.uk>

Alan Jenkins wrote:
> Hin-Tak Leung wrote:
>> --- On Tue, 30/12/08, Larry Finger <Larry.Finger@lwfinger.net> wrote:
>>   
>>> Alan Jenkins wrote:
>>>     
>>>> That won't fix hibernation though.  The disconnect
>>>>       
>>> method can still get
>>>     
>>>> called in resume from hibernation, before the
>>>>       
>>> workqueue gets unfrozen. 
>>>     
>>>> To be honest, I'm far more interested in
>>>>       
>>> suspend-to-disk than
>>>     
>>>> suspend-to-ram.
>>>>       
>>> Does this patch fix your problem? It works here, but I only
>>> did one
>>> test. The patch is for wireless-testing.
>>>
>>> Larry
>>>     
>> My test is that STD works alright, but STR still does not work. I haven't tried STD for a long time - I mostly use STR with SUSPEND_MODULES -, so I have no idea if STD works before the patch or not. With the patch, I can suspend to RAM (removing my SUSPEND_MODULES workaround temporarily), but it would not "defrost - this is similiar to  my failed attempt at fixing this - adding some skeleton _suspend()/_resume() routines can get the driver to STR, but it won't defrost. Without any _suspend()/_resume() routines, it won't even suspend, but just spend ages trying to suspend.
>>
>> I'll post your patch to http://bugzilla.kernel.org/show_bug.cgi?id=11887  as a work-in-progress...
>>
>> Hin-Tak
> 
> Thanks for the patch.  I tested STD too, and it doesn't seem to hang
> anymore.  I even tested removing the adaptor while the system was
> hibernated.  However, after resume the interface is useless.
> 
> I get a kernel error message, which repeats regularly (something like
> every 5 or 10 seconds):
> 
>     phy0: Reset timeout!
> 
> and it doesn't work:
> 
>     # iwlist wlan0 scan
>     wlan0     Interface doesn't support scanning : Network is down
> 
>     # ifconfig wlan0 up
>     SIOCSIFFLAGS: Connection timed out
> 
> I have to re-plug the adaptor, and then it works again.

Thanks for testing. I have some ideas on how to improve the
suspend/resume, but they will require some changes in the structure. I
will also try to determine why my system fails at STR so I can test
both flavors.

Larry

  parent reply	other threads:[~2008-12-31 19:33 UTC|newest]

Thread overview: 57+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-12-30 10:51 rtl8187 usb wifi adaptor causes suspend hang Alan Jenkins
2008-12-30 13:10 ` Alan Jenkins
2008-12-30 13:10 ` Alan Jenkins
2008-12-30 15:45   ` [linux-pm] " Alan Stern
2008-12-30 16:45     ` Bob Copeland
2008-12-30 16:48       ` Alan Jenkins
2008-12-30 16:48       ` [linux-pm] " Alan Jenkins
2008-12-30 17:25       ` Alan Stern
2008-12-30 17:25       ` [linux-pm] " Alan Stern
2008-12-30 17:31         ` Alan Jenkins
2008-12-30 17:31         ` [linux-pm] " Alan Jenkins
2008-12-30 17:52           ` Larry Finger
2008-12-30 18:05             ` Hin-Tak Leung
2008-12-30 18:05             ` [linux-pm] " Hin-Tak Leung
2008-12-30 18:05             ` Oliver Neukum
2008-12-30 18:05             ` [linux-pm] " Oliver Neukum
2008-12-30 18:09               ` Larry Finger
2008-12-30 18:09               ` Larry Finger
2008-12-31  0:59             ` Hin-Tak Leung
2008-12-31  0:59             ` [linux-pm] " Hin-Tak Leung
2008-12-31 14:33               ` Alan Jenkins
2008-12-31 15:57                 ` Bob Copeland
2008-12-31 15:57                 ` [linux-pm] " Bob Copeland
2008-12-31 17:10                   ` Alan Jenkins
2008-12-31 17:10                   ` Alan Jenkins
2008-12-31 17:16                 ` [linux-pm] " Oliver Neukum
2008-12-31 17:16                 ` Oliver Neukum
2008-12-31 19:33                 ` Larry Finger [this message]
2008-12-31 23:00                   ` Larry Finger
2008-12-31 23:00                   ` [linux-pm] " Larry Finger
2008-12-31 23:26                     ` Bob Copeland
2008-12-31 23:26                     ` [linux-pm] " Bob Copeland
2009-01-01  0:16                     ` Alan Stern
2009-01-01  0:16                     ` Alan Stern
2009-01-01  6:06                   ` Hin-Tak Leung
2009-01-01  6:06                   ` [linux-pm] " Hin-Tak Leung
2009-01-01 22:25                     ` Alan Stern
2009-01-01 23:42                       ` Hin-Tak Leung
2009-01-02  2:24                         ` Alan Stern
2009-01-02  2:24                         ` [linux-pm] " Alan Stern
2009-01-01 23:42                       ` Hin-Tak Leung
2009-01-01 22:25                     ` Alan Stern
2008-12-31 19:33                 ` Larry Finger
2009-01-01  6:18                 ` Hin-Tak Leung
2009-01-01  6:18                 ` [linux-pm] " Hin-Tak Leung
2009-01-01 13:07                   ` Alan Jenkins
2009-01-01 13:40                     ` Stefanik Gábor
2009-01-01 13:40                     ` [linux-pm] " Stefanik Gábor
2009-01-01 14:39                       ` Hin-Tak Leung
2009-01-01 16:38                         ` Larry Finger
2009-01-01 16:38                         ` [linux-pm] " Larry Finger
2009-01-01 14:39                       ` Hin-Tak Leung
2009-01-01 13:07                   ` Alan Jenkins
2008-12-31 14:33               ` Alan Jenkins
2008-12-30 17:52           ` Larry Finger
2008-12-30 16:45     ` Bob Copeland
2008-12-30 15:45   ` Alan Stern

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=495BC8F9.4040203@lwfinger.net \
    --to=larry.finger@lwfinger.net \
    --cc=alan-jenkins@tuffmail.co.uk \
    --cc=herton@mandriva.com.br \
    --cc=htl10@users.sourceforge.net \
    --cc=linux-pm@lists.linux-foundation.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=me@bobcopeland.com \
    --cc=stern@rowland.harvard.edu \
    /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.