From mboxrd@z Thu Jan 1 00:00:00 1970 From: Adrien Mazarguil Subject: [PATCH 00/10] Fix build errors related to exported headers Date: Tue, 5 Apr 2016 16:08:00 +0200 Message-ID: <1459865290-10248-1-git-send-email-adrien.mazarguil@6wind.com> Cc: Bruce Richardson , Thomas Monjalon To: dev@dpdk.org Return-path: Received: from mail-wm0-f47.google.com (mail-wm0-f47.google.com [74.125.82.47]) by dpdk.org (Postfix) with ESMTP id 5AC7C2C08 for ; Tue, 5 Apr 2016 16:08:26 +0200 (CEST) Received: by mail-wm0-f47.google.com with SMTP id l6so25379619wml.1 for ; Tue, 05 Apr 2016 07:08:26 -0700 (PDT) List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" DPDK uses GNU C language extensions in most of its code base. This is fine for internal source files whose compilation flags are controlled by DPDK, however user applications that use exported "public" headers may experience compilation failures when enabling strict error/standard checks (-std and -pedantic for instance). Exported headers are installed system-wide and must be as clean as possible so applications do not have to use workarounds for compilation issues. This patchset affects exported headers only, compilation problems are addressed as follows: - adding the __extension__ keyword to nonstandard constructs (same method as existing libraries when there is no other choice) - adding the __extension__ keyword to C11 constructs to remain compatible with pure C99 - adding missing includes so exported files can be included out of order and on their own - fixing GNU printf-like variadic macros as there is no magic keyword for these Adrien Mazarguil (10): lib: add extension keyword to braced-groups within expressions lib: add extension keyword to large enum values lib: use C99 syntax for zero-size arrays lib: add extension keyword to nonstandard bit-fields lib: add extension keyword to structs with no members lib: add extension keyword to unnamed structs/unions lib: fix missing include dependencies lib: add extension keyword to forward reference to enum types lib: remove named variadic macros in exported headers lib: hide static functions that are never defined lib/librte_acl/rte_acl.h | 2 +- lib/librte_cfgfile/rte_cfgfile.h | 2 ++ lib/librte_cmdline/cmdline.h | 1 + lib/librte_cmdline/cmdline_parse_portlist.h | 1 + lib/librte_cmdline/cmdline_socket.h | 3 ++ lib/librte_cryptodev/rte_crypto.h | 3 ++ lib/librte_cryptodev/rte_crypto_sym.h | 4 +++ lib/librte_cryptodev/rte_cryptodev.h | 41 ++++++++++++++-------- lib/librte_cryptodev/rte_cryptodev_pmd.h | 6 ++-- .../common/include/arch/arm/rte_byteorder.h | 3 ++ .../common/include/arch/arm/rte_memcpy_32.h | 3 +- .../common/include/arch/arm/rte_prefetch_32.h | 2 ++ .../common/include/arch/arm/rte_prefetch_64.h | 2 ++ lib/librte_eal/common/include/arch/arm/rte_vect.h | 1 + .../common/include/arch/ppc_64/rte_atomic.h | 2 ++ .../common/include/arch/ppc_64/rte_byteorder.h | 2 ++ .../common/include/arch/ppc_64/rte_cycles.h | 3 ++ .../common/include/arch/ppc_64/rte_memcpy.h | 3 +- .../common/include/arch/ppc_64/rte_prefetch.h | 2 ++ .../common/include/arch/x86/rte_atomic.h | 3 ++ .../common/include/arch/x86/rte_atomic_32.h | 9 +++++ .../common/include/arch/x86/rte_atomic_64.h | 8 +++++ .../common/include/arch/x86/rte_byteorder.h | 3 ++ .../common/include/arch/x86/rte_byteorder_32.h | 7 ++++ .../common/include/arch/x86/rte_byteorder_64.h | 7 ++++ .../common/include/arch/x86/rte_cycles.h | 3 ++ .../common/include/arch/x86/rte_memcpy.h | 4 +-- .../common/include/arch/x86/rte_prefetch.h | 2 ++ lib/librte_eal/common/include/arch/x86/rte_rtm.h | 1 + lib/librte_eal/common/include/arch/x86/rte_vect.h | 8 +++-- lib/librte_eal/common/include/generic/rte_atomic.h | 1 + .../common/include/generic/rte_byteorder.h | 2 ++ .../common/include/generic/rte_cpuflags.h | 3 ++ lib/librte_eal/common/include/generic/rte_memcpy.h | 7 ++-- lib/librte_eal/common/include/rte_common.h | 22 ++++++++++-- lib/librte_eal/common/include/rte_devargs.h | 1 + lib/librte_eal/common/include/rte_eal.h | 1 + lib/librte_eal/common/include/rte_interrupts.h | 2 ++ lib/librte_eal/common/include/rte_memory.h | 4 +++ lib/librte_eal/common/include/rte_memzone.h | 2 ++ lib/librte_eal/common/include/rte_time.h | 8 +++++ lib/librte_eal/common/include/rte_version.h | 1 + .../linuxapp/eal/include/exec-env/rte_interrupts.h | 1 + .../linuxapp/eal/include/exec-env/rte_kni_common.h | 2 +- lib/librte_ether/rte_dev_info.h | 2 ++ lib/librte_ether/rte_eth_ctrl.h | 4 +++ lib/librte_ether/rte_ethdev.h | 4 +++ lib/librte_hash/rte_fbk_hash.h | 2 +- lib/librte_hash/rte_thash.h | 4 +++ lib/librte_ip_frag/rte_ip_frag.h | 2 +- lib/librte_lpm/rte_lpm.h | 7 +++- lib/librte_lpm/rte_lpm6.h | 2 ++ lib/librte_lpm/rte_lpm_neon.h | 1 + lib/librte_lpm/rte_lpm_sse.h | 1 + lib/librte_mbuf/rte_mbuf.h | 9 +++++ lib/librte_mempool/rte_mempool.h | 1 + lib/librte_pipeline/rte_pipeline.h | 5 ++- lib/librte_reorder/rte_reorder.h | 2 ++ lib/librte_ring/rte_ring.h | 2 +- lib/librte_sched/rte_bitmap.h | 3 +- lib/librte_sched/rte_reciprocal.h | 2 ++ lib/librte_sched/rte_sched_common.h | 2 ++ lib/librte_timer/rte_timer.h | 2 ++ lib/librte_vhost/rte_virtio_net.h | 2 +- 64 files changed, 221 insertions(+), 36 deletions(-) -- 2.1.4