[Qemu-devel] [PATCH RFC 1/3] tests/vhost-user-bridge: disable debug output by default

Jens Freimann posted 3 patches 8 years, 7 months ago
[Qemu-devel] [PATCH RFC 1/3] tests/vhost-user-bridge: disable debug output by default
Posted by Jens Freimann 8 years, 7 months ago
From: Jens Freimann <jfreiman@redhat.com>

vhost-user-bridge prints out a lot of information, including dumps
of all transmitted data. When called from a testcase this output
clutters the actual test results, so let's make the default no debug
output.

Signed-off-by: Jens Freimann <jfreiman@redhat.com>
---
 tests/vhost-user-bridge.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/vhost-user-bridge.c b/tests/vhost-user-bridge.c
index 1e5b5ca..93d9535 100644
--- a/tests/vhost-user-bridge.c
+++ b/tests/vhost-user-bridge.c
@@ -34,7 +34,7 @@
 #include "standard-headers/linux/virtio_net.h"
 #include "contrib/libvhost-user/libvhost-user.h"
 
-#define VHOST_USER_BRIDGE_DEBUG 1
+#define VHOST_USER_BRIDGE_DEBUG 0
 
 #define DPRINT(...) \
     do { \
-- 
2.9.4


Re: [Qemu-devel] [PATCH RFC 1/3] tests/vhost-user-bridge: disable debug output by default
Posted by Maxime Coquelin 8 years, 7 months ago

On 07/12/2017 11:41 AM, Jens Freimann wrote:
> From: Jens Freimann <jfreiman@redhat.com>
> 
> vhost-user-bridge prints out a lot of information, including dumps
> of all transmitted data. When called from a testcase this output
> clutters the actual test results, so let's make the default no debug
> output.
> 
> Signed-off-by: Jens Freimann <jfreiman@redhat.com>
> ---
>   tests/vhost-user-bridge.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Makes sense to have debug logs disabled by default.

Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>

Thanks,
Maxime

> diff --git a/tests/vhost-user-bridge.c b/tests/vhost-user-bridge.c
> index 1e5b5ca..93d9535 100644
> --- a/tests/vhost-user-bridge.c
> +++ b/tests/vhost-user-bridge.c
> @@ -34,7 +34,7 @@
>   #include "standard-headers/linux/virtio_net.h"
>   #include "contrib/libvhost-user/libvhost-user.h"
>   
> -#define VHOST_USER_BRIDGE_DEBUG 1
> +#define VHOST_USER_BRIDGE_DEBUG 0
>   
>   #define DPRINT(...) \
>       do { \
>