From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pf0-f196.google.com (mail-pf0-f196.google.com [209.85.192.196]) by dpdk.org (Postfix) with ESMTP id 44F281BB4C for ; Fri, 11 May 2018 05:04:58 +0200 (CEST) Received: by mail-pf0-f196.google.com with SMTP id f20-v6so2017526pfn.0 for ; Thu, 10 May 2018 20:04:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=KLihQ6qc3yFzZfovzucTZe601Px37y2oY1Fh7DMHO/U=; b=eW3raqHogDT8F4l/oGnCtTjxNWroOnbOTl2Dtro1Mvnw6cuZmIsGSVa3a2oIVaVtIj XXEqy/wG8W0Aav0Z+DxynALYnfJ3znp9tqE0BEIFakLgOU9yn1HrDurcoLhsDQq/k2jy wYDQPGkFY55icYmXu7VOw2QSTJrfH4qrUow63QJCZI1dUX2ca6IEgYjvTvgUe9KBwNA/ 1PposqZfpLFkN7DbLV/Hs3fJXhkNcPJjQi4FzLQ7NlNSxf5lijAoLSwbyOFykX5+gAaa YEqh4EUnScZTuwskvMkuMm+6SoK1OxJM3v8a5EcH+HBNYehd/hSg0T4MxSKs3drUu1hG ik/w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=KLihQ6qc3yFzZfovzucTZe601Px37y2oY1Fh7DMHO/U=; b=tHrCURWf6IO97pFqZor+p0rBqKcD0e9HD7HCnGLV+axzcs+jIYO1qZPZ4s8PsIm7b4 mFsipiH8rOFMv44P0i3w9+FP3LBmscbG0U3lmRWEHnHTj92DfBrmnGFj2I0LdHjGbryZ 36oWWtb+4m4hDmGMPCft17030VsxD6qybmOwYT9lEuEIrsNDMILbeCVoYAib/V0oVnNE amOqpOuJvWa46aSdQwkSXmoNkVT13vkzMfPLMbKaz2jhFhkGRVywYmXYm/bJLyni1r67 uZfiNw69p57widAfEJDFG6j9MKNTZ06Tl0Sr1IZOPM7qirdrVALeE0CQeBBK/ORMX/k0 p8wA== X-Gm-Message-State: ALKqPwe3UTqlg1IYbMriWnf5a7fDNYG2m4lRLkLX8MZeDwemt1CSeRsK +OaQxm5iEWJFnPKPgl5kK1twkgKVGvM= X-Google-Smtp-Source: AB8JxZq5EzTdX9rgJf6CDyv04Vr5onym82HzmNxiWVhz4Hz8TuMwd+zG4jBK+rN5h8mjjQfT7+ggkg== X-Received: by 2002:a62:990f:: with SMTP id d15-v6mr3712514pfe.115.1526007897320; Thu, 10 May 2018 20:04:57 -0700 (PDT) Received: from xeon-e3 (204-195-35-107.wavecable.com. [204.195.35.107]) by smtp.gmail.com with ESMTPSA id u16-v6sm4592492pfl.39.2018.05.10.20.04.57 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 10 May 2018 20:04:57 -0700 (PDT) Date: Thu, 10 May 2018 20:04:54 -0700 From: Stephen Hemminger To: Andy Green Cc: dev@dpdk.org Message-ID: <20180510200454.75e6fdcc@xeon-e3> In-Reply-To: <152600382101.54839.18392182958431211001.stgit@localhost.localdomain> References: <152600382101.54839.18392182958431211001.stgit@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH v2 00/24] Fixes for GCC8 against lagopus 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: , X-List-Received-Date: Fri, 11 May 2018 03:04:58 -0000 On Fri, 11 May 2018 09:58:54 +0800 Andy Green wrote: > The following series fixes build problems in dpdk master > headers, found when using it as the dpdk subproject in > lagopus. These errors are coming when you try to use > the dpdk headers, not when you build dpdk itself. > > Two patches are updated to follow requests for larger > work. > > --- > > Andy Green (24): > lib/librte_eal: import libbsd strlcpy > lib/librte_ethdev: change eth-dev-ops API to return int > rte_common.h: cast gcc builtin result to avoid complaints > lib/librte_eal: explicit tmp cast > lib/librte_eal: explicit cast for signed change > /lib/librte_eal: stage cast from uint64 to long > rte_spinlock.h: stack declarations before code > rte_ring_generic.h: stack declarations before code > rte_ring.h: remove signed type flipflopping > rte_dev.h: stack declaration at top of own basic block > rte_mbuf.h: avoid warnings from inadvertant promotion > rte_mbuf.h: explicit casts for int16 to uint16 > rte_mbuf.h: make sure RTE-MIN compares same types > rte_mbuf.h: explicit cast restricting ptrdiff to uint16 > rte_mbuf.h: explicit cast for size type to uint32 > rte_mbuf.h: explicit casts to uint16 to avoid warnings > rte_byteorder.h: explicit cast for return promotion > rte_ether.h: explicit cast avoiding truncation warning > rte_ether.h: stack vars declared at top of function > rte_ethdev.h: align sign and scope of temp var > rte_ethdev.h: explicit cast for truncation > rte_hash_crc.h: stack vars declared at top of function > rte_hash_crc.h: explicit casts for truncation > test_table_pipeline: repair munged indirection level > > > drivers/net/ark/ark_ethdev_rx.c | 4 +- > drivers/net/ark/ark_ethdev_rx.h | 3 +- > drivers/net/avf/avf_rxtx.c | 4 +- > drivers/net/avf/avf_rxtx.h | 2 + > drivers/net/bnxt/bnxt_ethdev.c | 5 ++- > drivers/net/dpaa/dpaa_ethdev.c | 4 +- > drivers/net/dpaa2/dpaa2_ethdev.c | 6 ++- > drivers/net/e1000/e1000_ethdev.h | 6 +-- > drivers/net/e1000/em_rxtx.c | 4 +- > drivers/net/e1000/igb_rxtx.c | 4 +- > drivers/net/enic/enic_ethdev.c | 9 ++--- > drivers/net/i40e/i40e_rxtx.c | 4 +- > drivers/net/i40e/i40e_rxtx.h | 3 +- > drivers/net/ixgbe/ixgbe_ethdev.h | 3 +- > drivers/net/ixgbe/ixgbe_rxtx.c | 4 +- > drivers/net/nfp/nfp_net.c | 9 ++--- > drivers/net/sfc/sfc_ethdev.c | 4 +- > drivers/net/thunderx/nicvf_ethdev.c | 2 + > drivers/net/thunderx/nicvf_rxtx.c | 4 +- > drivers/net/thunderx/nicvf_rxtx.h | 2 + > drivers/net/vhost/rte_eth_vhost.c | 4 +- > examples/l3fwd-power/main.c | 2 + > lib/librte_eal/common/eal_common_string_fns.c | 34 +++++++++++++++++++ > .../common/include/arch/x86/rte_memcpy.h | 8 ++-- > .../common/include/arch/x86/rte_spinlock.h | 4 ++ > .../common/include/generic/rte_byteorder.h | 2 + > lib/librte_eal/common/include/rte_common.h | 2 + > lib/librte_eal/common/include/rte_dev.h | 15 +++++--- > lib/librte_eal/common/include/rte_lcore.h | 2 + > lib/librte_eal/common/include/rte_random.h | 6 ++- > lib/librte_eal/common/include/rte_string_fns.h | 7 +--- > lib/librte_ethdev/rte_ethdev.h | 30 ++++++++++------- > lib/librte_ethdev/rte_ethdev_core.h | 4 +- > lib/librte_hash/rte_hash_crc.h | 11 +++--- > lib/librte_mbuf/rte_mbuf.h | 36 +++++++++++--------- > lib/librte_net/rte_ether.h | 5 ++- > lib/librte_ring/rte_ring.h | 4 +- > lib/librte_ring/rte_ring_c11_mem.h | 2 + > lib/librte_ring/rte_ring_generic.h | 10 ++---- > test/test/test_table_pipeline.c | 4 +- > 40 files changed, 157 insertions(+), 121 deletions(-) > > -- > Signature All looks good to me. Reviewed-by: Stephen Hemminger