crypto: chelsio - This fixes the libkcapi's cbc(aes) aio fail test cases

The libkcapi "cbc(aes)" failed tests are
symmetric asynchronous cipher one shot multiple test,
symmetric asynchronous cipher stream multiple test,
Symmetric asynchronous cipher vmsplice multiple test

In this patch a wait_for_completion is added in the chcr_aes_encrypt function,
which completes when the response of comes from the hardware.
This adds serialization for encryption in cbc(aes) aio case.

Signed-off-by: Ayush Sawal <ayush.sawal@chelsio.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>

diff --git a/drivers/crypto/chelsio/chcr_algo.c b/drivers/crypto/chelsio/chcr_algo.c
index b4b9b22125d17773df2ab9cac17c72328eb9d772..699e3053895a824f7ef015165c66e794dac74e71 100644 (file)
--- a/drivers/crypto/chelsio/chcr_algo.c
+++ b/drivers/crypto/chelsio/chcr_algo.c
@@ -1102,6 +1102,7 @@ static int chcr_handle_cipher_resp(struct skcipher_request *req,
                                   unsigned char *input, int err)
 {
        struct crypto_skcipher *tfm = crypto_skcipher_reqtfm(req);
+       struct chcr_context *ctx = c_ctx(tfm);
        struct uld_ctx *u_ctx = ULD_CTX(c_ctx(tfm));
        struct ablk_ctx *ablkctx = ABLK_CTX(c_ctx(tfm));
        struct sk_buff *skb;
@@ -1166,10 +1167,20 @@ static int chcr_handle_cipher_resp(struct skcipher_request *req,
        chcr_send_wr(skb);
        reqctx->last_req_len = bytes;
        reqctx->processed += bytes;
+       if (get_cryptoalg_subtype(tfm) ==
+               CRYPTO_ALG_SUB_TYPE_CBC && req->base.flags ==
+                       CRYPTO_TFM_REQ_MAY_SLEEP ) {
+               complete(&ctx->cbc_aes_aio_done);
+       }
        return 0;
 unmap:
        chcr_cipher_dma_unmap(&ULD_CTX(c_ctx(tfm))->lldi.pdev->dev, req);
 complete:
+       if (get_cryptoalg_subtype(tfm) ==
+               CRYPTO_ALG_SUB_TYPE_CBC && req->base.flags ==
+                       CRYPTO_TFM_REQ_MAY_SLEEP ) {
+               complete(&ctx->cbc_aes_aio_done);
+       }
        chcr_dec_wrcount(dev);
        req->base.complete(&req->base, err);
        return err;
@@ -1289,6 +1300,7 @@ error:
 static int chcr_aes_encrypt(struct skcipher_request *req)
 {
        struct crypto_skcipher *tfm = crypto_skcipher_reqtfm(req);
+       struct chcr_context *ctx;
        struct chcr_dev *dev = c_ctx(tfm)->dev;
        struct sk_buff *skb = NULL;
        int err, isfull = 0;
@@ -1313,6 +1325,12 @@ static int chcr_aes_encrypt(struct skcipher_request *req)
        skb->dev = u_ctx->lldi.ports[0];
        set_wr_txq(skb, CPL_PRIORITY_DATA, c_ctx(tfm)->tx_qidx);
        chcr_send_wr(skb);
+       if (get_cryptoalg_subtype(tfm) ==
+               CRYPTO_ALG_SUB_TYPE_CBC && req->base.flags ==
+                       CRYPTO_TFM_REQ_MAY_SLEEP ) {
+                       ctx=c_ctx(tfm);
+                       wait_for_completion(&ctx->cbc_aes_aio_done);
+        }
        return isfull ? -EBUSY : -EINPROGRESS;
 error:
        chcr_dec_wrcount(dev);
@@ -1401,7 +1419,7 @@ static int chcr_init_tfm(struct crypto_skcipher *tfm)
                pr_err("failed to allocate fallback for %s\n", alg->base.cra_name);
                return PTR_ERR(ablkctx->sw_cipher);
        }
-
+       init_completion(&ctx->cbc_aes_aio_done);
        crypto_skcipher_set_reqsize(tfm, sizeof(struct chcr_skcipher_req_ctx));
 
        return chcr_device_init(ctx);

diff --git a/drivers/crypto/chelsio/chcr_crypto.h b/drivers/crypto/chelsio/chcr_crypto.h
index 6db2df8c8a050d085113b5f07e962226f2de35a5..9207d88c55384401fd8a18e1a153802886c5110c 100644 (file)
--- a/drivers/crypto/chelsio/chcr_crypto.h
+++ b/drivers/crypto/chelsio/chcr_crypto.h
@@ -254,6 +254,7 @@ struct chcr_context {
        unsigned char rx_qidx;
        unsigned char tx_chan_id;
        unsigned char pci_chan_id;
+       struct completion cbc_aes_aio_done;
        struct __crypto_ctx crypto_ctx[0];
 };