Received-SPF: pass (google.com: domain of linux-kernel+bounces-152125-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <acce93e6-4c8e-472d-a2bc-5cbad2950661@intel.com>
Date: Fri, 19 Apr 2024 21:06:58 -0700
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v1 00/31] x86/resctrl: Move the resctrl filesystem code to
 /fs/resctrl
To: Dave Martin <Dave.Martin@arm.com>
CC: James Morse <james.morse@arm.com>, <x86@kernel.org>,
	<linux-kernel@vger.kernel.org>, Fenghua Yu <fenghua.yu@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>, <shameerali.kolothum.thodi@huawei.com>, "D Scott
 Phillips OS" <scott@os.amperecomputing.com>, <carl@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>,
	David Hildenbrand <david@redhat.com>, Rex Nie <rex.nie@jaguarmicro.com>
References: <20240321165106.31602-1-james.morse@arm.com>
 <ed23454e-16df-48e9-a344-a6d8baf902a5@intel.com>
 <ZiE8/oXyjBef2qTy@e133380.arm.com>
Content-Language: en-US
From: Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <ZiE8/oXyjBef2qTy@e133380.arm.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?SDF6RUNuZTF4eU5sQ0xiNGRFSTMvcExRdEVFN0YvTTFlYnJlZ2dpWlhHcTVR?=
 =?utf-8?B?Vkd3R0lSS0szSThiRTVvalJpMXVqRThKb2xZL2FrWDFQdXNTWkRCQlpvb2hl?=
 =?utf-8?B?SE5wTmk2dTJrWVhkMkh5ZG5TT0J0eS9TWE9IcUVwazdFakV4ZVh4NmsyWi9H?=
 =?utf-8?B?Q1FBbXllVjFFM2wyL3NhMitDbkJJemNmVERrd1JrUHY1U1F6djdGcW9wOUV1?=
 =?utf-8?B?SDRiMXVMMGhCaC9oN29JTnd2RjY5STcwWmhQKzZ6L1pNRS9uVU9XNGNPTzVq?=
 =?utf-8?B?YlVVWkg5N1JKRXgvV3dDUng4Wk9yanZpL3l0TTd1dUx2V0p0K3EwRm5EU3RZ?=
 =?utf-8?B?ZWlHTXQ4ZDVRbmlER0p3WjF6YTFINThpNXZlcEpXWjBGY1cvV2ZSSUJWRllH?=
 =?utf-8?B?OU1LVFJNQUtJSW4zdVUwMG9VM0ZWUFp0K3hwb3BSYVI5Q0gvYVBPNFFPVTZM?=
 =?utf-8?B?SE9zckk0Q0tFZHNMNkhncDM0dXVVYnlkaDI4WHhKK0Z3Nk44YjdBbEsxS0dP?=
 =?utf-8?B?QXFwSUx4bUFaeUZ2VjIveUNSTzU1NkJwV21IY2QvdVNuZ2lqTVZMRFl6aWpn?=
 =?utf-8?B?Wi9naWxRMDZUSkg5NklrVnFkYVU2em82YTA3bWdPaTFFckV3VkZTRjF0QXo1?=
 =?utf-8?B?ZitYNkZqSU5qT1FSRHFpZG01d0VOWlNUSk84NkJMZmdkcFNaMWVrbnh1VGxK?=
 =?utf-8?B?R1dzZWFTUTQyQTFUMEJ1YjJKQjVHL1dkNG00cjBVbXFxSWdWOWlUck9Hd3Mx?=
 =?utf-8?B?WkpObmc2TE4yTWRWeTdVNjYvZnFKNStsbWRVWExicmQ3SXN0dVRZS09aM2Nj?=
 =?utf-8?B?S1d3YnVNSGtKWHFuQTZLMEwweU1aR1d0OG5VejduT1hWQVFxWUVDK0VRd3V4?=
 =?utf-8?B?VTV0NElsbnZTVkJKbXBEZlYwemgwRnZRTy80cDNtWVhjUk9ldzBRQmY3WXVh?=
 =?utf-8?B?MmJQTDFScmhrTzhsQTMxWVNvZktLK2tLRUE0cEdoQzdiL1lPSnArVkJ1Nnk5?=
 =?utf-8?B?anRBNktkN1JEc0Jvd2MwSDcvVHdzcWJvd1EwY0hqUUtVSUpjYm5mRTZuc296?=
 =?utf-8?B?RVNmMElXUnpRMWhvY2tCN05JMVpreTBmSXg0dC9ia1l5Vzk2cVB4K2pFbFFZ?=
 =?utf-8?B?ejZ2RGlKY2VScFg3L2pmVnBnbzhtTTBNVjUzaW1OcU9wZGVjRTh4Sm0rTFZj?=
 =?utf-8?B?eHRhVVlOd29MdWZvOWRQdkY3bVk1QTVFbmRPZHJpUXZNSXYwaEdlc2N2czZH?=
 =?utf-8?B?SCtzcnNOWmJVS2JYRE0xQlBDMkF4SWtQa1VhTUVMSHdmTG1aYUdhd054NmpT?=
 =?utf-8?B?cnhEY2tYcnhZVERxRExUVmx4MlFVMzM5SVM3QmZhRUZKNEN3ck5SSU9JWUZu?=
 =?utf-8?B?WEwvSUE4Z2ZVM2tESmR1V0MwRERCTkdOdGZ4MEVlbFdFenVvZTlCWlpaMFUy?=
 =?utf-8?B?Ym4rWU9jb2U0U0RrVlRITnJUckQ3L21aVExpTlBtWnY2L0x2blk0Y2hWYmNG?=
 =?utf-8?B?YkdSekZSdEU2dDRTSGlXZDg4OHpEbnlWZ25UWlIrQm9HcUR6UzVtVk1oMVd3?=
 =?utf-8?B?ZmZpQXZzMEhtZm9PWko2dUtKYWVMcHA3dnBiRE9QOFlLRjFUdUJSOUdYVzZx?=
 =?utf-8?B?Nm1zK1ptejFlcVBZR1d6dVBXY2ZhVnFwMGFYL0lraTFOTGZTak5Bc1FabUpI?=
 =?utf-8?B?L0luUjdzb0d3WElWNWNNUlhwNUlDT3pDRWZ4cGRVcFZ5SFF5Vis2WlUxZDJC?=
 =?utf-8?B?ay93YnJYeWprblBwSWdtaytKd2w2R2FGcEpoS3Z2TFdzOHpOYnFtdmcyc3JM?=
 =?utf-8?B?ZHZIVGpFb2U1VU5WRytGbk1ac0pOQWhMV3BiK2RQUGhrdkdqQmNqQk80RVRm?=
 =?utf-8?B?RnJjWFZoYktUKzFVeWo4Zml3Y045SGhtNGd0d0JlUzNHZlZ2dGR1RXYrR0d6?=
 =?utf-8?B?MmNIT0s0OHNxU1RkbVV5eU56SE15NHF4T2J2clF6Z3piYW05TG80ak93ZXZL?=
 =?utf-8?B?WTBiY0ppV2lmempxb1Q4dTNNVEJ0ZnprMVcxSm96MlNTWmYrM29Vd0lSUFpy?=
 =?utf-8?B?WjJ2SldWTUsxZ1Z2WmljQTJuVzNnWkpwVmlkbmc2Y0w3Q0hTcUtBMi9pYXBS?=
 =?utf-8?B?QUJvUUlkcGNNQ3FiUzhaUkwybUZaWEZUQm9oMWRGeUhwQm9xb2tuUUZ2Qzhi?=
 =?utf-8?B?SkE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: ab66b8ed-98cd-4a5b-960c-08dc60ef5454
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Apr 2024 04:07:00.9110
 (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: nPVN4R0HNRSCXb0kSUb9APgD4RmHU5VaHaxRuylZCyxn/nGLkY71bncGj6qyAfGW4IR4D046deZLKDMdjDYltBTaVM0/u0ErcaNDPEvHtnw=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY8PR11MB7948
X-OriginatorOrg: intel.com

Hi Dave,

On 4/18/2024 8:32 AM, Dave Martin wrote:
> Since we seem to be getting to the end of the outstanding discussion on
> this series, what's your preference for moving this forward?
> 
> James will be back towards the end of next week AIUI, so it should be
> possible to apply most of the changes that have been discussed and post
> a v2 for review fairly quickly after that.
> 
> I'm happy to apply my fixups branch and post an untested "v1.1" for
> review if you think it would be useful to see the de-noised output of
> the review so far, but I don't want to fork the discussion or create
> unnecessary work.  This interim respin would be subject to what James
> wants to pick up.

Please post v2 with discussions complete and feedback addressed
(unless there are some discussions that need to see how other changes turn out
before they can complete but I do not think this applies to this series).

To create some expectations, when you submit a new series it will go to the back of
my review queue and at this time there are three other resctrl pieces of work
waiting for review (btw ... one of them, [1], is waiting for feedback from Arm). 
 
> Alternatively, I can try to follow up on the string parsing discussion
> from patch #3 with some real code if you'd prefer to get a clean
> interface in place for that.

We can keep discussing that, sure.

Reinette

[1] https://lore.kernel.org/lkml/cover.1711674410.git.babu.moger@amd.com/