On Tue, Nov 12, 2024 at 07:10:26PM +0100, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Edgar E. Iglesias <edgar.iglesias@amd.com>
> ---
> hw/net/xilinx_ethlite.c | 5 +++--
> hw/net/trace-events | 4 ++++
> 2 files changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/hw/net/xilinx_ethlite.c b/hw/net/xilinx_ethlite.c
> index e84b4cdd35..bb330a233f 100644
> --- a/hw/net/xilinx_ethlite.c
> +++ b/hw/net/xilinx_ethlite.c
> @@ -30,6 +30,7 @@
> #include "hw/irq.h"
> #include "hw/qdev-properties.h"
> #include "net/net.h"
> +#include "trace.h"
>
> #define D(x)
> #define R_TX_BUF0 0
> @@ -198,13 +199,13 @@ static ssize_t eth_rx(NetClientState *nc, const uint8_t *buf, size_t size)
> return size;
>
> if (s->regs[rxbase + R_RX_CTRL0] & CTRL_S) {
> - D(qemu_log("ethlite lost packet %x\n", s->regs[R_RX_CTRL0]));
> + trace_ethlite_pkt_lost(s->regs[R_RX_CTRL0]);
> return -1;
> }
>
> D(qemu_log("%s %zd rxbase=%x\n", __func__, size, rxbase));
> if (size > (R_MAX - R_RX_BUF0 - rxbase) * 4) {
> - D(qemu_log("ethlite packet is too big, size=%x\n", size));
> + trace_ethlite_pkt_size_too_big(size);
> return -1;
> }
> memcpy(&s->regs[rxbase + R_RX_BUF0], buf, size);
> diff --git a/hw/net/trace-events b/hw/net/trace-events
> index d0f1d8c0fb..2b36cd967e 100644
> --- a/hw/net/trace-events
> +++ b/hw/net/trace-events
> @@ -511,3 +511,7 @@ xen_netdev_connect(int dev, unsigned int tx, unsigned int rx, int port) "vif%u t
> xen_netdev_frontend_changed(const char *dev, int state) "vif%s state %d"
> xen_netdev_tx(int dev, int ref, int off, int len, unsigned int flags, const char *c, const char *d, const char *m, const char *e) "vif%u ref %u off %u len %u flags 0x%x%s%s%s%s"
> xen_netdev_rx(int dev, int idx, int status, int flags) "vif%u idx %d status %d flags 0x%x"
> +
> +# xilinx_ethlite.c
> +ethlite_pkt_lost(uint32_t rx_ctrl) "rx_ctrl:0x%" PRIx32
> +ethlite_pkt_size_too_big(uint64_t size) "size:0x%" PRIx64
> --
> 2.45.2
>