Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Mime-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset=UTF-8
Date:   Fri, 11 Aug 2023 19:49:15 +0300
Message-Id: <CUPVBIYZ2B34.P1LAY8LPFQEQ@suppilovahvero>
To:     "Takashi Iwai" <tiwai@suse.de>
Cc:     <linux-integrity@vger.kernel.org>,
        "Linus Torvalds" <torvalds@linux-foundation.org>,
        <stable@vger.kernel.org>,
        "Lino Sanfilippo" <l.sanfilippo@kunbus.com>,
        <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] tpm/tpm_tis: Disable interrupts categorically for
 Lenovo
From:   "Jarkko Sakkinen" <jarkko@kernel.org>
References: <20230810182433.518523-1-jarkko@kernel.org>
 <87zg2yrqmy.wl-tiwai@suse.de> <87wmy2rqjy.wl-tiwai@suse.de>
In-Reply-To: <87wmy2rqjy.wl-tiwai@suse.de>
Precedence: bulk

On Fri Aug 11, 2023 at 10:22 AM EEST, Takashi Iwai wrote:
> On Fri, 11 Aug 2023 09:21:09 +0200,
> Takashi Iwai wrote:
> >=20
> > On Thu, 10 Aug 2023 20:24:33 +0200,
> > Jarkko Sakkinen wrote:
> > >=20
> > > By large most of the entries in tpm_tis_dmi_table[] are for Lenovo la=
ptops,
> > > and they keep on coming. Therefore, disable IRQs categorically for Le=
novo.
> > >=20
> > > Fixes: e644b2f498d2 ("tpm, tpm_tis: Enable interrupt test")
> > > Cc: <stable@vger.kernel.org> # v6.4+
> > > Reported-by: "Takashi Iwai" <tiwai@suse.de>
> > > Closes: https://lore.kernel.org/linux-integrity/87il9qhxjq.wl-tiwai@s=
use.de/
> > > Signed-off-by: Jarkko Sakkinen <jarkko@kernel.org>
> > > ---
> > > This will be included into v6.5-rc6 PR, as long as Takashi ack's it. =
I'm
> > > planning to send tomorrow morning (GMT+3).
> >=20
> > Feel free to take my ack:
> >=20
> > Acked-by: Takashi Iwai <tiwai@suse.de>
> >=20
> > I'll build a test kernel and ask reporters for testing the fix, too.
>
> Also, it's worth to put the link to the original bug report:
>
> Link: https://bugzilla.suse.com/show_bug.cgi?id=3D1213779

These days it is recommended afaik to use "Closes:" so I used that
(checkpatch even complains if it does not exist when you have a
reported-by tag).

BR, Jarkko