diff options
author | David S. Miller <davem@davemloft.net> | 2011-05-19 18:41:03 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-05-19 18:41:03 -0400 |
commit | 2a64255019e13dd975f1487cec32b873bcb76da6 (patch) | |
tree | 81c6fef1eb709a528bec87e9e26070a06050a434 /net/rxrpc | |
parent | c9d10c497c3728a5c798c11486dccdc01b2092a8 (diff) |
rxrpc: Fix set but unused variable 'usage' in rxrpc_get_peer().
I backed off from trying to just eliminate this variable, since
transforming atomic_inc_return() into atomic_inc() takes away
the memory barriers.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/rxrpc')
-rw-r--r-- | net/rxrpc/ar-peer.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/rxrpc/ar-peer.c b/net/rxrpc/ar-peer.c index b6ff06351d67..2754f098d436 100644 --- a/net/rxrpc/ar-peer.c +++ b/net/rxrpc/ar-peer.c | |||
@@ -157,6 +157,7 @@ struct rxrpc_peer *rxrpc_get_peer(struct sockaddr_rxrpc *srx, gfp_t gfp) | |||
157 | /* we can now add the new candidate to the list */ | 157 | /* we can now add the new candidate to the list */ |
158 | peer = candidate; | 158 | peer = candidate; |
159 | candidate = NULL; | 159 | candidate = NULL; |
160 | usage = atomic_read(&peer->usage); | ||
160 | 161 | ||
161 | list_add_tail(&peer->link, &rxrpc_peers); | 162 | list_add_tail(&peer->link, &rxrpc_peers); |
162 | write_unlock_bh(&rxrpc_peer_lock); | 163 | write_unlock_bh(&rxrpc_peer_lock); |
@@ -171,7 +172,7 @@ success: | |||
171 | &peer->srx.transport.sin.sin_addr, | 172 | &peer->srx.transport.sin.sin_addr, |
172 | ntohs(peer->srx.transport.sin.sin_port)); | 173 | ntohs(peer->srx.transport.sin.sin_port)); |
173 | 174 | ||
174 | _leave(" = %p {u=%d}", peer, atomic_read(&peer->usage)); | 175 | _leave(" = %p {u=%d}", peer, usage); |
175 | return peer; | 176 | return peer; |
176 | 177 | ||
177 | /* we found the peer in the list immediately */ | 178 | /* we found the peer in the list immediately */ |