mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Pavel Machek <pavel@ucw.cz>
To: Oliver Neukum <oliver@neukum.org>
Cc: David Brownell <david-b@pacbell.net>,
	Alan Stern <stern@rowland.harvard.edu>,
	linux-usb-devel@lists.sourceforge.net,
	linux-kernel@vger.kernel.org
Subject: Re: [linux-usb-devel] error to be returned while suspended
Date: Sun, 8 Oct 2006 21:19:37 +0200	[thread overview]
Message-ID: <20061008191937.GE3788@elf.ucw.cz> (raw)
In-Reply-To: <200610071916.27315.oliver@neukum.org>

Hi!

> > > OK, let me state the basics.
> > > 
> > > To get real power savings, we:
> > > - blank the display
> > > - spin down the hard drive
> > > - put the CPU into an ACPI sleep state
> > > 
> > > To do the latter well, we need to make sure there's no DMA. It is
> > > important that less or little DMA will not help. We need no DMA.
> > > So we need to handle the commonest scenarios fully.
> > > 
> > > I dare say that the commonest scenario involving USB is a laptop with
> > > an input device attached. Input devices are for practical purposes always
> > > opened. A simple resume upon open and suspend upon close is useless.
> > 
> > Okay, but you can simply do autosuspend with remote wakeup completely
> > inside input driver. You do ot need it to be controlled from X... at
> > most you need one variable ('autosuspend_inactivity_timeout')
> > controlled from userland.
> > 
> > That's what we already do for hdd spindown... you simply tell disk to
> > aitospindown after X seconds of inactivity.
> 
> The firmware in the drive supplies this function. It's hardly by choice
> that it is made available. The power management functions without
> timeout are also exported. For other power control features like
> cpu frequency considerable effort has been made to export them to
> user space.
> 
> A simple timeout solution has drawbacks.
> 
> - there's no guarantee the user wants wakeup (think laptop on
> crowded table)

If you do not want wakeups (=> do not want any input from that
device), then close that device.

> - you want to suspend immediately when you blank the screen (or switch to
> a text console)

I kind-of understand "when you blank", but I do not think this
mandatory. Why would you want to suspend when switching to text
console? Am I no longer allowed to use gpm?

> - you want to consider all devices' activity. I am not pleased if my mouse
> becomes less responsive just because I used only the keyboard for a
> few minutes. Coordinating this inside the driver is hard as some input
> devices might well be not usb (eg. bluetooth mouse, usb tablet)

Yep, that would be nice; but not at price of /sys/.../power/state like
monstrosity that never ever worked properly.
								Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

  parent reply	other threads:[~2006-10-08 19:19 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-03 11:23 Oliver Neukum
2006-10-03 12:51 ` linux-os (Dick Johnson)
2006-10-03 13:02   ` Oliver Neukum
2006-10-03 14:17     ` linux-os (Dick Johnson)
2006-10-04 11:19 ` Pavel Machek
2006-10-04 16:34   ` Oliver Neukum
2006-10-04 22:44     ` Pavel Machek
2006-10-05  7:07       ` Oliver Neukum
2006-10-05  8:57         ` Pavel Machek
2006-10-05 16:21         ` [linux-usb-devel] " Alan Stern
2006-10-05 16:35           ` Oliver Neukum
2006-10-05 18:24             ` Alan Stern
2006-10-05 18:43               ` Oliver Neukum
2006-10-05 20:48                 ` Alan Stern
2006-10-05 21:25                   ` Oliver Neukum
2006-10-05 21:45                     ` Alan Stern
2006-10-06  7:21                       ` Oliver Neukum
2006-10-06 17:48                         ` Alan Stern
2006-10-06 11:25                       ` Pavel Machek
2006-10-06  2:47                     ` David Brownell
2006-10-06  7:04                       ` Oliver Neukum
2006-10-06 11:27                         ` Pavel Machek
2006-10-06 14:09                           ` Oliver Neukum
2006-10-06 21:10                         ` David Brownell
2006-10-07 10:49                           ` Oliver Neukum
2006-10-07 11:08                             ` Pavel Machek
2006-10-07 17:16                               ` Oliver Neukum
2006-10-08  0:03                                 ` David Brownell
2006-10-08  2:03                                   ` Alan Stern
2006-10-08  7:07                                     ` Oliver Neukum
2006-10-08 14:27                                       ` Alan Stern
2006-10-08 19:36                                     ` Pavel Machek
2006-10-08 19:57                                       ` Oliver Neukum
2006-10-08 21:06                                         ` Pavel Machek
2006-10-08  6:40                                   ` Oliver Neukum
2006-10-09 15:56                                     ` David Brownell
2006-10-08  6:51                                   ` Oliver Neukum
2006-10-08  7:14                                   ` Oliver Neukum
2006-10-08 13:24                                   ` Oliver Neukum
2006-10-08 14:32                                     ` Alan Stern
2006-10-08 19:41                                     ` /sys/.../power/state " Pavel Machek
2006-10-08 19:19                                 ` Pavel Machek [this message]
     [not found]                   ` <200610080838.03488.oliver@neukum.org>
     [not found]                     ` <200610080020.49158.david-b@pacbell.net>
2006-10-08  8:39                       ` Oliver Neukum
2006-10-08 14:16                         ` Dmitry Torokhov
2006-10-06 11:23                 ` Pavel Machek
2006-10-06 11:21             ` Pavel Machek

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=20061008191937.GE3788@elf.ucw.cz \
    --to=pavel@ucw.cz \
    --cc=david-b@pacbell.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb-devel@lists.sourceforge.net \
    --cc=oliver@neukum.org \
    --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

Powered by JetHome