From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v8 01/25] eal: define macro container_of Date: Tue, 30 Aug 2016 12:30:22 +0200 Message-ID: <2199965.TtOmKzW9OO@xps13> References: <1466510566-9240-1-git-send-email-shreyansh.jain@nxp.com> <57C46637.3060707@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: Ferruh Yigit , dev@dpdk.org, viktorin@rehivetech.com, david.marchand@6wind.com, hemant.agrawal@nxp.com To: Shreyansh Jain Return-path: Received: from mail-wm0-f49.google.com (mail-wm0-f49.google.com [74.125.82.49]) by dpdk.org (Postfix) with ESMTP id 2D8E22BD5 for ; Tue, 30 Aug 2016 12:30:25 +0200 (CEST) Received: by mail-wm0-f49.google.com with SMTP id i5so27974357wmg.0 for ; Tue, 30 Aug 2016 03:30:25 -0700 (PDT) In-Reply-To: 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" 2016-08-30 09:57, Shreyansh Jain: > On Monday 29 August 2016 10:13 PM, Ferruh Yigit wrote: > > This gives compilation error for mlx5, because the libraries mlx depends > > defines same macro: > > ..../rte_common.h:338:9: error: 'container_of' macro redefined > > /usr/include/infiniband/verbs.h:77:9: note: previous definition is here > > I thought testing with scripts/test-build.sh and default configuration > would compile all drivers - I was wrong. I will retest the patches and > release again. > > Is there a better way to test that no driver breaks? Any particular > parameters I should use for test-build.sh? Yes I suggest to create a file ~/.config/dpdk/devel.config to adapt the configuration to your system. Once you have installed the required dependencies, you can make this kind of configuration: mlxdep=$root/mlx/mofed-3.3-1.0.0.0 szedep=$root/sze/usr-1.1.4 if echo $DPDK_TARGET | grep -q '^x86_64' ; then export DPDK_DEP_ARCHIVE=y export DPDK_DEP_ZLIB=y export DPDK_DEP_PCAP=y export DPDK_DEP_SSL=y export DPDK_DEP_MOFED=y export DPDK_DEP_SZE=y export DPDK_DEP_CFLAGS="-I$mlxdep/include -I$szedep/include" export DPDK_DEP_LDFLAGS="-L$mlxdep/lib -L$szedep/lib64 -rpath=$szedep/lib64" export AESNI_MULTI_BUFFER_LIB_PATH=$root/aesni/ipsec-043 export LIBSSO_SNOW3G_PATH=$root/libsso/libsso-snow3g-0.3.1 export LIBSSO_KASUMI_PATH=$root/libsso/libsso-kasumi-0.3.1 fi > I used 'x86_64-native-linuxapp-gcc+default+debug+shared' for all patches. It is a good idea to test also with clang (x86_64-native-linuxapp-clang) and another arch (e.g. arm64-thunderx-linuxapp-gcc).