Hello,
I think this just fixes a warning at the end of the DPDK compilation,
right?
[MW] Yes, exactly.
In any case, I prefer the version Flavio suggested
[MW]I have attached the patch, I think it is not worth to
start new mailing thread.
Br,
Michal.
-Original Message-
From: Daniel
> On Feb 19, 2016, at 4:40 PM, Ben Pfaff wrote:
>
> +pinctrl_handle_arp(const struct flow *ip_flow, struct ofpbuf *userdata)
> {
> ...
> +if (ip_flow->vlan_tci) {
> +eth_push_vlan(&packet, htons(ETH_TYPE_VLAN_8021Q),
> ip_flow->vlan_tci);
> +}
Not sure if it matters, since "ip_
'make check-system-userspace', 'make check-kernel' and 'make check-kmod'
work with real environment and can not be run simultaneously.
To prevent violation of the system environment strip out options for
parallel execution from TESTSUITEFLAGS for this targets.
Signed-off-by: Ilya Maximets
---
t
This prevents failure of test scenarios 'make check-kernel'
and 'make check-system-userspace' if creation of veth pairs or
network namespaces is not supported by kernel (or module isn't loaded).
Signed-off-by: Ilya Maximets
---
tests/system-common-macros.at | 6 +++---
1 file changed, 3 insertio
version 3:
* AT_SKIP_IF ---> AT_CHECK(... || return 77).
* Using of GNU make extentions removed.
version 2:
* 'testsuite: Add timeout to add_of_br() command.' removed
because already applied.
* New patch 'tests/automake.mk: Prohibition of parallel
Fixes failure of test due to not yet fully destroyed ports from
previous test if ovs-vswitchd was killed.
Example:
netdev_linux|WARN|br0: creating tap device failed: Device or resource busy
bridge|WARN|could not open network device br0 (Device or resource busy)
Signed-off-by: Ilya Maximets
---
On 29.02.2016 21:44, Ben Pfaff wrote:
> On Mon, Feb 29, 2016 at 04:06:52PM +0300, Ilya Maximets wrote:
>> 'make check-system-userspace', 'make check-kernel' and 'make check-kmod'
>> work with real environment and can not be run simultaneously.
>>
>> To prevent violation of the system environment st
On 29.02.2016 21:46, Ben Pfaff wrote:
> On Mon, Feb 29, 2016 at 04:06:50PM +0300, Ilya Maximets wrote:
>> This prevents failure of test scenarios 'make check-kernel'
>> and 'make check-system-userspace' if creation of veth pairs or
>> network namespaces is not supported by kernel (or module isn't l
On 29/02/2016 11:07, "Ben Pfaff" wrote:
>On Sun, Feb 28, 2016 at 12:13:12PM -0800, Daniele Di Proietto wrote:
>> Currently we treat set_multiq() calls specially in netdev and
>>dpif-netdev:
>> every pmd thread must be stopped and set_multiq() is allowed to destroy
>>and
>> recreate the device.
>
Thanks for reviewing the patch. I have sent a V2 with the default and
assert.
Thanks,
Sairam
On 2/29/16, 9:35 AM, "Alin Serdean"
wrote:
>Small nit can you please add a default to the switch and put an assert.
>
>
>
>Acked-by: Alin Gabriel Serdean
>
>
>
>
>
>> -Mesaj original-
>
>> De l
Alin,
Did you notice any change in the size of the binary? Eg. Ovs-vswitchd.exe?
With or without your change, the binary was the same size. I am wondering
if the default is /MT?
Thanks,
-- Nithin
-Original Message-
From: dev on behalf of Alin Serdean
Date: Friday, February 26, 2016 at
Michael Arnaldi wrote on 02/29/2016 09:55:54 AM:
> From: Michael Arnaldi
> To: Ryan Moats/Omaha/IBM@IBMUS
> Date: 02/29/2016 09:56 AM
> Subject: Re: [ovs-dev] [PATCH V4] Separation of NB & SB dbs.
>
> Hi Ryan,
>
> Sorry for delay it was a crazy week at work!
>
> I hope i will have time during th
On Mon, Feb 29, 2016 at 12:53:42PM -0800, Jarno Rajahalme wrote:
>
> > On Feb 26, 2016, at 4:48 PM, Ben Pfaff wrote:
> >
> > On Thu, Feb 25, 2016 at 01:20:17PM -0800, Jarno Rajahalme wrote:
> >> ONF Extension 230 adds support for OpenFlow 1.4 bundles to OpenFlow
> >> 1.3. Supporting this allows
On Mon, Feb 29, 2016 at 03:52:38PM -0500, Russell Bryant wrote:
> s/bofy/body/. I noticed this spelling typo while reading this header
> file.
>
> Signed-off-by: Russell Bryant
Acked-by: Ben Pfaff
___
dev mailing list
dev@openvswitch.org
http://openvs
> On Feb 26, 2016, at 4:48 PM, Ben Pfaff wrote:
>
> On Thu, Feb 25, 2016 at 01:20:17PM -0800, Jarno Rajahalme wrote:
>> ONF Extension 230 adds support for OpenFlow 1.4 bundles to OpenFlow
>> 1.3. Supporting this allows OpenFlow 1.3 controllers to start using
>> bundles. Also the ovs-ofctl '--b
s/bofy/body/. I noticed this spelling typo while reading this header
file.
Signed-off-by: Russell Bryant
---
lib/ofpbuf.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/ofpbuf.h b/lib/ofpbuf.h
index 489a543..8008cb4 100644
--- a/lib/ofpbuf.h
+++ b/lib/ofpbuf.h
@@ -39,7 +
Alin,
Can you pls. add KeMemoryBarrier(); after the ŒdataFlowState¹ was set to
OvsSwitchPaused?
What does a ŒNetEventFilterPreDetach¹ event mean in the context of a
Hyper-V switch? Will the OvsExtPause() get invoked after that? If so,
we¹ll have to update the ASSERT in OvsExtPause() which assumes
Currently, ``ovs-ctl start'' will attempt to start both the DB and
vswitchd. This is quite convenient when the database already has all of
the configuration values required, and when using a single services file
for systemd integration. The same goes for the ``ovs-ctl stop'' command.
However, ther
When a packet which needs segmentation is received, the header for
each segment is being calculated, i.e. IP length, checksum, TCP seq,
TCP checksum.
The problem with the current code is that it wrongly assumes that
the Ethernet frame payload is always an IPv4 packet.
This patch checks the EtherT
Thanks for the review.
Pushed to master,
Jarno
> On Feb 26, 2016, at 4:40 PM, Ben Pfaff wrote:
>
> On Thu, Feb 25, 2016 at 01:20:16PM -0800, Jarno Rajahalme wrote:
>> Even though the number of supported MPLS labels may vary between a
>> datapath and the OVS userspace, it is better to use the
Hi Sorin,
I look forward to the new patch. I had confused the usage of
OvsDeferredActionsQueueIsEmpty with the previous patch.
Thanks,
Sairam
On 2/27/16, 4:39 AM, "Sorin Vinturis"
wrote:
>Hi Sairam,
>
>Thank you for reviewing recirculation patches. Please see my answers
>inline.
>
>-Sorin
>
>-
On Sat, Feb 27, 2016 at 8:34 AM, Ben Pfaff wrote:
> On Fri, Feb 26, 2016 at 07:17:46PM -0800, Andy Zhou wrote:
> > When reporting remote status, A listening remote will randomly
> > pick a session and report its session status. This does not seem
> > to make much sense. It is probably better to l
On Fri, Feb 26, 2016 at 4:27 PM, Ben Pfaff wrote:
> On Wed, Feb 24, 2016 at 01:27:33PM -0800, Andy Zhou wrote:
> > Signed-off-by: Andy Zhou
>
> Acked-by: Ben Pfaff
>
Thanks joe and Ben for the review. Pushed to master.
___
dev mailing list
dev@openvsw
On Sun, Feb 28, 2016 at 12:13:12PM -0800, Daniele Di Proietto wrote:
> Currently we treat set_multiq() calls specially in netdev and dpif-netdev:
> every pmd thread must be stopped and set_multiq() is allowed to destroy and
> recreate the device.
>
> I think we can improve this by:
> * Generalizin
On Tue, Feb 23, 2016 at 01:31:01AM -0300, Flavio Leitner wrote:
> On Fri, 19 Feb 2016 14:30:31 -0500
> Lance Richardson wrote:
>
> > Add make targets for Fedora and RHEL7 RPMs, update INSTALL.Fedora.md
> > to document their use
> >
> > Added distribution tarball and rpm build directory to .gitig
On Mon, Feb 29, 2016 at 04:06:51PM +0300, Ilya Maximets wrote:
> Fixes failure of test due to not yet fully destroyed ports from
> previous test if ovs-vswitchd was killed.
>
> Example:
>
> netdev_linux|WARN|br0: creating tap device failed: Device or resource busy
> bridge|WARN|could not open net
On Mon, Feb 29, 2016 at 04:06:50PM +0300, Ilya Maximets wrote:
> This prevents failure of test scenarios 'make check-kernel'
> and 'make check-system-userspace' if creation of veth pairs or
> network namespaces is not supported by kernel (or module isn't loaded).
>
> Signed-off-by: Ilya Maximets
On Mon, Feb 29, 2016 at 04:06:52PM +0300, Ilya Maximets wrote:
> 'make check-system-userspace', 'make check-kernel' and 'make check-kmod'
> work with real environment and can not be run simultaneously.
>
> To prevent violation of the system environment strip out options for
> parallel execution fr
Thanks for the reminder Ben and sorry for the delay.
I think this just fixes a warning at the end of the DPDK compilation,
right?
In any case, I prefer the version Flavio suggested, which should be
equivalent
for the current OVS+DPDK build process.
Thanks
On 29/02/2016 10:22, "Ben Pfaff" wrote
On Thu, Feb 11, 2016 at 02:29:07PM +, Zoltán Balogh wrote:
> Hi,
>
> The idea is to use OFPPS_LIVE bit to propagate link aliveness state towards
> the controller also when sending port status.
> The ofport->may_enable flag could be used for this purpose. I updated some
> unit tests according
Daniele, have you taken a look at this patch?
On Mon, Feb 22, 2016 at 01:49:54PM -0300, Flavio Leitner wrote:
>
> Adding Daniele just in case.
>
> fbl
>
> On Mon, 22 Feb 2016 10:13:10 +
> "Weglicki, MichalX" wrote:
>
> > Hello,
> >
> > Just gentle reminder, is this patch fine? Or I'm su
On Wed, Jan 20, 2016 at 03:14:59PM +0900, Simon Horman wrote:
> This series implements support for layer 3 ports. By removing the
> assumption that all packets/flows have Ethernet header, this series adds
> generic support for layer 3 ports in OVS. Support for layer 3 GRE tunnels
> is also added b
That's not how the Python version of the IDL is meant to be used.
SchemaHelper operates on the schema that the client *wants*, not on the
schema that the database server *has*. That's why it reads the schema
from a local disk file by default. If Neutron does something else then
it should probably
> On Feb 29, 2016, at 8:09 AM, Russell Bryant wrote:
>
> On Sat, Feb 27, 2016 at 4:45 PM, Justin Pettit wrote:
>
> >> +v2.5.1 - 26 Feb 2016
> >> +-
> >> +
> >> +
> >
> > Did we release 2.5.1 yesterday too? :-)
>
> We always do that when we prepare for a release. That way
This macro is not required as we drop support for unsupported
kernel versions.
Signed-off-by: Pravin B Shelar
Acked-by: Jesse Gross
---
acinclude.m4 | 9 -
datapath/compat.h | 5 -
datapat
Currently OVS out of tree datapath supports a large number of kernel
versions. From 2.6.32 to 4.3 and various distribution-specific
kernels. But at this point major features are only available on more
recent kernels. For example, stateful services are only available
starting in kernel 3.10 and STT
You are right Sai, the flags need to be set before the checksum is computed.
I acked you patch.
Alin.
> -Mesaj original-
> De la: Sairam Venugopal [mailto:vsai...@vmware.com]
> Trimis: Friday, February 26, 2016 12:56 AM
> Către: Sorin Vinturis ;
> dev@openvswitch.org
> Cc: Alin Serdean
Small nit can you please add a default to the switch and put an assert.
Acked-by: Alin Gabriel Serdean
> -Mesaj original-
> De la: dev [mailto:dev-boun...@openvswitch.org] În numele Sairam
> Venugopal
> Trimis: Friday, February 26, 2016 1:07 AM
> Către: dev@openvswitch.org
> Subiect: [o
On 28 February 2016 at 22:33, Gurucharan Shetty wrote:
> This series adds support for OVN loadbalancing. This is very
> lightly tested (with a very simple OVN topology of 1 router
> and 2 lswitches), but the patches are far enough to get any
> early feedback.
>
> The patches were tested on Linux
From: Jarno Rajahalme
Signed-off-by: Jarno Rajahalme
---
lib/ofp-actions.c| 3 +-
ofproto/ofproto-dpif-xlate.c | 21
tests/system-traffic.at | 225 +++
3 files changed, 248 insertions(+), 1 deletion(-)
diff --git a/lib/ofp-action
Invalid packets get dropped. Established connectections
need to be sent through conntrack with just "nat" so
that the VIP is changed to a loadbalanced IP address (by
setting reg1 = 2). New connections need to be loadbalanced
to one of the destination IP addresses (by setting reg1 = 3).
This introd
Future patches introduce more tables between
pre-ACL and ACL processing. As such, it looks
easier to separate these out into separate
functions to enhance code readability.
Signed-off-by: Gurucharan Shetty
---
ovn/northd/ovn-northd.c | 26 ++
1 file changed, 18 insertions
This new table sits before the pre-Stateful table and sets
'reg0' as 1, if the destination ip address of a packet
is a VIP in a loadbalancer object. Setting 'reg0' as 1 will
send the packet through conntrack to get its status (or to
track it.)
Signed-off-by: Gurucharan Shetty
---
ovn/northd/ovn-
Signed-off-by: Gurucharan Shetty
---
ovn/controller/lflow.c | 3 +-
ovn/controller/lflow.h | 2 +
ovn/controller/ofctrl.c | 134 +-
ovn/controller/ofctrl.h | 3 +-
ovn/controller/ovn-controller.c | 22 ++-
ovn/lib/ac
Also add the ability to run database commands on the
new schema using ovn-nbctl.
Signed-off-by: Gurucharan Shetty
---
ovn/ovn-nb.ovsschema | 18 --
ovn/ovn-nb.xml| 24
ovn/utilities/ovn-nbctl.c | 4
3 files changed, 44 insertions(+)
On Mon, Feb 29, 2016 at 7:26 AM, Huang Lei <148012...@qq.com> wrote:
> From: Huang Lei
>
> For setting the inactivity probe interval of the json session from
> ovn-controller to the OVN southbound database.
>
> Signed-off-by: Huang Lei
> ---
> lib/ovsdb-idl.c | 10 ++
Currently, the only use of stateful services in conntrack is
OVN ACLs. In table ACL, we commit the packet to conntrack
via ct_commit action.
As we introduce more stateful services, the ACL feature will
have to share the conntrack module with others. As
preparation for more stateful features like N
Currently, the only use of stateful services in conntrack is
OVN ACLs. In table pre-ACL, we send the packet to conntrack
to track it (to get its status) via the ct_next action.
As we introduce more stateful services, the ACL feature will
have to share the conntrack module with others. As
preparati
This series adds support for OVN loadbalancing. This is very
lightly tested (with a very simple OVN topology of 1 router
and 2 lswitches), but the patches are far enough to get any
early feedback.
The patches were tested on Linux kernel 4.4 with Jarno's
NAT patches (sent for review upstream) compi
On Sat, Feb 27, 2016 at 4:45 PM, Justin Pettit wrote:
>
> > On Feb 27, 2016, at 12:08 PM, Joe Stringer wrote:
> >
> > On 24 February 2016 at 04:29, Justin Pettit wrote:
> >> Signed-off-by: Justin Pettit
> >> ---
> >> NEWS | 4
> >> configure.ac | 2 +-
> >> debian/changelog
Ryan Moats/Omaha/IBM wrote on 02/23/2016 12:44:53 PM:
> From: Ryan Moats/Omaha/IBM
> To: arnaldimich...@gmail.com
> Date: 02/23/2016 12:44 PM
> Subject: Fw: [ovs-dev] [PATCH V4] Separation of NB & SB dbs.
>
> FH
>
> > From V3 removed any unnecessary code change and removed cloning of the
> > runni
"discuss" wrote on 02/29/2016 08:37:02
AM:
> From: Ryan Moats/Omaha/IBM@IBMUS
> To: disc...@openvswitch.org
> Date: 02/29/2016 08:37 AM
> Subject: [ovs-discuss] OVSDB change tracking - is the delete case
> really usable?
> Sent by: "discuss"
>
> All-
>
> I bumped into the following while trying
On Fri, Feb 26, 2016 at 7:51 PM, Ben Pfaff wrote:
> On Thu, Feb 25, 2016 at 11:01:19PM -0500, Russell Bryant wrote:
> > From: Russell Bryant
> >
> > Add support for matching on ct_mark and ct_label in OVN logical flows.
> >
> > Signed-off-by: Russell Bryant
>
> Acked-by: Ben Pfaff
>
Thanks.
On 27.02.2016 03:59, Ben Pfaff wrote:
> On Fri, Feb 26, 2016 at 02:57:39PM +0300, Ilya Maximets wrote:
>> This prevents failure of test scenarios 'make check-kernel'
>> and 'make check-system-userspace' if creation of veth pairs or
>> network namespaces is not supported by kernel (or module isn't l
This prevents failure of test scenarios 'make check-kernel'
and 'make check-system-userspace' if creation of veth pairs or
network namespaces is not supported by kernel (or module isn't loaded).
Signed-off-by: Ilya Maximets
---
tests/system-common-macros.at | 6 +++---
1 file changed, 3 insertio
'make check-system-userspace', 'make check-kernel' and 'make check-kmod'
work with real environment and can not be run simultaneously.
To prevent violation of the system environment strip out options for
parallel execution from TESTSUITEFLAGS for this targets.
Signed-off-by: Ilya Maximets
---
t
Fixes failure of test due to not yet fully destroyed ports from
previous test if ovs-vswitchd was killed.
Example:
netdev_linux|WARN|br0: creating tap device failed: Device or resource busy
bridge|WARN|could not open network device br0 (Device or resource busy)
Signed-off-by: Ilya Maximets
---
version 2:
* 'testsuite: Add timeout to add_of_br() command.' removed
because already applied.
* New patch 'tests/automake.mk: Prohibition of parallel
system-traffic test execution.'
* delay after ovs-vswitchd killing replaced with 'Waiting
for
From: Huang Lei
For setting the inactivity probe interval of the json session from
ovn-controller to the OVN southbound database.
Signed-off-by: Huang Lei
---
lib/ovsdb-idl.c | 10 ++
lib/ovsdb-idl.h | 4
ovn/controller/ovn-controller.8.xm
Dear user of openvswitch.org,
We have detected that your email account has been used to send a large amount
of unsolicited email during this week.
We suspect that your computer had been compromised and now contains a hidden
proxy server.
We recommend you to follow instruction in order to keep y
On 27.02.2016 01:53, Joe Stringer wrote:
> On 26 February 2016 at 03:57, Ilya Maximets wrote:
>> Fixes failure of test due to not yet fully destroyed ports from
>> previous test if ovs-vswitchd was killed.
>>
>> Example:
>>
>> netdev_linux|WARN|br0: creating tap device failed: Device or resource b
Hi Dball,
I am not clear about the localnet workflow, pls clarify, thanks.
The current usage of ovn provider network is like this,
1, first create ovs bridge which connects to provider network, for example
br-provider
$ ovs-vsctl add-br br-provider
2, create a mapping between network and bridge
$
Take a loot at the contractor of IDL:
...
def __init__(self, remote, schema):
...
You can see that it gets schema as a parameter, which is actually
SchemaHelper object:
...
assert isinstance(schema, SchemaHelper)
...
This object holds the schema in a json string (schema_json) until you will
c
63 matches
Mail list logo
