Page MenuHomeFreeBSD
Differential D38555

cpuset: Add compat shim to the sched_affinity functions
ClosedPublic
Actions

Authored by dchagin on Feb 13 2023, 11:40 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 11, 3:09 PM
Unknown Object (File)
Sun, Dec 7, 3:58 PM
Unknown Object (File)
Nov 19 2025, 5:25 AM
Unknown Object (File)
Nov 16 2025, 11:11 AM
Unknown Object (File)
Nov 14 2025, 9:59 PM
Unknown Object (File)
Oct 31 2025, 9:29 PM
Unknown Object (File)
Oct 31 2025, 8:00 PM
Unknown Object (File)
Oct 31 2025, 5:43 PM
View All Files
Subscribers
imp

Details

Reviewers
kib
Commits
rGf3f350c5c8e4: cpuset: Add compat shim to the sched_affinity functions
rGcbc32e4c5e84: cpuset: Add compat shim to the sched_affinity functions
Summary

To allow to run a new world on the pre 1400079 kernel a compat shims to
the sched_affinity functions has beed added. PID_MAX is hardcoded as our
kernel does not expose it to the user space.

Reported by: antoine
Tested by: antoine

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dchagin created this revision.Feb 13 2023, 11:40 AM
Herald added a subscriber: imp. · View Herald TranscriptFeb 13 2023, 11:40 AM
dchagin requested review of this revision.Feb 13 2023, 11:40 AM
Harbormaster completed remote builds in B49729: Diff 117083.Feb 13 2023, 11:40 AM
dchagin edited the summary of this revision. (Show Details)Feb 13 2023, 3:59 PM
dchagin added a reviewer: kib.
kib added inline comments.Feb 14 2023, 12:41 AM
lib/libc/gen/sched_getaffinity.c
37

Look at sys/param.h P_OSREL_XXX macros.

dchagin updated this revision to Diff 117188.Feb 14 2023, 7:49 AM
dchagin edited the summary of this revision. (Show Details)

better (beauty) naming, thanks)

Harbormaster completed remote builds in B49768: Diff 117188.Feb 14 2023, 7:49 AM
kib added a comment.Feb 14 2023, 8:04 AM
In D38555#877823, @dchagin wrote:

better (beauty) naming, thanks)

Um, I mean that you should add your P_OSREL value to sys/param.h

dchagin added a comment.Feb 14 2023, 8:31 AM
In D38555#877827, @kib wrote:
In D38555#877823, @dchagin wrote:

better (beauty) naming, thanks)

Um, I mean that you should add your P_OSREL value to sys/param.h

sure? P_OSREL is under _KERNEL, thats why I do not do it

dchagin updated this revision to Diff 117192.Feb 14 2023, 11:07 AM

_WANT_OSREL use

Harbormaster completed remote builds in B49770: Diff 117192.Feb 14 2023, 11:07 AM
kib added inline comments.Feb 15 2023, 1:04 AM
lib/libc/gen/sched_getaffinity.c
37

I would move this definition into libc_private.h, and add a comment explaining why we hardcode the value there.

dchagin updated this revision to Diff 117281.Feb 15 2023, 9:12 AM

done, may be put _PID_MAX under _WANT_P_OSREL condition?

Harbormaster completed remote builds in B49796: Diff 117281.Feb 15 2023, 9:12 AM
kib accepted this revision.Feb 15 2023, 9:17 AM
In D38555#878467, @dchagin wrote:

done, may be put _PID_MAX under _WANT_P_OSREL condition?

No, they are unrelated.

lib/libc/include/libc_private.h
48

Add a blank line before this comment.

This revision is now accepted and ready to land.Feb 15 2023, 9:17 AM
Closed by commit rGcbc32e4c5e84: cpuset: Add compat shim to the sched_affinity functions (authored by dchagin). · Explain WhyFeb 15 2023, 9:24 AM
This revision was automatically updated to reflect the committed changes.
dchagin added a commit: rGcbc32e4c5e84: cpuset: Add compat shim to the sched_affinity functions.
dchagin added a commit: rGf3f350c5c8e4: cpuset: Add compat shim to the sched_affinity functions.Feb 19 2023, 7:26 AM

Revision Contents
Changeset List

PathSize
lib/
libc/
gen/
15 lines
14 lines
include/
6 lines
sys/
sys/
1 line

Diff 117282

View Options

lib/libc/gen/sched_getaffinity.c

Loading...
View Options

lib/libc/gen/sched_setaffinity.c

Loading...
View Options

lib/libc/include/libc_private.h

Loading...
View Options

sys/sys/param.h

Loading...