Looks good.
> -----Original Message-----
> From: Geanta Neag Horia Ioan-B05471
> Sent: Friday, May 10, 2013 5:39 PM
> To: linux-crypto@xxxxxxxxxxxxxxx
> Cc: Herbert Xu; stable@xxxxxxxxxxxxxxx; Phillips Kim-R1AAHA; Garg Vakul-
> B16394
> Subject: [PATCH] crypto: caam - fix inconsistent assoc dma mapping
> direction
>
> req->assoc is dma mapped BIDIRECTIONAL and unmapped TO_DEVICE.
> Since it is read-only for the device, use TO_DEVICE both for mapping and
> unmapping.
>
> Cc: <stable@xxxxxxxxxxxxxxx> # 3.9, 3.8
> Signed-off-by: Horia Geanta <horia.geanta@xxxxxxxxxxxxx>
> ---
> drivers/crypto/caam/caamalg.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/crypto/caam/caamalg.c
> b/drivers/crypto/caam/caamalg.c index 42420fb..bf7da50 100644
> --- a/drivers/crypto/caam/caamalg.c
> +++ b/drivers/crypto/caam/caamalg.c
> @@ -1154,7 +1154,7 @@ static struct aead_edesc *aead_edesc_alloc(struct
> aead_request *req,
> dst_nents = sg_count(req->dst, req->cryptlen, &dst_chained);
>
> sgc = dma_map_sg_chained(jrdev, req->assoc, assoc_nents ? : 1,
> - DMA_BIDIRECTIONAL, assoc_chained);
> + DMA_TO_DEVICE, assoc_chained);
> if (likely(req->src == req->dst)) {
> sgc = dma_map_sg_chained(jrdev, req->src, src_nents ? : 1,
> DMA_BIDIRECTIONAL, src_chained);
> @@ -1336,7 +1336,7 @@ static struct aead_edesc
> *aead_giv_edesc_alloc(struct aead_givcrypt_request
> dst_nents = sg_count(req->dst, req->cryptlen, &dst_chained);
>
> sgc = dma_map_sg_chained(jrdev, req->assoc, assoc_nents ? : 1,
> - DMA_BIDIRECTIONAL, assoc_chained);
> + DMA_TO_DEVICE, assoc_chained);
> if (likely(req->src == req->dst)) {
> sgc = dma_map_sg_chained(jrdev, req->src, src_nents ? : 1,
> DMA_BIDIRECTIONAL, src_chained);
> --
> 1.7.7.6
--
To unsubscribe from this list: send the line "unsubscribe linux-crypto" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
