From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id AAD43A0547 for ; Sun, 26 Sep 2021 23:17:51 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 968324003D; Sun, 26 Sep 2021 23:17:51 +0200 (CEST) Received: from mail-io1-f45.google.com (mail-io1-f45.google.com [209.85.166.45]) by mails.dpdk.org (Postfix) with ESMTP id 22BBF4003D for ; Sun, 26 Sep 2021 23:17:50 +0200 (CEST) Received: by mail-io1-f45.google.com with SMTP id h129so20288865iof.1 for ; Sun, 26 Sep 2021 14:17:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=qb5CcpQqz9rHR69e6s2vKAZORldzmxFgiEQ3iWa8TF8=; b=PI66OJQadYzk0B6P6VJYTDHdyoFnn3coYXjDvPwj0uVHxxrCYKFi3yxBaHfI+t5wDg GJ0ZhdeoHE1cqTx5GOfC4BhnYfbrkl+ojfmtaRE3p3JU4DA9od+gq6eTNukgVB6QaX5W Va31kZ0IBpQKuJpf+30LENT533d7gUtp94klA= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=qb5CcpQqz9rHR69e6s2vKAZORldzmxFgiEQ3iWa8TF8=; b=AbqXSRO9vTbL64PBnukWLjghDJYHV36IoROwqHb0NQamiarkH/QqFVk1p+tDhZaoA9 WPYFjw7JaRXa05i3kQ2VbSnt1uMk7Fuk24JoplM9yx7h/X1CnUnpEZovr1ZvBNN9Eb4Y ie+ZJtHlXt/H0gp/PiCDoZjztiHKTCDISpTZGbf6uvjwyijsH6iFpmT/j3lH0elgpU01 gryZ5HEZ5EP+qt/DkCxOu8gQjG6nugA0/PTqTYjty27gWKIoeMlc5ssseFLCKK9b7y6Z aQLePn+5lm9eVXmzXzcQazAkz402vJYufCYRsgdWefZBsTtLh/0WFuWNE1wloR8ksoUc gBNg== X-Gm-Message-State: AOAM531DVDSfn6dxXy+qgeenUCarw6irb61DFMZ9aeqJL+hsZ1pv6d6R cejBinYx34EydfZoMetjP0R0xsSVlxoQTEy+Vl8jiZpBdpI= X-Google-Smtp-Source: ABdhPJzuBtp3PtF7waNlR1UIhlL/96PnPb6xBQQfKcmgjL5Xi3QXRbOnLvwql6QzLz7YThQWAQHqeVYk2v5d8b+baTI= X-Received: by 2002:a05:6638:3890:: with SMTP id b16mr19362211jav.65.1632691069446; Sun, 26 Sep 2021 14:17:49 -0700 (PDT) MIME-Version: 1.0 References: <20210925005544.50518-1-ajit.khaparde@broadcom.com> In-Reply-To: <20210925005544.50518-1-ajit.khaparde@broadcom.com> From: Ajit Khaparde Date: Sun, 26 Sep 2021 14:17:33 -0700 Message-ID: To: dpdk-dev Cc: dpdk stable , Ferruh Yigit Content-Type: multipart/signed; protocol="application/pkcs7-signature"; micalg=sha-256; boundary="0000000000003d1cc205ccec83f4" X-Content-Filtered-By: Mailman/MimeDel 2.1.29 Subject: Re: [dpdk-stable] [PATCH] net/bnxt: fix Tx queue startup state X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" --0000000000003d1cc205ccec83f4 Content-Type: text/plain; charset="UTF-8" On Fri, Sep 24, 2021 at 5:55 PM Ajit Khaparde wrote: > > Default queue state of Tx queues on startup is not correct. > Fix this by setting the state when the port is started. > > Fixes: 6eb3cc2294fd ("net/bnxt: add initial Tx code") > Cc: stable@dpdk.org > > Signed-off-by: Ajit Khaparde > Reviewed-by: Lance Richardson Patch applied to dpdk-next-net-brcm. Thanks > --- > drivers/net/bnxt/bnxt_ethdev.c | 10 ++++++++++ > drivers/net/bnxt/bnxt_txq.c | 5 ----- > 2 files changed, 10 insertions(+), 5 deletions(-) > > diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c > index 097dd10de9..eb9147927e 100644 > --- a/drivers/net/bnxt/bnxt_ethdev.c > +++ b/drivers/net/bnxt/bnxt_ethdev.c > @@ -814,6 +814,16 @@ static int bnxt_start_nic(struct bnxt *bp) > } > } > > + for (j = 0; j < bp->tx_nr_rings; j++) { > + struct bnxt_tx_queue *txq = bp->tx_queues[j]; > + > + if (!txq->tx_deferred_start) { > + bp->eth_dev->data->tx_queue_state[j] = > + RTE_ETH_QUEUE_STATE_STARTED; > + txq->tx_started = true; > + } > + } > + > rc = bnxt_hwrm_cfa_l2_set_rx_mask(bp, &bp->vnic_info[0], 0, NULL); > if (rc) { > PMD_DRV_LOG(ERR, > diff --git a/drivers/net/bnxt/bnxt_txq.c b/drivers/net/bnxt/bnxt_txq.c > index 830416af3d..3ffc334cff 100644 > --- a/drivers/net/bnxt/bnxt_txq.c > +++ b/drivers/net/bnxt/bnxt_txq.c > @@ -166,11 +166,6 @@ int bnxt_tx_queue_setup_op(struct rte_eth_dev *eth_dev, > > eth_dev->data->tx_queues[queue_idx] = txq; > > - if (txq->tx_deferred_start) > - txq->tx_started = false; > - else > - txq->tx_started = true; > - > return 0; > err: > bnxt_tx_queue_release_op(txq); > -- > 2.30.1 (Apple Git-130) > --0000000000003d1cc205ccec83f4--