-
Notifications
You must be signed in to change notification settings - Fork 1k
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
freebsd proposal to move sockcred2 usage in the root mod while ignori… #2589
Conversation
r? @Amanieu (rust-highfive has picked a reviewer for you, use r? to override) |
a72a80d
to
9de7043
Compare
…ng it for FreeBSD 12.
9de7043
to
da089ce
Compare
@@ -3701,6 +3712,15 @@ f! { | |||
let (idx, offset) = (cpu / bitset_bits, cpu % bitset_bits); | |||
0 != cpuset.__bits[idx] & (1 << offset) | |||
} | |||
|
|||
pub fn SOCKCRED2SIZE(ngrps: usize) -> usize { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Won't it be an issue on freebsd 11 to link to a non-existing function?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah no, nevermind me...
Looks good to me since the struct has the same layout between 13 and 14 and the function is implemented, not linked. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. This is the best way to do with new symbols in the OS, rather than changed symbols.
@bors r+ |
📌 Commit da089ce has been approved by |
☀️ Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13 |
…ng it for FreeBSD 12.