<div dir="ltr">Hi Stephen,<br><br>Agreed. Since it gates nothing, I will delete it rather than document a no-op constant.<br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Mon, Sep 21, 2026 at 9:54 AM Stephen Hemminger <<a href="mailto:stephen@networkplumber.org">stephen@networkplumber.org</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, 20 Sep 2026 20:59:03 -0600<br>
Mohammad Shuab Siddique <<a href="mailto:mohammad-shuab.siddique@broadcom.com" target="_blank">mohammad-shuab.siddique@broadcom.com</a>> wrote:<br>
<br>
> From: Mohammad Shuab Siddique <<a href="mailto:mohammad-shuab.siddique@broadcom.com" target="_blank">mohammad-shuab.siddique@broadcom.com</a>><br>
> <br>
> The driver only supported queue sizes up to 4096 for Tx and 8192 for<br>
> Rx. Raise both to 16384. The completion ring for a Rx ring is sized<br>
> at 2x the Rx ring size, further multiplied by 4 when the aggregation<br>
> ring is in use (8x total), so at 16384 it can reach 131072, above<br>
> uint16_t range - widen the ring index/counter variables touched by<br>
> that path to uint32_t.<br>
> <br>
> Signed-off-by: Keegan Freyhof <<a href="mailto:keegan.freyhof@broadcom.com" target="_blank">keegan.freyhof@broadcom.com</a>><br>
> Signed-off-by: Mohammad Shuab Siddique <<a href="mailto:mohammad-shuab.siddique@broadcom.com" target="_blank">mohammad-shuab.siddique@broadcom.com</a>><br>
> ---<br>
[PATCH v2] net/bnxt: add support for queue size of 16384<br>
<br>
Warning: the change of BNXT_MAX_MTU and BNXT_NUM_VLANS to UL is<br>
unrelated to queue size and is not described. It changes the type of<br>
BNXT_MAX_PKT_LEN to unsigned long. Drop it, or split it out with a<br>
reason.<br>
<br>
Warning: MAX_CP_DESC_CNT is bumped to 32K and gains a comment saying<br>
it is unused and the real worst case is 131072. If it is unused,<br>
delete it.<br>
</blockquote></div><div><br clear="all"></div><div><br></div><span class="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature"><div dir="ltr">Regards,<div>Shuab</div></div></div>