tip-bot for Paul E. McKenney
2018-Dec-04 22:39 UTC
[tip:core/rcu] drivers/vhost: Replace synchronize_rcu_bh() with synchronize_rcu()
Commit-ID: d05faa5f1ac50beef77b4ceba0e8e157d41146e2 Gitweb: https://git.kernel.org/tip/d05faa5f1ac50beef77b4ceba0e8e157d41146e2 Author: Paul E. McKenney <paulmck at linux.ibm.com> AuthorDate: Mon, 5 Nov 2018 17:14:53 -0800 Committer: Paul E. McKenney <paulmck at linux.ibm.com> CommitDate: Tue, 27 Nov 2018 09:21:37 -0800 drivers/vhost: Replace synchronize_rcu_bh() with synchronize_rcu() Now that synchronize_rcu() waits for bh-disable regions of code as well as RCU read-side critical sections, synchronize_rcu_bh() can be replaced by synchronize_rcu(). This commit therefore makes this change. Signed-off-by: Paul E. McKenney <paulmck at linux.ibm.com> Cc: "Michael S. Tsirkin" <mst at redhat.com> Cc: Jason Wang <jasowang at redhat.com> Cc: <kvm at vger.kernel.org> Cc: <virtualization at lists.linux-foundation.org> Cc: <netdev at vger.kernel.org> --- drivers/vhost/net.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/vhost/net.c b/drivers/vhost/net.c index ab11b2bee273..564ead864028 100644 --- a/drivers/vhost/net.c +++ b/drivers/vhost/net.c @@ -1359,7 +1359,7 @@ static int vhost_net_release(struct inode *inode, struct file *f) if (rx_sock) sockfd_put(rx_sock); /* Make sure no callbacks are outstanding */ - synchronize_rcu_bh(); + synchronize_rcu(); /* We do an extra flush before freeing memory, * since jobs can re-queue themselves. */ vhost_net_flush(n);
Apparently Analagous Threads
- [PATCH net v2] vhost: fix a theoretical race in device cleanup
- [PATCH net v2] vhost: fix a theoretical race in device cleanup
- [PATCH net 3/3] vhost: fix a theoretical race in device cleanup
- [PATCH] Revert "vhost: fix release path lockdep checks"
- [PATCH RESEND] Revert "vhost: fix release path lockdep checks"