Remove the printk format %e
authorBarret Rhoden <brho@cs.berkeley.edu>
Sat, 13 Feb 2016 19:00:35 +0000 (14:00 -0500)
committerBarret Rhoden <brho@cs.berkeley.edu>
Sun, 14 Feb 2016 14:03:21 +0000 (09:03 -0500)
We were using it in two places, and one of them was incorrect (getchar()
wasn't returning an errno).

It was also blindly inferring errstr() from the context.  If we're going to
do that, we might as well not even pass in the err.  It was basically an
unused perror() for the kernel.

Signed-off-by: Barret Rhoden <brho@cs.berkeley.edu>
kern/src/printfmt.c
kern/src/process.c
kern/src/readline.c

index 9c27346..5128b41 100644 (file)
@@ -131,17 +131,6 @@ void vprintfmt(void (*putch)(int, void**), void **putdat, const char *fmt, va_li
                        putch(va_arg(ap, int), putdat);
                        break;
 
-               // error message
-               case 'e':
-                       err = va_arg(ap, int);
-                       if (err < 0)
-                               err = -err;
-                       if (err >= MAX_ERRNO)
-                               printfmt(putch, putdat, "error %d(%s)", err, get_cur_errbuf());
-                       else
-                               printfmt(putch, putdat, "%s (%s)", errno_to_string(err), get_cur_errbuf());
-                       break;
-
                case 'E': // ENET MAC
                        if ((mac = va_arg(ap, uint8_t *)) == NULL){
                                char *s = "00:00:00:00:00:00";
index dd561ca..cdb7b46 100644 (file)
@@ -431,14 +431,13 @@ void __proc_ready(struct proc *p)
        spin_unlock(&pid_hash_lock);
 }
 
-/* Creates a process from the specified file, argvs, and envps.  Tempted to get
- * rid of proc_alloc's style, but it is so quaint... */
+/* Creates a process from the specified file, argvs, and envps. */
 struct proc *proc_create(struct file *prog, char **argv, char **envp)
 {
        struct proc *p;
        error_t r;
        if ((r = proc_alloc(&p, current, 0 /* flags */)) < 0)
-               panic("proc_create: %e", r);    /* one of 3 quaint usages of %e */
+               panic("proc_create: %d", r);
        int argc = 0, envc = 0;
        if(argv) while(argv[argc]) argc++;
        if(envp) while(envp[envc]) envc++;
index 5565e24..32548a6 100644 (file)
@@ -20,7 +20,7 @@ int readline(char *buf, size_t buf_l, const char *prompt, ...)
        while (1) {
                c = getchar();
                if (c < 0) {
-                       printk("read error: %e\n", c);  /* %e! */
+                       printk("read error: %d\n", c);
                        retval = i;
                        break;
                } else if (c == '\b' || c == 0x7f) {