public inbox for linux-pm@vger.kernel.org
 help / color / mirror / Atom feed
From: Johannes Berg <johannes@sipsolutions.net>
To: Pavel Machek <pavel@ucw.cz>
Cc: linux-pm <linux-pm@lists.osdl.org>
Subject: Re: suspend to disk modes, default mode
Date: Tue, 20 Mar 2007 10:44:07 +0100	[thread overview]
Message-ID: <1174383848.3770.8.camel@johannes.berg> (raw)
In-Reply-To: <20070320093322.GE3922@elf.ucw.cz>


[-- Attachment #1.1: Type: text/plain, Size: 1016 bytes --]

Hi,

> > I also used to think this was for implementing suspend-to-both, is that
> > wrong? In any case, I can't live with it being platform by default
> 
> No, it is to get ACPI working after resume.

Yeah, I noticed later on. Sorry for the misunderstanding.

> > Maybe we just need a pm_disk_modes bitmap and a default_pm_disk_mode
> > member in struct pm_ops instead? What's firmware vs. platform anyway?
> 
> bitmap would be nice. Firmware used to mean "let the bios do the whole
> suspend", platform means "linux does the suspend, but we tell bios we
> are doing it", shutdown means "linux does the suspend, does not tell
> anyone".

Well, actually, "let the bios do the whole suspend" should be
implemented by a new PM_SUSPEND_FIRMWAREDISK mode that can be used
instead of "disk" and is invoked through pm_ops directly w/o hitting the
disk.c code. That would be the right way to invoke such a mode, however,
currently, no platforms even have such a mode as far as I can tell.

johannes

[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 190 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]



  reply	other threads:[~2007-03-20  9:44 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-20  0:47 suspend to disk modes, default mode Johannes Berg
2007-03-20  0:51 ` Johannes Berg
2007-03-20  1:03 ` Johannes Berg
2007-03-20  9:33 ` Pavel Machek
2007-03-20  9:44   ` Johannes Berg [this message]
2007-03-20 10:20     ` 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=1174383848.3770.8.camel@johannes.berg \
    --to=johannes@sipsolutions.net \
    --cc=linux-pm@lists.osdl.org \
    --cc=pavel@ucw.cz \
    /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