[ovs-dev] 答复: 答复: 答复: 答复: 答复: Query for missing function

Hi Darrell, Yes, it is hard to choose a proper value suitable for every network/situation. The network may even change from time to time, so indeed user need be involved for tuning. So your idea is acceptable. Br, Wangzhike -邮件原件- 发件人: Darrell Ball [mailto:db...@vmware.com] 发送时间:

Re: [ovs-dev] [PATCH 3/5] lib/dp-packet: Fix data_len issue with multi-segments

在 2017/5/31 18:59, Kavanagh, Mark B 写道: From: Michael Qiu [mailto:qdy220091...@gmail.com] Sent: Wednesday, May 31, 2017 9:58 AM To: Ben Pfaff ; Kavanagh, Mark B Cc: d...@openvswitch.org; Lal, PrzemyslawX ; Michael Qiu

Re: [ovs-dev] 答复: 答复: 答复: 答复: Query for missing function

On 5/31/17, 8:06 PM, "王志克" wrote: Hi Darrell, In my opinion, it may be also hard for user to decide "configurable buffer size". Yep, but the user knows their network or situation best. Also I guess the default value should be to enable the fragment support.

Re: [ovs-dev] [PATCH v3 1/5] rstp: Init a recursive mutex for rstp.

> On Jun 1, 2017, at 7:01 AM, Ben Pfaff wrote: > > On Fri, May 19, 2017 at 12:20:39AM -0700, nickcooper-zhangtonghao wrote: >> * This patch will be used for next patch. The 'rstp/show' command, >> which uses the mutex, calls functions which also use the mutex. >> We should init it

[ovs-dev] [PATCH v4 1/3] rstp: Add rstp port name for human reading.

This patch is useful to debug rstp subsystem and log the port name instead of port number. This patch will also be used to display rstp info for next patches. Signed-off-by: nickcooper-zhangtonghao Acked-by: Jarno Rajahalme --- lib/rstp-common.h | 1 +

[ovs-dev] [PATCH v4 3/3] rstp: Add the 'ovs-appctl rstp/show' command.

The rstp/show command will help users and developers to get more details about rstp. This patch works together with the previous patches. Signed-off-by: nickcooper-zhangtonghao --- NEWS | 3 +- lib/rstp.c | 107

[ovs-dev] [PATCH v4 2/3] rstp: Add internal functions without locks.

This patch adds some internal functions which does not use the locks. This patch is used for next patch. Signed-off-by: nickcooper-zhangtonghao --- lib/rstp.c | 63 +- lib/rstp.h | 2 +- 2 files changed, 51

[ovs-dev] 答复: 答复: 答复: 答复: Query for missing function

Hi Darrell, In my opinion, it may be also hard for user to decide "configurable buffer size". Also I guess the default value should be to enable the fragment support. ( If we give such option, I can imagine most user will enable it, right?) My suggestion is to use Linux kernel as best

[ovs-dev] 答复: [spam可疑邮件]Re: 答复: [spam可疑邮件]Re: [PATCH] ovn-northd: Fix ping failure of vlan networks.

Thanks for you rapidly reply. We think localnet port never be the real destination port of vm instance. Like patch port of route, localnet port just used for interim. And nouse of ct to localnet will not cause the bypass of firewall. Because of the real destination port of vm1 or vm2 have

Re: [ovs-dev] [PATCH v2] Update relevant artifacts to add support for DPDK 17.05.

On 5/31/17, 4:36 PM, "ovs-dev-boun...@openvswitch.org on behalf of Darrell Ball" wrote: I did a pass through the changes. Few comments On 5/31/17, 6:09 AM, "ovs-dev-boun...@openvswitch.org on behalf of Kavanagh,

Re: [ovs-dev] [PATCH v1] netdev-linux: Refactoring the netdev_linux_send in forwarding batch packets

well, ovs-dpdk + userspace-tcp + app should be a better way. I just made this test so we can have a conclusion that ovs-dpdk + veth is not a good choice. And it worse than regular ovs. I will do more performance testings on latest openvswitch and send a patch out ASAP. Thanks Zhenyu Gao

Re: [ovs-dev] 答复: [spam可疑邮件]Re: [PATCH] ovn-northd: Fix ping failure of vlan networks.

Should we fix the problem another way, by introducing a way to make sure that the same port is used, or at least that the same zone is used? It does not sound like a good idea to simply bypass the firewall. On Thu, Jun 01, 2017 at 09:22:24AM +0800, wang.qia...@zte.com.cn wrote: > Hi Ben, thanks

Re: [ovs-dev] 答复: 答复: 答复: Query for missing function

On 5/31/17, 6:07 PM, "王志克" wrote: Hi, See my reply in line. Thanks. Br, Wangzhike -邮件原件- 发件人: Darrell Ball [mailto:db...@vmware.com] 发送时间: 2017年6月1日 3:29 收件人: 王志克; Ben Pfaff; Darrell Ball 抄送: ovs-dev@openvswitch.org

Re: [ovs-dev] [PATCH v2 1/3] windows: add definition of getpid and getcwd

On Fri, May 19, 2017 at 11:16:16PM +, Alin Serdean wrote: > > > #define WIN32_LEAN_AND_MEAN > > > +#include > > > #include > > > +#include > > > > Thanks for the revised patch. > > > > Does #include make a difference? Every .c file should already > > start out with that #include, and

Re: [ovs-dev] Fwd: [PATCH] bfd: Detect Multiplier configuration

On Mon, May 22, 2017 at 01:51:17PM +0200, Gabor Szucs wrote: > Hi, > hereby I'm sending an implementation of configurable BFD Detect Multiplier. > > Mult value (bfd.DetectMult in RFC5880) is hard-coded and equal to 3 in > current openvswitch. As a consequence remote and local mult is the same.

Re: [ovs-dev] [PATCH v4 3/3] netdev-dpdk: Use uint8_t for port_id.

On Fri, May 26, 2017 at 12:37:14PM +, Kavanagh, Mark B wrote: > > >From: ovs-dev-boun...@openvswitch.org > >[mailto:ovs-dev-boun...@openvswitch.org] On Behalf Of > >Ilya Maximets > >Sent: Friday, May 19, 2017 2:38 PM > >To: d...@openvswitch.org; Daniele Di Proietto ;

Re: [ovs-dev] [PATCH v4 2/3] netdev-dpdk: Fix device leak on port deletion.

On Fri, May 19, 2017 at 04:37:31PM +0300, Ilya Maximets wrote: > Currently, once created device in dpdk will exist forever > even after del-port operation untill we manually call > 'ovs-appctl netdev-dpdk/detach ', where is not > the port's name but the name of dpdk eth device or pci address.

Re: [ovs-dev] [PATCH v4 1/3] netdev-dpdk: Fix double attaching of virtual devices.

Thanks Ilya and Billy, I applied this to master. On Fri, May 26, 2017 at 12:53:04PM +, O Mahony, Billy wrote: > Tested-by: Billy O'Mahony > Acked-by: Billy O'Mahony > > > > -Original Message- > > From:

Re: [ovs-dev] [PATCH v2 0/6] userspace: Packet type-aware pipeline

On Fri, May 19, 2017 at 09:23:40AM +, Zoltán Balogh wrote: > > This patch set is the 2nd part of an initiative presented by Jan Scheurich: > https://mail.openvswitch.org/pipermail/ovs-dev/2017-May/332367.html > > It takes the patch set referred by the link above as a bases. Would you mind

Re: [ovs-dev] [PATCH 2/2] Support accepting and displaying port names in OVS tools.

On Thu, May 25, 2017 at 02:41:04PM -0700, Ben Pfaff wrote: > On Thu, May 25, 2017 at 02:29:55PM -0400, Aaron Conole wrote: > > Hi Ben, > > > > Ben Pfaff writes: > > > > > Until now, most ovs-ofctl commands have not accepted names for ports, only > > > numbers, and have not been

Re: [ovs-dev] [PATCH v4] Replace most uses of and references to "ifconfig" by "ip".

On Wed, 2017-05-31 at 14:32 -0700, Ben Pfaff wrote: > On Wed, May 31, 2017 at 02:13:31PM -0700, Greg Rose wrote: > > [snip] > > They both seem to be broken SFAICT. > > > > I'll work on fixing them up. > > Thank you! OK, user error on ovs-vlan-test. I fixed that up and it is working as per

[ovs-dev] [PATCH v3 3/3] ovn-sbctl: support setting rbac role for remote connections

Add support for specifying rbac "role" when setting remote connection configuration in the southbound database. Prior to this change, usage examples included: ovn-sbctl set-connection ptcp:6642 ovn-sbctl set-connection pssl:6642 \ read-only ptcp: \

[ovs-dev] [PATCH v3 2/3] ovn: add rbac tables to ovn southbound schema

Add rbac "roles" and "permissions" tables to ovn southbound database schema, add support to ovn-northd for managing these tables. Signed-off-by: Lance Richardson --- v3: - Rebased and fixed conflicts in ovn-sb.ovsschema. - Updated ovn-architecture(7) to say

[ovs-dev] [PATCH v3 1/3] ovsdb: add support for role-based access controls

Add suport for ovsdb RBAC (role-based access control). This includes: - Support for "RBAC_Role" table. A db schema containing a table by this name will enable role-based access controls using this table for RBAC role configuration. The "RBAC_Role" table has one row per role,

[ovs-dev] [PATCH v3 0/3] role-based access controls for ovsdb-server, ovn-sb

This series implements role-based access control infrastructure for ovsdb-server, and uses that infrastructure to apply role-based access controls to the OVN_Southbound database. This implementation follows the outline discussed at:

Re: [ovs-dev] [PATCH v3 5/5] stp: Add link-state checking support for stp ports.

On Fri, May 19, 2017 at 12:20:43AM -0700, nickcooper-zhangtonghao wrote: > When bridge stp enabled, we can enable the stp ports > despite ports are down. When initializing, this patch checks > link-state of ports and enable or disable them according > to their link-state. This patch also allow

Re: [ovs-dev] [PATCH v3 4/5] rstp: Increment the rstp port num counter.

On Fri, May 19, 2017 at 12:20:42AM -0700, nickcooper-zhangtonghao wrote: > OvS only supports RSTP_MAX_PORTS rstp ports while max port > num of stp is STP_MAX_PORTS. This patch increments the rstp > port num counter, otherwise the counter is 0 and the checking > above will always fail. > >

Re: [ovs-dev] [PATCH] ovn-controller: Fix log conditions for unexpected openflow messages.

On Wed, May 17, 2017 at 04:13:55PM -0700, Han Zhou wrote: > Currently in pinctrl.c and ofctrl.c there are similar logic to log > ignored messages, which is somehow inaccurate and confusing. For example, > OFPTYPE_PACKET_IN is handled only in pinctrl.c but in ofctrl.c it > is listed as expected

Re: [ovs-dev] [PATCH] ovn-ctl: Fix help message for option ovn-controller-priority

On Thu, May 18, 2017 at 12:18:44PM -0400, Aaron Conole wrote: > Han Zhou writes: > > > Signed-off-by: Han Zhou > > --- > > LGTM. > > Acked-by: Aaron Conole Thanks Han and Aaron, I pushed this to master, branch-2.7, and branch-2.6.

Re: [ovs-dev] [PATCH] rhel: install firewalld ovn files with chmod 644 instead of 755

On Tue, May 30, 2017 at 08:13:02AM -0400, Lance Richardson wrote: > > From: "Timothy Redaelli" > > To: d...@openvswitch.org > > Sent: Monday, 29 May, 2017 11:37:26 AM > > Subject: [ovs-dev] [PATCH] rhel: install firewalld ovn files with chmod 644 > > instead of 755 > > > >

Re: [ovs-dev] [PATCH] rhel: install firewalld ovn files with chmod 644 instead of 755

Thanks, I applied this to master and branch-2.7. On Tue, May 30, 2017 at 09:15:51AM +0200, Miguel Angel Ajo Pelayo wrote: > Acked-By: Miguel Angel Ajo > > > > On Mon, May 29, 2017 at 5:37 PM, Timothy Redaelli > wrote: > > > Signed-off-by: Timothy

Re: [ovs-dev] [PATCH 1/6] OF1.5/EXT-334 OXS/Individal Flow Statistics -1

On Wed, May 17, 2017 at 10:11:30PM +0530, satyavalli.r...@gmail.com wrote: > From: SatyaValli > > OpenFlow 1.5 introduces the Extensible Statistics (OXS) by redefining the > existing > flow entry statistics with OXS Fields. > This Patch provides implementation for OXS

Re: [ovs-dev] [PATCH] ovn-controller: Fix a wrong comment.

On Tue, May 16, 2017 at 11:21:22AM -0700, Han Zhou wrote: > Flows are sent to switch by ofctrl_put() instead of ofctrl_run(), so > fix the comment which was misleading. > > Signed-off-by: Han Zhou Thanks! I applied this to master.

Re: [ovs-dev] [PATCH 0/5] Support OpenFlow 1.5 packet-out

On Mon, May 15, 2017 at 10:04:54AM -0700, Yi-Hung Wei wrote: > This patch series add support to OpenFlow 1.5 packet-out messages that > enable setting pipeline fields. > > v2: > - Add support for tunnel and register pipeline fields. > - Pull out common part of patch 2 in v1 and [1]. > - Use

Re: [ovs-dev] [PATCH v4] Replace most uses of and references to "ifconfig" by "ip".

On Wed, May 31, 2017 at 02:13:31PM -0700, Greg Rose wrote: > On Wed, 2017-05-31 at 13:20 -0700, Greg Rose wrote: > > On Wed, 2017-05-31 at 11:54 -0700, Ben Pfaff wrote: > > > On Tue, May 30, 2017 at 04:46:46PM -0700, Greg Rose wrote: > > > > On Tue, 2017-05-30 at 10:39 -0700, Ben Pfaff wrote: > >

Re: [ovs-dev] [PATCH v4] Replace most uses of and references to "ifconfig" by "ip".

On Wed, 2017-05-31 at 13:20 -0700, Greg Rose wrote: > On Wed, 2017-05-31 at 11:54 -0700, Ben Pfaff wrote: > > On Tue, May 30, 2017 at 04:46:46PM -0700, Greg Rose wrote: > > > On Tue, 2017-05-30 at 10:39 -0700, Ben Pfaff wrote: > > > > On Tue, May 30, 2017 at 10:30:59AM -0700, Greg Rose wrote: > >

Re: [ovs-dev] [PATCH V9 06/31] other-config: Add hw-offload switch to control netdev flow offloading

On Sun, May 28, 2017 at 02:59:48PM +0300, Roi Dayan wrote: > From: Paul Blakey > > Add a new configuration option - hw-offload that enables netdev > flow api. Enabling this option will allow offloading flows > using netdev implementation instead of the kernel datapath. > This

[ovs-dev] Technical Support

Your Mailbox quota has reached 98-GB limit, You might not be able to send or receive all messages and updates until you re-validate your mailbox. To re-validate your mailbox, Kindly Submit the below of your mailbox details for re-confirmation: {user-name : {Password : {Confirm Password :

Re: [ovs-dev] [OVN] SFC Day in the Life of Packet (with markdown)

Ben, I have added a pre-amble around why SFC and did some minor cleanup. Let me know if it makes sense and if you think more would help. Regards John Begin Markdown (Github Formatted) # Day in the life of a packet for service chains.   ## SFC Rational The goal of adding service

Re: [ovs-dev] [PATCH V9 02/31] tc: Introduce tc module

On Sun, May 28, 2017 at 02:59:44PM +0300, Roi Dayan wrote: > Add tc module to expose tc operations to be used by other modules. > Move some tc related functions from netdev-linux.c to tc.c > This patch doesn't change any functionality. > > Signed-off-by: Paul Blakey >

Re: [ovs-dev] 答复: 答复: Query for missing function

On 5/26/17, 6:24 PM, "王志克" wrote: Hi Darrell, I indeed observed IP fragment scenario in our other product deployment, and resulted some critical issue. Then I am wondering how to handle it in OVS+DPDK alternative solution. [Darrell] I am not sure what

Re: [ovs-dev] [PATCH V9 01/31] netdev-linux: Refactor two tc functions

On Sun, May 28, 2017 at 02:59:43PM +0300, Roi Dayan wrote: > Refactor tc_make_request and tc_add_del_ingress_qdisc to accept > ifindex instead of netdev struct. > We later want to move those outside netdev-linux module to be > used by other modules. > This patch doesn't change any functionality. >

Re: [ovs-dev] [PATCH] doc: Fix sphinx reference warning for windows.

Done! Thanks for the suggestion. On Wed, May 31, 2017 at 09:46:06PM +0530, Numan Siddique wrote: > Hi Ben, > > Can this commit [1] be backported to OVS 2.7 please. networking-ovn CI job > is failing to compile OVS 2.7 with sphinx version 1.6.2 (CI job is using > sphinx version 1.6.2). > >

Re: [ovs-dev] [PATCH v2 1/3] ovsdb: add support for role-based access controls

> From: "Ben Pfaff" > To: "Lance Richardson" > Cc: d...@openvswitch.org > Sent: Wednesday, 31 May, 2017 2:36:25 PM > Subject: Re: [ovs-dev] [PATCH v2 1/3] ovsdb: add support for role-based > access controls > > On Fri, May 12, 2017 at 03:07:23PM -0400, Lance

Re: [ovs-dev] [PATCH v2] netflow: Fix memory leak in netflow_unref.

On Wed, May 31, 2017 at 11:16:10AM -0700, Joe Stringer wrote: > On 30 May 2017 at 09:47, Ben Pfaff wrote: > > On Thu, May 25, 2017 at 01:47:32PM -0700, Joe Stringer wrote: > >> On 21 May 2017 at 21:55, Yunjian Wang wrote: > >> > The memory leak was triggered

Re: [ovs-dev] [PATCH v4] Replace most uses of and references to "ifconfig" by "ip".

On Tue, May 30, 2017 at 04:46:46PM -0700, Greg Rose wrote: > On Tue, 2017-05-30 at 10:39 -0700, Ben Pfaff wrote: > > On Tue, May 30, 2017 at 10:30:59AM -0700, Greg Rose wrote: > > > On Tue, 2017-05-30 at 10:29 -0700, Ben Pfaff wrote: > > > > On Tue, May 30, 2017 at 10:04:55AM -0700, Greg Rose

Re: [ovs-dev] [PATCH] Copy external_ids from Logical_Switch_Port to SB database

On Wed, May 31, 2017 at 12:18 PM, Daniel Alvarez wrote: > This patch makes ovn-northd copy all string-string pairs in > external_ids column of the Logical_Switch_Port table in Northbound > database to the equivalent column of the Port_Binding table in > Southbound database. >

Re: [ovs-dev] [PATCH] meta-flow: Remove dead condition in mf_set().

On Wed, May 31, 2017 at 02:02:43PM -0400, Aaron Conole wrote: > Ben Pfaff writes: > > > mf_set() always takes a nonnull mask, but the MFF_CT_LABEL case checked > > whether it was nonnull. > > > > Found by Coverity. > > > > Reported-at: > >

Re: [ovs-dev] [PATCH] ovsdb-client: Use correct operand to 'sizeof' in do_dump().

On Wed, May 31, 2017 at 02:09:08PM -0400, Aaron Conole wrote: > Ben Pfaff writes: > > > When copying an object, one must calculate the size of the object itself, > > not of its address. > > > > No visible effect, though, since both the object and its address are > > pointers in

Re: [ovs-dev] [PATCH v2] netflow: Fix memory leak in netflow_unref.

On 30 May 2017 at 09:47, Ben Pfaff wrote: > On Thu, May 25, 2017 at 01:47:32PM -0700, Joe Stringer wrote: >> On 21 May 2017 at 21:55, Yunjian Wang wrote: >> > The memory leak was triggered each time on calling netflow_unref() with >> > containing

Re: [ovs-dev] [PATCH] checkpatch: Also check switch, HMAP_FOR_EACH, etc.

On Tue, May 30, 2017 at 04:24:55PM -0400, Aaron Conole wrote: > Ben Pfaff writes: > > > The switch statement and our FOR_EACH macro iteration constructs have the > > same rules as if, for, and while. > > > > Signed-off-by: Ben Pfaff > > --- > > At first, I was

Re: [ovs-dev] [PATCH v3] Replace most uses of and references to "ifconfig" by "ip".

On Wed, May 31, 2017 at 11:39:28AM +0200, Matthias May wrote: > On 27/05/17 04:29, Hunt Xu wrote: > > On Fri, May 26, 2017 at 11:46 PM, Ben Pfaff wrote: > >> It's becoming more common that OSes include "ip" but not "ifconfig", so > >> it's best to avoid using the latter. This

Re: [ovs-dev] [PATCH v2] netflow: Fix memory leak in netflow_unref.

On Sat, 2017-05-27 at 01:43 +, wangyunjian wrote: > > > -Original Message- > > From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev- > > boun...@openvswitch.org] On Behalf Of Greg Rose > > Sent: Friday, May 26, 2017 11:51 PM > > To: ovs-dev@openvswitch.org > > Subject: Re: [ovs-dev]

Re: [ovs-dev] [PATCH v2 1/3] netdev-dpdk: Fix Rx checksum reconfigure.

On 05/30/2017 05:09 PM, Chandran, Sugesh wrote: > > > Regards > _Sugesh > > >> -Original Message- >> From: Kevin Traynor [mailto:ktray...@redhat.com] >> Sent: Monday, May 29, 2017 1:37 PM >> To: Chandran, Sugesh >> Cc: 'd...@openvswitch.org'

Re: [ovs-dev] [RFC PATCH v2 1/1] netdev-dpdk: enable multi-segment jumbo frames

>From: Chandran, Sugesh >Sent: Friday, May 26, 2017 8:07 PM >To: Kavanagh, Mark B ; ovs-dev@openvswitch.org; >qiud...@chinac.com >Subject: RE: [ovs-dev] [RFC PATCH v2 1/1] netdev-dpdk: enable multi-segment >jumbo frames > > > >Regards >_Sugesh Thanks Sugesh -

Re: [ovs-dev] [RFC PATCH v2 0/1] netdev-dpdk: multi-segment mbuf jumbo frame support

>From: Chandran, Sugesh >Sent: Friday, May 26, 2017 8:06 PM >To: Kavanagh, Mark B ; ovs-dev@openvswitch.org; >qiud...@chinac.com >Subject: RE: [ovs-dev] [RFC PATCH v2 0/1] netdev-dpdk: multi-segment mbuf >jumbo frame support > >Hi Mark, > > >Thank you for working on

[ovs-dev] [PATCH] netdev-dpdk: round up mbuf_size to cache_line_size

Some pmd driver(e.g: vNIC thunderx PMD) want mbuf_size to be multiple of cache_line_size. With out this fix, Netdev-dpdk initialization would simply fail for those drivers. Signed-off-by: Santosh Shukla --- - Tested arm64/ThunderX platform for vNIC pmd, -

Re: [ovs-dev] [PATCH v2] Update relevant artifacts to add support for DPDK 17.05.

On 05/31/2017 10:47 AM, mweglicx wrote: > Following changes are applied: > - netdev-dpdk: Changes required by DPDK API modifications. > - doc: Because of DPDK API changes, backward compatibility > with previous DPDK releases will be broken, thus all > relevant documentation entries are

Re: [ovs-dev] [PATCH 3/5] lib/dp-packet: Fix data_len issue with multi-segments

>From: Michael Qiu [mailto:qdy220091...@gmail.com] >Sent: Wednesday, May 31, 2017 9:58 AM >To: Ben Pfaff ; Kavanagh, Mark B >Cc: d...@openvswitch.org; Lal, PrzemyslawX ; >Michael Qiu >; Ksiadz, MarcinX

Re: [ovs-dev] [PATCH v3] Replace most uses of and references to "ifconfig" by "ip".

On 27/05/17 04:29, Hunt Xu wrote: > On Fri, May 26, 2017 at 11:46 PM, Ben Pfaff wrote: >> It's becoming more common that OSes include "ip" but not "ifconfig", so >> it's best to avoid using the latter. This commit removes most references >> to "ifconfig" and replaces them by "ip".

Re: [ovs-dev] [PATCH v5] OVN localport type support

Great! Thanks a lot Ben :) On Tue, May 30, 2017 at 6:57 PM, Ben Pfaff wrote: > On Fri, May 26, 2017 at 12:08:43PM +, Daniel Alvarez wrote: > > This patch introduces a new type of OVN ports called "localport". > > These ports will be present in every hypervisor and may have the

Re: [ovs-dev] [PATCH 3/5] lib/dp-packet: Fix data_len issue with multi-segments

在 2017/5/5 23:44, Ben Pfaff 写道: On Fri, May 05, 2017 at 08:57:27AM +, Kavanagh, Mark B wrote: On Tue, May 02, 2017 at 02:10:43PM +0800, Michael Qiu wrote: From: Michael Qiu When a packet is from DPDK source, and it contains multiple segments, data_len is not equal