If we do not add a new type of switch, we should write flag to a reg to
indicate the matched packets which are cloned to monitor.
This reg should add to all the pipeline stages of logical switch(both
ingress and egress) to distinguish from normal flow. Is this right for
Russell's point?
If we
-Original Message-
From: on behalf of Bhanuprakash Bodireddy
Date: Thursday, June 29, 2017 at 3:39 PM
To: "d...@openvswitch.org"
Subject: [ovs-dev] [PATCH v3 6/6] dpif-netdev: Flush the packets in
intermediate queue.
Under low rate traffic conditions, there can be 2 issues.
-Original Message-
From: on behalf of Bhanuprakash Bodireddy
Date: Thursday, June 29, 2017 at 3:39 PM
To: "d...@openvswitch.org"
Subject: [ovs-dev] [PATCH v3 4/6] netdev-dpdk: Add intermediate queue support.
This commit introduces netdev_dpdk_eth_tx_queue() function that
im
Hi Bhanu
Would it be possible to combine patches 1 and 2, rather than initially defining
an empty
netdev_txq_flush for dpdk ? I think the combined patch would have more context.
Darrell
-Original Message-
From: on behalf of Bhanuprakash Bodireddy
Date: Thursday, June 29, 2017 at 3:3
Sorry, I was multitasking last week and did not get a chance to finish the
responses on Friday
I looked thru. the code for all the patches
The last 3 patches of V3 needed a manual merge; as you know, the series needs a
rebase after recent commits.
For a full o/p batch case, I see about a 10% dro
openvswitch 2.6.2~pre+git20161223-3 is marked for autoremoval from testing on
2017-09-05
It (build-)depends on packages with these RC bugs:
853457: ipsec-tools: ftbfs with GCC-7
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mai
These are normal and unavoidable, because the vifs
disappear from the kernel before they are removed them from the OVS
database.
Signed-off-by: Roi Dayan
Reviewed-by: Paul Blakey
---
lib/netdev-linux.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/lib/netdev-linux.
From: Paul Blakey
Always implement get_ifindex without checking if offload is
enabled or not as this should not be related. From ovs-dpctl
we cannot tell if offload is enabled or not as other_config is
not being read.
Signed-off-by: Paul Blakey
Reviewed-by: Roi Dayan
---
lib/netdev-vport.c |
Hi,
The first patches is a fix for dumping vxlan rules with ovs-dpctl.
The second patch is to avoid false errors about getting ifindex
as the device might disappear already.
Thanks,
Roi
Paul Blakey (1):
netdev-vport: Always implement get_ifindex for netdev-vport
Roi Dayan (1):
netdev-linux
I am having a very hard time understanding what you're writing here.
Russell's point makes sense to me, but I don't understand your response.
Can you give some examples?
On Mon, Aug 07, 2017 at 09:40:06AM +0800, wang.qia...@zte.com.cn wrote:
> Not add new logical_mirror_switch, just use logical_sw
Not add new logical_mirror_switch, just use logical_switch of course can
capture the use case. But logical_switch pipeline is complex for flow
monitor. Flow monitor should ignore some tables such as port_security, lb
and so on. And also should consider normal function for normal ports. I
think
ALG infra and FTP (both V4 and V6) support is added to the userspace
datapath. Also, NAT support is included.
Signed-off-by: Darrell Ball
---
lib/conntrack-private.h | 40 +-
lib/conntrack.c | 1048 +++
lib/conntrack.h | 10 +-
3 f
In order to have full coverage of ALGs for the userspace
datapath, it is necessary to add 4 new tests. Three of these will
cover passive ftp, including basic V6 passive ftp, V4 passive ftp
with NAT and sequence skew and V6 passive ftp with NAT. The last
test will cover tftp with NAT. Before these a
Signed-off-by: Darrell Ball
---
NEWS | 1 +
1 file changed, 1 insertion(+)
diff --git a/NEWS b/NEWS
index 4df823d..7fe5aa7 100644
--- a/NEWS
+++ b/NEWS
@@ -36,6 +36,7 @@ v2.8.0 - xx xxx
abbreviated to 4 hex digits.
- Userspace Datapath:
* Added NAT support for userspace data
Both ipv4 and ipv6 are supported. Also, NAT support is included.
Signed-off-by: Darrell Ball
---
include/sparse/netinet/in.h | 3 ++-
lib/conntrack.c | 39 ++-
2 files changed, 40 insertions(+), 2 deletions(-)
diff --git a/include/sparse/netinet/
Signed-off-by: Darrell Ball
---
tests/system-userspace-macros.at | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/tests/system-userspace-macros.at b/tests/system-userspace-macros.at
index 3fc4ef9..f3337f0 100644
--- a/tests/system-userspace-macros.at
+++ b/tests/system-u
strcasestr is not defined for Windows, so implement a version
that could be used on Windows. This is needed for an upcoming
patch.
Signed-off-by: Darrell Ball
Co-authored-by: Ben Pfaff
Signed-off-by: Ben Pfaff
---
lib/string.c| 22 --
lib/string.h.in | 3 ++-
2 files c
ALG infra is added with support for FTP and TFTP.
Both V4 and V6 are supported. Also, NAT is supported.
Three passive ftp system tests are added to complete testing
coverage of ftp for the userspace datapath, as the existing
coverage of passive ftp was limited to one part of one test
for V4 only.
A new function conn_key_cmp() is introduced and used to replace
memcmp of conn_keys. Given that OVS runs on with many compilers and
on many architectures, it seems prudent to avoid memcmp in case
existing and future holes in conn_key are not handled by a given
compiler for a given architecture.
Si
Russell Bryant writes:
> On Fri, Aug 4, 2017 at 1:00 PM, Aaron Conole wrote:
>> After this commit, users may start a dpdk-enabled ovs setup as a
>> non-root user. This is accomplished by exporting the $HOME directory,
>> which dpdk uses to fill in it's semi-persistent RTE configuration.
>>
>> T
After splitting the unions of tcp/udp the sctp was forgotten
when parsing flower back to match.
Fixes: 2b1d9fa90909 ("tc: Split IPs and transport layer ports unions in flower
struct")
Signed-off-by: Roi Dayan
Reviewed-by: Paul Blakey
---
lib/netdev-tc-offloads.c |3 +++
1 files changed, 3
21 matches
Mail list logo
