public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: "H. Peter Anvin" <hpa@zytor.com>
To: "Eric W. Biederman" <ebiederm@xmission.com>
Cc: Zhao Yakui <yakui.zhao@intel.com>, Avi Kivity <avi@redhat.com>,
	Ingo Molnar <mingo@elte.hu>, Len Brown <lenb@kernel.org>,
	Stephen Rothwell <sfr@canb.auug.org.au>,
	Andrew Morton <akpm@linux-foundation.org>,
	Matthew Garrett <mjg@redhat.com>,
	Andrey Borzenkov <arvidjaar@mail.ru>,
	"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
	"Brown, Len" <len.brown@intel.com>,
	Thomas Gleixner <tglx@linutronix.de>,
	Eduardo Habkost <ehabkost@redhat.com>
Subject: Re: [PATCH 10/10] x86, ACPI: default to reboot via ACPI (again)
Date: Wed, 12 Nov 2008 20:34:11 -0800	[thread overview]
Message-ID: <491BAE43.5080309@zytor.com> (raw)
In-Reply-To: <m1vdusnu4d.fsf@frodo.ebiederm.org>

Eric W. Biederman wrote:
> 
> Looking at the coreboot source.  It is indeed defined for Nvidia, SIS,
> AMD and several others.
> 
> A lot of boards use:
> outb(0x02, 0xcf9);
> outb(0x06, 0xcf9);
> 
> Instead of just writing a plain 6.   I think at least on some machines
> there is a requirement for a low to hight transition.
> 

That is correct; that is what my patch has, with a 50 µs delay in
between (I also don't touch the ~6 bits.)

> 
> Right.  The only scary question is will a motherboard hang if you write
> to 0xcf9.  If we don't know of an example where writing to 0xcf9 will prevent
> us from getting to the next reset method then adding a generic 0xcf9 is
> safe.
> 

It would be highly surprising, except possibly on pre-PCI systems, which
is why my patch skips CF9 if the standard PCI ports are not detected.
Although CF9 isn't defined in the PCI spec, it falls in the PCI
configuration port range.

	-hpa
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2008-11-13  4:35 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-07  4:45 ACPI patchese on test branch Len Brown
2008-11-07  4:45 ` [PATCH 01/10] ACPI: pci_link: remove acpi_irq_balance_set() interface Len Brown
2008-11-07  4:45   ` [PATCH 02/10] ACPI: Disambiguate processor declaration type Len Brown
2008-11-07  4:45   ` [PATCH 03/10] ACPI: Behave uniquely based on processor declaration definition type Len Brown
2008-11-07  4:45   ` [PATCH 04/10] ACPI: 80 column adherence and spelling fix (no functional change) Len Brown
2008-11-07  4:45   ` [PATCH 05/10] Hibernate: Call platform_begin before swsusp_shrink_memory Len Brown
2008-11-07  4:45   ` [PATCH 06/10] ACPI hibernate: Add a mechanism to save/restore ACPI NVS memory Len Brown
2008-11-07  4:45   ` [PATCH 07/10] x86 hibernate: Mark ACPI NVS memory region at startup Len Brown
2008-11-07  4:45   ` [PATCH 08/10] ACPI hibernate: Introduce new kernel parameter acpi_sleep=s4_nonvs Len Brown
2008-11-07  4:45   ` [PATCH 09/10] compal-laptop: use rfkill switch subsystem Len Brown
2008-11-07  4:45   ` [PATCH 10/10] x86, ACPI: default to reboot via ACPI (again) Len Brown
2008-11-07  7:25     ` Ingo Molnar
2008-11-08  1:41       ` Len Brown
2008-11-08  6:30         ` Andrey Borzenkov
2008-11-08  7:12           ` Len Brown
2008-11-08  7:50             ` Andrey Borzenkov
2008-11-08 11:59               ` Ingo Molnar
2008-11-09  9:55                 ` Avi Kivity
2008-11-09 10:00                   ` H. Peter Anvin
2008-11-10  8:39                     ` Ingo Molnar
2008-11-10  8:54                       ` Avi Kivity
2008-11-10  9:02                         ` Ingo Molnar
2008-11-11 18:26                           ` H. Peter Anvin
2008-11-11 20:29                             ` Eric W. Biederman
2008-11-11 20:44                               ` Ingo Molnar
2008-11-10 11:59                         ` Matthew Garrett
2008-11-10 11:57                       ` Matthew Garrett
2008-11-10 12:56                         ` Ingo Molnar
2008-11-10 13:00                           ` Matthew Garrett
2008-11-11 23:14                             ` Len Brown
2008-11-12  0:25                               ` Attempt rebooting via port CF9 if it seems to be available H. Peter Anvin
2008-11-12 18:49                                 ` Andrey Borzenkov
2008-11-12  0:27                               ` [PATCH 10/10] x86, ACPI: default to reboot via ACPI (again) Matthew Garrett
2008-11-12 11:58                               ` Ingo Molnar
2008-11-12 12:23                                 ` Avi Kivity
2008-11-13  3:23                                   ` Zhao Yakui
2008-11-13  3:18                                     ` H. Peter Anvin
2008-11-13  3:43                                       ` Zhao Yakui
2008-11-13  4:10                                     ` Eric W. Biederman
2008-11-13  4:34                                       ` H. Peter Anvin [this message]
2008-11-13  4:14                                     ` Eric W. Biederman
2008-11-13  5:29                                       ` Zhao Yakui
2008-11-13  5:25                                         ` H. Peter Anvin
2008-11-13  6:56                                         ` Eric W. Biederman
2008-11-13  6:58                                         ` Eric W. Biederman
2008-11-13  9:06                                           ` Zhao Yakui
2008-11-13 17:42                                             ` H. Peter Anvin
2008-11-14  1:29                                               ` Zhao Yakui
2008-11-14  1:22                                                 ` H. Peter Anvin
2008-11-14  1:49                                                   ` Zhao Yakui
2008-11-13  3:29                                 ` Stephen Rothwell
2008-11-08 12:40         ` Rafael J. Wysocki

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=491BAE43.5080309@zytor.com \
    --to=hpa@zytor.com \
    --cc=akpm@linux-foundation.org \
    --cc=arvidjaar@mail.ru \
    --cc=avi@redhat.com \
    --cc=ebiederm@xmission.com \
    --cc=ehabkost@redhat.com \
    --cc=len.brown@intel.com \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=mjg@redhat.com \
    --cc=sfr@canb.auug.org.au \
    --cc=tglx@linutronix.de \
    --cc=yakui.zhao@intel.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