nv-tegra-mirror/linux-nv-oot
1
0
Fork 0
mirror of git://nv-tegra.nvidia.com/linux-nv-oot.git synced 2025-12-22 17:25:35 +03:00
Code Issues Packages Projects Releases Wiki Activity

drivers: mttcan: fix bus-off restart txfer

Browse Source 
[Issue]: CAN freezes/stops to send messages after restart from bus-off state.
Throws following log from kernel: "write: No buffer space available"

[Reason]: When message txfer starts, tx_object (which keeps track of active tx)
gets filled. If CAN goes to bus-off state, txfer remains incomplete for some
messages. In such case, tx_object bits will not get cleared. It will stop
adding more messages in controller RAM.

Along with tx_object, from network layer, there are socket echo buffers.
When CAN is initialized and up on network, netif_start_queue is pushed to start
transmission. When msg txfer starts, socket buffer gets filled and freed only
when txfer completes. During bus-off, since network queue remains ON, all the
queued msgs get filled in socket buffers and does not allow upcoming msgs.
Therefore we see "write: No buffer space available".

[Fix]: Clear tx_object when device goes to bus-off state and stop network queue.
Start network queue again during restart from bus-off.

Bug 4438223

Change-Id: I3cbc6529a90f357372c8b0095bdce4217b133e9b
Signed-off-by: Shubhi Garg <shgarg@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nv-oot/+/3142091
GVS: buildbot_gerritrpt <buildbot_gerritrpt@nvidia.com>
Reviewed-by: Bibek Basu <bbasu@nvidia.com>
This commit is contained in:
Shubhi Garg
2024-05-21 14:23:13 +05:30
committed by mobile promotions
parent d95c0d0add
commit f902d95962
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
drivers/net/can/mttcan/native/m_ttcan_linux.c
View File

@@ -1,6 +1,6 @@
/* SPDX-License-Identifier: GPL-2.0-only */ /* SPDX-License-Identifier: GPL-2.0-only */
/* /*
* Copyright (c) 2022-2023, NVIDIA CORPORATION & AFFILIATES. All rights reserved. * Copyright (c) 2022-2024, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
*/ */
#include <nvidia/conftest.h> #include <nvidia/conftest.h>
@@ -505,7 +505,8 @@ static int mttcan_state_change(struct net_device *dev,
*/ */
ttcan_set_intrpts(priv->ttcan, 0); ttcan_set_intrpts(priv->ttcan, 0);
priv->can.can_stats.bus_off++; priv->can.can_stats.bus_off++;
priv->ttcan->tx_object = 0;
netif_stop_queue(dev);
netif_carrier_off(dev); netif_carrier_off(dev);
if (priv->can.restart_ms) if (priv->can.restart_ms)
@@ -1114,6 +1115,7 @@ restart:
priv->can.can_stats.restarts++; priv->can.can_stats.restarts++;
mttcan_start(dev); mttcan_start(dev);
netif_start_queue(dev);
netif_carrier_on(dev); netif_carrier_on(dev);
} }