Received-SPF: pass (google.com: domain of linux-kernel+bounces-109084-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Wed, 20 Mar 2024 16:21:10 +0100
From: Maciej Wieczor-Retman <maciej.wieczor-retman@intel.com>
To: Reinette Chatre <reinette.chatre@intel.com>
CC: Tony Luck <tony.luck@intel.com>, Fenghua Yu <fenghua.yu@intel.com>, "Peter
 Newman" <peternewman@google.com>, Jonathan Corbet <corbet@lwn.net>, "Shuah
 Khan" <skhan@linuxfoundation.org>, <x86@kernel.org>, Shaopeng Tan
	<tan.shaopeng@fujitsu.com>, James Morse <james.morse@arm.com>, Jamie Iles
	<quic_jiles@quicinc.com>, Babu Moger <babu.moger@amd.com>, Randy Dunlap
	<rdunlap@infradead.org>, Drew Fustini <dfustini@baylibre.com>,
	<linux-kernel@vger.kernel.org>, <linux-doc@vger.kernel.org>,
	<patches@lists.linux.dev>
Subject: Re: [PATCH v16 0/9] Add support for Sub-NUMA cluster (SNC) systems
Message-ID: <5vnrboxpkrriiqcbwrxtbextu6pduqjfchlvsdn5nwe7lr7vou@dngffvu7a5fh>
References: <20240312214247.91772-1-tony.luck@intel.com>
 <8ee6f553-16c0-4097-b5d8-af1598d1b85a@intel.com>
Content-Type: text/plain; charset="iso-8859-1"
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <8ee6f553-16c0-4097-b5d8-af1598d1b85a@intel.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?iso-8859-1?Q?AFhLG2GPMuiFMnrIsmfBrB1Gf1XzFkv8DsY44xg5JGUoJqbC+ML4JdQfyN?=
 =?iso-8859-1?Q?G76xQRxK4hLZInEUNoT4zo0SPagp+S3bCAOH8cuzkXeMh4kAks2/Q4nt8O?=
 =?iso-8859-1?Q?vlvmQFmscbInTwsBefQR4GBQJ4Uwp2SPz8zExhevFS2XD04d9LmBHpKyJV?=
 =?iso-8859-1?Q?rh1LwV9xckVduhV45HivYDhyRuXGUGf7ru6qcyIlO54INAj8dn77Ip3t7P?=
 =?iso-8859-1?Q?lQk+UD/TQpLmz5MnLsV3mnSsLzPeff4/ID/Q/MqySrBoxE6FpL8qg+F8HI?=
 =?iso-8859-1?Q?+Uf8S4prRiGoBLFKKbpZjQxff/s7+uCVBvwuoOft9pCd7m9yFfS7hbbPL6?=
 =?iso-8859-1?Q?3qCH+1Lv5gCTNlfLBM25UPODxF1B9m+le0o8SBfQWuJocTrxLcfotTSvX2?=
 =?iso-8859-1?Q?y6ah457RL38fsxjZorKteUQU3ihzwqMNpiK4pqCc6kUpDay0baK8sTnm3w?=
 =?iso-8859-1?Q?AHyzuf4WfqLegLrul/tzWsAKUTCXER5JMpIJ/gknMr7xugtp9KJ2L3N2+A?=
 =?iso-8859-1?Q?y4D5mkO4bV3iCFERSO1itPivMJUXyiQQ0d8JmMeU1Qiyk0k5igC7YF2Qxw?=
 =?iso-8859-1?Q?2ryqByaSEu6GlDiyw1rlgHI/K09rn+6oc0AsqEO+BsnSyqDMEKST1TvMsn?=
 =?iso-8859-1?Q?Ct35HhvJakAyMG+6ZEWPES4uR9abyQV1SOqEoRieOZJ8InR6pT2V4JunZo?=
 =?iso-8859-1?Q?0+/j8cJTGHCCPkCp43x1bCTKvjzgBO2mcNuI1Q5aOpIAcoQii2ryeJjHtp?=
 =?iso-8859-1?Q?e0dRJbLb7Nv19yQsNPAWcCVcCCRPWoAvK/v1jJ/1JTIa+YDd30eBmKNwxp?=
 =?iso-8859-1?Q?QcCGYNb/9OC7CG3Wm+EJdda9ArTFOpfPwh0V8lcCRnt96ETASILO1sC2lP?=
 =?iso-8859-1?Q?OEMDnNO/pzJsEK+QMYBfHyOhvHdddzqf7S72U7KIEbROPboJ4iQrCTKMUl?=
 =?iso-8859-1?Q?UErMgi0zunIZv17oTU1izA8XIvDeA7oQ67/cfEY+LCakLkd1YbMnizOlM7?=
 =?iso-8859-1?Q?stj/yyhkxBBaCO8ist5P0Pn2uDkDeVYorGWN6uGSAtd6aSeev5KBoPJlwR?=
 =?iso-8859-1?Q?hxxmuVgQgZB2KcoZXr8KOpA90PlCDlBK8jQ9yAtLmcL7z6Y1Bmg+aoP1br?=
 =?iso-8859-1?Q?OY7H3ZzVTemz86nAebDwNRVItgbDjXCWJtZEsDfB4uMy/nru7Z96TAM11X?=
 =?iso-8859-1?Q?sSQoC7a76aRTlBIykyGL+kUrJQM48M4ZMLdON3584JDMY1IMJ0niYJocL5?=
 =?iso-8859-1?Q?cPmMoiDEA/+AHTvFqle3H7X7iDWad+tZ54Y73F5D01cxVkkc9qGVPuT1l2?=
 =?iso-8859-1?Q?mHy3hVpU0MXXcCCVV2SphTL5XRh+C5IvvJdNFFDg6/JwioUM/9bg3hFhkF?=
 =?iso-8859-1?Q?NLyka+UB+Wt3c4IYDcVrCdQZm85Ftz/MHS7pSjI/BMoOu/IffAZFqU3A7B?=
 =?iso-8859-1?Q?C3dCx4G+5t0Wf52kklVhpDrC+qMDsgNIvhnk+ZFPRhZad6/SuR4sxqETG3?=
 =?iso-8859-1?Q?Vt3o+6smZKXCOUKwQmn+nViDYs3KtUN8xzhses8JSO/iuwgJ5p+axE0q6K?=
 =?iso-8859-1?Q?GWDOvfRobtz3lc3ergj2RS6wkQRiksnivWBGtBcUDVKvohlJWf91qx0vFX?=
 =?iso-8859-1?Q?x77fr+aGpIQvCBXIMLRMAiDOcaZEseNdEb1aboKAnkpR51+AWyP2/gofSa?=
 =?iso-8859-1?Q?pEZ4JBPNrF+DsCAdbrY=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 3c2f1969-98cd-4f0b-8d39-08dc48f167b6
X-MS-Exchange-CrossTenant-AuthSource: MN0PR11MB6231.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Mar 2024 15:21:24.6781
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: TVvYAr8NftYtoO7j63d0hODlPej/uuIzzZNfrIIt1q9M8IoVKnlBWK8QYdzaoMP3xnDyvDRpg0BnBMBsKfn6nUZp0pxOfWHbJxwkZag8k1k=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL3PR11MB6316
X-OriginatorOrg: intel.com

Hi Reinette,

On 2024-03-19 at 10:51:14 -0700, Reinette Chatre wrote:
>What remains is the user interface that continues to gather opinions [3]. These new
>discussions were prompted by user space needing a way to determine if resctrl supports
>SNC. This started by using the "size" file but thinking about it more user space could
>also look at whether the number of L3 control domains are different from the number
>of L3 monitoring domains? I am adding Maciej for his opinion (please also include him
>in future versions of this series). 

By this do you mean comparing the contents of main "schemata" file with the
number of mon_L3_* files?

>Apart from the user space requirement to know if SNC is supported by resctrl there
>is also the interface with which user space obtains the monitoring data.
>James highlighted [1] that the interface used in this series uses existing files to
>represent different content, and can thus be considered as "broken". It is not obvious
>to me how to "fix" this. Should we continue to explore interfaces like [2] that
>attempts to add SNC support into resctrl or should the message continue to be
>that SNC "plays havoc with the RDT monitoring features" and users wanting to use
>SNC and RDT at the same time are expected to adapt to the peculiar interface ...
>or is the preference that after this series "SNC and RDT are compatible" and
>thus presented with an intuitive interface?

I kind of liked this idea [1]. Hiding SNC related information behind some not
obvious text parsing and size comparisons might eliminate any ease of use for
userspace applications. But I agree with you [2] that it's hard to predict the
future for this interface and any potential problems with setting up this
file structure.

[1] https://lore.kernel.org/all/SJ1PR11MB608309F47C00F964E16205D6FC2D2@SJ1PR11MB6083.namprd11.prod.outlook.com/
[2] https://lore.kernel.org/all/7f15a700-f23a-48f9-b335-13ea1735ad84@intel.com/

-- 
Kind regards
Maciej Wiecz?r-Retman