mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: Stefan Richter <stefanr@s5r6.in-berlin.de>, Pavel Machek <pavel@ucw.cz>
Cc: Ben Collins <ben.collins@ubuntu.com>,
	kernel list <linux-kernel@vger.kernel.org>,
	linux1394-devel@lists.sourceforge.net,
	Ingo Molnar <mingo@elte.hu>
Subject: Re: regression: fireware causes oops during system
Date: Sun, 16 Sep 2007 22:16:01 +0200	[thread overview]
Message-ID: <200709162216.02448.rjw@sisk.pl> (raw)
In-Reply-To: <46ED8AD0.2040004@s5r6.in-berlin.de>

On Sunday, 16 September 2007 21:58, Stefan Richter wrote:
> Rafael J. Wysocki wrote:
> >> Pavel Machek wrote:
> >>> commit 831441862956fffa17b9801db37e6ea1650b0f69
> >>> tree b0334921341f8f1734bdd3243de76d676329d21c
> >>> parent 787d2214c19bcc9b6ac48af0ce098277a801eded
> >>> author Rafael J. Wysocki <rjw@sisk.pl> Tue, 17 Jul 2007 04:03:35 -0700
> >>> committer Linus Torvalds <torvalds@woody.linux-foundation.org> Tue, 17
> >>> Jul 2007 10:23:02 -0700
> >>>
> >>>     Freezer: make kernel threads nonfreezable by default
> > 
> > Well, I don't think so.
> > 
> > nodemgr_host_thread() calls set_freezable() as it should.
> 
> This commit is certainly OK, as it should merely preserve status quo.
> Also note that Pavel wrote in his initial post that the problem became
> apparent way after -rc1.  Full quote:
> 
> | I noticed empty suspend stopped working around 2.6.23-rc4, and it is
> | still present in 2.6.23-rc6. To reproduce
> |
> | swapoff -a
> | echo disk > /sys/power/state
> | echo disk > /sys/power/state
> |
> | Unsetting
> |
> | CONFIG_IEEE1394=y
> |
> | solves the problem.

Yes.

I thought that there might be a later change that exposed a bug in it.

Hm.  Pavel, can you do

# echo test > /sys/power/disk
# echo disk > /sys/power/state

and see what happens?

Greetings,
Rafael

  reply	other threads:[~2007-09-16 20:03 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-09-11 19:00 Pavel Machek
2007-09-11 19:08 ` Ben Collins
2007-09-11 19:29   ` Stefan Richter
2007-09-11 19:45     ` Stefan Richter
2007-09-16 17:54       ` Pavel Machek
2007-09-16 18:41         ` Stefan Richter
2007-09-16 19:52           ` Rafael J. Wysocki
2007-09-16 19:58             ` Stefan Richter
2007-09-16 20:16               ` Rafael J. Wysocki [this message]
2007-10-05  7:08                 ` Pavel Machek
2007-10-05 21:28                   ` Rafael J. Wysocki
2007-09-16 17:57       ` 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=200709162216.02448.rjw@sisk.pl \
    --to=rjw@sisk.pl \
    --cc=ben.collins@ubuntu.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux1394-devel@lists.sourceforge.net \
    --cc=mingo@elte.hu \
    --cc=pavel@ucw.cz \
    --cc=stefanr@s5r6.in-berlin.de \
    /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

Powered by JetHome