Date: Sun, 6 Apr 2014 18:40:23 +0100
From: One Thousand Gnomes <gnomes@lxorguk.ukuu.org.uk>
To: Matthew Garrett <mjg59@srcf.ucam.org>
Cc: "Li, Aubrey" <aubrey.li@linux.intel.com>, Ingo Molnar <mingo@kernel.org>,
        hpa@zytor.com, linux-kernel@vger.kernel.org,
        torvalds@linux-foundation.org, rostedt@goodmis.org, tglx@linutronix.de,
        linux-tip-commits@vger.kernel.org,
        Adam Williamson <awilliam@redhat.com>
Subject: Re: [tip:x86/reboot] [PATCH] x86: Try the BIOS reboot method before
 the PCI reboot method
Message-ID: <20140406184023.1e31412d@alan.etchedpixels.co.uk>
In-Reply-To: <20140404151359.GB12370@srcf.ucam.org>
References: <20140404064120.GB11877@gmail.com>
	<tip-d122ab8d224a59aa0c3e4ba3540c6ab99556c3e3@git.kernel.org>
	<20140404080006.GB6944@gmail.com>
	<533EB5F1.8060002@linux.intel.com>
	<20140404151359.GB12370@srcf.ucam.org>
Organization: Intel Corporation
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org

On Fri, 4 Apr 2014 16:13:59 +0100
Matthew Garrett <mjg59@srcf.ucam.org> wrote:

> On Fri, Apr 04, 2014 at 09:38:57PM +0800, Li, Aubrey wrote:
> 
> > I don't think there is enough justification to revert the patch. We
> > introduce EFI into the default list, we didn't see any reason to remove
> > it out so far.
> 
> We should be using the EFI method, yes. But it seems that CF9 isn't 
> safe, so we should drop that.
>

Windows does not use CF9 and there are known, documented cases where CF9
will not work reliably. That includes reliable reboot on some Baytrail
systems. (Erratum VLT60 and VLT62).

So we really shouldn't be doing 0xCF9 except as a last resort if we are
still alive and have been through the official reboot methods.

Alan

--
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/