From a18da25e7355979d0f26cfd39dc0032172e8b135 Mon Sep 17 00:00:00 2001
From: Anton Kling <anton@kling.gg>
Date: Thu, 22 Feb 2024 17:28:08 +0100
Subject: Kernel: Fix undefined behavior and cleanup functions.

The int_syscall caused crashes for higher level of optimizations. To fix
this I rewrote the function in assembly as I deemed it to be simpler.
---
 userland/windowserver/Makefile | 5 +++--
 userland/windowserver/ws.c     | 4 ++--
 2 files changed, 5 insertions(+), 4 deletions(-)

(limited to 'userland/windowserver')

diff --git a/userland/windowserver/Makefile b/userland/windowserver/Makefile
index 7f80f2e..0adbdaa 100644
--- a/userland/windowserver/Makefile
+++ b/userland/windowserver/Makefile
@@ -1,5 +1,6 @@
 CC="i686-sb-gcc"
-CFLAGS = -ggdb -ffreestanding -O2 -Wall -Wextra -pedantic -mgeneral-regs-only -Wimplicit-fallthrough
+#CFLAGS = -ggdb -ffreestanding -O2 -Wall -Wextra -pedantic -mgeneral-regs-only -Wimplicit-fallthrough
+CFLAGS = -ggdb -ffreestanding -Ofast -Wall -Wextra -pedantic -mgeneral-regs-only -Wimplicit-fallthrough
 BIN=ws
 LIB=-lgcc
 INC=-I../libgui/
@@ -13,4 +14,4 @@ clean:
 	rm $(OBJ) ws
 
 $(BIN): $(OBJ)
-	$(CC) -o $(BIN) $(CFLAGS) $(OBJ) $(LIB)
+	$(CC) -flto -o $(BIN) $(CFLAGS) $(OBJ) $(LIB)
diff --git a/userland/windowserver/ws.c b/userland/windowserver/ws.c
index a438566..a4f5a59 100644
--- a/userland/windowserver/ws.c
+++ b/userland/windowserver/ws.c
@@ -333,8 +333,8 @@ void parse_mouse_event(int fd) {
         x |= 0xFF00;
       if (ys)
         y |= 0xFF00;
-      xc += *(int16_t *)&x;
-      yc += *(int16_t *)&y;
+      xc += x;
+      yc += y;
     }
   }
   mouse_x += xc;
-- 
cgit v1.2.3