linux-kernel - Re: [PATCH v4 0/2] thermal: rcar_gen3

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] [day] [month] [year] [list]
Message-ID: <20190510155005.GA499@vmlxhi-102.adit-jv.com>
Date:   Fri, 10 May 2019 17:50:05 +0200
From:   Eugeniu Rosca <erosca@...adit-jv.com>
To:     Niklas Söderlund <niklas.soderlund@...natech.se>
CC:     Jiada Wang <jiada_wang@...tor.com>, <rui.zhang@...el.com>,
        <edubezval@...il.com>, <daniel.lezcano@...aro.org>,
        <linux-pm@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <horms+renesas@...ge.net.au>, <geert+renesas@...der.be>,
        <sergei.shtylyov@...entembedded.com>,
        <marek.vasut+renesas@...il.com>,
        <kuninori.morimoto.gx@...esas.com>, <hien.dang.eb@...esas.com>,
        <fabrizio.castro@...renesas.com>, <dien.pham.ry@...esas.com>,
        <biju.das@...renesas.com>, <george_davis@...tor.com>,
        <joshua_frkuska@...tor.com>,
        Eugeniu Rosca <roscaeugeniu@...il.com>,
        Eugeniu Rosca <erosca@...adit-jv.com>
Subject: Re: [PATCH v4 0/2] thermal: rcar_gen3_thermal: fix IRQ issues

Hi Niklas,

On Fri, May 10, 2019 at 01:36:08PM +0200, Niklas Söderlund wrote:
> Hi Eugeniu,
> 
> On 2019-05-10 12:42:31 +0200, Eugeniu Rosca wrote:
> > Hi Niklas,
> > 
> > On Wed, May 08, 2019 at 01:54:03AM +0200, Niklas Söderlund wrote:
> > > Hi Jiada,
> > [..]
> > > I really like this series, nice work.
> > > 
> > > Tested-by: Niklas Söderlund <niklas.soderlund+renesas@...natech.se>
> > > Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@...natech.se>
> > 
> > Is there anything off-the-shelf available for testing the rcar3
> > thermal driver, to avoid reinventing the wheel via
> > https://patchwork.kernel.org/cover/10913163/#22602335
> 
> Not that I know of, unfortunately :-(
> 
> I have a private home hacked testing framework (don't we all?) based on 
> tcl+expect where I have two basic tests for rcar_gen3_thermal. I'm 
> willing to share the tests if you by chance want them, but be warned 
> that they are highly specialised for my needs and I'm reluctant to 
> publish my whole hack tool as it just a ugly hack ;-)
> 
> On a high level the tests I have are
> 
> 1. thermal-load
>     Generates load on target and observes the temperature is increased 
>     using the /sys/class/thermal/thermal_zone*/temp" interface. This 
>     seems similar to the test case your reference using stress-ng.
> 
> 2. thermal-cooling
>     Emulate the passive trip point temperatures using the 
>     /sys/class/thermal/*/emul_temp interface and observe that the 
>     specified cooling state is achieved.
> 
> I should add a third test to make sure IRQ fires but this is just a pet 
> project for me so maybe I will get around to it sometime...
> 
> If you know of anything around to test thermal drivers or if you create 
> something please let me know so I can add it to my tests. And let me 
> know if you want my hacks for inspiration for your own testing.

Thanks for this summary. It would be definitely convenient to have
a set of tests covering the most important features of the driver.

I was particularly thinking of the test procedure in light of below:
 - I still can reproduce a few UBSAN (signed integer overflow) and
   KASAN (use-after-free) reports with the most recent vanilla driver.
 - There are a couple of thermal commits in rcar-3.9.x pending for
   mainline submission:

https://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas-bsp.git/commit/?id=fe7d0d1c77f9 ("thermal: rcar_gen3_thermal: Use FUSE values if they are available")
https://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas-bsp.git/commit/?id=2776ccd63649 ("thermal: rcar_gen3_thermal: Fix interrupt count issue")
https://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas-bsp.git/commit/?id=9146af785f41 ("thermal: rcar_gen3_thermal: Enable selection between polling/interrupt mode")
https://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas-bsp.git/commit/?id=55b262766ec2 ("thermal: rcar_gen3_thermal: PIO-INT can be selected for each TSC separately")
https://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas-bsp.git/commit/?id=d323d9de0683 ("thermal: rcar_gen3_thermal: Add support for r8a77990")
https://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas-bsp.git/commit/?id=fb8efb8bac29 ("thermal: rcar_gen3_thermal: Fix interrupts are not raised issue on E3")
https://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas-bsp.git/commit/?id=5627c42a1bd5 ("thermal: rcar_gen3_thermal: Use DIV_ROUND_CLOSEST correctly as its description")
https://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas-bsp.git/commit/?id=d4e41702e53b ("thermal: rcar_gen3_thermal: [H3/M3N] Update calculation formula due to HW evaluation")
https://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas-bsp.git/commit/?id=958bd36e03b7 ("thermal: rcar_gen3_thermal: [E3] Update calculation formula due to HW evaluation")

Long story short, I think we will review more thermal commits in
hopefully not so distant future and it would be helpful to reach some
common understanding what kind of testing the new patches should pass.

Your summary already gives some insight in that direction. Thanks.

-- 
Best Regards,
Eugeniu.