Remove SEM_TRACE_BLOCKERS and TRACEME
authorBarret Rhoden <brho@cs.berkeley.edu>
Wed, 27 Jul 2016 23:34:02 +0000 (19:34 -0400)
committerBarret Rhoden <brho@cs.berkeley.edu>
Fri, 29 Jul 2016 21:43:07 +0000 (17:43 -0400)
Those are leftovers from the old op2 style tracing.  I noticed this
because trace_printk popped up in an NMI-enabled trace of block_test.
Gotta love the NMI sampling.

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

diff --git a/Kconfig b/Kconfig
index 1414f23..5844844 100644 (file)
--- a/Kconfig
+++ b/Kconfig
@@ -184,14 +184,6 @@ config SEM_SPINWAIT_NR_LOOPS
        help
                How many times to poll a busy semaphore before going to sleep.
 
-config SEM_TRACE_BLOCKERS
-       bool "Semaphore Blocker Tracing"
-       default n
-       help
-               Drops a TRACEME whenever a semaphore would block (barring slight
-               races).  To access the trace data, you'll need to use the opcontrols
-               and process the data with op2.
-
 config LARGE_KSTACKS
        bool "Large (two-page) Kernel Stacks"
        default y
index bf4f474..e16cc7e 100644 (file)
@@ -11,8 +11,6 @@ struct symtab_entry {
        uintptr_t addr;
 };
 
-#define TRACEME() trace_printk(TRUE, "%s(%d)", __FILE__, __LINE__)
-
 /* An alternative here is to have backtrace_list kmalloc an array.  The downside
  * is that we're calling into the allocator in potentially-delicate situations,
  * such as the NMI handler. */
index 61e7ff3..88ac162 100644 (file)
@@ -358,9 +358,6 @@ void sem_down(struct semaphore *sem)
        if (sem_trydown(sem))
                goto block_return_path;
 #endif
-#ifdef CONFIG_SEM_TRACE_BLOCKERS
-       TRACEME();
-#endif
        /* We're probably going to sleep, so get ready.  We'll check again later. */
        kthread = pcpui->cur_kthread;
        /* We need to have a spare slot for restart, so we also use it when