linux-kernel - Re: CONFIG_SENSORS_OCC_P8

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]

Date:   Sun, 4 Dec 2022 13:32:50 -0800
From:   Guenter Roeck <linux@...ck-us.net>
To:     Borislav Petkov <bp@...en8.de>
Cc:     Eddie James <eajames@...ux.vnet.ibm.com>,
        linux-hwmon@...r.kernel.org, lkml <linux-kernel@...r.kernel.org>
Subject: Re: CONFIG_SENSORS_OCC_P8_I2C

On 12/4/22 11:13, Borislav Petkov wrote:
> On Sun, Dec 04, 2022 at 10:37:09AM -0800, Guenter Roeck wrote:
>> I don't immediately see why this should depend on a P8 configuration.
>> The driver runs on a BMC, after all. My understanding is that a BMC can be
>> of any architecture. Is there a guarantee that there are and never will be
>> any x86 BMCs, and/or that BMCs are limited to a certain subset of
>> architectures ?
> 
> Well, then it should depend on the architectures which are currently
> running BMCs for P8 machines. ARM maybe?
> 
> At least on x86 the majority of BMCs are ARM-based AFAIK.
> 

We are not typically that restrictive in the hwmon subsystem. That isn't done
for many of the other drivers which are typically only used on x86 either.
Why is that suddenly a problem ? Just don't enable it if you don't need it.

Anyway,

commit 58c0399db8bf6bf88c7580b742b0cb45e02f55d5
Author:     Jeremy Kerr <jk@...econstruct.com.au>
AuthorDate: Mon Oct 24 16:15:27 2022 +0800
Commit:     Guenter Roeck <linux@...ck-us.net>
CommitDate: Thu Dec 1 09:21:06 2022 -0800

     hwmon: (occ) OCC sensors aren't arch-specific

     Commit c112d75840fb ("hwmon: OCC drivers are ARM-only") made the OCC
     sensor drivers not selectable on powerpc64:

       These drivers are for a BMC inside PowerPC servers. The BMC runs on
       ARM hardware, so only propose the drivers on this architecture, unless
       build-testing.

     ... but we now have a powerpc64 BMC (still for a powerpc64 host), so
     drop the `depends on` that excludes building for this platform.

     Signed-off-by: Jeremy Kerr <jk@...econstruct.com.au>
     Acked-by: Joel Stanley <joel@....id.au>
     Link: https://lore.kernel.org/r/20221024081527.3842565-1-jk@codeconstruct.com.au
     Signed-off-by: Guenter Roeck <linux@...ck-us.net>

I really don't want to end up in a situation where I constantly have
to deal with changing architecture support, so I would strongly prefer
to keep it that way unless it is a real problem.

Guenter