<HTML><BODY>> > diff --git a/drivers/net/bnxt/bnxt_ethdev.c > > b/drivers/net/bnxt/bnxt_ethdev.c > > index b677f9491d..bb02a2579b 100644 > > --- a/drivers/net/bnxt/bnxt_ethdev.c > > +++ b/drivers/net/bnxt/bnxt_ethdev.c > > @@ -652,8 +652,9 @@ static int bnxt_init_fc_ctx_mem(struct bnxt *bp) > > > > max_fc = bp->flow_stat->max_fc; > > > > - sprintf(type, "bnxt_rx_fc_in_" PCI_PRI_FMT, pdev->addr.domain, > > - pdev->addr.bus, pdev->addr.devid, pdev->addr.function); > > + snprintf(type, RTE_MEMZONE_NAMESIZE, "bnxt_rx_fc_in_" PCI_PRI_FMT, > > + pdev->addr.domain, pdev->addr.bus, > > + pdev->addr.devid, pdev->addr.function); > > /* 4 bytes for each counter-id */ >Rather than silently truncating, it should return an error. As far as I understand, these strings are used as diagnostic information when tracing is enabled. Is the truncation of a debug string so critical that we should interrupt device initialization by returning an error? For example, Linux kernel truncates worker thread names if the strings are too long пятница, 17 апреля 2026г., 20:04 +03:00 от Stephen Hemminger <a href="mailto:stephen@networkplumber.org">stephen@networkplumber.org</a>: <blockquote id="mail-app-auto-quote" cite="17764454431941097980" style="border-left:1px solid #297BEF; margin:0px 0px 0px 10px; padding:0px 0px 0px 10px;"><div class="js-helper js-readmsg-msg"> <style type="text/css"></style> <base target="_self" href="https://e.mail.ru/" /> <div id="style_17764454431941097980"> <div id="style_17764454431941097980_BODY">On Fri, 17 Apr 2026 12:43:07 -0400 Kishore Padmanabha <<a href="/compose?To=kishore.padmanabha@broadcom.com">kishore.padmanabha@broadcom.com</a>> wrote: > > diff --git a/drivers/net/bnxt/bnxt_ethdev.c > > b/drivers/net/bnxt/bnxt_ethdev.c > > index b677f9491d..bb02a2579b 100644 > > --- a/drivers/net/bnxt/bnxt_ethdev.c > > +++ b/drivers/net/bnxt/bnxt_ethdev.c > > @@ -652,8 +652,9 @@ static int bnxt_init_fc_ctx_mem(struct bnxt *bp) > > > > max_fc = bp->flow_stat->max_fc; > > > > - sprintf(type, "bnxt_rx_fc_in_" PCI_PRI_FMT, pdev->addr.domain, > > - pdev->addr.bus, pdev->addr.devid, pdev->addr.function); > > + snprintf(type, RTE_MEMZONE_NAMESIZE, "bnxt_rx_fc_in_" PCI_PRI_FMT, > > + pdev->addr.domain, pdev->addr.bus, > > + pdev->addr.devid, pdev->addr.function); > > /* 4 bytes for each counter-id */ Rather than silently truncating, it should return an error.</div> </div> </div></blockquote></BODY></HTML>