Re: [PATCH] cpufreq: intel_pstate: Make hwp_notify_lock a raw spinlock

[xiao sheng wen(肖盛文) <atzlinux@xxxxxxxx>]

在 2024/9/28 19:51, Uwe Kleine-König 写道:
> Hello,
>
> On Fri, Sep 27, 2024 at 03:47:08AM -0700, srinivas pandruvada wrote:
> > On Thu, 2024-09-19 at 10:11 +0200, Uwe Kleine-König wrote:
> > > notify_hwp_interrupt() is called via sysvec_thermal() ->
> > > smp_thermal_vector() -> intel_thermal_interrupt() in hard irq
> > > context.
> > > For this reason it must not use a simple spin_lock that sleeps with
> > > PREEMPT_RT enabled. So convert it to a raw spinlock.
> > >
> > > Reported-by: xiao sheng wen <atzlinux@xxxxxxxx>
> > > Link: https://bugs.debian.org/1076483
> > > Signed-off-by: Uwe Kleine-König <ukleinek@xxxxxxxxxx>
> >
> > Missing Tested-by?
>
> Neither I nor anybody else did test that patch before I sent it to the
> list (apart from a build test). I guess xiao sheng wen might have
> replied with a Tested-by tag.
Yes, I had tested this patch on my notebook(base on Debian kernel 
6.10.11-1 (2024-09-22) source code).
Work perfect!

This bug exist in 6.9.9-rt-amd64 #1  Debian 6.9.9-1:
https://linux-hardware.org/?probe=d57f4d29a8&log=dmesg

This bug fixed in PREEMPT_RT atzlinux 6.10.11-1 (2024-09-22) with this 
patch:
https://linux-hardware.org/?probe=de5a44374a&log=dmesg

Tested-by: xiao sheng wen <atzlinux@xxxxxxxx>

> >      Acked-by: Srinivas Pandruvada <srinivas.pandruvada@xxxxxxxxxxxxxxx>
>
> Thanks
> Uwe

Regards,
--
肖盛文 xiao sheng wen
Debian QA page: 
https://qa.debian.org/developer.php?login=atzlinux%40sina.com
Debian salsa: https://salsa.debian.org/atzlinux-guest
GnuPG Public Key: 0x00186602339240CB

Attachment: OpenPGP_signature.asc
Description: OpenPGP digital signature