提交 89f3e4a5 编写于 作者: P Preethi Banala 提交者: Jeff Kirsher

ice: Do not bail out when filter already exists

If filter already exists, do not go through error path flow but instead
continue to process rest of the function. Hence have an appropriate check
after adding MAC filters.
Signed-off-by: NPreethi Banala <preethi.banala@intel.com>
Signed-off-by: NAnirudh Venkataramanan <anirudh.venkataramanan@intel.com>
Tested-by: NAndrew Bowers <andrewx.bowers@intel.com>
Signed-off-by: NJeff Kirsher <jeffrey.t.kirsher@intel.com>
上级 4e1af7bf
...@@ -260,7 +260,11 @@ static int ice_vsi_sync_fltr(struct ice_vsi *vsi) ...@@ -260,7 +260,11 @@ static int ice_vsi_sync_fltr(struct ice_vsi *vsi)
/* Add mac addresses in the sync list */ /* Add mac addresses in the sync list */
status = ice_add_mac(hw, &vsi->tmp_sync_list); status = ice_add_mac(hw, &vsi->tmp_sync_list);
ice_free_fltr_list(dev, &vsi->tmp_sync_list); ice_free_fltr_list(dev, &vsi->tmp_sync_list);
if (status) { /* If filter is added successfully or already exists, do not go into
* 'if' condition and report it as error. Instead continue processing
* rest of the function.
*/
if (status && status != ICE_ERR_ALREADY_EXISTS) {
netdev_err(netdev, "Failed to add MAC filters\n"); netdev_err(netdev, "Failed to add MAC filters\n");
/* If there is no more space for new umac filters, vsi /* If there is no more space for new umac filters, vsi
* should go into promiscuous mode. There should be some * should go into promiscuous mode. There should be some
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册