Linux Documentation
 help / color / mirror / Atom feed
From: Thomas Gleixner <tglx@linutronix.de>
To: Sehee Jeong <sehee1.jeong@samsung.com>,
	anna-maria@linutronix.de, frederic@kernel.org, corbet@lwn.net
Cc: linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org,
	sehee1.jeong@samsung.com
Subject: Re: timers/migration: add 'notmigr' kernel parameter to disable timer migration
Date: Wed, 03 Sep 2025 17:37:12 +0200	[thread overview]
Message-ID: <87v7lzy2c7.ffs@tglx> (raw)
In-Reply-To: <20250807064849.3988-1-sehee1.jeong@samsung.com>

On Thu, Aug 07 2025 at 15:48, Sehee Jeong wrote:
> On heterogeneous systems with big.LITTLE architectures, timer migration
> may cause timers from little cores to run on big cores, or vice versa,
> because core type differences are not taken into account in the current
> timer migration logic.

And what's the reason why this should be done?

> This can be undesirable in systems that require strict power
> management, predictable latency, or core isolation.

Undesirable is not really a technical argument. Can you please describe
the actual problems instead of handwaving?

> This patch does not attempt to solve the structural limitation,

# git grep "This patch" Documentation/process/

> but provides a workaround by introducing an optional early boot parameter:
>
>     notmigr
>
> When specified, timer migration initialization is skipped entirely.

I agree with Frederic that this naming is suboptimal and should be
tmigr=[on/off].

But aside of that, disabling timer migration causes other power related
issues as it keeps timers always CPU local and therefore fails to let
idle CPUs stay truly idle.

So instead of just having a lazy work around and disabling everything,
this should really be addressed properly. Though without a proper
technical problem description, that's pretty much impossible.

Thanks,

        tglx



      parent reply	other threads:[~2025-09-03 15:37 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CGME20250807064855epcas2p3079c241a4da07d478e713021ca488d13@epcas2p3.samsung.com>
2025-08-07  6:48 ` timers/migration: add 'notmigr' kernel parameter to disable timer migration Sehee Jeong
2025-09-02 14:29   ` Frederic Weisbecker
2025-09-03 15:37   ` Thomas Gleixner [this message]

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=87v7lzy2c7.ffs@tglx \
    --to=tglx@linutronix.de \
    --cc=anna-maria@linutronix.de \
    --cc=corbet@lwn.net \
    --cc=frederic@kernel.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sehee1.jeong@samsung.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