| Message ID | 1464873292-17011-1-git-send-email-ivan.khoronzhuk@linaro.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On 02.06.16 16:14, Ivan Khoronzhuk wrote: > The rx-usecs shouldn't be changed while interface down/up. > Currently, for instance, if it's set to 100us, after interface > down/up it's 500us. It's a hidden bug that can lead to lavish > interrupt pacing time increasing while "down/up" up to max value. > > Steps to reproduce: > - set rx-usecs to be 100us > - down/up interface > - read new unexpected rx-usecs > > Signed-off-by: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org> > --- > > Based on ti-linux-4.4.y Please ignore this line, it's added by mistake. The patch is based on master. > > drivers/net/ethernet/ti/cpsw.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/ti/cpsw.c b/drivers/net/ethernet/ti/cpsw.c > index 7b44587..9919cb3 100644 > --- a/drivers/net/ethernet/ti/cpsw.c > +++ b/drivers/net/ethernet/ti/cpsw.c > @@ -1352,7 +1352,7 @@ static int cpsw_ndo_open(struct net_device *ndev) > if (priv->coal_intvl != 0) { > struct ethtool_coalesce coal; > > - coal.rx_coalesce_usecs = (priv->coal_intvl << 4); > + coal.rx_coalesce_usecs = priv->coal_intvl; > cpsw_set_coalesce(ndev, &coal); > } > >
On 06/02/2016 04:30 PM, Ivan Khoronzhuk wrote: > On 02.06.16 16:14, Ivan Khoronzhuk wrote: >> The rx-usecs shouldn't be changed while interface down/up. >> Currently, for instance, if it's set to 100us, after interface >> down/up it's 500us. It's a hidden bug that can lead to lavish >> interrupt pacing time increasing while "down/up" up to max value. >> >> Steps to reproduce: >> - set rx-usecs to be 100us >> - down/up interface >> - read new unexpected rx-usecs >> >> Signed-off-by: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org> >> --- >> >> Based on ti-linux-4.4.y > Please ignore this line, it's added by mistake. > The patch is based on master. > Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com> > >> >> drivers/net/ethernet/ti/cpsw.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/net/ethernet/ti/cpsw.c >> b/drivers/net/ethernet/ti/cpsw.c >> index 7b44587..9919cb3 100644 >> --- a/drivers/net/ethernet/ti/cpsw.c >> +++ b/drivers/net/ethernet/ti/cpsw.c >> @@ -1352,7 +1352,7 @@ static int cpsw_ndo_open(struct net_device *ndev) >> if (priv->coal_intvl != 0) { >> struct ethtool_coalesce coal; >> >> - coal.rx_coalesce_usecs = (priv->coal_intvl << 4); >> + coal.rx_coalesce_usecs = priv->coal_intvl; >> cpsw_set_coalesce(ndev, &coal); >> } >> >> >
From: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org> Date: Thu, 2 Jun 2016 16:14:52 +0300 > The rx-usecs shouldn't be changed while interface down/up. > Currently, for instance, if it's set to 100us, after interface > down/up it's 500us. It's a hidden bug that can lead to lavish > interrupt pacing time increasing while "down/up" up to max value. > > Steps to reproduce: > - set rx-usecs to be 100us > - down/up interface > - read new unexpected rx-usecs > > Signed-off-by: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org> Applied. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/ethernet/ti/cpsw.c b/drivers/net/ethernet/ti/cpsw.c index 7b44587..9919cb3 100644 --- a/drivers/net/ethernet/ti/cpsw.c +++ b/drivers/net/ethernet/ti/cpsw.c @@ -1352,7 +1352,7 @@ static int cpsw_ndo_open(struct net_device *ndev) if (priv->coal_intvl != 0) { struct ethtool_coalesce coal; - coal.rx_coalesce_usecs = (priv->coal_intvl << 4); + coal.rx_coalesce_usecs = priv->coal_intvl; cpsw_set_coalesce(ndev, &coal); }
The rx-usecs shouldn't be changed while interface down/up. Currently, for instance, if it's set to 100us, after interface down/up it's 500us. It's a hidden bug that can lead to lavish interrupt pacing time increasing while "down/up" up to max value. Steps to reproduce: - set rx-usecs to be 100us - down/up interface - read new unexpected rx-usecs Signed-off-by: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org> --- Based on ti-linux-4.4.y drivers/net/ethernet/ti/cpsw.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)