aboutsummaryrefslogtreecommitdiffstats
path: root/target
diff options
context:
space:
mode:
authorFelix Fietkau <nbd@openwrt.org>2015-01-13 00:34:38 +0000
committerFelix Fietkau <nbd@openwrt.org>2015-01-13 00:34:38 +0000
commit05d4b8c79b6e4d74b7d4b96c1a08fde6c6177e79 (patch)
treef27e3729ffdead8f3d2485bd5637f45ba7ac7a73 /target
parentadaac86c7fb1f81ac602f7eeba8fdd97313dd024 (diff)
downloadupstream-05d4b8c79b6e4d74b7d4b96c1a08fde6c6177e79.tar.gz
upstream-05d4b8c79b6e4d74b7d4b96c1a08fde6c6177e79.tar.bz2
upstream-05d4b8c79b6e4d74b7d4b96c1a08fde6c6177e79.zip
ramips: remove interrupt coalescing, it is unnecessary with napi polling and could reduce throughput
Signed-off-by: Felix Fietkau <nbd@openwrt.org> SVN-Revision: 43952
Diffstat (limited to 'target')
-rw-r--r--target/linux/ramips/files/drivers/net/ethernet/ralink/ralink_ethtool.c50
-rw-r--r--target/linux/ramips/files/drivers/net/ethernet/ralink/ralink_soc_eth.c26
-rw-r--r--target/linux/ramips/files/drivers/net/ethernet/ralink/ralink_soc_eth.h18
-rw-r--r--target/linux/ramips/files/drivers/net/ethernet/ralink/soc_mt7620.c8
-rw-r--r--target/linux/ramips/files/drivers/net/ethernet/ralink/soc_rt2880.c4
-rw-r--r--target/linux/ramips/files/drivers/net/ethernet/ralink/soc_rt305x.c8
-rw-r--r--target/linux/ramips/files/drivers/net/ethernet/ralink/soc_rt3883.c4
7 files changed, 40 insertions, 78 deletions
diff --git a/target/linux/ramips/files/drivers/net/ethernet/ralink/ralink_ethtool.c b/target/linux/ramips/files/drivers/net/ethernet/ralink/ralink_ethtool.c
index b1077bc9f2..c7744ba5c1 100644
--- a/target/linux/ramips/files/drivers/net/ethernet/ralink/ralink_ethtool.c
+++ b/target/linux/ramips/files/drivers/net/ethernet/ralink/ralink_ethtool.c
@@ -140,54 +140,6 @@ static void fe_get_ringparam(struct net_device *dev,
ring->tx_pending = NUM_DMA_DESC;
}
-static int fe_get_coalesce(struct net_device *dev,
- struct ethtool_coalesce *coal)
-{
- u32 delay_cfg = fe_reg_r32(FE_REG_DLY_INT_CFG);
-
- coal->rx_coalesce_usecs = (delay_cfg & 0xff) * FE_DELAY_TIME;
- coal->rx_max_coalesced_frames = ((delay_cfg >> 8) & 0x7f);
- coal->use_adaptive_rx_coalesce = (delay_cfg >> 15) & 0x1;
-
- coal->tx_coalesce_usecs = ((delay_cfg >> 16 )& 0xff) * FE_DELAY_TIME;
- coal->tx_max_coalesced_frames = ((delay_cfg >> 24) & 0x7f);
- coal->use_adaptive_tx_coalesce = (delay_cfg >> 31) & 0x1;
-
- return 0;
-}
-
-static int fe_set_coalesce(struct net_device *dev,
- struct ethtool_coalesce *coal)
-{
- u32 delay_cfg;
- u32 rx_usecs, tx_usecs;
- u32 rx_frames, tx_frames;
-
- if (!coal->use_adaptive_rx_coalesce || !coal->use_adaptive_tx_coalesce)
- return -EINVAL;
-
- rx_usecs = DIV_ROUND_UP(coal->rx_coalesce_usecs, FE_DELAY_TIME);
- rx_frames = coal->rx_max_coalesced_frames;
- tx_usecs = DIV_ROUND_UP(coal->tx_coalesce_usecs, FE_DELAY_TIME);
- tx_frames = coal->tx_max_coalesced_frames;
-
- if (((tx_usecs == 0) && (tx_frames ==0)) ||
- ((rx_usecs == 0) && (rx_frames ==0)))
- return -EINVAL;
-
- if (rx_usecs > 0xff) rx_usecs = 0xff;
- if (rx_frames > 0x7f) rx_frames = 0x7f;
- if (tx_usecs > 0xff) tx_usecs = 0xff;
- if (tx_frames > 0x7f) tx_frames = 0x7f;
-
- delay_cfg = ((((FE_DELAY_EN_INT | tx_frames) << 8) | tx_usecs) << 16) |
- (((FE_DELAY_EN_INT | rx_frames) << 8) | rx_usecs);
-
- fe_reg_w32(delay_cfg, FE_REG_DLY_INT_CFG);
-
- return 0;
-}
-
static void fe_get_strings(struct net_device *dev, u32 stringset, u8 *data)
{
switch (stringset) {
@@ -243,8 +195,6 @@ static struct ethtool_ops fe_ethtool_ops = {
.nway_reset = fe_nway_reset,
.get_link = fe_get_link,
.get_ringparam = fe_get_ringparam,
- .get_coalesce = fe_get_coalesce,
- .set_coalesce = fe_set_coalesce,
};
void fe_set_ethtool_ops(struct net_device *netdev)
diff --git a/target/linux/ramips/files/drivers/net/ethernet/ralink/ralink_soc_eth.c b/target/linux/ramips/files/drivers/net/ethernet/ralink/ralink_soc_eth.c
index 068366a7c6..67934c61a0 100644
--- a/target/linux/ramips/files/drivers/net/ethernet/ralink/ralink_soc_eth.c
+++ b/target/linux/ramips/files/drivers/net/ethernet/ralink/ralink_soc_eth.c
@@ -842,8 +842,8 @@ static int fe_poll(struct napi_struct *napi, int budget)
u32 tx_intr, rx_intr;
status = fe_reg_r32(FE_REG_FE_INT_STATUS);
- tx_intr = priv->soc->tx_dly_int;
- rx_intr = priv->soc->rx_dly_int;
+ tx_intr = priv->soc->tx_int;
+ rx_intr = priv->soc->rx_int;
tx_done = rx_done = 0;
poll_again:
@@ -907,16 +907,16 @@ static void fe_tx_timeout(struct net_device *dev)
static irqreturn_t fe_handle_irq(int irq, void *dev)
{
struct fe_priv *priv = netdev_priv(dev);
- u32 status, dly_int;
+ u32 status, int_mask;
status = fe_reg_r32(FE_REG_FE_INT_STATUS);
if (unlikely(!status))
return IRQ_NONE;
- dly_int = (priv->soc->rx_dly_int | priv->soc->tx_dly_int);
- if (likely(status & dly_int)) {
- fe_int_disable(dly_int);
+ int_mask = (priv->soc->rx_int | priv->soc->tx_int);
+ if (likely(status & int_mask)) {
+ fe_int_disable(int_mask);
napi_schedule(&priv->rx_napi);
} else {
fe_reg_w32(status, FE_REG_FE_INT_STATUS);
@@ -929,11 +929,11 @@ static irqreturn_t fe_handle_irq(int irq, void *dev)
static void fe_poll_controller(struct net_device *dev)
{
struct fe_priv *priv = netdev_priv(dev);
- u32 dly_int = priv->soc->tx_dly_int | priv->soc->rx_dly_int;
+ u32 int_mask = priv->soc->tx_int | priv->soc->rx_int;
- fe_int_disable(dly_int);
+ fe_int_disable(int_mask);
fe_handle_irq(dev->irq, dev);
- fe_int_enable(dly_int);
+ fe_int_enable(int_mask);
}
#endif
@@ -1018,9 +1018,7 @@ static int fe_hw_init(struct net_device *dev)
else
fe_hw_set_macaddr(priv, dev->dev_addr);
- fe_reg_w32(FE_DELAY_INIT, FE_REG_DLY_INT_CFG);
-
- fe_int_disable(priv->soc->tx_dly_int | priv->soc->rx_dly_int);
+ fe_int_disable(priv->soc->tx_int | priv->soc->rx_int);
/* frame engine will push VLAN tag regarding to VIDX feild in Tx desc. */
if (fe_reg_table[FE_REG_FE_DMA_VID_BASE])
@@ -1068,7 +1066,7 @@ static int fe_open(struct net_device *dev)
netif_carrier_on(dev);
netif_start_queue(dev);
- fe_int_enable(priv->soc->tx_dly_int | priv->soc->rx_dly_int);
+ fe_int_enable(priv->soc->tx_int | priv->soc->rx_int);
return 0;
@@ -1083,7 +1081,7 @@ static int fe_stop(struct net_device *dev)
unsigned long flags;
int i;
- fe_int_disable(priv->soc->tx_dly_int | priv->soc->rx_dly_int);
+ fe_int_disable(priv->soc->tx_int | priv->soc->rx_int);
netif_tx_disable(dev);
diff --git a/target/linux/ramips/files/drivers/net/ethernet/ralink/ralink_soc_eth.h b/target/linux/ramips/files/drivers/net/ethernet/ralink/ralink_soc_eth.h
index 8245330fbc..d26acf23d5 100644
--- a/target/linux/ramips/files/drivers/net/ethernet/ralink/ralink_soc_eth.h
+++ b/target/linux/ramips/files/drivers/net/ethernet/ralink/ralink_soc_eth.h
@@ -79,8 +79,22 @@ enum fe_reg {
#define FE_TX_DLY_INT BIT(1)
#define FE_RX_DLY_INT BIT(0)
+#define FE_RX_DONE_INT FE_RX_DONE_INT0
+#define FE_TX_DONE_INT (FE_TX_DONE_INT0 | FE_TX_DONE_INT1 | \
+ FE_TX_DONE_INT2 | FE_TX_DONE_INT3)
+
#define RT5350_RX_DLY_INT BIT(30)
#define RT5350_TX_DLY_INT BIT(28)
+#define RT5350_RX_DONE_INT1 BIT(17)
+#define RT5350_RX_DONE_INT0 BIT(16)
+#define RT5350_TX_DONE_INT3 BIT(3)
+#define RT5350_TX_DONE_INT2 BIT(2)
+#define RT5350_TX_DONE_INT1 BIT(1)
+#define RT5350_TX_DONE_INT0 BIT(0)
+
+#define RT5350_RX_DONE_INT (RT5350_RX_DONE_INT0 | RT5350_RX_DONE_INT1)
+#define RT5350_TX_DONE_INT (RT5350_TX_DONE_INT0 | RT5350_TX_DONE_INT1 | \
+ RT5350_TX_DONE_INT2 | RT5350_TX_DONE_INT3)
/* registers */
#define FE_FE_OFFSET 0x0000
@@ -367,8 +381,8 @@ struct fe_soc_data
void *swpriv;
u32 pdma_glo_cfg;
- u32 rx_dly_int;
- u32 tx_dly_int;
+ u32 rx_int;
+ u32 tx_int;
u32 checksum_bit;
u32 tx_udf_bit;
};
diff --git a/target/linux/ramips/files/drivers/net/ethernet/ralink/soc_mt7620.c b/target/linux/ramips/files/drivers/net/ethernet/ralink/soc_mt7620.c
index 60666e7886..e481c40dcb 100644
--- a/target/linux/ramips/files/drivers/net/ethernet/ralink/soc_mt7620.c
+++ b/target/linux/ramips/files/drivers/net/ethernet/ralink/soc_mt7620.c
@@ -225,8 +225,8 @@ static struct fe_soc_data mt7620_data = {
.port_init = mt7620_port_init,
.reg_table = mt7620_reg_table,
.pdma_glo_cfg = FE_PDMA_SIZE_16DWORDS | MT7620A_DMA_2B_OFFSET,
- .rx_dly_int = RT5350_RX_DLY_INT,
- .tx_dly_int = RT5350_TX_DLY_INT,
+ .rx_int = RT5350_RX_DONE_INT,
+ .tx_int = RT5350_TX_DONE_INT,
.checksum_bit = MT7620_L4_VALID,
.tx_udf_bit = MT7620_TX_DMA_UDF,
.has_carrier = mt7620a_has_carrier,
@@ -247,8 +247,8 @@ static struct fe_soc_data mt7621_data = {
.switch_config = mt7621_gsw_config,
.reg_table = mt7621_reg_table,
.pdma_glo_cfg = FE_PDMA_SIZE_16DWORDS | MT7620A_DMA_2B_OFFSET,
- .rx_dly_int = RT5350_RX_DLY_INT,
- .tx_dly_int = RT5350_TX_DLY_INT,
+ .rx_int = RT5350_RX_DONE_INT,
+ .tx_int = RT5350_TX_DONE_INT,
.checksum_bit = MT7621_L4_VALID,
.tx_udf_bit = MT7621_TX_DMA_UDF,
.has_carrier = mt7620a_has_carrier,
diff --git a/target/linux/ramips/files/drivers/net/ethernet/ralink/soc_rt2880.c b/target/linux/ramips/files/drivers/net/ethernet/ralink/soc_rt2880.c
index 61f6dcb01d..4602745542 100644
--- a/target/linux/ramips/files/drivers/net/ethernet/ralink/soc_rt2880.c
+++ b/target/linux/ramips/files/drivers/net/ethernet/ralink/soc_rt2880.c
@@ -65,8 +65,8 @@ struct fe_soc_data rt2880_data = {
.pdma_glo_cfg = FE_PDMA_SIZE_8DWORDS,
.checksum_bit = RX_DMA_L4VALID,
.tx_udf_bit = TX_DMA_UDF,
- .rx_dly_int = FE_RX_DLY_INT,
- .tx_dly_int = FE_TX_DLY_INT,
+ .rx_int = FE_RX_DONE_INT,
+ .tx_int = FE_TX_DONE_INT,
.mdio_read = rt2880_mdio_read,
.mdio_write = rt2880_mdio_write,
.mdio_adjust_link = rt2880_mdio_link_adjust,
diff --git a/target/linux/ramips/files/drivers/net/ethernet/ralink/soc_rt305x.c b/target/linux/ramips/files/drivers/net/ethernet/ralink/soc_rt305x.c
index 1ece54855c..2a8e5f7279 100644
--- a/target/linux/ramips/files/drivers/net/ethernet/ralink/soc_rt305x.c
+++ b/target/linux/ramips/files/drivers/net/ethernet/ralink/soc_rt305x.c
@@ -133,8 +133,8 @@ static struct fe_soc_data rt3050_data = {
.pdma_glo_cfg = FE_PDMA_SIZE_8DWORDS,
.checksum_bit = RX_DMA_L4VALID,
.tx_udf_bit = TX_DMA_UDF,
- .rx_dly_int = FE_RX_DLY_INT,
- .tx_dly_int = FE_TX_DLY_INT,
+ .rx_int = FE_RX_DONE_INT,
+ .tx_int = FE_TX_DONE_INT,
};
static struct fe_soc_data rt5350_data = {
@@ -148,8 +148,8 @@ static struct fe_soc_data rt5350_data = {
.pdma_glo_cfg = FE_PDMA_SIZE_8DWORDS,
.checksum_bit = RX_DMA_L4VALID,
.tx_udf_bit = TX_DMA_UDF,
- .rx_dly_int = RT5350_RX_DLY_INT,
- .tx_dly_int = RT5350_TX_DLY_INT,
+ .rx_int = RT5350_RX_DONE_INT,
+ .tx_int = RT5350_TX_DONE_INT,
};
const struct of_device_id of_fe_match[] = {
diff --git a/target/linux/ramips/files/drivers/net/ethernet/ralink/soc_rt3883.c b/target/linux/ramips/files/drivers/net/ethernet/ralink/soc_rt3883.c
index d935b967cc..1393f02929 100644
--- a/target/linux/ramips/files/drivers/net/ethernet/ralink/soc_rt3883.c
+++ b/target/linux/ramips/files/drivers/net/ethernet/ralink/soc_rt3883.c
@@ -69,8 +69,8 @@ static struct fe_soc_data rt3883_data = {
.reset_fe = rt3883_fe_reset,
.fwd_config = rt3883_fwd_config,
.pdma_glo_cfg = FE_PDMA_SIZE_8DWORDS,
- .rx_dly_int = FE_RX_DLY_INT,
- .tx_dly_int = FE_TX_DLY_INT,
+ .rx_int = FE_RX_DONE_INT,
+ .tx_int = FE_TX_DONE_INT,
.checksum_bit = RX_DMA_L4VALID,
.tx_udf_bit = TX_DMA_UDF,
.mdio_read = rt2880_mdio_read,