[2/2] net/bnxt: suppress supprious error log

Message ID 20190314213214.26957-3-stephen@networkplumber.org
State Accepted
Delegated to: Ferruh Yigit
Headers show
Series
  • net/bnxt: supress warnings
Related show

Checks

Context Check Description
ci/Intel-compilation success Compilation OK
ci/checkpatch success coding style OK

Commit Message

Stephen Hemminger March 14, 2019, 9:32 p.m.
The driver multiple rxq allocation logs a message at error level
but it really is a debug message.

Fixes: 51fafb89a9a0 ("net/bnxt: get rid of ff pools and use VNIC info array")
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/bnxt/bnxt_rxq.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Somnath Kotur March 15, 2019, 12:04 a.m. | #1
On Fri, Mar 15, 2019 at 3:02 AM Stephen Hemminger
<stephen@networkplumber.org> wrote:
>
> The driver multiple rxq allocation logs a message at error level
> but it really is a debug message.
>
> Fixes: 51fafb89a9a0 ("net/bnxt: get rid of ff pools and use VNIC info array")
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---
>  drivers/net/bnxt/bnxt_rxq.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/bnxt/bnxt_rxq.c b/drivers/net/bnxt/bnxt_rxq.c
> index 5345d3938311..17e2909a7e5c 100644
> --- a/drivers/net/bnxt/bnxt_rxq.c
> +++ b/drivers/net/bnxt/bnxt_rxq.c
> @@ -100,7 +100,8 @@ int bnxt_mq_rx_configure(struct bnxt *bp)
>                 }
>         }
>         nb_q_per_grp = bp->rx_cp_nr_rings / pools;
> -       PMD_DRV_LOG(ERR, "pools = %u nb_q_per_grp = %u\n", pools, nb_q_per_grp);
> +       PMD_DRV_LOG(DEBUG, "pools = %u nb_q_per_grp = %u\n",
> +                   pools, nb_q_per_grp);
>         start_grp_id = 0;
>         end_grp_id = nb_q_per_grp;
>
> --
> 2.17.1
>
Acked-by: Somnath Kotur (somnath.kotur@broadcom.com

Patch

diff --git a/drivers/net/bnxt/bnxt_rxq.c b/drivers/net/bnxt/bnxt_rxq.c
index 5345d3938311..17e2909a7e5c 100644
--- a/drivers/net/bnxt/bnxt_rxq.c
+++ b/drivers/net/bnxt/bnxt_rxq.c
@@ -100,7 +100,8 @@  int bnxt_mq_rx_configure(struct bnxt *bp)
 		}
 	}
 	nb_q_per_grp = bp->rx_cp_nr_rings / pools;
-	PMD_DRV_LOG(ERR, "pools = %u nb_q_per_grp = %u\n", pools, nb_q_per_grp);
+	PMD_DRV_LOG(DEBUG, "pools = %u nb_q_per_grp = %u\n",
+		    pools, nb_q_per_grp);
 	start_grp_id = 0;
 	end_grp_id = nb_q_per_grp;