bnxt_en: Retain user settings on a VF after RESET_NOTIFY event.
authorVasundhara Volam <vasundhara-v.volam@broadcom.com>
Fri, 30 Aug 2019 03:55:01 +0000 (23:55 -0400)
committerDavid S. Miller <davem@davemloft.net>
Fri, 30 Aug 2019 21:02:19 +0000 (14:02 -0700)
Retain the VF MAC address, default VLAN, TX rate control, trust settings
of VFs after firmware reset.

Signed-off-by: Vasundhara Volam <vasundhara-v.volam@broadcom.com>
Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/bnxt/bnxt.c
drivers/net/ethernet/broadcom/bnxt/bnxt_sriov.c
drivers/net/ethernet/broadcom/bnxt/bnxt_sriov.h

index cd20067fff05b91e376e6b74af09c553756f6919..ff911d66bd748f3ad5fa41883ecf6304f88317bf 100644 (file)
@@ -9211,7 +9211,7 @@ static int bnxt_open(struct net_device *dev)
                        int n = pf->active_vfs;
 
                        if (n)
-                               bnxt_cfg_hw_sriov(bp, &n);
+                               bnxt_cfg_hw_sriov(bp, &n, true);
                }
                bnxt_hwmon_open(bp);
        }
index 4aacfc33a8cc6efaa20b7a6fac191ebc585a5ed9..f6f3454d605979b43f1a7db2ea6caca9bd49b06a 100644 (file)
@@ -470,10 +470,43 @@ static int bnxt_hwrm_func_buf_rgtr(struct bnxt *bp)
        return hwrm_send_message(bp, &req, sizeof(req), HWRM_CMD_TIMEOUT);
 }
 
+/* Caller holds bp->hwrm_cmd_lock mutex lock */
+static void __bnxt_set_vf_params(struct bnxt *bp, int vf_id)
+{
+       struct hwrm_func_cfg_input req = {0};
+       struct bnxt_vf_info *vf;
+
+       vf = &bp->pf.vf[vf_id];
+       bnxt_hwrm_cmd_hdr_init(bp, &req, HWRM_FUNC_CFG, -1, -1);
+       req.fid = cpu_to_le16(vf->fw_fid);
+       req.flags = cpu_to_le32(vf->func_flags);
+
+       if (is_valid_ether_addr(vf->mac_addr)) {
+               req.enables |= cpu_to_le32(FUNC_CFG_REQ_ENABLES_DFLT_MAC_ADDR);
+               memcpy(req.dflt_mac_addr, vf->mac_addr, ETH_ALEN);
+       }
+       if (vf->vlan) {
+               req.enables |= cpu_to_le32(FUNC_CFG_REQ_ENABLES_DFLT_VLAN);
+               req.dflt_vlan = cpu_to_le16(vf->vlan);
+       }
+       if (vf->max_tx_rate) {
+               req.enables |= cpu_to_le32(FUNC_CFG_REQ_ENABLES_MAX_BW);
+               req.max_bw = cpu_to_le32(vf->max_tx_rate);
+#ifdef HAVE_IFLA_TX_RATE
+               req.enables |= cpu_to_le32(FUNC_CFG_REQ_ENABLES_MIN_BW);
+               req.min_bw = cpu_to_le32(vf->min_tx_rate);
+#endif
+       }
+       if (vf->flags & BNXT_VF_TRUST)
+               req.flags |= cpu_to_le32(FUNC_CFG_REQ_FLAGS_TRUSTED_VF_ENABLE);
+
+       _hwrm_send_message(bp, &req, sizeof(req), HWRM_CMD_TIMEOUT);
+}
+
 /* Only called by PF to reserve resources for VFs, returns actual number of
  * VFs configured, or < 0 on error.
  */
-static int bnxt_hwrm_func_vf_resc_cfg(struct bnxt *bp, int num_vfs)
+static int bnxt_hwrm_func_vf_resc_cfg(struct bnxt *bp, int num_vfs, bool reset)
 {
        struct hwrm_func_vf_resource_cfg_input req = {0};
        struct bnxt_hw_resc *hw_resc = &bp->hw_resc;
@@ -545,6 +578,9 @@ static int bnxt_hwrm_func_vf_resc_cfg(struct bnxt *bp, int num_vfs)
 
        mutex_lock(&bp->hwrm_cmd_lock);
        for (i = 0; i < num_vfs; i++) {
+               if (reset)
+                       __bnxt_set_vf_params(bp, i);
+
                req.vf_id = cpu_to_le16(pf->first_vf_id + i);
                rc = _hwrm_send_message(bp, &req, sizeof(req),
                                        HWRM_CMD_TIMEOUT);
@@ -659,15 +695,15 @@ static int bnxt_hwrm_func_cfg(struct bnxt *bp, int num_vfs)
        return rc;
 }
 
-static int bnxt_func_cfg(struct bnxt *bp, int num_vfs)
+static int bnxt_func_cfg(struct bnxt *bp, int num_vfs, bool reset)
 {
        if (BNXT_NEW_RM(bp))
-               return bnxt_hwrm_func_vf_resc_cfg(bp, num_vfs);
+               return bnxt_hwrm_func_vf_resc_cfg(bp, num_vfs, reset);
        else
                return bnxt_hwrm_func_cfg(bp, num_vfs);
 }
 
-int bnxt_cfg_hw_sriov(struct bnxt *bp, int *num_vfs)
+int bnxt_cfg_hw_sriov(struct bnxt *bp, int *num_vfs, bool reset)
 {
        int rc;
 
@@ -677,7 +713,7 @@ int bnxt_cfg_hw_sriov(struct bnxt *bp, int *num_vfs)
                return rc;
 
        /* Reserve resources for VFs */
-       rc = bnxt_func_cfg(bp, *num_vfs);
+       rc = bnxt_func_cfg(bp, *num_vfs, reset);
        if (rc != *num_vfs) {
                if (rc <= 0) {
                        netdev_warn(bp->dev, "Unable to reserve resources for SRIOV.\n");
@@ -758,7 +794,7 @@ static int bnxt_sriov_enable(struct bnxt *bp, int *num_vfs)
        if (rc)
                goto err_out1;
 
-       rc = bnxt_cfg_hw_sriov(bp, num_vfs);
+       rc = bnxt_cfg_hw_sriov(bp, num_vfs, false);
        if (rc)
                goto err_out2;
 
@@ -1144,7 +1180,7 @@ mac_done:
 }
 #else
 
-int bnxt_cfg_hw_sriov(struct bnxt *bp, int *num_vfs)
+int bnxt_cfg_hw_sriov(struct bnxt *bp, int *num_vfs, bool reset)
 {
        if (*num_vfs)
                return -EOPNOTSUPP;
index 0abf18e70feed9c7b7d6918d44681be95fcae7a1..629641bf6fc5719305c252acb7349da2995a2027 100644 (file)
@@ -36,7 +36,7 @@ int bnxt_set_vf_link_state(struct net_device *, int, int);
 int bnxt_set_vf_spoofchk(struct net_device *, int, bool);
 int bnxt_set_vf_trust(struct net_device *dev, int vf_id, bool trust);
 int bnxt_sriov_configure(struct pci_dev *pdev, int num_vfs);
-int bnxt_cfg_hw_sriov(struct bnxt *bp, int *num_vfs);
+int bnxt_cfg_hw_sriov(struct bnxt *bp, int *num_vfs, bool reset);
 void bnxt_sriov_disable(struct bnxt *);
 void bnxt_hwrm_exec_fwd_req(struct bnxt *);
 void bnxt_update_vf_mac(struct bnxt *);