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

[Xiaojun Liu <xiaojun.liu@silicom.co.il>]

Hi Xiaolong,

Thank you! My gcc version is 4.8.5. I will try 7.5.0.

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