linux-kernel - Re: [PATCH] x86/reboot: Add Dell Optiplex 7450 AIO 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 for Android: free password hash cracker in your pocket

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

Message-ID: <20160714125816.GA21281@gmail.com>
Date:	Thu, 14 Jul 2016 14:58:17 +0200
From:	Ingo Molnar <mingo@...nel.org>
To:	Alex Hung <alex.hung@...onical.com>
Cc:	tglx@...utronix.de, mingo@...hat.com, hpa@...or.com,
	x86@...nel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] x86/reboot: Add Dell Optiplex 7450 AIO reboot quirk


* Alex Hung <alex.hung@...onical.com> wrote:

> Dell Optiplex 7450 AIO works with BOOT_ACPI; however, the quirk for
> "OptiPlex 745" changes its boot method to BOOT_BIOS and causes 7450 AIO
> hangs when rebooting; as a result, 7450 AIO is appended to overwrite
> BOOT_BIOS by BOOT_ACPI in order not to break the original 745 series
> 
> Signed-off-by: Alex Hung <alex.hung@...onical.com>
> ---
>  arch/x86/kernel/reboot.c | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)
> 
> diff --git a/arch/x86/kernel/reboot.c b/arch/x86/kernel/reboot.c
> index b7e1d5f..b404d05 100644
> --- a/arch/x86/kernel/reboot.c
> +++ b/arch/x86/kernel/reboot.c
> @@ -55,6 +55,19 @@ bool port_cf9_safe = false;
>   */
>  
>  /*
> + * Some machines require the "reboot=a" commandline options
> + */
> +static int __init set_acpi_reboot(const struct dmi_system_id *d)
> +{
> +	if (reboot_type != BOOT_ACPI) {
> +		reboot_type = BOOT_ACPI;
> +		pr_info("%s series board detected. Selecting %s-method for reboots.\n",
> +			d->ident, "ACPI");
> +	}
> +	return 0;
> +}
> +
> +/*
>   * Some machines require the "reboot=b" or "reboot=k"  commandline options,
>   * this quirk makes that automatic.
>   */
> @@ -395,6 +408,14 @@ static struct dmi_system_id __initdata reboot_dmi_table[] = {
>  			DMI_MATCH(DMI_PRODUCT_NAME, "Dell XPS710"),
>  		},
>  	},
> +	{	/* Handle problems with rebooting on Dell Optiplex 7450 AIO */
> +		.callback = set_acpi_reboot,
> +		.ident = "Dell OptiPlex 7450 AIO",
> +		.matches = {
> +			DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> +			DMI_MATCH(DMI_PRODUCT_NAME, "OptiPlex 7450 AIO"),
> +		},
> +	},

I think it would be better to limit the original "Dell Optiplex 745" quirk to that 
machine alone instead of quirking the quirk on one of the (known) newer models 
with longer strings.

Thanks,

	Ingo