alarm/cons/eventfd/random: improve tap errstr message
authorBarret Rhoden <brho@cs.berkeley.edu>
Thu, 20 Jun 2019 15:43:27 +0000 (11:43 -0400)
committerBarret Rhoden <brho@cs.berkeley.edu>
Thu, 11 Jul 2019 18:29:21 +0000 (14:29 -0400)
It's nice to know what the attempted tap was.

Signed-off-by: Barret Rhoden <brho@cs.berkeley.edu>
kern/drivers/dev/alarm.c
kern/drivers/dev/cons.c
kern/drivers/dev/eventfd.c
kern/drivers/dev/random.c

index 2186af2..1042803 100644 (file)
@@ -498,8 +498,8 @@ static int tap_alarm(struct proc_alarm *a, struct fd_tap *tap, int cmd,
        int ret;
 
        if (tap->filter & ~legal_filter) {
-               set_error(ENOSYS, "Unsupported #%s tap, must be %p", devname(),
-                                 legal_filter);
+               set_error(ENOSYS, "Unsupported #%s tap %p, must be %p",
+                         devname(), tap->filter, legal_filter);
                return -1;
        }
        cv_lock(&a->cv);
index e012e68..34daf07 100644 (file)
@@ -1302,8 +1302,8 @@ static int tap_stdin(struct chan *c, struct fd_tap *tap, int cmd)
 #define CONS_STDIN_TAPS (FDTAP_FILT_READABLE | FDTAP_FILT_HANGUP)
 
        if (tap->filter & ~CONS_STDIN_TAPS) {
-               set_error(ENOSYS, "Unsupported #%s tap, must be %p", devname(),
-                         CONS_STDIN_TAPS);
+               set_error(ENOSYS, "Unsupported #%s tap %p, must be %p",
+                         devname(), tap->filter, CONS_STDIN_TAPS);
                return -1;
        }
        spin_lock(&cons_q_lock);
@@ -1342,8 +1342,8 @@ static int tap_null(struct chan *c, struct fd_tap *tap, int cmd)
 #define CONS_NULL_TAPS (FDTAP_FILT_WRITABLE | FDTAP_FILT_HANGUP)
 
        if (tap->filter & ~CONS_NULL_TAPS) {
-               set_error(ENOSYS, "Unsupported #%s tap, must be %p", devname(),
-                         CONS_NULL_TAPS);
+               set_error(ENOSYS, "Unsupported #%s tap %p, must be %p",
+                         devname(), tap->filter, CONS_NULL_TAPS);
                return -1;
        }
        return 0;
index 479666c..8430348 100644 (file)
@@ -295,8 +295,8 @@ static int efd_tapfd(struct chan *c, struct fd_tap *tap, int cmd)
        switch (c->qid.path) {
        case Qefd:
                if (tap->filter & ~EFD_LEGAL_TAPS) {
-                       set_error(ENOSYS, "Unsupported #%s tap, must be %p",
-                                 devname(), EFD_LEGAL_TAPS);
+                       set_error(ENOSYS, "Unsupported #%s tap %p, must be %p",
+                                 devname(), tap->filter, EFD_LEGAL_TAPS);
                        return -1;
                }
                spin_lock(&efd->tap_lock);
index 75e4acb..da416b1 100644 (file)
@@ -203,8 +203,8 @@ static int random_tapfd(struct chan *c, struct fd_tap *tap, int cmd)
        #define RANDOM_TAPS (FDTAP_FILT_READABLE | FDTAP_FILT_HANGUP)
 
        if (tap->filter & ~RANDOM_TAPS) {
-               set_error(ENOSYS, "Unsupported #%s tap, must be %p", devname(),
-                         RANDOM_TAPS);
+               set_error(ENOSYS, "Unsupported #%s tap %p, must be %p",
+                         devname(), tap->filter, RANDOM_TAPS);
                return -1;
        }
        switch (c->qid.path) {