summaryrefslogtreecommitdiff
path: root/net/phonet
diff options
context:
space:
mode:
authorHoratiu Vultur <horatiu.vultur@microchip.com>2024-02-19 09:00:43 +0100
committerJakub Kicinski <kuba@kernel.org>2024-02-21 17:12:43 -0800
commit603ead96582d85903baec2d55f021b8dac5c25d2 (patch)
tree7de91c8e7d66f40bb2577fd45381cf5f9964b1ed /net/phonet
parent1fde0ca3a0de7e9f917668941156959dd5e9108b (diff)
net: sparx5: Add spinlock for frame transmission from CPU
Both registers used when doing manual injection or fdma injection are shared between all the net devices of the switch. It was noticed that when having two process which each of them trying to inject frames on different ethernet ports, that the HW started to behave strange, by sending out more frames then expected. When doing fdma injection it is required to set the frame in the DCB and then make sure that the next pointer of the last DCB is invalid. But because there is no locks for this, then easily this pointer between the DCB can be broken and then it would create a loop of DCBs. And that means that the HW will continuously transmit these frames in a loop. Until the SW will break this loop. Therefore to fix this issue, add a spin lock for when accessing the registers for manual or fdma injection. Signed-off-by: Horatiu Vultur <horatiu.vultur@microchip.com> Reviewed-by: Daniel Machon <daniel.machon@microchip.com> Fixes: f3cad2611a77 ("net: sparx5: add hostmode with phylink support") Link: https://lore.kernel.org/r/20240219080043.1561014-1-horatiu.vultur@microchip.com Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'net/phonet')
0 files changed, 0 insertions, 0 deletions