linux-kernel - Re: DMI_MATCH patch in reboot.c for SFF Dell OptiPlex 745

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <Pine.GSO.4.64.0706050918350.24899@holyrood.ed.ac.uk>
Date:	Tue, 5 Jun 2007 09:42:46 +0100 (BST)
From:	"James E. Jarvis" <jjarvis@...ffmail.ed.ac.uk>
To:	Robert Hancock <hancockr@...w.ca>
cc:	Linux-Kernel@...r.kernel.org,
	Kenneth MacDonald <kenneth.macdonald@...ac.uk>
Subject: Re: DMI_MATCH patch in reboot.c for SFF Dell OptiPlex 745 - fixes
 hang on reboot

On Mon, 4 Jun 2007, Robert Hancock wrote:

> James Jarvis wrote:
>> I trust this is the right list...
>> 
>> The following patch enables reboot through BIOS on the Dell Optiplex 745 
>> Small Form Factor base, on which reboot hangs. The larger form factor does 
>> not require this, hence the match on DMI_BOARD_NAME.
>> 
>> --- arch/i386/kernel/reboot.c.orig      2007-06-01 12:22:53.000000000 
>> +0100
>> +++ arch/i386/kernel/reboot.c   2007-06-04 16:53:36.000000000 +0100
>> @@ -104,6 +104,15 @@
>>                        DMI_MATCH(DMI_PRODUCT_NAME, "PowerEdge 300/"),
>>                },
>>        },
>> +       {       /* Handle problems with rebooting on Dell Optiplex 745's 
>> SFF*/
>> +               .callback = set_bios_reboot,
>> +               .ident = "Dell OptiPlex 745",
>> +               .matches = {
>> +                       DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
>> +                       DMI_MATCH(DMI_PRODUCT_NAME, "OptiPlex 745"),
>> +                       DMI_MATCH(DMI_BOARD_NAME, "0WF810"),
>> +               },
>> +       },
>>        {       /* Handle problems with rebooting on Dell 2400's */
>>                .callback = set_bios_reboot,
>>                .ident = "Dell PowerEdge 2400",
>
> This blacklist seems to be growing. Isn't there something better we can do 
> here? Can we just reboot through the BIOS by default, is that really broken 
> on some systems?

Agreed, it is ugly. However I don't like leavng the determination of 
whether the system board will only reboot through BIOS to the end user. 
Whilst using the kernel boot argument "reboot=b" is an alternative, it 
works only _if_ one knows the platform in advance. However our boot media 
(and indeed live-cds generally) are used such that this is not practical.

As for whether rebooting through BIOS by default would solve this matter - 
I don't know. It may break on some machines. Given that EFI may (???) 
replace traditional BIOS in the future is it worth re-engineering this?

James Jarvis
Uinversity of Edinburgh
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/