Received-SPF: pass (google.com: domain of linux-kernel+bounces-153498-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Message-ID: <94c3bfd0-e221-46c7-86c4-f0545e66f17d@amd.com>
Date: Mon, 22 Apr 2024 08:51:16 -0500
User-Agent: Mozilla Thunderbird
Reply-To: babu.moger@amd.com
Subject: Re: [PATCH v1 00/31] x86/resctrl: Move the resctrl filesystem code to
 /fs/resctrl
To: Reinette Chatre <reinette.chatre@intel.com>,
 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>,
 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>
 <acce93e6-4c8e-472d-a2bc-5cbad2950661@intel.com>
Content-Language: en-US
From: "Moger, Babu" <babu.moger@amd.com>
In-Reply-To: <acce93e6-4c8e-472d-a2bc-5cbad2950661@intel.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?Q2pPODlMVFluRHU2aDNwWWFuQStiSHp5ZDk2QkRBSHZMNU4vK0xqeG8zbUhX?=
 =?utf-8?B?dzdnTDQzUDZkK0QrdnJxSVlWM1IyTHUwcG4yeE9QcWJoV1dLR3htV3FZdzh3?=
 =?utf-8?B?SHYwYm1DbVIzM290Mk5zYUg1aFNTV0wwM3NGQTE4TzBtNWgySFAyc3FFTVRa?=
 =?utf-8?B?bExJWTJlS2tJa0JraWtjODZzRkpBU0JHZXFBNGUyamFJaWFDakZCTEZsUFhV?=
 =?utf-8?B?L0hvdm04enNTc3lqQmlvUldITVhiYTNaaEZKb1lGWGhTUTFFWEUrTmFST1Rv?=
 =?utf-8?B?L0JqSG02dnFwR1lvMmFGMFFwL2dJRFRtRGxTNy8ydEhxeXF6NFl4ZXlYU0tD?=
 =?utf-8?B?TXFxN2p6eFN5dVJyV1h6Y2VMa012bmMxUGxoZ3lxQjloS21SbVhMclpOaTBB?=
 =?utf-8?B?NVZWWE5GYTZhYjNlaDJTTUQ1cGQ0SlNBVWNBVG5BcFlXVm93TEMyUG9rRmhy?=
 =?utf-8?B?dk8yaGhydHl2UENJZXhWM1NlUUswRCtobkRxU2FhdldPMFNveTV5bGNtUnR4?=
 =?utf-8?B?WUdSRlZ4VkdGQ2VpbHVDUHhHZUl6SmxOQm9HdFh3TVhkWlYzZmE3NERpVWls?=
 =?utf-8?B?RFMrckRsblNQVUZaV1VhWVVKd09QUUlCY3NPU1VvWFUvS0ZNa3N4ZUFQZEZB?=
 =?utf-8?B?cDF2UlZlV3VodjZYUlVpZnNyUE80YjdjU1NBOFpKaEt3cnFwNzVJV0orRThD?=
 =?utf-8?B?VXJzTkNqQzNlQjEyeUpQcXovWnI3dTR0cytGb2VibUVKbjI4WXV5SkEzcWZh?=
 =?utf-8?B?Mk1FSmIrSEZrdjg1MkF0bmhhR2txbldxNE1QZENHRktDZ0pJeUEwWFdIU2p4?=
 =?utf-8?B?NXo0WUZtRDRGUXhMeTB6VFZYUE1INk1KQUVRRVREdHJQVWhKQXpKVjJZQitJ?=
 =?utf-8?B?ZDdWdW00Sjgyck5ZajJCVThheWo4MEhkWEthNUpmU092azBnRUd1REduQXRm?=
 =?utf-8?B?WXFKMG1CVVBteWVWNXpKTWlVK0JqOXBveGttTmp4a2dQRk9zeWV6ZityNWgr?=
 =?utf-8?B?TmxPeFBqczJ4M2h4QU5xRElMM1d3WDFVcHdCVjU3SUxVVVpkbWFMakNwdk5K?=
 =?utf-8?B?VGw5S1pub1lCMnFYNFZNZ0kyYXpZZE43WVJwRkRKR2o1dDd0SEtCWWxsYmVT?=
 =?utf-8?B?Mk55c1JzaHdKbXVMcUwvMzYzUHduSWJuOW11ME1Wa3d2Y08wa1BqZzBmMGNr?=
 =?utf-8?B?S3JuUFNVVWQwWldmbnFTSUVEeDVCSmtFbnNtYTNRbTIzWkwySGNOSXRuSWND?=
 =?utf-8?B?aC9UQTlsVHVwN05MTFVjVXdOeWthWHg4bVlINndwanJFeHBxUno2VlVJaEVJ?=
 =?utf-8?B?QkRMVWFSOVBvbWNBS0ZYUi8vU2R5N2p1R05NMjkrSE4zREpRV0dsajJjNUhM?=
 =?utf-8?B?UWtWY05jVEhXdkpWaTJkYWNBUndFN3U2c0ZZVWNGRUtXUUJGbXgrSVp5aUJN?=
 =?utf-8?B?SkhsUUNUeTB1WlM1VTNGZU5Ta3pNSHc1N0p6SUo3dFlpVDM2MFhqY0txcXZF?=
 =?utf-8?B?NmxRN1MxTDE3ajF5WkZPRUE0N1dBZU11RjYxMkVIcXJrM2VoaVhidHZsZ2Zj?=
 =?utf-8?B?cXl2SHk3cUtSMlV2dTBJRUtQMmo1SXJPV3BUNk1md0lKVFkyOGUvaklWR21w?=
 =?utf-8?B?ZGFzZDBPcTNDbDJuaU42ZUdkdjlFUjlrWXN6TXBoeTJIcGdoWDdRQkFGdVVj?=
 =?utf-8?B?Rm1DYi90dnNaRGNJTGloTENuQ3lscHBTb0twYlBPaERaY1huRGNrMzdzQnFN?=
 =?utf-8?B?VThCUE9VTVhEb0hBcWNlYUNjV2VqUUtxRWpOMVNPeGY1L0JpQUV4akVtZkdG?=
 =?utf-8?B?Wm53VHZ0bEEyL3lkRWFyL3VuTU12NjkwcDM0TjM5Sm90TE1CblR1SFRWeVdN?=
 =?utf-8?B?RnBiaC9lNkE2M0c3c1J2YUdXbjA0MnkwZVBreXNTdFprdHJKcmw1aW1oWEdn?=
 =?utf-8?B?SkdrUkIwMzRTYWFML0ZnMUkzOGtLSUFqekZ0dlN6Y1JsUVhtMU1lMzVXYVpB?=
 =?utf-8?B?ZzRrSFg1ZTJvWjhRV2xXeVFVaGVtWjV1am5VSGtlanlBWU1GY04rb3VxMGFN?=
 =?utf-8?B?MFhGKzEwanhjb3FWOFVTcWxVampOQktxWkNVZnBEM2hOV1Jnb3lXWUVwdUxJ?=
 =?utf-8?Q?rJIE=3D?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 70a4d7f9-260b-41e6-9a0b-08dc62d349e6
X-MS-Exchange-CrossTenant-AuthSource: MW3PR12MB4553.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Apr 2024 13:51:19.8548
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 1vdU6GSFNx80ungic64pfyijqXr2rbRna2yor0fKkSpxSQLtsEIdffKPX+T+5Qdc
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR12MB4359

Hi Reinette,

On 4/19/24 23:06, Reinette Chatre wrote:
> 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/

Do you have any more feedback on this series. I have few feedbacks from
Peter. I was planning to work on v4 of this series.

-- 
Thanks
Babu Moger