Mirrors_Framework/nuttx
1
0
Fork 0
mirror of https://github.com/apache/nuttx.git synced 2026-05-28 11:56:10 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

arm/imx9: sched_unlock should called after spin_unlock()

Browse Source 
fix deadlock if sched_unlock switch out to other task

Signed-off-by: chao an <anchao.archer@bytedance.com>
This commit is contained in:
chao an
2025-01-14 00:36:58 +08:00
committed by Xiang Xiao
parent b7003e55b9
commit 3d3db4a375
1 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
arch/arm/src/imx9/imx9_flexcan.c
+4 -5
View File
@@ -807,6 +807,7 @@ static int imx9_txpoll(struct net_driver_s *dev)
struct imx9_driver_s *priv = struct imx9_driver_s *priv =
(struct imx9_driver_s *)dev->d_private; (struct imx9_driver_s *)dev->d_private;
irqstate_t flags; irqstate_t flags;
int ret = 0;
/* If the polling resulted in data that should be sent out on the network, /* If the polling resulted in data that should be sent out on the network,
* the field d_len is set to a value > 0. * the field d_len is set to a value > 0.
@@ -829,20 +830,18 @@ static int imx9_txpoll(struct net_driver_s *dev)
if (imx9_txringfull(priv)) if (imx9_txringfull(priv))
{ {
sched_unlock(); ret = -EBUSY;
spin_unlock_irqrestore(&priv->lock, flags);
return -EBUSY;
} }
} }
sched_unlock();
spin_unlock_irqrestore(&priv->lock, flags); spin_unlock_irqrestore(&priv->lock, flags);
sched_unlock();
/* If zero is returned, the polling will continue until all connections /* If zero is returned, the polling will continue until all connections
* have been examined. * have been examined.
*/ */
return 0; return ret;
} }
/**************************************************************************** /****************************************************************************