All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Cadence: gem: fix wraparound in 64bit descriptors
@ 2020-04-16  9:02 Ramon Fried
  2020-04-16  9:39   ` no-reply
                   ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: Ramon Fried @ 2020-04-16  9:02 UTC (permalink / raw)
  To: edgar.iglesias, alistair, peter.maydell, jasowang, qemu-arm
  Cc: Ramon Fried, qemu-devel

Wraparound of TX descriptor cyclic buffer only updated
the low 32 bits of the descriptor.
Fix that by checking if we're working with 64bit descriptors.

Signed-off-by: Ramon Fried <rfried.dev@gmail.com>
---
 hw/net/cadence_gem.c | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/hw/net/cadence_gem.c b/hw/net/cadence_gem.c
index 51ec5a072d..b8ae21cc0d 100644
--- a/hw/net/cadence_gem.c
+++ b/hw/net/cadence_gem.c
@@ -1238,7 +1238,14 @@ static void gem_transmit(CadenceGEMState *s)
             /* read next descriptor */
             if (tx_desc_get_wrap(desc)) {
                 tx_desc_set_last(desc);
-                packet_desc_addr = s->regs[GEM_TXQBASE];
+
+            if (s->regs[GEM_DMACFG] & GEM_DMACFG_ADDR_64B) {
+                packet_desc_addr = s->regs[GEM_TBQPH];
+                packet_desc_addr <<= 32;
+            } else {
+                packet_desc_addr = 0;
+            }
+                packet_desc_addr |= s->regs[GEM_TXQBASE];
             } else {
                 packet_desc_addr += 4 * gem_get_desc_len(s, false);
             }
-- 
2.26.0


^ permalink raw reply related	[flat|nested] 9+ messages in thread
end of thread, other threads:[~2020-04-17 17:18 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-04-16  9:02 [PATCH] Cadence: gem: fix wraparound in 64bit descriptors Ramon Fried
2020-04-16  9:39 ` no-reply
2020-04-16  9:39   ` no-reply
2020-04-16 11:09   ` Philippe Mathieu-Daudé
2020-04-17 13:37 ` Peter Maydell
2020-04-17 13:37   ` Peter Maydell
2020-04-17 17:16   ` Ramon Fried
2020-04-17 17:16     ` Ramon Fried
2020-04-17 15:46 ` Edgar E. Iglesias

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.