mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Pavel Machek <pavel@ucw.cz>
To: hugang <hugang@soulinfo.com>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>, linux-kernel@vger.kernel.org
Subject: Re: software suspend in 2.5.70-mm3.
Date: Tue, 3 Jun 2003 20:55:51 +0200	[thread overview]
Message-ID: <20030603185551.GA3274@zaurus.ucw.cz> (raw)
In-Reply-To: <20030603223511.155ea2cc.hugang@soulinfo.com>

Hi!

> > > Hi Pavel Machek:
> > > 
> > > I try the 2.5.70-mm3 with software suspend function. When suspend it will oops at ide-disk.c 1526 line
> > >    BUG_ON (HWGROUP(drive)->handler);
> > > 
> > > I'm disable this check, The software suspend can work, and also can resumed. But this fix is not best way. I found in ide-io.c 1196
> > >    hwgroup->handler = NULL;
> > > is the problem.
> > 
> > The only way to make the suspend work properly is to queue the suspend
> > sequence wit the other requests. Ben was doing some playing with this
> > but I'm not sure what happened to it.
> > 
> Yes the above patch is not safe, When i'm run updatedb and suspsned, After resume will oops at kjournal. 
> 
> Here is another test on it, it can works with updatedb.
> -
> I found a best way to fix it. here is it. With the patch, I'm run updatedb and suspend for 5 counts, every things is ok.
> 


Locating Ben's patch and forward-porting 
it would be way better...
-- 
				Pavel
Written on sharp zaurus, because my Velo1 broke. If you have Velo you don't need...


  parent reply	other threads:[~2003-06-03 18:46 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-06-03 13:11 hugang
2003-06-03 13:22 ` Alan Cox
2003-06-03 14:35   ` hugang
2003-06-03 14:08     ` Alan Cox
2003-06-03 17:04       ` Benjamin Herrenschmidt
2003-06-04 13:14         ` IDE Power Management (Was: software suspend in 2.5.70-mm3) Benjamin Herrenschmidt
2003-06-04 13:51           ` Michael Frank
2003-06-04 13:58             ` Benjamin Herrenschmidt
2003-06-04 14:10             ` Michael Frank
2003-06-04 14:29               ` Benjamin Herrenschmidt
2003-06-04 14:35                 ` Michael Frank
2003-06-04 19:46                 ` Pavel Machek
2003-06-04 20:16           ` Pavel Machek
2003-06-03 18:55     ` Pavel Machek [this message]
2003-06-08  2:26       ` software suspend in 2.5.70-mm3 hugang
2003-06-08 18:29         ` Pavel Machek
2003-06-08  2:27       ` hugang

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=20030603185551.GA3274@zaurus.ucw.cz \
    --to=pavel@ucw.cz \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=hugang@soulinfo.com \
    --cc=linux-kernel@vger.kernel.org \
    /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

all inboxes | Powered by JetHome®