signal: Remove old way of sending signals and instead use procfs

15 files changed, 59 insertions, 102 deletions

diff --git a/kernel/cpu/syscall.c b/kernel/cpu/syscall.c
index 31f1fc5..de765b4 100644
--- a/kernel/cpu/syscall.c
+++ b/kernel/cpu/syscall.c
@@ -173,17 +173,6 @@ int syscall_isatty(int fd) {
   return 1;
 }
 
-int syscall_kill(int fd, int sig) {
-  vfs_fd_t *fd_ptr = get_vfs_fd(fd, NULL);
-  if (!fd_ptr) {
-    return -EBADF;
-  }
-  if (!fd_ptr->inode->send_signal) {
-    return -EBADF;
-  }
-  return process_signal(fd_ptr, sig);
-}
-
 int syscall_lseek(int fd, int offset, int whence) {
   return vfs_lseek(fd, offset, whence);
 }
@@ -223,29 +212,6 @@ int syscall_open(const char *file, int flags, mode_t mode) {
   return vfs_open(_file, _flags, _mode);
 }
 
-int syscall_open_process(int pid) {
-  // TODO: Permission check
-  process_t *process = (process_t *)ready_queue;
-  for (; process; process = process->next) {
-    if (pid == process->pid) {
-      break;
-    }
-  }
-  if (!process) {
-    return -ESRCH;
-  }
-
-  vfs_inode_t *inode = vfs_create_inode(
-      process->pid, 0 /*type*/, 0 /*has_data*/, 0 /*can_write*/, 1 /*is_open*/,
-      0, process /*internal_object*/, 0 /*file_size*/, NULL /*open*/,
-      NULL /*create_file*/, NULL /*read*/, NULL /*write*/, NULL /*close*/,
-      NULL /*create_directory*/, NULL /*get_vm_object*/, NULL /*truncate*/,
-      NULL /*stat*/, process_signal, NULL /*connect*/);
-  int rc = vfs_create_fd(0, 0, 0, inode, NULL);
-  assert(rc >= 0);
-  return rc;
-}
-
 int syscall_poll(SYS_POLL_PARAMS *args) {
   struct pollfd *fds = args->fds;
   size_t nfds = args->nfds;
@@ -654,14 +620,12 @@ int (*syscall_functions[])() = {
     (void(*))syscall_mkdir,
     (void(*))syscall_recvfrom,
     (void(*))syscall_sendto,
-    (void(*))syscall_kill,
     (void(*))syscall_sigaction,
     (void(*))syscall_chdir,
     (void(*))syscall_getcwd,
     (void(*))syscall_isatty,
     (void(*))syscall_randomfill,
     (void(*))syscall_munmap,
-    (void(*))syscall_open_process,
     (void(*))syscall_lseek,
     (void(*))syscall_connect,
     (void(*))syscall_setsockopt,
diff --git a/kernel/fs/ext2.c b/kernel/fs/ext2.c
index fe09834..0f8e344 100644
--- a/kernel/fs/ext2.c
+++ b/kernel/fs/ext2.c
@@ -745,7 +745,7 @@ vfs_inode_t *ext2_open(const char *path) {
       inode_num, type, NULL, NULL, 1 /*is_open*/, 0, NULL /*internal_object*/,
       file_size, ext2_open, ext2_create_file, ext2_read, ext2_write, ext2_close,
       ext2_create_directory, NULL /*get_vm_object*/, ext2_truncate /*truncate*/,
-      ext2_stat, NULL /*send_signal*/, NULL /*connect*/);
+      ext2_stat, NULL /*connect*/);
 }
 
 u64 end_of_last_entry_position(int dir_inode, u64 *entry_offset,
@@ -960,7 +960,7 @@ vfs_inode_t *ext2_mount(void) {
       0 /*is_open*/, 0, NULL /*internal_object*/, 0 /*file_size*/, ext2_open,
       ext2_create_file, ext2_read, ext2_write, ext2_close,
       ext2_create_directory, NULL /*get_vm_object*/, ext2_truncate /*truncate*/,
-      ext2_stat, NULL /*send_signal*/, NULL /*connect*/);
+      ext2_stat, NULL /*connect*/);
   if (!inode) {
     goto ext2_mount_error;
   }
diff --git a/kernel/fs/procfs.c b/kernel/fs/procfs.c
index f6a4fef..69022d4 100644
--- a/kernel/fs/procfs.c
+++ b/kernel/fs/procfs.c
@@ -134,7 +134,7 @@ vfs_inode_t *open_process(u64 pid, int id) {
       0 /*is_open*/, 0, process /*internal_object*/, 0 /*file_size*/,
       procfs_open, NULL /*create_file*/, process_read, process_write,
       process_close, NULL /*create_directory*/, NULL /*get_vm_object*/,
-      NULL /*truncate*/, NULL /*stat*/, NULL /*send_signal*/, NULL /*connect*/);
+      NULL /*truncate*/, NULL /*stat*/, NULL /*connect*/);
   if (!inode) {
     return NULL;
   }
@@ -152,8 +152,7 @@ vfs_inode_t *procfs_open(const char *p) {
         0 /*is_open*/, 0, NULL /*internal_object*/, 0 /*file_size*/,
         procfs_open, NULL /*create_file*/, procfs_read, NULL /* write */,
         procfs_close, NULL /*create_directory*/, NULL /*get_vm_object*/,
-        NULL /*truncate*/, NULL /*stat*/, NULL /*send_signal*/,
-        NULL /*connect*/);
+        NULL /*truncate*/, NULL /*stat*/, NULL /*connect*/);
   }
 
   int got_num;
@@ -180,6 +179,6 @@ vfs_inode_t *procfs_mount(void) {
       0 /*is_open*/, 0, NULL /*internal_object*/, 0 /*file_size*/, procfs_open,
       NULL /*create_file*/, procfs_read, NULL /* write */, procfs_close,
       NULL /*create_directory*/, NULL /*get_vm_object*/, NULL /*truncate*/,
-      NULL /*stat*/, NULL /*send_signal*/, NULL /*connect*/);
+      NULL /*stat*/, NULL /*connect*/);
   return inode;
 }
diff --git a/kernel/fs/shm.c b/kernel/fs/shm.c
index 5e44b97..be80de5 100644
--- a/kernel/fs/shm.c
+++ b/kernel/fs/shm.c
@@ -101,8 +101,7 @@ int shm_open(const char *name, int oflag, mode_t mode) {
       0 /*inode_num*/, 0 /*type*/, NULL, NULL, 1 /*is_open*/, 0,
       internal_object, 0 /*file_size*/, NULL /*open*/, NULL /*create_file*/,
       shm_read, shm_write, shm_close, NULL /*create_directory*/,
-      shm_get_vm_object, shm_ftruncate, NULL /*stat*/, NULL /*send_signal*/,
-      NULL /*connect*/);
+      shm_get_vm_object, shm_ftruncate, NULL /*stat*/, NULL /*connect*/);
 
   vfs_fd_t *fd_ptr;
   int fd = vfs_create_fd(oflag, mode, 0 /*is_tty*/, inode, &fd_ptr);
diff --git a/kernel/fs/tmpfs.c b/kernel/fs/tmpfs.c
index f979d92..3040622 100644
--- a/kernel/fs/tmpfs.c
+++ b/kernel/fs/tmpfs.c
@@ -62,7 +62,7 @@ void dual_pipe(int fd[2]) {
         internal_object, 0 /*file_size*/, NULL /*open*/, NULL /*create_file*/,
         tmp_read, tmp_write, tmp_close, NULL /*create_directory*/,
         NULL /*get_vm_object*/, NULL /*truncate*/, NULL /*stat*/,
-        NULL /*send_signal*/, NULL /*connect*/);
+        NULL /*connect*/);
     assert(inode);
 
     fd[i] =
@@ -93,7 +93,7 @@ void pipe(int fd[2]) {
         internal_object, 0 /*file_size*/, NULL /*open*/, NULL /*create_file*/,
         tmp_read, tmp_write, tmp_close, NULL /*create_directory*/,
         NULL /*get_vm_object*/, NULL /*truncate*/, NULL /*stat*/,
-        NULL /*send_signal*/, NULL /*connect*/);
+        NULL /*connect*/);
     assert(inode);
 
     fd[i] = vfs_create_fd(O_RDWR, 0, 0 /*is_tty*/, inode, &fd_ptrs[i]);
diff --git a/kernel/fs/vfs.c b/kernel/fs/vfs.c
index 7e23870..f8655bd 100644
--- a/kernel/fs/vfs.c
+++ b/kernel/fs/vfs.c
@@ -38,7 +38,6 @@ vfs_inode_t *vfs_create_inode(
     vfs_vm_object_t *(*get_vm_object)(u64 length, u64 offset, vfs_fd_t *fd),
     int (*truncate)(vfs_fd_t *fd, size_t length),
     int (*stat)(vfs_fd_t *fd, struct stat *buf),
-    int (*send_signal)(vfs_fd_t *fd, int signal),
     int (*connect)(vfs_fd_t *fd, const struct sockaddr *addr,
                    socklen_t addrlen)) {
   vfs_inode_t *r = kcalloc(1, sizeof(inode_t));
@@ -59,7 +58,6 @@ vfs_inode_t *vfs_create_inode(
   r->get_vm_object = get_vm_object;
   r->truncate = truncate;
   r->stat = stat;
-  r->send_signal = send_signal;
   r->connect = connect;
   return r;
 }
diff --git a/kernel/fs/vfs.h b/kernel/fs/vfs.h
index 98220d3..518f304 100644
--- a/kernel/fs/vfs.h
+++ b/kernel/fs/vfs.h
@@ -63,7 +63,6 @@ struct vfs_inode {
   vfs_vm_object_t *(*get_vm_object)(u64 length, u64 offset, vfs_fd_t *fd);
   int (*truncate)(vfs_fd_t *fd, size_t length);
   int (*stat)(vfs_fd_t *fd, struct stat *buf);
-  int (*send_signal)(vfs_fd_t *fd, int signal);
   int (*connect)(vfs_fd_t *fd, const struct sockaddr *addr, socklen_t addrlen);
 };
 
@@ -102,7 +101,6 @@ vfs_inode_t *vfs_create_inode(
     vfs_vm_object_t *(*get_vm_object)(u64 length, u64 offset, vfs_fd_t *fd),
     int (*truncate)(vfs_fd_t *fd, size_t length),
     int (*stat)(vfs_fd_t *fd, struct stat *buf),
-    int (*send_signal)(vfs_fd_t *fd, int signal),
     int (*connect)(vfs_fd_t *fd, const struct sockaddr *addr,
                    socklen_t addrlen));
 #endif
diff --git a/kernel/queue.c b/kernel/queue.c
index e1b2943..7aa9a56 100644
--- a/kernel/queue.c
+++ b/kernel/queue.c
@@ -73,11 +73,10 @@ int queue_create(void) {
   relist_init(&list->entries);
   list->process = current_task;
 
-  vfs_inode_t *inode =
-      vfs_create_inode(0, 0, queue_has_data, NULL, 1 /*is_open*/, OBJECT_QUEUE,
-                       list /*internal_object*/, 0, NULL, NULL, NULL, NULL,
-                       queue_close, NULL, NULL /*get_vm_object*/, NULL, NULL,
-                       NULL /*send_signal*/, NULL /*connect*/);
+  vfs_inode_t *inode = vfs_create_inode(
+      0, 0, queue_has_data, NULL, 1 /*is_open*/, OBJECT_QUEUE,
+      list /*internal_object*/, 0, NULL, NULL, NULL, NULL, queue_close, NULL,
+      NULL /*get_vm_object*/, NULL, NULL, NULL /*connect*/);
   assert(inode);
   return vfs_create_fd(0, 0, 0, inode, NULL);
 }
diff --git a/kernel/socket.c b/kernel/socket.c
index 7ed6769..57f869a 100644
--- a/kernel/socket.c
+++ b/kernel/socket.c
@@ -634,7 +634,7 @@ int accept(int socket, struct sockaddr *address, socklen_t *address_len) {
         OBJECT_TCP, connection, 0 /*file_size*/, NULL /*open*/,
         NULL /*create_file*/, tcp_read, tcp_write, tcp_close /*close*/,
         NULL /*create_directory*/, NULL /*get_vm_object*/, NULL /*truncate*/,
-        NULL /*stat*/, NULL /*send_signal*/, NULL /*connect*/);
+        NULL /*stat*/, NULL /*connect*/);
     inode->_is_open = tcp_is_open;
     assert(inode);
     return vfs_create_fd(O_RDWR, 0, 0 /*is_tty*/, inode, NULL);
@@ -782,8 +782,7 @@ int tcp_create_fd(int is_nonblock) {
       0 /*inode_num*/, FS_TYPE_UNIX_SOCKET, NULL, always_can_write, 1,
       OBJECT_TCP, con, 0 /*file_size*/, NULL /*open*/, NULL /*create_file*/,
       tcp_read, tcp_write, NULL /*close*/, NULL /*create_directory*/,
-      NULL /*get_vm_object*/, NULL /*truncate*/, NULL /*stat*/,
-      NULL /*send_signal*/, tcp_connect);
+      NULL /*get_vm_object*/, NULL /*truncate*/, NULL /*stat*/, tcp_connect);
   if (!inode) {
     kfree(con);
     return -ENOMEM;
@@ -825,7 +824,7 @@ int socket(int domain, int type, int protocol) {
         0 /*file_size*/, NULL /*open*/, NULL /*create_file*/, socket_read,
         socket_write, socket_close, NULL /*create_directory*/,
         NULL /*get_vm_object*/, NULL /*truncate*/, NULL /*stat*/,
-        NULL /*send_signal*/, NULL /*connect*/);
+        NULL /*connect*/);
     if (!inode) {
       rc = -ENOMEM;
       goto socket_error;
@@ -855,7 +854,7 @@ int socket(int domain, int type, int protocol) {
         OBJECT_UNIX, new_socket, 0 /*file_size*/, NULL /*open*/,
         NULL /*create_file*/, NULL, NULL, NULL /*close*/,
         NULL /*create_directory*/, NULL /*get_vm_object*/, NULL /*truncate*/,
-        NULL /*stat*/, NULL /*send_signal*/, udp_connect);
+        NULL /*stat*/, udp_connect);
     if (!inode) {
       rc = -ENOMEM;
       goto socket_error;
diff --git a/userland/libc/Makefile b/userland/libc/Makefile
index 0e56a4a..e5aeff1 100644
--- a/userland/libc/Makefile
+++ b/userland/libc/Makefile
@@ -2,7 +2,7 @@ CC="i686-sb-gcc"
 AR="i686-sb-ar"
 AS="i686-sb-as"
 CFLAGS = -ggdb -ffreestanding -O2 -Wall -Wextra -pedantic -Werror -Wimplicit-fallthrough -I./include/ -static -I../../include/ -Wno-int-conversion -Wno-unused-parameter
-OBJ=crt0.o libc.o malloc/malloc.o pty.o sys/mman/mmap.o sys/mman/munmap.o memset.o assert.o stdio/snprintf.o stdio/vfprintf.o string/memcpy.o string/memcmp.o string/strcmp.o ubsan.o string/strcpy.o isspace.o stdio/puts.o stdio/putchar.o dirent/opendir.o dirent/readdir.o dirent/closedir.o unistd/getopt.o dirent/scandir.o dirent/alphasort.o stdio/printf.o stdio/vdprintf.o stdio/vprintf.o stdio/dprintf.o stdio/vprintf.o string/strlen.o string/strnlen.o stdio/stdin.o stdio/getchar.o stdio/fgetc.o arpa/inet/htons.o arpa/inet/htonl.o stdio/fread.o stdio/fwrite.o stdio/fopen.o stdio/fclose.o stdio/fseek.o ctype/isascii.o stdio/fprintf.o stdlib/atoi.o stdlib/strtol.o ctype/toupper.o ctype/tolower.o string/strcat.o string/strchr.o string/sscanf.o sys/stat/stat.o stdlib/getenv.o string/strrchr.o stdio/ftell.o stdio/tmpfile.o stdio/fgets.o stdio/feof.o stdio/fscanf.o stdio/ungetc.o string/strncmp.o stdio/fputc.o string/strncpy.o stdio/remove.o stdio/ferror.o stdio/fputs.o stdlib/rand.o stdlib/srand.o unistd/getpid.o stdlib/strtoul.o stdio/fflush.o stdlib/abort.o string/strcspn.o time/localtime.o time/time.o time/clock_gettime.o time/gmtime.o time/strftime.o string/strpbrk.o ctype/isdigit.o ctype/isalpha.o ctype/isxdigit.o ctype/ispunct.o stdio/setvbuf.o stdio/fileno.o stdio/putc.o stdio/sprintf.o stdlib/abs.o string/strspn.o stdlib/qsort.o string/memmove.o setjmp/longjmp.o setjmp/setjmp.o libgen/basename.o string/strdup.o string/strndup.o string/strlcpy.o stdlib/atexit.o stdio/open_memstream.o libgen/dirname.o unistd/unlink.o string/strstr.o string/strcasecmp.o string/strncasecmp.o stdlib/mkstemp.o string/strtok.o unistd/execvp.o unistd/_exit.o ctype/isalnum.o time/ctime_r.o stdlib/strtold.o sys/time/gettimeofday.o stdio/fgetpos.o stdio/fsetpos.o ctype/isprint.o stdlib/system.o stdio/tmpnam.o unistd/msleep.o stdlib/atof.o stdlib/strtod.o stdio/rename.o sys/stat/mkdir.o unistd/uptime.o unistd/ftruncate.o sys/socket/recvfrom.o sys/socket/sendto.o signal/kill.o signal/sigaction.o unistd/chdir.o unistd/getcwd.o stdio/getdelim.o stdio/getline.o unistd/isatty.o sys/socket/listen.o stdlib/realpath.o systemcall.o sys/random/randomfill.o fcntl/open.o unistd/write.o unistd/pwrite.o fcntl/open_process.o tb/sb.o tb/sv.o string/memchr.o stdlib/atol.o stdlib/atoll.o stdlib/strtoll.o sys/stat/fstat.o unistd/lseek.o ctype/isupper.o ctype/islower.o ctype/isblank.o ctype/isgraph.o ctype/iscntrl.o math/ldexp.o sys/socket/connect.o sys/socket/setsockopt.o arpa/inet/ntohl.o arpa/inet/ntohs.o netdb/getaddrinfo.o tb/sha1.o sys/socket/getpeername.o fcntl/fcntl.o queue.o sys/sendfile.o stdio/fdopen.o unistd/dup.o
+OBJ=crt0.o libc.o malloc/malloc.o pty.o sys/mman/mmap.o sys/mman/munmap.o memset.o assert.o stdio/snprintf.o stdio/vfprintf.o string/memcpy.o string/memcmp.o string/strcmp.o ubsan.o string/strcpy.o isspace.o stdio/puts.o stdio/putchar.o dirent/opendir.o dirent/readdir.o dirent/closedir.o unistd/getopt.o dirent/scandir.o dirent/alphasort.o stdio/printf.o stdio/vdprintf.o stdio/vprintf.o stdio/dprintf.o stdio/vprintf.o string/strlen.o string/strnlen.o stdio/stdin.o stdio/getchar.o stdio/fgetc.o arpa/inet/htons.o arpa/inet/htonl.o stdio/fread.o stdio/fwrite.o stdio/fopen.o stdio/fclose.o stdio/fseek.o ctype/isascii.o stdio/fprintf.o stdlib/atoi.o stdlib/strtol.o ctype/toupper.o ctype/tolower.o string/strcat.o string/strchr.o string/sscanf.o sys/stat/stat.o stdlib/getenv.o string/strrchr.o stdio/ftell.o stdio/tmpfile.o stdio/fgets.o stdio/feof.o stdio/fscanf.o stdio/ungetc.o string/strncmp.o stdio/fputc.o string/strncpy.o stdio/remove.o stdio/ferror.o stdio/fputs.o stdlib/rand.o stdlib/srand.o unistd/getpid.o stdlib/strtoul.o stdio/fflush.o stdlib/abort.o string/strcspn.o time/localtime.o time/time.o time/clock_gettime.o time/gmtime.o time/strftime.o string/strpbrk.o ctype/isdigit.o ctype/isalpha.o ctype/isxdigit.o ctype/ispunct.o stdio/setvbuf.o stdio/fileno.o stdio/putc.o stdio/sprintf.o stdlib/abs.o string/strspn.o stdlib/qsort.o string/memmove.o setjmp/longjmp.o setjmp/setjmp.o libgen/basename.o string/strdup.o string/strndup.o string/strlcpy.o stdlib/atexit.o stdio/open_memstream.o libgen/dirname.o unistd/unlink.o string/strstr.o string/strcasecmp.o string/strncasecmp.o stdlib/mkstemp.o string/strtok.o unistd/execvp.o unistd/_exit.o ctype/isalnum.o time/ctime_r.o stdlib/strtold.o sys/time/gettimeofday.o stdio/fgetpos.o stdio/fsetpos.o ctype/isprint.o stdlib/system.o stdio/tmpnam.o unistd/msleep.o stdlib/atof.o stdlib/strtod.o stdio/rename.o sys/stat/mkdir.o unistd/uptime.o unistd/ftruncate.o sys/socket/recvfrom.o sys/socket/sendto.o signal/kill.o signal/sigaction.o unistd/chdir.o unistd/getcwd.o stdio/getdelim.o stdio/getline.o unistd/isatty.o sys/socket/listen.o stdlib/realpath.o systemcall.o sys/random/randomfill.o fcntl/open.o unistd/write.o unistd/pwrite.o tb/sb.o tb/sv.o string/memchr.o stdlib/atol.o stdlib/atoll.o stdlib/strtoll.o sys/stat/fstat.o unistd/lseek.o ctype/isupper.o ctype/islower.o ctype/isblank.o ctype/isgraph.o ctype/iscntrl.o math/ldexp.o sys/socket/connect.o sys/socket/setsockopt.o arpa/inet/ntohl.o arpa/inet/ntohs.o netdb/getaddrinfo.o tb/sha1.o sys/socket/getpeername.o fcntl/fcntl.o queue.o sys/sendfile.o stdio/fdopen.o unistd/dup.o
 all: libc.a
 
 %.o: %.c
diff --git a/userland/libc/fcntl/open_process.c b/userland/libc/fcntl/open_process.c
deleted file mode 100644
index 8486728..0000000
--- a/userland/libc/fcntl/open_process.c
+++ /dev/null
@@ -1,6 +0,0 @@
-#include <fcntl.h>
-#include <syscall.h>
-
-int open_process(int pid) {
-  RC_ERRNO(syscall(SYS_OPEN_PROCESS, pid, 0, 0, 0, 0));
-}
diff --git a/userland/libc/include/syscall.h b/userland/libc/include/syscall.h
index 95ab817..3699854 100644
--- a/userland/libc/include/syscall.h
+++ b/userland/libc/include/syscall.h
@@ -34,26 +34,24 @@
 #define SYS_MKDIR 26
 #define SYS_RECVFROM 27
 #define SYS_SENDTO 28
-#define SYS_KILL 29
-#define SYS_SIGACTION 30
-#define SYS_CHDIR 31
-#define SYS_GETCWD 32
-#define SYS_ISATTY 33
-#define SYS_RANDOMFILL 34
-#define SYS_MUNMAP 35
-#define SYS_OPEN_PROCESS 36
-#define SYS_LSEEK 37
-#define SYS_CONNECT 38
-#define SYS_SETSOCKOPT 39
-#define SYS_GETPEERNAME 40
-#define SYS_FCNTL 41
-#define SYS_CLOCK_GETTIME 42
-#define SYS_QUEUE_CREATE 43
-#define SYS_QUEUE_MOD_ENTRIES 44
-#define SYS_QUEUE_WAIT 45
-#define SYS_SENDFILE 46
-#define SYS_SHM_UNLINK 47
-#define SYS_DUP 48
+#define SYS_SIGACTION 29
+#define SYS_CHDIR 30
+#define SYS_GETCWD 31
+#define SYS_ISATTY 32
+#define SYS_RANDOMFILL 33
+#define SYS_MUNMAP 34
+#define SYS_LSEEK 35
+#define SYS_CONNECT 36
+#define SYS_SETSOCKOPT 37
+#define SYS_GETPEERNAME 38
+#define SYS_FCNTL 39
+#define SYS_CLOCK_GETTIME 40
+#define SYS_QUEUE_CREATE 41
+#define SYS_QUEUE_MOD_ENTRIES 42
+#define SYS_QUEUE_WAIT 43
+#define SYS_SENDFILE 44
+#define SYS_SHM_UNLINK 45
+#define SYS_DUP 46
 
 int syscall(uint32_t eax, uint32_t ebx, uint32_t ecx, uint32_t edx,
             uint32_t esi, uint32_t edi);
diff --git a/userland/libc/signal/kill.c b/userland/libc/signal/kill.c
index c8a368f..4518b49 100644
--- a/userland/libc/signal/kill.c
+++ b/userland/libc/signal/kill.c
@@ -1,15 +1,20 @@
+#include <assert.h>
 #include <fcntl.h>
-#include <unistd.h>
 #include <signal.h>
+#include <stdio.h>
 #include <syscall.h>
-
-int kill_fd(int fd, int sig) {
-  RC_ERRNO(syscall(SYS_KILL, fd, sig, 0, 0, 0))
-}
+#include <unistd.h>
 
 int kill(int pid, int sig) {
-  int fd = open_process(pid);
-  int rc = kill_fd(fd, sig);
-  close(fd);
-  return rc;
+  char buffer[4096];
+  snprintf(buffer, sizeof(buffer), "/proc/%d/signal", pid);
+  int fd = open(buffer, O_WRITE);
+  if (-1 == fd) {
+    return -1;
+  }
+  if (-1 == dprintf(fd, "%d", sig)) {
+    return -1;
+  }
+  assert(-1 != close(fd));
+  return 0;
 }
diff --git a/userland/minibox/utilities/ls.c b/userland/minibox/utilities/ls.c
index 0ce70f8..5a8f8ea 100644
--- a/userland/minibox/utilities/ls.c
+++ b/userland/minibox/utilities/ls.c
@@ -20,10 +20,16 @@ int ls_main(int argc, char **argv) {
                           newline = 1;
                           break;
                   }*/
+  char *path = argv[1];
+  char path_buffer[256];
+
+  if (!path) {
+    (void)getcwd(path_buffer, 256);
+    path = path_buffer;
+  }
+
   struct dirent **namelist;
   int n;
-  char path[256];
-  (void)getcwd(path, 256);
   COND_PERROR_EXP(-1 == (n = scandir(path, &namelist, 0, 0)), "scandir",
                   return 1);
 
diff --git a/userland/minibox/utilities/rdate.c b/userland/minibox/utilities/rdate.c
index 11398c8..54a2746 100644
--- a/userland/minibox/utilities/rdate.c
+++ b/userland/minibox/utilities/rdate.c
@@ -42,9 +42,7 @@ int rdate_main(int argc, char **argv) {
   int64_t unix_time = (t - 2208988800) * 1000;
   int clock_fd = open("/dev/clock", O_RDWR);
   int64_t current;
-  read(clock_fd, &current, sizeof(int64_t));
-  write(clock_fd, &unix_time, sizeof(int64_t));
-  int64_t delta = (current / 1000) - (unix_time / 1000);
-  printf("delta: %d\n", delta);
+  dprintf(clock_fd, "%lld", unix_time);
+  close(clock_fd);
   return 0;
 }