diff options
| author | David Howells <dhowells@redhat.com> | 2020-01-24 10:21:15 +0000 |
|---|---|---|
| committer | David Howells <dhowells@redhat.com> | 2022-12-01 13:36:42 +0000 |
| commit | 3dd9c8b5f09fd24652729a3da5c5efa3ec2c4590 (patch) | |
| tree | 7325670c28397dae62cf553efe6fba29ac39e921 /net/rxrpc/output.c | |
| parent | 5e6ef4f1017c7f844e305283bbd8875af475e2fc (diff) | |
| download | lwn-3dd9c8b5f09fd24652729a3da5c5efa3ec2c4590.tar.gz lwn-3dd9c8b5f09fd24652729a3da5c5efa3ec2c4590.zip | |
rxrpc: Remove the _bh annotation from all the spinlocks
None of the spinlocks in rxrpc need a _bh annotation now as the RCU
callback routines no longer take spinlocks and the bulk of the packet
wrangling code is now run in the I/O thread, not softirq context.
Signed-off-by: David Howells <dhowells@redhat.com>
cc: Marc Dionne <marc.dionne@auristor.com>
cc: linux-afs@lists.infradead.org
Diffstat (limited to 'net/rxrpc/output.c')
| -rw-r--r-- | net/rxrpc/output.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/net/rxrpc/output.c b/net/rxrpc/output.c index 2ea1fa1b8a6f..e5d715b855fc 100644 --- a/net/rxrpc/output.c +++ b/net/rxrpc/output.c @@ -286,9 +286,9 @@ void rxrpc_transmit_ack_packets(struct rxrpc_local *local) if (list_empty(&local->ack_tx_queue)) return; - spin_lock_bh(&local->ack_tx_lock); + spin_lock(&local->ack_tx_lock); list_splice_tail_init(&local->ack_tx_queue, &queue); - spin_unlock_bh(&local->ack_tx_lock); + spin_unlock(&local->ack_tx_lock); while (!list_empty(&queue)) { struct rxrpc_txbuf *txb = @@ -296,9 +296,9 @@ void rxrpc_transmit_ack_packets(struct rxrpc_local *local) ret = rxrpc_send_ack_packet(local, txb); if (ret < 0 && ret != -ECONNRESET) { - spin_lock_bh(&local->ack_tx_lock); + spin_lock(&local->ack_tx_lock); list_splice_init(&queue, &local->ack_tx_queue); - spin_unlock_bh(&local->ack_tx_lock); + spin_unlock(&local->ack_tx_lock); break; } |