From mboxrd@z Thu Jan  1 00:00:00 1970
From: Niklas Cassel <niklas.cassel@linaro.org>
Subject: Request for stable backport: stmmac: Use correct values in TQS/RQS
 fields
Date: Wed, 19 Dec 2018 10:55:16 +0100
Message-ID: <20181219095516.GA7747@centauri.lan>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: netdev@vger.kernel.org, vinod.koul@linaro.org
To: davem@davemloft.net
Return-path: <netdev-owner@vger.kernel.org>
Received: from mail-lf1-f43.google.com ([209.85.167.43]:34950 "EHLO
        mail-lf1-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1726537AbeLSJzU (ORCPT
        <rfc822;netdev@vger.kernel.org>); Wed, 19 Dec 2018 04:55:20 -0500
Received: by mail-lf1-f43.google.com with SMTP id e26so14568726lfc.2
        for <netdev@vger.kernel.org>; Wed, 19 Dec 2018 01:55:19 -0800 (PST)
Content-Disposition: inline
Sender: netdev-owner@vger.kernel.org
List-ID: <netdev.vger.kernel.org>

Hello David,

I can observe a netdev watchdog timeout on kernel 4.14.78 when using stmmac
with multiple tx queues.

Backporting the following commit:

commit 52a76235d0c4dd259cd0df503afed4757c04ba1d                                                  
Author: Jose Abreu <Jose.Abreu@synopsys.com>
Date:   Fri Oct 13 10:58:36 2017 +0100                                                           
                                                                                                 
    net: stmmac: Use correct values in TQS/RQS fields                                            

    Currently we are using all the available fifo size in RQS and                                
    TQS fields. This will not work correctly in multi-queues IP's                                
    because total fifo size must be splitted to the enabled queues.                              
                                                                                                 
    Correct this by computing the available fifo size per queue and                              
    setting the right value in TQS and RQS fields.                                               
                                                                                                 
    Signed-off-by: Jose Abreu <joabreu@synopsys.com>                                             
    Cc: David S. Miller <davem@davemloft.net>                                                    
    Cc: Joao Pinto <jpinto@synopsys.com>
    Cc: Giuseppe Cavallaro <peppe.cavallaro@st.com>                                              
    Cc: Alexandre Torgue <alexandre.torgue@st.com>                                               
    Signed-off-by: David S. Miller <davem@davemloft.net>


resolves the issue.

The fix was first included in v4.15
$ git tag --contains 52a76235d0c4dd259cd0df503afed4757c04ba1d
v4.15
v4.15-rc1
v4.15-rc2

Could you please queue it up for 4.14 stable?


Kind regards,
Niklas