public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: David Laight <david.laight.linux@gmail.com>
To: Attila Fazekas <afazekas@redhat.com>
Cc: qyousef@layalina.io, dietmar.eggemann@arm.com,
	frederic@kernel.org, jstultz@google.com, juri.lelli@redhat.com,
	linux-kernel@vger.kernel.org, mingo@kernel.org,
	peterz@infradead.org, rostedt@goodmis.org, saravanak@google.com,
	tglx@linutronix.de, vincent.guittot@linaro.org
Subject: Re: [PATCH] Kconfig.hz: Change default HZ to 1000
Date: Sun, 2 Mar 2025 12:05:30 +0000	[thread overview]
Message-ID: <20250302120530.1e742e06@pumpkin> (raw)
In-Reply-To: <20250228103304.302289-1-afazekas@redhat.com>

On Fri, 28 Feb 2025 11:33:04 +0100
Attila Fazekas <afazekas@redhat.com> wrote:

> The 250Hz was set as a middle ground.
> 
> There are still workloads which sensitive to cache misses
> and to the time spent in ticks.
> 
> You still can loose ~1% performance with higher Hz,
> It might not sound much, but if you are running thousands
> of servers 1% loss can be a high cost.
> 
> There are many workloads out there where 100Hz is the better choice.

Are there are real issues with changing HZ to 1000, but adding an option
for the timer tick interval (in ms).
So 'jiffies' would always count milliseconds.
That would make is easy to make the actual 'clock tick' be boot time
selectable (or run-time if you get brave!).
The 'timer wheel' code would really need to work on actual ticks,
but I doubt nothing else cares.

That would allow HZ be the same for all architectures, even though
m68k might really want a 50Hz interrupt.

That is much better than any plan to make HZ a variable - which will
bloat code (and with divisions) and not be valid for static initialisers.

	David

      reply	other threads:[~2025-03-02 12:05 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-02-10  0:19 [PATCH] Kconfig.hz: Change default HZ to 1000 Qais Yousef
2025-02-12 14:50 ` Joel Fernandes
2025-02-16 16:11   ` Qais Yousef
2025-02-23  7:50   ` Andrea Righi
2025-02-23 10:00     ` Ingo Molnar
2025-02-23 23:21       ` Qais Yousef
2025-02-24 13:16         ` Vincent Guittot
2025-02-24 23:23       ` Sean Christopherson
2025-02-13  8:24 ` Saravana Kannan
2025-02-16 16:29   ` Qais Yousef
2025-02-16 19:05 ` David Laight
2025-02-22 23:47   ` Qais Yousef
2025-02-28 10:33 ` Attila Fazekas
2025-03-02 12:05   ` David Laight [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=20250302120530.1e742e06@pumpkin \
    --to=david.laight.linux@gmail.com \
    --cc=afazekas@redhat.com \
    --cc=dietmar.eggemann@arm.com \
    --cc=frederic@kernel.org \
    --cc=jstultz@google.com \
    --cc=juri.lelli@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@kernel.org \
    --cc=peterz@infradead.org \
    --cc=qyousef@layalina.io \
    --cc=rostedt@goodmis.org \
    --cc=saravanak@google.com \
    --cc=tglx@linutronix.de \
    --cc=vincent.guittot@linaro.org \
    /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