crypto: caam - remove unneeded edesc zeroization
authorHoria Geantă <horia.geanta@nxp.com>
Wed, 1 Nov 2017 07:13:10 +0000 (09:13 +0200)
committerHerbert Xu <herbert@gondor.apana.org.au>
Mon, 6 Nov 2017 06:45:00 +0000 (14:45 +0800)
Extended descriptor allocation has been changed by
commit dde20ae9d6383 ("crypto: caam - Change kmalloc to kzalloc to avoid residual data")
to provide zeroized memory, meaning we no longer have to sanitize
its members - edesc->src_nents and edesc->dst_dma.

Signed-off-by: Horia Geantă <horia.geanta@nxp.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/caam/caamhash.c

index 698580b60b2f1b8fed004fec52a73dfdf917a390..7b1167437b8844b3ebd4ff784c777fabcf54ee28 100644 (file)
@@ -871,7 +871,6 @@ static int ahash_final_ctx(struct ahash_request *req)
        desc = edesc->hw_desc;
 
        edesc->sec4_sg_bytes = sec4_sg_bytes;
-       edesc->src_nents = 0;
 
        ret = ctx_map_to_sec4_sg(desc, jrdev, state, ctx->ctx_len,
                                 edesc->sec4_sg, DMA_TO_DEVICE);
@@ -1123,7 +1122,6 @@ static int ahash_final_no_ctx(struct ahash_request *req)
                dev_err(jrdev, "unable to map dst\n");
                goto unmap;
        }
-       edesc->src_nents = 0;
 
 #ifdef DEBUG
        print_hex_dump(KERN_ERR, "jobdesc@"__stringify(__LINE__)": ",
@@ -1205,7 +1203,6 @@ static int ahash_update_no_ctx(struct ahash_request *req)
 
                edesc->src_nents = src_nents;
                edesc->sec4_sg_bytes = sec4_sg_bytes;
-               edesc->dst_dma = 0;
 
                ret = buf_map_to_sec4_sg(jrdev, edesc->sec4_sg, state);
                if (ret)
@@ -1417,7 +1414,6 @@ static int ahash_update_first(struct ahash_request *req)
                }
 
                edesc->src_nents = src_nents;
-               edesc->dst_dma = 0;
 
                ret = ahash_edesc_add_src(ctx, edesc, req, mapped_nents, 0, 0,
                                          to_hash);