From: Reinette Chatre <reinette.chatre@intel.com>
To: Maciej Wieczor-Retman <maciej.wieczor-retman@intel.com>,
<fenghua.yu@intel.com>, <shuah@kernel.org>
Cc: <linux-kselftest@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
<ilpo.jarvinen@linux.intel.com>, <tony.luck@intel.com>
Subject: Re: [PATCH v8 1/2] selftests/resctrl: Adjust effective L3 cache size with SNC enabled
Date: Mon, 16 Dec 2024 13:43:15 -0800 [thread overview]
Message-ID: <6c540c6d-680d-4a22-bec9-e4184bff051f@intel.com> (raw)
In-Reply-To: <83dcba6a462ec010a3c80bd8c6d66f49502b75ae.1734361935.git.maciej.wieczor-retman@intel.com>
Hi Maciej,
On 12/16/24 7:18 AM, Maciej Wieczor-Retman wrote:
> Sub-NUMA Cluster divides CPUs sharing an L3 cache into separate NUMA
> nodes. Systems may support splitting into either two, three, four or six
> nodes. When SNC mode is enabled the effective amount of L3 cache
> available for allocation is divided by the number of nodes per L3.
>
> It's possible to detect which SNC mode is active by comparing the number
> of CPUs that share a cache with CPU0, with the number of CPUs on node0.
>
> Detect SNC mode once and let other tests inherit that information.
>
> Update CFLAGS after including lib.mk in the Makefile so that fallthrough
> macro can be used.
>
> To check if SNC detection is reliable one can check the
> /sys/devices/system/cpu/offline file. If it's empty, it means all cores
> are operational and the ratio should be calculated correctly. If it has
> any contents, it means the detected SNC mode can't be trusted and should
> be disabled.
>
> Check if detection was not reliable due to offline cpus. If it was skip
> running tests since the results couldn't be trusted.
>
> Co-developed-by: Tony Luck <tony.luck@intel.com>
> Signed-off-by: Tony Luck <tony.luck@intel.com>
> Signed-off-by: Maciej Wieczor-Retman <maciej.wieczor-retman@intel.com>
> ---
Thank you very much.
Reviewed-by: Reinette Chatre <reinette.chatre@intel.com>
Reinette
next prev parent reply other threads:[~2024-12-16 21:44 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-12-16 15:18 [PATCH v8 0/2] selftests/resctrl: SNC kernel support discovery Maciej Wieczor-Retman
2024-12-16 15:18 ` [PATCH v8 1/2] selftests/resctrl: Adjust effective L3 cache size with SNC enabled Maciej Wieczor-Retman
2024-12-16 21:43 ` Reinette Chatre [this message]
2024-12-16 15:18 ` [PATCH v8 2/2] selftests/resctrl: Discover SNC kernel support and adjust messages Maciej Wieczor-Retman
2024-12-19 18:35 ` [PATCH v8 0/2] selftests/resctrl: SNC kernel support discovery Reinette Chatre
2025-01-14 22:30 ` Reinette Chatre
2025-01-14 23:25 ` Shuah Khan
2025-01-14 23:28 ` Reinette Chatre
2025-01-14 23:58 ` Shuah Khan
2025-01-15 8:48 ` Maciej Wieczor-Retman
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=6c540c6d-680d-4a22-bec9-e4184bff051f@intel.com \
--to=reinette.chatre@intel.com \
--cc=fenghua.yu@intel.com \
--cc=ilpo.jarvinen@linux.intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-kselftest@vger.kernel.org \
--cc=maciej.wieczor-retman@intel.com \
--cc=shuah@kernel.org \
--cc=tony.luck@intel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.