summary refs log tree commit diff stats
path: root/hw/net/npcm_gmac.c
diff options
context:
space:
mode:
authorNabih Estefan <nabihestefan@google.com>2024-11-05 10:10:00 +0000
committerPeter Maydell <peter.maydell@linaro.org>2024-11-05 10:10:00 +0000
commitab4b56d981d753eb2ecc610d2b5d5689716aae9c (patch)
tree9324b7578f521484c2b5205e1037a5936533a569 /hw/net/npcm_gmac.c
parent3647dca9fbf4fb267c401cdd94519b7a60af0b0b (diff)
downloadfocaccia-qemu-ab4b56d981d753eb2ecc610d2b5d5689716aae9c.tar.gz
focaccia-qemu-ab4b56d981d753eb2ecc610d2b5d5689716aae9c.zip
hw/net/npcm_gmac: Change error log to trace event
Convert the LOG_GUEST_ERROR for the "tx descriptor is owned
by software" to a trace message. This condition is normal
when there is there is nothing to transmit, and we would
otherwise spam the logs with it in that situation.

Signed-off-by: Nabih Estefan <nabihestefan@google.com>
Signed-off-by: Roque Arcudia Hernandez <roqueh@google.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-id: 20241014184847.1594056-1-roqueh@google.com
[PMM: tweaked commit message]
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Diffstat (limited to 'hw/net/npcm_gmac.c')
-rw-r--r--hw/net/npcm_gmac.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/hw/net/npcm_gmac.c b/hw/net/npcm_gmac.c
index 6fa6bece61..685905f9e2 100644
--- a/hw/net/npcm_gmac.c
+++ b/hw/net/npcm_gmac.c
@@ -546,9 +546,8 @@ static void gmac_try_send_next_packet(NPCMGMACState *gmac)
 
         /* 1 = DMA Owned, 0 = Software Owned */
         if (!(tx_desc.tdes0 & TX_DESC_TDES0_OWN)) {
-            qemu_log_mask(LOG_GUEST_ERROR,
-                          "TX Descriptor @ 0x%x is owned by software\n",
-                          desc_addr);
+            trace_npcm_gmac_tx_desc_owner(DEVICE(gmac)->canonical_path,
+                                          desc_addr);
             gmac->regs[R_NPCM_DMA_STATUS] |= NPCM_DMA_STATUS_TU;
             gmac_dma_set_state(gmac, NPCM_DMA_STATUS_TX_PROCESS_STATE_SHIFT,
                 NPCM_DMA_STATUS_TX_SUSPENDED_STATE);