From: Jesper Dangaard Brouer <hawk@kernel.org>
To: Shakeel Butt <shakeel.butt@linux.dev>
Cc: Yosry Ahmed <yosryahmed@google.com>,
tj@kernel.org, cgroups@vger.kernel.org, hannes@cmpxchg.org,
lizefan.x@bytedance.com, longman@redhat.com,
kernel-team@cloudflare.com, linux-mm@kvack.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH V7 1/2] cgroup/rstat: Avoid thundering herd problem by kswapd across NUMA nodes
Date: Fri, 19 Jul 2024 09:54:41 +0200 [thread overview]
Message-ID: <5ccc693a-2142-489d-b3f1-426758883c1e@kernel.org> (raw)
In-Reply-To: <k3aiufe36mb2re3fyfzam4hqdeshvbqcashxiyb5grn7w2iz2s@2oeaei6klok3>
On 19/07/2024 02.40, Shakeel Butt wrote:
> Hi Jesper,
>
> On Wed, Jul 17, 2024 at 06:36:28PM GMT, Jesper Dangaard Brouer wrote:
>>
> [...]
>>
>>
>> Looking at the production numbers for the time the lock is held for level 0:
>>
>> @locked_time_level[0]:
>> [4M, 8M) 623 |@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@ |
>> [8M, 16M) 860 |@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@|
>> [16M, 32M) 295 |@@@@@@@@@@@@@@@@@ |
>> [32M, 64M) 275 |@@@@@@@@@@@@@@@@ |
>>
>
> Is it possible to get the above histogram for other levels as well?
Data from other levels available in [1]:
[1]
https://lore.kernel.org/all/8c123882-a5c5-409a-938b-cb5aec9b9ab5@kernel.org/
IMHO the data shows we will get most out of skipping level-0 root-cgroup
flushes.
Copy-pasted here (adj whitespaces hoping it better fit email):
Time below is in nanosec:
- M -> ms
- K -> usec
@locked_time_level[0]:
[4M, 8M) 623 |@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@ |
[8M, 16M) 860 |@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@|
[16M, 32M) 295 |@@@@@@@@@@@@@@@@@ |
[32M, 64M) 275 |@@@@@@@@@@@@@@@@ |
@locked_time_level[1]:
[4K, 8K) 6 |@@@@ |
[8K, 16K) 65 |@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@|
[16K, 32K) 52 |@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@ |
[32K, 64K) 23 |@@@@@@@@@@@@@@@@@@ |
[64K, 128K) 15 |@@@@@@@@@@@@ |
[128K, 256K) 10 |@@@@@@@@ |
[256K, 512K) 6 |@@@@ |
[512K, 1M) 15 |@@@@@@@@@@@@ |
[1M, 2M) 2 |@ |
[2M, 4M) 14 |@@@@@@@@@@@ |
[4M, 8M) 6 |@@@@ |
[8M, 16M) 7 |@@@@@ |
[16M, 32M) 1 | |
@locked_time_level[2]:
[2K, 4K) 1 | |
[4K, 8K) 160 |@@@@@@@@@ |
[8K, 16K) 733 |@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@ |
[16K, 32K) 901 |@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@|
[32K, 64K) 191 |@@@@@@@@@@@ |
[64K, 128K) 115 |@@@@@@ |
[128K, 256K) 61 |@@@ |
[256K, 512K) 70 |@@@@ |
[512K, 1M) 59 |@@@ |
[1M, 2M) 27 |@ |
[2M, 4M) 9 | |
@locked_time_level[3]:
[1K, 2K) 3 | |
[2K, 4K) 2 | |
[4K, 8K) 5 | |
[8K, 16K) 147 |@@@@@@ |
[16K, 32K) 1222 |@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@|
[32K, 64K) 266 |@@@@@@@@@@@ |
[64K, 128K) 199 |@@@@@@@@ |
[128K, 256K) 146 |@@@@@@ |
[256K, 512K) 124 |@@@@@ |
[512K, 1M) 17 | |
[1M, 2M) 0 | |
[2M, 4M) 0 | |
[4M, 8M) 1 | |
@locked_time_level[4]:
[4K, 8K) 2 |@@ |
[8K, 16K) 17 |@@@@@@@@@@@@@@@@@@@@@@ |
[16K, 32K) 40 |@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@|
[32K, 64K) 4 |@@@@@ |
> I know this is 12 numa node machine, how many total CPUs are there?
192 CPU cores (incl HyperThreaded).
--Jesper
next prev parent reply other threads:[~2024-07-19 7:54 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-11 13:28 [PATCH V7 1/2] cgroup/rstat: Avoid thundering herd problem by kswapd across NUMA nodes Jesper Dangaard Brouer
2024-07-11 13:29 ` [PATCH V7 2/2 RFC] cgroup/rstat: add tracepoint for ongoing flusher waits Jesper Dangaard Brouer
2024-07-16 8:42 ` [PATCH V7 1/2] cgroup/rstat: Avoid thundering herd problem by kswapd across NUMA nodes Jesper Dangaard Brouer
2024-07-17 0:35 ` Yosry Ahmed
2024-07-17 3:00 ` Waiman Long
2024-07-17 16:05 ` Yosry Ahmed
2024-07-17 16:36 ` Jesper Dangaard Brouer
2024-07-17 16:49 ` Yosry Ahmed
2024-07-18 8:12 ` Jesper Dangaard Brouer
2024-07-18 15:55 ` Yosry Ahmed
2024-07-19 0:40 ` Shakeel Butt
2024-07-19 3:11 ` Yosry Ahmed
2024-07-19 23:01 ` Shakeel Butt
2024-07-19 7:54 ` Jesper Dangaard Brouer [this message]
2024-07-19 22:47 ` Shakeel Butt
2024-07-20 4:52 ` Yosry Ahmed
[not found] ` <CAJD7tkaypFa3Nk0jh_ZYJX8YB0i7h9VY2YFXMg7GKzSS+f8H5g@mail.gmail.com>
2024-07-20 15:05 ` Jesper Dangaard Brouer
2024-07-22 20:02 ` Shakeel Butt
2024-07-22 20:12 ` Yosry Ahmed
2024-07-22 21:32 ` Shakeel Butt
2024-07-22 22:58 ` Shakeel Butt
2024-07-23 6:24 ` Yosry Ahmed
2024-07-17 0:30 ` Yosry Ahmed
2024-07-17 7:32 ` Jesper Dangaard Brouer
2024-07-17 16:31 ` Yosry Ahmed
2024-07-17 18:17 ` Jesper Dangaard Brouer
2024-07-17 18:43 ` Yosry Ahmed
2024-07-19 15:07 ` Jesper Dangaard Brouer
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=5ccc693a-2142-489d-b3f1-426758883c1e@kernel.org \
--to=hawk@kernel.org \
--cc=cgroups@vger.kernel.org \
--cc=hannes@cmpxchg.org \
--cc=kernel-team@cloudflare.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=lizefan.x@bytedance.com \
--cc=longman@redhat.com \
--cc=shakeel.butt@linux.dev \
--cc=tj@kernel.org \
--cc=yosryahmed@google.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;
as well as URLs for NNTP newsgroup(s).