[Intel-wired-lan] [PATCH net-next] ice: fix broken Rx on VFs

[Loktionov, Aleksandr]

> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces at osuosl.org> On Behalf
> Of Alexander Lobakin
> Sent: Monday, November 24, 2025 6:18 PM
> To: Andrew Lunn <andrew+netdev at lunn.ch>; David S. Miller
> <davem at davemloft.net>; Eric Dumazet <edumazet at google.com>; Jakub
> Kicinski <kuba at kernel.org>; Paolo Abeni <pabeni at redhat.com>
> Cc: Kitszel, Przemyslaw <przemyslaw.kitszel at intel.com>; Nguyen,
> Anthony L <anthony.l.nguyen at intel.com>; Slepecki, Jakub
> <jakub.slepecki at intel.com>; NXNE CNSE OSDT ITP Upstreaming
> <nxne.cnse.osdt.itp.upstreaming at intel.com>; intel-wired-
> lan at lists.osuosl.org; netdev at vger.kernel.org; linux-
> kernel at vger.kernel.org
> Subject: Re: [Intel-wired-lan] [PATCH net-next] ice: fix broken Rx on
> VFs
> 
> From: Alexander Lobakin <aleksander.lobakin at intel.com>
> Date: Mon, 24 Nov 2025 18:07:35 +0100
> 
> Ooops, missed a tag, sorry...
> 
> > Since the tagged commit, ice stopped respecting Rx buffer length
> > passed from VFs.
> > At that point, the buffer length was hardcoded in ice, so VFs still
> > worked up to some point (until, for example, a VF wanted an MTU
> larger
> > than its PF).
> > The next commit 93f53db9f9dc ("ice: switch to Page Pool"), broke Rx
> on
> > VFs completely since ice started accounting per-queue buffer lengths
> > again, but now VF queues always had their length zeroed, as ice was
> > already ignoring what iavf was passing to it.
> >
> > Restore the line that initializes the buffer length on VF queues
> > basing on the virtchnl messages.
> >
> > Fixes: 3a4f419f7509 ("ice: drop page splitting and recycling")
> > Reported-by: Jakub Slepecki <jakub.slepecki at intel.com>
> 
> Suggested-by: Michal Swiatkowski <michal.swiatkowski at linux.intel.com>
> 
> > Reviewed-by: Tony Nguyen <anthony.l.nguyen at intel.com>
> > Signed-off-by: Alexander Lobakin <aleksander.lobakin at intel.com>
> > ---
> > I'd like this to go directly to net-next to quickly unbreak VFs (the
> > related commits are not in the mainline yet).
> Thanks,
> Olek

Reviewed-by: Aleksandr Loktionov <aleksandr.loktionov at intel.com>