Message-ID: <3a302f40-2c2c-daa2-4281-39b1ff7e90ea@oracle.com>
Date:   Tue, 7 Mar 2023 21:46:43 +0530
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.9.0
Subject: Re: [PATCH 5/5] cgroup/cpuset: Minor updates to test_cpuset_prs.sh
Content-Language: en-US
To:     Waiman Long <longman@redhat.com>, Tejun Heo <tj@kernel.org>,
        Zefan Li <lizefan.x@bytedance.com>,
        Johannes Weiner <hannes@cmpxchg.org>,
        Shuah Khan <shuah@kernel.org>
Cc:     cgroups@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-kselftest@vger.kernel.org, Will Deacon <will@kernel.org>,
        Peter Zijlstra <peterz@infradead.org>
References: <20230306200849.376804-1-longman@redhat.com>
 <20230306200849.376804-6-longman@redhat.com>
From:   Kamalesh Babulal <kamalesh.babulal@oracle.com>
In-Reply-To: <20230306200849.376804-6-longman@redhat.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?d0R6SHphUGYrWHMwV2R5cUxlV1JsSHphRSt3eUtjUDB5OFJSRGtCT1hsWkVE?=
 =?utf-8?B?S08wVEN0QVljSTRMS3NnSjU4QmZHU2kyeFBHMzFPYXhNeldGd3R4cXUxeU5P?=
 =?utf-8?B?OXRnTnk3N1R0R2JheGtnalFnMkw2RkYrQys4dCtOUmRCWUJSbmNIaEVhOVdo?=
 =?utf-8?B?VkFlcXpjNjFnVlRRNW4ramRNK2I2SVZJWEYrWGZKREVxbmNsU1FBa1lXeUVF?=
 =?utf-8?B?a1QvWVJlZEduRytkYXdvSUtSVys2aVNUMTFmYUZJcmNEZnB3dHBtczBHSlc4?=
 =?utf-8?B?S2xOVEpxdlRsU215akd4bnN0Q0tubWNlaDErVEMzZ0M1WldlYSttN0pQWUNJ?=
 =?utf-8?B?Zm0wYi9rTVU5bTBwM0lnRVp4OG4vZEI4ZFl2Q3dKaEIzZzkzZVZGbEJLTXd1?=
 =?utf-8?B?WHF2NmlEazZpQTZZSWV5eVFPTWRvYnliL2cyU0ppM21Zb2kxajZHa3EwaTBI?=
 =?utf-8?B?akVOdUh6UWR0SXE5dkI2ZGZFSjZROFhxTW5QRHpiblZNMnlZaHdPYS9YSXVY?=
 =?utf-8?B?eXNOWnp6NzlTKzVJcnJ6aFc1dEZuNEpaNTNTZnVDMXljNTJWeXc4WWJOK3Z3?=
 =?utf-8?B?RWowTUVsQy9rbjQ5YkpTTkpqOE5ka3ZSbkI2NEdhY01FTUlGWkRVa21ZWis5?=
 =?utf-8?B?MWxXc2VTN0hKWlZKT3cwRzlhUE9oRFNNTWY5Z2xZUEJBMWNjcVZwa0pxSXFr?=
 =?utf-8?B?NXBCb0kzcnFTa1JFZDBneGJzTUxseEpLcVBqVTdLM040SmdhRzA1NGMyUVlU?=
 =?utf-8?B?MU1yQkZud05wYjVJVFg3alJhQXg4WEtmSzk1ZkxRcmtjamtBTlpLWGJMalNq?=
 =?utf-8?B?dkRLaG4wbHFuc2Z2OVh0ZEJNeE1OQVRVWEFOQjJya3QwQmIzSWo1ckFkNERt?=
 =?utf-8?B?cTl5cy9UN3FDODZSbkVwRUtqMXBsNldEZ281WWxGcithY0dwLzRxYktlNHFh?=
 =?utf-8?B?QnJFUnA3SUFVNHdYR0FEbVQvbzZoTDBFc1BqWEhFWXZMZWtPVGZpUGptV1B2?=
 =?utf-8?B?R2RTZ090SnBIMTU4eEpSWCtRUi9MbGtIc2JtZ3ZYcmdtcmZCZHlqVnNCdzUw?=
 =?utf-8?B?MHFra2E5ZmZvZU1oZUFtSlg2U2YwMSt1UkQvdGVuR2llS0RyQzlWcmM1aFVN?=
 =?utf-8?B?dWI5MjZNWjVYUFZCcEZ0MzRwdjlHY0E1TlF4UFBhTVUrRE1SYzlWWDlnc2p1?=
 =?utf-8?B?bk9TbjVhU0o0ZU9kcTZkQnBDMWMxZ2ZTcDFIazZUbmVsMXA4WURDZkRxWmVE?=
 =?utf-8?B?QWR2UHBRQnVCVDFxdUtJUkNSOGlkdlowOUlVU0RkcHhHYVljNkxGU2hUdkJ0?=
 =?utf-8?B?NUdHdTFNMzdEcmVZSG92QTZoVlZobkloTWI3cTJIUGRxYXd0L2hsMzI1ZWdt?=
 =?utf-8?B?ZHlJNVZSa1NNUHZHVGdEUm82YUxsQ3RkNTMvZytHQmVJQ2UrVjNqckNHcjBY?=
 =?utf-8?B?NHVnblZTTzR4ZDVZODJTYXdqdHJ3cWJrUFlMTnJGVzJ3eG9CVHFsZ0xMMXUy?=
 =?utf-8?B?OE5hdnRTSllGYWZHeWFUWFZmMUtIeFVwdG4rbzZidUJwNlRSQWxHNmNlVGhX?=
 =?utf-8?B?Y0dzMjRJMHFJdncxWFczNDRMOFIyQVpkaS9WRHdOU0xweHBzcnIrUnEvclZU?=
 =?utf-8?B?YnZVY0ZaNzIrbU5RY2RmRm00TnRrUk5TTk9DZUJ4TllpV2dhVFhuTENnSlN2?=
 =?utf-8?B?MEJxTy9kQ3ZKTS9nc2VYUStuU0xYNmZrWENvRjFja09NaXFTcGkwSXFPT3RT?=
 =?utf-8?B?bnVlRFBtUXQ1NTNabk8rdm16YjBZSGFMZnZqN3JuNndnQ1FNRjFlc0VrZVFW?=
 =?utf-8?B?N2tlRkRNYjNSZFdPUmdLS1g2RkI3YVNvMFJPUWtVSmk1UWREalRHaGZLRDZW?=
 =?utf-8?B?K2xkMUFmRks0Z3NYdWw1YURhTm5jRDJwTlBQSGlCNUFreVMxVDdQUGU4LzVl?=
 =?utf-8?B?UENXZXY1dW0yS3pCOUR5bTZjbmxzazdyVXZoYURIQ3U4anpHSnhtckR0bFlM?=
 =?utf-8?B?REtOdHJtcEZWK2xva1F2OFg5U2J4VFk3dWdiYjJwTTB2ME5QZUw5RFJQZDNm?=
 =?utf-8?B?dTBrdWo0cXdXWEpZVlhmYW00RW9WWkZHdjhscy9XeTlZeEJoeDFJWjVGR1lT?=
 =?utf-8?B?QTF2aU9xalRRaERZS1lUWUFzaUxzSjFpZytHVytDNHJSVnA3MElmS1FPelpU?=
 =?utf-8?B?a1E9PQ==?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: EXw1TkqMoHgMJbt1zTpNZPOesCpwUZfbsCjIv119SsrLAsrKGsZPNhxhJZkRqywni8I+VAyG/8O/SVX3tq0ixW+TBxFy1gkoOvHgYIEqjM62+D+xGXINXCbkkgGqJe4U9HPnTmLH3QYpl3kzbxWv1XiFUpR3/vu7nK1w3FsS4IHnaTpcC9FKE5pxLBLLxwlm8Ly/Kwfbg1LO5AbRJLWVIUUlejPkX+uhvsrJGiABJ/avRSnJlELENs5CFDPcXWnpEjpEnGZ7eb6ybwjufbMkmAaU180cRit+PnIbROG3OsPhDIZ7IMfOg776inrYcCrqD+L8vRIiagYpBs1nUrUHkFF+zGfog5WG8gYdCQqHQ17NLNWlwopgtOz8dUGSm+SpxYB7wNaNqVHr29angox0VWW1tEASNqauAEx6AWU07cYIdtA7GIeDaKHneKqX+Iln7aj2RtXiwibNVaYcgYDsfXrUlD/4Me4R7quDI/S9OKHRcZDOACgD+k8pwwnq+ldyb4ZW7MW5uuC9TrVBhUhSIbrt1YzcARkdCLuZTdWP8oXSzfbzLN9XyGk7UlItuzCMbhz+ecGlTOPrviMTYlTOUhzTi+94Ws3dXGfaHRp+7v7ZOyrzcaImAFWRFqdnZULMuXOxWw/9N4LsB/kXOEChSfu43M2Dfqfp7b+Thy6mTgFka1ed1v5x6WuFhuJM9BhgOrgd0hE9ebHsnLgzyyw9FcUa7O/fAvRSvjfpKyMrJA1pwULaFkKeJKcAsupW5F+Rjnp3vwgpqopIrDRU7n1kFegZUGPLnrjU2TzH6QAoobxFf5uxqAqZCLZk1+I3B+qiTWfoR4vUpng1VzDIDeQM9I4L9pd0gkXvwar3k2ieeSPoXFjLwpURExWe9wdwh3UgIrbT1pFtDcpOptA7CV3TWfJvI2D/sZcL2tDNKBH9yic=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 6f90b812-a183-4bf0-f3e8-08db1f27604d
X-MS-Exchange-CrossTenant-AuthSource: DM6PR10MB3001.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Mar 2023 16:16:58.9526
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: OR7utJgDrm/jHtRfIms8ix3gsm95wGyu11NFKkfzmL/rdgOTWMBmCvRf54tWk+tU+OxiSCTfnYdYgEEX8hU75jtMfZSjuHqSAFNyCsmSlbg=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR10MB7474
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.219,Aquarius:18.0.942,Hydra:6.0.573,FMLib:17.11.170.22
 definitions=2023-03-07_11,2023-03-07_01,2023-02-09_01
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxscore=0 phishscore=0 bulkscore=0
 adultscore=0 suspectscore=0 spamscore=0 mlxlogscore=901 malwarescore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2212070000
 definitions=main-2303070146
X-Proofpoint-GUID: ky2CsnxKFHqv-mHivfwCUE9iO47d_S15
X-Proofpoint-ORIG-GUID: ky2CsnxKFHqv-mHivfwCUE9iO47d_S15
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 3/7/23 01:38, Waiman Long wrote:
> This patch makes the following minor updates to the cpuset partition
> testing script test_cpuset_prs.sh.
> 
>  - Remove online_cpus function call as it will be called anyway on exit
>    in cleanup.
>  - Make the enabling of sched/verbose debugfs flag conditional on the
>    "-v" verbose option and set DELAY_FACTOR to 2 in this case as cpuset
>    partition operations are likely to be slowed down by enabling that.
> 
> Signed-off-by: Waiman Long <longman@redhat.com>

Reviewed-by: Kamalesh Babulal <kamalesh.babulal@oracle.com>