DPDK patches and discussions
 help / color / mirror / Atom feed
From: Ye Xiaolong <xiaolong.ye@intel.com>
To: Xiaojun Liu <xiaojun.liu@silicom.co.il>
Cc: Ferruh Yigit <ferruh.yigit@intel.com>,
	"xiao.w.wang@intel.com" <xiao.w.wang@intel.com>,
	"qi.z.zhang@intel.com" <qi.z.zhang@intel.com>,
	"ngai-mint.kwan@intel.com" <ngai-mint.kwan@intel.com>,
	"jacob.e.keller@intel.com" <jacob.e.keller@intel.com>,
	"dev@dpdk.org" <dev@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH v4 0/5] support fm10k switch management
Date: Mon, 20 Apr 2020 16:14:47 +0800	[thread overview]
Message-ID: <20200420081447.GD33101@intel.com> (raw)
In-Reply-To: <VI1PR04MB5150A0D62885C482C5D1B480BDD40@VI1PR04MB5150.eurprd04.prod.outlook.com>

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

  reply	other threads:[~2020-04-20  8:19 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1584687199-19198-1-git-send-email-xiaojun.liu@silicom.co.il>
2020-04-09  6:26 ` Xiaojun Liu
2020-04-09  6:26   ` [dpdk-dev] [PATCH v4 1/5] net/fm10k: add basic functions for " Xiaojun Liu
2020-04-17 13:56     ` Ferruh Yigit
2020-04-09  6:26   ` [dpdk-dev] [PATCH v4 2/5] net/fm10k: add epl serdes and port control functions Xiaojun Liu
2020-04-17 13:57     ` Ferruh Yigit
2020-04-09  6:26   ` [dpdk-dev] [PATCH v4 3/5] net/fm10k: add ffu and statistics and config file functions Xiaojun Liu
2020-04-09  6:26   ` [dpdk-dev] [PATCH v4 4/5] net/fm10k: add flow interface and switch management Xiaojun Liu
2020-04-09  6:26   ` [dpdk-dev] [PATCH v4 5/5] net/fm10k: add switch management support Xiaojun Liu
2020-04-20  5:09     ` Ye Xiaolong
2020-04-16  8:33   ` [dpdk-dev] [PATCH v4 0/5] support fm10k switch management Wang, Xiao W
2020-04-17  4:02     ` Ye Xiaolong
2020-04-17 14:18   ` Ferruh Yigit
2020-04-20  4:55     ` Ye Xiaolong
2020-04-20  6:37       ` Xiaojun Liu
2020-04-20  8:14         ` Ye Xiaolong [this message]
2020-04-20  8:24           ` Xiaojun Liu
2020-04-20 13:05             ` Ferruh Yigit

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200420081447.GD33101@intel.com \
    --to=xiaolong.ye@intel.com \
    --cc=dev@dpdk.org \
    --cc=ferruh.yigit@intel.com \
    --cc=jacob.e.keller@intel.com \
    --cc=ngai-mint.kwan@intel.com \
    --cc=qi.z.zhang@intel.com \
    --cc=xiao.w.wang@intel.com \
    --cc=xiaojun.liu@silicom.co.il \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).