| From foo@baz Fri Feb 23 11:48:40 CET 2018 |
| From: Christophe JAILLET <christophe.jaillet@wanadoo.fr> |
| Date: Fri, 17 Nov 2017 22:37:53 +0100 |
| Subject: dmaengine: ioat: Fix error handling path |
| |
| From: Christophe JAILLET <christophe.jaillet@wanadoo.fr> |
| |
| |
| [ Upstream commit 5c9afbda911ce20b3f2181d1e440a0222e1027dd ] |
| |
| If the last test in 'ioat_dma_self_test()' fails, we must release all |
| the allocated resources and not just part of them. |
| |
| Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr> |
| Acked-by: Dave Jiang <dave.jiang@intel.com> |
| Signed-off-by: Vinod Koul <vinod.koul@intel.com> |
| Signed-off-by: Sasha Levin <alexander.levin@microsoft.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| --- |
| drivers/dma/ioat/init.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/drivers/dma/ioat/init.c |
| +++ b/drivers/dma/ioat/init.c |
| @@ -388,7 +388,7 @@ static int ioat_dma_self_test(struct ioa |
| if (memcmp(src, dest, IOAT_TEST_SIZE)) { |
| dev_err(dev, "Self-test copy failed compare, disabling\n"); |
| err = -ENODEV; |
| - goto free_resources; |
| + goto unmap_dma; |
| } |
| |
| unmap_dma: |
