Skip to content

Update statx to Linux 6.14 - #1467

Open
ginnyTheCat wants to merge 2 commits into
bytecodealliance:mainfrom
ginnyTheCat:statx
Open

ginnyTheCat wants to merge 2 commits into
bytecodealliance:mainfrom
ginnyTheCat:statx

Conversation

@ginnyTheCat

@ginnyTheCat ginnyTheCat commented May 23, 2025

Copy link
Copy Markdown

Fix #1385. Waiting on sunfishcode/linux-raw-sys#155 + release and libc 0.2.190 for all constants.

@rusty-snake rusty-snake left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread src/fs/statx.rs
@ginnyTheCat ginnyTheCat changed the title Update statx to Linux 6.11 Update statx to Linux 6.14 Jun 26, 2025
@sunfishcode

Copy link
Copy Markdown
Member

I took the liberty of rebasing on trunk, which already had the stx_dio_read_offset_align field, but not yet the flags. Thanks for the PR!

@sunfishcode

Copy link
Copy Markdown
Member

Ah, this still depends on rust-lang/libc#4401.

@valentynkit

Copy link
Copy Markdown

Ah, this still depends on rust-lang/libc#4401.

Just FYI.
Missing statx maks and fields were added into rust libc: rust-lang/libc#5412 (masks), rust-lang/libc#4621 (fields)

@ginnyTheCat

Copy link
Copy Markdown
Author

Thanks for letting us know. Sadly it's not in a 0.2 release yet.

@xtqqczze

Copy link
Copy Markdown
Contributor

@ginnyTheCat merge conflicts need to be resolved

@ginnyTheCat

Copy link
Copy Markdown
Author

I'm aware of that, but until there is a libc 0.2.190 release the CI is just gonna fail again afaik.

@xtqqczze

xtqqczze commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

You could convert to draft and continue development with the unreleased libc 0.2.190 by adding this to Cargo.toml:

[patch.crates-io]
libc = { git = "https://github.com/rust-lang/libc", branch = "libc-0.2" }

@ginnyTheCat

Copy link
Copy Markdown
Author

Ah, because of #1676. Now i understand. I will try that, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Newer statx features

5 participants