Re: 2.6.9-rc3 software suspend (pmdisk) stopped working

From: Kevin Fenzi (kevin-linux-kernel_at_scrye.com)
Date: 10/01/04

  • Next message: Denis Vlasenko: "[PATCH] small sha512 cleanup"
    Date:	Fri, 1 Oct 2004 13:25:27 -0600
    To: linux-kernel@vger.kernel.org
    To: rjw@sisk.pl ("Rafael J. Wysocki")
    
    

    -----BEGIN PGP SIGNED MESSAGE-----
    Hash: SHA1

    >>>>> "Rafael" == Rafael J Wysocki <rjw@sisk.pl> writes:

    Rafael> On Friday 01 of October 2004 18:03, Kevin Fenzi wrote:
    >> >>>>> "Pavel" == Pavel Machek <pavel@ucw.cz> writes:
    >>
    Pavel> Hi!
    >> >> Anyone noticed that pmdisk software suspend stopped working in
    >> -rc3 >> ? In -rc2 it worked just fine. My script was
    >> >>
    >> >> chvt 1 echo -n shutdown >/sys/power/disk echo -n disk >>
    >> >/sys/power/state chvt 7
    >> >>
    >> >> In -rc3 it appears to write pages out to disk, but never shuts
    >> down >> the machine. Is there something else i need to do or am
    >> missing ?
    >>
    Pavel> You are not missing anything, it is somehow broken. I'll try to
    Pavel> find out what went wrong and fix it. In the meantime, look at
    Pavel> -mm series, it works there. Pavel
    >> I finally had a chance to try 2.6.9-rc3 here last night.
    >>
    >> It suspended ok for me, but on resume it would load in the cache
    >> and then reboot. :(

    Rafael> Always? I mean, is it reproducible? I have a similar
    Rafael> problem, but it is not reproducible, apparently. Sometimes it
    Rafael> reboots, sometimes it reports a double fault, but most often
    Rafael> it resumes just fine.

    Well, I only tried it twice, but it rebooted both times.

    After applying the patch below from Pavel on top of 2.6.9-rc3 it now
    seems to work (again, I have only done a few cycles).

    Do you have HIMEM enabled? Does the patch below make it more stable
    for you?

    Rafael> Greets, RJW

    kevin

    - --- clean/kernel/power/swsusp.c 2004-10-01 00:30:32.000000000 +0200
    +++ clean-mm/kernel/power/swsusp.c 2004-09-26 01:34:27.000000000 +0200
    @@ -856,7 +860,9 @@
             local_irq_disable();
             save_processor_state();
             error = swsusp_arch_suspend();
    + /* Restore control flow magically appears here */
             restore_processor_state();
    + restore_highmem();
             local_irq_enable();
             return error;
     }
    -----BEGIN PGP SIGNATURE-----
    Version: GnuPG v1.2.4 (GNU/Linux)
    Comment: Processed by Mailcrypt 3.5.8 <http://mailcrypt.sourceforge.net/>

    iD8DBQFBXa8q3imCezTjY0ERAv55AKCHAvg2sqBYi4p2cR+ZIJ6Y1bZeiwCfc1cN
    9m/3MpImIeu1LfchBRK8LEE=
    =fIt6
    -----END PGP SIGNATURE-----
    -
    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/


  • Next message: Denis Vlasenko: "[PATCH] small sha512 cleanup"

    Relevant Pages


    Loading