All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nigel Cunningham <nigel@nigel.suspend2.net>
To: "Rafael J. Wysocki" <rjw@sisk.pl>
Cc: nigel@suspend2.net, Matthew Garrett <mjg59@srcf.ucam.org>,
	David Brownell <david-b@pacbell.net>,
	linux-pm@lists.linux-foundation.org,
	Marcelo Tosatti <marcelo@kvack.org>,
	linux-acpi@vger.kernel.org, Richard Hughes <hughsient@gmail.com>,
	rtc-linux@googlegroups.com
Subject: Re: [linux-pm] Re: Hibernate after alarm wakes from STR
Date: Wed, 11 Jul 2007 20:14:23 +1000	[thread overview]
Message-ID: <200707112014.28958.nigel@nigel.suspend2.net> (raw)
In-Reply-To: <200707111209.05163.rjw@sisk.pl>

[-- Attachment #1: Type: text/plain, Size: 3204 bytes --]

Hi.

On Wednesday 11 July 2007 20:09:04 Rafael J. Wysocki wrote:
> On Wednesday, 11 July 2007 05:14, Nigel Cunningham wrote:
> > On Wednesday 11 July 2007 11:59:48 Matthew Garrett wrote:
> > > On Wed, Jul 11, 2007 at 11:39:37AM +1000, Nigel Cunningham wrote:
> > > 
> > > > Yeah, that is a bit confusing. At the moment, I'm doing the suspend to 
ram 
> > > > platform dependent preparation and cleanup in this scenario. That's 
> > > > definitely the right thing to do in the case where we write an image, 
then 
> > > > suspend to ram, wake and continue working without running running out 
of 
> > > > battery (writing the image is redundant in that case). Where we end up 
> > > > properly powering down after suspending to ram, I believe we don't run 
the 
> > > > pm_ops->finish after doing the atomic restore when resuming the image.
> > > 
> > > I'm not convinced this can work terribly well. It's not unlikely that 
> > > hardware will need different state stored over different types of 
> > > suspend. Can you separate out the saving of kernel memory and userspace 
> > > memory, then resume/suspend/save the new kernel state without touching 
> > > the userspace state?
> > 
> > Yeah, we could redo and resave the atomic copy, but it doesn't seem to be 
> > necessary at the moment; it has been working reliably, regardless of which 
> > combination of events occurs. If/when I come across a case where we have 
> > problems, I'll give resaving the atomic copy a go.
> > 
> > (Thinks some more). Ah, I think we're already doing the right thing, if 
I'm 
> > recalling the order of actions right. If I'm remembering correctly, prior 
to 
> > the atomic copy, we do hibernation prep, then after the atomic copy, 
> > hibernation cleanup. Then, if suspending to ram, we do the 
prep/enter/cleanup 
> > after the image has finished writing. If we lose power from suspend to 
ram, 
> > it doesn't matter because we're just doing a normal resume then, with the 
> > hibernation cleanup post atomic restore machine the prep that was done 
prior 
> > to the atomic copy.
> > 
> > To summarise:
> > 
> > Hibernate + STR + full wake.
> >  Hibernation prep
> >  (Atomic copy)
> >  Hibernation cleanup
> >  STR prep
> >  STR enter
> >  STR cleanup		
> >  Remove hibernation image
> > 
> > Hibernate + STR + poweroff + hibernate resume:
> > 
> >  Hibernation prep
> >  (Atomic copy)
> >  Hibernation cleanup
> >  STR prep
> >  STR enter
> >  <power out> or <STR wake + power off> (STR prep/enter no longer matters)
> >  (Fresh boot)
> >  Atomic restore
> >  Hibernation cleanup (matching prep at start)
> >  Remove hibernation image
> 
> Yes, I think that this is the right ordering, but for some graphics adapters
> we need to do some tricks from the user space before 'STR prep' and after
> 'STR cleanup', which is theoretically possible with uswsusp.

Yeah. I haven't done it yet, but intend to implement that using userui.

Regards,

Nigel
-- 
Nigel Cunningham
Christian Reformed Church of Cobden
103 Curdie Street, Cobden 3266, Victoria, Australia
Ph. +61 3 5595 1185 / +61 417 100 574
Communal Worship: 11 am Sunday.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

  parent reply	other threads:[~2007-07-11 10:14 UTC|newest]

Thread overview: 126+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-30 23:57 Power Mangement Interfaces Jordan Crouse
2007-03-31  0:18 ` Johannes Berg
2007-03-31  0:21   ` Johannes Berg
2007-03-31  4:33     ` [linux-pm] " Gopi P.M.
2007-03-31 15:20     ` Jordan Crouse
2007-03-31 16:12     ` David Brownell
2007-03-31 15:16   ` Jordan Crouse
2007-04-02  7:38     ` [linux-pm] " Pavel Machek
2007-03-31 16:57   ` David Brownell
2007-04-04 16:15     ` Johannes Berg
2007-04-04 18:44       ` David Brownell
2007-04-04 21:49         ` Johannes Berg
2007-04-05  5:05           ` David Brownell
2007-04-05  8:02             ` Johannes Berg
2007-04-05  8:32         ` Zhang Rui
2007-03-31 16:52 ` David Brownell
2007-03-31 18:16   ` Jordan Crouse
2007-03-31 18:57     ` David Brownell
2007-04-01  1:01       ` Jordan Crouse
2007-04-01  3:01         ` David Brownell
2007-04-01  3:01         ` David Brownell
2007-04-01 16:56           ` Jordan Crouse
2007-04-01 16:56           ` Jordan Crouse
2007-04-02  0:28             ` David Brownell
2007-04-02 16:55               ` Jordan Crouse
2007-04-02 16:55               ` Jordan Crouse
2007-04-02 17:53                 ` David Brownell
2007-04-02 17:53                 ` David Brownell
2007-07-08  3:46                 ` rtc-cmos not supporting RTC_AIE? Marcelo Tosatti
2007-07-08  3:46                 ` Marcelo Tosatti
2007-07-08  5:26                   ` David Brownell
2007-07-08  5:26                   ` David Brownell
2007-07-08 19:03                     ` Marcelo Tosatti
2007-07-08 19:03                     ` Marcelo Tosatti
2007-07-08 19:17                       ` David Brownell
2007-07-08 19:31                         ` Richard Hughes
2007-07-08 20:15                           ` Hibernate after alarm wakes from STR David Brownell
2007-07-08 20:15                           ` David Brownell
2007-07-08 22:31                             ` Marcelo Tosatti
2007-07-08 22:31                             ` Marcelo Tosatti
2007-07-09  2:44                               ` David Brownell
2007-07-09  2:44                               ` David Brownell
2007-07-09  8:34                                 ` Richard Hughes
2007-07-09  8:34                                 ` Richard Hughes
2007-07-09 15:40                                 ` Marcelo Tosatti
2007-07-09 16:26                                   ` David Brownell
2007-07-10  2:45                                     ` [linux-pm] " Nigel Cunningham
2007-07-10 16:51                                       ` David Brownell
2007-07-10 22:16                                         ` Nigel Cunningham
2007-07-11  0:45                                           ` Matthew Garrett
2007-07-11  0:45                                           ` [linux-pm] " Matthew Garrett
2007-07-11  0:53                                             ` Nigel Cunningham
2007-07-11  1:23                                               ` Matthew Garrett
2007-07-11  1:23                                               ` [linux-pm] " Matthew Garrett
2007-07-11  1:39                                                 ` Nigel Cunningham
2007-07-11  1:39                                                 ` [linux-pm] " Nigel Cunningham
2007-07-11  1:59                                                   ` Matthew Garrett
2007-07-11  1:59                                                   ` [linux-pm] " Matthew Garrett
2007-07-11  3:14                                                     ` Nigel Cunningham
2007-07-11  3:14                                                     ` [linux-pm] " Nigel Cunningham
2007-07-11 10:09                                                       ` Rafael J. Wysocki
2007-07-11 10:09                                                       ` [linux-pm] " Rafael J. Wysocki
2007-07-11 10:14                                                         ` Nigel Cunningham
2007-07-11 10:14                                                         ` Nigel Cunningham [this message]
2007-07-11 10:31                                                           ` Rafael J. Wysocki
2007-07-11 10:31                                                           ` [linux-pm] " Rafael J. Wysocki
2007-07-11  0:53                                             ` Nigel Cunningham
2007-07-11 16:04                                           ` David Brownell
2007-07-11 16:04                                           ` [linux-pm] " David Brownell
2007-07-11 22:48                                             ` Nigel Cunningham
2007-07-11 22:48                                             ` Nigel Cunningham
2007-07-10 22:16                                         ` Nigel Cunningham
2007-07-10 16:51                                       ` David Brownell
2007-07-10  2:45                                     ` Nigel Cunningham
2007-07-09 16:26                                   ` David Brownell
2007-07-09 15:40                                 ` Marcelo Tosatti
2007-07-08 19:31                         ` rtc-cmos not supporting RTC_AIE? Richard Hughes
2007-07-08 19:17                       ` David Brownell
2007-07-08  3:49                 ` [PATCH] rtc-cmos: use cmos_rtc_board_info to determine wake_on callback Marcelo Tosatti
2007-07-08  5:06                   ` David Brownell
2007-07-08  5:06                   ` David Brownell
2007-07-08  3:49                 ` Marcelo Tosatti
2007-07-08  3:55                 ` [PATCH] OLPC rtc-cmos support Marcelo Tosatti
2007-07-08  3:55                 ` Marcelo Tosatti
2007-07-08  5:13                   ` David Brownell
2007-07-08 18:40                     ` Marcelo Tosatti
2007-07-08 18:40                     ` Marcelo Tosatti
2007-07-08 19:10                       ` David Brownell
2007-07-08 19:10                       ` David Brownell
2007-07-08 20:17                         ` Marcelo Tosatti
2007-07-08 20:47                           ` David Brownell
2007-07-08 20:47                           ` David Brownell
2007-07-08 20:17                         ` Marcelo Tosatti
2007-07-08  5:13                   ` David Brownell
2007-06-19 17:00               ` Power Mangement Interfaces Marcelo Tosatti
2007-06-19 19:17                 ` Jens Axboe
2007-06-19 19:17                 ` Jens Axboe
2007-06-19 19:41                 ` Woodruff, Richard
2007-06-21  1:30                 ` David Brownell
2007-07-08 22:10                   ` [PATCH] add powerbutton and lid platform devices Marcelo Tosatti
2007-07-08 22:10                   ` Marcelo Tosatti
2007-07-09 15:05                     ` Jordan Crouse
2007-07-09 16:30                       ` David Brownell
2007-07-09 16:30                       ` David Brownell
2007-07-09 16:36                         ` Jordan Crouse
2007-07-09 16:36                         ` Jordan Crouse
2007-07-16  8:51                         ` Richard Hughes
2007-07-16  8:51                         ` Richard Hughes
2007-07-16 17:11                       ` C. Scott Ananian
2007-07-16 17:11                       ` C. Scott Ananian
2007-07-09 15:05                     ` Jordan Crouse
2007-06-21  1:30                 ` Power Mangement Interfaces David Brownell
2007-06-19 17:00               ` Marcelo Tosatti
2007-04-02  0:28             ` David Brownell
2007-04-02 10:23           ` Zhang Rui
2007-04-02 18:24             ` David Brownell
2007-04-02 19:40               ` Matthew Garrett
2007-04-02 21:31                 ` David Brownell
2007-04-02 21:31                 ` David Brownell
2007-04-02 19:40               ` Matthew Garrett
2007-04-05  8:20               ` Zhang Rui
2007-04-05  8:20               ` Zhang Rui
2007-04-02 18:24             ` David Brownell
2007-04-02 10:07       ` Zhang Rui
2007-03-31 19:14     ` Jim Gettys
2007-04-02  9:36 ` Zhang Rui

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=200707112014.28958.nigel@nigel.suspend2.net \
    --to=nigel@nigel.suspend2.net \
    --cc=david-b@pacbell.net \
    --cc=hughsient@gmail.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-pm@lists.linux-foundation.org \
    --cc=marcelo@kvack.org \
    --cc=mjg59@srcf.ucam.org \
    --cc=nigel@suspend2.net \
    --cc=rjw@sisk.pl \
    --cc=rtc-linux@googlegroups.com \
    /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.