linux-kernel - Re: [PATCH 1/2] sched_ext: change the variable name for slice refill event

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <a247e016-494d-42a6-8f6a-2ff08366404b@126.com>
Date: Fri, 18 Apr 2025 10:05:08 +0800
From: Honglei Wang <jameshongleiwang@....com>
To: Andrea Righi <arighi@...dia.com>
Cc: tj@...nel.org, void@...ifault.com, changwoo@...lia.com, mingo@...hat.com,
 peterz@...radead.org, juri.lelli@...hat.com, vincent.guittot@...aro.org,
 dietmar.eggemann@....com, rostedt@...dmis.org, bsegall@...gle.com,
 mgorman@...e.de, vschneid@...hat.com, joshdon@...gle.com, brho@...gle.com,
 linux-kernel@...r.kernel.org, bpf@...r.kernel.org
Subject: Re: [PATCH 1/2] sched_ext: change the variable name for slice refill
 event

Hi Andrea,

On 2025/4/18 02:30, Andrea Righi wrote:
> Hi Honglei,
> 
> On Thu, Apr 17, 2025 at 04:07:07PM +0800, Honglei Wang wrote:
>> SCX_EV_ENQ_SLICE_DFL gives the impression that the event only occurs
>> when the tasks were enqueued, which seems not accurate. What it actually
>> means is the refilling with defalt slice, and this can occur either when
>> enqueue or pick_task. Let's change the variable to
>> SCX_EV_REFILL_SLICE_DFL.
>>
>> Signed-off-by: Honglei Wang <jameshongleiwang@....com>
>> ---
>>  kernel/sched/ext.c             | 22 +++++++++++-----------
>>  tools/sched_ext/scx_qmap.bpf.c |  4 ++--
>>  2 files changed, 13 insertions(+), 13 deletions(-)
>>
>> diff --git a/kernel/sched/ext.c b/kernel/sched/ext.c
>> index 66bcd40a28ca..594087ac4c9e 100644
>> --- a/kernel/sched/ext.c
>> +++ b/kernel/sched/ext.c
>> @@ -1517,10 +1517,10 @@ struct scx_event_stats {
>>  	s64		SCX_EV_ENQ_SKIP_MIGRATION_DISABLED;
>>  
>>  	/*
>> -	 * The total number of tasks enqueued (or pick_task-ed) with a
>> -	 * default time slice (SCX_SLICE_DFL).
>> +	 * The total number of tasks slice refill with default time slice
>> +	 * (SCX_SLICE_DFL).
> 
> Nit, how about:
> 
> Total number of times a task's time slice was refilled with the default
> value (SCX_SLICE_DFL).
> 

OK, will send a update later.

Thanks,
Honglei

> Thanks,
> -Andrea