crypto: mv_cesa - Fix situation where the dest sglist is organized differently than...
authorUri Simchoni <uri@jdland.co.il>
Thu, 8 Apr 2010 16:26:34 +0000 (19:26 +0300)
committerHerbert Xu <herbert@gondor.apana.org.au>
Tue, 13 Apr 2010 08:50:47 +0000 (16:50 +0800)
Bugfix for situations where the destination scatterlist has a different
buffer structure than the source scatterlist (e.g. source has one 2K
buffer and dest has 2 1K buffers)

Signed-off-by: Uri Simchoni <uri@jdland.co.il>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/mv_cesa.c

index 37d9f0688e7575a3e366f6bb9eda5adc5db807b5..018a95ce0c9b0d6e6a13bff1522630799bc445b3 100644 (file)
@@ -242,6 +242,8 @@ static void dequeue_complete_req(void)
        struct ablkcipher_request *req = cpg->cur_req;
        void *buf;
        int ret;
+       int need_copy_len = cpg->p.crypt_len;
+       int sram_offset = 0;
 
        cpg->p.total_req_bytes += cpg->p.crypt_len;
        do {
@@ -257,14 +259,16 @@ static void dequeue_complete_req(void)
                buf = cpg->p.dst_sg_it.addr;
                buf += cpg->p.dst_start;
 
-               dst_copy = min(cpg->p.crypt_len, cpg->p.sg_dst_left);
-
-               memcpy(buf, cpg->sram + SRAM_DATA_OUT_START, dst_copy);
+               dst_copy = min(need_copy_len, cpg->p.sg_dst_left);
 
+               memcpy(buf,
+                      cpg->sram + SRAM_DATA_OUT_START + sram_offset,
+                      dst_copy);
+               sram_offset += dst_copy;
                cpg->p.sg_dst_left -= dst_copy;
-               cpg->p.crypt_len -= dst_copy;
+               need_copy_len -= dst_copy;
                cpg->p.dst_start += dst_copy;
-       } while (cpg->p.crypt_len > 0);
+       } while (need_copy_len > 0);
 
        BUG_ON(cpg->eng_st != ENGINE_W_DEQUEUE);
        if (cpg->p.total_req_bytes < req->nbytes) {