crypto: chtls - Corrected function call context
authorVinay Kumar Yadav <vinay.yadav@chelsio.com>
Tue, 14 Jan 2020 12:28:47 +0000 (17:58 +0530)
committerHerbert Xu <herbert@gondor.apana.org.au>
Wed, 22 Jan 2020 08:21:08 +0000 (16:21 +0800)
corrected function call context and moved t4_defer_reply
to apropriate location.

Signed-off-by: Vinay Kumar Yadav <vinay.yadav@chelsio.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/chelsio/chtls/chtls_cm.c

index dffa2aa855fdd4f9a9ef67e18d875cf8f2978556..d4674589e1aad8412bc39227f5d70a7013f52d0c 100644 (file)
@@ -1829,6 +1829,20 @@ static void send_defer_abort_rpl(struct chtls_dev *cdev, struct sk_buff *skb)
        kfree_skb(skb);
 }
 
+/*
+ * Add an skb to the deferred skb queue for processing from process context.
+ */
+static void t4_defer_reply(struct sk_buff *skb, struct chtls_dev *cdev,
+                          defer_handler_t handler)
+{
+       DEFERRED_SKB_CB(skb)->handler = handler;
+       spin_lock_bh(&cdev->deferq.lock);
+       __skb_queue_tail(&cdev->deferq, skb);
+       if (skb_queue_len(&cdev->deferq) == 1)
+               schedule_work(&cdev->deferq_task);
+       spin_unlock_bh(&cdev->deferq.lock);
+}
+
 static void send_abort_rpl(struct sock *sk, struct sk_buff *skb,
                           struct chtls_dev *cdev, int status, int queue)
 {
@@ -1843,7 +1857,7 @@ static void send_abort_rpl(struct sock *sk, struct sk_buff *skb,
 
        if (!reply_skb) {
                req->status = (queue << 1);
-               send_defer_abort_rpl(cdev, skb);
+               t4_defer_reply(skb, cdev, send_defer_abort_rpl);
                return;
        }
 
@@ -1862,20 +1876,6 @@ static void send_abort_rpl(struct sock *sk, struct sk_buff *skb,
        cxgb4_ofld_send(cdev->lldi->ports[0], reply_skb);
 }
 
-/*
- * Add an skb to the deferred skb queue for processing from process context.
- */
-static void t4_defer_reply(struct sk_buff *skb, struct chtls_dev *cdev,
-                          defer_handler_t handler)
-{
-       DEFERRED_SKB_CB(skb)->handler = handler;
-       spin_lock_bh(&cdev->deferq.lock);
-       __skb_queue_tail(&cdev->deferq, skb);
-       if (skb_queue_len(&cdev->deferq) == 1)
-               schedule_work(&cdev->deferq_task);
-       spin_unlock_bh(&cdev->deferq.lock);
-}
-
 static void chtls_send_abort_rpl(struct sock *sk, struct sk_buff *skb,
                                 struct chtls_dev *cdev,
                                 int status, int queue)