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

Sagi Grimberg [off-list ref] writes:

quoted

+     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