From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 38E835A71 for ; Tue, 17 Nov 2015 13:21:34 +0100 (CET) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga103.fm.intel.com with ESMTP; 17 Nov 2015 04:21:33 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.20,307,1444719600"; d="scan'208";a="822103294" Received: from irvmail001.ir.intel.com ([163.33.26.43]) by orsmga001.jf.intel.com with ESMTP; 17 Nov 2015 04:21:32 -0800 Received: from sivswdev01.ir.intel.com (sivswdev01.ir.intel.com [10.237.217.45]) by irvmail001.ir.intel.com (8.14.3/8.13.6/MailSET/Hub) with ESMTP id tAHCLVh7013684; Tue, 17 Nov 2015 12:21:31 GMT Received: from sivswdev01.ir.intel.com (localhost [127.0.0.1]) by sivswdev01.ir.intel.com with ESMTP id tAHCLVcJ032192; Tue, 17 Nov 2015 12:21:31 GMT Received: (from bricha3@localhost) by sivswdev01.ir.intel.com with id tAHCLVJD032188; Tue, 17 Nov 2015 12:21:31 GMT From: Bruce Richardson To: dev@dpdk.org Date: Tue, 17 Nov 2015 12:21:05 +0000 Message-Id: <1447762867-32124-1-git-send-email-bruce.richardson@intel.com> X-Mailer: git-send-email 1.7.4.1 In-Reply-To: <1446552059-5446-1-git-send-email-bruce.richardson@intel.com> References: <1446552059-5446-1-git-send-email-bruce.richardson@intel.com> Subject: [dpdk-dev] [PATCH v4 0/2] ethdev: debug code cleanup X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 17 Nov 2015 12:21:34 -0000 This patchset performs two cleanups: 1. Four functions in ethdev.c which were enabled for debug only have been merged into their inlined header-file counterparts. This change required that a number of macros be renamed and moved to the header file too. The macro changes are in patches 1 & 2, and the elimination of the separate debug fns are in patch 3. 2. Checks for valid function pointers are added to the API calls for reading the descriptor ring count, and checking for a valid descriptor. This is because these functions are not implemented by most drivers, and so it's far safer to have the check. NOTE: This patchset now depends upon the cryptodev patchset --- V4 Changes: * Originally this was a 4-patch set, but patches 1 and 2 duplicated changes being made in the patchset to add crypto device support. Therefore this set has been reduced to two patches to sit on top of that set. * As suggested on-list, when adding checks for the function pointers being valid we can also add in the similarly lightweight checks for the port id being valid. V3 Changes: * Rebased to latest DPDK codebase * Fixed checkpatch issues in patches 2 and 3. V2 Changes: * Rebased to latest DPDK codebase * Changed type from uint32_t to int for the count function, on the basis of feedback received. Bruce Richardson (2): ethdev: remove duplicated debug functions ethdev: add sanity checks to functions lib/librte_ether/rte_ethdev.c | 64 ------------------------------------------- lib/librte_ether/rte_ethdev.h | 62 ++++++++++++++++++++--------------------- 2 files changed, 30 insertions(+), 96 deletions(-) -- 2.5.0