Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH] net: ethernet: ti: fix netdev_queue compiling error
To:     Chen Jiahao <chenjiahao16@huawei.com>, <davem@davemloft.net>,
        <kuba@kernel.org>, <jesse.brandeburg@intel.com>, <vigneshr@ti.com>,
        <netdev@vger.kernel.org>, <linux-kernel@vger.kernel.org>
CC:     <heying24@huawei.com>
References: <20210617112838.143314-1-chenjiahao16@huawei.com>
From:   Grygorii Strashko <grygorii.strashko@ti.com>
Message-ID: <6dbabec2-25df-7a4a-457f-d738479d36b1@ti.com>
Date:   Fri, 18 Jun 2021 13:40:50 +0300
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.8.1
MIME-Version: 1.0
In-Reply-To: <20210617112838.143314-1-chenjiahao16@huawei.com>
Content-Type: text/plain; charset="utf-8"; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 8bit
Precedence: bulk


On 17/06/2021 14:28, Chen Jiahao wrote:
> There is a compiling error in am65-cpsw-nuss.c while not selecting
> CONFIG_BQL:
> 
> drivers/net/ethernet/ti/am65-cpsw-nuss.c: In function
> ‘am65_cpsw_nuss_ndo_host_tx_timeout’:
> drivers/net/ethernet/ti/am65-cpsw-nuss.c:353:26: error:
> ‘struct netdev_queue’ has no member named ‘dql’
>    353 |      dql_avail(&netif_txq->dql),
>        |                          ^~
> 
> This problem is solved by adding the #ifdef CONFIG_BQL directive
> where struct dql is used.
> 
> Fixes: 93a76530316a ("net: ethernet: ti: introduce am65x/j721e gigabit eth subsystem driver")
> Signed-off-by: Chen Jiahao <chenjiahao16@huawei.com>
> ---
>   drivers/net/ethernet/ti/am65-cpsw-nuss.c | 8 ++++++++
>   1 file changed, 8 insertions(+)
> 
> diff --git a/drivers/net/ethernet/ti/am65-cpsw-nuss.c b/drivers/net/ethernet/ti/am65-cpsw-nuss.c
> index 6a67b026df0b..a0b30bb763ea 100644
> --- a/drivers/net/ethernet/ti/am65-cpsw-nuss.c
> +++ b/drivers/net/ethernet/ti/am65-cpsw-nuss.c
> @@ -346,12 +346,20 @@ static void am65_cpsw_nuss_ndo_host_tx_timeout(struct net_device *ndev,
>   	tx_chn = &common->tx_chns[txqueue];
>   	trans_start = netif_txq->trans_start;
>   
> +#ifdef CONFIG_BQL
>   	netdev_err(ndev, "txq:%d DRV_XOFF:%d tmo:%u dql_avail:%d free_desc:%zu\n",
>   		   txqueue,
>   		   netif_tx_queue_stopped(netif_txq),
>   		   jiffies_to_msecs(jiffies - trans_start),
>   		   dql_avail(&netif_txq->dql),
>   		   k3_cppi_desc_pool_avail(tx_chn->desc_pool));
> +#else
> +	netdev_err(ndev, "txq:%d DRV_XOFF:%d tmo:%u free_desc:%zu\n",
> +		   txqueue,
> +		   netif_tx_queue_stopped(netif_txq),
> +		   jiffies_to_msecs(jiffies - trans_start),
> +		   k3_cppi_desc_pool_avail(tx_chn->desc_pool));
> +#endif
>   
>   	if (netif_tx_queue_stopped(netif_txq)) {
>   		/* try recover if stopped by us */
> 

Seems like there is right helper available - qdisc_avail_bulklimit().

Any way, it most probably has to be solved in generic way on netdev/dql level.

-- 
Best regards,
grygorii