Re: [PATCH v12 10/26] nvme-tcp: Deal with netdevice DOWN events

[Aurelien Aptel <aaptel@nvidia.com>]

Sagi Grimberg <sagi@grimberg.me> writes:
>> +     switch (event) {
>> +     case NETDEV_GOING_DOWN:
>> +             mutex_lock(&nvme_tcp_ctrl_mutex);
>> +             list_for_each_entry(ctrl, &nvme_tcp_ctrl_list, list) {
>> +                     if (ndev == ctrl->offloading_netdev)
>> +                             nvme_tcp_error_recovery(&ctrl->ctrl);
>> +             }
>> +             mutex_unlock(&nvme_tcp_ctrl_mutex);
>> +             flush_workqueue(nvme_reset_wq);
>
> In what context is this called? because every time we flush a workqueue,
> lockdep finds another reason to complain about something...

Thanks for highlighting this, we re-checked it and we found that we are
covered by nvme_tcp_error_recovery(), we can remove the
flush_workqueue() call above.

> Otherwise looks good,
> Reviewed-by: Sagi Grimberg <sagi@grimberg.me>

Thanks