diff options
author | Igor Druzhinin <igor.druzhinin@citrix.com> | 2017-01-17 15:49:38 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-01-18 15:11:20 -0500 |
commit | f16f1df65f1cf139ff9e9f84661e6573d6bb27fc (patch) | |
tree | 6f3d46c2653ace73426558feacde02c97c7adbeb /drivers/net/xen-netback/xenbus.c | |
parent | 9a6cdf52b85ea5fb21d2bb31e4a7bc61b79923a7 (diff) |
xen-netback: protect resource cleaning on XenBus disconnect
vif->lock is used to protect statistics gathering agents from using the
queue structure during cleaning.
Signed-off-by: Igor Druzhinin <igor.druzhinin@citrix.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
Reviewed-by: Paul Durrant <paul.durrant@citrix.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/xen-netback/xenbus.c')
-rw-r--r-- | drivers/net/xen-netback/xenbus.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/net/xen-netback/xenbus.c b/drivers/net/xen-netback/xenbus.c index 34e4af9272ea..85b742e1c42f 100644 --- a/drivers/net/xen-netback/xenbus.c +++ b/drivers/net/xen-netback/xenbus.c | |||
@@ -503,9 +503,11 @@ static void backend_disconnect(struct backend_info *be) | |||
503 | for (queue_index = 0; queue_index < be->vif->num_queues; ++queue_index) | 503 | for (queue_index = 0; queue_index < be->vif->num_queues; ++queue_index) |
504 | xenvif_deinit_queue(&be->vif->queues[queue_index]); | 504 | xenvif_deinit_queue(&be->vif->queues[queue_index]); |
505 | 505 | ||
506 | spin_lock(&be->vif->lock); | ||
506 | vfree(be->vif->queues); | 507 | vfree(be->vif->queues); |
507 | be->vif->num_queues = 0; | 508 | be->vif->num_queues = 0; |
508 | be->vif->queues = NULL; | 509 | be->vif->queues = NULL; |
510 | spin_unlock(&be->vif->lock); | ||
509 | 511 | ||
510 | xenvif_disconnect_ctrl(be->vif); | 512 | xenvif_disconnect_ctrl(be->vif); |
511 | } | 513 | } |