cxgb4: fix MPS index overwrite when setting MAC address
authorHerat Ramani <herat@chelsio.com>
Tue, 31 Mar 2020 19:46:09 +0000 (01:16 +0530)
committerDavid S. Miller <davem@davemloft.net>
Wed, 1 Apr 2020 18:15:49 +0000 (11:15 -0700)
commit41aa8561ca3fc5748391f08cc5f3e561923da52c
tree6c18a1ed78b9f9cd18047bcec13795488579f94d
parent744fdc8233f6aa9582ce08a51ca06e59796a3196
cxgb4: fix MPS index overwrite when setting MAC address

cxgb4_update_mac_filt() earlier requests firmware to add a new MAC
address into MPS TCAM. The MPS TCAM index returned by firmware is
stored in pi->xact_addr_filt. However, the saved MPS TCAM index gets
overwritten again with the return value of cxgb4_update_mac_filt(),
which is wrong.

When trying to update to another MAC address later, the wrong MPS TCAM
index is sent to firmware, which causes firmware to return error,
because it's not the same MPS TCAM index that firmware had sent
earlier to driver.

So, fix by removing the wrong overwrite being done after call to
cxgb4_update_mac_filt().

Fixes: 3f8cfd0d95e6 ("cxgb4/cxgb4vf: Program hash region for {t4/t4vf}_change_mac()")
Signed-off-by: Herat Ramani <herat@chelsio.com>
Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c