Re: [PATCH net] bnxt_en: Gate TPH enablement behind BNXT_SUPPORTS_QUEUE_API check
From: Thomas Walsh
Date: Thu Aug 20 2026 - 17:27:21 EST
On Thu, Aug 20, 2026 at 5:15 PM Jakub Kicinski <kuba@xxxxxxxxxx> wrote:
>
> On Thu, 20 Aug 2026 12:16:07 -0700 Michael Chan wrote:
> > On Thu, Aug 20, 2026 at 11:59 AM Michael Chan <michael.chan@xxxxxxxxxxxx> wrote:
> > >
> > > On Thu, Aug 20, 2026 at 11:30 AM Thomas Walsh <thwalsh@xxxxxxxxxx> wrote:
> > > > NIC is BCM57414 NetXtreme-E 10Gb/25Gb RDMA Ethernet Controller
> > > > [14e4:16d7]
> > > > FW version shared with us was 238.1.138.6 upgraded from 236.1.167.0
> > >
> > > This NIC does not support queue management. I think it also doesn't
> > > support TPH, but I need to check and confirm.
> >
> > Confirmed. Only Thor and newer (P5_PLUS) supports TPH.
> >
> > So the patch looks fine to me. TPH requires queue management to
> > restart the queue. And NICs that support queue management (with
> > updated FW) all support TPH. Thanks.
> >
> > Reviewed-by: Michael Chan <michael.chan@xxxxxxxxxxxx>
>
> Thanks!
>
> Thomas, please update commit msg with the info from Michael, rebase on
> up-to-date net/main and repost.
> --
> pw-bot: cr
>
Thank you both!
I'll get the v2 out shortly.