collect all Fuchsia bindings into the fuchsia module

The Fuchsia bindings are currently spread out across multiple modules in `sys/pal/unix` leading to unnecessary duplication. This PR moves all of these definitions into `sys::pal::unix::fuchsia` and additionally:
* deduplicates the definitions
* makes the error names consistent
* marks some extern functions as safe
* removes unused items (there's no need to maintain these bindings if we're not going to use them)
* removes the documentation for the definitions (contributors should always consult the platform documentation, duplicating that here is just an extra maintenance burden)

Author: joboet

library/std/src/sys/pal/unix/fuchsia.rs

@@ -1,48 +1,35 @@
-#![allow(non_camel_case_types, unused)]
+#![expect(non_camel_case_types)]
 
-use libc::{c_int, c_void,size_t};
+use libc::size_t;
 
+usecrate::ffi::{c_char, c_int, c_void};
 usecrate::io;
-usecrate::mem::MaybeUninit;
-usecrate::os::raw::c_char;
 
-pubtypezx_handle_t = u32;
-pubtypezx_vaddr_t = usize;
-pubtypezx_rights_t = u32;
-pubtypezx_status_t = i32;
-
-pubconstZX_HANDLE_INVALID:zx_handle_t = 0;
+//////////
+// Time //
+//////////
 
 pubtypezx_time_t = i64;
-pubconstZX_TIME_INFINITE:zx_time_t = i64::MAX;
-
-pubtypezx_signals_t = u32;
-
-pubconstZX_OBJECT_SIGNAL_3:zx_signals_t = 1 << 3;
 
-pubconstZX_TASK_TERMINATED:zx_signals_t = ZX_OBJECT_SIGNAL_3;
+pubconstZX_TIME_INFINITE:zx_time_t = i64::MAX;
 
-pubconstZX_RIGHT_SAME_RIGHTS:zx_rights_t = 1 << 31;
+unsafeextern"C"{
+pub safe fnzx_clock_get_monotonic() -> zx_time_t;
+}
 
-// The upper four bits gives the minor version.
-pubtypezx_object_info_topic_t = u32;
+/////////////
+// Handles //
+/////////////
 
-pubconstZX_INFO_PROCESS:zx_object_info_topic_t = 3 | (1 << 28);
+pubtypezx_handle_t = u32;
 
-pubtypezx_info_process_flags_t = u32;
+pubconstZX_HANDLE_INVALID:zx_handle_t = 0;
 
-pubfnzx_cvt<T>(t:T) -> io::Result<T>
-where
-T:TryInto<zx_status_t> + Copy,
-{
-ifletOk(status) = TryInto::try_into(t){
-if status < 0{Err(io::Error::from_raw_os_error(status))}else{Ok(t)}
-}else{
-Err(io::Error::last_os_error())
-}
+unsafeextern"C"{
+pubfnzx_handle_close(handle:zx_handle_t) -> zx_status_t;
 }
 
-// Safe wrapper around zx_handle_t
+/// A safe wrapper around `zx_handle_t`.
 pubstructHandle{
 raw:zx_handle_t,
 }
@@ -65,6 +52,66 @@ impl Drop for Handle {
 }
 }
 
+///////////
+// Futex //
+///////////
+
+pubtypezx_futex_t = crate::sync::atomic::Atomic<u32>;
+
+unsafeextern"C"{
+pubfnzx_object_wait_one(
+handle:zx_handle_t,
+signals:zx_signals_t,
+timeout:zx_time_t,
+pending:*mutzx_signals_t,
+) -> zx_status_t;
+
+pubfnzx_futex_wait(
+value_ptr:*constzx_futex_t,
+current_value:zx_futex_t,
+new_futex_owner:zx_handle_t,
+deadline:zx_time_t,
+) -> zx_status_t;
+pubfnzx_futex_wake(value_ptr:*constzx_futex_t,wake_count:u32) -> zx_status_t;
+pubfnzx_futex_wake_single_owner(value_ptr:*constzx_futex_t) -> zx_status_t;
+pub safe fnzx_thread_self() -> zx_handle_t;
+}
+
+////////////////
+// Properties //
+////////////////
+
+pubconstZX_PROP_NAME:u32 = 3;
+
+unsafeextern"C"{
+pubfnzx_object_set_property(
+handle:zx_handle_t,
+property:u32,
+value:*const libc::c_void,
+value_size: libc::size_t,
+) -> zx_status_t;
+}
+
+/////////////
+// Signals //
+/////////////
+
+pubtypezx_signals_t = u32;
+
+pubconstZX_OBJECT_SIGNAL_3:zx_signals_t = 1 << 3;
+pubconstZX_TASK_TERMINATED:zx_signals_t = ZX_OBJECT_SIGNAL_3;
+
+/////////////////
+// Object info //
+/////////////////
+
+// The upper four bits gives the minor version.
+pubtypezx_object_info_topic_t = u32;
+
+pubconstZX_INFO_PROCESS:zx_object_info_topic_t = 3 | (1 << 28);
+
+pubtypezx_info_process_flags_t = u32;
+
 // Returned for topic ZX_INFO_PROCESS
 #[derive(Default)]
 #[repr(C)]
@@ -76,25 +123,6 @@ pub struct zx_info_process_t {
 }
 
 unsafeextern"C"{
-pubfnzx_job_default() -> zx_handle_t;
-
-pubfnzx_task_kill(handle:zx_handle_t) -> zx_status_t;
-
-pubfnzx_handle_close(handle:zx_handle_t) -> zx_status_t;
-
-pubfnzx_handle_duplicate(
-handle:zx_handle_t,
-rights:zx_rights_t,
-out:*constzx_handle_t,
-) -> zx_handle_t;
-
-pubfnzx_object_wait_one(
-handle:zx_handle_t,
-signals:zx_signals_t,
-timeout:zx_time_t,
-pending:*mutzx_signals_t,
-) -> zx_status_t;
-
 pubfnzx_object_get_info(
 handle:zx_handle_t,
 topic:u32,
@@ -105,6 +133,10 @@ unsafe extern "C" {
 ) -> zx_status_t;
 }
 
+///////////////
+// Processes //
+///////////////
+
 #[derive(Default)]
 #[repr(C)]
 pubstructfdio_spawn_action_t{
@@ -130,180 +162,43 @@ unsafe extern "C" {
 
 pubfnfdio_fd_clone(fd:c_int,out_handle:*mutzx_handle_t) -> zx_status_t;
 pubfnfdio_fd_create(handle:zx_handle_t,fd:*mutc_int) -> zx_status_t;
+
+pubfnzx_task_kill(handle:zx_handle_t) -> zx_status_t;
 }
 
 // fdio_spawn_etc flags
 
 pubconstFDIO_SPAWN_CLONE_JOB:u32 = 0x0001;
 pubconstFDIO_SPAWN_CLONE_LDSVC:u32 = 0x0002;
 pubconstFDIO_SPAWN_CLONE_NAMESPACE:u32 = 0x0004;
-pubconstFDIO_SPAWN_CLONE_STDIO:u32 = 0x0008;
 pubconstFDIO_SPAWN_CLONE_ENVIRON:u32 = 0x0010;
 pubconstFDIO_SPAWN_CLONE_UTC_CLOCK:u32 = 0x0020;
-pubconstFDIO_SPAWN_CLONE_ALL:u32 = 0xFFFF;
 
 // fdio_spawn_etc actions
 
-pubconstFDIO_SPAWN_ACTION_CLONE_FD:u32 = 0x0001;
 pubconstFDIO_SPAWN_ACTION_TRANSFER_FD:u32 = 0x0002;
 
-// Errors
-
-#[allow(unused)]
-pubconstERR_INTERNAL:zx_status_t = -1;
-
-// ERR_NOT_SUPPORTED: The operation is not implemented, supported,
-// or enabled.
-#[allow(unused)]
-pubconstERR_NOT_SUPPORTED:zx_status_t = -2;
-
-// ERR_NO_RESOURCES: The system was not able to allocate some resource
-// needed for the operation.
-#[allow(unused)]
-pubconstERR_NO_RESOURCES:zx_status_t = -3;
-
-// ERR_NO_MEMORY: The system was not able to allocate memory needed
-// for the operation.
-#[allow(unused)]
-pubconstERR_NO_MEMORY:zx_status_t = -4;
-
-// ERR_CALL_FAILED: The second phase of zx_channel_call(; did not complete
-// successfully.
-#[allow(unused)]
-pubconstERR_CALL_FAILED:zx_status_t = -5;
-
-// ERR_INTERRUPTED_RETRY: The system call was interrupted, but should be
-// retried. This should not be seen outside of the VDSO.
-#[allow(unused)]
-pubconstERR_INTERRUPTED_RETRY:zx_status_t = -6;
-
-// ======= Parameter errors =======
-// ERR_INVALID_ARGS: an argument is invalid, ex. null pointer
-#[allow(unused)]
-pubconstERR_INVALID_ARGS:zx_status_t = -10;
-
-// ERR_BAD_HANDLE: A specified handle value does not refer to a handle.
-#[allow(unused)]
-pubconstERR_BAD_HANDLE:zx_status_t = -11;
-
-// ERR_WRONG_TYPE: The subject of the operation is the wrong type to
-// perform the operation.
-// Example: Attempting a message_read on a thread handle.
-#[allow(unused)]
-pubconstERR_WRONG_TYPE:zx_status_t = -12;
-
-// ERR_BAD_SYSCALL: The specified syscall number is invalid.
-#[allow(unused)]
-pubconstERR_BAD_SYSCALL:zx_status_t = -13;
-
-// ERR_OUT_OF_RANGE: An argument is outside the valid range for this
-// operation.
-#[allow(unused)]
-pubconstERR_OUT_OF_RANGE:zx_status_t = -14;
-
-// ERR_BUFFER_TOO_SMALL: A caller provided buffer is too small for
-// this operation.
-#[allow(unused)]
-pubconstERR_BUFFER_TOO_SMALL:zx_status_t = -15;
-
-// ======= Precondition or state errors =======
-// ERR_BAD_STATE: operation failed because the current state of the
-// object does not allow it, or a precondition of the operation is
-// not satisfied
-#[allow(unused)]
-pubconstERR_BAD_STATE:zx_status_t = -20;
-
-// ERR_TIMED_OUT: The time limit for the operation elapsed before
-// the operation completed.
-#[allow(unused)]
-pubconstERR_TIMED_OUT:zx_status_t = -21;
-
-// ERR_SHOULD_WAIT: The operation cannot be performed currently but
-// potentially could succeed if the caller waits for a prerequisite
-// to be satisfied, for example waiting for a handle to be readable
-// or writable.
-// Example: Attempting to read from a message pipe that has no
-// messages waiting but has an open remote will return ERR_SHOULD_WAIT.
-// Attempting to read from a message pipe that has no messages waiting
-// and has a closed remote end will return ERR_REMOTE_CLOSED.
-#[allow(unused)]
-pubconstERR_SHOULD_WAIT:zx_status_t = -22;
-
-// ERR_CANCELED: The in-progress operation (e.g., a wait) has been
-// // canceled.
-#[allow(unused)]
-pubconstERR_CANCELED:zx_status_t = -23;
-
-// ERR_PEER_CLOSED: The operation failed because the remote end
-// of the subject of the operation was closed.
-#[allow(unused)]
-pubconstERR_PEER_CLOSED:zx_status_t = -24;
-
-// ERR_NOT_FOUND: The requested entity is not found.
-#[allow(unused)]
-pubconstERR_NOT_FOUND:zx_status_t = -25;
-
-// ERR_ALREADY_EXISTS: An object with the specified identifier
-// already exists.
-// Example: Attempting to create a file when a file already exists
-// with that name.
-#[allow(unused)]
-pubconstERR_ALREADY_EXISTS:zx_status_t = -26;
-
-// ERR_ALREADY_BOUND: The operation failed because the named entity
-// is already owned or controlled by another entity. The operation
-// could succeed later if the current owner releases the entity.
-#[allow(unused)]
-pubconstERR_ALREADY_BOUND:zx_status_t = -27;
-
-// ERR_UNAVAILABLE: The subject of the operation is currently unable
-// to perform the operation.
-// Note: This is used when there's no direct way for the caller to
-// observe when the subject will be able to perform the operation
-// and should thus retry.
-#[allow(unused)]
-pubconstERR_UNAVAILABLE:zx_status_t = -28;
-
-// ======= Permission check errors =======
-// ERR_ACCESS_DENIED: The caller did not have permission to perform
-// the specified operation.
-#[allow(unused)]
-pubconstERR_ACCESS_DENIED:zx_status_t = -30;
-
-// ======= Input-output errors =======
-// ERR_IO: Otherwise unspecified error occurred during I/O.
-#[allow(unused)]
-pubconstERR_IO:zx_status_t = -40;
-
-// ERR_REFUSED: The entity the I/O operation is being performed on
-// rejected the operation.
-// Example: an I2C device NAK'ing a transaction or a disk controller
-// rejecting an invalid command.
-#[allow(unused)]
-pubconstERR_IO_REFUSED:zx_status_t = -41;
-
-// ERR_IO_DATA_INTEGRITY: The data in the operation failed an integrity
-// check and is possibly corrupted.
-// Example: CRC or Parity error.
-#[allow(unused)]
-pubconstERR_IO_DATA_INTEGRITY:zx_status_t = -42;
-
-// ERR_IO_DATA_LOSS: The data in the operation is currently unavailable
-// and may be permanently lost.
-// Example: A disk block is irrecoverably damaged.
-#[allow(unused)]
-pubconstERR_IO_DATA_LOSS:zx_status_t = -43;
-
-// Filesystem specific errors
-#[allow(unused)]
-pubconstERR_BAD_PATH:zx_status_t = -50;
-#[allow(unused)]
-pubconstERR_NOT_DIR:zx_status_t = -51;
-#[allow(unused)]
-pubconstERR_NOT_FILE:zx_status_t = -52;
-// ERR_FILE_BIG: A file exceeds a filesystem-specific size limit.
-#[allow(unused)]
-pubconstERR_FILE_BIG:zx_status_t = -53;
-// ERR_NO_SPACE: Filesystem or device space is exhausted.
-#[allow(unused)]
-pubconstERR_NO_SPACE:zx_status_t = -54;
+////////////
+// Errors //
+////////////
+
+pubtypezx_status_t = i32;
+
+pubconstZX_OK:zx_status_t = 0;
+pubconstZX_ERR_NOT_SUPPORTED:zx_status_t = -2;
+pubconstZX_ERR_INVALID_ARGS:zx_status_t = -10;
+pubconstZX_ERR_BAD_HANDLE:zx_status_t = -11;
+pubconstZX_ERR_WRONG_TYPE:zx_status_t = -12;
+pubconstZX_ERR_BAD_STATE:zx_status_t = -20;
+pubconstZX_ERR_TIMED_OUT:zx_status_t = -21;
+
+pubfnzx_cvt<T>(t:T) -> io::Result<T>
+where
+T:TryInto<zx_status_t> + Copy,
+{
+ifletOk(status) = TryInto::try_into(t){
+if status < 0{Err(io::Error::from_raw_os_error(status))}else{Ok(t)}
+}else{
+Err(io::Error::last_os_error())
+}
+}