public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: David Brownell <david-b@pacbell.net>
To: Mattia Dongili <malattia@linux.it>
Cc: Alan Stern <stern@rowland.harvard.edu>,
	Zhang Rui <rui.zhang@intel.com>,
	linux-acpi@vger.kernel.org, linux-pm@lists.linux-foundation.org,
	akpm@linux-foundation.org
Subject: Re: [linux-pm] Re: [2.6.22-rc1-mm1] vaio laptop (SZ72B) immediately resumes after STR
Date: Fri, 22 Jun 2007 02:43:55 -0700	[thread overview]
Message-ID: <200706220243.56056.david-b@pacbell.net> (raw)
In-Reply-To: <20070622092258.GC25100@inferi.kami.home>


> > > Just to simplify, in this situation, if I had an usb mouse attached to
> > > this usb controller and removed the mouse while suspended, would the
> > > laptop wakeup?
> > 
> > If the USB controller was enabled for wakeup and was working correctly,
> > it would wake up the laptop.  USB wakeup events are defined as: new
> > connection, disconnection, overcurrent, plus any wakeup requests
> > received from devices on the USB bus.
> > 
> > > If so that's not what I most probably want. :)

You have a workaround in hand to achieve what you want.


> > Maybe we should change the kernel so that the default wakeup setting 
> > for USB controllers is disabled. 

I'd rather not go that route.  It may well be that there's
something odd in the current handling of wakeup events with
EHCI.  I've not personally touched that particular code in
several years now, and a lot of code round it has changed
since then.  Plus of course, problems that don't reproduce
on my hardware can't get much attention from me ... there's
something like a factor of ten or more in terms of effort
to do that sort of remote debugging.


> > Then people would have to enable  
> > wakeup explicitly if they wanted to be able to start up their system by 
> > typing on a USB keyboard, for instance.
> 
> Don't know, I guess this could be easy automated in userspace actually.
> But if connect/disconnect are wakeup events my vote goes to default to
> disabled for usb controllers and manage selective enable with some nice
> userspace tool. Not that my opinion counts that much :)

A userspace tool would certainly be doable.  That's part of why
the sysfs wakeup attributes exist:  to cope with hardware oddities
the kernel doesn't know about, or to cope with user preferences.

- Dave


  reply	other threads:[~2007-06-22  9:43 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-18  7:15 [2.6.22-rc1-mm1] vaio laptop (SZ72B) immediately resumes after STR Mattia Dongili
2007-05-18  7:22 ` Andrew Morton
2007-05-19  6:48   ` Mattia Dongili
2007-05-19  7:04     ` Andrew Morton
2007-05-19 15:11       ` 2.6.22-rc1 regression: tifm prevents suspending [was: Re: [2.6.22-rc1-mm1] vaio laptop (SZ72B) immediately resumes after STR] Mattia Dongili
2007-05-19 16:43         ` Mattia Dongili
2007-05-19 17:17         ` Michal Piotrowski
2007-05-19 18:29           ` Rafael J. Wysocki
2007-05-19 20:50             ` Michal Piotrowski
2007-05-20  6:14       ` [2.6.22-rc1-mm1] vaio laptop (SZ72B) immediately resumes after STR Mattia Dongili
2007-05-20  6:47         ` Andrew Morton
2007-05-20  6:59           ` Mattia Dongili
2007-05-20 18:38         ` David Brownell
2007-05-21  0:41           ` Mattia Dongili
2007-05-21  1:22             ` David Brownell
2007-05-21  2:05               ` Mattia Dongili
2007-06-19  8:57         ` Zhang Rui
2007-06-19  9:30           ` Mattia Dongili
2007-06-19  9:55             ` Zhang Rui
2007-06-19 11:26               ` Rafael J. Wysocki
2007-06-20  1:52                 ` Zhang Rui
2007-06-20 21:39                   ` Rafael J. Wysocki
2007-06-22 10:08                     ` David Brownell
2007-06-22 14:33                       ` [linux-pm] " Alan Stern
2007-06-19 10:09           ` Mattia Dongili
2007-06-19 14:20             ` Alan Stern
2007-06-20 14:45               ` [linux-pm] " Mattia Dongili
2007-06-20 16:33                 ` Alan Stern
2007-06-21  6:42                   ` Mattia Dongili
2007-06-21  7:28                     ` Zhang Rui
2007-06-21  9:22                       ` Mattia Dongili
2007-06-21 15:45                     ` Alan Stern
2007-06-22  9:22                       ` Mattia Dongili
2007-06-22  9:43                         ` David Brownell [this message]
2007-06-22 14:07                           ` Alan Stern
2007-06-22 10:02                 ` David Brownell
2007-06-22 14:22                   ` 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=200706220243.56056.david-b@pacbell.net \
    --to=david-b@pacbell.net \
    --cc=akpm@linux-foundation.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-pm@lists.linux-foundation.org \
    --cc=malattia@linux.it \
    --cc=rui.zhang@intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox