Received-SPF: pass (google.com: domain of linux-kernel+bounces-206592-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Message-ID: <45758766-f3f6-455d-ba34-36b9fbad80cb@intel.com>
Date: Fri, 7 Jun 2024 11:16:24 -0700
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v6 15/16] selftests/resctrl: Remove mongrp from CMT test
To: =?UTF-8?Q?Ilpo_J=C3=A4rvinen?= <ilpo.jarvinen@linux.intel.com>,
	<linux-kselftest@vger.kernel.org>, Shuah Khan <shuah@kernel.org>,
	=?UTF-8?Q?Maciej_Wiecz=C3=B3r-Retman?= <maciej.wieczor-retman@intel.com>,
	Babu Moger <babu.moger@amd.com>
CC: Fenghua Yu <fenghua.yu@intel.com>, <linux-kernel@vger.kernel.org>, "Shuah
 Khan" <skhan@linuxfoundation.org>
References: <20240607125316.7089-1-ilpo.jarvinen@linux.intel.com>
 <20240607125316.7089-16-ilpo.jarvinen@linux.intel.com>
From: Reinette Chatre <reinette.chatre@intel.com>
Content-Language: en-US
In-Reply-To: <20240607125316.7089-16-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?aFQwTGYvM09vVHNJNzRYazFtMXZQUm9qZHlhVkJJTHUyRVZoZGtLdkRFaGZL?=
 =?utf-8?B?UnZMSVpPOHFxa0VDVU1nZnFacDNqQVBaZVNXRm00NVozUXVLdGdhOUhwN0dJ?=
 =?utf-8?B?amE0YlJoS3BWZFZNOEZEdjFhZ2pUc3h5VnZmdHBBVDlzZzFVUllZWXQ0d0U1?=
 =?utf-8?B?bHdtV1B5a3huUWV2NDhRUzNWRHZNL09JOFpmbXpaaTNSZDcyUVBNOWlVcUQ0?=
 =?utf-8?B?WitWbWxXOENaWk00RElNeFZDWStqUjQ0YTgvdllzcy9LNEVTWDFjUnF1Ulpa?=
 =?utf-8?B?M3orcC8yT2MrRDFLcktIdEpzOFlxWjJlNDZ3d2g5VURjZVc2Z1FlcnMzY2du?=
 =?utf-8?B?bVlVQ0pRQ2RUNzVLNnM2NHZCRVArMmxJOWFrYVVOVS9IT2JsSG9Hd1c2QkRX?=
 =?utf-8?B?WHQ4b28zOXdGQ242bEZwanZER0ZyNXk4Wmkydi9ScklTVWJ2Z2U4M3BnakhE?=
 =?utf-8?B?Qjl2aDBBWUV3V0g4bmpMVnoyaU1aaS8yWG9IT0JSeUlMZmFRSlFMZ0RXOFlR?=
 =?utf-8?B?bE40MkhKbGZaNHBWa0k4bzZlZW5ISVpFR0k0em5MdS9tVnFVL3N1V0xibFZq?=
 =?utf-8?B?WFBQcmhxY3ZDNzdoZE5tRE1VL0d1Vmc4UXFKMUpZTWpnWWJjckxlSXl0NW5Q?=
 =?utf-8?B?NW0zUUpwb1ZSR0FDL05samxPaldnMVZBYzNRbks3dkVjSldTVlp6UlJLYVdK?=
 =?utf-8?B?aHlmRGVldVFvM1JSb2Y3cmZsazIwNzV4NzhxWmIraTNEcm1DNHpVQ0d3dkZX?=
 =?utf-8?B?d0ZVM3VYTHBXNzhlczNJcW92SzdkViswTWk5OFVQbEZVQlBVeUtQV0hEMDlo?=
 =?utf-8?B?N28xSnE4TXJjS3crUGhsUGpaS1JiTDZRWkxlOXlBYTQ5SmJVT2lVRy9JSkRp?=
 =?utf-8?B?dlA0Vyt4bnpWQ25IOGxyN0RhTkRTdkNqYk1vU29oMktpUEtSRDdHRHZ5TEEv?=
 =?utf-8?B?cFAyWFhicVpDcTlqNVNyd1cxaGJFdE1ZRG1za21HUUpmL20zUmRNMTZaV2po?=
 =?utf-8?B?KzhTK3Y5WjA5MTdmc1FKY1dBRUh3WnN5RlZxeHNYYzFCeFVuRUxLeFoxVW9S?=
 =?utf-8?B?aUFFcjZhcHNwU1dFUks2TFV0d0tPdjdCWkczUS9TaFVFZGZXN1FVT1h5cytG?=
 =?utf-8?B?a3p2NXFCUGVUR28zT3huZGRUalkwZFBXek1SUCtaMFBXMFJYaWt2M1BRZWNX?=
 =?utf-8?B?Q2Z6U0hZTHh5ckwraHYxVU4waGZHZG1vRkpWVzRzZ2cwUWhVdnRYVlBlYTJY?=
 =?utf-8?B?K2ZlNTdndDNHL2hMMk9hK3lIaHZrc2EwWHk3SEFNVlFFQzllVjY2YnFCL0Jy?=
 =?utf-8?B?TU52YTYvOE1oRjA3QVpndS9SVlF5SEdCVU1ZK0xTSlNOOE51S1AzcnhpRUZp?=
 =?utf-8?B?d2ZOekVGaVpRSGluYTYwVnZjcVNqRjBCa0swSWxKajZMcmJpbERzdFVJRExF?=
 =?utf-8?B?R3A5c1RROHkzK0JtQUdXWGszM1ZGTHdlQStpSmNhZUhaRjU3ZG5vWE1KNU1q?=
 =?utf-8?B?V2Jna3lYbS94LzFGWlBlVGx6aXBaNFA3M3hEd0hQYTFDbEJIOGc0a1lyem9V?=
 =?utf-8?B?RjR1K1Z2NzhYM0FxUU5NdHRGM3hUUHkxazYxTEZ4aFdtNWJvMkg0Q2JiVWdm?=
 =?utf-8?B?ckwzZUhjU1B1MXYvSzNWVFp6dlk3Y0tpTmNuMWtZeWE3TGdHWHVTUE1HSDZj?=
 =?utf-8?B?ajY2WGRsei90VkFmQWYwbm9wbm9xZGgrY2VoREd0MEVUT1dhRGlJNlBudHlv?=
 =?utf-8?B?ZGkzWjQ3RTJjdUpnOXNVdUlibU5xdmhObUN0V3BaTWUxc0RFYVJLaUh1TTN1?=
 =?utf-8?B?bDVoWWx5cUZ6UEhEaEwzb1BXM2JVMHhNMnZKWDBlTWhLSmJSMTcrVjd3SFlx?=
 =?utf-8?B?T0tRVFdWMnFlMjNqYjErZmxKNVN1QmpZdzFReUJhdWhhUE5DNnhsTEVWWFNm?=
 =?utf-8?B?WDdHbEFJcmFwU0lmb3Nmb0h1azNxOFpJUTV3R0s1RXZnSS9MKzdYeEJyUmV5?=
 =?utf-8?B?djBkYWFVZ3pMUEhGZ1lXbzY2S3Uzc0xoT1RjQ3VWTVdQZlc4M0ZEdWNsTXhl?=
 =?utf-8?B?NHo0VnY0aFMzU1l6R3d5OUhvd1p5djdHWFFFZGcydUV4bGpRQnp0NUdnaWo5?=
 =?utf-8?B?WWxnZ1NCZENhazJGcWpZL0k4blJHSlZ1djFwOWR6MUFZUEpqZkxVZ2NGOHpm?=
 =?utf-8?B?alE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: bbd7559b-939a-4494-1d8c-08dc871df261
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Jun 2024 18:16:27.1431
 (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: 0EgFaJaQoqd546PvIArIuJ8uqAjRVe8bguwwBsehUPpLA5djKPcQiRj47MUZzVDbG7OQNRzUPtqxLG6ocD/D1prDZQCTwIPu6t/x3QlCogU=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR11MB7605
X-OriginatorOrg: intel.com

Hi Ilpo,

On 6/7/24 5:53 AM, Ilpo Järvinen wrote:
> The CMT selftest instantiates a monitor group to read LLC occupancy.
> Since the test also creates a control group, it is unnecessary to
> create another one for monitoring because control groups already
> provide monitoring too.
> 
> Remove the unnecessary monitor group from the CMT selftest.
> 
> Suggested-by: Reinette Chatre <reinette.chatre@intel.com>
> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
> Tested-by: Babu Moger <babu.moger@amd.com>
> Reviewed-by: Reinette Chatre <reinette.chatre@intel.com>
> --

"--" -> "---" to prevent snippet below from being included in
changelog.

Reinette