From mboxrd@z Thu Jan 1 00:00:00 1970 From: "tip-bot for Paul E. McKenney" Subject: [tip:core/rcu] drivers/vhost: Replace synchronize_rcu_bh() with synchronize_rcu() Date: Tue, 4 Dec 2018 14:39:33 -0800 Message-ID: Reply-To: tglx@linutronix.de, linux-kernel@vger.kernel.org, mst@redhat.com, virtualization@lists.linux-foundation.org, jasowang@redhat.com, netdev@vger.kernel.org, mingo@kernel.org, paulmck@linux.ibm.com, hpa@zytor.com, kvm@vger.kernel.org Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org To: linux-tip-commits@vger.kernel.org Cc: kvm@vger.kernel.org, mst@redhat.com, netdev@vger.kernel.org, virtualization@lists.linux-foundation.org, hpa@zytor.com, tglx@linutronix.de, paulmck@linux.ibm.com, mingo@kernel.org List-Id: virtualization@lists.linuxfoundation.org Commit-ID: d05faa5f1ac50beef77b4ceba0e8e157d41146e2 Gitweb: https://git.kernel.org/tip/d05faa5f1ac50beef77b4ceba0e8e157d41146e2 Author: Paul E. McKenney AuthorDate: Mon, 5 Nov 2018 17:14:53 -0800 Committer: Paul E. McKenney 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 Cc: "Michael S. Tsirkin" Cc: Jason Wang Cc: Cc: Cc: --- 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);