Remove getcallerpc()
authorBarret Rhoden <brho@cs.berkeley.edu>
Mon, 8 Apr 2019 13:42:28 +0000 (09:42 -0400)
committerBarret Rhoden <brho@cs.berkeley.edu>
Mon, 8 Apr 2019 13:48:36 +0000 (09:48 -0400)
It did nothing and was just confusing.  For example, the panic message
for a double-close was just: "cclose 0x0000000000000000".

Signed-off-by: Barret Rhoden <brho@cs.berkeley.edu>
kern/drivers/dev/cons.c
kern/include/ns.h
kern/src/ns/chan.c
kern/src/ns/qio.c

index 4fdb4ea..809a325 100644 (file)
@@ -323,11 +323,6 @@ void sysfatal(char *fmt, ...)
        panic("sysfatal: %s", err);
 }
 
-void _assert(char *fmt)
-{
-       panic("assert failed at %#p: %s", getcallerpc(&fmt), fmt);
-}
-
 #if 0
 int pprint(char *fmt, ...)
 {
index 5615df0..35c6933 100644 (file)
@@ -67,10 +67,6 @@ extern double pow10(int);
  * one-of-a-kind
  */
 extern char *cleanname(char *unused_char_p_t);
-static inline uint32_t getcallerpc(void *v)
-{
-       return 0;
-}
 
 extern int getfields(char *unused_char_p_t, char **unused_char_pp_t,
                                         int unused_int, int, char *);
index 02f11fd..ae16089 100644 (file)
@@ -331,7 +331,7 @@ void cclose(struct chan *c)
                return;
 
        if (c->flag & CFREE)
-               panic("cclose %p", getcallerpc(&c));
+               panic("double cclose %p.  maybe kfunc channame and hexdump", c);
 
        kref_put(&c->ref);
 }
index 791fa0c..1c41787 100644 (file)
@@ -141,7 +141,7 @@ struct block *padblock(struct block *bp, int size)
 
                PANIC_EXTRA(bp);
                if (bp->next)
-                       panic("padblock %p", getcallerpc(&bp));
+                       panic("%s %p had a next", __func__, bp);
                n = BLEN(bp);
                padblockcnt++;
                nbp = block_alloc(size + n, MEM_WAIT);
@@ -158,7 +158,7 @@ struct block *padblock(struct block *bp, int size)
                PANIC_EXTRA(bp);
 
                if (bp->next)
-                       panic("padblock %p", getcallerpc(&bp));
+                       panic("%s %p had a next", __func__, bp);
 
                if (bp->lim - bp->wp >= size)
                        return bp;