Re: [EXT] Re: [PATCH net-next v2 2/9] octeon_ep: poll for control messages

From: Jakub Kicinski
Date: Tue Dec 06 2022 - 12:23:59 EST


On Tue, 6 Dec 2022 10:58:47 +0200 Leon Romanovsky wrote:
> > Polling for control messages every 100ms? Sure.
> >
> > You say "valid in netdev" so perhaps you can educate us where/why it
> > would not be?
>
> It doesn't seem right to me that idle device burns CPU cycles, while it
> supports interrupts. If it needs "listen to FW", it will be much nicer to
> install interrupts immediately and don't wait for netdev.

No doubt, if there is an alternative we can push for it to be
implemented. I guess this being yet another "IPU" there could
be possible workarounds in FW? As always with IPUs - hard to tell :/

If there is no alternative - it is what it is.
It's up to customers to buy good HW.

That said, looking at what this set does - how are the VFs configured?
That's the showstopper for the series in my mind.