Re: [PATCH v6] watchdog/hardlockup: simplify perf event probe and remove per-cpu dependency
From: Qiliang Yuan
Date: Wed Jan 28 2026 - 01:14:10 EST
Hi Doug,
On Tue, Jan 27, 2026 at 8:08 PM Doug Anderson <dianders@xxxxxxxxxxxx> wrote:
> One last note is that your Signed-off-by tags don't match. When I
> apply your patch, I see:
>
> Author: Qiliang Yuan <realwujing@xxxxxxxxx>
>
> ...but your Signed-off-by is your "@chinatelecom.cn" address. That's
> generally not okay. You need to do something to make those match.
Thank you for pointing this out. I've updated my git configuration and
the patch to ensure the Signed-off-by matches the Author's Gmail address.
> Given our newest understanding, the cpu_hotplug_disable() isn't truly
> needed anymore, but I'm still good keeping it.
Since the probe is now fully stateless and no longer touches per-cpu
variables, I've opted to remove the redundant cpu_hotplug_disable() in
v7 to make the code cleaner, as you suggested it wasn't strictly necessary.
v7 Link: https://lore.kernel.org/all/20260128060833.1715622-1-realwujing@xxxxxxxxx/
Thanks again for your detailed review and guidance!
Best regards,
Qiliang