Commit aebfc816 authored by Shannon Nelson's avatar Shannon Nelson Committed by Jeff Kirsher

i40e: don't give up on DCB error after reset

We don't need to give up in the reset/rebuild process if the DCB setup failed,
so handle it here the same as in the probe setup.  Also adjust the log strings
a little to look less scary.

Change-ID: I57308d703047e61d3f1a5e471ea77be232444ca0
Signed-off-by: default avatarShannon Nelson <shannon.nelson@intel.com>
Signed-off-by: default avatarJeff Kirsher <jeffrey.t.kirsher@intel.com>
parent e240f674
...@@ -4533,7 +4533,8 @@ static int i40e_init_pf_dcb(struct i40e_pf *pf) ...@@ -4533,7 +4533,8 @@ static int i40e_init_pf_dcb(struct i40e_pf *pf)
"DCBX offload is supported for this PF.\n"); "DCBX offload is supported for this PF.\n");
} }
} else { } else {
dev_info(&pf->pdev->dev, "AQ Querying DCB configuration failed: %d\n", dev_info(&pf->pdev->dev,
"AQ Querying DCB configuration failed: aq_err %d\n",
pf->hw.aq.asq_last_status); pf->hw.aq.asq_last_status);
} }
...@@ -6188,8 +6189,9 @@ static void i40e_reset_and_rebuild(struct i40e_pf *pf, bool reinit) ...@@ -6188,8 +6189,9 @@ static void i40e_reset_and_rebuild(struct i40e_pf *pf, bool reinit)
#ifdef CONFIG_I40E_DCB #ifdef CONFIG_I40E_DCB
ret = i40e_init_pf_dcb(pf); ret = i40e_init_pf_dcb(pf);
if (ret) { if (ret) {
dev_info(&pf->pdev->dev, "init_pf_dcb failed: %d\n", ret); dev_info(&pf->pdev->dev, "DCB init failed %d, disabled\n", ret);
goto end_core_reset; pf->flags &= ~I40E_FLAG_DCB_CAPABLE;
/* Continue without DCB enabled */
} }
#endif /* CONFIG_I40E_DCB */ #endif /* CONFIG_I40E_DCB */
#ifdef I40E_FCOE #ifdef I40E_FCOE
...@@ -9269,7 +9271,7 @@ static int i40e_probe(struct pci_dev *pdev, const struct pci_device_id *ent) ...@@ -9269,7 +9271,7 @@ static int i40e_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
#ifdef CONFIG_I40E_DCB #ifdef CONFIG_I40E_DCB
err = i40e_init_pf_dcb(pf); err = i40e_init_pf_dcb(pf);
if (err) { if (err) {
dev_info(&pdev->dev, "init_pf_dcb failed: %d\n", err); dev_info(&pdev->dev, "DCB init failed %d, disabled\n", err);
pf->flags &= ~I40E_FLAG_DCB_CAPABLE; pf->flags &= ~I40E_FLAG_DCB_CAPABLE;
/* Continue without DCB enabled */ /* Continue without DCB enabled */
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment