linux-kernel - Re: [PATCH 2/3] sched_ext: Rename scx_bpf_consume() to scx_bpf_dsq_move_to

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 for Android: free password hash cracker in your pocket

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

Message-ID: <81281555-d57f-4507-a3a5-ea9d1aa27871@gmail.com>
Date: Sun, 10 Nov 2024 19:03:03 +0900
From: Changwoo Min <multics69@...il.com>
To: tj@...nel.org, void@...ifault.com
Cc: linux-kernel@...r.kernel.org, kernel-team@...a.com, sched-ext@...a.com,
 arighi@...dia.com, me@...tlynerdless.de, ggherdovich@...e.com,
 dschatzberg@...a.com, yougmark94@...il.com, changwoo@...lia.com,
 kernel-dev@...lia.com
Subject: Re: [PATCH 2/3] sched_ext: Rename scx_bpf_consume() to
 scx_bpf_dsq_move_to_local()

Hello Tejun,

Overall looks good to me. I have one minior comment:

On 24. 11. 10. 04:48, Tejun Heo wrote:
> @@ -6372,9 +6372,8 @@ __bpf_kfunc void scx_bpf_dispatch(struct task_struct *p, u64 dsq_id, u64 slice,
>    * @enq_flags: SCX_ENQ_*
>    *
>    * Insert @p into the vtime priority queue of the DSQ identified by @dsq_id.
> - * Tasks queued into the priority queue are ordered by @vtime and always
> - * consumed after the tasks in the FIFO queue. All other aspects are identical
> - * to scx_bpf_dsq_insert().
> + * Tasks queued into the priority queue are ordered by @vtime. All other aspects
> + * are identical to scx_bpf_dsq_insert().

I suggest keeping this part, "and always consumed after the tasks
in the FIFO queue." Otherwise, IIRC, there is no place to explain
the priority between FIFO and priority DSQs explicitly.

Regards,
Changwoo Min