Re: [dpdk-dev] [PATCH v4 0/5] support fm10k switch management

[Ferruh Yigit <ferruh.yigit@intel.com>]

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.ye@intel.com] 
> Sent: Monday, April 20, 2020 4:15 PM
> To: Xiaojun Liu
> Cc: Ferruh Yigit; xiao.w.wang@intel.com; qi.z.zhang@intel.com; ngai-mint.kwan@intel.com; jacob.e.keller@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.ye@intel.com] 
>> Sent: Monday, April 20, 2020 12:56 PM
>> To: Ferruh Yigit
>> Cc: Xiaojun Liu; xiao.w.wang@intel.com; qi.z.zhang@intel.com; ngai-mint.kwan@intel.com; jacob.e.keller@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