Linux: print after BUG() is pretty useless
authorSimon Wilkinson <sxw@your-file-system.com>
Sun, 12 Sep 2010 11:07:59 +0000 (12:07 +0100)
committerDerrick Brashear <shadow@dementia.org>
Mon, 20 Sep 2010 06:44:40 +0000 (23:44 -0700)
Calling BUG() panics the machine, so having a print statement with
further debugging information after it doesn't help much.

Swap the ordering, so we at least see the message.

Change-Id: I3aa3b8ba5faaa40cef7bb046093b322212eea2b8
Reviewed-on: http://gerrit.openafs.org/2754
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>

src/afs/LINUX/osi_alloc.c
src/afs/LINUX24/osi_alloc.c

index b586d7b..b10bbab 100644 (file)
@@ -347,8 +347,8 @@ osi_linux_free(void *addr)
        afs_atomlist_put(al_mem_pool, lmp);     /* return osi_linux_mem struct to pool */
        afs_linux_cur_allocs--;
     } else {
-       BUG();
        printf("osi_linux_free: failed to remove chunk from hashtable\n");
+       BUG();
     }
 
     up(&afs_linux_alloc_sem);
index 30a69a5..d4f91cb 100644 (file)
@@ -364,8 +364,8 @@ osi_linux_free(void *addr)
        afs_atomlist_put(al_mem_pool, lmp);     /* return osi_linux_mem struct to pool */
        afs_linux_cur_allocs--;
     } else {
-       BUG();
        printf("osi_linux_free: failed to remove chunk from hashtable\n");
+       BUG();
     }
 
     up(&afs_linux_alloc_sem);