Received-SPF: pass (google.com: domain of linux-kernel+bounces-194529-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: <083f8f41-5192-48df-9498-b448cad2d23c@intel.com>
Date: Wed, 29 May 2024 10:47:10 -0700
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v4 12/16] selftests/resctrl: Make some strings passed to
 resctrlfs functions const
To: =?UTF-8?Q?Ilpo_J=C3=A4rvinen?= <ilpo.jarvinen@linux.intel.com>,
	<linux-kselftest@vger.kernel.org>, Shuah Khan <shuah@kernel.org>, Babu Moger
	<babu.moger@amd.com>, =?UTF-8?Q?Maciej_Wiecz=C3=B3r-Retman?=
	<maciej.wieczor-retman@intel.com>
CC: <linux-kernel@vger.kernel.org>, Fenghua Yu <fenghua.yu@intel.com>, "Shuah
 Khan" <skhan@linuxfoundation.org>
References: <20240520123020.18938-1-ilpo.jarvinen@linux.intel.com>
 <20240520123020.18938-13-ilpo.jarvinen@linux.intel.com>
Content-Language: en-US
From: Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <20240520123020.18938-13-ilpo.jarvinen@linux.intel.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?S3FDeklsSHJjZEZGdXlBaVJkaDlsTFJUZDZmbFN6MVpjc2RpOFFnY1hNK1du?=
 =?utf-8?B?SDNiMUV2cXE1MldScUJCdUxua2RCQjU3WHlMUmJKTHhQYVhhN0hRVEFGejRC?=
 =?utf-8?B?Q0c2N1AvaTRSY2g2alhIWmdUQjBLRWxuak4yUjhvTEZlSnNPU3JadDRsejBW?=
 =?utf-8?B?ZWltRVZUNXdTRXJKa0hvTjU4a0VDZ3k2ZzFBRFZyRWRrdlU2SWx6WUV5WnNr?=
 =?utf-8?B?UVovaG9GdW9TVmFLU2ZoMHQ0WGxFdmM5Vk93dmswd2ZUYXh6RGhmSDhHaUNZ?=
 =?utf-8?B?c3kra1FZQ25kRjRRakdBdzluSE5MSVdlVW1VWnlEZTlxazN4akN1SWFtOXg4?=
 =?utf-8?B?cWVLK1NtdTlJNzd2c0FoakVNS25ZS1NkUGRKZHZBTEU4elhWa1NNZ1d5TzFq?=
 =?utf-8?B?R0kvZXkrN05kZy95RDlVMWdhRmFyNW5aTS9sTGtmK1VkUE5XZ0N6ZVhVNzBG?=
 =?utf-8?B?aUNBdzJpa2ZmMzNBVVR5VEpNRDNXMzN5Z1phVjZXSVUwcDFqRkxERGtuSDNn?=
 =?utf-8?B?d3cyRFpPbllndEdzSzFVby9MOEMwNG9USjUwdkhUOE4ycis1bk1ObjdlOFZX?=
 =?utf-8?B?RnV3SVFFZGhHZ3BwVEh3UEtWWkxoM3hTeGMvam5Kbmp4SHByTGFzUUdhZklw?=
 =?utf-8?B?ZkVPbmNSRXc4RndtSS8zNU0rL0w2MGFDcXI2bU1YQVYrdTRxVmVSbWZodFZU?=
 =?utf-8?B?d2lFSXAzNUlUL1c1VjRIM3EvbGdtbjhxYW5ma1ZqVXU4aHVXVlhJSXVKaWR1?=
 =?utf-8?B?UDkvODZwZUJTRnU4dHllaFp5NEIyTzJ6OVBxb3k5dDB0RHgzbmo3cWFmcnJD?=
 =?utf-8?B?MU5Rbkd4U0tsbG9VRjJOMDk5dGk1TkUyMGozTkszVjc0Y1kyWkorcG13VzB2?=
 =?utf-8?B?U3ZlM3Nib2JhMEZSS3h2aXdnN2xUSCtnNENlM1p3dUdQM1MrUlRqM3JxZlJB?=
 =?utf-8?B?N053QXpGQ3ArUHFoU0tNcFVvcC9CSTZyNHBjR1lyMTdlSU5VWlphemRTVDI4?=
 =?utf-8?B?cnBwRlloK252NHFyLys4b2JaWklLdGsyTTdwSk9KQW1oeThYcVJRcDNnNytm?=
 =?utf-8?B?Qm02KzFBdW5QanJZL2Q2cGQrRXIxb3lPdlRRRWdSOERDeGtVVG92aXNqS3cx?=
 =?utf-8?B?eHg2eGMzWG5VTXZIejRRd0tGUXdrN25uNThpSTJiS2ZFNXU3MFlkOForUTFp?=
 =?utf-8?B?dVBVWURsTDMxQU1zQkNJcDdwS2J5MnJzVk0xSHFLd29ndzUxcGo3QWtnOUFo?=
 =?utf-8?B?bDcrVTVmYVgrM3hwV1BQODZCZnlHNEp0eUxIeDJoMyt1TGVFNzZ6cmduZ0NG?=
 =?utf-8?B?Z1RyOG5KbFBhZ1NHTnlWWkVRV3d6NkVVOHFKcUYzd0lwTGVXZkJLK21tbXRu?=
 =?utf-8?B?SVl1UXhKY1dyOXNLZGYwZDAvMTM2ZHdXUVo0bFR4bVNOby9aYmYwcWszSUJv?=
 =?utf-8?B?V1RBSy95dkE0MTY5TEhBTTZDbFhNZjB1VVlnczE3dHB3S2Q2TjFkdGpXb29s?=
 =?utf-8?B?L3o4Y0ZrVHVVL3prd0ZDRENFT2NON0ZFcGJ5Z1J5ekpueEJZMHhma0ptZ1NV?=
 =?utf-8?B?OUpJZWNqMERWVUFtU3Q2YWJLcWNyYWhuL1pvOFJ0dGpPM2Q4aWE2eS9NdHRP?=
 =?utf-8?B?cWtjNG9vUXVITjVHaWF2ZjNmSUFlYSs3WGRrZXdyRXNtRjJGY0ozNk81T2J0?=
 =?utf-8?B?WEFpYWI1NUhIRWROaUxwZzUzVkZ1WEV6NGVQZEI1RTk0VkNBV1lySjBvV0Yr?=
 =?utf-8?B?ZGxqUGIyZnZLdXdRZi9kZ3ZsVytMYVZmeEw5MjlQWExXc0ppUkRPVVRCV2tD?=
 =?utf-8?B?aThmeXBOTTdSMXE1UjNmc0Q0Wm12Mm02eTB1dWFVVHlBMmxNdUkvZHZ0Rk5r?=
 =?utf-8?B?QU1PMTQ5U0d6Qk9pTFFWSXQzSzhPVk1lRWFqeldVN0REQ1hvSXhvajFETjVk?=
 =?utf-8?B?MzB4NS91SWpHdVpqT3ZPSXFwR3FjMGpER20yRit3RGdkOFdvcXg4VUV3MXpS?=
 =?utf-8?B?aHk5YkV2WHdTaU9RVUtxNEU4VzZrVnE5QjZueGNxZXJVclNBVG05bnlDNEVM?=
 =?utf-8?B?T3BtTDN3aVFyYWRWR0ErQ3p1K0w1UUtjQjFpb0FiVGxyRjFPK1c3cHJDQmhZ?=
 =?utf-8?B?OWFnZFhOdEEvcVZIakE0VnRMb3cwendkR3lORSt0cGIyNFpUWmNYaExZWWVl?=
 =?utf-8?B?RUE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: a27b6c9b-b57b-4d7e-94d9-08dc80075ecc
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 May 2024 17:47:12.5189
 (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: bizJhyo+PrFPQEUEPMBfpssh/J2IR6e4WkQIKWvf9w6435VKaCTUZnnrIKI0AmTYhT1oh5OadNTwpTEfFpcp8LhdzIqfxhFMXaNjnx5qFAw=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW3PR11MB4587
X-OriginatorOrg: intel.com

Hi Ilpo,

On 5/20/24 5:30 AM, Ilpo Järvinen wrote:
> Control group, monitor group and resctrl_val are not mutated and
> should not be mutated within resctrlfs.c functions.
> 
> Mark this by using const char * for the arguments.
> 
> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
> ---

Reviewed-by: Reinette Chatre <reinette.chatre@intel.com>

Reinette