From patchwork Fri Jul 5 17:22:43 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Anatoly Burakov X-Patchwork-Id: 56184 Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 9C4D81BE1B; Fri, 5 Jul 2019 19:22:48 +0200 (CEST) Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by dpdk.org (Postfix) with ESMTP id 6AAEB559A for ; Fri, 5 Jul 2019 19:22:46 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 05 Jul 2019 10:22:45 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.63,455,1557212400"; d="scan'208";a="185217212" Received: from silpixa00399498.ir.intel.com (HELO silpixa00399498.ger.corp.intel.com) ([10.237.223.125]) by fmsmga001.fm.intel.com with ESMTP; 05 Jul 2019 10:22:44 -0700 From: Anatoly Burakov To: dev@dpdk.org Date: Fri, 5 Jul 2019 18:22:43 +0100 Message-Id: X-Mailer: git-send-email 2.17.1 In-Reply-To: References: Subject: [dpdk-dev] [PATCH v3 0/1] Fix timer resource leak 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" Previous attempts [1] at fixing the resource leak have been deemed unsuccessful because of limitations around what can be done without breaking the ABI. Now that we've broken the EAL ABI, we can fix this issue properly. This patchset is adding a new lock API, as well as fixes the actual issue. The patchset depends on mem config patchset [2]. v3: - Accidentally squashed with previous patch, reverted v2: - Addressed review comments and squashed patches [1] http://patches.dpdk.org/patch/53334/ [2] http://patches.dpdk.org/project/dpdk/list/?series=5369 Anatoly Burakov (1): timer: fix resource leak in finalize lib/librte_eal/common/eal_common_mcfg.c | 14 +++++++ lib/librte_eal/common/eal_memcfg.h | 2 + .../common/include/rte_eal_memconfig.h | 22 ++++++++++ lib/librte_eal/rte_eal_version.map | 2 + lib/librte_timer/rte_timer.c | 41 +++++++++++++------ lib/librte_timer/rte_timer.h | 5 ++- 6 files changed, 71 insertions(+), 15 deletions(-)