> -----Original Message----- > From: Intel-wired-lan <[email protected]> On Behalf Of Simon > Horman > Sent: Wednesday, April 15, 2026 3:53 PM > To: Loktionov, Aleksandr <[email protected]> > Cc: [email protected]; Nguyen, Anthony L > <[email protected]>; [email protected]; Kiran Patil > <[email protected]> > Subject: Re: [Intel-wired-lan] [PATCH iwl-net 5/5] iavf: return 0 when TC > flower > filter not found after qdisc teardown > > On Mon, Apr 13, 2026 at 09:30:35AM +0200, Aleksandr Loktionov wrote: > > From: Kiran Patil <[email protected]> > > > > When an egress qdisc is destroyed, the driver proactively deletes all > > associated cloud filters to prevent stale hardware state, decrementing > > num_cloud_filters to zero in the process. > > > > The kernel netdev layer is unaware of this implicit cleanup and may > > still try to delete the same filters individually. If the filter is > > not found in the driver's list and num_cloud_filters is already zero, > > return 0 instead of -EINVAL to avoid confusing upper layers that > > believe the filter is still offloaded in hardware. > > > > Fixes: 0075fa0fadd0 ("i40evf: Add support to apply cloud filters") > > Signed-off-by: Kiran Patil <[email protected]> > > Signed-off-by: Aleksandr Loktionov <[email protected]> > > Reviewed-by: Simon Horman <[email protected]> > > Sashiko has some comments on this function - which do not seem related to the > logic this patch touches. > > I'd encourage you to take a look at some point as a follow-up activity. > > ...
Tested-by: Rafal Romanowski <[email protected]>
