Message-ID: <50d0e974-478f-ea60-9b7b-50982697ef96@intel.com>
Date:   Wed, 15 Mar 2023 17:04:10 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Firefox/102.0 Thunderbird/102.8.0
Subject: Re: [PATCH v2 9/9] selftests/resctrl: Correct get_llc_perf() param in
 function comment
Content-Language: en-US
To:     =?UTF-8?Q?Ilpo_J=c3=a4rvinen?= <ilpo.jarvinen@linux.intel.com>,
        <linux-kselftest@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        Fenghua Yu <fenghua.yu@intel.com>,
        Shuah Khan <shuah@kernel.org>
References: <20230215130605.31583-1-ilpo.jarvinen@linux.intel.com>
 <20230215130605.31583-10-ilpo.jarvinen@linux.intel.com>
From:   Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <20230215130605.31583-10-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?bzh2aXVaWmt6LzJtaVJtU2JQZVRaRERTdTdtYVRpUGlmcy9GOFlIQWViQThY?=
 =?utf-8?B?RkFrWHdEWXlvczNQYmk1d0c1My9NZmJDSWlWYU9tN09GdVlGWUNsVEk3N0Zw?=
 =?utf-8?B?bmxUZWIzendqais5TjI4RGlpMDhIcVZzdjNmS0JpYW95YzdPVDVZN2NYQnQx?=
 =?utf-8?B?SjFYbk9EaGo2Z1dvVm1Ca09xSVRXRUFCV0NJMXpLamVqMDN0Mm1aWjZlbG1s?=
 =?utf-8?B?cUMySmdrQ0VMYjZjc1RaRkttZi90RVlObGw5cUZGbXpQLzF5aG1UY2YybGd2?=
 =?utf-8?B?aUpUeVVLQ3ZrQ2VGRjY4Z0lzOTJDd1QzT3d1K09zVmhqdThwbjlKZGxBNTln?=
 =?utf-8?B?U3pEMjlBTUlDWWQ5TmNDZ0JCS2gyMzRnYy8wZ3k1Sno3Wm9GZ1d1ck03a1J4?=
 =?utf-8?B?Wm10ZTcvb0ZxdThPRWtFa3BDZUhOaE5FRDhFOXBvbEQwbWM5ZGlXQWhNQ3NX?=
 =?utf-8?B?bVNxZUhMa1pyOWhrK0xjY1cxa0NjN2l3enR3cGJ1d2xBMUNFZmszd21nU0Zx?=
 =?utf-8?B?NXoxMStOS095WnhrcUJ1NFNDbXNjNVBuWmxiN0tCdEsyV3liVkJiSmdXaHZu?=
 =?utf-8?B?aUlLZHBNWFM0bDVCUytXS2Z1UDliK0JpYmF4emJQcDkyWWRMSzNjU21zMG9W?=
 =?utf-8?B?UmxkR2lHWW1VZEp6QkU2WnQxMFc0eUppZ041ZFRKN3FubDJORDR1WlFhNkE2?=
 =?utf-8?B?SzJxeDFEdUVGeklIODkrVjY4K1kzWkE0VTNEZDRIaVlvN0NHTWNudTRQd3ly?=
 =?utf-8?B?TmU0TWFDUExQZHFkVjByMkNSS1RieitPQytuWTBaMlJtNlk0MmEyRzAwci9D?=
 =?utf-8?B?UGNGVEtOeC8wTUdqUTJ0K0RXNW5rVWhnTU44NWhpcHdWU3Q4YnVxdnVvSWVH?=
 =?utf-8?B?cWtZRFc1UE4raEtpN2tuektWc294dU1VUmo1TkxRMVFsTTlXblZPblZyalNH?=
 =?utf-8?B?cnozc1E5TGJPRmo5UjBKQ2UxNkdGbkE2MU5FM2dMenhaZHVQRVNHQm1Sd2kx?=
 =?utf-8?B?eTdaTUllZXJLZ1BWR1dzaithaWFyV3ZrMytIS1l4Ri80WjVibGpYNGRsR1RD?=
 =?utf-8?B?eUo3NDhOTFo5RTdhZDA3SUFYTk1NZnlhWmowTnllOTVEYTlBbFRITUxXVzBh?=
 =?utf-8?B?U2xJU0ZwK2pSWUZtblZMblkzOHFYamVxZExncnl4WEs5c2N4cU9qOUx2RzVy?=
 =?utf-8?B?M2UrbFZES3JCVlJqbVlTcWhGazAvZ25FWFN5L0xMY0FYSzFZVy9UWWZtdGtU?=
 =?utf-8?B?SXFSd2NkQjAvTU5Vc0dtMVhXcjhqcENFTnkzMmM2aXpTNVdYRFpTQUYrSTFx?=
 =?utf-8?B?RjQzS1NRNytPNGZpM0VsY3IzRmovVjd1U0k5bmc2Z0V5LzYzMDFFQUVDd2JB?=
 =?utf-8?B?Y251SjdoamlxYjdDTC9ITmV4aURGQ2FFOFI2VlV4M0RXSmV6d3FPT1lrcDZt?=
 =?utf-8?B?WEZtOHh4NVlreDVHY1NJZ1EveU9pMXhYMVJnOWo3WlNacHRvK0pFNWM4NldE?=
 =?utf-8?B?ZUZUK3RuQy9uaU1HcUYvMkh5N3djMmV3OUc2NEhZYVBIL0FPdWdxUVh1MDBr?=
 =?utf-8?B?RlVLWlc0ZjV4WUF2Z0hZWHhlMk12MmxFU1JIdlNsME55QndYanBJbVZNOVBu?=
 =?utf-8?B?eXd3bHdzOVJPdXpDOWlZWElCMFdXK2t3QlZlV1VuTUd6MTRLTkU2Y2NMekdp?=
 =?utf-8?B?WS8zN2tzdGRCVW1EY1Q0ZDRRQzNQc0xLejlXZmhnME4wRTRzTVBwdVkwTWVs?=
 =?utf-8?B?L0Vzc3FPdjhvM2VRSCtVVXU3YXJUZEpYVnBjaVFqMFBFUGxac2Mrc0FDUS9V?=
 =?utf-8?B?SU5hVDFZc0dpQ0RIWGdBbTEvV3UySktDd3QvMW1pVlZCMmtmYTVVcVRHMWNY?=
 =?utf-8?B?UGI5WkpZb29UK1ZUanpMc0hZRFM2TGdEblFNa3BHMkx6OVZ2THZkL1dnNWto?=
 =?utf-8?B?S1hYTDFINWpULzNEeFByNERFU05vbkluRDRlcFQ1T3UwU3hWVVdDRDNQSGl4?=
 =?utf-8?B?SmprOWZySnNuM3NPdlc2TmJqODgxNVBtK0k0WENvSDY5NkJsVEluQXVlVTJX?=
 =?utf-8?B?bzFETE9sRk9McUk1NXEreXE2Sm93Z2lvTmxOMDRxR0kwTTZaWWVWNXdGZWJm?=
 =?utf-8?B?TlBkd2xRYWVkMEkrSDRzKzFpcDNRWTNkNEhmSmJBcUQxZGpVdC9EV051Qi81?=
 =?utf-8?B?dHc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: a901d44f-2c0d-49aa-80db-08db25b1f99b
X-MS-Exchange-CrossTenant-AuthSource: CY4PR11MB1862.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Mar 2023 00:04:13.2557
 (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: CF+y/f+N1IHxEQ2Hfduw0TbK6B52MghCxY6/CrTDa7lm+SqljVb7JBFW3+RjLuX1tcc23EmMTSwA3FPHgMulwoaPMFaMjskekV0BqmLSNI4=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN9PR11MB5340
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Ilpo,

On 2/15/2023 5:06 AM, Ilpo Järvinen wrote:
> get_llc_perf() function comment refers to cpu_no parameter that does
> not exist.
> 
> Correct get_llc_perf() the comment to document llc_perf_miss instead.

"Correct the get_llc_perf() comment"? This is so minor and I do not think
a reason to resubmit whole series.

> 
> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
> ---

Thank you

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

Reinette