Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <b9c0402a-5471-c8ce-9cf4-aa1a1a629b07@intel.com>
Date:   Thu, 13 Jul 2023 15:59:25 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Firefox/102.0 Thunderbird/102.13.0
Subject: Re: [PATCH v4 08/19] selftests/resctrl: Remove mum_resctrlfs from
 struct resctrl_val_param
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>,
        Fenghua Yu <fenghua.yu@intel.com>,
        <linux-kernel@vger.kernel.org>
References: <20230713131932.133258-1-ilpo.jarvinen@linux.intel.com>
 <20230713131932.133258-9-ilpo.jarvinen@linux.intel.com>
From:   Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <20230713131932.133258-9-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?UGZLQkRLS05SaFFSZW02cy9VZlI3eVdoRXVCTHlpcUswaUdFVkJXWVM0eldr?=
 =?utf-8?B?cGRTSWRZQU91SXI2dlk2Z0dvMGFCRVhEV3NXanplR2RxalJzQ2tqYWhYeDUv?=
 =?utf-8?B?WnJQL0NuQXB3bGhZalZHZEdjL0twTC9RN05BdFJ0cVd3YnhuMHVYQTNaNHV1?=
 =?utf-8?B?empCSjlBSzFCNThuK3pHYURDRVFCRktkZWJJVThKSDBDR1ByMHM1VjBJaWZh?=
 =?utf-8?B?eDRKL1EzMXJmTGxGdHRCUE5odlhBcDlqdGorYzR5ZUVKcmNsdmVaMnp1TDY2?=
 =?utf-8?B?TDZtVDE3SmM1MWZUcG9YZkRxMHFMdUlHeWhWai9hNVJSaG5rVmRPelNzSXlE?=
 =?utf-8?B?T3FBZ1JCVUdkSEg3QndYU0YyVC9uLzVPYkNOZ2luQ3phdEFwcTdVS3pYQjBP?=
 =?utf-8?B?blByeExGTkRiSk01QzJubTdSRGtzQWwwaDFNbGdmaWtzdWlGN2UxOUlyY0pR?=
 =?utf-8?B?OWc3dTVwMHlFMlIyVHZXNGN1Y3hBNzQzd2xmZTRGTGNmODJLL0ZtTVBDa2Q4?=
 =?utf-8?B?djA2bjhBa3FzNGtVNEM5MUFjUHFEcm9HaUUzenhWS01hTm01UFdMcjJJQ0ZU?=
 =?utf-8?B?VWFrUzFDSmlvSG44cjdvOTJsZFF2YTVhR0V2OEIrWit0a3B2Z0lGNmdHR0Rs?=
 =?utf-8?B?eVpOZW02bHdiWm1VeVcrSDNwQ09SazZhTnNvMC9YU3I0YlBHVUtmZW4yR3pH?=
 =?utf-8?B?Z3psSzNuejdXM2Rxc1cxeFZ4NW1vRVB0VFlEWFBjclp6a05VQ3VjQXNXTjNS?=
 =?utf-8?B?YnZLRFI2N2RWeU1mOU5wbENHaDZKeWZqTmJ2bmtzeTl4bmJHdW1wdU1ZaEZa?=
 =?utf-8?B?UDI5ZDFibFJ0eG5mL1NlYWlJUHpzTHRFeWRKSHc3cVhBNVZKSllKb2xLbjFh?=
 =?utf-8?B?N05na2V1em5GOFRhOVJ3RlpqTitpeTVyR004UXdWK3phUnZpWDZMZ3gyUUZT?=
 =?utf-8?B?VkFzQy8wUUZLZUhnRWRRNjRrekxWNGpWQndxUVU5bUZRdTA1d3ZmeWlzU296?=
 =?utf-8?B?M1puc2tYNytyMnNma05wSUxtTnVKYVQrM0VIUGlVUklaMURLa2xNWmE5cVBV?=
 =?utf-8?B?V0I3U0wwN09RL2ErNjNtRjFRdUZzNGVZV3JnOTRNaG0vQzlmdGRmVHBWY1hI?=
 =?utf-8?B?aVo1U3BzL3hsd2JvdlJ2TGcxMW1rOVN0aGQxdmxXNmFmK0Y2NDhIZjA5N1FM?=
 =?utf-8?B?bFZFNmp2NHR0czl5YlJJUWlwUWpXOVgzNlV5Q2tzdmIyVWlHRm1Jd05mMjRR?=
 =?utf-8?B?Z2xpbitzbm41M1BDU011RGViY0E4THMzRzc4aTNscGFaN2RxOGdvQmRTMFhx?=
 =?utf-8?B?d29ZNFRaeGM0aHFxN3VGTStVaXpBVUxIbzhOcUJaRmllVHlYSndZMldOL1NT?=
 =?utf-8?B?NjgrRE1jdHRRY1VGSm9tdmpNcktyeXRGUjBtSkpGbUhHZ2s1bGhETUloLytH?=
 =?utf-8?B?SFNkU2lqbVg4Y3Y5V1o3YmNyWkNyZGxzcnZJT3NaeVhmWU41ZS9sT1ZlRERz?=
 =?utf-8?B?cUhtNzR3M2tkY0V2eDM0aEFPSGJjclNFNnR2bXVRbmdHeEJjb2dsOXJVTWdy?=
 =?utf-8?B?K1hMbWNvVEtOS2xoeFBNaTlPL1dyaHRyN2JRcjhBWVJ1c2ZjdmtVRHNQbmEv?=
 =?utf-8?B?cUxVbElJWWhvVUY1YmsyT2NJTFk2dVVnYmZDaEs3WUVrVlJmTndsSis0R3hB?=
 =?utf-8?B?eVN2N3l6ckxBZEM1UlJUTDFRUlJkYlRmd081OU1zQVMyMVAvbGlST3lDMW5J?=
 =?utf-8?B?R3VMTjUvUU0vMy9Qc0NLN1pnUVVXcTQ0a2dNb2ZVWlduL1YrYStGTEs3MFVv?=
 =?utf-8?B?cEdaZFlaQi9EZW5RbHFuK2Z3UUlUa3pGZDNiNHU1WUx0QStRMWd2YmRDeWN4?=
 =?utf-8?B?bEVxZlVObW5XanNtdUhDSGUzQmJ0TG5aWERtc3BwTmpoQk9yUHRMTmg5TFUr?=
 =?utf-8?B?andtZTU2R0NJUjRaMENISU0wb3BjOWNBQ0lvNDBOSi9jT2lGbjdqSDI2WnE0?=
 =?utf-8?B?UElkSXVIbDJ2TkNYbGwyMmlkbkJzVFdxNmtvZ1lvV0JqdGh1Qk9FMjZ2WXRn?=
 =?utf-8?B?aDJPc0NtdzZmOCtBcC9JZDFNYjBRQ3FEMW9pUnRYWFgyd0RCS3JxZ1ZKSnRY?=
 =?utf-8?B?eVNVRi9NVkluRHNhUTEwdk4yczNYU0oxUG9HeHJETG1zWWthTnlHNDFTQUEx?=
 =?utf-8?B?aVE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 60bd242f-ce49-4907-efff-08db83f4cf61
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Jul 2023 22:59:27.8921
 (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: RVBX54HTje5fEF/TK0TEpZWF3vhYlxs2msMzUxOWvN3RpqYD8wfVG4IzIirJ55GdsqA3RsAefULHRtQLx9k+sJ5yfnFtItIrMcCf7pEWFY8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO6PR11MB5586
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.5 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_MED,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

Hi Ilpo,

On 7/13/2023 6:19 AM, Ilpo Järvinen wrote:
> Resctrl FS mount/umount are now cleanly paired leaving .mum_resctrlfs
> in the struct resctrl_val_param unused.
> 
> Remove .mum_resctrlfs from the struct resctrl_val_param that is
> leftover from the remount trickery and no longer needed.

s/needed/used. Actually, considering the first paragraph the
second paragraph can just be:
"Remove .mum_resctrlfs from struct resctrl_val_param."

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

With the changelog updated:

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

Reinette