Cleaned up some printking
authorBarret Rhoden <brho@cs.berkeley.edu>
Mon, 29 Sep 2014 04:06:46 +0000 (21:06 -0700)
committerBarret Rhoden <brho@cs.berkeley.edu>
Mon, 29 Sep 2014 04:06:46 +0000 (21:06 -0700)
The ifconfig is a bit tricky.  Somewhere we set errno in the kernel,
probably during the various bind internal calls.  It's probably fine,
but my BB echo hacks thinks it is an error.

The kprof extra printk isn't needed since echo now prints errors.

kern/drivers/dev/kprof.c
kern/kfs/ifconfig
kern/src/oprofile/cpu_buffer.c

index 4418928..edf2f19 100644 (file)
@@ -97,7 +97,6 @@ kprofattach(char *spec)
        kprof.nbuf = (kprof.maxpc-kprof.minpc) >> LRES;
        n = kprof.nbuf*CELLSIZE;
        if(kprof.buf == 0) {
-               printk("Allocate %d bytes\n", n);
                kprof.buf = kzmalloc(n, KMALLOC_WAIT);
                if(kprof.buf == 0)
                        error(Enomem);
@@ -434,7 +433,6 @@ kprofwrite(struct chan *c, void *a, long n, int64_t unused)
                }else if(strncmp(a, "opstop", 6) == 0) {
                        oprofile_control_trace(0);
                } else  {
-                       printk("startclr|start|stop|clear|opstart|opstop");
                        error("startclr|start|stop|clear|opstart|opstop");
                }
                break;
index 7b7d802..4b13d3f 100755 (executable)
@@ -99,7 +99,9 @@ bind -a \#l$NIC /net
 #get an instance of a network stack
 i=`cat /net/ipifc/clone`
 # bind the ether0 to it
-echo "bind ether /net/ether$NIC " > /net/ipifc/$i/ctl
+# the kernel sets errno, though it doesn't return -1 or anything.  but our bb
+# hacks doesn't know any better, and echo will think it was an error
+echo "bind ether /net/ether$NIC " > /net/ipifc/$i/ctl 2> /dev/null
 # ifconfig it
 echo "add $me $mask $remote" > /net/ipifc/$i/ctl
 echo "add 0 0 $route" > /net/iproute
index d55901b..1968dc0 100644 (file)
@@ -145,7 +145,6 @@ int alloc_cpu_buffers(void)
        /* we *really* don't want to block. Losing data is better. */
        qnoblock(opq, 1);
        if (!op_cpu_buffer) {
-               printk("ALlocate %d bytes\n", sizeof(*op_cpu_buffer) * num_cpus);
                op_cpu_buffer =
                        kzmalloc(sizeof(*op_cpu_buffer) * num_cpus, KMALLOC_WAIT);
                if (!op_cpu_buffer)