-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Ktls #4482
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
base: main
Are you sure you want to change the base?
Ktls #4482
Conversation
b60cffa
to
7e42c78
Compare
For nightly freebsd-15 ci task to success, you would need to update the ci freebsd env with the new headers. I have no idea why freebsd-13 is failing. |
@asomers would you mind reviewing this since it's FreeBSD?
If there is a version bump available, feel freeto submit a PR updating it. I'm not sure about freebsd-13 either, ctest seems extremely flaky on that target. |
Is there an existing example of the PR with such update? |
You just want to update a minor freebsd version? I don't know one off the top of my head but it shouldn't be too hard, the FreeBSD CI config is in https://github.com/rust-lang/libc/blob/9c8cd59b2367ce9cd5acd6247c3764cdf670a9f8/.cirrus.yml |
Our CI config uses the latest FreeBSD 15 snapshot on GCE. So we don't need to do anything. If we just wait, GCE will have the latest release soon enough. |
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.
Cool project, kib! I look forward to trying it.
pub _xig_spare32: u32, | ||
pub xig_gen: inp_gen_t, | ||
pub xig_sogen: so_gen_t, | ||
pub _xig_spare64: [u64; 4], |
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.
The convention used elsewhere in libc is to make spare and padding fields private. The forces consumers to initialize with mem::zeroed
, but it also makes it easier for future versions of libc to replace those spare fields with no backwards-compatibility problems.
pub _xig_spare32: u32, | |
pub xig_gen: inp_gen_t, | |
pub xig_sogen: so_gen_t, | |
pub _xig_spare64: [u64; 4], | |
_xig_spare32: u32, | |
pub xig_gen: inp_gen_t, | |
pub xig_sogen: so_gen_t, | |
_xig_spare64: [u64; 4], |
} | ||
|
||
pub struct in_addr_4in6 { | ||
pub ia46_pad32: [u32; 3], |
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.
pub ia46_pad32: [u32; 3], | |
ia46_pad32: [u32; 3], |
pub inp_gencnt: u64, | ||
pub so_pcb: kvaddr_t, | ||
pub coninf: crate::in_conninfo, | ||
pub rx_vlan_id: c_short, |
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.
pub rx_vlan_id: c_short, | |
pub rx_vlan_id: c_ushort, |
Description
The PR adds definitions for kernel-offloaded TLS TCP session information sysctls. It is used by the utility to list kTLS connections https://github.com/kostikbel/ktcplist
Sources
https://github.com/freebsd/freebsd-src/blob/4078e0d1d6f8189bff44fcad04df256220035f76/sys/sys/ktls.h
freebsd/freebsd-src@c9e9a0f
@rustbot label +stable-nominated