Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) client-ip=2620:137:e000::3:7;
Message-ID: <8436206b-b9a8-4ef9-9f9e-8fd5272e2ecd@intel.com>
Date:   Tue, 28 Nov 2023 14:11:18 -0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v2 05/26] selftests/resctrl: Mark get_cache_size()
 cache_type const
Content-Language: en-US
To:     =?UTF-8?Q?Ilpo_J=C3=A4rvinen?= <ilpo.jarvinen@linux.intel.com>,
        <linux-kselftest@vger.kernel.org>, Shuah Khan <shuah@kernel.org>,
        "Shaopeng Tan" <tan.shaopeng@jp.fujitsu.com>,
        =?UTF-8?Q?Maciej_Wiecz=C3=B3r-Retman?= 
        <maciej.wieczor-retman@intel.com>,
        Fenghua Yu <fenghua.yu@intel.com>
CC:     <linux-kernel@vger.kernel.org>
References: <20231120111340.7805-1-ilpo.jarvinen@linux.intel.com>
 <20231120111340.7805-6-ilpo.jarvinen@linux.intel.com>
From:   Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <20231120111340.7805-6-ilpo.jarvinen@linux.intel.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?WDRjbURCR3l1THBnWWs4b1hJcFpDcHllMUxEQ1l4M0lWTzFCVmdYNnhYbWQ0?=
 =?utf-8?B?blZSR0NPRWF4Mm95aUhUY3lha2YyaW5ES1o0WW5MOGl0Sjc4eG5zU0ZQYXlt?=
 =?utf-8?B?bngxUFFFQXB0eko2TGF3aHZzZlkvMS9XMEUvaUMwcS9EZWVwOXl5Q2NBZ1NS?=
 =?utf-8?B?MTlVNDZQNGZtWG1hTWNtMHQ5cm0vdWpZWHFTeTJncHZQWmd3NHUwRkY4V2xV?=
 =?utf-8?B?cHlnanJuVytGMkhXV3pBVlN0U0U2dlJZalVnaDNWa3pZV3hpR0JlTTQzaDM3?=
 =?utf-8?B?a1MyekhVdzJJaXVNcUpzWWlEUGt0NHNvTDVOejNiOVU2eWI5SVhNcTFqeVN1?=
 =?utf-8?B?VlVncnVCekRQRVRxTzNDbnZ4YlpIaXR3RktHZlV0SjlJRExyblMvbXY5SFFw?=
 =?utf-8?B?a0lwS0J6NXlrVWNMZmZxSEtNU1NFdEt5cVpheldDOVAySlZ0YzdjblpDZDdL?=
 =?utf-8?B?N3lHNGhIQW43bGlWZ1ZZdUtrbkdKMmJlSEhpYnZaN1NkT1R6TVlPY21xeXlh?=
 =?utf-8?B?UlN2cGRwRjNaQ29NaHNBTUpyN2pkd2pNdisxcmtNZHhYMVQ5VXA3cWk4emFt?=
 =?utf-8?B?NGVQd0ZHdE4zOGZzbHRMNUxSMFFrREJXcUpubTNueFhRd2VBUzEySkNhdFNM?=
 =?utf-8?B?eWtYTXFzaE9Oek5tNnRGNzBRYzZDUjFBazRxM0o5NGZhYWRkTklYZGwvb2xI?=
 =?utf-8?B?ckhmTDk4WDVDQml3ay83QU9Td3dWNTlKK0VoTXBSNFlpL0RQamoySzJPZXZw?=
 =?utf-8?B?UTF2dzFDT296UDlPMU80ZE1DYlZBbXRFeE9tUHpibEVqUTFaRkRBS0M5N1N4?=
 =?utf-8?B?eFNKT1o2OUU2RWxhTEtGYlJaQjBvOVFHWCtvMnZXZkhIR1dLYWZaZnQzTEVq?=
 =?utf-8?B?OEJSZGVsVTJ3R3hCRDc3N3EvK2RRQWt1T0svRExXb0pPRitNL0ZyaGZjUks3?=
 =?utf-8?B?WEJUMTdobjJqa2hSRVB3UTgxYXRObngzRjM5V21KejZtSXJaTlhqRUdoa0JJ?=
 =?utf-8?B?cjFGUFBidTRqTUR6RXQxVHpUY1MweWxUd3dyNllKaE1idTBuSk1TS0VSdFR3?=
 =?utf-8?B?VjFtUVN5VWdyK3Z4cWp1anE3bjA1cW1WdGJTbzYvS3BkOFF6NTRqS1NTQ1Az?=
 =?utf-8?B?TW5TZzRwZVNKRGdUb1ZZa2llTVZmMlIwYlU1WElkcTI2THJrKzl2SnA2Y3NK?=
 =?utf-8?B?SEhRcXFXNDllTGJBTW5HY3B4QW5odEVJdmZ3SmlXN0V6NjhnNTd1Q2hUMXZq?=
 =?utf-8?B?Vi9TaWU3K1hGUnNwODRvRE1rcFh0T1ZFaXFQMEc3RU4wUEVhVFFnVDRhNXZQ?=
 =?utf-8?B?eVdpOUdxVDUzdWpRZ1hxbEZ1THdmTUwyQUlEdklWd3gwb2FoSWs4ZDFlTWtE?=
 =?utf-8?B?TWdpcmh5RVVxRXNLalpIYW5tWGM4ZkZXMWUyM041d3F5SVNYRGVLNWZiM0VY?=
 =?utf-8?B?RkkzZEhQQ2wyVkFTamloeUdsOFM2U05nOWQyUWpwMlFUNkh0VGZkYnRYdFEz?=
 =?utf-8?B?cGc0L3NpTXFKblNuZnY4aGZwdEdQbm9WRHFFU1RpcGY0OXY5WjlLem5MOWhv?=
 =?utf-8?B?aERDek5wRkJUWHRvMHo5U3Q5TkNaTkpqSzdBd2RJOUpYN2JpblE4QVkxRFhK?=
 =?utf-8?B?dzRyNjQ2YWV1OEZ1WnA2SkxBTVgybVBpbmJvMVFaRXBwSVZqQnpDeXUzdzBv?=
 =?utf-8?B?d2xGMWJOUU5Cd3hXSEdCaDlZYnZlUmNwQ3FCVUFOaW1QZmJ0MXRMaDZFVXpJ?=
 =?utf-8?B?bnNOTXNHdlVHd09OUCtiaUZQN1drZjY5d0Iya3NjS3BaMGdUbGR2cHlUZUd1?=
 =?utf-8?B?QS9HbWhsRkhKTjFDRHB4M2d5YmhUbmtVVmxzazlOdjJSb200cDBSeUdhZThY?=
 =?utf-8?B?cmNVUFNub1YxdnEydFpOb0l5T2NVWnVFME9vWFRqMHFRRmQ3R3dhL1ladStI?=
 =?utf-8?B?WDR2YlNxK1FxcllLSXVueFZxUzNUT0grYWYraUpBV3Zqc3c4UmpyTUFhNnFE?=
 =?utf-8?B?bkY0UllqNkVIMmVFVXp5U0NjL1g2R1VURkFETGw4UEd3NEd5VWIzSnpKZGZR?=
 =?utf-8?B?Ly8yZGo0dnRySzd1ZzJ1TjgwWXBZdHFISkc2NXZlNzc5NmVsKytURUQzeVIw?=
 =?utf-8?B?dUtHdUlIQXoyNnVIcmxXQWd2YmJKUGlDRnNYZ3BzZWswcGVySUZEOGlrYUI3?=
 =?utf-8?B?YlE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: f003d29a-2105-4076-7832-08dbf05ef26b
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Nov 2023 22:11:19.0557
 (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: mp0xgZ1sy7T6yUY9t+hCKZkF1UnicocAtmBLlQ/MLSi15/O+cNE8J2fLyv4QKxeE/secSUJz8bl6cESr1kRlMoIbONTrXw2MZ5y61qQKal8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR11MB6328
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,
        RCVD_IN_DNSWL_BLOCKED,SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Tue, 28 Nov 2023 14:11:47 -0800 (PST)

Hi Ilpo,

On 11/20/2023 3:13 AM, Ilpo Järvinen wrote:
> get_cache_size() does not modify cache_type so it could be const.
> 
> Mark cache_type const so that const char * can be passed to it. This
> prevents warnings once many of the test parameters are marked const.
> 
> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
> ---

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

Reinette