From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id BB488A09D9; Wed, 11 Nov 2020 21:26:16 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 1E7DD5AA7; Wed, 11 Nov 2020 21:25:43 +0100 (CET) Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id C08515AA6 for ; Wed, 11 Nov 2020 21:25:41 +0100 (CET) IronPort-SDR: 8ETIZItN5zYrs6EuDKyx9mqIWTvGayLhhZS11VSbOz61kMUESfgAk64yltGXKgxf4b+Ec4+w/M Mvc1NjFzaZBA== X-IronPort-AV: E=McAfee;i="6000,8403,9802"; a="167627811" X-IronPort-AV: E=Sophos;i="5.77,470,1596524400"; d="scan'208";a="167627811" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by fmsmga104.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Nov 2020 12:25:33 -0800 IronPort-SDR: XY4AVbCsScpNRv7ftwXPo7Dz/4pjVsAX0DSF5xrgjv3UDJaxZLNLQEv3P3K3NgxVHB7JG5Dasq j/GlwMMFwY3w== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.77,470,1596524400"; d="scan'208";a="531855321" Received: from txasoft-yocto.an.intel.com ([10.123.72.192]) by fmsmga005.fm.intel.com with ESMTP; 11 Nov 2020 12:25:32 -0800 From: Timothy McDaniel To: Cc: dev@dpdk.org, erik.g.carrillo@intel.com, gage.eads@intel.com, harry.van.haaren@intel.com, jerinj@marvell.com, thomas@monjalon.net, david.marchand@redhat.com Date: Wed, 11 Nov 2020 14:27:01 -0600 Message-Id: <1605126422-522-3-git-send-email-timothy.mcdaniel@intel.com> X-Mailer: git-send-email 1.7.10 In-Reply-To: <1605126422-522-1-git-send-email-timothy.mcdaniel@intel.com> References: <1605126422-522-1-git-send-email-timothy.mcdaniel@intel.com> Subject: [dpdk-dev] [PATCH] event/dlb: do not free memzone if port create succeeds 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" Add missing returns so that the memzone free is not called if port create is successful. Signed-off-by: Timothy McDaniel --- drivers/event/dlb/pf/dlb_pf.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/event/dlb/pf/dlb_pf.c b/drivers/event/dlb/pf/dlb_pf.c index cf88c49..3aeef6f 100644 --- a/drivers/event/dlb/pf/dlb_pf.c +++ b/drivers/event/dlb/pf/dlb_pf.c @@ -342,6 +342,7 @@ dlb_pf_ldb_port_create(struct dlb_hw_dev *handle, *(struct dlb_cmd_response *)cfg->response = response; DLB_INFO(dev->dlb_device, "Exiting %s() with ret=%d\n", __func__, ret); + return 0; create_port_err: @@ -419,6 +420,7 @@ dlb_pf_dir_port_create(struct dlb_hw_dev *handle, *(struct dlb_cmd_response *)cfg->response = response; DLB_INFO(dev->dlb_device, "Exiting %s() with ret=%d\n", __func__, ret); + return 0; create_port_err: -- 2.6.4