net: dwc_eth_qos: Fix hang when freeing packet after stop
authorSamuel Holland <samuel.holland@sifive.com>
Mon, 14 Apr 2025 19:30:11 +0000 (12:30 -0700)
committerTom Rini <trini@konsulko.com>
Thu, 24 Apr 2025 14:22:59 +0000 (08:22 -0600)
commita948f5711844498421432719c4c19d9629cb9c05
tree019551fc13f12742433c6c250f49cd13fbe15e0c
parent233fda6af674736dbc6ff37a9ef003b9fa4b8074
net: dwc_eth_qos: Fix hang when freeing packet after stop

If eqos_free_pkt() is called after eqos_stop(), eqos_stop_resets() will
have been called already. This may prevent accessing the MMIO space to
update the RX descriptor tail pointer, so we must skip the descriptor
maintenance logic. This is okay because the descriptors and tail pointer
will all be rewritten anyway during the next call to eqos_start().

This hang was observed after a failed TFTP transaction:

  eqos_recv(dev=000000047fb57330, flags=1):
  eqos_recv: *packetp=000000c3ffb5c080, length=151

  TFTP error: 'file <FILE> not found for <IP>' (1)
  Not retrying...
  eqos_stop(dev=000000047fb57330):
  eqos_stop: OK
  eqos_free_pkt(packet=000000c3ffb5c080, length=151)
  <HANG>

Fixes: ba4dfef1469f ("net: add driver for Synopsys Ethernet QoS device")
Signed-off-by: Samuel Holland <samuel.holland@sifive.com>
drivers/net/dwc_eth_qos.c