net/af_iucv: don't lookup the socket on TX notification

Whoever called iucv_sk(sk)->sk_txnotify() must already know that they're
dealing with an af_iucv socket.

Signed-off-by: Julian Wiedmann <jwi@linux.ibm.com>
Acked-by: Willem de Bruijn <willemb@google.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

diff --git a/net/iucv/af_iucv.c b/net/iucv/af_iucv.c
index 427a1abce0a8ab2d9dea673449b913d42ceb6180..8683b6939f459bab761b05cd2764b1bea4aa544c 100644 (file)
--- a/net/iucv/af_iucv.c
+++ b/net/iucv/af_iucv.c
@@ -2134,23 +2134,14 @@ static int afiucv_hs_rcv(struct sk_buff *skb, struct net_device *dev,
 static void afiucv_hs_callback_txnotify(struct sk_buff *skb,
                                        enum iucv_tx_notify n)
 {
-       struct sock *isk = skb->sk;
-       struct sock *sk = NULL;
-       struct iucv_sock *iucv = NULL;
+       struct iucv_sock *iucv = iucv_sk(skb->sk);
+       struct sock *sk = skb->sk;
        struct sk_buff_head *list;
        struct sk_buff *list_skb;
        struct sk_buff *nskb;
        unsigned long flags;
 
-       read_lock_irqsave(&iucv_sk_list.lock, flags);
-       sk_for_each(sk, &iucv_sk_list.head)
-               if (sk == isk) {
-                       iucv = iucv_sk(sk);
-                       break;
-               }
-       read_unlock_irqrestore(&iucv_sk_list.lock, flags);
-
-       if (!iucv || sock_flag(sk, SOCK_ZAPPED))
+       if (sock_flag(sk, SOCK_ZAPPED))
                return;
 
        list = &iucv->send_skb_q;