You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
72 lines
2.5 KiB
72 lines
2.5 KiB
![]()
8 years ago
|
From dd08d1ac4cfc86fbea5ee207b9615922ede88ec6 Mon Sep 17 00:00:00 2001
|
||
|
From: John Crispin <john@phrozen.org>
|
||
|
Date: Tue, 17 May 2016 06:01:45 +0200
|
||
|
Subject: [PATCH 095/102] net-next: mediatek: add IRQ locking
|
||
![]()
8 years ago
|
|
||
|
The code that enables and disables IRQs is missing proper locking. After
|
||
|
adding the IRQ separation patch and routing the putting the RX and TX IRQs
|
||
|
on different cores we experienced IRQ stalls. Fix this by adding proper
|
||
|
locking. We use a dedicated lock to reduce the latency if the IRQ code.
|
||
|
Otherwise it might wait for bottom code to finish before reenabling or
|
||
|
disabling IRQs.
|
||
|
|
||
![]()
8 years ago
|
Signed-off-by: Sean Wang <keyhaede@gmail.com>
|
||
|
Signed-off-by: John Crispin <john@phrozen.org>
|
||
![]()
8 years ago
|
---
|
||
|
drivers/net/ethernet/mediatek/mtk_eth_soc.c | 7 +++++++
|
||
|
drivers/net/ethernet/mediatek/mtk_eth_soc.h | 1 +
|
||
|
2 files changed, 8 insertions(+)
|
||
|
|
||
![]()
8 years ago
|
diff --git a/drivers/net/ethernet/mediatek/mtk_eth_soc.c b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
|
||
|
index 13ee15f..c869064 100644
|
||
![]()
8 years ago
|
--- a/drivers/net/ethernet/mediatek/mtk_eth_soc.c
|
||
|
+++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
|
||
![]()
8 years ago
|
@@ -331,18 +331,24 @@ static void mtk_mdio_cleanup(struct mtk_eth *eth)
|
||
![]()
8 years ago
|
|
||
|
static inline void mtk_irq_disable(struct mtk_eth *eth, u32 mask)
|
||
|
{
|
||
|
+ unsigned long flags;
|
||
|
u32 val;
|
||
|
|
||
|
+ spin_lock_irqsave(ð->irq_lock, flags);
|
||
|
val = mtk_r32(eth, MTK_QDMA_INT_MASK);
|
||
|
mtk_w32(eth, val & ~mask, MTK_QDMA_INT_MASK);
|
||
|
+ spin_unlock_irqrestore(ð->irq_lock, flags);
|
||
|
}
|
||
|
|
||
|
static inline void mtk_irq_enable(struct mtk_eth *eth, u32 mask)
|
||
|
{
|
||
|
+ unsigned long flags;
|
||
|
u32 val;
|
||
|
|
||
|
+ spin_lock_irqsave(ð->irq_lock, flags);
|
||
|
val = mtk_r32(eth, MTK_QDMA_INT_MASK);
|
||
|
mtk_w32(eth, val | mask, MTK_QDMA_INT_MASK);
|
||
|
+ spin_unlock_irqrestore(ð->irq_lock, flags);
|
||
|
}
|
||
|
|
||
|
static int mtk_set_mac_address(struct net_device *dev, void *p)
|
||
![]()
8 years ago
|
@@ -1771,6 +1777,7 @@ static int mtk_probe(struct platform_device *pdev)
|
||
![]()
8 years ago
|
return -EADDRNOTAVAIL;
|
||
|
|
||
|
spin_lock_init(ð->page_lock);
|
||
|
+ spin_lock_init(ð->irq_lock);
|
||
|
|
||
|
eth->ethsys = syscon_regmap_lookup_by_phandle(pdev->dev.of_node,
|
||
|
"mediatek,ethsys");
|
||
![]()
8 years ago
|
diff --git a/drivers/net/ethernet/mediatek/mtk_eth_soc.h b/drivers/net/ethernet/mediatek/mtk_eth_soc.h
|
||
|
index a5eb7c6..3159d2a 100644
|
||
![]()
8 years ago
|
--- a/drivers/net/ethernet/mediatek/mtk_eth_soc.h
|
||
|
+++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.h
|
||
![]()
8 years ago
|
@@ -373,6 +373,7 @@ struct mtk_eth {
|
||
![]()
8 years ago
|
void __iomem *base;
|
||
|
struct reset_control *rstc;
|
||
|
spinlock_t page_lock;
|
||
|
+ spinlock_t irq_lock;
|
||
|
struct net_device dummy_dev;
|
||
|
struct net_device *netdev[MTK_MAX_DEVS];
|
||
|
struct mtk_mac *mac[MTK_MAX_DEVS];
|
||
![]()
8 years ago
|
--
|
||
|
1.7.10.4
|
||
|
|