Skip to content
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

Enable statx on musl-libc #3976

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Commits on Oct 16, 2024

  1. linux_like: Unify statx definitions

    The statx system call and corresponding constants are defined by the
    Linux kernel and don't depend on the libc or architecture. The only
    difference is whether a libc exports the statx syscall wrapper or not.
    
    We can thus unify the statx definitions for all Linux "like" platforms:
    GNU (glibc), Android (bionic), and (in a later commit) musl.
    
    The statx struct is in a separate file because the style check doesn't
    allow multiple s! macros in one file, and #[cfg()] doesn't work in s!.
    
    Plain u64 (or uint64_t in C) can't be used for the statx fields because
    bionic defines them as __u64, and provides incompatible definitions of
    uint64_t and __u64.
    neuschaefer committed Oct 16, 2024
    Configuration menu
    Copy the full SHA
    c3bc406 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    23ad220 View commit details
    Browse the repository at this point in the history
  3. Enable statx on musl-libc

    Version 1.2.5 of musl-libc added support for the statx system call[1].
    
    [1]: https://musl.libc.org/releases.html
    neuschaefer committed Oct 16, 2024
    Configuration menu
    Copy the full SHA
    48e8413 View commit details
    Browse the repository at this point in the history