<div dir="ltr"><div>The size of the string should not exceed 27 or 28 bytes less than RTE_MEMZONE_NAMESIZE. </div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Wed, Apr 22, 2026 at 12:44 AM Денис Люлин <<a href="mailto:lyulin.2003@mail.ru">lyulin.2003@mail.ru</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">
<div><p style="margin-top:0px" dir="ltr">> > diff --git a/drivers/net/bnxt/bnxt_ethdev.c<br>
> > b/drivers/net/bnxt/bnxt_ethdev.c<br>
> > index b677f9491d..bb02a2579b 100644<br>
> > --- a/drivers/net/bnxt/bnxt_ethdev.c<br>
> > +++ b/drivers/net/bnxt/bnxt_ethdev.c<br>
> > @@ -652,8 +652,9 @@ static int bnxt_init_fc_ctx_mem(struct bnxt *bp)<br>
> ><br>
> > max_fc = bp->flow_stat->max_fc;<br>
> ><br>
> > - sprintf(type, "bnxt_rx_fc_in_" PCI_PRI_FMT, pdev->addr.domain,<br>
> > - pdev->addr.bus, pdev->addr.devid, pdev->addr.function);<br>
> > + snprintf(type, RTE_MEMZONE_NAMESIZE, "bnxt_rx_fc_in_" PCI_PRI_FMT,<br>
> > + pdev->addr.domain, pdev->addr.bus,<br>
> > + pdev->addr.devid, pdev->addr.function);<br>
> > /* 4 bytes for each counter-id */</p>
<p dir="ltr">>Rather than silently truncating, it should return an error.</p>
<p dir="ltr">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</p>
пятница, 17 апреля 2026г., 20:04 +03:00 от Stephen Hemminger <a href="mailto:stephen@networkplumber.org" target="_blank">stephen@networkplumber.org</a>:<br><br><blockquote id="m_5315368748831621944mail-app-auto-quote" cite="http://17764454431941097980" style="border-left:1px solid rgb(41,123,239);margin:0px 0px 0px 10px;padding:0px 0px 0px 10px"><div>
    
    
    
    <div id="m_5315368748831621944style_17764454431941097980">
        <div id="m_5315368748831621944style_17764454431941097980_BODY">On Fri, 17 Apr 2026 12:43:07 -0400<br>
Kishore Padmanabha <<a href="https://e.mail.ru/compose?To=kishore.padmanabha@broadcom.com" target="_blank">kishore.padmanabha@broadcom.com</a>> wrote:<br>
<br>
> > diff --git a/drivers/net/bnxt/bnxt_ethdev.c<br>
> > b/drivers/net/bnxt/bnxt_ethdev.c<br>
> > index b677f9491d..bb02a2579b 100644<br>
> > --- a/drivers/net/bnxt/bnxt_ethdev.c<br>
> > +++ b/drivers/net/bnxt/bnxt_ethdev.c<br>
> > @@ -652,8 +652,9 @@ static int bnxt_init_fc_ctx_mem(struct bnxt *bp)<br>
> ><br>
> >         max_fc = bp->flow_stat->max_fc;<br>
> ><br>
> > -       sprintf(type, "bnxt_rx_fc_in_" PCI_PRI_FMT, pdev->addr.domain,<br>
> > -               pdev->addr.bus, pdev->addr.devid, pdev->addr.function);<br>
> > +       snprintf(type, RTE_MEMZONE_NAMESIZE, "bnxt_rx_fc_in_" PCI_PRI_FMT,<br>
> > +               pdev->addr.domain, pdev->addr.bus,<br>
> > +               pdev->addr.devid, pdev->addr.function);<br>
> >         /* 4 bytes for each counter-id */<br>
<br>
Rather than silently truncating, it should return an error.</div>
        
    </div>
</div></blockquote></div>
</blockquote></div></div>