From mboxrd@z Thu Jan 1 00:00:00 1970 From: Adrien Mazarguil Subject: [PATCH v3 00/14] Fixes for exported headers Date: Wed, 26 Apr 2017 14:07:08 +0200 Message-ID: References: To: dev@dpdk.org Return-path: Received: from mail-wm0-f52.google.com (mail-wm0-f52.google.com [74.125.82.52]) by dpdk.org (Postfix) with ESMTP id 563042C50 for ; Wed, 26 Apr 2017 14:07:34 +0200 (CEST) Received: by mail-wm0-f52.google.com with SMTP id r190so2017087wme.1 for ; Wed, 26 Apr 2017 05:07:34 -0700 (PDT) Received: from 6wind.com (host.78.145.23.62.rev.coltfrance.com. [62.23.145.78]) by smtp.gmail.com with ESMTPSA id k199sm8910972wmd.20.2017.04.26.05.07.32 for (version=TLS1_2 cipher=AES128-SHA bits=128/128); Wed, 26 Apr 2017 05:07:32 -0700 (PDT) In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" This series addresses the remaining issues seen by check-includes.sh in exported headers. Most of them may cause compilation errors in user applications: - Dependencies on missing includes. - Non-standard C/C++ constructs usage without associated safeties. - Missing C++ awareness blocks. It also addresses the incomplete implementation of E-Tag and NVGRE flow API pattern items. Changes in v2: - Fixed (still) incomplete E-tag and NVGRE flow API documentation. - Renamed E-Tag item parameter "ecid_b" to "grp_ecid_b" in testpmd, to reflect that both fields are set at once. - Fixed remaining compilation issue with RTE_LIBRTE_EVENTDEV_DEBUG. - Fixed avp include to avoid kernel module compilation issue pointed out by Allain. Changes in v3: - Reverted regression introduced in v2 related to RTE_PMD_DEBUG_TRACE() and RTE_LIBRTE_EVENTDEV_DEBUG. - Addressed original issue with RTE_PMD_DEBUG_TRACE() in a new separate commit. - Fixed remaining regression on the kernel side of avp. Adrien Mazarguil (14): crypto/scheduler: fix missing includes eventdev: fix errors with strict compilation flags latency: fix missing includes in exported header net: fix missing include in exported header vhost: fix errors with strict compilation flags mbuf: fix missing includes in exported header net/avp: fix errors in exported headers bitrate: fix errors in exported header efd: fix missing include in exported header metrics: fix errors in exported header ethdev: fix C++ errors in flow API ethdev: fix C++ errors in flow API (MPLS, GRE) ethdev: fix incomplete items in flow API eal: fix debug macro redefinition app/test-pmd/cmdline_flow.c | 46 ++++++++++++++++++ devtools/check-includes.sh | 4 +- doc/guides/prog_guide/rte_flow.rst | 26 ++++++++++ doc/guides/testpmd_app_ug/testpmd_funcs.rst | 8 +++ .../crypto/scheduler/rte_cryptodev_scheduler.h | 1 + .../rte_cryptodev_scheduler_operations.h | 1 + drivers/net/avp/rte_avp_common.h | 18 ++++++- drivers/net/avp/rte_avp_fifo.h | 12 +++++ lib/librte_bitratestats/rte_bitrate.h | 10 ++++ lib/librte_cryptodev/rte_cryptodev_pmd.h | 8 --- lib/librte_eal/common/include/rte_dev.h | 14 ++++++ lib/librte_efd/rte_efd.h | 2 + lib/librte_ether/rte_ethdev.h | 9 ---- lib/librte_ether/rte_flow.h | 51 ++++++++++++++++++++ lib/librte_eventdev/rte_eventdev.h | 3 +- lib/librte_eventdev/rte_eventdev_pmd.h | 23 ++++----- lib/librte_latencystats/rte_latencystats.h | 2 + lib/librte_mbuf/rte_mbuf_ptype.h | 3 ++ lib/librte_metrics/rte_metrics.h | 10 ++++ lib/librte_net/rte_net_crc.h | 2 + lib/librte_vhost/rte_vhost.h | 16 ++++-- 21 files changed, 231 insertions(+), 38 deletions(-) -- 2.1.4