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: <96b6b5ae-078f-fbde-7cd9-d64d766f6727@intel.com>
Date:   Wed, 12 Apr 2023 10:30:47 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Firefox/102.0 Thunderbird/102.9.1
Subject: Re: [PATCH v8 0/6] Some improvements of resctrl selftest
Content-Language: en-US
To:     Shaopeng Tan <tan.shaopeng@jp.fujitsu.com>,
        Fenghua Yu <fenghua.yu@intel.com>,
        Shuah Khan <shuah@kernel.org>
CC:     <linux-kernel@vger.kernel.org>, <linux-kselftest@vger.kernel.org>
References: <20230215083230.3155897-1-tan.shaopeng@jp.fujitsu.com>
From:   Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <20230215083230.3155897-1-tan.shaopeng@jp.fujitsu.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?NEFNdllxcGIveU9IZlV6akJSMGxGcEsydHJwQkVpLzV4dDBJeDhWbjVaTkgw?=
 =?utf-8?B?YXlCRmZQMzNSQkQ4ODJGMUE3RVJPdmdpZCsyVnhERk5Ocnc0alFON0svY1BK?=
 =?utf-8?B?YXJLMlRNZFBwamdZSFlOU2xidjdkRGo5VEVYUUNvOWZNZ29ETFZMa3l0ejky?=
 =?utf-8?B?Wkd5RnVTRUc4Q24wZWkzQWZidTE4cTV2ZGNJZUpSZ0FpdXJVZ2ljbFVobDFG?=
 =?utf-8?B?Q0pDMkNXSWdBYjdzMUFCYmlvU0djZkVPc0ZQcXZNUC9KVEJHSUdaR3cxOG4x?=
 =?utf-8?B?RllxVnN2QTVoa2FrUE5hbDU5ejJpRGJBVTJOTStIZ1ZDNE5TcDVMM2dHWWE0?=
 =?utf-8?B?cG9TeUwyUXZ2YTJKd1ZIUDgyWEpvUUR4SVJMSTJoaWhidVNyalZlL0wvZGo4?=
 =?utf-8?B?WEFBUVQwV01LUDFDa0ovTmdUNzBBZDgzR0pMQ1ZEcnhpTDlGeFdvSDRBZVJr?=
 =?utf-8?B?djQzWitDeWhTRFVKSVBqRkFUTzBtOE1SWlg3MHdZREFsdGxFeUdVR0hsMmlM?=
 =?utf-8?B?dGIvR1NQS2ZZV2RXY3JucWpjL05nVDVlNE9EMmdWbmV1RXMxMDZkNmRQUDFx?=
 =?utf-8?B?eHo4NDU0M1FSWVNZQ0pCdnhtdm5UMmdvMVR3cmVvNFRYYmp5cXc5TDBQSC8z?=
 =?utf-8?B?L0gzM2ZiMEFlcVdsRUpvRk16a2FqYnQvWGx2YWk3TXQ2VVNQL0VjNEMrTksy?=
 =?utf-8?B?cjcxdXlLWVJqRS9iWGhFUUFZa3RNeVozT3B2NmhRWmZHTzdmOXQ3cFRGajFT?=
 =?utf-8?B?bkdxWHhwNTdXb2U1azNkZlBHQUhJUXVKZTFyWDFiWG9RWUxadVlpeFFmbXpt?=
 =?utf-8?B?akp0ZGYrTzc1TDZxd3FObFpUMVBvTXpUOGhuV2tua3dabjdtNDltb2N2R1BL?=
 =?utf-8?B?eVordFhCdmRaVjkrL3ZVcjBaMENPZTVEa3J5WWxjR3hFVElQRitNTTdhcE5S?=
 =?utf-8?B?blhWa1pMdzVwUVJ6QWwvZ2NjZ054ejV6bzRuTTZ0dEFXTWdXbENLaDlVNitB?=
 =?utf-8?B?bDMzVkNGaHcrQ0FRUE95cGZ6dHdaOGc0SUtRREptOTdlamljRitFWlZFN21G?=
 =?utf-8?B?L3BqVlI3MVRJbkdhZUJEVnRPVi9Zb3o2blBuNjFPMzdzaVdQNFNBVTE0OVFK?=
 =?utf-8?B?ZFVJUmYxdUg4MDhWQlg5YjZjbEluNmdsZTJVLzhhd3lVeWoxQXNsQ0FNWWdW?=
 =?utf-8?B?bVhMbkY5Q0EwUzBiNXA3c04za2dXSElZUmZrbTRoR1RTNU1EeC9wTFhCTWtU?=
 =?utf-8?B?VW90NDhEZU9UeE5QWmNKS0YwMEdWRHdPMHR5TUlJeHdtdkREUDhKTnE2UW8z?=
 =?utf-8?B?Uy96cWFqMS9NOE9xbjhwVHVtdjQrTjM2YWZrT0x2QUphUDFLZ0k2MXNGWE5D?=
 =?utf-8?B?azk1cnR4d01MR1AwT1I3blVYTzNIc3crdHk1VUY4ZnV5Z2owSEp5Y0xUY1gr?=
 =?utf-8?B?NFRyMmthd3R3bzg5KzJzREFMQ3JSWktIUDEwdEZ0YmRaRFlZN2J2eTk5Q2Rm?=
 =?utf-8?B?K1pQVkh0bngvM3lpazZtRm1WQTdIRlBiVS8yM1c2eXBqdFVIdlJBSTFrSFdS?=
 =?utf-8?B?ZjhGQkIrcXNpSWVPbFJycStmbURtNzEzRVlJMkpCSDMzV3pvWmY2djIraXh1?=
 =?utf-8?B?a2kzNzhwd0ZJNjBjSTBKT0ZTNmVmYVdCd0NHd0p6UWhGcjFhV2JaV09OeVpQ?=
 =?utf-8?B?NGFrOVlOd3Z2dFhPRFNOb3RYQ3lmbkRrbHFrMEgxSWhsOHNCcElMMDFjNnpn?=
 =?utf-8?B?UVEyQ3h2djQzbzFHRWxMcjY2VlZTdkxJanMxRkd2anJHUkExdWJVZ3Z3blhv?=
 =?utf-8?B?d2oyR0Ura3VOOXI2dVJMN2xibG5pSHBxRUdJaFk1enhNM3lGNFhCNlVFenM4?=
 =?utf-8?B?aDBXVnRPNnU0R0xHTHBLdUxGVFNoRVVOemU2Um00eFFyYlk5R0lETnZBN25X?=
 =?utf-8?B?TTB1OUluZER3ZW9nR1dLREQwK3J2SDZRejBIY3Q0NFM5VTBsNEJFZ2RtSERS?=
 =?utf-8?B?WEhwRGszN3BmeXVoMk5iemhvM09JV0dZc2w3WUFVZlVnYWxaYlRva2JueTFK?=
 =?utf-8?B?dEx2QlJpazdsSTdVS2ZMVU1YQ1NESFUrcERuTzFLU1lRbytFcW1QTWhya3k2?=
 =?utf-8?B?Mlhjb3dkVmVHUWlPdlNUZnduZzZ5dlJTcjFrazB1Vm5vYkg3NTVQTXJ5ckVl?=
 =?utf-8?B?QUE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: e10f9553-a05e-47bf-a6b4-08db3b7ba8e0
X-MS-Exchange-CrossTenant-AuthSource: CY4PR11MB1862.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Apr 2023 17:30:50.5682
 (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: LY3tRC+Rj8NMQ+9sV4wfztgWGe5o6twkAmPA32Ub9RT6dBQDILhTTUi5oiAlMr3notOwwl9quJi9/iKpZ3K+7OlmPv/qeHnVL8uonyG5ZG4=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY8PR11MB6817
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-5.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,URIBL_BLOCKED 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 Shaopeng,

On 2/15/2023 12:32 AM, Shaopeng Tan wrote:
> Hello,
> 
> The aim of this patch series is to improve the resctrl selftest.
> Without these fixes, some unnecessary processing will be executed
> and test results will be confusing. 
> There is no behavior change in test themselves.
> 
> [patch 1] Make write_schemata() run to set up shemata with 100% allocation
> 	  on first run in MBM test.
> [patch 2] The MBA test result message is always output as "ok",
> 	  make output message to be "not ok" if MBA check result is failed.
> [patch 3] When a child process is created by fork(), the buffer of the 
> 	  parent process is also copied. Flush the buffer before
> 	  executing fork().
> [patch 4] An error occurs whether in parents process or child process,
> 	  the parents process always kills child process and runs
> 	  umount_resctrlfs(), and the child process always waits to be
> 	  killed by the parent process.
> [patch 5] If a signal received, to cleanup properly before exiting the
> 	  parent process, commonize the signal handler registered for 
> 	  CMT/MBM/MBA tests and reuse it in CAT, also unregister the 
> 	  signal handler at the end of each test.
> [patch 6] Before exiting each test CMT/CAT/MBM/MBA, clear test result 
> 	  files function cat/cmt/mbm/mba_test_cleanup() are called
> 	  twice. Delete once.
> 
> This patch series is based on Linux v6.2-rc7.
> 

A later patch series was picked up by kselftest causing a conflict with
this series. Could you please submit a new version that is based on
the "next" branch of
git://git.kernel.org/pub/scm/linux/kernel/git/shuah/linux-kselftest.git ?

Thank you very much

Reinette