diff options
author | Eli Cohen <eli@dev.mellanox.co.il> | 2014-11-06 12:51:22 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-11-06 16:40:36 -0500 |
commit | 364d1798efdf13c7f2b9d902228adf8e84f1d963 (patch) | |
tree | 7662a703252362ffbca459f891a34f73e6140c8c /drivers/net/ethernet | |
parent | a158906dd7d4379e85ec371a14edfe1ce5f2318d (diff) | |
download | op-kernel-dev-364d1798efdf13c7f2b9d902228adf8e84f1d963.zip op-kernel-dev-364d1798efdf13c7f2b9d902228adf8e84f1d963.tar.gz |
net/mlx5_core: Fix race on driver load
When events arrive at driver load, the event handler gets called even before
the spinlock and list are initialized. Fix this by moving the initialization
before EQs creation.
Signed-off-by: Eli Cohen <eli@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet')
-rw-r--r-- | drivers/net/ethernet/mellanox/mlx5/core/main.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/main.c b/drivers/net/ethernet/mellanox/mlx5/core/main.c index 3d8e8e4..71b10b2 100644 --- a/drivers/net/ethernet/mellanox/mlx5/core/main.c +++ b/drivers/net/ethernet/mellanox/mlx5/core/main.c @@ -864,14 +864,14 @@ static int init_one(struct pci_dev *pdev, dev->profile = &profile[prof_sel]; dev->event = mlx5_core_event; + INIT_LIST_HEAD(&priv->ctx_list); + spin_lock_init(&priv->ctx_lock); err = mlx5_dev_init(dev, pdev); if (err) { dev_err(&pdev->dev, "mlx5_dev_init failed %d\n", err); goto out; } - INIT_LIST_HEAD(&priv->ctx_list); - spin_lock_init(&priv->ctx_lock); err = mlx5_register_device(dev); if (err) { dev_err(&pdev->dev, "mlx5_register_device failed %d\n", err); |