On 4/20/2020 9:24 AM, Xiaojun Liu wrote: > Hi Xiaolong, > > Thank you! My gcc version is 4.8.5. I will try 7.5.0.
The compiler versions I am using: - gcc (GCC) 9.3.1 20200408 (Red Hat 9.3.1-2) - clang version 9.0.1 (Fedora 9.0.1-2.fc31) - aarch64-buildroot-linux-gnu-gcc.br_real (Buildroot 2018.08-git-00807-g0184fa3) 8.2.0 - powerpc64le-linux-gnu-gcc (GCC) 6.2.1 20160822 (Advance-Toolchain-at10.0) IBM AT 10 branch, based on subversion id 239677. - icc (ICC) 19.1.0.166 20191121 > > Best regards, > Xiaojun > > -----Original Message----- > From: Ye Xiaolong [mailto:xiaolong...@intel.com] > Sent: Monday, April 20, 2020 4:15 PM > To: Xiaojun Liu > Cc: Ferruh Yigit; xiao.w.w...@intel.com; qi.z.zh...@intel.com; > ngai-mint.k...@intel.com; jacob.e.kel...@intel.com; dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v4 0/5] support fm10k switch management > > On 04/20, Xiaojun Liu wrote: >> Hi Ferruh, >> >> Thanks! I checked the building by "devtools/test-build.sh >> x86_64-native-linux-gcc+next+shared", I didn't see any error. >> Could you tell me how do you build it and what's the error ? > > Which gcc are you system using? I can reproduce the build error with gcc 7.5.0 > via ./devtools/test-build.sh x86_64-native-linux-gcc+next+shared. > > > Thanks, > Xiaolong > >> >> Best regards, >> Xiaojun >> >> -----Original Message----- >> From: Ye Xiaolong [mailto:xiaolong...@intel.com] >> Sent: Monday, April 20, 2020 12:56 PM >> To: Ferruh Yigit >> Cc: Xiaojun Liu; xiao.w.w...@intel.com; qi.z.zh...@intel.com; >> ngai-mint.k...@intel.com; jacob.e.kel...@intel.com; dev@dpdk.org >> Subject: Re: [dpdk-dev] [PATCH v4 0/5] support fm10k switch management >> >> On 04/17, Ferruh Yigit wrote: >>> On 4/9/2020 7:26 AM, Xiaojun Liu wrote: >>>> V4: >>>> * Add I2C to control the inside LED and PHY. >>>> * Add SBUS to communicate with spico. >>>> * Add registers defination. >>>> * Add switch management log API. >>>> * Add switch management structures. >>>> * Add epl serdes include loading spico, >>>> controlling pcsl, dma, dfe, ical. >>>> * Add spico code. >>>> * Add state machine for epl lane and port. >>>> * Add external port management. >>>> * Add ffu to support offload flow into HW. >>>> It supports forward, mirror, push VLAN, pop VLAN. >>>> * Add config file to configure debug log, port speed, >>>> epl port mapping dpdk port, flowset. >>>> * Add statistics includes epl port, ffu rule, dpdk port. >>>> * Add flow interface to support offload flow into HW. >>>> * Add switch management, includes initialization, >>>> port mapping, epl port link, LED controller, interrupt handler. >>>> * Split dev_init to 2 parts in ethdev. First only register >>>> the port in switch management; second init hook will be >>>> called after all the pf are registered. >>>> * Add switch interrupt support. >>>> * Add fm10k_mirror_rule_set/fm10k_mirror_rule_reset. >>>> * Add fm10k_dev_filter_ctrl to support flow operation. >>>> * Add dpdk port and pf mapping. >>>> * Modify Makefile to add new files building >>>> >>>> Xiaojun Liu (5): >>>> net/fm10k: add basic functions for switch management >>>> net/fm10k: add epl serdes and port control functions >>>> net/fm10k: add ffu and statistics and config file functions >>>> net/fm10k: add flow interface and switch management >>>> net/fm10k: add switch management support >>>> >>> >>> With 'CONFIG_RTE_FM10K_SWITCH_MANAGEMENT=y' I am getting following build >>> errors, >>> can you please check them? >>> >> >> Thanks Ferruh for catching the build error, forgot to build test with >> CONFIG_RTE_FM10K_SWITCH_MANAGEMENT=y >> when applying it :( >> >> Thanks, >> Xiaolong