public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Zilvinas Valinskas <zilvinas@gemtek.lt>
To: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: Andrew Morton <akpm@osdl.org>, Linus Torvalds <torvalds@osdl.org>,
	Linux Kernel list <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] Remove suspend() calls from shutdown path
Date: Thu, 04 Aug 2005 19:02:09 +0300	[thread overview]
Message-ID: <1123171329.9251.4.camel@swoop.gemtek.lt> (raw)
In-Reply-To: <1123168844.30257.64.camel@gaston>

On Thu, 2005-08-04 at 17:20 +0200, Benjamin Herrenschmidt wrote:
> On Thu, 2005-08-04 at 15:16 +0300, Zilvinas Valinskas wrote:
> > Hello Ben, Andrew, 
> > 
> > This patch helps me if I disconnect all USB peripherals before shutting
> > down notebook. With connected peripherals (USB mouse, PL2303
> > USB<->serial converter/port) - powering off process stops right after
> > unmounting filesystems but before hda power off ... 
> > 
> > There is a bug report for this too:
> > http://bugzilla.kernel.org/show_bug.cgi?id=4992
> 
> This is unclear.
> 
> I would expect the behaviour you report to happen _without_ this patch,
> that is with current git tree, and I would expect this patch to fix it
> by reverting to the previous 2.6.12 behaviour...
> 
> Ben.

Sys-rq - T: shows device_suspend() is called, perhaps that explains. It
seems that any attempt to suspend either USB hub or device connected to
it results in freeze. :\ Just guessing. 

Anything else should I try ?


hcd_submit_urb
wait_for_completion
default_wake_function
usb_start_wait_urb
timeout_kill
usb_internal_control_msg
usb_control_msg
hub_port_suspend
__usb_suspend_device
locktree
usb_suspend_device
__link_walk_path
device_suspend   <----- still called ?
ohci_reboot
generic_ide_ioctl
activate_task
__group_send_sig_info
sys_kill
block_ioctl
block_ioctl
do_ioctl
vfs_ioctl
get_name
sys_ioctl
sys_enter_esp


  reply	other threads:[~2005-08-04 16:04 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-08-04  9:36 [PATCH] Remove suspend() calls from shutdown path Benjamin Herrenschmidt
2005-08-04 12:16 ` Zilvinas Valinskas
2005-08-04 15:20   ` Benjamin Herrenschmidt
2005-08-04 16:02     ` Zilvinas Valinskas [this message]
2005-08-04 20:51     ` Rafael J. Wysocki
2005-08-04 20:58       ` Andrew Morton
2005-08-04 21:30         ` Rafael J. Wysocki

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=1123171329.9251.4.camel@swoop.gemtek.lt \
    --to=zilvinas@gemtek.lt \
    --cc=akpm@osdl.org \
    --cc=benh@kernel.crashing.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=torvalds@osdl.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox