Received-SPF: pass (google.com: domain of linux-kernel+bounces-186726-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Message-ID: <e107e6bc-3fa0-4ad7-8749-498defe87094@intel.com>
Date: Wed, 22 May 2024 14:15:33 -0700
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v18 10/17] x86/resctrl: Refactor mkdir_mondata_subdir()
 with a helper function
To: Tony Luck <tony.luck@intel.com>, Fenghua Yu <fenghua.yu@intel.com>,
	"Maciej Wieczor-Retman" <maciej.wieczor-retman@intel.com>, Peter Newman
	<peternewman@google.com>, James Morse <james.morse@arm.com>, Babu Moger
	<babu.moger@amd.com>, Drew Fustini <dfustini@baylibre.com>, Dave Martin
	<Dave.Martin@arm.com>
CC: <x86@kernel.org>, <linux-kernel@vger.kernel.org>,
	<patches@lists.linux.dev>
References: <20240515222326.74166-1-tony.luck@intel.com>
 <20240515222326.74166-11-tony.luck@intel.com>
Content-Language: en-US
From: Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <20240515222326.74166-11-tony.luck@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?WXcrQjZ5QXVkSThYUXc4OFJDQmxVQVZGRnUyL0hxd0V5bmxyN3pNV3ovcC9q?=
 =?utf-8?B?WmE5aThORzJUM2xyZmpIbzVnOGUraU00VmxhSWpIekdtZjhJU3lIUit6aEl5?=
 =?utf-8?B?YXlmQWNGVGkrb091M0V4QmZ4MmFyNFROOXU5aGtwc3RuVTM0N1Mwd2lwbnh0?=
 =?utf-8?B?SUN0UlpGMUF3UngyM3hUcnRmTXhudGM4cmNFbDJoUWh1d1h1UkdRRmRBVGRt?=
 =?utf-8?B?ajFlakc0aTFSUVByYi9pQnY4bHduekQ2MXIwU0NwMWduamdYTE5BYjI0R011?=
 =?utf-8?B?aUpOQ3JkL1R0ODdNaTBqeW0yS0pBSkk5MXQreEhWK2dKZytuNVhsLzhBRGp1?=
 =?utf-8?B?V2pIU1pLSnpURnZHWWVUK3R6VVFGTjNNUjFkM3JmK3dLZ0VWeEUwQ2tTUXY0?=
 =?utf-8?B?TFZockhCUFF1K0hqZ3R6ZkttNVM2YytiYm5iM0k0NllNcit2NXBERFY5SFE1?=
 =?utf-8?B?cmF1R2JRbHp2cWt1ZU9DUzZ0L0VFd3d6L3g3TWVnZjJVSERqV3VqVnZtUWJL?=
 =?utf-8?B?dndyTk82WWtzVysyZjA3OUdCNUhCU3liemZCbGFaVG01c1RtVGgraXR6Qmgr?=
 =?utf-8?B?NUpkYjkyZWRBTmo2RGhFdkhWMkdGM0psUldHbWJ1cituOFFUZTdZQko2TkJm?=
 =?utf-8?B?WVM1K0tmMjRuMk1GMGRjVmE2K2NqMXJYcVNKRUd6VlpxMDJUUnlFdnVNNVFx?=
 =?utf-8?B?b2hFYVo1N1lKZnJOeFZZN01UZUlvcXhSeWdVSmhEQklmMmJ4RDJNTUNLdWN0?=
 =?utf-8?B?NXB3MDF4S3BSanAxVzBCVlpkdlRiS2F5S0U0N3g2UG1GSlR0NDVrNVpGUXVC?=
 =?utf-8?B?ZzYvbkJnZWdTV3EvUUx1dkJLVXkybHFYcE1qbEtoUEZoRnVsdUNGcUtZdFV6?=
 =?utf-8?B?TzM3V3V5UUNCQ1hGRW00RGorUk1qS29aTEo2d1grL2pJUWRES29DOXFyUmZK?=
 =?utf-8?B?SllvcUl0NzAzQ0t1dHRlaDNiTVM0VW1sQUk3QWh2SjBNcFphUGdQS1U3RHZD?=
 =?utf-8?B?WDM4T1liZU5qNitpanVUMC9PYXlmNDh1RHZqVGdkY2FZeGNiTEwrY3RIb1or?=
 =?utf-8?B?T1h0YjVtdlZtbDJwa1pmVHVlMkMxTStuNjNpek5sSURxTXRZOEZsdGRJNmdK?=
 =?utf-8?B?MVJPemc1OGxQWWcrYTZnYUpnb2hyNTM5ZnU5TDNxMnFzZUc5NGIyWVhHZXUx?=
 =?utf-8?B?WjdmL1FtRE54cDRrZTRiTEQ2QVJJRlhyTE5QNGdMSUdzWmhTelZqcGU1bHNI?=
 =?utf-8?B?NVp4WkNQN3V1TFFVRTZyMVJBVkgwQmF5dzVYNkdEZ0ttRFB1ck14Z1J5VWNh?=
 =?utf-8?B?Mm5Fc0duWHFrcEFhaTc1VldhTkwzYS9tVXZiZS9TM1ZOWU9SQ2puZnRVNGMr?=
 =?utf-8?B?NUtBRDZ3M2QrYktLU0UvTnBsenNnR21IYlcwT3Jia1J6OCtzVitLSmZPekVa?=
 =?utf-8?B?UDQvRzdCOTh1N1R6d3VobExVandmOEFISk0zUzVrMFlINGdEbHpOc0VHK1k2?=
 =?utf-8?B?Z3JaVU0za0w2QWI3ek5ENnFkdjJXUEZ2eml4VEliUDJ6b1htZGU3bmlXM1pX?=
 =?utf-8?B?TUp2QWtJcDFwVUNKNXFqNDVNMnN2bzVGU3YvSTg4Q3NKNGJaOGszdlNkQTA4?=
 =?utf-8?B?SXF6LzdlU1ZVL0p0QXkreDRrNzQyMDA0aWVhNENERUF6RUU0bXlNWlhwY1ZQ?=
 =?utf-8?B?VTdDaXp3UDFWL3VBZVlmVGFSKzR4Qm9tNkVFNGFZeHN3a1hJWmJZbXdJWHZY?=
 =?utf-8?B?TXdTc29zSUZrQWt5U0JMMTdlYVJtYUpCL3FkdDdaL0ROSmhMOUJwK1dpVkRQ?=
 =?utf-8?B?RHZnM2FmcGJnWlQ0SEQrV21GR2NFVlBiQVk5bUppTFFpR1JwekdHV3RnbWd2?=
 =?utf-8?B?bmVTVjdUSTJ2SUdzaW5IUVhlWWU4a3NjeURJUlRhODdXVDk2VThFYUh6UDFE?=
 =?utf-8?B?SW8wV25wVStma3U0RUk3Lzh3bnJvWnB2ajRoNTY5ZEdJYXdBTlBuQlRtU2NC?=
 =?utf-8?B?TDRnMnd3ekQrMlJrUHZqRGtZTkp6MnFFbDhOMjdkblliQ3NMS2xBMGhqZW9t?=
 =?utf-8?B?SmE4R29wSkR3Z2JvRzYrRWZ4b2toZzF5Q1RJNGNwQ21HWUJGcHVUUEdlNkhm?=
 =?utf-8?B?REI1eFdSbnZQOEpoQjJORC9JMHB5ZmN6aHZHTDk2M20yRXhpc1JiRTRSVENp?=
 =?utf-8?B?VUE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: e76b45df-6ef4-49d6-6f9e-08dc7aa45262
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 May 2024 21:15:35.6911
 (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: 292nXlOfCYW5kv/3/O7QKQYXAMyj3KTkP5pS0r9BDPp/gqFKo4mNu2mdCuvLKZjsmGUjWvUA83YV1IFWSW9llvsLorxY6Jk/WW4Ol+dNihc=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ2PR11MB7671
X-OriginatorOrg: intel.com

Hi Tony,

On 5/15/2024 3:23 PM, Tony Luck wrote:
> Move the creation of monitoring files into a helper function.

Could you please elaborate why this is needed using a changelog 
following x86 customs?

Reinette