fix shitty attributes

arch/x86_64/archctl.c

@@ -12,21 +12,21 @@ struct syscall_ret arch_archctl(uintptr_t op, uintptr_t arg) {
     switch (op) {
     case __ARCHCTL_SET_FSBASE:
         if (!arch_mem_addr_is_user(arg)) {
-            ret.errno = EINVAL;
+            ret.error = EINVAL;
             return ret;
         }
         asm_wrmsr(MSR_FS_BASE, arg);
         return ret;
     case __ARCHCTL_SET_GSBASE:
         if (!arch_mem_addr_is_user(arg)) {
-            ret.errno = EINVAL;
+            ret.error = EINVAL;
             return ret;
         }
         // Update the inactive GSBASE MSR, not ours.
         asm_wrmsr(MSR_KERNEL_GS_BASE, arg);
         return ret;
     default:
-        ret.errno = EINVAL;
+        ret.error = EINVAL;
         return ret;
     }
 }

include/zinnia/compiler.h

@@ -14,18 +14,10 @@
 #define __format(like, ...) gnu::format(like, __VA_ARGS__)
 #define __always_inline     gnu::always_inline
 #define __visibility        gnu::visibility
-
+#define __pure              gnu::pure
-// TODO: Workaround for clang which doesn't have these C23 standard attributes yet.
-#if !__has_c_attribute(unsequenced)
-#define unsequenced gnu::const
-#endif
-
-#if !__has_c_attribute(reproducible)
-#define reproducible gnu::pure
-#endif
 
 // Only clang has these hints, GCC would throw a warning here.
-// clang::noderef also don't support the C23 attribute format.
+// clang::noderef also doesn't support the C23 attribute format.
 #ifdef __clang__
 #define __user __attribute__((noderef, address_space(1)))
 #define __phys __attribute__((noderef, address_space(2)))

include/zinnia/syscall.h

@@ -4,7 +4,7 @@
 
 struct syscall_ret {
     uintptr_t value;
-    int errno;
+    int error;
 };
 
 struct syscall {

include/zinnia/syscall_list.inc

@@ -1,5 +1,5 @@
 #ifndef SYSCALL_DECLARE
-#define SYSCALL_DECLARE(number, name, num_args) extern void sys_##name(...);
+#define SYSCALL_DECLARE(number, name, num_args)
 #endif
 
 SYSCALL_DECLARE(0, exit, 1)

syscalls/dispatch.c

@@ -8,6 +8,7 @@
 #include <bits/sched.h>
 
 // Declare all syscalls.
+#define SYSCALL_DECLARE(number, name, num_args) extern void sys_##name(...);
 #include <zinnia/syscall_list.inc>
 #undef SYSCALL_DECLARE
 
@@ -43,5 +44,5 @@ void syscall_dispatch(struct arch_context* ctx) {
     }
 
     ctx->ARCH_CTX_RET0 = ret.value;
-    ctx->ARCH_CTX_RET1 = ret.errno;
+    ctx->ARCH_CTX_RET1 = ret.error;
 }

syscalls/exit.c

@@ -4,7 +4,7 @@
 struct syscall_ret sys_exit(int err) {
     struct syscall_ret ret = {};
 
-    ret.errno = ENOSYS;
+    ret.error = ENOSYS;
 
     panic("sys::exit() not implemented!\n");
 

syscalls/uname.c

@@ -20,7 +20,7 @@ struct syscall_ret sys_setuname(const __user struct __utsname* ptr) {
     struct syscall_ret ret = {};
 
     if (!ptr) {
-        ret.errno = EINVAL;
+        ret.error = EINVAL;
         return ret;
     }
 
@@ -28,7 +28,7 @@ struct syscall_ret sys_setuname(const __user struct __utsname* ptr) {
 
     struct __utsname read_buf;
     if (!usercopy_read(&read_buf, ptr, sizeof(read_buf))) {
-        ret.errno = EFAULT;
+        ret.error = EFAULT;
         return ret;
     }
 
@@ -50,7 +50,7 @@ struct syscall_ret sys_getuname(__user struct __utsname* ptr) {
     struct syscall_ret ret = {};
 
     if (!ptr) {
-        ret.errno = EINVAL;
+        ret.error = EINVAL;
         return ret;
     }
 
@@ -59,7 +59,7 @@ struct syscall_ret sys_getuname(__user struct __utsname* ptr) {
     spin_unlock(&uname_lock);
 
     if (!usercopy_write(ptr, &write_buf, sizeof(write_buf)))
-        ret.errno = EFAULT;
+        ret.error = EFAULT;
 
     return ret;
 }