async_tx/raid6: add missing dma_unmap calls to the async fail case
Dan Williams [Mon, 21 Sep 2009 17:47:40 +0000 (10:47 -0700)]
If we are unable to offload async_mult() or async_sum_product(), then
unmap the buffers before falling through to the synchronous path.

Signed-off-by: Dan Williams <dan.j.williams@intel.com>

crypto/async_tx/async_raid6_recov.c

index 822a42d..6d73dde 100644 (file)
@@ -55,6 +55,13 @@ async_sum_product(struct page *dest, struct page **srcs, unsigned char *coef,
                        async_tx_submit(chan, tx, submit);
                        return tx;
                }
+
+               /* could not get a descriptor, unmap and fall through to
+                * the synchronous path
+                */
+               dma_unmap_page(dev, dma_dest[1], len, DMA_BIDIRECTIONAL);
+               dma_unmap_page(dev, dma_src[0], len, DMA_TO_DEVICE);
+               dma_unmap_page(dev, dma_src[1], len, DMA_TO_DEVICE);
        }
 
        /* run the operation synchronously */
@@ -101,6 +108,12 @@ async_mult(struct page *dest, struct page *src, u8 coef, size_t len,
                        async_tx_submit(chan, tx, submit);
                        return tx;
                }
+
+               /* could not get a descriptor, unmap and fall through to
+                * the synchronous path
+                */
+               dma_unmap_page(dev, dma_dest[1], len, DMA_BIDIRECTIONAL);
+               dma_unmap_page(dev, dma_src[0], len, DMA_TO_DEVICE);
        }
 
        /* no channel available, or failed to allocate a descriptor, so