Re: [PATCH V4 11/12] bnxt_en: Add TPH support in BNXT driver

Jakub Kicinski
2024-08-27T19:05:46Z

On Tue, 27 Aug 2024 10:50:51 -0400 Andy Gospodarek wrote:
> > The merge window is in 3 weeks or so, so this can wait.
>
> Are you asking for the patch for this feature to include the queue
> stop/start instead of this?

Yes, indeed.

> I just checked linux-pci and it does have bnxt_queue_stop/bnxt_queue_start.
>
> > I'm worried we'll find out later that the current queue reset
> > implementation in bnxt turns out to be insufficient. And we'll
> > be stuck with yet another close/open in this driver.
>
> The queue reset _has_ to work. We will ensure that it does and fix any
> problems found. Note that these have been under test already internally
> and fixes are/will be posted to the list as they are made. Holding this
> patch because an API that it uses might not work seems odd.

Not holding because API may not work, holding because (I thought)
API isn't in place at all. If bnxt_queue_stop/bnxt_queue_start are in
linux-pci please rewrite the patch to use those and then all clear
from my PoV.