Received-SPF: pass (google.com: domain of linux-kernel+bounces-136173-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <ed23454e-16df-48e9-a344-a6d8baf902a5@intel.com>
Date: Mon, 8 Apr 2024 20:13:19 -0700
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v1 00/31] x86/resctrl: Move the resctrl filesystem code to
 /fs/resctrl
To: James Morse <james.morse@arm.com>, <x86@kernel.org>,
	<linux-kernel@vger.kernel.org>
CC: 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>, Dave
 Martin <dave.martin@arm.com>
References: <20240321165106.31602-1-james.morse@arm.com>
Content-Language: en-US
From: Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <20240321165106.31602-1-james.morse@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?RGcyUkR1enVzd0NLWWZIYXdlUVQvZm9YL0xMVUFtU2lJS3ZmK2FUa2draGFW?=
 =?utf-8?B?SHliSENoRkU0NTUxWXVMSVFPTFZPR3ZzeUJHb1J3NlUvNFZPTjVJNWJXZVFi?=
 =?utf-8?B?clUvQnhGMm5UT1ltOWZCVWdkR3B1T2VBRmRWZG1WT3Q0SkZ2c1Q4Q1R6M2p3?=
 =?utf-8?B?NTRvM0pDSi93MFo2aFV2Y1hMUVlsQ00rVHVuK2dUVnpiaS9WVDl2aVdkQkMx?=
 =?utf-8?B?alI3TzV1YjBLcjlSTlFROEYxRjllUmVXMUtTTitTbTZhNnZSVjhSL3RYUzhw?=
 =?utf-8?B?Q1NGNlpuY0RCMmxqTndNTzRvdEJQZFhBWkg0RExPL2Y3Z0l1WERHT294SUVB?=
 =?utf-8?B?Ykg1MzFCVWtMUklxUTJBS1FLTjhiaGVhei9kcHUrMXRIaTBVT1FGTGVDWkVx?=
 =?utf-8?B?Q2pJRTdwcTNkaXVwdlo5M0g3aW8rTUZ1MHBwUm5RcW5hanBWRkpjeWxhNXB1?=
 =?utf-8?B?cGx3ZDcvK0RQL3pVZjNWVmdPdnFoczJSU1F3MFkwMXJnODZqbXVNTGtXdWZi?=
 =?utf-8?B?V2NKTTN0UHFPOUNiQ1k3YURrRC93N0liRXpTNGgvd0tNbit4UUtqajl0Q1ll?=
 =?utf-8?B?V1luaXFESHpVNmJaN3BQTUhENmFCNlh0VGQ1K0R2cEw4TVJ2d25yOVQ3Njc5?=
 =?utf-8?B?RU5vR1VMT1pyblkwQURTZTlmVkdNOVVOQUc3WVlvWEU4UExyS3V5WGM3VjU0?=
 =?utf-8?B?L3c4QUxQQ25XS2dvbUJOdGZxeTFEMTRDQlIwazRUME9rbTQ3ZEZzTy90VWNo?=
 =?utf-8?B?VmxVbm1YL2c3Z2dsRE4vdjNwb05Ua2ZWTzhKK055cFVIRStaa1pTTkFoUlA4?=
 =?utf-8?B?ZjY5QWg1SXlkOWZlSStoNnBuQVA0TVhHVzI5WGlmaTNvOG5rZEdlcklxRXMr?=
 =?utf-8?B?K0xLRVNTQ0FBelZqdWh5QXN5K24rWlgxZlQ0NDB1TFVuN2xrYXRrNVBHa0hK?=
 =?utf-8?B?NmlDbms4MXg1MmhaVStOdkNSbzZCVXNjQVlOVnlnS29WWVZOQmU2UmRVeDNV?=
 =?utf-8?B?Y3N0L1N0UVVjVGp6Y1ljV2gxMXFJeHVpRW41UVgxRzRmZjFnMFkrSThLREVm?=
 =?utf-8?B?OEtmMmQyK2x6UmpnYVkwQVk5bmhHOHRCL0xIQVlTS0NYcUNaSUN0ZE1Ya085?=
 =?utf-8?B?QnlQZURSeUMrYkhyTGk2MCswODRwWUxaZDNYMkJ4blhJdVJqdzB6Y09NZHJq?=
 =?utf-8?B?bnZQTTVYUTFnNUdtT1Y4UytYb210Uk53dEw2VC9GbWdrSEx1WDk4aGQvZHhB?=
 =?utf-8?B?MWZYSzhqcWh3dk1aRGRMWWxsdTN2Zk1aZWN4K09Bd3V1ZkRFbXMzak5zdmND?=
 =?utf-8?B?WEdkQ2gzNnlLSm1ENVpvOEhWN0hCcS9mcCtLZEdHZ2xVU1FyK3owVWt3QVQy?=
 =?utf-8?B?YXczbzVXUFRLTHZpaGNZTDA5eW9zTEhYRDdvTDRmL05RUXJRY2ljRlRESWFW?=
 =?utf-8?B?a1dkQkZuMHBoMm1LS2lzbGUvTzFRK2dRUk1kWExXUFNLN0JxdU5pVHFuZXY0?=
 =?utf-8?B?VDBSbUVBWnk4VitjRWlTKzBPVjJwQVhUL1M3cE4zK29wMmVOSUovVGFFeFJX?=
 =?utf-8?B?d283M21xaENzbkxxTzgwcmVnclZHL2xycmkzWjU5eHF3TEp3c25LcllnUEZD?=
 =?utf-8?B?NmZBVWkrZFdpdUNaSW85V01UeFhmUXZxSXBrOWhyVEJUWEVzdG9BTS9aYU12?=
 =?utf-8?B?cWNJMHo3K1VPSDl5M2FoK0pZeEVWVU95d29udCtKZzBYcFh1UXFjbHF4T1F3?=
 =?utf-8?B?ZXRTRFF2L0s2MHFaQSt2Rm9hdm16V3ZSRVBkeFA3a3dzOERkd0JvMlVqNFRI?=
 =?utf-8?B?eHVsckRYeVpwQjVjNGJ0UjMzczV2Ry9LM2lHZWJpQjkrcVVZVGRKejBjZndt?=
 =?utf-8?B?N2hrOUtkZnp3UUxHUDRkV1VobjMxOFJ2NHI0NTh0b1ZWaVp1bmJ2OFR1eThD?=
 =?utf-8?B?QTlHSlB1dzRrZ1RROTlkT1M3Y0N4bEtsL1FENnB2Y3piSmp2TG9vRGZlNWxB?=
 =?utf-8?B?T0ZJMGRNNFhmbWtHa00yMXZZdVYxUEJGN1BwZmp4bWtyWmxYNnhKU2YvZFFG?=
 =?utf-8?B?eS85WDdjTzY1TlVSTnZSV0phVnJSeHVLZXMrWk8zZFV6Y1RsWmFZaVdMYlR3?=
 =?utf-8?B?ZkhyM2hMSHduUlQ4VG5NWVFRRlRLc1dsY1BNWmtaR1lveFl5dE5obVh1WGFQ?=
 =?utf-8?B?MWc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 620aa439-6d1d-43de-20f2-08dc58430366
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Apr 2024 03:13:22.5505
 (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: c2LDD4YA+llG/Y/7oFKavnBXJ2SR6TrOakgl3ObjblA+JG80rr+Wvt6PTnAERn9OmUEKy9nTU9i1x9jACE0yY3TLnPLYXo0Ja1r7IpTcHnA=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR11MB4783
X-OriginatorOrg: intel.com

Hi James and x86 Maintainers,

Please consider the file movements as captured in the diffstat below:

On 3/21/2024 9:50 AM, James Morse wrote:
>  MAINTAINERS                               |    2 +
>  arch/Kconfig                              |    8 +
>  arch/x86/Kconfig                          |    5 +-
>  arch/x86/include/asm/resctrl.h            |   45 +-
>  arch/x86/kernel/cpu/resctrl/Makefile      |    5 +-
>  arch/x86/kernel/cpu/resctrl/core.c        |  119 +-
>  arch/x86/kernel/cpu/resctrl/ctrlmondata.c |  506 +--
>  arch/x86/kernel/cpu/resctrl/internal.h    |  433 +--
>  arch/x86/kernel/cpu/resctrl/monitor.c     |  813 +----
>  arch/x86/kernel/cpu/resctrl/pseudo_lock.c | 1126 +-----
>  arch/x86/kernel/cpu/resctrl/rdtgroup.c    | 3929 +-------------------
>  arch/x86/kernel/process_32.c              |    2 +-
>  arch/x86/kernel/process_64.c              |    2 +-
>  fs/Kconfig                                |    1 +
>  fs/Makefile                               |    1 +
>  fs/resctrl/Kconfig                        |   23 +
>  fs/resctrl/Makefile                       |    3 +
>  fs/resctrl/ctrlmondata.c                  |  527 +++
>  fs/resctrl/internal.h                     |  340 ++
>  fs/resctrl/monitor.c                      |  843 +++++
>  fs/resctrl/psuedo_lock.c                  | 1122 ++++++

(sidenote: James, please note typo in psuedo_lock.c)

>  fs/resctrl/rdtgroup.c                     | 4013 +++++++++++++++++++++
>  include/linux/resctrl.h                   |  153 +-
>  include/linux/resctrl_types.h             |   98 +
>  24 files changed, 7244 insertions(+), 6875 deletions(-)
>  create mode 100644 fs/resctrl/Kconfig
>  create mode 100644 fs/resctrl/Makefile
>  create mode 100644 fs/resctrl/ctrlmondata.c
>  create mode 100644 fs/resctrl/internal.h
>  create mode 100644 fs/resctrl/monitor.c
>  create mode 100644 fs/resctrl/psuedo_lock.c
>  create mode 100644 fs/resctrl/rdtgroup.c
>  create mode 100644 include/linux/resctrl_types.h

I would like to check in on the sentiments regarding maintaining the resctrl
contributions after this work is merged. Considering that resctrl will 
be split between fs/resctrl and arch/x86, would it still be acceptable for
resctrl code to both areas (filesystem as well as arch) to flow via the tip tree with
help from x86 maintainers?

How will Arm patches flow?

James, are you planning a separate MAINTAINERS entry for the Arm specific code?
I would like to propose that you add yourself as a reviewer to the existing resctrl
MAINTAINERS entry to learn when any changes are submitted that may impact Arm. 

Reinette