linux-kernel - Re: [PATCH v2] thermal/drivers/int340x: Fix RFIM mailbox write commands

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: <CAJZ5v0hj83Bmw0-Yhb22xQHGauRWL-RADkiKQRuvcpYi3A45WA@mail.gmail.com>
Date:   Thu, 30 Dec 2021 16:44:30 +0100
From:   "Rafael J. Wysocki" <rafael@...nel.org>
To:     srinivas pandruvada <srinivas.pandruvada@...ux.intel.com>,
        Sumeet Pawnikar <sumeet.r.pawnikar@...el.com>
Cc:     "Rafael J. Wysocki" <rafael@...nel.org>,
        Daniel Lezcano <daniel.lezcano@...aro.org>,
        Linux PM <linux-pm@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Stable <stable@...r.kernel.org>
Subject: Re: [PATCH v2] thermal/drivers/int340x: Fix RFIM mailbox write commands

On Thu, Dec 23, 2021 at 4:31 PM srinivas pandruvada
<srinivas.pandruvada@...ux.intel.com> wrote:
>
> On Thu, 2021-12-23 at 15:12 +0530, Sumeet Pawnikar wrote:
> > The existing mail mechanism only supports writing of workload types.
> > But mailbox command for RFIM (cmd = 0x08) also requires write operation
> > which was ignored. This results in failing to store RFI restriction.
> > This requires enhancing mailbox writes for non workload commands also.
> > So, remove the check for MBOX_CMD_WORKLOAD_TYPE_WRITE in mailbox write,
> > with this other write commands also can be supoorted. But at the same
> > time, we have to make sure that there is no impact on read commands,
> > by not writing anything in mailbox data register.
> > To properly implement, add two separate functions for mbox read and
> > write
> > command for processor thermal workload command type. This helps to
> > differentiate the read and write workload command types while sending
> > mbox
> > command.
> >
> > Fixes: 5d6fbc96bd36 ("thermal/drivers/int340x: processor_thermal:
> > Export additional attributes")
> > Signed-off-by: Sumeet Pawnikar <sumeet.r.pawnikar@...el.com>
> > Cc: stable@...r.kernel.org # 5.14+
> Acked-by: Srinivas Pandruvada <srinivas.pandruvada@...ux.intel.com>

Applied (with some edits in the changelog) as 5.17 material, thanks!