Dear Aleksandr,

Thank you for your patch.

Am 12.03.24 um 09:13 schrieb Aleksandr Loktionov:
Fix counter to be bumped only for new or active filters.

Although it’s a small diffstat, could you please elaborate, and at least describe the problem in more detail.

How can this be tested?

Fixes: 621650cabee5 ("i40e: Refactoring VF MAC filters counting to make more 
reliable")
Signed-off-by: Aleksandr Loktionov <[email protected]>
Reviewed-by: Arkadiusz Kubalewski <[email protected]>
---
  drivers/net/ethernet/intel/i40e/i40e_main.c | 7 +++++--
  1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c 
b/drivers/net/ethernet/intel/i40e/i40e_main.c
index 89a3401..6010a49 100644
--- a/drivers/net/ethernet/intel/i40e/i40e_main.c
+++ b/drivers/net/ethernet/intel/i40e/i40e_main.c
@@ -1257,8 +1257,11 @@ int i40e_count_filters(struct i40e_vsi *vsi)
        int bkt;
        int cnt = 0;
- hash_for_each_safe(vsi->mac_filter_hash, bkt, h, f, hlist)
-               ++cnt;
+       hash_for_each_safe(vsi->mac_filter_hash, bkt, h, f, hlist) {
+               if (f->state == I40E_FILTER_NEW ||
+                   f->state == I40E_FILTER_ACTIVE)
+                       ++cnt;
+       }
return cnt;
  }


Kind regards,

Paul

Reply via email to