All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pavel Machek <pavel@ucw.cz>
To: Oliver Neukum <oliver@neukum.org>
Cc: "Rafael J. Wysocki" <rjw@sisk.pl>,
	Alan Stern <stern@rowland.harvard.edu>,
	Oliver Neukum <oliver@neukum.name>,
	pm list <linux-pm@lists.osdl.org>,
	linux-kernel@vger.kernel.org
Subject: Re: [linux-pm] question on resume()
Date: Wed, 31 Jan 2007 11:30:45 +0100	[thread overview]
Message-ID: <20070131103045.GA19640@elf.ucw.cz> (raw)
In-Reply-To: <200701311114.55685.oliver@neukum.org>

Hi!

> > > So, this means, on suspend():
> > > 
> > > 1. Don't worry about TASK_UNINTERRUPTIBLE
> > > 2. Do worry about TASK_INTERRUPTIBLE
> > > We have to cease IO and must not call wake_up_interruptible()
> > 
> > "cease IO"? No, I believe it is enough not to start new I/O. Userspace
> > is frozen at that point, it can't ask you to do I/O.
> >
> > > Isn't that a race until suspend() is called?
> > 
> > I do not think so.
> 
> What about URBs in flight which are waited for with TASK_INTERRUPTIBLE?

Same thing as if someone sent a signal to that process. (We are
talking about user processes here, right? Kernel threads have to take
care themselves).

> > > On resume():
> > > 
> > > 1. Don't worry about TASK_UNINTERRUPTIBLE
> > > 2. Do not restart IO that may call wake_up_interruptible()
> > > 
> > > When do we restart such IO?
> > 
> > We reuse signal handling code to do that for us. It is same situation
> > as when someone signals task doing I/O.
> 
> What happens to tasks in TASK_INTERRUPTIBLE which are frozen?
> Are they interrupted and frozen?

Interrupted with fake signal, then frozen, yes.
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

  reply	other threads:[~2007-01-31 10:30 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-29 11:06 question on resume() Oliver Neukum
2007-01-29 11:24 ` Nigel Cunningham
2007-01-29 11:34   ` Oliver Neukum
2007-01-29 20:14     ` Nigel Cunningham
2007-01-29 21:04       ` Oliver Neukum
2007-01-29 21:21         ` Nigel Cunningham
2007-01-29 23:10           ` Rafael J. Wysocki
2007-01-30 16:32             ` Rafael J. Wysocki
2007-01-30 16:50               ` Oliver Neukum
2007-01-30 22:32                 ` Rafael J. Wysocki
2007-01-30 22:32                   ` Rafael J. Wysocki
2007-01-31  8:33                   ` Rafael J. Wysocki
2007-01-31  8:33                     ` Rafael J. Wysocki
2007-01-31  8:40                     ` Oliver Neukum
2007-01-31  8:40                       ` Oliver Neukum
2007-01-31  8:49                       ` Rafael J. Wysocki
2007-01-31  9:04                         ` Oliver Neukum
2007-01-31  9:04                           ` Oliver Neukum
2007-01-31  9:36                           ` Pavel Machek
2007-01-31  9:36                             ` [linux-pm] " Pavel Machek
2007-01-31 10:14                             ` Oliver Neukum
2007-01-31 10:14                               ` [linux-pm] " Oliver Neukum
2007-01-31 10:30                               ` Pavel Machek [this message]
2007-01-31 15:54                             ` Alan Stern
2007-01-31 15:54                               ` [linux-pm] " Alan Stern
2007-01-31 16:12                               ` Oliver Neukum
2007-01-31 16:12                                 ` [linux-pm] " Oliver Neukum
2007-01-31 16:27                                 ` Alan Stern
2007-01-31 16:27                                   ` [linux-pm] " Alan Stern
2007-01-31 18:04                               ` Woodruff, Richard
2007-01-31 18:04                                 ` [linux-pm] " Woodruff, Richard
2007-01-31 15:48                     ` Alan Stern
2007-01-31 15:48                       ` [linux-pm] " Alan Stern
2007-01-31 18:52                       ` Rafael J. Wysocki

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=20070131103045.GA19640@elf.ucw.cz \
    --to=pavel@ucw.cz \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@lists.osdl.org \
    --cc=oliver@neukum.name \
    --cc=oliver@neukum.org \
    --cc=rjw@sisk.pl \
    --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 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.