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

Jakub Kicinski
2024-08-26T22:49:14Z

On Mon, 26 Aug 2024 16:56:36 -0400 Andy Gospodarek wrote:
> We plan to replace these calls with calls to stop and start only that
> ring via netdev_rx_queue_restart as soon as these calls all land in
> the same tree. Since this set is [presumably] coming through
> linux-pci we didn't think we could do that yet.
>
> Thoughts?

The merge window is in 3 weeks or so, so this can wait.
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.

While we talk about affinity settings in bnxt -- it'd be great
if it maintained the mapping and XPS settings across reconfiguration
(like queue count changes or attaching XDP).