Received-SPF: pass (google.com: domain of linux-kernel+bounces-77834-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <f32b90fe-2d56-4e9e-8f44-c7ee662af43d@amd.com>
Date: Fri, 23 Feb 2024 11:58:09 +0530
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 00/30] PREEMPT_AUTO: support lazy rescheduling
To: Ankur Arora <ankur.a.arora@oracle.com>,
 Thomas Gleixner <tglx@linutronix.de>
Cc: linux-kernel@vger.kernel.org, peterz@infradead.org,
 torvalds@linux-foundation.org, paulmck@kernel.org,
 akpm@linux-foundation.org, luto@kernel.org, bp@alien8.de,
 dave.hansen@linux.intel.com, hpa@zytor.com, mingo@redhat.com,
 juri.lelli@redhat.com, vincent.guittot@linaro.org, willy@infradead.org,
 mgorman@suse.de, jpoimboe@kernel.org, mark.rutland@arm.com, jgross@suse.com,
 andrew.cooper3@citrix.com, bristot@kernel.org,
 mathieu.desnoyers@efficios.com, geert@linux-m68k.org,
 glaubitz@physik.fu-berlin.de, anton.ivanov@cambridgegreys.com,
 mattst88@gmail.com, krypton@ulrich-teichert.org, rostedt@goodmis.org,
 David.Laight@ACULAB.COM, richard@nod.at, mjguzik@gmail.com,
 jon.grimm@amd.com, bharata@amd.com, boris.ostrovsky@oracle.com,
 konrad.wilk@oracle.com
References: <20240213055554.1802415-1-ankur.a.arora@oracle.com>
 <15fcf98c-58c5-fa08-2e21-f3b0baf058e6@amd.com> <87zfvtg2mh.ffs@tglx>
 <f1a5fe09-f4bc-0dc5-edda-6b3e690bc0e3@amd.com> <87ttm0dwhk.ffs@tglx>
 <87bk87c1n7.fsf@oracle.com>
Content-Language: en-US
From: Raghavendra K T <raghavendra.kt@amd.com>
In-Reply-To: <87bk87c1n7.fsf@oracle.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?NGdlVTlsbG9ZdW1UYWhsRENTYUZkcnRYaXVDTHNuSFBTbnJadWZwclhaWEZ3?=
 =?utf-8?B?ZnJERFVlUHA1NHd1eDVNdWd2aGdnMi9aMW5ZVUI5WHZYZWhGcVJ0YVdTMjMw?=
 =?utf-8?B?dnhoS2ZvTHdVUE9Vd21FODJEd096dERFTUV4VUo1NkgwUGRBelBTM2k5UVNU?=
 =?utf-8?B?UzNONlJMdnBQNlN2ZWlUSXJhcUxvcUpWTEdOT2JkM2I3Sk1TK1VndDZhQTU5?=
 =?utf-8?B?OHBqWXhNeWx4eGN5Ukg4VG1NUnRmS1AxOVdIblZCZDdDeTcvdHVKWndhbUxw?=
 =?utf-8?B?cjFhM3UySWpoRURFN3M1eWxIR3l1d1J1Qk1VT1A1WjZ4NWZIbGVNRVlsVUIz?=
 =?utf-8?B?djIvNlIvaWlQeU51QkFpTlFrSEFnd3laMHNxUXlwRVMyeE0xTHZLTFNQTktV?=
 =?utf-8?B?L0N4VFcyWmdjQVhpcHZoZFppL0p0cW9jK3FFREFGYWJkb0tnRVpLck5jMktM?=
 =?utf-8?B?Skwra2tMVGJ6RjZDY3dtZEJuNXUyZGl6bDFZTnhqT2RFeUVDVFJVK1Q3cThC?=
 =?utf-8?B?QnVCZi9rMCtEZFMwZEw4ZDU2eGZyWGI5WkJBc0FzTXdESE5weTc1VDNsRUc1?=
 =?utf-8?B?SzFsaTg2VlJzTURTQm5uZDZyS0w0dWg0d1crd2FISGEvZndSa0NRWjR3eStO?=
 =?utf-8?B?Lzd5ZzRrUWFJeVduVUkza0FlWGw0cVgwdFFLSkNzUG9OajVHRXF0Y3JHVklX?=
 =?utf-8?B?N2NocXhQNVBZQU9KcjQyTW5DcWN4TENvb0dxSjFqL2Y0YUhpVFIrVmNNaDd0?=
 =?utf-8?B?SXJsZ1VLZjAyTDBSWmI5Sys5YnZzc1IvTmZzZCtXb2RhbkV1aTdUQmVlc3By?=
 =?utf-8?B?UElpdGNMM011U3hzek5FZzhQNlZYdlZ1UHJxQlE4YnVIWlNIV1RGODh0Nm96?=
 =?utf-8?B?aW5xcWdkcnVHMVY5bnI1ci93c2VFNzJuaklBUllDRlBPRW0yem9VVTJmTEhn?=
 =?utf-8?B?UWxlOEVTenBBVEd4Q0NsOCtFTlMvb2JPYndnZjZWS3RFbE9hMVZOeUtwT1dz?=
 =?utf-8?B?K1IrOUhKSXVLU2dBV3pSa3JURnZWWEluZzhFcEtFeXYyWFRLVk9QODIwVWZm?=
 =?utf-8?B?VnVyWHhTa0UrajB6YjVCWHFKSWtENWszZzJrdGlsK21EcDVyRFRzN0tmNFds?=
 =?utf-8?B?RkFjVkdHd0xCRThxK3pOMExwZjJJQnd3MDVRUVdtYXhLcEtzYUxVS3ZHNGdp?=
 =?utf-8?B?K1BOYXhpZ2FZTzB6a21sd0JoblRZSmQvaWNRVHBiTmZCWTJ2VitSTVgzR25N?=
 =?utf-8?B?aXNLanM4S0VEM0RLUndZZHUvdENLS3JFV0V3bVlHZ1BUSkl2TzFwR2Y4WSt1?=
 =?utf-8?B?bzZXeWJQRzVqNHp3WDZ6N0NJWG82bzJldkJaS3R2L2JFNUZjWnJMN0NnaktQ?=
 =?utf-8?B?aVpMMENuZmU2dm5TbTFjZUdNVVBGRmIzbm00akxzS2ZZU2FpOTdUb1c3UzVh?=
 =?utf-8?B?cFNSR2dROHo4S21MQW1iVXlIR0R4TFdsZUNDRzMvNFNabHZMajVhbysrQ1dT?=
 =?utf-8?B?L3V3bmZYeDd5bUtzeTdBNDVwdTMxSUJNR3hOaW9xOUlNcHRiWGdXVXVPaVFs?=
 =?utf-8?B?UU5JTm5NME1CWVd1N05jWDl3MHMvalVnaXZ0VzdHUlFDcEV5aE80RjFQWU9F?=
 =?utf-8?B?K3UzZm9kM1lnbmYwdEhRUUpLSmxFalB2NzB2WWdtLzN1VExoYk1rdSs1TnRD?=
 =?utf-8?B?Mk1WSjRFbEJ5RTkraXBtbG9YNmRmV3lpR0g3aVo4MDIvenp4UjMwcWRMN2tK?=
 =?utf-8?B?Y3ZCR2FIb3R6cklDd3Z2VXQ5UjZWVkhHNHhwM1RuemMwYm1qY1ozcGRNcU5y?=
 =?utf-8?B?d2paRzNPV2pVdml4RVAxWXNyRE5RWit3Vk5FVjVQeWVZTUU2ejVlYzRtbDhq?=
 =?utf-8?B?bkMrS0FvZnZSUVg0YWIvemYzeXV1S0ppU0xMNlNZYlBWZExsYm5VcC9JK1F0?=
 =?utf-8?B?T2dHUWxOVVRwbkVLZHVtRVAwMWRNcUZwQVZPSG0ySEdOeDhRZXY2MVNnRThY?=
 =?utf-8?B?S1JGcE9kYmZucnN4V3BMMWFXTnFIeUh6S05iK3dVSjMxMHBtYkZCMk5meTll?=
 =?utf-8?B?bnRHSXJKdm5iWXBTNHNKblJDbWRuK2I4QVFvSkQwYk85U1RnWllNMENKRHRl?=
 =?utf-8?Q?VJEXd0udKb5C5aGyp6wQKBz/h?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 59c50f25-ba4e-4456-740f-08dc3438a3a4
X-MS-Exchange-CrossTenant-AuthSource: PH7PR12MB5805.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Feb 2024 06:28:25.1456
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: REY5qAgUTCKw65C9r1iVNJTdpR8NPCiOm54LwycQsPq8I5h0SxW86KekjKChL00cb0HSLDeyjzyLlS//GfcjnQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR12MB5469

On 2/23/2024 8:44 AM, Ankur Arora wrote:
> 
> Thomas Gleixner <tglx@linutronix.de> writes:
> 
>> On Wed, Feb 21 2024 at 22:57, Raghavendra K T wrote:
>>> On 2/21/2024 10:45 PM, Thomas Gleixner wrote:
>>>> On Wed, Feb 21 2024 at 17:53, Raghavendra K T wrote:
>>>>> Configuration tested.
>>>>> a) Base kernel (6.7),
>>>>
>>>> Which scheduling model is the baseline using?
>>>>
>>>
>>> baseline is also PREEMPT_DYNAMIC with voluntary preemption
>>>
>>>>> b) patched with PREEMPT_AUTO voluntary preemption.
>>>>> c) patched with PREEMPT_DYNAMIC voluntary preemption.
>>
>> Which RCU variant do you have enabled with a, b, c ?
>>
>> I.e. PREEMPT_RCU=?
> 
> Raghu please confirm this, but if the defaults were chosen
> then we should have:
> 
>>> baseline is also PREEMPT_DYNAMIC with voluntary preemption
> PREEMPT_RCU=y
> 
>>>>> b) patched with PREEMPT_AUTO voluntary preemption.
> 
> If this was built with PREEMPT_VOLUNTARY then, PREEMPT_RCU=n.
> If with CONFIG_PREEMPT, PREEMPT_RCU=y.
> 
> Might be worth rerunning the tests with the other combination
> as well (still with voluntary preemption).
> 
>>>>> c) patched with PREEMPT_DYNAMIC voluntary preemption.
> PREEMPT_RCU=y

Hello Thomas, Ankur,
Yes, Ankur's understanding is right, defaults were chosen all the time so
for
a) base 6.7.0+ + PREEMPT_DYNAMIC with voluntary preemption PREEMPT_RCU=y
b) patched + PREEMPT_AUTO voluntary preemption. PREEMPT_RCU = n
c) patched + PREEMPT_DYNAMIC with voluntary preemption PREEMPT_RCU=y

I will check with other combination (CONFIG_PREEMPT/PREEMPT_RCU) for (b)
and comeback if I see anything interesting.

Thanks and Regards
- Raghu