linux-kernel - Re: [PATCH v2] x86/reboot/quirks: Add GIGABYTE BRIX BXBT-2807 reboot quirk

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAJAOzpAC8V7iNnXDjdJS8Gu0pWsdZgy77ofcZftmfyOoOTDNeg@mail.gmail.com>
Date:   Thu, 10 Dec 2020 16:01:06 -0700
From:   Dan Nicholson <dbn@...lessos.org>
To:     Borislav Petkov <bp@...en8.de>
Cc:     Chris Chiu <chiu@...lessos.org>, tglx@...utronix.de,
        mingo@...hat.com, hpa@...or.com, peterz@...radead.org,
        Andy Lutomirski <luto@...nel.org>, x86@...nel.org,
        linux-kernel@...r.kernel.org, linux@...lessos.org,
        Dan Nicholson <nicholson@...lessos.org>,
        Jian-Hong Pan <jhp@...lessos.org>
Subject: Re: [PATCH v2] x86/reboot/quirks: Add GIGABYTE BRIX BXBT-2807 reboot quirk

On Thu, Dec 10, 2020 at 11:49 AM Borislav Petkov <bp@...en8.de> wrote:
>
> On Thu, Dec 10, 2020 at 12:19:46PM +0800, Chris Chiu wrote:
> > From: Dan Nicholson <nicholson@...lessos.org>
> >
> > The GIGABYTE BRIX BXBT-2807 always hangs with the normal acpi
> > reboot.
>
> Is that what the "hard disk crash" in the comment below, refers to?

Hmm, when I wrote this patch several years back there wasn't any hang
when rebooting. According to our internal bug tracker, you could hear
the disk head crashing on reboot. The workaround was to do a BIOS
reboot rather than an ACPI reboot. My original commit message was:

    x86/reboot: Use bios reboot for GIGABYTE BRIX BXBT-2807

    Using ACPI, the hard disk crashes on reboot. This seems to be avoided
    when using the BIOS to reboot.

I don't know if this became a hang with more recent kernels as
described in this commit message. Chris?

--
Dan