ibmvnic: register netdev after init of adapter
commit 570425f8c7c18b14fa8a2a58a0adb431968ad118 upstream.
Finish initializing the adapter before registering netdev so state
is consistent.
Fixes: c26eba03e4
("ibmvnic: Update reset infrastructure to support tunable parameters")
Signed-off-by: Sukadev Bhattiprolu <sukadev@linux.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:

committed by
Greg Kroah-Hartman

parent
6e0f986032
commit
92b791771a
@@ -5383,6 +5383,12 @@ static int ibmvnic_probe(struct vio_dev *dev, const struct vio_device_id *id)
|
|||||||
goto ibmvnic_dev_file_err;
|
goto ibmvnic_dev_file_err;
|
||||||
|
|
||||||
netif_carrier_off(netdev);
|
netif_carrier_off(netdev);
|
||||||
|
|
||||||
|
adapter->state = VNIC_PROBED;
|
||||||
|
|
||||||
|
adapter->wait_for_reset = false;
|
||||||
|
adapter->last_reset_time = jiffies;
|
||||||
|
|
||||||
rc = register_netdev(netdev);
|
rc = register_netdev(netdev);
|
||||||
if (rc) {
|
if (rc) {
|
||||||
dev_err(&dev->dev, "failed to register netdev rc=%d\n", rc);
|
dev_err(&dev->dev, "failed to register netdev rc=%d\n", rc);
|
||||||
@@ -5390,10 +5396,6 @@ static int ibmvnic_probe(struct vio_dev *dev, const struct vio_device_id *id)
|
|||||||
}
|
}
|
||||||
dev_info(&dev->dev, "ibmvnic registered\n");
|
dev_info(&dev->dev, "ibmvnic registered\n");
|
||||||
|
|
||||||
adapter->state = VNIC_PROBED;
|
|
||||||
|
|
||||||
adapter->wait_for_reset = false;
|
|
||||||
adapter->last_reset_time = jiffies;
|
|
||||||
return 0;
|
return 0;
|
||||||
|
|
||||||
ibmvnic_register_fail:
|
ibmvnic_register_fail:
|
||||||
|
Reference in New Issue
Block a user