On Tue, 27 Aug 2024 10:50:51 -0400 Andy Gospodarek wrote:

quoted

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.

quoted

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.