linux-kernel - Re: [PATCH] hwmon: (dell-smm) Add Dell OptiPlex 7060 to DMI table

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: <20240628215704.i6ohbuz2zgegr27p@pali>
Date: Fri, 28 Jun 2024 23:57:04 +0200
From: Pali Rohár <pali@...nel.org>
To: Armin Wolf <W_Armin@....de>
Cc: jdelvare@...e.com, linux@...ck-us.net, linux-hwmon@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] hwmon: (dell-smm) Add Dell OptiPlex 7060 to DMI table

On Friday 28 June 2024 23:47:23 Armin Wolf wrote:
> The BIOS on this machine is buggy and will in some cases return
> an error when trying to get the fan state, but reading of the
> RPM values and the temperature sensors still works.

Does this error affects machine usage (e.g. freeze of CPU or some
erratic fan behavior)? Or just kernel does not receive fan state and is
unable to report meaningful value to userspace?

> 
> Closes: https://github.com/vitorafsr/i8kutils/issues/38
> Signed-off-by: Armin Wolf <W_Armin@....de>
> ---
>  Documentation/hwmon/dell-smm-hwmon.rst | 2 ++
>  drivers/hwmon/dell-smm-hwmon.c         | 7 +++++++
>  2 files changed, 9 insertions(+)
> 
> diff --git a/Documentation/hwmon/dell-smm-hwmon.rst b/Documentation/hwmon/dell-smm-hwmon.rst
> index 977263cb57a8..74905675d71f 100644
> --- a/Documentation/hwmon/dell-smm-hwmon.rst
> +++ b/Documentation/hwmon/dell-smm-hwmon.rst
> @@ -360,6 +360,8 @@ Firmware Bug                                            Affected Machines
>  ======================================================= =================
>  Reading of fan states return spurious errors.           Precision 490
> 
> +                                                        OptiPlex 7060
> +
>  Reading of fan types causes erratic fan behaviour.      Studio XPS 8000
> 
>                                                          Studio XPS 8100
> diff --git a/drivers/hwmon/dell-smm-hwmon.c b/drivers/hwmon/dell-smm-hwmon.c
> index 48a81c64f00d..c75bfe93f2f6 100644
> --- a/drivers/hwmon/dell-smm-hwmon.c
> +++ b/drivers/hwmon/dell-smm-hwmon.c
> @@ -1263,6 +1263,13 @@ static const struct dmi_system_id i8k_dmi_table[] __initconst = {
>  			DMI_MATCH(DMI_PRODUCT_NAME, "MP061"),
>  		},
>  	},
> +	{
> +		.ident = "Dell OptiPlex 7060",
> +		.matches = {
> +			DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> +			DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "OptiPlex 7060"),
> +		},
> +	},
>  	{
>  		.ident = "Dell Precision",
>  		.matches = {
> --
> 2.39.2
>