crypto: inside-secure - acknowledge the result requests all at once
authorAntoine Ténart <antoine.tenart@free-electrons.com>
Thu, 14 Dec 2017 14:26:55 +0000 (15:26 +0100)
committerHerbert Xu <herbert@gondor.apana.org.au>
Fri, 22 Dec 2017 09:03:34 +0000 (20:03 +1100)
This patches moves the result request acknowledgment from a per request
process to acknowledging all the result requests handled at once.

Suggested-by: Ofer Heifetz <oferh@marvell.com>
Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/inside-secure/safexcel.c

index db7ad9d3eeed8e8f9781794d42d3a17f9d5c3af3..36c9a72c47b19c703a963fd5e8d8c158209f8619 100644 (file)
@@ -600,7 +600,7 @@ static inline void safexcel_handle_result_descriptor(struct safexcel_crypto_priv
 {
        struct safexcel_request *sreq;
        struct safexcel_context *ctx;
-       int ret, i, nreq, ndesc = 0, done;
+       int ret, i, nreq, ndesc = 0, tot_descs = 0, done;
        bool should_complete;
 
        nreq = readl(priv->base + EIP197_HIA_RDR(ring) + EIP197_HIA_xDR_PROC_COUNT);
@@ -622,13 +622,9 @@ static inline void safexcel_handle_result_descriptor(struct safexcel_crypto_priv
                if (ndesc < 0) {
                        kfree(sreq);
                        dev_err(priv->dev, "failed to handle result (%d)", ndesc);
-                       goto requests_left;
+                       goto acknowledge;
                }
 
-               writel(EIP197_xDR_PROC_xD_PKT(1) |
-                      EIP197_xDR_PROC_xD_COUNT(ndesc * priv->config.rd_offset),
-                      priv->base + EIP197_HIA_RDR(ring) + EIP197_HIA_xDR_PROC_COUNT);
-
                if (should_complete) {
                        local_bh_disable();
                        sreq->req->complete(sreq->req, ret);
@@ -636,6 +632,14 @@ static inline void safexcel_handle_result_descriptor(struct safexcel_crypto_priv
                }
 
                kfree(sreq);
+               tot_descs += ndesc;
+       }
+
+acknowledge:
+       if (i) {
+               writel(EIP197_xDR_PROC_xD_PKT(i) |
+                      EIP197_xDR_PROC_xD_COUNT(tot_descs * priv->config.rd_offset),
+                      priv->base + EIP197_HIA_RDR(ring) + EIP197_HIA_xDR_PROC_COUNT);
        }
 
 requests_left: