Clean up aliased monitor commands
authorBarret Rhoden <brho@cs.berkeley.edu>
Mon, 26 Oct 2015 21:51:45 +0000 (17:51 -0400)
committerBarret Rhoden <brho@cs.berkeley.edu>
Wed, 28 Oct 2015 16:05:14 +0000 (12:05 -0400)
The better way to alias a monitor command is to just point the aliased
entry at the real function, not to implement the function a second time.

Signed-off-by: Barret Rhoden <brho@cs.berkeley.edu>
kern/include/monitor.h
kern/src/monitor.c

index 5c62a67..6e95b47 100644 (file)
@@ -13,7 +13,6 @@ void monitor(struct hw_trapframe *hw_tf);
 int mon_help(int argc, char **argv, struct hw_trapframe *hw_tf);
 int mon_kerninfo(int argc, char **argv, struct hw_trapframe *hw_tf);
 int mon_backtrace(int argc, char **argv, struct hw_trapframe *hw_tf);
-int mon_bt(int argc, char **argv, struct hw_trapframe *hw_tf);
 int mon_ps(int argc, char **argv, struct hw_trapframe *hw_tf);
 int mon_reboot(int argc, char **argv, struct hw_trapframe *hw_tf);
 int mon_showmapping(int argc, char **argv, struct hw_trapframe *hw_tf);
index c44122a..3283d8b 100644 (file)
@@ -41,7 +41,7 @@ static command_t commands[] = {
        { "help", "Display this list of commands", mon_help },
        { "kerninfo", "Display information about the kernel", mon_kerninfo },
        { "backtrace", "Dump a backtrace", mon_backtrace },
-       { "bt", "Dump a backtrace", mon_bt },
+       { "bt", "Dump a backtrace", mon_backtrace },
        { "reboot", "Take a ride to the South Bay", mon_reboot },
        { "showmapping", "Shows VA->PA mappings", mon_showmapping},
        { "sm", "Shows VA->PA mappings", mon_sm},
@@ -128,11 +128,6 @@ int mon_backtrace(int argc, char **argv, struct hw_trapframe *hw_tf)
        return __backtrace(argc, argv, hw_tf);
 }
 
-int mon_bt(int argc, char **argv, struct hw_trapframe *hw_tf)
-{
-       return __backtrace(argc, argv, hw_tf);
-}
-
 int mon_reboot(int argc, char **argv, struct hw_trapframe *hw_tf)
 {
        cprintf("[Scottish Accent]: She's goin' down, Cap'n!\n");