Received-SPF: pass (google.com: domain of linux-kernel+bounces-10088-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
From: Carl Worth <carl@os.amperecomputing.com>
To: James Morse <james.morse@arm.com>, x86@kernel.org,
 linux-kernel@vger.kernel.org
Cc: Fenghua Yu <fenghua.yu@intel.com>, Reinette Chatre
 <reinette.chatre@intel.com>, Thomas Gleixner <tglx@linutronix.de>, Ingo
 Molnar <mingo@redhat.com>, Borislav Petkov <bp@alien8.de>, H Peter Anvin
 <hpa@zytor.com>, Babu Moger <Babu.Moger@amd.com>, James Morse
 <james.morse@arm.com>, shameerali.kolothum.thodi@huawei.com, D Scott
 Phillips OS <scott@os.amperecomputing.com>, lcherian@marvell.com,
 bobo.shaobowang@huawei.com, tan.shaopeng@fujitsu.com,
 baolin.wang@linux.alibaba.com, Jamie Iles <quic_jiles@quicinc.com>, Xin
 Hao <xhao@linux.alibaba.com>, peternewman@google.com,
 dfustini@baylibre.com, amitsinght@marvell.com
Subject: Re: [PATCH v8 00/24] x86/resctrl: monitored closid+rmid together,
 separate arch/fs locking
In-Reply-To: <20231215174343.13872-1-james.morse@arm.com>
References: <20231215174343.13872-1-james.morse@arm.com>
Date: Fri, 22 Dec 2023 14:43:43 -0800
Message-ID: <874jg9rh7k.fsf@rasp.cworth.amperemail.amperecomputing.com>
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?16Uq4/LyY4MRDiwwD/b6p7GwPeGUJklknpiy5Pzq714h7m6jXMbiHEoRmlF1?=
 =?us-ascii?Q?yW3iPWEk1WjAkx98IkfOQmlR4dp1sOLRHWBss1ZzBb0Yq4kbNyiB6U7uCSwR?=
 =?us-ascii?Q?HyMwYt+fkLsZk2ai8dtG7IifZujLGEmjxfSgV7lOLma2pSYATzaNnyu4YQ+M?=
 =?us-ascii?Q?/udgc5kNkZuFRP+0gcu/w1nXXqAnO8K1pyjsPMIGuchDCSnZhTc/i7KOWRzF?=
 =?us-ascii?Q?+Bvbzt3hzi3V0fzvXjVZM11I7trQ8h8qdbwAjaws/OkGjFsAPVy3HYqMsW4K?=
 =?us-ascii?Q?tIt7Z5rc/1SR6n2xfUK4lACmUwDQMmJ0MH64c806rY+AkpO0uccmGEa6zVON?=
 =?us-ascii?Q?ZvVGCxI7d9lhKIJOh26Cw+K3U7r5MRGzfqYdaVklx9M6FROELmO2DmVtmeIX?=
 =?us-ascii?Q?Lrmohmi5wfGl66I6FW6jbTnHnOJfYI56tFCupOQprTdgkKgmSFGfMV8ourR3?=
 =?us-ascii?Q?1A7cWT0C6qRtqIrwybmz/MPm2d5hpsQtDXrc8mZ3cUTKu23hSkSV/Kx3T8Qf?=
 =?us-ascii?Q?1Hva+CPF1NVX/U9u0XYnGeH5RUim6JKBL4lm38NsJ1q78G4njJVf7079JOiz?=
 =?us-ascii?Q?nXWwMCrVKYJHHlQwu2V8lwOtSfzOOM04j3fhahtbeJKKqWofDDRKNJ0CRYAV?=
 =?us-ascii?Q?QCX2RD7G679GI9NnQ8zQbFX9fUg37hF0/7LzlucXO9WEroRag0Bu7MSNPPLZ?=
 =?us-ascii?Q?DwXRhM2CGLlmtdGJDy6q/dKbJ1TenR87dgbux2i7KJzxIvtf5xWc9BtIm7r9?=
 =?us-ascii?Q?Sr0cgK+2JEY4AJw9ez3rx/A6vnDLunIWmOrs5nKYipSL37rUrfTgEBBq20dD?=
 =?us-ascii?Q?XhhQx865Ob1O/n1bKpm6eFhftlSb/W69PCPiCWrDJ4+I0qhLSIJ7s5yGZXXF?=
 =?us-ascii?Q?ytrDKnt8SZuXVAw6hEK/v6fwf28xsttoHcaGBr9JBtk0bCSgMhNV5FT4g5Ls?=
 =?us-ascii?Q?kjRbBoEgHmzbZPDStutWASYdD/1+ecBSR3AIA/Gs6jDHAHPwqBu58dLLBbZ8?=
 =?us-ascii?Q?ewW6CumvvOlmTp449PpsP+nDvYZSfqhfRM1rR2Nsnx+AgpNGj7qq+HffudMn?=
 =?us-ascii?Q?YSfAg3erPjDJ2OIqDWKgHS4YHV8XjdXzw/nM7NYQjPd29VBgu7+0bYmsNbBF?=
 =?us-ascii?Q?e3Sq1W01tEMzDL2g6v7vmKxid34ryxqTnw0L9NfDXjjhd4nAfEc7g8q6yRJW?=
 =?us-ascii?Q?9HqEHjS1YRrvG3ivQQ2N8n5MPzJMujClp+Lz23O8jbd1YrvBmc6y2RLhDYi+?=
 =?us-ascii?Q?A0t/1C2l/bU1igwUY6TgloOCwO5HWXYUK7+611l7zdrcLxypis+XCsVyPbJR?=
 =?us-ascii?Q?/LVVkmPQ0Y93fUwq0uDvITF7C/UHqEzxCwhLaK8wd4MyBWRWce7Frm7+fmiw?=
 =?us-ascii?Q?nhwlzoY/vQaC2M8DdMkFqVBmHmib1LHXq+1XvImlRmxMGtNfD3OjzntWf/GM?=
 =?us-ascii?Q?dfHntlH5GzmFJDDxKbfeLEqRnZwT22p+Vsj9GCllLxUV81/y6L97qPlqmPIi?=
 =?us-ascii?Q?nUytynOv2lm6PwGKE20OcUQYosFCuzZrfXh77h5G96grCbaM1ZJMpI+5OZmJ?=
 =?us-ascii?Q?Lg8/bXXoKIUA7sMdEmXcZOWnL4wgdWsO0wZ5blpHi3sdeYtFG0InjXRIZJBo?=
 =?us-ascii?Q?d37D+HeuPGZbq3l9RDrM0+I=3D?=
X-OriginatorOrg: os.amperecomputing.com
X-MS-Exchange-CrossTenant-Network-Message-Id: cfa46b63-518d-4b18-cf1a-08dc033f7617
X-MS-Exchange-CrossTenant-AuthSource: PH0PR01MB8024.prod.exchangelabs.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Dec 2023 22:43:48.1733
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3bc2b170-fd94-476d-b0ce-4229bdc904a7
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 92VuehLEmexrj5PloUWkoI3S6OhLeelV9bX78jGJFqGhxKTwwfD/H1sTHoVXFGMIh1he1fHN5sY5U4IhTiUo6+b0z1G+8w4CYzmspST7BTc=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA0PR01MB6282

James Morse <james.morse@arm.com> writes:
> This series does two things, it changes resctrl to call resctrl_arch_rmid_read()
> in a way that works for MPAM, and it separates the locking so that the arch code
> and filesystem code don't have to share a mutex. I tried to split this as two
> series, but these touch similar call sites, so it would create more work.
>
> (What's MPAM? See the cover letter of the first series. [1])

Thanks, James. This is really useful for us at Ampere since it enables
the MPAM driver on top of this series.

I've tested this series on an Ampere implementation, by successfully
using resctrl to configure and exercise MPAM functionality. I can't
speak to the effects of the refactor on x86 since I have not tested that
all.

For the series:

Tested-by: Carl Worth <carl@os.amperecomputing.com>

-Carl