-
Notifications
You must be signed in to change notification settings - Fork 1.1k
linux: Add new netlink flags #4288
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
Conversation
Some changes occurred in the Android module cc @maurer |
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.
Are these available on Sparc? CI is throwing some errors, but this may just be because our headers are somewhat old.
If that is the case, libc-test/build.rs
will need to be updated to exclude them.
@tgross35 Thanks for getting back to me on this. I'm willing to investigate and see which constants are available on what platform in recent versions of the headers but I'm not quite sure what the best way to look into this is. What would you recommend? |
Actually just try rebasing, the sparc headers in CI were updated recently so maybe things fixed themselves. |
6498f48
to
70527d1
Compare
Okay, I've rebased, I'll keep an eye on CI. |
Okay it appears that CI is passing! |
A permalink to the header is preferable over docs links anyway, since things change. I updated the top post 👍 Thanks! @rustbot label +stable-nominated |
(backport <rust-lang#4288>) (cherry picked from commit 70527d1)
Description
This change adds some new constants for the DELETE and ACK subset of netlink commands.
Sources
The documentation appears to be out of date, but the header is
/usr/include/linux/netlink.h
and includes the following:Header: https://github.com/torvalds/linux/blob/0fed89a961ea851945d23cc35beb59d6e56c0964/include/uapi/linux/netlink.h#L82-L87
Checklist
libc-test/semver
have been updated*LAST
or*MAX
areincluded (see #3131)
cd libc-test && cargo test --target mytarget
);especially relevant for platforms that may not be checked in CI