[dpdk-dev] Question about patchset order.

2016-03-02 Thread Bruce Richardson
On Tue, Feb 23, 2016 at 06:34:38PM +0100, Thomas Monjalon wrote: > 2016-02-23 16:17, Kobylinski, MichalX: > > Hi Thomas, > > I sent in January a patch-set that extends to 24 bits a next_hop field in > > lpm library: > > http://dpdk.org/dev/patchwork/patch/10249/ > >

[dpdk-dev] [PATCH v2] I217 and I218 changes

2016-03-02 Thread Lu, Wenzhuo
Hi Ravi, >Do you want me to resend it as 'v1' and include your comments in commit >message? let me know. I think it?s better if you withdraw this one, send a new one and make the commit log easy to understand ? And you can keep my ack.

[dpdk-dev] [PATCH v1] virtio: Use cpuflag for vector api

2016-03-02 Thread Qiu, Michael
On 3/1/2016 5:46 PM, Santosh Shukla wrote: > On Tue, Mar 1, 2016 at 2:41 PM, Qiu, Michael wrote: >> On 2/26/2016 4:53 PM, Santosh Shukla wrote: >>> Check cpuflag macro before using vectored api. >>> -virtio_recv_pkts_vec() uses _sse3__ simd instruction for now so added >>> cpuflag. >>> - Also

[dpdk-dev] [PATCH v9 0/2] Add VHOST PMD

2016-03-02 Thread Qiu, Michael
On 3/1/2016 10:19 AM, Tetsuya Mukawa wrote: > On 2016/03/01 11:00, Qiu, Michael wrote: >> On 2/26/2016 4:36 PM, Tetsuya Mukawa wrote: >>> On 2016/02/26 13:29, Tetsuya Mukawa wrote: [...] BTW, I have set the frontend mergeable off. I have checked below cases. - Case1:

[dpdk-dev] [PATCH 2/2 v3] i40e: Add floating VEB support in i40e

2016-03-02 Thread Xu, Qian Q
Suggest to add document for why to have it, the basic usage and the sample of how to use it. Thanks Qian -Original Message- From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Zhe Tao Sent: Thursday, February 25, 2016 2:32 PM To: dev at dpdk.org Subject: [dpdk-dev] [PATCH 2/2 v3]

[dpdk-dev] [PATCH] hash: fix memcmp function pointer in multi-process

2016-03-02 Thread Qiu, Michael
On 3/2/2016 2:57 AM, Dhananjaya Reddy Eadala wrote: > Hi > > We found a problem in dpdk-2.2 using under multi-process environment. > Here is the brief description how we are using the dpdk: > > We have two processes proc1, proc2 using dpdk. These proc1 and proc2 are > two different compiled

[dpdk-dev] [PATCH v1] virtio: Use cpuflag for vector api

2016-03-02 Thread Yuanhan Liu
On Wed, Mar 02, 2016 at 02:10:14AM +, Qiu, Michael wrote: > On 3/1/2016 5:46 PM, Santosh Shukla wrote: > > On Tue, Mar 1, 2016 at 2:41 PM, Qiu, Michael > > wrote: > >> On 2/26/2016 4:53 PM, Santosh Shukla wrote: > >>> Check cpuflag macro before using vectored api. > >>>

[dpdk-dev] [PATCH v6 1/4] lib/ether: optimize the'rte_eth_tunnel_filter_conf' structure

2016-03-02 Thread Sun, Xutao
Hi, Panu > -Original Message- > From: Panu Matilainen [mailto:pmatilai at redhat.com] > Sent: Tuesday, March 01, 2016 4:54 PM > To: Sun, Xutao ; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v6 1/4] lib/ether: optimize > the'rte_eth_tunnel_filter_conf' structure > > On 03/01/2016

[dpdk-dev] tcp/ip stack based on dpdk is ready

2016-03-02 Thread zimeiw
hi, The tcp/ip stack is developed based on dpdk. tcp/ip stack and APP deployment. |---| |---| |---| | APP | | APP | | APP | | | | | | | | | | | | |

[dpdk-dev] [PATCH v2] ethdev: fix byte order inconsistence between fdir flow and mask

2016-03-02 Thread Lu, Wenzhuo
Hi, > -Original Message- > From: Wu, Jingjing > Sent: Monday, February 1, 2016 10:48 AM > To: dev at dpdk.org > Cc: Wu, Jingjing; Lu, Wenzhuo; Pei, Yulong; yaacovh at mellanox.com > Subject: [PATCH v2] ethdev: fix byte order inconsistence between fdir flow and > mask > > Fixed issue of

[dpdk-dev] [PATCH v4 1/3] fm10k: enable FTAG based forwarding

2016-03-02 Thread Wang, Xiao W
Hi, > -Original Message- > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > Sent: Wednesday, March 2, 2016 6:38 AM > To: Wang, Xiao W > Cc: Chen, Jing D ; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v4 1/3] fm10k: enable FTAG based forwarding > > On Tue, 1 Mar 2016

[dpdk-dev] [PATCH v2] doc: Malicious Driver Detection not supported by ixgbe

2016-03-02 Thread He, Shaopeng
Hi, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Wenzhuo Lu > Sent: Friday, February 26, 2016 12:49 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v2] doc: Malicious Driver Detection not supported > by ixgbe > > Announce that Malicious Driver

[dpdk-dev] [PATCH v4 1/3] lpm: make rte_lpm_lookupx4 API definition architecture agnostic

2016-03-02 Thread Jerin Jacob
On Tue, Mar 01, 2016 at 06:42:35PM +0100, Thomas Monjalon wrote: > 2016-02-12 17:58, Jerin Jacob: > > -Used architecture agnostic xmm_t to represent 128 bit SIMD variable > > > > -Introduced vect_* API abstraction in app/test to test rte_lpm_lookupx4 > > API in architecture agnostic way > > > >

[dpdk-dev] [PATCH v4 2/3] lpm: add support for NEON

2016-03-02 Thread Jerin Jacob
On Tue, Mar 01, 2016 at 06:46:04PM +0100, Thomas Monjalon wrote: > 2016-02-12 17:58, Jerin Jacob: > > # fails to compile on ARM > > -CONFIG_RTE_LIBRTE_LPM=n > > -CONFIG_RTE_LIBRTE_TABLE=n > > -CONFIG_RTE_LIBRTE_PIPELINE=n > > The associated examples cannot compile. > Maybe it's too early to

[dpdk-dev] [PATCH v5 1/5] lib/librte_ether: change function name of tunnel port config

2016-03-02 Thread Wenzhuo Lu
The names of function for tunnel port configuration are not accurate. They're tunnel_add/del, better change them to tunnel_port_add/del. As it may be an ABI change if change the names directly, the new functions are added but not remove the old ones. The old ones will be removed in the next

[dpdk-dev] [PATCH v5 2/5] i40e: rename the tunnel port config functions

2016-03-02 Thread Wenzhuo Lu
As the names of tunnel port config functions are not accurate, change them from tunnel_add/del to tunnel_port_add/del. And support both the old and new rte ops. Signed-off-by: Wenzhuo Lu --- drivers/net/i40e/i40e_ethdev.c | 22 -- 1 file changed, 12 insertions(+), 10

[dpdk-dev] [PATCH v5 4/5] ixgbe: support VxLAN & NVGRE RX checksum off-load

2016-03-02 Thread Wenzhuo Lu
X550 will do VxLAN & NVGRE RX checksum off-load automatically. This patch exposes the result of the checksum off-load. Signed-off-by: Wenzhuo Lu --- drivers/net/ixgbe/ixgbe_ethdev.c | 4 drivers/net/ixgbe/ixgbe_rxtx.c | 11 ++- lib/librte_ether/rte_ethdev.h| 1 +

[dpdk-dev] [PATCH v5 3/5] ixgbe: support UDP tunnel port config

2016-03-02 Thread Wenzhuo Lu
Add UDP tunnel port add/del support on ixgbe. Now only support VxLAN port configuration. Although according to the specification the VxLAN port has a default value 4789, it can be changed. We support VxLAN port configuration to meet the change. Note, the default value of VxLAN port in ixgbe NICs

[dpdk-dev] [PATCH v5 5/5] ixgbe: support VxLAN & NVGRE TX checksum off-load

2016-03-02 Thread Wenzhuo Lu
The patch add VxLAN & NVGRE TX checksum off-load. When the flag of outer IP header checksum offload is set, we'll set the context descriptor to enable this checksum off-load. Also update release note for VxLAN & NVGRE checksum off-load support. Signed-off-by: Wenzhuo Lu ---

[dpdk-dev] [PATCH v4 3/3] maintainers: claim responsibility for arm64 specific files of hash and lpm

2016-03-02 Thread Jerin Jacob
On Tue, Mar 01, 2016 at 06:47:46PM +0100, Thomas Monjalon wrote: > 2016-02-12 17:58, Jerin Jacob: > > +F: lib/librte_lpm/rte_lpm_neon.h > > This line should be in the previous patch. Will fix in v5 > > > +F: lib/librte_hash/rte_crc_arm64.h > > +F: lib/librte_hash/rte_cmp_arm64.h > > Yes,

[dpdk-dev] [PATCH v7 0/4] Add tunnel filter support for IP in GRE on i40e

2016-03-02 Thread Liu, Yong
Tested-by: Yong Liu - Tested Branch: dpdk-next-net - Tested Commit: 5fa83b5398e26af7537b09605432fcb3d0cc1d41 - OS: Fedora20 3.11.10-301.fc20.x86_64 - GCC: gcc version 4.8.3 20140911 - CPU: Intel(R) Xeon(R) CPU E5-2699 v3 @ 2.30GHz - NIC: Intel Corporation Device RedrockCanyou [8086:15a4] -

[dpdk-dev] [PATCH v6 1/4] lib/ether: optimize the'rte_eth_tunnel_filter_conf' structure

2016-03-02 Thread Panu Matilainen
On 03/02/2016 05:10 AM, Sun, Xutao wrote: > Hi, Panu > >> -Original Message- >> From: Panu Matilainen [mailto:pmatilai at redhat.com] >> Sent: Tuesday, March 01, 2016 4:54 PM >> To: Sun, Xutao ; dev at dpdk.org >> Subject: Re: [dpdk-dev] [PATCH v6 1/4] lib/ether: optimize >>

[dpdk-dev] [PATCH v7 0/4] Add tunnel filter support for IP in GRE on i40e

2016-03-02 Thread Wu, Jingjing
> -Original Message- > From: Sun, Xutao > Sent: Tuesday, March 01, 2016 4:41 PM > To: dev at dpdk.org > Cc: Wu, Jingjing; Zhang, Helin > Subject: [PATCH v7 0/4] Add tunnel filter support for IP in GRE on i40e > > This patch set adds tunnel filter support for IP in GRE on i40e. > > v2

[dpdk-dev] [PATCH 0/2 v4] i40e: Add floating VEB support for i40e

2016-03-02 Thread Zhe Tao
This patch-set add the support for floating VEB in i40e. All the VFs VSIs can decide whether to connect to the legacy VEB/VEPA or the floating VEB. When connect to the floating VEB a new floating VEB is created. Now all the VFs need to connect to floating VEB or legacy VEB, cannot connect to both

[dpdk-dev] [PATCH 1/2 v4] i40e: support floating VEB config

2016-03-02 Thread Zhe Tao
Add the new floating related argument option in the EAL. Using this parameter, all the samples can decide whether to use legacy VEB/VEPA or floating VEB. Even the floating argument is provided in the EAL, but this floating feature are only support for FVL so far. Signed-off-by: Zhe Tao ---

[dpdk-dev] [PATCH 2/2 v4] i40e: Add floating VEB support in i40e

2016-03-02 Thread Zhe Tao
This patch add the support for floating VEB in i40e. All the VFs VSIs can decide whether to connect to the legacy VEB/VEPA or the floating VEB. When connect to the floating VEB a new floating VEB is created. Now all the VFs need to connect to floating VEB or legacy VEB, cannot connect to both of

[dpdk-dev] [PATCH v4 4/4] ena: DPDK polling-mode driver for Amazon Elastic Network Adapters (ENA)

2016-03-02 Thread Panu Matilainen
On 03/01/2016 06:46 PM, Jan Medala wrote: > This is a PMD for the Amazon ethernet ENA family. > The driver operates variety of ENA adapters through feature negotiation > with the adapter and upgradable commands set. > ENA driver handles PCI Physical and Virtual ENA functions. > > Signed-off-by:

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-03-02 Thread Panu Matilainen
On 03/02/2016 04:02 AM, Stephen Hemminger wrote: > On Mon, 29 Feb 2016 08:33:25 -0600 > Jay Rolette wrote: > >> On Mon, Feb 29, 2016 at 5:06 AM, Thomas Monjalon > 6wind.com> >> wrote: >> >>> Hi, >>> I totally agree with Avi's comments. >>> This topic is really important for the future of DPDK.

[dpdk-dev] [PATCH v1 0/3] virtio vector and misc

2016-03-02 Thread Yuanhan Liu
On Tue, Mar 01, 2016 at 03:32:17PM +0530, Santosh Shukla wrote: > - 1st patch: let non-x86 arch use virtio pmd driver in non-vec > - 2nd patch: enable virtio arm support > - 3rd patch: update virtio for arm feature entry in release guide. Series looks good to me: Acked-by: Yuanhan Liu However,

[dpdk-dev] [PATCH v2] ethdev: fix byte order inconsistence between fdir flow and mask

2016-03-02 Thread Zhe Tao
On Mon, Feb 01, 2016 at 10:48:21AM +0800, Jingjing Wu wrote: > Fixed issue of byte order in ethdev library that the structure > for setting fdir's mask and flow entry is inconsist and made > inputs of mask be in big endian. > > Fixes: 76c6f89e80d4 ("ixgbe: support new flow director masks") >

[dpdk-dev] [PATCH v1 0/3] virtio vector and misc

2016-03-02 Thread Santosh Shukla
On Wed, Mar 2, 2016 at 2:02 PM, Yuanhan Liu wrote: > On Tue, Mar 01, 2016 at 03:32:17PM +0530, Santosh Shukla wrote: >> - 1st patch: let non-x86 arch use virtio pmd driver in non-vec >> - 2nd patch: enable virtio arm support >> - 3rd patch: update virtio for arm feature entry in release guide. >

[dpdk-dev] [PATCH v2 4/6] bond mode 4: allow external state machine

2016-03-02 Thread Iremonger, Bernard
Hi Eric, > > > > @@ -157,6 +159,7 @@ struct rte_eth_bond_8023ad_conf { > > > > uint32_t tx_period_ms; > > > > uint32_t rx_marker_period_ms; > > > > uint32_t update_timeout_ms; > > > > + rte_eth_bond_8023ad_ext_slowrx_fn slowrx_cb; > > > > }; > > > > > > This

[dpdk-dev] New driver (large patch) question.

2016-03-02 Thread Thomas Monjalon
Hi, 2016-03-01 19:56, Stephen Hurd: > I submitted a new driver on Friday, and it was rejected for being over 300k. > > The rejection email suggested contacing dev-owner at dpdk.org, which I did on > Monday with no reply. > > What's the process to submit patches larger than the mailing list size

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-03-02 Thread Vincent JARDIN
Le 02/03/2016 09:27, Panu Matilainen a ?crit : >>> I'd like to see these be merged. >>> >>> Jay >> >> The code is really not ready. I am okay with cooperative development >> but the current code needs to go into a staging type tree. >> No compatibility, no ABI guarantees, more of an RFC. >> Don't

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-03-02 Thread Jim Thompson
> On Mar 2, 2016, at 4:47 AM, Vincent JARDIN > wrote: > > Le 02/03/2016 09:27, Panu Matilainen a ?crit : I'd like to see these be merged. Jay >>> >>> The code is really not ready. I am okay with cooperative development >>> but the current code needs to go into a staging type

[dpdk-dev] [PATCH v2] app/testpmd Fix max_socket detection

2016-03-02 Thread De Lara Guarch, Pablo
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Stephen Hurd > Sent: Wednesday, January 13, 2016 10:24 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v2] app/testpmd Fix max_socket detection > > Previously, max_socket was set to the highest numbered

[dpdk-dev] [PATCH v4 2/4] kcp: add kernel control path kernel module

2016-03-02 Thread Ferruh Yigit
On 3/1/2016 11:06 PM, Stephen Hemminger wrote: > On Tue, 1 Mar 2016 15:41:58 + > Ferruh Yigit wrote: > >> +#ifdef RTE_KCP_KO_DEBUG >> +#define KCP_DBG(args...) pr_info(args) >> +#else >> +#define KCP_DBG(args...) >> +#en > > Why not use pr_debug() which is compile enabled already and

[dpdk-dev] [PATCH v4 2/4] kcp: add kernel control path kernel module

2016-03-02 Thread Ferruh Yigit
On 3/1/2016 11:10 PM, Stephen Hemminger wrote: > On Tue, 1 Mar 2016 15:41:58 + > Ferruh Yigit wrote: > >> +struct kcp_ethtool_msg { >> +int cmd_id; >> +int port_id; >> +unsigned int flag; >> +char input_buffer[KCP_ETHTOOL_MSG_LEN]; >> +char

[dpdk-dev] [PATCH v5 0/2] fm10k: enable FTAG based forwarding

2016-03-02 Thread Wang Xiao W
v5: * Used kvargs api to parse the devargs parameter. * Put release note into the driver patch. v4: * Removed the build time config option, used devargs to config FTAG. * Rebased on head of dpdk-next-net/rel_16_04 branch. v3: * Removed "\n" in PMD_INIT_LOG. * Returned "-ENOTSUP" instead of -1 in

[dpdk-dev] [PATCH v5 1/2] fm10k: enable FTAG based forwarding

2016-03-02 Thread Wang Xiao W
This patch enables reading sglort info into mbuf for RX and inserting an FTAG at the beginning of the packet for TX. The vlan_tci_outer field selected from rte_mbuf structure for sglort is not used in fm10k now. In FTAG based forwarding mode, the switch will forward packets according to glort info

[dpdk-dev] [PATCH v5 2/2] doc: add introduction for fm10k FTAG based forwarding

2016-03-02 Thread Wang Xiao W
Add a brief introduction on FTAG, describe what's FTAG and how it works in forwarding. Signed-off-by: Wang Xiao W --- doc/guides/nics/fm10k.rst | 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/doc/guides/nics/fm10k.rst b/doc/guides/nics/fm10k.rst index

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-03-02 Thread Thomas Monjalon
2016-03-02 11:47, Vincent JARDIN: > Le 02/03/2016 09:27, Panu Matilainen a ?crit : > >>> I'd like to see these be merged. > >>> > >>> Jay > >> > >> The code is really not ready. I am okay with cooperative development > >> but the current code needs to go into a staging type tree. > >> No

[dpdk-dev] [PATCH v2 00/12] extend flow director's fields in i40e driver

2016-03-02 Thread Jingjing Wu
v2 changes: - rebase on dpdk-next-net/rel_16_04 - comments rewording. - redefine the value of RTE_ETH_INPUT_SET_L3_IP4_TTL to avoid ABI breaking. - remove ABI announce in Deprecation. - fix the ethertype setting when program filter in v1 patch set. This patch set extends flow director to

[dpdk-dev] [PATCH v2 01/12] ethdev: extend flow director to support input set selection

2016-03-02 Thread Jingjing Wu
This patch added RTE_ETH_INPUT_SET_L3_IP4_TTL, RTE_ETH_INPUT_SET_L3_IP6_HOP_LIMITS input field type and extended struct rte_eth_ipv4_flow and rte_eth_ipv6_flow to support filtering by tos, protocol and ttl. Signed-off-by: Jingjing Wu --- lib/librte_ether/rte_eth_ctrl.h | 8 1 file

[dpdk-dev] [PATCH v2 02/12] i40e: split function for input set change of hash and fdir

2016-03-02 Thread Jingjing Wu
This patch split function for input set changing of hash and fdir to avoid multiple check on different situation. Signed-off-by: Jingjing Wu --- drivers/net/i40e/i40e_ethdev.c | 233 + drivers/net/i40e/i40e_ethdev.h | 11 +- drivers/net/i40e/i40e_fdir.c

[dpdk-dev] [PATCH v2 03/12] i40e: remove flex payload from INPUT_SET_SELECT operation

2016-03-02 Thread Jingjing Wu
In this patch, flex payload is removed from valid fdir input set values. It is because all flex payload configuration can be set in struct rte_fdir_conf during device configure phase. And it is a more flexible configuration including flexpayload's selection, input set selection by word and mask

[dpdk-dev] [PATCH v2 04/12] i40e: restore default setting on input set of filters

2016-03-02 Thread Jingjing Wu
This patch added a new function to set the input set to default when initialization. Signed-off-by: Jingjing Wu --- drivers/net/i40e/i40e_ethdev.c | 56 ++ 1 file changed, 56 insertions(+) diff --git a/drivers/net/i40e/i40e_ethdev.c

[dpdk-dev] [PATCH v2 05/12] i40e: extend flow director to filter by more IP Header fields

2016-03-02 Thread Jingjing Wu
This patch extended flow director to select more IP Header fields as filter input set. Signed-off-by: Jingjing Wu --- drivers/net/i40e/i40e_ethdev.c | 69 ++ drivers/net/i40e/i40e_fdir.c | 26 +++- 2 files changed, 75 insertions(+), 20

[dpdk-dev] [PATCH v2 06/12] testpmd: extend commands for filter's input set changing

2016-03-02 Thread Jingjing Wu
This patch extended commands for filter's input set changing. It added tos, protocol and ttl as filter's input fields, and remove the words selection from flex payloads for flow director. Signed-off-by: Jingjing Wu --- app/test-pmd/cmdline.c | 100

[dpdk-dev] [PATCH v2 07/12] librte_ether: extend rte_eth_fdir_flow to support tunnel format

2016-03-02 Thread Jingjing Wu
This patch changed rte_eth_fdir_flow from union to struct to support more packets formats, for example, Vxlan and GRE tunnel packets with IP inner frame. This patch also add new RTE_FDIR_TUNNEL_TYPE_GRE enum. Signed-off-by: Jingjing Wu --- doc/guides/rel_notes/deprecation.rst | 4

[dpdk-dev] [PATCH v2 09/12] testpmd: extend commands for fdir's tunnel id input set

2016-03-02 Thread Jingjing Wu
This patch extended commands for filter's input set changing. It added GRE/Vxlan Tunnel as filter's input fields. Signed-off-by: Jingjing Wu --- app/test-pmd/cmdline.c | 27 +-- doc/guides/testpmd_app_ug/testpmd_funcs.rst | 22 --

[dpdk-dev] [PATCH v2 08/12] i40e: extend flow director to filter by tunnel ID

2016-03-02 Thread Jingjing Wu
This patch extended flow director to select Vxlan/GRE tunnel ID as filter's input set and program the filter rule with the defined tunnel type. Signed-off-by: Jingjing Wu --- drivers/net/i40e/i40e_ethdev.c | 11 +++ drivers/net/i40e/i40e_fdir.c | 150 +++--

[dpdk-dev] [PATCH v2 11/12] i40e: extend flow director to filter by vlan id

2016-03-02 Thread Jingjing Wu
This patch extended flow director to select vlan id as filter's input set and program the filter rule with vlan id. Signed-off-by: Jingjing Wu --- doc/guides/rel_notes/release_16_04.rst | 2 ++ drivers/net/i40e/i40e_ethdev.c | 11 drivers/net/i40e/i40e_fdir.c | 49

[dpdk-dev] [PATCH v2 12/12] testpmd: extend commands for fdir's vlan input set

2016-03-02 Thread Jingjing Wu
This patch extended commands for filter's input set changing. It added vlan as filter's input fields. Signed-off-by: Jingjing Wu --- app/test-pmd/cmdline.c | 6 +++--- doc/guides/testpmd_app_ug/testpmd_funcs.rst | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff

[dpdk-dev] [PATCH v2 10/12] i40e: fix VLAN bitmasks for hash/fdir input sets for tunnels

2016-03-02 Thread Jingjing Wu
From: Andrey Chilikin This patch adds missing VLAN bitmask for inner frame in case of tunneling and fixes VLAN tags bitmasks for single or outer frame in case of tunneling. Fixes: 98f055707685 ("i40e: configure input fields for RSS or flow director") Signed-off-by:

[dpdk-dev] [PATCH v3] af_packet: make the device detachable

2016-03-02 Thread Wojciech Żmuda
Hi Panu, >I think its okay to remove without going through the deprecation process. > just drop the accidentally exported symbol from the 2.0 definition. Well, this is what I've done so far. I'm going to post v4 patch modifying release_16_04.rst instead of release_2_3.rst, then. Thank you for

[dpdk-dev] [PATCH v4] af_packet: make the device detachable

2016-03-02 Thread Wojciech Zmuda
Allow dynamic deallocation of af_packet device through proper API functions. To achieve this: * set device flag to RTE_ETH_DEV_DETACHABLE * implement rte_pmd_af_packet_devuninit() and expose it through rte_driver.uninit() * copy device name to ethdev->data to make discoverable with

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-03-02 Thread Vincent JARDIN
Le 02/03/2016 11:51, Jim Thompson a ?crit : > Can we take it as a requirement to support FreeBSD this time around? Of course, all OS should be on the loop, but I guess, it would be per kernel specific. What is ethtool on FreeBSD? Or can you start porting ethtool on FreeBSD?

[dpdk-dev] [PATCH v3 0/5] add dpdk packet capture support for tcpdump

2016-03-02 Thread Reshma Pattan
This patch set include the design to capture packets from dpdk ports for tcpdump. This patch set include test-pmd changes to verify below patch set. http://dpdk.org/dev/patchwork/patch/9750/ http://dpdk.org/dev/patchwork/patch/9751/ http://dpdk.org/dev/patchwork/patch/9752/ Dependencies 1:

[dpdk-dev] [PATCH v3 1/5] app/test-pmd: fix nb_rxq and nb_txq checks

2016-03-02 Thread Reshma Pattan
Made testpmd changes to validate nb_rxq/nb_txq zero value changes of librte_ether. Signed-off-by: Reshma Pattan --- app/test-pmd/cmdline.c| 11 +-- app/test-pmd/parameters.c | 14 +- app/test-pmd/testpmd.c| 28 +--- 3 files changed, 39

[dpdk-dev] [PATCH v3 2/5] drivers/net/pcap: add public api to create pcap device

2016-03-02 Thread Reshma Pattan
Added new public api to create pcap device from pcaps. Added new header file for API declaration. Added new public api to version map Signed-off-by: Reshma Pattan --- drivers/net/pcap/Makefile |4 +- drivers/net/pcap/rte_eth_pcap.c | 156

[dpdk-dev] [PATCH v3 4/5] lib/librte_eal: add tcpdump support in primary process

2016-03-02 Thread Reshma Pattan
Added tcpdump functionality to eal interrupt thread. Enhanced interrupt thread to support tcpdump socket and message processing from secondary. Created new mempool and rings to handle packets of tcpdump. Added rte_eth_rxtx_callbacks for ingress/egress packets processing for tcpdump. Added

[dpdk-dev] [PATCH v3 5/5] doc: update doc for tcpdump feature

2016-03-02 Thread Reshma Pattan
Added tcpdump design changes to proc_info section of sample application user guide. Added tcpdump design changes to env abstraction layer section of programmers guide. Updated Release notes Signed-off-by: Reshma Pattan --- doc/guides/prog_guide/env_abstraction_layer.rst | 43 -

[dpdk-dev] [PATCH v3 3/5] app/proc_info: add tcpdump support in secondary process

2016-03-02 Thread Reshma Pattan
Added below optional command line options for tcpdump support. 1)--tcupdump '(port,queue)' ==> port id and queue can be valid queue number or '*'. User need to specify specific queue id to capture packets on given queue for given port. (OR) '*' to capture packets from all queues of given port.

[dpdk-dev] [PATCH v2] mk: replace the combined library with a linker script

2016-03-02 Thread Panu Matilainen
On 03/01/2016 04:48 PM, Panu Matilainen wrote: > On 03/01/2016 03:40 PM, Thomas Monjalon wrote: >> ping >> I would like to be sure nothing is forgotten in this new revision. > > Sorry, didn't realize you were waiting for input from me, it feels a bit > strange to comment on something supposedly

[dpdk-dev] [PATCH v2] mk: replace the combined library with a linker script

2016-03-02 Thread Thomas Monjalon
2016-03-02 14:30, Panu Matilainen: > On 03/01/2016 04:48 PM, Panu Matilainen wrote: > > On 03/01/2016 03:40 PM, Thomas Monjalon wrote: > >> ping > >> I would like to be sure nothing is forgotten in this new revision. > > > > Sorry, didn't realize you were waiting for input from me, it feels a bit

[dpdk-dev] [PATCH v2] mk: replace the combined library with a linker script

2016-03-02 Thread Panu Matilainen
On 03/02/2016 02:40 PM, Thomas Monjalon wrote: > 2016-03-02 14:30, Panu Matilainen: >> On 03/01/2016 04:48 PM, Panu Matilainen wrote: >>> On 03/01/2016 03:40 PM, Thomas Monjalon wrote: ping I would like to be sure nothing is forgotten in this new revision. >>> >>> Sorry, didn't realize

[dpdk-dev] [PATCH] eal: arm64: introduce RTE_CPUFLAG_ATOMICS

2016-03-02 Thread Jerin Jacob
armv8.1 adds support for new atomic instructions. Linux kernel v4.3 onwards, the presence of atomic instruction support can detect through HWCAP_ATOMICS Signed-off-by: Jerin Jacob --- app/test/test_cpuflags.c | 3 +++ lib/librte_eal/common/arch/arm/rte_cpuflags.c

[dpdk-dev] [PATCH v4 1/3] fm10k: enable FTAG based forwarding

2016-03-02 Thread Thomas Monjalon
2016-03-01 14:37, Stephen Hemminger: > On Tue, 1 Mar 2016 13:36:39 +0800 > Wang Xiao W wrote: > > +static int > > +fm10k_check_ftag(struct rte_devargs *devargs) > > +{ > > + if (devargs == NULL) > > + return 0; > > + > > + if (strstr(devargs->args, "enable_ftag=1") == NULL) > > +

[dpdk-dev] [PATCH v1] I217 and I218 changes

2016-03-02 Thread Ravi Kerur
v1: Make necessary changes to support I217 and I218 NICs. Use v2' incorporating internal review comments as a base. Internal review done by Wenzhou Lu (Intel) and internal review versions and testing shown below v2': Incorporate Wenzhou's

[dpdk-dev] [PATCH v1] I217 and I218 changes

2016-03-02 Thread Ravi Kerur
v1: Make necessary changes to support I217 and I218 NICs. Use v2' incorporating internal review comments as a base. Internal review done by Wenzhuo Lu (Intel) and internal review versions and testing shown below v2': Incorporate Wenzhuo's

[dpdk-dev] [PATCH] hash: fix memcmp function pointer in multi-process

2016-03-02 Thread Dhananjaya Reddy Eadala
Michael Please find the attached is the patch file generated from "git format-patch -1" Thanks Dhana On Tue, Mar 1, 2016 at 9:29 PM, Qiu, Michael wrote: > On 3/2/2016 2:57 AM, Dhananjaya Reddy Eadala wrote: > > Hi > > > > We found a problem in dpdk-2.2 using under multi-process environment.

[dpdk-dev] [PATCH v2] I217 and I218 changes

2016-03-02 Thread Ravi Kerur
Hi, Wenzhuo, Bruce, I have superseded old patch with a new submission. Please let me know if that approach is ok. Thanks, Ravi On Tue, Mar 1, 2016 at 5:21 PM, Lu, Wenzhuo wrote: > Hi Ravi, > > > > >Do you want me to resend it as 'v1' and include your comments in commit > message? let me know.

[dpdk-dev] [PATCH] eal: arm64: introduce RTE_CPUFLAG_ATOMICS

2016-03-02 Thread Jan Viktorin
On Wed, 2 Mar 2016 18:50:59 +0530 Jerin Jacob wrote: > armv8.1 adds support for new atomic instructions. > Linux kernel v4.3 onwards, the presence of atomic instruction > support can detect through HWCAP_ATOMICS > > Signed-off-by: Jerin Jacob Reviewed-by: Jan Viktorin

[dpdk-dev] Multi Queue Support With Mellanox 40GbE NIC - MT27500 Family [ConnectX-3]

2016-03-02 Thread Olga Shern
Hi Suresh, RSS configuration is not supported with ConnectX-3 PMD. We added this support for ConnectX-4 PMD, available in DPDK 2.2. Best Regards, Olga -Original Message- From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Suresh Paruchuri Sent: Tuesday, March 01, 2016 8:03 PM To:

[dpdk-dev] [PATCH] hash: fix memcmp function pointer in multi-process

2016-03-02 Thread De Lara Guarch, Pablo
Hi Dhana, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Dhananjaya Reddy > Eadala > Sent: Wednesday, March 02, 2016 2:02 PM > To: Qiu, Michael > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] hash: fix memcmp function pointer in multi- > process >

[dpdk-dev] [PATCH v3] mk: stop on warning only in developer build

2016-03-02 Thread Thomas Monjalon
From: Panu Matilainen Add RTE_DEVEL_BUILD make-variable which can be used to do things differently when doing development vs building a release, autodetected from source root .git presence and overridable via commandline. It is used it to enable -Werror compiler flag and may

[dpdk-dev] [PATCH v2 1/2] lpm: extend ip4 next_hop and add config structure

2016-03-02 Thread Thomas Monjalon
Hi, As suggested by Bruce, I'm looking to apply this patchset before the ARM one. Please check the comments below. Is it possible to split the changes in next_hop and config structure? > --- a/doc/guides/rel_notes/release_2_3.rst > +++ b/doc/guides/rel_notes/release_2_3.rst > @@ -18,7 +18,11 @@

[dpdk-dev] [PATCH] mk: fix error message

2016-03-02 Thread Thomas Monjalon
When specifying a wrong directory with RTE_SDK and RTE_TARGET to build an application, the error message about missing config file was wrong. Fixes: 6b62a72a70d0 ("mk: install a standard cutomizable tree") Reported-by: Steeven Lee Signed-off-by: Thomas Monjalon --- mk/internal/rte.extvars.mk

[dpdk-dev] [PATCH] e1000: fix setting of VF MAC address

2016-03-02 Thread Bernard Iremonger
Allow reprogramming of the RAR with a zero mac address, to ensure that the VF traffic goes to the PF after stop, close and detach of the VF. Fixes: be2d648a2dd3 ("igb: add PF support") Fixes: d82170d27918 ("igb: add VF support") Signed-off-by: Bernard Iremonger ---

[dpdk-dev] [PATCH v5 1/1] eal/linux: change hugepage sorting to avoid overlapping memcpy

2016-03-02 Thread Thomas Monjalon
2016-01-07 15:54, Ralf Hoffmann: > with only one hugepage or already sorted hugepage addresses, the sort > function called memcpy with same src and dst pointer. Debugging with > valgrind will issue a warning about overlapping area. This patch changes > the sort method to qsort to avoid this

[dpdk-dev] [PATCH v2 1/2] lpm: extend ip4 next_hop and add config structure

2016-03-02 Thread Kobylinski, MichalX
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Wednesday, March 2, 2016 4:29 PM > To: Kobylinski, MichalX > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 1/2] lpm: extend ip4 next_hop and add config > structure > Importance: High > >

[dpdk-dev] New driver (large patch) question.

2016-03-02 Thread Stephen Hemminger
On Wed, 02 Mar 2016 11:21:26 +0100 Thomas Monjalon wrote: > Hi, > > 2016-03-01 19:56, Stephen Hurd: > > I submitted a new driver on Friday, and it was rejected for being over 300k. > > > > The rejection email suggested contacing dev-owner at dpdk.org, which I did > > on > > Monday with no

[dpdk-dev] [PATCH v2 1/2] lpm: extend ip4 next_hop and add config structure

2016-03-02 Thread Thomas Monjalon
2016-03-02 16:23, Kobylinski, MichalX: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > Sent: Wednesday, March 2, 2016 4:29 PM > > Is it possible to split the changes in next_hop and config structure? > > It is possible. Do you mean to put config structure into seperate

[dpdk-dev] [PATCH] eal: fix compile error in eal_timer.c caused by hpet

2016-03-02 Thread Thomas Monjalon
2016-01-28 14:16, ? ?: > Fix compile error when enable CONFIG_RTE_LIBEAL_USE_HPET. > > Error messages: > /root/dpdk-2.2.0/lib/librte_eal/linuxapp/eal/eal_timer.c: In function > ?rte_eal_hpet_init?: > /root/dpdk-2.2.0/lib/librte_eal/linuxapp/eal/eal_timer.c:222:2: error: > implicit declaration

[dpdk-dev] [PATCH v4 05/12] pmd/fm10k: add dev_ptype_info_get implementation

2016-03-02 Thread Chen, Jing D
Hi, -Original Message- From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Jianfeng Tan Sent: Thursday, February 25, 2016 6:09 PM To: dev at dpdk.org Subject: [dpdk-dev] [PATCH v4 05/12] pmd/fm10k: add dev_ptype_info_get implementation Signed-off-by: Jianfeng Tan ---

[dpdk-dev] [PATCH v3 1/2] librte_pipeline: add support for packet redirection at action handlers

2016-03-02 Thread Jasvinder Singh
Currently, there is no mechanism that allows the pipeline ports (in/out) and table action handlers to override the default forwarding decision (as previously configured per input port or in the table entry). Therefore, new pipeline API functions have been added which allows action handlers to

[dpdk-dev] [PATCH v3 2/2] modify action handlers in test_pipeline and ip_pipeline

2016-03-02 Thread Jasvinder Singh
Changes are made to the ports and table action handlers defined in app/test_pipeline and ip_pipeline sample application. Signed-off-by: Jasvinder Singh Acked-by: Cristian Dumitrescu --- app/test-pipeline/pipeline_acl.c | 3 +- app/test-pipeline/pipeline_hash.c

[dpdk-dev] [PATCH] nfp: fix non-x86 build

2016-03-02 Thread Bruce Richardson
On Mon, Feb 08, 2016 at 02:27:59PM +, Alejandro Lucero wrote: > On Sat, Feb 6, 2016 at 9:51 PM, Thomas Monjalon > wrote: > > > The file sys/io.h was included but it can be unavailable in some > > non-x86 toolchains. > > As others system includes in the file nfp_net.c, it seems useless, > >

[dpdk-dev] [PATCH] librte_port: fix segmentation fault for ethdev writer nodrop

2016-03-02 Thread Jasvinder Singh
Error log: [APP] Initializing PIPELINE0 ... pipeline> [APP] Initializing PIPELINE1 ... [PIPELINE1] Pass-through Segmentation fault (core dumped) Fixes: 304c8091e90a ("port: add ethdev writer nodrop") Signed-off-by: Jasvinder Singh Acked-by: Cristian Dumitrescu ---

[dpdk-dev] [PATCH] librte_port: fix segmentation fault for ring writer nodrop

2016-03-02 Thread Jasvinder Singh
Error log: [APP] Initializing PIPELINE0 ... pipeline> [APP] Initializing PIPELINE1 ... [PIPELINE1] Pass-through [APP] Initializing PIPELINE2 ... [PIPELINE2] Pass-through Segmentation fault (core dumped) Fixes: 5f4cd47309d6 ("port: add ring writer nodrop") Fixes: d58f69c54172 ("port: add

[dpdk-dev] [PATCH 0/3] add flag to disable CRC checksum offloading

2016-03-02 Thread Bruce Richardson
On Mon, Feb 15, 2016 at 05:55:22PM +0100, Paul Emmerich wrote: > This patch adds a new tx checksum offloading flag: PKT_TX_NO_CRC_CSUM. > This allows disabling CRC checksum offloading on a per-packet basis. > Doing this can be useful if you want to send out invalid packets on > purpose, e.g. in a

[dpdk-dev] [PATCH v2] I217 and I218 changes

2016-03-02 Thread Bruce Richardson
On Wed, Mar 02, 2016 at 06:02:02AM -0800, Ravi Kerur wrote: > Hi, Wenzhuo, Bruce, I have superseded old patch with a new submission. > Please let me know if that approach is ok. > > Thanks, > Ravi Superceding patches is standard. Please mark the old patch as superceded in patchwork so it doesn't

[dpdk-dev] New driver (large patch) question.

2016-03-02 Thread Stephen Hurd
The bulk of the patch is the hardware interface header file. With all the comments, it weighs in around 800k. If I strip the comments, it's around 300k. If I both strip all the comments and remove all the currently unused structures, I can get the entire patch down just below 300k, but that

[dpdk-dev] [PATCH] drivers/net/bnxt New driver for Broadcom bnxt

2016-03-02 Thread Stephen Hurd
Initial new driver for Broadcom bnxt (Cumulus) devices. - Adds drivers/net/bnxt and the librte_pmd_bnxt - Adds bnxt PCI IDs - Adds support for 2/2.5/25/50Gbps modes to rte_ethdev.h Signed-off-by: Stephen Hurd --- MAINTAINERS |4 + config/common_bsdapp

[dpdk-dev] [PATCH] drivers/net/bnxt New driver for Broadcom bnxt

2016-03-02 Thread Stephen Hemminger
Please split into pieces, at a minimum: * changes to base DPDK code, you are changing lib/librte_ether * driver files * config to enable, MAINTAINERS etc. Also lots and lots of checkpatch errors: Results of: DPDK_CHECKPATCH_PATH=$HOME/kernel/linux/scripts/checkpatch.pl

[dpdk-dev] New driver (large patch) question.

2016-03-02 Thread Thomas Monjalon
2016-03-02 13:30, Stephen Hurd: > The bulk of the patch is the hardware interface header file. With all the > comments, it weighs in around 800k. If I strip the comments, it's around > 300k. If I both strip all the comments and remove all the currently unused > structures, I can get the entire

[dpdk-dev] [PATCH] drivers/net/bnxt New driver for Broadcom bnxt

2016-03-02 Thread Thomas Monjalon
Please do not submit some disabled code and avoid #ifdef. 2016-03-02 13:36, Stephen Hurd: > +static struct eth_dev_ops bnxt_dev_ops = { > + .dev_infos_get = bnxt_dev_info_get_op, > + .dev_configure = bnxt_dev_configure_op, > + .dev_start = bnxt_dev_start_op, > + .dev_stop

[dpdk-dev] [PATCH v3] mk: stop on warning only in developer build

2016-03-02 Thread Bruce Richardson
On Wed, Mar 02, 2016 at 03:22:23PM +0100, Thomas Monjalon wrote: > From: Panu Matilainen > > Add RTE_DEVEL_BUILD make-variable which can be used to do things > differently when doing development vs building a release, > autodetected from source root .git presence and overridable via >

[dpdk-dev] New driver (large patch) question.

2016-03-02 Thread Stephen Hurd
The issue is that the big header in question is the whole hardware/firmware interface definition. The comments in it are the only publicly available documentation on the hardware I'm aware of. The driver itself doesn't have a lot of optional features in it, it's the header file that's too big.

  1   2   >