diff options
| author | Jason Wang <jasowang@redhat.com> | 2011-01-17 03:11:17 -0500 |
|---|---|---|
| committer | Michael S. Tsirkin <mst@redhat.com> | 2011-03-13 17:08:04 -0400 |
| commit | 783e3988544b94ff3918666b9f36866ac547fba1 (patch) | |
| tree | e05c17f7e368490e3d34750cc2f6b8e2092ef1e5 /drivers/vhost | |
| parent | 94249369e9930276e30087da205349a55478cbb5 (diff) | |
vhost: lock receive queue, not the socket
vhost takes a sock lock to try and prevent
the skb from being pulled from the receive queue
after skb_peek. However this is not the right lock to use for that,
sk_receive_queue.lock is. Fix that up.
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Diffstat (limited to 'drivers/vhost')
| -rw-r--r-- | drivers/vhost/net.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/drivers/vhost/net.c b/drivers/vhost/net.c index 0329c411bbf1..57203014c457 100644 --- a/drivers/vhost/net.c +++ b/drivers/vhost/net.c | |||
| @@ -213,12 +213,13 @@ static int peek_head_len(struct sock *sk) | |||
| 213 | { | 213 | { |
| 214 | struct sk_buff *head; | 214 | struct sk_buff *head; |
| 215 | int len = 0; | 215 | int len = 0; |
| 216 | unsigned long flags; | ||
| 216 | 217 | ||
| 217 | lock_sock(sk); | 218 | spin_lock_irqsave(&sk->sk_receive_queue.lock, flags); |
| 218 | head = skb_peek(&sk->sk_receive_queue); | 219 | head = skb_peek(&sk->sk_receive_queue); |
| 219 | if (head) | 220 | if (likely(head)) |
| 220 | len = head->len; | 221 | len = head->len; |
| 221 | release_sock(sk); | 222 | spin_unlock_irqrestore(&sk->sk_receive_queue.lock, flags); |
| 222 | return len; | 223 | return len; |
| 223 | } | 224 | } |
| 224 | 225 | ||
