slirp: Cleanup and basic reanimation of debug code

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>

diff --git a/Makefile b/Makefile
index f8dc5065dd27010bfdbb6bcfb0c6e3af25024cdb..8a03d8719669d04965d9e183659844bb272e1110 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -190,7 +190,7 @@ ifdef CONFIG_SLIRP
 CPPFLAGS+=-I$(SRC_PATH)/slirp
 SLIRP_OBJS=cksum.o if.o ip_icmp.o ip_input.o ip_output.o \
 slirp.o mbuf.o misc.o sbuf.o socket.o tcp_input.o tcp_output.o \
-tcp_subr.o tcp_timer.o udp.o bootp.o debug.o tftp.o
+tcp_subr.o tcp_timer.o udp.o bootp.o tftp.o
 OBJS+=$(addprefix slirp/, $(SLIRP_OBJS))
 endif
 

diff --git a/slirp/bootp.c b/slirp/bootp.c
index 97c2811fbf38cd04d7c468e8ec6b2eb307ca21c0..fec52c0a84a98565eac415ef917629d634700ed0 100644 (file)
--- a/slirp/bootp.c
+++ b/slirp/bootp.c
@@ -42,7 +42,7 @@ static const uint8_t rfc1533_cookie[] = { RFC1533_COOKIE };
 
 #ifdef DEBUG
 #define dprintf(fmt, ...) \
-if (slirp_debug & DBG_CALL) { fprintf(dfd, fmt, ##  __VA_ARGS__); fflush(dfd); }
+do if (slirp_debug & DBG_CALL) { fprintf(dfd, fmt, ##  __VA_ARGS__); fflush(dfd); } while (0)
 #else
 #define dprintf(fmt, ...)
 #endif

diff --git a/slirp/debug.c b/slirp/debug.c
deleted file mode 100644 (file)
index ad02df7..0000000
--- a/slirp/debug.c
+++ /dev/null
@@ -1,12 +0,0 @@
-/*
- * Copyright (c) 1995 Danny Gasparovski.
- * Portions copyright (c) 2000 Kelly Price.
- *
- * Please read the file COPYRIGHT for the
- * terms and conditions of the copyright.
- */
-
-#include <slirp.h>
-
-FILE *dfd = NULL;
-int slirp_debug = 0;

diff --git a/slirp/debug.h b/slirp/debug.h
index 963ffda0728b83cd24a0413693c9977ff98ac0ce..6cfa61edb32ff30c688eee7b18cb3a24f80933e3 100644 (file)
--- a/slirp/debug.h
+++ b/slirp/debug.h
@@ -5,25 +5,24 @@
  * terms and conditions of the copyright.
  */
 
-#define PRN_STDERR     1
-#define PRN_SPRINTF    2
+//#define DEBUG 1
 
-extern FILE *dfd;
-extern int slirp_debug;
+#ifdef DEBUG
 
 #define DBG_CALL 0x1
 #define DBG_MISC 0x2
 #define DBG_ERROR 0x4
-#define DEBUG_DEFAULT DBG_CALL|DBG_MISC|DBG_ERROR
 
-#ifdef DEBUG
+#define dfd stderr
+
+extern int slirp_debug;
+
 #define DEBUG_CALL(x) if (slirp_debug & DBG_CALL) { fprintf(dfd, "%s...\n", x); fflush(dfd); }
 #define DEBUG_ARG(x, y) if (slirp_debug & DBG_CALL) { fputc(' ', dfd); fprintf(dfd, x, y); fputc('\n', dfd); fflush(dfd); }
 #define DEBUG_ARGS(x) if (slirp_debug & DBG_CALL) { fprintf x ; fflush(dfd); }
 #define DEBUG_MISC(x) if (slirp_debug & DBG_MISC) { fprintf x ; fflush(dfd); }
 #define DEBUG_ERROR(x) if (slirp_debug & DBG_ERROR) {fprintf x ; fflush(dfd); }
 
-
 #else
 
 #define DEBUG_CALL(x)
@@ -33,5 +32,3 @@ extern int slirp_debug;
 #define DEBUG_ERROR(x)
 
 #endif
-
-void debug_init _P((char *, int));

diff --git a/slirp/misc.c b/slirp/misc.c
index 958711298ebd351c11728221e03ae410c399ffe8..ae6f2114907e09a8a8009c62d7c7900ed8711afb 100644 (file)
--- a/slirp/misc.c
+++ b/slirp/misc.c
@@ -10,6 +10,10 @@
 
 #include "monitor.h"
 
+#ifdef DEBUG
+int slirp_debug = DBG_CALL|DBG_MISC|DBG_ERROR;
+#endif
+
 u_int curtime, time_fasttimo, last_slowtimo;
 
 /*

diff --git a/slirp/slirp.h b/slirp/slirp.h
index c30858275e4e8312cc742157268534b65fe2bab4..1bd7c89b419d6fbdaf49abc0dc9129991f65a964 100644 (file)
--- a/slirp/slirp.h
+++ b/slirp/slirp.h
@@ -1,8 +1,6 @@
 #ifndef __COMMON_H__
 #define __COMMON_H__
 
-//#define DEBUG 1
-
 #include "config-host.h"
 #include "slirp_config.h"
 

diff --git a/slirp/socket.c b/slirp/socket.c
index a911b29d901d9ed4a516969365a45dccec44a52d..f7b212fba8aad0963e369491dd7a1b3386037ea7 100644 (file)
--- a/slirp/socket.c
+++ b/slirp/socket.c
@@ -584,7 +584,8 @@ tcp_listen(u_int32_t haddr, u_int hport, u_int32_t laddr, u_int lport, int flags
        socklen_t addrlen = sizeof(addr);
 
        DEBUG_CALL("tcp_listen");
-       DEBUG_ARG("port = %d", port);
+       DEBUG_ARG("haddr = %x", haddr);
+       DEBUG_ARG("hport = %d", hport);
        DEBUG_ARG("laddr = %x", laddr);
        DEBUG_ARG("lport = %d", lport);
        DEBUG_ARG("flags = %x", flags);