From patchwork Tue Oct 10 14:12:03 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Hemant Agrawal X-Patchwork-Id: 115405 Delivered-To: patch@linaro.org Received: by 10.140.22.163 with SMTP id 32csp3804306qgn; Tue, 10 Oct 2017 07:12:44 -0700 (PDT) X-Google-Smtp-Source: AOwi7QAa8JAlrvLrJphHoVjEFo8gtK/1ahc2P/MdQt2VmUW2KiMO6awpZf25dYkprFx275tZLh0o X-Received: by 10.223.184.75 with SMTP id u11mr8527612wrf.13.1507644764823; Tue, 10 Oct 2017 07:12:44 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1507644764; cv=none; d=google.com; s=arc-20160816; b=bMYeY14iuCrbT+FIFblSPGC9OVAoZaYWM7ydj0Su1E6F5BUtglGxzNA27qFE+H0VEg OE31t5XzSttnwtKCZhcbw4lpPmi1u+0FL2zZUpPAorGpVolv97piRwEN5Tq5bO2FiDKW 0QWwGo3Jw7sU1vWaYiBvYTL3FOO/v+zCeWOlGuaLib0DtP4RgCEj5uXngsVOj9XOpnTQ B9dyNcd7OniDGUEzMEvxOjEf3MsUzpZW3Utrpw9HlwTXcia8I63sW2WHow/ecKjePqgg kk9pIoeKn75MjNXdVnQdoq/lJXuhxfseDOpjkG33oNrayWAHV7A+O9GDFqQO0BKFtGVS wTuw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:subject:spamdiagnosticmetadata :spamdiagnosticoutput:mime-version:message-id:date:cc:to:from :arc-authentication-results; bh=F/IcXAr0HBnVBWjTISrGlPfMYPBD8mBYF0dJ+21vvPE=; b=gnAe29EkDJD3JascyJcFA+pOcNZBtp6Ma1I2Jijn9+t0g3XGGnV9aWRr+FPnYFGeOP j1Cs14/KmMjfdpd5/wbhxjbj7qdnz53fWcJVP1UY+jrK9UTmYzuWzu/+GSmZAJQ+MbN+ bSBiZs5G7kl3LsFAI2MVhvpiRxVK1Z53KkEgl9DomaAOpxQKg/UOFlAtOa89cTO2kH+v IVDMXExlYanvGDvPESBBbntks5Gy/cXsnFwp5tHIMoUIP3YErnQwxgeghBsnuMVLND/k sBgDY1KZ1oGMmCPVLl7TLLKMHsddPtAjagJAhQAwKwJXY5uxUnoQzqYSa22UaAz8yXhG GB0w== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) smtp.mailfrom=dev-bounces@dpdk.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=nxp.com Return-Path: Received: from dpdk.org (dpdk.org. [92.243.14.124]) by mx.google.com with ESMTP id b76si8718335wmg.157.2017.10.10.07.12.44; Tue, 10 Oct 2017 07:12:44 -0700 (PDT) Received-SPF: pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) client-ip=92.243.14.124; Authentication-Results: mx.google.com; spf=pass (google.com: domain of dev-bounces@dpdk.org designates 92.243.14.124 as permitted sender) smtp.mailfrom=dev-bounces@dpdk.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=nxp.com Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 48D681B2C0; Tue, 10 Oct 2017 16:12:43 +0200 (CEST) Received: from NAM02-BL2-obe.outbound.protection.outlook.com (mail-bl2nam02on0043.outbound.protection.outlook.com [104.47.38.43]) by dpdk.org (Postfix) with ESMTP id 10E651B260 for ; Tue, 10 Oct 2017 16:12:41 +0200 (CEST) Received: from BN3PR03CA0106.namprd03.prod.outlook.com (10.174.66.24) by BN6PR03MB2690.namprd03.prod.outlook.com (10.173.144.9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.77.7; Tue, 10 Oct 2017 14:12:39 +0000 Received: from BN1BFFO11FD024.protection.gbl (2a01:111:f400:7c10::1:147) by BN3PR03CA0106.outlook.office365.com (2603:10b6:400:4::24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.77.7 via Frontend Transport; Tue, 10 Oct 2017 14:12:40 +0000 Authentication-Results: spf=fail (sender IP is 192.88.158.2) smtp.mailfrom=nxp.com; dpdk.org; dkim=none (message not signed) header.d=none;dpdk.org; dmarc=fail action=none header.from=nxp.com; Received-SPF: Fail (protection.outlook.com: domain of nxp.com does not designate 192.88.158.2 as permitted sender) receiver=protection.outlook.com; client-ip=192.88.158.2; helo=az84smr01.freescale.net; Received: from az84smr01.freescale.net (192.88.158.2) by BN1BFFO11FD024.mail.protection.outlook.com (10.58.144.87) with Microsoft SMTP Server (version=TLS1_0, cipher=TLS_RSA_WITH_AES_256_CBC_SHA) id 15.20.77.10 via Frontend Transport; Tue, 10 Oct 2017 14:12:40 +0000 Received: from bf-netperf1.ap.freescale.net (bf-netperf1.ap.freescale.net [10.232.134.28]) by az84smr01.freescale.net (8.14.3/8.14.0) with ESMTP id v9AECaDl004024; Tue, 10 Oct 2017 07:12:37 -0700 From: Hemant Agrawal To: CC: Date: Tue, 10 Oct 2017 19:42:03 +0530 Message-ID: <1507644723-21398-1-git-send-email-hemant.agrawal@nxp.com> X-Mailer: git-send-email 2.7.4 X-EOPAttributedMessage: 0 X-Matching-Connectors: 131521183602714026; (91ab9b29-cfa4-454e-5278-08d120cd25b8); () X-Forefront-Antispam-Report: CIP:192.88.158.2; IPV:NLI; CTRY:US; EFV:NLI; SFV:NSPM; SFS:(10009020)(6009001)(336005)(7966004)(39380400002)(39860400002)(376002)(346002)(2980300002)(1110001)(1109001)(339900001)(189002)(199003)(104016004)(16586007)(69596002)(5660300001)(33646002)(356003)(36756003)(498600001)(105606002)(97736004)(316002)(2351001)(106466001)(81156014)(81166006)(5003940100001)(305945005)(50466002)(50986999)(8676002)(86362001)(53936002)(2906002)(68736007)(8936002)(189998001)(48376002)(4326008)(77096006)(50226002)(47776003)(6916009)(85426001)(6666003); DIR:OUT; SFP:1101; SCL:1; SRVR:BN6PR03MB2690; H:az84smr01.freescale.net; FPR:; SPF:Fail; PTR:InfoDomainNonexistent; MX:1; A:1; LANG:en; X-Microsoft-Exchange-Diagnostics: 1; BN1BFFO11FD024; 1:XGTbIMgxs96Ooq/p5S6eB05g7D2oXLSbO7MFeBflzKL4qU3jK/d1rC/xWdH0/hb0sTVPX1DGYL87BI7mHaON+4tSP4QvFciipZE37djT5SVVabBmVITyzDPDg/pVS48r MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 761a324f-0e02-4df0-c449-08d50fe8f788 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(2017052603199)(201703131430075)(201703131517081); SRVR:BN6PR03MB2690; X-Microsoft-Exchange-Diagnostics: 1; BN6PR03MB2690; 3:Du0Q1I069jU77GVMgxxTaHcPDn6BNEzdXZJkiMh61/6kuZA8Z7GNlSmLj6uQXdkI1CzWmLxecUVqm1wv2a8zKoHNN27x9eN77vCGKWqGamaEb+Rwe4xaJrDULqGDapO+wPllU6uSMS0X24fKI1CzyO9Y50B9Wi38vNTEdWAYH+4WZnP/xt4CUdgPKIbIp11a7VCRon9n2pfUDMpxJDanezrU3Ah97rzsiOv/uclfTcyZYqyRjiachL/4oiZ1b2GNX3dT41CY3i2UOkaleABpZZsyaJEIdCQRSZl3W1QxDGXXS88S89hrs+iMn4yYCljWrCyFiwQwrVuELchcHn9oqFTrXR6Hi4iizch0CpHbAkY=; 25:c6l27zXrGUj8XQsxP/ut7ix3INm94MckmaLyN+m7OzbKdf9YhVInnl5FmDtoTaaxIdrNc/Q+s/xWY17SqYIWZ3vECic67i5EqQ5HEqBbECnOhDBQhbPKuXTA5mTTAf1BgN5JNu8fwvilUTXrSn7tMpLDrF6bGOEt9e8Gn0J9sCVmCJSXNOkAlg/GllSuhiQIwLv36KgQMZ0HcY+vyXD8eUbnvLu8AxSr4DNGn5bkah3gXBCkh1pbUqUx3392FSVrk0tKGIjvju9kQVRnyy/zzeAdiq9a6yhRGa4lPuup9aVTBdZPLyfNhxs6t8Dy6tADKYXPUDCuHHbBBV5wnHX/Nw== X-MS-TrafficTypeDiagnostic: BN6PR03MB2690: X-Microsoft-Exchange-Diagnostics: 1; BN6PR03MB2690; 31:gqQhf9a2xFa0H3JPtpcNXyhcTBgZ4Ofw3aGxBabDHl/C0jJqqyJTcvfE5k9xzG0NC2oKiCBfFP5Jscv8OxQKwffO+vMe7ObJgBpsKSD58U0RHvD1VNJQTwz2qhMVSGxXBlFFi0CXq7up7BPpeNWwtLlKb2r6nA8jkkRNj6qLKpJ5UVbjN8xNxSm/PekqCaQWuly+YybxVofgFzOPt2Y5iO2+EQqJRN6B5U/xFcjrxTg=; 4:cf+voPmLH1FxlIEujWJdTWJYzlkpdCTTlqxGMW7830ks+33VlFHu6NryeEkL3s/Mv2P+y5NQdZnbhtk7lg5+q2PtiXE3XYTqr5XgPEtqazjB/mfbdlbchTgq+OA4jLmy6AbJqrvYZJeJBEyAsPGqLq5OA78PyGZhoxjZmdv4nNz8CpIrJp0gCdNUlxQkUMFXccurkLFyRLcqmqMhWlmxpkC5L8xv/0sN6LHYctNa8gQrr+dn8uz+RRHpvYNkyfePuIpQUj8yE25KRoTFFPFfIkTsRN86sSXkq8Uqcr5ayE44Sy2xofL25U1mzVBzBokvkGCRhj0HZOEQKxAtiYOUag== X-Exchange-Antispam-Report-Test: UriScan:(185117386973197)(275809806118684); X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6095135)(2401047)(8121501046)(5005006)(10201501046)(3002001)(100000703101)(100105400095)(93006095)(93001095)(6055026)(6096035)(20161123565025)(20161123559100)(20161123561025)(20161123563025)(20161123556025)(201703131430075)(201703131441075)(201703131448075)(201703131433075)(201703161259150)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:BN6PR03MB2690; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(400006)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:BN6PR03MB2690; X-Forefront-PRVS: 04569283F9 X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BN6PR03MB2690; 23:a2lU3w2fM7jjMTjTxqRdRGCwx+meaNo96c++5R6dE?= J/lW7Pf2aERy8WDspQyYeLjUb0T5RUEHTr/u/Ut5SEP/jUlSOvUBt6uGQp+i4HqdmXP0uDjwVuP+o7iwI8A9ipgQTrFpLU9kGYsh7FcITzJwWZcxsBz5wH3rpxAi9WIcWZ9G0raj4TXS97nP6jpdkc8IXFZXABl7iCkZnKhIOtAvV7Q/WqONFacFb5bTkxiXxFbAAm4vY5zwEaBMlwyE6sOhCnJXSFb86JO/cYpk0KAK028KtZxBjPki95k/QhUqM/k7ijN7VGk4sQGWIftXa/KfbywFzD8AAGGy237OjaJmrVmLh7e7mOGYZ6djrm+4th6tKMtN//KeYza+XI5rUSFgM0b28D4Qf4wAejeJaCmg9/OvUDZdoCgdaD/rRyR7tg7O1fyhh9XnPZZ1zh+/fK/0bNpJLaFFt5ajbf88QRYPUArWMh6J4G0PJOi5cKnQwawCKE8ZypzWjx+6ahxqQ7Q72CXtUZhAw0e1xUK+/iBAwl4Qe11+oUVvEFbUfywlB2s0RSJbNbgVWIC0ATAmzh9JSpua7hG/nB1CluEd0CjqhKNyKusxBLmeqVd/frBl9TF3ShWNSptR49r2/UgXIjLyl1+s+dcC2NGK3Ymm02EKd6js1vWTH/5E+Qk4GREFp9m0HSVWt1OqZaeJxSdutwuxuiGLiWxsZ7VPPUrNoa303MMhrGlMOo0ppSbTQHASVP+qVdHVNB/aspfGHSzS1vU1u2ZIZo8NtRnRQl0tZI2f4S2L+EMs6RYVPnT5R3tEnXMpAspk1T6EM2o+bX7+5k5XajG0gF1j4dpOysACcD0cXNSUrsnbTGcc9/hT2OG2VDsOyg3kxb/dJKLCuCIiYb8JAMXXrAc/Zb40pq2pnmdUffch9Dgfd7grlhFYy7v8MlYjA0W/MV5A5MSAbbZmZjEI+cqidnPLivdx51Q1y7OJWiOoEQu0q0ah3bwtZ1oGiiXBKy+VkauX/eBNWd2cv+wnowyEJ1S31WfyH7brrYXSH8ZOXbeMFk4JxiQHVYHyHuwFiVwl7sNMNRW22bKuKIJwcbbsy1OF/vRdG1vh7squwNK8vb5imef7jxvITcSO4o= X-Microsoft-Exchange-Diagnostics: 1; BN6PR03MB2690; 6:5VtpUN62HLboRKb1RUXjcgajXUOc/OiSUhTKM8sK3NbJpYHGIyqkeiTQN0Ow99E4ZJLKLSRCGfQezYQB+kHMTR2XRGyoI2GW41zFi21L5dyfIy/ejEmb6B6aL8QrovlGEyE1M9LrWuOVSeLiiIEBZMeM6y/eSwTmcaqWO6/q31RuK7j1RQTp/yBFVLg/LMZdTx7Fxioyz6qkHf9WWfzcKufKx+LxRuVzAUmaZhNhvLc1Av2uGkkn04YudX8HydrgCAyxsj/ZzK8LPLguZm0U9Zvzc1IyECk7oAhJolDH6sieydtq/trwZyBEFhu1hP/vd6oNUA4a4wiJ8wgNRcCWOw==; 5:O5QZ5jhfWgv0w5E0hukrjS3xGK+vfFLGQ8ier5LJEX9YivsJCEGMYAXKk5FKHv1NuoMyhvFWU57zIzbeZfuQTrgf9DGquFlIPDob6Qezal0zG2by22icD2nezq6U1G9gdwrC56jdXlX562RjFyPoZ15j9OeRaH39WZ0T3sZV4Tg=; 24:88RbsSEfFFdXPCiQ/0WnJ3lR8f8NLCULJ8QkpeV72ImeE3QtEc3JeHzUrTl80VwEMxn7k8VO9Gp4Nieoqdg1+V7gCCeBJKZV/nj72sYDtBM=; 7:ISoqJ9yrBjXy6/AAwv05jiGOFRwJD/EQXhkSofW4o9J5BiBGSeDBWw3edNbTWFys4Tr3bLVfERmTIcrljnx/XCXzY5gu7GgAWnm99kd2FTK5elqx7ZkAdHetQrELwK/A39ETUZCcaZNWJRquv/N+yXUFfU9HVUHZVyigDubdX/oca7K6SUPgC/dxkyGhJZcB7Yw7v+Rzdef/ptUuvapyM/1rCSmdTg7igNJ/p0/zRZE= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Oct 2017 14:12:40.0374 (UTC) X-MS-Exchange-CrossTenant-Id: 5afe0b00-7697-4969-b663-5eab37d5f47e X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=5afe0b00-7697-4969-b663-5eab37d5f47e; Ip=[192.88.158.2]; Helo=[az84smr01.freescale.net] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN6PR03MB2690 Subject: [dpdk-dev] [PATCH] mempool/dpaa2: improve error handling X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Reverting the config on encountering errors. Signed-off-by: Hemant Agrawal --- drivers/mempool/dpaa2/dpaa2_hw_mempool.c | 34 +++++++++++++++++++++----------- 1 file changed, 23 insertions(+), 11 deletions(-) -- 2.7.4 diff --git a/drivers/mempool/dpaa2/dpaa2_hw_mempool.c b/drivers/mempool/dpaa2/dpaa2_hw_mempool.c index 6df203f..d664929 100644 --- a/drivers/mempool/dpaa2/dpaa2_hw_mempool.c +++ b/drivers/mempool/dpaa2/dpaa2_hw_mempool.c @@ -65,7 +65,7 @@ rte_hw_mbuf_create_pool(struct rte_mempool *mp) struct dpaa2_bp_info *bp_info; struct dpbp_attr dpbp_attr; uint32_t bpid; - int ret, p_ret; + int ret; avail_dpbp = dpaa2_alloc_dpbp_dev(); @@ -78,7 +78,7 @@ rte_hw_mbuf_create_pool(struct rte_mempool *mp) ret = dpaa2_affine_qbman_swp(); if (ret) { RTE_LOG(ERR, PMD, "Failure in affining portal\n"); - return ret; + goto err1; } } @@ -86,7 +86,7 @@ rte_hw_mbuf_create_pool(struct rte_mempool *mp) if (ret != 0) { PMD_INIT_LOG(ERR, "Resource enable failure with" " err code: %d\n", ret); - return ret; + goto err1; } ret = dpbp_get_attributes(&avail_dpbp->dpbp, CMD_PRI_LOW, @@ -94,10 +94,16 @@ rte_hw_mbuf_create_pool(struct rte_mempool *mp) if (ret != 0) { PMD_INIT_LOG(ERR, "Resource read failure with" " err code: %d\n", ret); - p_ret = ret; - ret = dpbp_disable(&avail_dpbp->dpbp, CMD_PRI_LOW, - avail_dpbp->token); - return p_ret; + goto err2; + } + + bp_info = rte_malloc(NULL, + sizeof(struct dpaa2_bp_info), + RTE_CACHE_LINE_SIZE); + if (!bp_info) { + PMD_INIT_LOG(ERR, "No heap memory available for bp_info"); + ret = -ENOMEM; + goto err2; } /* Allocate the bp_list which will be added into global_bp_list */ @@ -105,7 +111,8 @@ rte_hw_mbuf_create_pool(struct rte_mempool *mp) RTE_CACHE_LINE_SIZE); if (!bp_list) { PMD_INIT_LOG(ERR, "No heap memory available"); - return -ENOMEM; + ret = -ENOMEM; + goto err3; } /* Set parameters of buffer pool list */ @@ -127,9 +134,6 @@ rte_hw_mbuf_create_pool(struct rte_mempool *mp) rte_dpaa2_bpid_info[bpid].bp_list = bp_list; rte_dpaa2_bpid_info[bpid].bpid = bpid; - bp_info = rte_malloc(NULL, - sizeof(struct dpaa2_bp_info), - RTE_CACHE_LINE_SIZE); rte_memcpy(bp_info, (void *)&rte_dpaa2_bpid_info[bpid], sizeof(struct dpaa2_bp_info)); mp->pool_data = (void *)bp_info; @@ -138,6 +142,14 @@ rte_hw_mbuf_create_pool(struct rte_mempool *mp) h_bp_list = bp_list; return 0; +err3: + rte_free(bp_info); +err2: + dpbp_disable(&avail_dpbp->dpbp, CMD_PRI_LOW, avail_dpbp->token); +err1: + dpaa2_free_dpbp_dev(avail_dpbp); + + return ret; } static void