> From: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> diff --git a/drivers/infiniband/hw/hfi1/vnic_main.c
> b/drivers/infiniband/hw/hfi1/vnic_main.c
> index b183c56b7b6a4..f89d0cb1c7204 100644
> --- a/drivers/infiniband/hw/hfi1/vnic_main.c
> +++ b/drivers/infiniband/hw/hfi1/vnic_main.c
> @@ -512,7 +512,8 @@ static int hfi1_vnic_init(struct hfi1_vnic_vport_info
> *vinfo)
> goto txreq_fail;
> }
>
> - if (hfi1_netdev_rx_init(dd)) {
> + rc = hfi1_netdev_rx_init(dd);
> + if (rc) {
> dd_dev_err(dd, "Unable to initialize netdev contexts\n");
> goto alloc_fail;
> }
Dan,
This is definitely wrong, but another call to hfi1_netdev_rx_init() exists in hfi1_vnic_up() that needs to be fixed.
Can you address that too?
Mike
