On Tue, Jan 13, 2026 at 3:03 AM Yodel Eldar <[email protected]> wrote:
>
> vhost-user-bridge debug prints UDP TX hexdumps in its transmit handler,
> but does not for receives, even though they are beneficial for testing.
> Add an RX hexdump in the receive callback.
>
> To delineate between transmits and receives, also add a debug print
> indicating that the program is in the transmit handler.
>
> Signed-off-by: Yodel Eldar <[email protected]>

Reviewed-by: Marc-André Lureau <[email protected]>

> ---
>  contrib/vhost-user-bridge/vhost-user-bridge.c | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/contrib/vhost-user-bridge/vhost-user-bridge.c 
> b/contrib/vhost-user-bridge/vhost-user-bridge.c
> index ce4c3426d3..3f0fd0fa49 100644
> --- a/contrib/vhost-user-bridge/vhost-user-bridge.c
> +++ b/contrib/vhost-user-bridge/vhost-user-bridge.c
> @@ -179,6 +179,9 @@ vubr_handle_tx(VuDev *dev, int qidx)
>
>      assert(qidx % 2);
>
> +    DPRINT("\n\n   ***   IN UDP TRANSMIT HANDLER    ***\n\n");
> +    DPRINT("    hdrlen = %d\n", hdrlen);
> +
>      for (;;) {
>          ssize_t ret;
>          unsigned int out_num;
> @@ -333,6 +336,10 @@ vubr_backend_recv_cb(int sock, void *ctx)
>          };
>          ret = RETRY_ON_EINTR(recvmsg(vubr->backend_udp_sock, &msg, 0));
>
> +        if (ret > 0 && VHOST_USER_BRIDGE_DEBUG) {
> +            iov_hexdump(sg, num, stderr, "RX:", ret);
> +        }
> +
>          if (i == 0) {
>              iov_restore_front(elem->in_sg, sg, hdrlen);
>          }
> --
> 2.52.0
>
>


-- 
Marc-André Lureau

Reply via email to