[Intel-wired-lan] [PATCH net] ice: fix vf->num_mac count with port representors

[Buvaneswaran, Sujai]

> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces at osuosl.org> On Behalf Of
> Jacob Keller
> Sent: Thursday, April 10, 2025 11:44 PM
> To: Nguyen, Anthony L <anthony.l.nguyen at intel.com>; Intel Wired LAN
> <intel-wired-lan at lists.osuosl.org>; netdev <netdev at vger.kernel.org>
> Cc: Keller, Jacob E <jacob.e.keller at intel.com>
> Subject: [Intel-wired-lan] [PATCH net] ice: fix vf->num_mac count with port
> representors
> 
> The ice_vc_repr_add_mac() function indicates that it does not store the MAC
> address filters in the firmware. However, it still increments vf->num_mac.
> This is incorrect, as vf->num_mac should represent the number of MAC
> filters currently programmed to firmware.
> 
> Indeed, we only perform this increment if the requested filter is a unicast
> address that doesn't match the existing vf->hw_lan_addr. In addition,
> ice_vc_repr_del_mac() does not decrement the vf->num_mac counter. This
> results in the counter becoming out of sync with the actual count.
> 
> As it turns out, vf->num_mac is currently only used in legacy made without
> port representors. The single place where the value is checked is for
> enforcing a filter limit on untrusted VFs.
> 
> Upcoming patches to support VF Live Migration will use this value when
> determining the size of the TLV for MAC address filters. Fix the representor
> mode function to stop incrementing the counter incorrectly.
> 
> Fixes: ac19e03ef780 ("ice: allow process VF opcodes in different ways")
> Signed-off-by: Jacob Keller <jacob.e.keller at intel.com>
> ---
> I am not certain if there is currently a way to trigger a bug from userspace
> due to this incorrect count, but I think it still warrants a net fix.
> ---
>  drivers/net/ethernet/intel/ice/ice_virtchnl.c | 1 -
>  1 file changed, 1 deletion(-)
> 
Tested-by: Sujai Buvaneswaran <sujai.buvaneswaran at intel.com>