mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Thomas Gleixner <tglx@linutronix.de>
To: 刘术高 <liushugao@xiaomi.com>, 朱恺乾 <zhukaiqian@xiaomi.com>,
	"Daniel Lezcano" <daniel.lezcano@linaro.org>,
	张嘉伟 <zhangjiawei8@xiaomi.com>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	王韬 <lingyue@xiaomi.com>, 熊亮 <xiongliang@xiaomi.com>,
	"isaacmanjarres@google.com" <isaacmanjarres@google.com>,
	"Frederic Weisbecker" <frederic@kernel.org>,
	"Anna-Maria Behnsen" <anna-maria@linutronix.de>,
	梁伟鹏 <weipengliang@xiaomi.com>, 翁金飞 <wengjinfei@xiaomi.com>
Subject: Re: 回复: [External Mail][PATCH] tick/broadcast: Plug clockevents replacement race
Date: Sat, 05 Sep 2026 22:59:54 +0200	[thread overview]
Message-ID: <87o6eb4do5.ffs@fw13> (raw)
In-Reply-To: <6eb4fac303f4457f9686510664e79bff@xiaomi.com>

On Fri, Aug 28 2026 at 09:08, 刘术高 wrote:

Can you please adhere to the kernel mailing list netiquette:

  https://people.kernel.org/tglx/notes-about-netiquette

> We have verified this patch on our platform (kernel v6.18 / Android 17)
> and can confirm it resolves the race condition issue. The problem is that
> tick_install_broadcast_device() calls clockevents_exchange_device() to
> move the old device to the released list (setting it to DETACHED state),
> then updates tick_broadcast_device.evtdev to the new device — but this
> sequence is not protected by tick_broadcast_lock. The cpuidle/hrtimer
> paths can observe the old device in DETACHED state during this window,
> leading to unexpected behavior.
>
> After applying the patch, the issue no longer reproduces under our
> concurrent CPU idle + clockevent device registration testing.
>
> Could you please share the expected timeline for merging this fix into
> the mainline kernel? We have downstream products depending on this fix
> and would appreciate it if the merge could be expedited.
>
> Test environment:
>   - Kernel: 6.18
>   - Platform: Android 17
>   - Test: Concurrent CPU idle + clockevent device registration

and to the documented process in Documentation/process/* which clearly
states that the way to confirm that a bug is fixed is expressed by a

Tested-by: ....

tag, which allows tools to pick it up automatically.

Thanks,

        tglx

  parent reply	other threads:[~2026-09-05 20:59 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <042520850d394f0bb0004a226db63d0d@xiaomi.com>
2024-06-27 11:26 ` Race condition when replacing the broadcast timer Thomas Gleixner
2024-06-28  1:59   ` [External Mail]Re: " 朱恺乾
2024-06-28  7:22     ` Daniel Lezcano
2024-07-01  2:11       ` 朱恺乾
2024-07-10 20:30         ` Thomas Gleixner
2024-07-29 11:44           ` Thomas Gleixner
2024-08-12 14:19             ` [PATCH] tick/broadcast: Plug clockevents replacement race Thomas Gleixner
2024-09-25  9:45               ` Anna-Maria Behnsen
2024-10-17 16:16               ` Frederic Weisbecker
2026-09-05 20:54                 ` Thomas Gleixner
2026-08-28  9:08               ` 回复: [External Mail][PATCH] " 刘术高
2026-09-03  6:33                 ` 刘术高
2026-09-03 15:12                   ` Bradley Morgan
2026-09-05 21:21                     ` Thomas Gleixner
2026-09-05 21:24                       ` Bradley Morgan
2026-09-05 22:00                         ` Thomas Gleixner
2026-09-05 22:06                           ` Bradley Morgan
2026-09-05 23:15                             ` Thomas Gleixner
2026-09-06 10:40                               ` Bradley Morgan
2026-09-05 22:05                         ` Thomas Gleixner
2026-09-05 21:19                   ` Thomas Gleixner
2026-09-07  6:24                     ` 回复: " 刘术高
2026-09-05 20:59                 ` Thomas Gleixner [this message]
2026-09-07  9:15               ` [tip: timers/urgent] " tip-bot2 for Thomas Gleixner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87o6eb4do5.ffs@fw13 \
    --to=tglx@linutronix.de \
    --cc=anna-maria@linutronix.de \
    --cc=daniel.lezcano@linaro.org \
    --cc=frederic@kernel.org \
    --cc=isaacmanjarres@google.com \
    --cc=lingyue@xiaomi.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=liushugao@xiaomi.com \
    --cc=weipengliang@xiaomi.com \
    --cc=wengjinfei@xiaomi.com \
    --cc=xiongliang@xiaomi.com \
    --cc=zhangjiawei8@xiaomi.com \
    --cc=zhukaiqian@xiaomi.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®