mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Aaron Durbin" <adurbin@google.com>
To: "Yinghai Lu" <yhlu.kernel@gmail.com>
Cc: "Andi Kleen" <ak@suse.de>,
	linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org,
	len.brown@intel.com, akpm@osdl.org
Subject: Re: [PATCH] ACPI: Add reboot mechanism
Date: Tue, 17 Jul 2007 08:08:19 -0700	[thread overview]
Message-ID: <8f95bb250707170808n19799ba5l3acd13e2ee7e0f08@mail.gmail.com> (raw)
In-Reply-To: <86802c440707170740i30ed2110l130dafbec8e4d4a5@mail.gmail.com>

On 7/17/07, Yinghai Lu <yhlu.kernel@gmail.com> wrote:
> On 7/17/07, Andi Kleen <ak@suse.de> wrote:
> > On Monday 16 July 2007 20:00:19 Aaron Durbin wrote:
> > >
> > > Add the ability to reset the machine using the RESET_REG in ACPI's FADT table.
> >
> > Why? I had such a patch at some point as experiment, but it never
> > helped actually fix a box.
> >
> system without superio, should still have SMI to simulate one keyboard
> controller. So the key board reset still works.

This is true if SMIs are still flowing, and the SMI handler correctly handles
the reset properly. I have seen cases where the SMI handler is broken which
causes the system to perpetually hang. Utilizing the exposed reset mechanism
in ACPI should reset the system every time. However, if the reset fails it
will fall back to the keyboard controller.

-Aaron

  reply	other threads:[~2007-07-17 15:08 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-16 18:00 Aaron Durbin
2007-07-16 18:17 ` Arjan van de Ven
2007-07-16 18:22   ` Aaron Durbin
2007-07-17  1:46 ` Shaohua Li
2007-07-17  2:10   ` Aaron Durbin
2007-07-17 11:31 ` Andi Kleen
2007-07-17 14:40   ` Yinghai Lu
2007-07-17 15:08     ` Aaron Durbin [this message]
2007-07-17 15:42       ` Andi Kleen
2007-07-17 15:47         ` Aaron Durbin
2007-07-17 15:08   ` Aaron Durbin
2007-07-27  6:17     ` Pavel Machek
     [not found] <fa.IMXKKgfBkv/U+MG0xx5sLkpV2Ws@ifi.uio.no>
     [not found] ` <fa.+iOE+OqrQUbecN788fV4aEhLLtg@ifi.uio.no>
2007-07-17 14:39   ` Robert Hancock
2007-07-17 15:41     ` Andi Kleen

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=8f95bb250707170808n19799ba5l3acd13e2ee7e0f08@mail.gmail.com \
    --to=adurbin@google.com \
    --cc=ak@suse.de \
    --cc=akpm@osdl.org \
    --cc=len.brown@intel.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=yhlu.kernel@gmail.com \
    /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®