kernel: Add initial read() implementation

Signed-off-by: Ian Moffett <ian@osmora.org>

2 files changed, 68 insertions, 0 deletions

diff --git a/sys/kern/kern_filedesc.c b/sys/kern/kern_filedesc.c
index 6f59508..c610747 100644
--- a/sys/kern/kern_filedesc.c
+++ b/sys/kern/kern_filedesc.c
@@ -298,6 +298,36 @@ open(const char *pathname, int oflag)
 }
 
 /*
+ * Read file into a buffer.
+ *
+ * @fd: File descriptor number.
+ * @buf: Buffer to read to.
+ * @count: Number of bytes to read.
+ */
+int
+read(int fd, void *buf, size_t count)
+{
+    ssize_t bytes_read;
+    struct vnode *vnode;
+    struct filedesc *fd_desc;
+
+    fd_desc = fd_from_fdnum(this_td(), fd);
+
+    if (fd_desc == NULL) {
+        return -EBADF;
+    }
+
+    vnode = fd_desc->vnode;
+
+    if (count > MAX_RW_SIZE) {
+        return -EINVAL;
+    }
+
+    bytes_read = vfs_read(vnode, buf, count);
+    return bytes_read;
+}
+
+/*
  * arg0: int fd
  * arg1: const void *buf
  * arg2: count
@@ -340,3 +370,40 @@ sys_close(struct syscall_args *args)
     fd_close_fdnum(this_td(), args->arg0);
     return 0;
 }
+
+/*
+ * arg0: fd
+ * arg1: char *buf
+ * arg2: size_t count
+ */
+uint64_t
+sys_read(struct syscall_args *args)
+{
+    char *kbuf;
+    ssize_t bytes_read;
+
+    if (args->arg2 > MAX_RW_SIZE || args->arg2 == 0) {
+        return -EINVAL;
+    }
+
+    kbuf = dynalloc(args->arg2);
+
+    if (kbuf == NULL) {
+        return -ENOMEM;
+    }
+
+    /*
+     * Try to read into our kernel buffer then copy out
+     * to userspace.
+     */
+    if ((bytes_read = read(args->arg0, kbuf, args->arg2)) < 0) {
+        /* Failure */
+        dynfree(kbuf);
+        return bytes_read;
+    }
+
+    if (copyout(kbuf, args->arg1, bytes_read) != 0) {
+        invalid_uaddr(args->arg1);
+    }
+    return bytes_read;
+}
diff --git a/sys/kern/kern_syscall.c b/sys/kern/kern_syscall.c
index 4acfdc5..0f45469 100644
--- a/sys/kern/kern_syscall.c
+++ b/sys/kern/kern_syscall.c
@@ -45,4 +45,5 @@ uint64_t(*g_syscall_table[__MAX_SYSCALLS])(struct syscall_args *args) = {
     sys_write,
     sys_open,
     sys_close,
+    sys_read,
 };