> -----Original Message-----
> From: Jose Ignacio Tornos Martinez <[email protected]>
> Sent: Friday, April 24, 2026 12:33 PM
> To: Loktionov, Aleksandr <[email protected]>
> Cc: Nguyen, Anthony L <[email protected]>;
> [email protected]; [email protected]; [email protected]; intel-
> [email protected]; Keller, Jacob E
> <[email protected]>; [email protected];
> [email protected]; [email protected]; [email protected];
> [email protected]; Kitszel, Przemyslaw <[email protected]>
> Subject: RE: [PATCH net v4 4/4] ice: skip unnecessary VF reset when
> setting trust
>
> Hello Aleksandr,
>
> > For me it looks like cc: [email protected] must be added
> I am not sure about that, because the bugs fixed here (vf->trusted
> ordering and race condition) only trigger when MAC LLDP filters exist,
> which is an uncommon scenario.
> And most users will benefit from the performance improvement that is
> an optimization rather than the bug fixes.
> I mean, I included the commit fixed as a reference but due to
> optimization as the main reason, I didn't dare to request this for
> older versions.
Ok I see your point. As it's optimization, you are right.
>
> > You declare ice_vf_clear_all_promisc_modes() returning int, but
> ignore
> > the return value.
> > Looks suspicious isn't it?
> Well, it is used like that when the funciton is called locally (the
> function is not modifiedi, just made public), and really my intention
> was to clean as much as possible (so error checking is not necessary).
> In my opinion it would be enough to warn about the possible problems
> (already done in the existing function).
Can you go extra mile and add error code handling?
Or at least document it in the code why you don't do it?
>
> Anyway, if, despite the reasons I have tried to explain, you still
> think the same way, please let me know so I can adjust them (if you
> don't mind, I would wait for more reviews to include them in a next
> version).
>
> Thanks
>
> Best regards
> Jose Ignacio
Thank you
Alex
