Minor formatting changes
[akaros.git] / kern / src / devfs.c
index 7769379..2cc2764 100644 (file)
@@ -62,8 +62,7 @@ struct file *make_device(char *path, int mode, int type,
 /* We provide a separate set of f_ops for devices (char and block), and the fops
  * is the only thing that differs from the regular KFS.  We need to do some
  * ghetto-overriding of these ops after we create them. */
-
-off_t dev_c_llseek(struct file *file, off_t offset, int whence)
+int dev_c_llseek(struct file *file, off64_t offset, off64_t *ret, int whence)
 {
        set_errno(EINVAL);
        return -1;
@@ -79,16 +78,14 @@ int dev_mmap(struct file *file, struct vm_region *vmr)
 /* this is really /dev/console, and will need some tty work.  for now, no matter
  * how much they ask for, we return one character at a time. */
 ssize_t dev_stdin_read(struct file *file, char *buf, size_t count,
-                       off_t *offset)
+                       off64_t *offset)
 {
        char c;
        extern struct kb_buffer cons_buf;
+
+       if (!count)
+               return 0;
        kb_get_from_buf(&cons_buf, &c, 1);
-       /* this is faking some tcgetattr attributes */
-       /* applications (ash) expect a \n instead of a \r */
-       if (c == '\r')
-               c = '\n';
-       printk("%c", c);
        /* TODO UMEM */
        if (current)
                memcpy_to_user_errno(current, buf, &c, 1);
@@ -98,7 +95,7 @@ ssize_t dev_stdin_read(struct file *file, char *buf, size_t count,
 }
 
 ssize_t dev_stdout_write(struct file *file, const char *buf, size_t count,
-                         off_t *offset)
+                         off64_t *offset)
 {
        char *t_buf;
        struct proc *p = current;
@@ -108,7 +105,14 @@ ssize_t dev_stdout_write(struct file *file, const char *buf, size_t count,
                t_buf = (char*)buf;
        if (!t_buf)
                return -1;
-       printk("%s", t_buf);
+       /* TODO: tty hack.  they are sending us an escape sequence, and the keyboard
+        * would try to print it (which it can't do yet).  The hack is even dirtier
+        * in that we only detect it if it is the first char, and we ignore
+        * everything else. */
+       if (t_buf[0] != '\033') /* 0x1b */
+               cputbuf(t_buf, count);
+       if (p)
+               user_memdup_free(p, t_buf);
        return count;
 }
 
@@ -147,16 +151,22 @@ struct file_operations dev_f_op_stdout = {
        kfs_check_flags,
 };
 
+ssize_t dev_null_read(struct file *file, char *buf, size_t count,
+                      off64_t *offset)
+{
+       return 0;
+}
+
 /* /dev/null: just take whatever was given and pretend it was written */
 ssize_t dev_null_write(struct file *file, const char *buf, size_t count,
-                       off_t *offset)
+                       off64_t *offset)
 {
        return count;
 }
 
 struct file_operations dev_f_op_null = {
        dev_c_llseek,
-       0,      /* read - can't read null */
+       dev_null_read,
        dev_null_write,
        kfs_readdir,    /* this will fail gracefully */
        dev_mmap,