diff options
| author |   Michael Chan <michael.chan@broadcom.com> | 2020-01-17 00:32:47 -0500 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2020-01-18 14:38:29 +0100 |
| commit | d061b2411d5f3d6272187ab734ce0640827fca13 (patch) | |
| tree | 382730a0bc17f312dbe442ceee5445b8a71aad93 /drivers/net/ethernet/broadcom/bnxt/bnxt_vfr.c | |
| parent | bnxt_en: Fix ipv6 RFS filter matching logic. (diff) | |
| download | linux-dev-d061b2411d5f3d6272187ab734ce0640827fca13.tar.xz linux-dev-d061b2411d5f3d6272187ab734ce0640827fca13.zip | |
bnxt_en: Do not treat DSN (Digital Serial Number) read failure as fatal.
DSN read can fail, for example on a kdump kernel without PCIe extended
config space support. If DSN read fails, don't set the
BNXT_FLAG_DSN_VALID flag and continue loading. Check the flag
to see if the stored DSN is valid before using it. Only VF reps
creation should fail without valid DSN.
Fixes: 03213a996531 ("bnxt: move bp->switch_id initialization to PF probe")
Reported-by: Marc Smith <msmith626@gmail.com>
Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
| -rw-r--r-- | drivers/net/ethernet/broadcom/bnxt/bnxt_vfr.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_vfr.c b/drivers/net/ethernet/broadcom/bnxt/bnxt_vfr.c index f9bf7d7250ab..b010b34cdaf8 100644 --- a/drivers/net/ethernet/broadcom/bnxt/bnxt_vfr.c +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_vfr.c @@ -398,6 +398,9 @@ static int bnxt_vf_reps_create(struct bnxt *bp) struct net_device *dev; int rc, i; + if (!(bp->flags & BNXT_FLAG_DSN_VALID)) + return -ENODEV; + bp->vf_reps = kcalloc(num_vfs, sizeof(vf_rep), GFP_KERNEL); if (!bp->vf_reps) return -ENOMEM; |