Message ID | 20181029100808.31923-1-alexandru.ardelean@analog.com (mailing list archive) |
---|---|
State | Accepted |
Headers | show
Return-Path: <dmaengine-owner@kernel.org> Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 5972C14BD for <patchwork-dmaengine@patchwork.kernel.org>; Mon, 29 Oct 2018 10:08:26 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 48246290BC for <patchwork-dmaengine@patchwork.kernel.org>; Mon, 29 Oct 2018 10:08:26 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 3C330294FD; Mon, 29 Oct 2018 10:08:26 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.9 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id B2DA3290BC for <patchwork-dmaengine@patchwork.kernel.org>; Mon, 29 Oct 2018 10:08:25 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729634AbeJ2S4X (ORCPT <rfc822;patchwork-dmaengine@patchwork.kernel.org>); Mon, 29 Oct 2018 14:56:23 -0400 Received: from mail-bl2nam02on0073.outbound.protection.outlook.com ([104.47.38.73]:47808 "EHLO NAM02-BL2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1729608AbeJ2S4X (ORCPT <rfc822;dmaengine@vger.kernel.org>); Mon, 29 Oct 2018 14:56:23 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=analog.onmicrosoft.com; s=selector1-analog-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=gHLUmO4wojVgHUv3G7Biwwip447HhdQDR4e3Pfu/mCs=; b=gkIdF/e3CUQ71ZreOCzcTIi12TJ0AIEwW04P1KGSGdAPNiQgFLuHUCXk+oLyeNZ9zvrbyrYGwh/oWcwZcewYFuy5fcopL9Rh3grpiZ09qG/Kuu8EEolESRohcSMCFI75w7nL2WiFSzMksmR6Lke8c2BBHdTTLr+iJQuXc+w4R4U= Received: from CY1PR03CA0005.namprd03.prod.outlook.com (2603:10b6:600::15) by DM2PR0301MB1232.namprd03.prod.outlook.com (2a01:111:e400:5020::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1273.19; Mon, 29 Oct 2018 10:08:22 +0000 Received: from BN1AFFO11FD013.protection.gbl (2a01:111:f400:7c10::194) by CY1PR03CA0005.outlook.office365.com (2603:10b6:600::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.1273.18 via Frontend Transport; Mon, 29 Oct 2018 10:08:21 +0000 Authentication-Results: spf=pass (sender IP is 137.71.25.57) smtp.mailfrom=analog.com; intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=bestguesspass action=none header.from=analog.com; Received-SPF: Pass (protection.outlook.com: domain of analog.com designates 137.71.25.57 as permitted sender) receiver=protection.outlook.com; client-ip=137.71.25.57; helo=nwd2mta4.analog.com; Received: from nwd2mta4.analog.com (137.71.25.57) by BN1AFFO11FD013.mail.protection.outlook.com (10.58.52.73) with Microsoft SMTP Server (version=TLS1_0, cipher=TLS_RSA_WITH_AES_256_CBC_SHA) id 15.20.1294.14 via Frontend Transport; Mon, 29 Oct 2018 10:08:21 +0000 Received: from NWD2HUBCAS8.ad.analog.com (nwd2hubcas8.ad.analog.com [10.64.69.108]) by nwd2mta4.analog.com (8.13.8/8.13.8) with ESMTP id w9TA8KBJ031220 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=OK); Mon, 29 Oct 2018 03:08:20 -0700 Received: from zeus.spd.analog.com (10.64.82.11) by NWD2HUBCAS8.ad.analog.com (10.64.69.108) with Microsoft SMTP Server id 14.3.301.0; Mon, 29 Oct 2018 06:08:20 -0400 Received: from saturn.analog.com ([10.50.1.244]) by zeus.spd.analog.com (8.15.1/8.15.1) with ESMTP id w9TA8JSG007246; Mon, 29 Oct 2018 06:08:19 -0400 From: Alexandru Ardelean <alexandru.ardelean@analog.com> To: <dmaengine@vger.kernel.org>, <dan.j.williams@intel.com> CC: Alexandru Ardelean <alexandru.ardelean@analog.com> Subject: [PATCH V2] dmaengine: dmatest: unmap data on a single code-path when xfer done Date: Mon, 29 Oct 2018 12:08:08 +0200 Message-ID: <20181029100808.31923-1-alexandru.ardelean@analog.com> X-Mailer: git-send-email 2.17.1 MIME-Version: 1.0 Content-Type: text/plain X-ADIRoutedOnPrem: True X-EOPAttributedMessage: 0 X-MS-Office365-Filtering-HT: Tenant X-Forefront-Antispam-Report: CIP:137.71.25.57;IPV:NLI;CTRY:US;EFV:NLI;SFV:NSPM;SFS:(10009020)(396003)(39860400002)(376002)(136003)(346002)(2980300002)(438002)(3190300001)(189003)(54534003)(199004)(316002)(16586007)(246002)(2616005)(8676002)(110136005)(86362001)(1076002)(7696005)(106466001)(476003)(106002)(2906002)(5660300001)(107886003)(36756003)(51416003)(47776003)(26005)(53416004)(8936002)(478600001)(50466002)(48376002)(50226002)(77096007)(4326008)(44832011)(336012)(186003)(6666004)(356004)(7636002)(305945005)(486006)(126002)(426003)(2101003);DIR:OUT;SFP:1101;SCL:1;SRVR:DM2PR0301MB1232;H:nwd2mta4.analog.com;FPR:;SPF:Pass;LANG:en;PTR:nwd2mail11.analog.com;MX:1;A:1; X-Microsoft-Exchange-Diagnostics: 1;BN1AFFO11FD013;1:fouYov0QZdn1B+4iz07uFn9h4YRt6tiYF9QPeuUilLH7ES459OJb9RsSAaljvbBClSosXJjpmP4UlHWnzEl0JexMoP81kTn3X8zDhFVbK/pU1sveY+9oGYoRxDDY5L7+ X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 27a6e3ef-2c8b-44d9-d3c6-08d63d8674c1 X-Microsoft-Antispam: BCL:0;PCL:0;RULEID:(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600074)(711020)(4608076)(2017052603328)(7153060);SRVR:DM2PR0301MB1232; X-Microsoft-Exchange-Diagnostics: 1;DM2PR0301MB1232;3:8IrAS6ab4DoyIyMQKO+RoEn/gzqZtNZ+09dkd0xplzDyTCmBkCs1qUOg8DZjuvRND7LrNp00sjp4JEtHaPRyQsDwleF0tENehUMXNnMDzPQmhCfYjQAWMiqgNLDcliGk2I0NUjtZ+u7wokunABL8cjmP0kBwNFhfyrQgA3ZTrWpiv3nrHfHb5+BhWxKhsi02/Wz7goH02C1N+uyqGE95rboN1UKOmzlfnBa2jbbgg22iztXDmPiM58jxBvYReOmVVbHiMNCcy2d1FBZpupqSqUZDeZ3m7pugm3QsBjJCqbYfPqwggZUZzuLXHb7DzjIbkQgItzSSBHl87OqMy/Yj4nqviabI7uHr+8rIRNwP/PE=;25:PhRufaWju1BTXEklgN4SJuiNRmI5zptU6TeaE9xJBBbIvYRKEO4GsAZ5CFpZS4XwEjmOQset5EmNzPjtfrLR4vdG7SV7HdflGVk1DVlb8u2M6FQ18pbaiiZY5PNt8qLTEKVcRja1urxhLlk3a8iJodsofX6impq/Xk2pZoRxFTGoEz6UjL4FR79ewbCJj4E6ClJI/XqUbCU4ChbIwSHBnQm2tryJV/dfAQdM3r5qhaACUOGcVe96uD1OHzX05ml7r925s6bHPjxiKFvKZlA6wrnXa+itMbYYhswrPnMrHWA5RXAIgSqgSKsXhKTu2NiC3vbXhnl4T6sCwFhqG8X6T2ZNO798mJ9av0EYk62BAqo= X-MS-TrafficTypeDiagnostic: DM2PR0301MB1232: X-Microsoft-Exchange-Diagnostics: 1;DM2PR0301MB1232;31:nDCO1bJqPCTbnwrpi8rDbI3Fam4LPxUDfpto76x1kMVOU2rkqFuQgmzT7wfjtfmKvx4mjBycTlesh/6nutPOpg+yyi2rc/uEPWFr2RuBDZHxkZhwD8SLv0swAKUiOs7b/Ribo9uj+304CjGxPXfbzdcQ+11u3wytzxLa4nXt9JG0tSs0CcFfUQIvyaMngb/kCtWDEplnfp3suiV9qCeOW4LYZDgY6XAfHcXu5vLG/fw=;20:MIlLVV8JEFsagHvsowZaEvriLkomkIrXyyGQJbo2Gsr8NSI8vIyShDMuq5N/BOf9LK/jdRvNrJgn+t4g5cntMGEgBgUokSrQ9IWRe/pkkVg25hn6gk4Ovl1udZA09HwjaFm4lVrNykYclWeE16c/JVNuIAPYN0d0Hn5xoyhl9wxg8pGNTKbp3BybA915XMvmeXFApm/qLYTznENPdzkHPfxAR1h1+P2xcPfImyC7KxZush9yWt+/8QN3vjE/+PFVcrcX6cBBzsu2zxV6dl4bH7NpJzZOb2Ut5XwdmtsqkWyZ6B8namm9FXCGuvC1RFTyfrUgjUvfRau9UJpMe/CVAcD8hcCxCGSlu8ezhfNxlJ420QtFhxmyVCJjbAhXmyuCNVVTuVl3Pw/JvF+olwUbTQJs/KEU+rwaR3UdBkCBTJ7CeaLLQ50fYiQ3sDN24maAMpvZCBJ2tK+SQ+3IRsLXdf0ccjLLVo98XinFw7UlX3FwEo60dwxVZvQ37sd/8Qlk X-Microsoft-Antispam-PRVS: <DM2PR0301MB123294642D12A0EEEDF2E057F9F30@DM2PR0301MB1232.namprd03.prod.outlook.com> X-Exchange-Antispam-Report-Test: UriScan:(232431446821674); X-MS-Exchange-SenderADCheck: 1 X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(3231382)(944501410)(52105095)(3002001)(10201501046)(93006095)(93004095)(6055026)(148016)(149066)(150057)(6041310)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123562045)(20161123560045)(20161123564045)(20161123558120)(201708071742011)(7699051)(76991095);SRVR:DM2PR0301MB1232;BCL:0;PCL:0;RULEID:;SRVR:DM2PR0301MB1232; X-Microsoft-Exchange-Diagnostics: 1;DM2PR0301MB1232;4:kKNEVTr2TZqHRdeJs24EYGJVXiobNp+zhHJHMan5WHKebiqk8ALJSofPFRQkaWHCPzl/Jgqpb64UYbW9FKFPPc1vRxS0eBZphgdG8NpKuIsiXGpkIf0jz5QaLfc8fM0GpFXaTYbVDgpuNTdg+kGb1esA6LdT1hsHGrcx0Loy5IbTprt0d2ZsoYwhLqnQnZDccF/7x9xQDE4H6JEkhjVJJT+krmlISvHlUZ6vb+mwbzNdt3VyVVoYsN+EGwUH3OQIIo4tH1ri2zX2QfkeCqNnZBhVc4NeK95WAjkDR+8kwfW9iQdK8sdCBhFaecWEAy1k X-Forefront-PRVS: 084080FC15 X-Microsoft-Exchange-Diagnostics: 1;DM2PR0301MB1232;23:Jqsyne0R8gBHe0Qb/rdoKDQiwEfqP8ZqNY/lSIar8Ad6nsn8NB4l6pxTjwc9Sr6NtbRQ/M+pxC6tRldK3plzcEkhmX5ayA6Syjh84G5vHa/yKkrT067jCx23Q0/YidjuDTxkU6DZzxUWh7PAFqg8yXKwksHk9bbaGKOYUonXcWxZ/rrepKFNS0sPSuXg0iVPvfF+P15lK7O4IyIWLAZz5SB3Vtrij19GU3KQ7ItAW+85V6/1FQT1p08dViPtj3VmA/SmYBHf0zPO4pNBF3o/L08DhWLXd+I1+S+JqzhaykjTlGLZwbwgwY1LOZLnGgUbzYpM1eCgBrbQw/y3j7vugjrw/fwstQiZsgW/uVVsHSe2jBld+lPswmOjsQ2Pl54GbWRGiQCBuXrvV+KCimKz1heCzeiRsFCy7PxozIuprlESHf3oX4jTW/4q9mgOM6PUw37bkhIWeo3HU9I4sOMp6s+Je6JJvQ9k/+upkkqD0N9/znXJWpsjgYOr28XNT6cZwUuKjCP1s8UjyQanOSRX9S8qc//vGFUtaGN1WBWY7EM3guh5L4g/eC5icO0h7D4LICv3uIS+v7VFulOJZbQa5X4miCoS175gOiq+osZ0UEyP4hF85GlwHweSCDTckLlnHeRxzWve4CQ9pmB7ysMFf46Eu+EQlqI0PpyK8BZiE2P6tHCaDqKA4Ak1vT5DTqgH3YdGXqtU6CBmq/juWmPWHmJyK7Q4uIyTla1Pfboq05HldYOfULnGqgvpe3oQdgv4e79vDOBooMsaFZ6H3MLFPU3qWMS/1UROCNLCbl73tZcBdgXkjAeCAvcADANX8QKkmu+hhJDqmscm9dkr0EpzHUQiYpZYDpc1V1oisMyy/dftPILhp5MKyQnNtbcMZDK/BCNMm1puja2ejPK5MH/IvqJ3fRpHuoLTpccRxTd+TEeeVnwd+6sMmTCAIoYqHRpHb9tsrcK6oY76ebjOrd9xo/l/lq0UJZWK6AUlJJ2oIdIJgOFOZtGI6Xig9+qKmWf1vR1KDAD2Trmj7evlw6CQHZ5hrdY/Rdw2nLHRSkp6tKNjIDfGSux2RSKNd5gvBDPW X-Microsoft-Antispam-Message-Info: nbsQY28sdQADBPIMn1YnHjiMZbXXBdSWhIrZVuVarV4jGVTHWarXJS47mwe/9AXReFP5tOYObe4HH5ap2Sr6gNYd12HfTBKDlsyoLRYc66JB+E1JIzzX7D/uhzgCoWk/yvJxYMWMWuU9aJ6Z5EA0n/0ZeszUusa+Pph+HKUFM5QBeollgxXpfMUi87dvx25sWsdGWouOIRlNhIx4cXcPPjwzjdJClOxNXvn86YwAD6sa/g6Ln+Qcavi2HrnJztp3VI0hUQUg+8dd6Gf9wQh3aq9kAYNG8IcakvebTRh3sjo8e7rSO8cp8YAiJTnY6FEdeqBpNrIAr0NnqKQguERjxXy+dygpYqlM+Gq1q3PGk6Q= X-Microsoft-Exchange-Diagnostics: 1;DM2PR0301MB1232;6:7H08/DWu+jtSjbADq0xsOfyA3/Yel0zrpyrybmuOve/iFCYgCwlRxCLoQdY8zg1FzQ/IkIJRVGNPvF6+hScVQJ0yBQdf2FDaLYVnvrlz/u/DWde40/X4Ip3pHMXP9V4ipmpPwZa4mI/CHV5c/8xBVi3PRa9ck//k78nw4DDOR1N1IK3cfKoLOIVHBWfgcPP3asMjMR2FO2faE7J09QRWBaZQ4xUEwE2kTvPkonq92NFrloWLjfGmJ027jWtarlyPVNQgeQ/9uKRGneWrX9R/LAFxsZLNVjigr2Qmg5RgKW7rMpER5hATi4a9Tx9INoCdhu5ahAfhNfnoYG444sqLXirZdXhPSd8juImCnlB49uaoFVS5YF8YxA/FQf5BTjo/mqViQGM9Q4xgfsvQTf7Wz2zHGoffprpnFFaGs2xVKh4tuGJztG16dfZOjixRLG0Jsro1/893d9Kn68Aw7GhphAKJBvto2sdA+MW4A9P4FPQ=;5:5NzRiY4tQu/268tLM2U5VZT0h2Hyo2jBCvGEb+3t+SMosUqBBHfDP+P5riMoGn6jYGClFKI0D9l1ZBwiW6qbSIQ2CRyl/9pkwZ/QSUunZ9AoxfwCpwQ27VP5ue7ufvvsHW86AaI/rG8x1b5xXHiICKY6kSecIsOht5XUYTPpfds=;7:oC59VzvcuRHEhUjcFcZrujdvTKKF6LYpc1xsEf9kf7NHyvCtwvMrAuyox2ExCbpdBuGGwJz35wYIEUUg36bc0Kg0GKMmBBQGJWTNFylg9ct4ikypo+IQlFZCqcLVygFc7dWbj0EgkME5umJWrstr/A== SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: analog.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Oct 2018 10:08:21.2025 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 27a6e3ef-2c8b-44d9-d3c6-08d63d8674c1 X-MS-Exchange-CrossTenant-Id: eaa689b4-8f87-40e0-9c6f-7228de4d754a X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=eaa689b4-8f87-40e0-9c6f-7228de4d754a;Ip=[137.71.25.57];Helo=[nwd2mta4.analog.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM2PR0301MB1232 Sender: dmaengine-owner@vger.kernel.org Precedence: bulk List-ID: <dmaengine.vger.kernel.org> X-Mailing-List: dmaengine@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP |
Series |
[V2] dmaengine: dmatest: unmap data on a single code-path when xfer done
|
expand
|
On 29-10-18, 12:08, Alexandru Ardelean wrote: > After the DMA transfer is done, we don't need to call the un-mapping code > in 3 places. One is enough. Applied, thanks
diff --git a/drivers/dma/dmatest.c b/drivers/dma/dmatest.c index 9cd09e8f9b4f..214391ba019a 100644 --- a/drivers/dma/dmatest.c +++ b/drivers/dma/dmatest.c @@ -721,14 +721,14 @@ static int dmatest_func(void *data) status = dma_async_is_tx_complete(chan, cookie, NULL, NULL); + dmaengine_unmap_put(um); + if (!done->done) { - dmaengine_unmap_put(um); result("test timed out", total_tests, src_off, dst_off, len, 0); failed_tests++; continue; } else if (status != DMA_COMPLETE) { - dmaengine_unmap_put(um); result(status == DMA_ERROR ? "completion error status" : "completion busy status", total_tests, src_off, @@ -737,8 +737,6 @@ static int dmatest_func(void *data) continue; } - dmaengine_unmap_put(um); - if (params->noverify) { verbose_result("test passed", total_tests, src_off, dst_off, len, 0);
After the DMA transfer is done, we don't need to call the un-mapping code in 3 places. One is enough. Signed-off-by: Alexandru Ardelean <alexandru.ardelean@analog.com> --- Changelog v1 -> v2: - re-applied on an upstream base; V1 was applied on another patch drivers/dma/dmatest.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-)