[src/trunk]: src/lib/libpuffs Be slightly more forgiving about unknown operat...

[pooka <pooka%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/b984a54a98e3
branches:  trunk
changeset: 750599:b984a54a98e3
user:      pooka <pooka%NetBSD.org@localhost>
date:      Thu Jan 07 22:46:11 2010 +0000

description:
Be slightly more forgiving about unknown operation types.

diffstat:

 lib/libpuffs/opdump.c |  13 ++++++++-----
 1 files changed, 8 insertions(+), 5 deletions(-)

diffs (44 lines):

diff -r a367d61b1228 -r b984a54a98e3 lib/libpuffs/opdump.c
--- a/lib/libpuffs/opdump.c     Thu Jan 07 22:45:31 2010 +0000
+++ b/lib/libpuffs/opdump.c     Thu Jan 07 22:46:11 2010 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: opdump.c,v 1.29 2010/01/07 20:47:47 pooka Exp $        */
+/*     $NetBSD: opdump.c,v 1.30 2010/01/07 22:46:11 pooka Exp $        */
 
 /*
  * Copyright (c) 2005, 2006  Antti Kantee.  All Rights Reserved.
@@ -35,7 +35,7 @@
 
 #include <sys/cdefs.h>
 #if !defined(lint)
-__RCSID("$NetBSD: opdump.c,v 1.29 2010/01/07 20:47:47 pooka Exp $");
+__RCSID("$NetBSD: opdump.c,v 1.30 2010/01/07 22:46:11 pooka Exp $");
 #endif /* !lint */
 
 #include <sys/types.h>
@@ -147,7 +147,7 @@
        const char **map;
        int isvn = 0;
 
-       map = NULL; /* yes, we are all interested in your opinion, gcc */
+       printf("reqid: %" PRIu64 ", ", preq->preq_id);
        switch (PUFFSOP_OPCLASS(preq->preq_opclass)) {
        case PUFFSOP_VFS:
                map = vfsop_revmap;
@@ -165,11 +165,14 @@
        case PUFFSOP_FLUSH:
                map = flush_revmap;
                break;
+       default:
+               printf("unhandled opclass\n");
+               return;
        }
 
-       printf("reqid: %" PRIu64 ", opclass %d%s, optype: %s, "
+       printf("opclass %d%s, optype: %s, "
            "cookie: %p,\n" DINT "aux: %p, auxlen: %zu, pid: %d, lwpid: %d\n",
-           preq->preq_id, PUFFSOP_OPCLASS(preq->preq_opclass),
+           PUFFSOP_OPCLASS(preq->preq_opclass),
            PUFFSOP_WANTREPLY(preq->preq_opclass) ? "" : " (FAF)",
            map[preq->preq_optype], preq->preq_cookie,
            preq->preq_buf, preq->preq_buflen,