Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

scx: Build fix after kernel update #75

Merged
merged 1 commit into from
Jan 9, 2024
Merged

scx: Build fix after kernel update #75

merged 1 commit into from
Jan 9, 2024

Conversation

htejun
Copy link
Contributor

@htejun htejun commented Jan 9, 2024

In the latest kernel, sched_ext API has changed in two areas:

  • ops.prep_enable/cancel_enable/enable/disable() replaced with ops.init_task/enable/disable/exit_task().

  • scx_bpf_dispatch() can now be called from ops.select_cpu(). Also, SCX_ENQ_LOCAL flag is removed. Instead, users can call scx_bpf_select_cpu_dfl() from ops.select_cpu() and use the @is_idle out param value to determine whether to dispatch directly.

This commit updates all schedules so that they build.

  • Init functions renamed / merged / split.

  • ops.select_cpu() is added to several schedulers and local direct disptching logic is moved there.

This is the minimum update which is need to make the schedulers build and work. It needs further update to e.g. move vtime udpates to ops.enable().

In the latest kernel, sched_ext API has changed in two areas:

- ops.prep_enable/cancel_enable/enable/disable() replaced with
  ops.init_task/enable/disable/exit_task().

- scx_bpf_dispatch() can now be called from ops.select_cpu(). Also,
  SCX_ENQ_LOCAL flag is removed. Instead, users can call
  scx_bpf_select_cpu_dfl() from ops.select_cpu() and use the @is_idle out
  param value to determine whether to dispatch directly.

This commit updates all schedules so that they build.

- Init functions renamed / merged / split.

- ops.select_cpu() is added to several schedulers and local direct
  disptching logic is moved there.

This is the minimum update which is need to make the schedulers build and
work. It needs further update to e.g. move vtime udpates to ops.enable().
@htejun htejun requested review from Byte-Lab and arighi January 9, 2024 00:55
@Byte-Lab Byte-Lab merged commit 4ff504a into main Jan 9, 2024
0 of 2 checks passed
@htejun htejun deleted the htejun branch January 9, 2024 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants