<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 28, 2022 at 3:46 PM Ajit Khaparde <<a href="mailto:ajit.khaparde@broadcom.com">ajit.khaparde@broadcom.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Sun, Mar 27, 2022 at 11:29 PM Christian Ehrhardt<br>
<<a href="mailto:christian.ehrhardt@canonical.com" target="_blank">christian.ehrhardt@canonical.com</a>> wrote:<br>
><br>
> On Fri, Mar 25, 2022 at 9:15 AM Kalesh A P<br>
> <<a href="mailto:kalesh-anakkur.purayil@broadcom.com" target="_blank">kalesh-anakkur.purayil@broadcom.com</a>> wrote:<br>
> ><br>
> > From: Kalesh AP <<a href="mailto:kalesh-anakkur.purayil@broadcom.com" target="_blank">kalesh-anakkur.purayil@broadcom.com</a>><br>
> ><br>
> > This patch fixes a typo introduced in the last backport.<br>
> > Fixed a wrong check.<br>
><br>
> Hi Kalesh,<br>
> thanks for the fix.<br>
><br>
> Since we already have entered -rc1 and testing I wanted to ask how<br>
> sever this issue is.<br>
> The options we have are<br>
> a) not really an important issue, hold it back not and make it part of<br>
> 19.11.13 later this year<br>
> b) breaking bnxt too much, needs to get into 19.11.12 - but we would<br>
> not reset testing (no new RC), you'd cover some bnxt related tests<br>
> c) breaking bnxt too much and having potential to influence all<br>
> things, taking into 19.11.12 and casting an -rc2 resetting tests for<br>
> everyone<br>
><br>
> I'm tempted to consider this a case for (b), but wanted to know if you<br>
> agree and if you could make the related bnxt based re-test happen?<br>
Christian, option (b) is fine. We can take care of bnxt related tests.<br></blockquote><div><br></div><div>Thank you, applied without tagging -rc2 as agreed.</div><div><br></div><div>For your testing it is here</div><div><a href="https://git.dpdk.org/dpdk-stable/log/?h=19.11">https://git.dpdk.org/dpdk-stable/log/?h=19.11</a><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Thanks<br>
Ajit<br>
><br>
> > Upstream code does not have this issue.<br>
> ><br>
> > Bugzilla ID: 977<br>
> > Fixes: 942eb8e842fc ("net/bnxt: fix xstats names query overrun")<br>
> ><br>
> > Signed-off-by: Kalesh AP <<a href="mailto:kalesh-anakkur.purayil@broadcom.com" target="_blank">kalesh-anakkur.purayil@broadcom.com</a>><br>
> > ---<br>
> >  drivers/net/bnxt/bnxt_stats.c | 2 +-<br>
> >  1 file changed, 1 insertion(+), 1 deletion(-)<br>
> ><br>
> > diff --git a/drivers/net/bnxt/bnxt_stats.c b/drivers/net/bnxt/bnxt_stats.c<br>
> > index 39fd100..bc181db 100644<br>
> > --- a/drivers/net/bnxt/bnxt_stats.c<br>
> > +++ b/drivers/net/bnxt/bnxt_stats.c<br>
> > @@ -612,7 +612,7 @@ int bnxt_dev_xstats_get_names_op(__rte_unused struct rte_eth_dev *eth_dev,<br>
> >         if (rc)<br>
> >                 return rc;<br>
> ><br>
> > -       if (xstats_names != NULL || size < stat_cnt)<br>
> > +       if (xstats_names == NULL || size < stat_cnt)<br>
> >                 return stat_cnt;<br>
> ><br>
> >         for (i = 0; i < RTE_DIM(bnxt_rx_stats_strings); i++) {<br>
> > --<br>
> > 2.10.1<br>
> ><br>
><br>
><br>
> --<br>
> Christian Ehrhardt<br>
> Staff Engineer, Ubuntu Server<br>
> Canonical Ltd<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature">Christian Ehrhardt<br>Staff Engineer, Ubuntu Server<br>Canonical Ltd</div></div>