Re: CONFIG_BREAK_MY_MACHINE

Hans de Goede
2007-05-17T08:23:49Z

Jean Delvare wrote:
> On Tue, 15 May 2007 20:31:00 +0000, Pavel Machek wrote:
>> Hi!
>>
>>>> Hardware Monitoring support --->
>>>> <*> Hardware Monitoring support
>>>> <*> Abit uGuru
>>>> <M> VIA686A
>>>> <*> IBM Hard Drive Active Protection System (hdaps)
>>>>
>>>> But I'm quite sure that the only module used is VIA686A (I'm
>>>> rebuilding to confirm).
>>> This is a rather bad idea to build the abituguru and hdaps drivers into
>>> your kernel if you don't have these devices. Especially abituguru, as
>>> it does arbitrary port probing.
>> hdaps should be safe (DMI based whitelist, no?)
>
> Correct.
>
>> If abitguru breaks random machines, we probably should DMI whitelist,
>> too.
>
> I never said it was breaking machines, just that it was accessing
> arbitrary I/O ports.
>
> This was already discussed with the driver's author (Hans de Goede,
> Cc'd) and I think we agreed on the principle, but it didn't happen yet.
> This device only exists on Abit motherboards so it would be easy enough
> to check the DMI vendor. A more detailed white list is also possible,
> but I'm not insisting on it.
>
> The driver could also be made to depend on X86, as this is the only
> architecture where it is useful.
>
> Hans, can you please submit a patch doing this?
>

As you can see from the CC I've send you, I've just mailed my private army of
testers / known uguru users, requesting them to send me the output of dmidecode
on their uguru mobo's. Once I have this info, I'll see if there is some
consistency in the manufacturer DMI field.

I've mailed them, since the DMI info is needed anyways to put uguru detection
in sensors-detect, but I'm not sure I want to add this to the driver too. Many
other ISA drivers do io probe's too, if a users decides to start loading random
drivers, he is asking for problems, should we really even try to protect
against this?

Regards,

Hans