macos: update vm_statistics64 to latest sdk#5253
Conversation
This comment has been minimized.
This comment has been minimized.
|
|
This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
There was a problem hiding this comment.
Could you remove the skip at
Lines 353 to 354 in 5848d31
Please also add a link to https://github.com/apple-oss-distributions/xnu/blob/f6217f891ac0bb64f3d375211650a4c1ff8ca1ea/osfmk/mach/vm_statistics.h#L142-L177 in your commit message, just saying that xnu hasn't yet been updated.
|
Reminder, once the PR becomes ready for a review, use |
XNU headers have not yet been updated to match the updated revisions https://github.com/apple-oss-distributions/xnu/blob/f6217f891ac0bb64f3d375211650a4c1ff8ca1ea/osfmk/mach/vm_statistics.h#L142-L177
|
In that case, could you update // existing
let x86_64 = target.contains("x86_64");
let i686 = target.contains("i686");
// new
let macos = VERSIONS.macos;
// ...
match ... {
"vm_statistics64" => macos.unwrap() < (27, 0),
}and make sure it tests locally? |
|
@tgross35 made these changes and it tests locally. Additionally, I spotted that Additional permalink that will be updated: https://github.com/apple-oss-distributions/xnu/blob/f6217f891ac0bb64f3d375211650a4c1ff8ca1ea/bsd/sys/aio.h#L165 |
|
For future reference, we usually shouldn't take PRs updating to API for not-yet-released platform versions, they tend to change. This is fine since you've already gone through the work here, but if you were planning on more you should hold off until xnu is updated (ideally also GitHub actions so we don't have to rely on local tests). |
|
@tgross35 my main motivation was trying to fix a program I use on the beta, it lead me down this rabbit hole to finding out the declaration had been updated. Thanks for taking a look at it. |
XNU headers have not yet been updated to match the updated revisions https://github.com/apple-oss-distributions/xnu/blob/f6217f891ac0bb64f3d375211650a4c1ff8ca1ea/osfmk/mach/vm_statistics.h#L142-L177 (backport <rust-lang#5253>) (cherry picked from commit 36a4c58)
Will be updated at https://github.com/apple-oss-distributions/xnu/blob/f6217f891ac0bb64f3d375211650a4c1ff8ca1ea/bsd/sys/aio.h#L165 (backport <rust-lang#5253>) (cherry picked from commit 328f309)
XNU headers have not yet been updated to match the updated revisions https://github.com/apple-oss-distributions/xnu/blob/f6217f891ac0bb64f3d375211650a4c1ff8ca1ea/osfmk/mach/vm_statistics.h#L142-L177 (backport <rust-lang#5253>) (cherry picked from commit 36a4c58)
Will be updated at https://github.com/apple-oss-distributions/xnu/blob/f6217f891ac0bb64f3d375211650a4c1ff8ca1ea/bsd/sys/aio.h#L165 (backport <rust-lang#5253>) (cherry picked from commit 328f309)
XNU headers have not yet been updated to match the updated revisions https://github.com/apple-oss-distributions/xnu/blob/f6217f891ac0bb64f3d375211650a4c1ff8ca1ea/osfmk/mach/vm_statistics.h#L142-L177 (backport <rust-lang#5253>) (cherry picked from commit 36a4c58)
Will be updated at https://github.com/apple-oss-distributions/xnu/blob/f6217f891ac0bb64f3d375211650a4c1ff8ca1ea/bsd/sys/aio.h#L165 (backport <rust-lang#5253>) (cherry picked from commit 328f309)
Description
Updates
vm_statistics64to include the new values added in more recent MacOS SDK versions.Sources
Relevant header snippet from MacOS27.sdk
Checklist
libc-test/semverhave been updated*LASTor*MAXareincluded (see #3131)
cd libc-test && cargo test --target mytarget);especially relevant for platforms that may not be checked in CI
@rustbot label +stable-nominated