Rustix for android, part deux#1577
Open
gurchetansingh wants to merge 4 commits intobytecodealliance:mainfrom
Open
Rustix for android, part deux#1577gurchetansingh wants to merge 4 commits intobytecodealliance:mainfrom
gurchetansingh wants to merge 4 commits intobytecodealliance:mainfrom
Conversation
Contributor
gurchetansingh
commented
Feb 2, 2026
gurchetansingh
commented
gurchetansingh
force-pushed
the
rustix-for-android-part-deux
branch
from
e97ec4e to
f5b00ae
Compare
gurchetansingh
changed the title
Contributor
Author
Contributor
Author
|
Also, getting a new release for this would be most terrific for downstream integrators. |
notgull
reviewed
Feb 5, 2026
|
|
||
| /// Convenience macro for Rustix's ioctl read | ||
| #[macro_export] | ||
| macro_rules! ioctl_readwrite { |
Contributor
There was a problem hiding this comment.
These seem like weird inclusions for the public API.
Contributor
Author
There was a problem hiding this comment.
so nix has them too:
https://github.com/nix-rust/nix/blob/master/src/sys/ioctl/mod.rs#L652
I know of two places that are duplicating this logic:
https://github.com/Smithay/drm-rs/blob/develop/drm-ffi/src/ioctl.rs#L6
https://github.com/magma-gpu/rutabaga_gfx/blob/main/third_party/mesa3d/src/magma/sys/linux/amdgpu.rs#L43 (the project I work on)
Anything particularly wrong with these APIs? Otherwise, people have to excessively define things like ioctl::Updater and ioctl::opcode::read_write.
gurchetansingh
pushed a commit
to magma-gpu/rutabaga_gfx
that referenced
this pull request
Feb 18, 2026
We would like to land: #36 But that'll break Android until the following rustix change lands: bytecodealliance/rustix#1577 and it can take a few months to get something through rustix due to limited maintainer bandwidth. Stopgap solution: cut releases and branches for libkrun until Rustix change is merged. Unlike the "chromeos" branch, the libkrun branch will eventually be merged with main.
gurchetansingh
pushed a commit
to magma-gpu/rutabaga_gfx
that referenced
this pull request
Feb 19, 2026
We would like to land: #36 But that'll break Android until the following rustix change lands: bytecodealliance/rustix#1577 and it can take a few months to get something through rustix due to limited maintainer bandwidth. Stopgap solution: cut releases and branches for libkrun until Rustix change is merged. Unlike the "chromeos" branch, the libkrun branch will eventually be merged with main.
Contributor
Author
|
gentle ping. Perhaps added more reviewers/maintainers could help with the backlog? |
sunfishcode
reviewed
Feb 21, 2026
|
|
||
| /// Convenience macro for Rustix's ioctl read | ||
| #[macro_export] | ||
| macro_rules! ioctl_readwrite { |
These ioctl wrappers are used by rustix users. For example: https://github.com/Smithay/drm-rs/blob/develop/drm-ffi/src/ioctl.rs#L6 Roughly, they match the ones provided by nix. I'm upstreaming the portion I have personally tested.
Android likes to build host tools (adb, Cuttlefish, ..) via musl, using the same hermetic tree as for Bionic build. And linux-raw-sys isn't in that hermetic tree, so add support for the libc-backend.
It's desirable to enable futex for some cases that will be run on Android. The strategy is just to define the constants needed by the libc ourselves, if libc doesn't provide the necessary constant. Long-term, that could make the libc backend only depend on libc.
Rustix has gone back-and-forth on it's policy on Android. Android platform developers don't want a dependency on linux-raw-sys. This is documented here: bytecodealliance#1095 bytecodealliance#1478 rust-vsock/vsock-rs#60 Android app developers seem to want to use linux-raw-sys though: bytecodealliance#1528 The Android platform developers don't REALLY care about the cargo build though. CI/CD testing on the particular build options Android likes is useful. This MR adds another non-cargo build system (Meson) to the CI/CD, that mirrors the options Android likes. Meson is renowned for being a easy-to-maintain build system and arguably the favorite among Linux-distro maintainers. Right now, meson currently does not use Cargo.toml as a source of truth for dependency, since a dependency's build.rs file can basically do anything, and there's no way for an external build system to determine a build.rs file is doing [1]. As such, the standard way to include dependencies is are meson subprojects, for each dependency. There are various proposals to improve the situation [2], but those would require a bleeding edge meson not in CI/CD bots yet. That said, there's an easy way to test from the root of the directory: sudo apt install meson ninja-build meson setup rustix-build ninja -C rustix-build Meson closely mirrors what Android does. By adding it here, we ensure the continue use of Rustix in core Android platform code. [1] mesonbuild/meson#2173 [2] mesonbuild/meson#15223
gurchetansingh
force-pushed
the
rustix-for-android-part-deux
branch
from
f5b00ae to
3ec91c2
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.