public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jonathan Sambrook <jonathan.sambrook@dsvr.co.uk>
To: Pavel Machek <pavel@suse.cz>
Cc: linux-kernel@vger.kernel.org,
	Swsusp mailing list <swsusp-devel@lists.sourceforge.net>
Subject: Re: [Swsusp-devel] Re: swsusp problems [was Re: Your opinion on the merge?]
Date: Wed, 24 Mar 2004 20:22:59 +0000	[thread overview]
Message-ID: <20040324202259.GJ20333@jsambrook> (raw)
In-Reply-To: <20040324151831.GB25738@atrey.karlin.mff.cuni.cz>

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

At 16:18 on Wed 24/03/04, pavel@suse.cz masquerading as 'Pavel Machek' wrote:
> Hi!
> 
> > > Except when it is not in syslog, because it was after atomic copy or
> > > before atomic copy back. swsusp is pretty unique in this respect.
> > >
> > 
> > And I would consider an error that happens after atomic copy critical. If
> > this happens all attempts to draw progress bar, etc. should be stopped and
> > suspend should probably abort as well.
> 
> Well, not all printks() are errors this hard. And at some points, it
> is no longer possible to abort (after pagedir is on disk, its okay to
> panic (machine will resume normally after that), but its not okay to
> simply return. You could fix signature then return, but it would be hard). 
> 
> > What happens if swsusp1 gets such an error during atomic phase? Will it
> > continue or abort? If it continues I would imagine user not noticing the
> > message it it flashes the split second before the box powers off. 
> 
> It continues. Fortunately powerdown takes enough time on most machines
> that messages can be readed ... 

> if you pay attetion.

Which is not a normal usage scenario.

Common scenario:

 suspend machine then go home/to bed


 BTW Pavel I'm not arguing that the code has to stay in without
 modification
 (e.g. massively stripped down or whatever). But this is one place where
 kernel code is a lot closer to Joe Enduser's awareness than is usually
 the case, so IMUO, the priorities shift.

 That said, this discussion seems to be necessary to refine what is
 necessary from what has been creatively evolved.

 Jonathan

> 								Pavel
> -- 
> Horseback riding is like software...
> ...vgf orggre jura vgf serr.
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/

-- 
                   
 Jonathan Sambrook 
Software  Developer 
 Designer  Servers

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

  reply	other threads:[~2004-03-24 20:23 UTC|newest]

Thread overview: 80+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1079659165.15559.34.camel@calvin.wpcb.org.au>
     [not found] ` <20040318193703.4c02f7f5.akpm@osdl.org>
     [not found]   ` <1079661410.15557.38.camel@calvin.wpcb.org.au>
     [not found]     ` <20040318200513.287ebcf0.akpm@osdl.org>
     [not found]       ` <1079664318.15559.41.camel@calvin.wpcb.org.au>
2004-03-21 22:00         ` swsusp problems [was Re: Your opinion on the merge?] Pavel Machek
2004-03-22  0:29           ` Pavel Machek
2004-03-22 20:55           ` Nigel Cunningham
2004-03-22 23:17             ` Pavel Machek
2004-03-23  9:53               ` [Swsusp-devel] " Jonathan Sambrook
2004-03-23 15:25                 ` Micha Feigin
2004-03-23 21:47                 ` Pavel Machek
2004-03-23 21:08                   ` Nigel Cunningham
2004-03-23 22:17                     ` Pavel Machek
2004-03-23 22:38                       ` Nigel Cunningham
2004-03-23 23:07                     ` Michael Frank
2004-03-23 23:17                       ` Pavel Machek
2004-03-23 22:36                         ` Nigel Cunningham
2004-03-23 23:45                           ` Dumitru Ciobarcianu
2004-03-23 22:52                             ` Nigel Cunningham
2004-03-24  0:05                               ` Joel Jaeggli
2004-03-23 23:10                                 ` Nigel Cunningham
2004-03-23 22:43                   ` Dmitry Torokhov
2004-03-23 23:32                     ` Pavel Machek
2004-03-23 22:40                       ` Nigel Cunningham
2004-03-23 23:44                         ` Pavel Machek
2004-03-24  3:12                           ` Michael Frank
2004-03-24 10:17                             ` Pavel Machek
2004-03-24 22:46                               ` Michael Frank
2004-03-24 23:23                                 ` Pavel Machek
2004-03-24 23:56                                   ` Michael Frank
2004-03-25  0:23                                     ` -nice tree [was Re: [Swsusp-devel] Re: swsusp problems [was Re: Your opinion on the merge?]] Pavel Machek
2004-03-25  0:50                                       ` Michael Frank
2004-03-25  1:41                                         ` Pavel Machek
     [not found]                                           ` <200403250857.08920.matthias.wieser@hiasl.net>
2004-03-25 20:39                                             ` Nigel Cunningham
2004-03-25 22:27                                               ` Pavel Machek
2004-03-25 21:38                                                 ` Nigel Cunningham
2004-03-25 22:54                                                   ` Pavel Machek
2004-03-26  5:59                                                     ` Michael Frank
2004-03-26 10:50                                                       ` Pavel Machek
2004-03-26 13:17                                                         ` Michael Frank
2004-03-27 14:49                                                   ` Jamie Lokier
2004-03-29 12:10                                                     ` fast compressed fs (was Re: -nice tree) Pavel Machek
2004-03-26  5:59                                                 ` -nice tree [was Re: [Swsusp-devel] Re: swsusp problems [was Re: Your opinion on the merge?]] Michael Frank
2004-03-26 10:22                                                   ` Pavel Machek
2004-03-26 20:30                                                     ` Nigel Cunningham
2004-03-26 22:22                                                       ` Pavel Machek
2004-03-27  2:08                                                         ` Nigel Cunningham
2004-03-27  3:37                                                           ` Luke-Jr
2004-03-27  4:28                                                             ` Micha Feigin
2004-03-27  4:40                                                               ` Luke-Jr
2004-03-27 19:50                                                                 ` Micha Feigin
2004-03-27 20:03                                                                   ` Luke-Jr
2004-03-27 21:01                                                                     ` Paranoia is fun [Was Re: -nice tree [was Re: [Swsusp-devel] Re: swsusp problems [was Re: Your opinion on the merge?]]] Michael Frank
2004-03-27 21:40                                                                       ` Luke-Jr
2004-03-27 21:29                                                             ` -nice tree [was Re: [Swsusp-devel] Re: swsusp problems [was Re: Your opinion on the merge?]] Pavel Machek
2004-03-28  0:27                                                               ` Micha Feigin
2004-03-27  2:21                                                 ` Micha Feigin
2004-03-29 12:13                                                   ` Pavel Machek
2004-03-24  4:52                       ` [Swsusp-devel] Re: swsusp problems [was Re: Your opinion on the merge?] Dmitry Torokhov
2004-03-24  5:04                         ` Nigel Cunningham
2004-03-24  6:22                           ` Michael Frank
2004-03-24  5:46                             ` Nigel Cunningham
2004-03-24  7:31                               ` Michael Frank
2004-03-24 10:26                                 ` Pavel Machek
2004-03-25  0:06                                   ` Jonathan Sambrook
2004-03-24  9:32                             ` Karol Kozimor
2004-03-24 14:15                               ` Michael Frank
2004-03-25  0:35                                 ` Karol Kozimor
2004-03-24  5:06                         ` sleeping in request function mohanlal jangir
     [not found]                           ` <20040324083313.GG3377@suse.de>
2004-03-25  3:45                             ` mohanlal jangir
2004-03-24 10:22                         ` [Swsusp-devel] Re: swsusp problems [was Re: Your opinion on the merge?] Pavel Machek
2004-03-24 12:48                           ` Dmitry Torokhov
2004-03-24 15:18                             ` Pavel Machek
2004-03-24 20:22                               ` Jonathan Sambrook [this message]
2004-03-24 21:06                                 ` Michael Frank
2004-03-25 22:13                                   ` swsusp is not reliable. Face it. [was Re: [Swsusp-devel] Re: swsusp problems] Pavel Machek
2004-03-25 21:33                                     ` Nigel Cunningham
2004-03-25 22:52                                       ` Pavel Machek
2004-03-26  5:59                                     ` Michael Frank
2004-03-26  9:59                                       ` Pavel Machek
2004-03-26 10:21                                         ` Michael Frank
2004-03-25 22:08                                 ` [Swsusp-devel] Re: swsusp problems [was Re: Your opinion on the merge?] Pavel Machek
2004-03-24 22:05                           ` Markus Gaugusch
2004-03-24 21:25                             ` Nigel Cunningham

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=20040324202259.GJ20333@jsambrook \
    --to=jonathan.sambrook@dsvr.co.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pavel@suse.cz \
    --cc=swsusp-devel@lists.sourceforge.net \
    /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