public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Nigel Cunningham <nigel@nigel.suspend2.net>
To: Miklos Szeredi <miklos@szeredi.hu>
Cc: rjw@sisk.pl, a1426z@gawab.com, jeremy@goop.org, jbms@cmu.edu,
	pavel@ucw.cz, nickpiggin@yahoo.com.au,
	linux-kernel@vger.kernel.org, akpm@linux-foundation.org
Subject: Re: Hibernation Redesign
Date: Wed, 11 Jul 2007 22:17:18 +1000	[thread overview]
Message-ID: <200707112217.19220.nigel@nigel.suspend2.net> (raw)
In-Reply-To: <E1I8b0l-0000DB-00@dorka.pomaz.szeredi.hu>

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

Hi.

On Wednesday 11 July 2007 22:09:39 Miklos Szeredi wrote:
> > > Freezing of tasks is slowing down suspend.  Don't know how serious
> > > this is, suspend is pretty fast, but could possibly be even faster.
> > 
> > It's FUD. Freezing of tasks normally takes next to no time. I've never 
> > understood the rediculously long timeout it has. If freezing succeeds, all 
> > processes are frozen within 1/2 a second tops. If it fails, nothing is 
going 
> > to change in the following 19.5 seconds (or whatever it is if I don't 
> > remember the value properly).
> 
> Right.  The 20s timeout is again a sign of brokenness.
> 
> If we expect something to fail, it should fail immediately, without
> waiting for arbitrary timeouts.
> 
> And if we don't expect it to fail, why the timeout?

Two reasons:

1) The processing does take time. We're sending (pseudo-)signals to other 
processes, and the amount of time it will take for them to enter the 
refrigerator will depend upon the number of processes there are, the 
scheduling algorithm being used, the rate at which they're forking and what 
other activity is occuring. That's why I said "normally" above.

2) Things do get broken. New functionality gets added which sometimes doesn't 
immediately play well with the freezer. In such cases, we want to fail nicely 
rather than wrongly assuming it will always work and waiting indefinitely.

Regards,

Nigel
-- 
See http://www.tuxonice.net for Howtos, FAQs, mailing
lists, wiki and bugzilla info.

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

  reply	other threads:[~2007-07-11 12:17 UTC|newest]

Thread overview: 60+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-08 14:37 Hibernation Redesign (was: malicious filesystems (was Re: [linux-pm] Re: [PATCH] Remove process freezer from suspend to RAM) Al Boldi
2007-07-09  4:11 ` Hibernation Redesign Jeremy Maitin-Shepard
2007-07-09  4:29   ` Nick Piggin
2007-07-09  4:36     ` Jeremy Maitin-Shepard
2007-07-09  4:45       ` Nick Piggin
2007-07-09  4:54         ` Jeremy Maitin-Shepard
2007-07-09  4:58           ` Nick Piggin
2007-07-09  5:33             ` Nick Piggin
2007-07-09  6:22           ` Jeremy Fitzhardinge
2007-07-09 13:45       ` Pavel Machek
2007-07-09 14:02         ` Oliver Neukum
2007-07-09 14:26           ` Jeremy Maitin-Shepard
2007-07-09 15:09             ` Oliver Neukum
2007-07-09 15:27               ` Jeremy Maitin-Shepard
2007-07-09  4:39     ` Nick Piggin
2007-07-09 13:52       ` Pavel Machek
2007-07-09 15:30         ` Al Boldi
2007-07-10  1:29           ` Nick Piggin
2007-07-10  2:28           ` Jeremy Maitin-Shepard
2007-07-10 14:57             ` Jeremy Fitzhardinge
2007-07-10 17:25               ` Jeremy Maitin-Shepard
2007-07-10 22:59                 ` Jeremy Fitzhardinge
2007-07-11  4:11                   ` Al Boldi
2007-07-11 10:27                     ` Rafael J. Wysocki
2007-07-11 10:42                       ` Miklos Szeredi
2007-07-11 11:04                         ` Rafael J. Wysocki
2007-07-11 11:11                           ` Miklos Szeredi
2007-07-11 11:50                             ` Rafael J. Wysocki
2007-07-11 11:54                               ` Miklos Szeredi
2007-07-11 12:00                                 ` Nigel Cunningham
2007-07-11 12:09                                   ` Miklos Szeredi
2007-07-11 12:17                                     ` Nigel Cunningham [this message]
2007-07-11 12:27                                     ` Rafael J. Wysocki
2007-07-11 12:29                                       ` Miklos Szeredi
2007-07-11 21:04                                         ` Rafael J. Wysocki
2007-07-12  9:15                                     ` Pavel Machek
2007-07-12 22:13                                       ` Miklos Szeredi
2007-07-11 12:19                                 ` Rafael J. Wysocki
2007-07-11 12:49                                   ` Nigel Cunningham
2007-07-11 21:06                                     ` Rafael J. Wysocki
2007-07-11 12:11                             ` Nigel Cunningham
2007-07-11 12:24                               ` Miklos Szeredi
2007-07-11 12:46                                 ` Nigel Cunningham
2007-07-11 12:55                                   ` Miklos Szeredi
2007-07-11 13:16                               ` Jeremy Maitin-Shepard
2007-07-11 20:48                                 ` Mark Lord
2007-07-11 21:14                                   ` Rafael J. Wysocki
2007-07-11 22:17                                   ` Al Boldi
2007-07-11 22:34                                     ` Rafael J. Wysocki
2007-07-11 23:12                                       ` Al Boldi
2007-07-11 23:31                                         ` Nigel Cunningham
2007-07-12  3:11                                           ` Al Boldi
2007-07-12 13:20                                         ` Rafael J. Wysocki
2007-07-16  6:04                                           ` Nick Piggin
2007-07-12 20:29                                     ` Jeremy Maitin-Shepard
2007-07-11 23:46                                 ` Nigel Cunningham
2007-07-11 17:55                               ` david
2007-07-11 22:54                                 ` Nigel Cunningham
2007-07-10 17:45               ` Al Boldi
2007-07-10 18:20                 ` Jeremy Maitin-Shepard

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=200707112217.19220.nigel@nigel.suspend2.net \
    --to=nigel@nigel.suspend2.net \
    --cc=a1426z@gawab.com \
    --cc=akpm@linux-foundation.org \
    --cc=jbms@cmu.edu \
    --cc=jeremy@goop.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=miklos@szeredi.hu \
    --cc=nickpiggin@yahoo.com.au \
    --cc=nigel@suspend2.net \
    --cc=pavel@ucw.cz \
    --cc=rjw@sisk.pl \
    /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