net: tests: Clean up logging of net_buf tests

Change-Id: I8f82c586c730691c72f8996a25843ba786c73fe6
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
This commit is contained in:
Johan Hedberg 2016-06-09 10:21:51 +03:00
parent bab34ea49c
commit c78a5a9aae
2 changed files with 19 additions and 11 deletions

View file

@ -1,2 +1,2 @@
CONFIG_NET_BUF=y CONFIG_NET_BUF=y
CONFIG_NET_BUF_DEBUG=y #CONFIG_NET_BUF_DEBUG=y

View file

@ -23,6 +23,12 @@
#include <net/buf.h> #include <net/buf.h>
#if defined(CONFIG_NET_BUF_DEBUG)
#define DBG(fmt, ...) printk(fmt, ##__VA_ARGS__)
#else
#define DBG(fmt, ...)
#endif
#define TEST_TIMEOUT SECONDS(1) #define TEST_TIMEOUT SECONDS(1)
struct bt_data { struct bt_data {
@ -48,7 +54,7 @@ static void buf_destroy(struct net_buf *buf)
printk("Invalid free pointer in buffer!\n"); printk("Invalid free pointer in buffer!\n");
} }
printk("destroying %p\n", buf); DBG("destroying %p\n", buf);
nano_fifo_put(buf->free, buf); nano_fifo_put(buf->free, buf);
} }
@ -97,7 +103,7 @@ static bool net_buf_test_2(void)
return false; return false;
} }
printk("Fragment list head %p\n", head); DBG("Fragment list head %p\n", head);
frag = head; frag = head;
for (i = 0; i < ARRAY_SIZE(bufs_pool) - 1; i++) { for (i = 0; i < ARRAY_SIZE(bufs_pool) - 1; i++) {
@ -106,11 +112,11 @@ static bool net_buf_test_2(void)
printk("Failed to get fragment!\n"); printk("Failed to get fragment!\n");
return false; return false;
} }
printk("%p -> %p\n", frag, frag->frags); DBG("%p -> %p\n", frag, frag->frags);
frag = frag->frags; frag = frag->frags;
} }
printk("%p -> %p\n", frag, frag->frags); DBG("%p -> %p\n", frag, frag->frags);
nano_fifo_init(&fifo); nano_fifo_init(&fifo);
net_buf_put(&fifo, head); net_buf_put(&fifo, head);
@ -142,7 +148,7 @@ static void test_3_fiber(int arg1, int arg2)
return; return;
} }
printk("Got buffer from FIFO\n"); DBG("Got buffer %p from FIFO\n", buf);
destroy_called = 0; destroy_called = 0;
net_buf_unref(buf); net_buf_unref(buf);
@ -170,7 +176,7 @@ static bool net_buf_test_3(void)
return false; return false;
} }
printk("Fragment list head %p\n", head); DBG("Fragment list head %p\n", head);
frag = head; frag = head;
for (i = 0; i < ARRAY_SIZE(bufs_pool) - 1; i++) { for (i = 0; i < ARRAY_SIZE(bufs_pool) - 1; i++) {
@ -179,11 +185,11 @@ static bool net_buf_test_3(void)
printk("Failed to get fragment!\n"); printk("Failed to get fragment!\n");
return false; return false;
} }
printk("%p -> %p\n", frag, frag->frags); DBG("%p -> %p\n", frag, frag->frags);
frag = frag->frags; frag = frag->frags;
} }
printk("%p -> %p\n", frag, frag->frags); DBG("%p -> %p\n", frag, frag->frags);
nano_fifo_init(&fifo); nano_fifo_init(&fifo);
nano_sem_init(&sema); nano_sem_init(&sema);
@ -196,7 +202,7 @@ static bool net_buf_test_3(void)
return false; return false;
} }
printk("calling net_buf_put\n"); DBG("calling net_buf_put\n");
net_buf_put(&fifo, head); net_buf_put(&fifo, head);
@ -227,9 +233,11 @@ void main(void)
net_buf_pool_init(bufs_pool); net_buf_pool_init(bufs_pool);
for (count = 0, pass = 0; count < ARRAY_SIZE(tests); count++) { for (count = 0, pass = 0; count < ARRAY_SIZE(tests); count++) {
printk("Running %s... ", tests[count].name);
if (!tests[count].func()) { if (!tests[count].func()) {
printk("%s failed!\n", tests[count].name); printk("failed!\n");
} else { } else {
printk("passed!\n");
pass++; pass++;
} }
} }