public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: Ducrot Bruno <ducrot-kk6yZipjEM5g9hUCZPvPmw@public.gmane.org>
To: Dave Jones <davej-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
Cc: Len Brown <len.brown-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
	ak-h9bWGtP8wOw@public.gmane.org,
	ACPI Developers
	<acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org>
Subject: Re: [Fwd: [PATCH] Don't save state in ACPI on shutdown/reboot]
Date: Thu, 13 Nov 2003 00:07:56 +0100	[thread overview]
Message-ID: <20031112230756.GD32464@poupinou.org> (raw)
In-Reply-To: <20031112220559.GR31026-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>

On Wed, Nov 12, 2003 at 10:05:59PM +0000, Dave Jones wrote:
> On Tue, Nov 11, 2003 at 06:00:35PM +0100, Ducrot Bruno wrote:
> 
>  > > The ACPI code in 2.4 currently saves the machine state as for a 
>  > > suspend when doing an reboot or shutdown. This fails on x86-64
>  > > on some boxes.
>  > > 
>  > > Don't save/restore state in ACPI when entering S5.
>  > 
>  > That break also some i386 boxes.  That why I submitted another patch (do
>  > I have to post it again?)
> 
> Do you mean Andi's patch breaks i386 boxes, or this bug also appears
> on i386 ? The former seems unlikely from the look of Andi's patch.
> 

I wanted to point that some i386 boxes are broken as well, and
then a patch is needed.  And yes, when I read again my email, that
a bit confusing.

The patch I send actually implement the exact same
behaviour of 2.6 (minus S5 via the proc file, disabled on 2.6).

Also, it is much likely a design fix.  'power off' shall not
be dependant of CONFIG_ACPI_SLEEP (even though than one is always
yes for 2.4 if CONFIG_ACPI, at least on i386).

Cheers,

-- 
Ducrot Bruno

--  Which is worse:  ignorance or apathy?
--  Don't know.  Don't care.


-------------------------------------------------------
This SF.Net email sponsored by: ApacheCon 2003,
16-19 November in Las Vegas. Learn firsthand the latest
developments in Apache, PHP, Perl, XML, Java, MySQL,
WebDAV, and more! http://www.apachecon.com/

      parent reply	other threads:[~2003-11-12 23:07 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-11-11 16:23 [Fwd: [PATCH] Don't save state in ACPI on shutdown/reboot] Len Brown
     [not found] ` <1068567824.24982.21.camel-D2Zvc0uNKG8@public.gmane.org>
2003-11-11 17:00   ` Ducrot Bruno
     [not found]     ` <20031111170035.GD29175-kk6yZipjEM5g9hUCZPvPmw@public.gmane.org>
2003-11-12 22:05       ` Dave Jones
     [not found]         ` <20031112220559.GR31026-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2003-11-12 23:07           ` Ducrot Bruno [this message]

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=20031112230756.GD32464@poupinou.org \
    --to=ducrot-kk6yzipjem5g9huczpvpmw@public.gmane.org \
    --cc=acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
    --cc=ak-h9bWGtP8wOw@public.gmane.org \
    --cc=davej-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=len.brown-ral2JQCrhuEAvxtiuMwx3w@public.gmane.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