mirror of
https://codeberg.org/ziglang/zig.git
synced 2026-03-08 07:04:50 +01:00
6216922a9d
Maintaining the POSIX `stat` bits for Zig is a pain. The order and bit-length of members differ between all architectures, and int types can be signed or unsigned. The libcs deal with this by introducing the own version of `struct stat` and copying the kernel structure members to it. In the case of glibc, they did it twice thanks to the largefile transition! In practice, the project needs to maintain three versions of `struct stat`: - What the kernel defines. - What musl wants for `struct stat`. - What glibc wants for `struct stat64`. Make sure to use `fstatat64`! This isn't as simple as running `zig translate-c`. In #21440 I had to: - Compile toolchains for each arch+glibc/musl combo. - Create a test `fstat` program with/without `FILE_OFFSET_BITS=64`. - Dump the value for `struct stat`. - Stare at `std.os.linux`/`std.c` and cry. - Add some missing padding. The fact that so many target checks in the `linux` and `posix` tests exist is most likely due to writing to padding bits and failing later. The solution to this madness is `statx(2)`: - It takes a single structure that is the same for all arches AND libcs. - It uses a custom timestamp format, but it is 64-bit ready. - It gives the same info as `fstatat(2)` and more! - Unlike `fstatat(2)`, you can request a subset of the info required based on passing a mask. It's so good that modern Linux arches (e.g. riscv) don't even implement `stat`, with the libcs using a generic `struct stat` and copying from `struct statx`. Therefore, this commit rips out all the `stat` bits from `std.os.linux` and `std.c`. `std.posix.Stat` is now `void`, and calling `std.posix.*stat` is an compile-time error. A wrapper around `statx` has been added to `std.os.linux`, and callers have been upgraded to use it. Tests have also been updated to use `statx` where possible. While I was here, I converted the mask and file attributes to be packed struct bitfields. A nice side effect is checking that you actually recieved the members you asked for via `Statx.mask`, which I have used by adding `assert`s at specific callsites. |
||
|---|---|---|
| .. | ||
| c_compiler | ||
| c_embed_path | ||
| child_process | ||
| cmakedefine | ||
| coff_dwarf | ||
| compile_asm | ||
| compiler_rt_panic | ||
| config_header | ||
| dep_diamond | ||
| dep_duplicate_module | ||
| dep_lazypath | ||
| dep_mutually_recursive | ||
| dep_recursive | ||
| dep_shared_builtin | ||
| dep_triangle | ||
| depend_on_main_mod | ||
| dependency_options | ||
| dependencyFromBuildZig | ||
| dirname | ||
| embed_generated_file | ||
| emit_asm_and_bin | ||
| emit_asm_no_bin | ||
| emit_llvm_no_bin | ||
| empty_env | ||
| empty_global_error_set | ||
| entry_point | ||
| env_vars | ||
| extern | ||
| glibc_compat | ||
| global_linkage | ||
| install_headers | ||
| install_raw_hex | ||
| ios | ||
| issue_339 | ||
| issue_794 | ||
| issue_5825 | ||
| issue_11595 | ||
| issue_12706 | ||
| libcxx | ||
| libfuzzer | ||
| load_dynamic_library | ||
| mix_c_files | ||
| mix_o_files | ||
| pkg_import | ||
| posix | ||
| run_cwd | ||
| run_output_caching | ||
| run_output_paths | ||
| self_exe_symlink | ||
| shared_library | ||
| simple | ||
| static_c_lib | ||
| strip_empty_loop | ||
| strip_struct_init | ||
| test_obj_link_run | ||
| test_runner_module_imports | ||
| test_runner_path | ||
| tsan | ||
| windows_argv | ||
| windows_bat_args | ||
| windows_entry_points | ||
| windows_paths | ||
| windows_resources | ||
| windows_spawn | ||
| zerolength_check | ||
| build.zig | ||
| build.zig.zon | ||