linux-kernel - Re: [PATCH] iio: inkern: fix channel read regression

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: <Zl1vttlXz3FRVyYS@hovoldconsulting.com>
Date: Mon, 3 Jun 2024 09:24:38 +0200
From: Johan Hovold <johan@...nel.org>
To: Jonathan Cameron <jic23@...nel.org>
Cc: Johan Hovold <johan+linaro@...nel.org>,
	Lars-Peter Clausen <lars@...afoo.de>, linux-iio@...r.kernel.org,
	linux-kernel@...r.kernel.org, Nuno Sa <nuno.sa@...log.com>
Subject: Re: [PATCH] iio: inkern: fix channel read regression

Hi Jonathan,

On Sat, Jun 01, 2024 at 02:21:47PM +0100, Jonathan Cameron wrote:
> On Thu, 30 May 2024 09:44:16 +0200
> Johan Hovold <johan+linaro@...nel.org> wrote:
> 
> > A recent "cleanup" broke IIO channel read outs and thereby thermal
> > mitigation on the Lenovo ThinkPad X13s by returning zero instead of the
> > expected IIO value type in iio_read_channel_processed_scale():
> > 
> > 	thermal thermal_zone12: failed to read out thermal zone (-22)
> > 
> > Fixes: 3092bde731ca ("iio: inkern: move to the cleanup.h magic")
> > Cc: Nuno Sa <nuno.sa@...log.com>
> > Signed-off-by: Johan Hovold <johan+linaro@...nel.org>

> In meantime, Nuno please take another look at these and see if
> we have additional problem cases like this.  Given the patch
> queue I have and a busy few days it will be a while before I
> get to it but I'll try and take a close look soon as well.

Please consider getting this one into mainline as soon as possible as it
breaks thermal mitigation, which can cause literal skin burns, on all
Qualcomm platforms (and probably other platforms too).

> Longer term, in my view the readability and chance of bugs
> is reduced, but churn always introduces the possibility of
> issues like this in the short term :(

I just worry about some of the cleanup.h conversion I've seen where
inexperienced developers potentially break tested and reviewed code for
something which is often not very readable and for very little gain.

Johan