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 3C439A2EDB for ; Fri, 6 Sep 2019 15:22:00 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 0672F1F3EE; Fri, 6 Sep 2019 15:22:00 +0200 (CEST) Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by dpdk.org (Postfix) with ESMTP id 0548F1F3E5 for ; Fri, 6 Sep 2019 15:21:57 +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 fmsmga105.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 06 Sep 2019 06:21:57 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,473,1559545200"; d="scan'208";a="199518009" Received: from marcinzx-mobl.ger.corp.intel.com ([10.103.104.109]) by fmsmga001.fm.intel.com with ESMTP; 06 Sep 2019 06:21:56 -0700 From: Marcin Zapolski To: dev@dpdk.org Cc: Marcin Zapolski Date: Fri, 6 Sep 2019 15:18:10 +0200 Message-Id: <20190906131813.1343-1-marcinx.a.zapolski@intel.com> X-Mailer: git-send-email 2.22.0.windows.1 In-Reply-To: <20190730124950.1293-1-marcinx.a.zapolski@intel.com> References: <20190730124950.1293-1-marcinx.a.zapolski@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-dev] [RFC 19.11 v2 0/3] Hide DPDK internal struct from public API 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" Several DPDK internal structures are exposed to direct access by user applications. This patch removes them from public API, and makes core DPDK functions that use them non-inline. v2: This patch set no longer makes internal DPDK functions non-inline. Instead it splits the rte_eth_dev structure to private and public part and modifies function arguments of rx and tx functions. This should bring less performance impact, but at the cost of needing to modify every PMD to use new rx and tx functions. For testing purposes, the ixgbe and i40e drivers are modified to acommodate for the changes. Marcin Zapolski (3): ethdev: hide key ethdev structures from public API i40e: make driver compatible with changes in ethdev ixgbe: make driver compatible with changes in ethdev drivers/net/i40e/i40e_ethdev.c | 10 +- drivers/net/i40e/i40e_ethdev.h | 1 + drivers/net/i40e/i40e_ethdev_vf.c | 8 +- drivers/net/i40e/i40e_rxtx.c | 119 ++-- drivers/net/i40e/i40e_rxtx.h | 33 +- drivers/net/i40e/i40e_rxtx_vec_altivec.c | 23 +- drivers/net/i40e/i40e_rxtx_vec_avx2.c | 45 +- drivers/net/i40e/i40e_rxtx_vec_neon.c | 23 +- drivers/net/i40e/i40e_rxtx_vec_sse.c | 23 +- drivers/net/i40e/i40e_vf_representor.c | 12 +- drivers/net/ixgbe/ixgbe_ethdev.c | 30 +- drivers/net/ixgbe/ixgbe_ethdev.h | 23 +- drivers/net/ixgbe/ixgbe_rxtx.c | 111 ++-- drivers/net/ixgbe/ixgbe_rxtx.h | 9 +- drivers/net/ixgbe/ixgbe_rxtx_vec_neon.c | 22 +- drivers/net/ixgbe/ixgbe_rxtx_vec_sse.c | 23 +- drivers/net/ixgbe/ixgbe_vf_representor.c | 10 +- lib/librte_bitratestats/rte_bitrate.c | 2 +- lib/librte_bpf/bpf_pkt.c | 2 +- lib/librte_ethdev/ethdev_private.c | 1 + lib/librte_ethdev/ethdev_profile.h | 1 + lib/librte_ethdev/rte_ethdev.c | 122 +++- lib/librte_ethdev/rte_ethdev.h | 222 +++++--- lib/librte_ethdev/rte_ethdev_core.h | 531 +----------------- lib/librte_ethdev/rte_ethdev_driver.h | 482 ++++++++++++++++ lib/librte_ethdev/rte_ethdev_version.map | 11 + lib/librte_ethdev/rte_flow.c | 2 +- lib/librte_ethdev/rte_mtr.c | 2 +- lib/librte_ethdev/rte_tm.c | 2 +- lib/librte_ethdev/rte_tm_driver.h | 1 + .../rte_event_eth_rx_adapter.c | 2 +- .../rte_event_eth_tx_adapter.c | 2 +- lib/librte_eventdev/rte_eventdev.c | 2 +- lib/librte_flow_classify/rte_flow_classify.h | 2 +- .../rte_flow_classify_parse.h | 2 +- lib/librte_gro/gro_tcp4.c | 2 +- lib/librte_gro/gro_vxlan_tcp4.c | 2 +- lib/librte_gro/rte_gro.c | 2 +- lib/librte_gso/rte_gso.c | 2 +- lib/librte_kni/rte_kni.c | 2 +- lib/librte_latencystats/rte_latencystats.c | 2 +- lib/librte_pdump/rte_pdump.c | 2 +- lib/librte_port/rte_port_ethdev.c | 2 +- lib/librte_telemetry/rte_telemetry.c | 2 +- lib/librte_telemetry/rte_telemetry_parser.c | 2 +- lib/librte_vhost/vhost.c | 2 +- 46 files changed, 1053 insertions(+), 885 deletions(-) -- 2.17.1