On Thu, Jul 16, 2015 at 12:26:20PM +0530, Saloni Jain wrote:
So what I think that instead of ovs-ofctl table-mod to use an
OFPMP_TABLE_DESC request to obtain the current configuration, then
modify it according to the user's request, ovs-vswitchd should query
and store the current
On Tue, Jul 14, 2015 at 12:28:16PM -0300, Thadeu Lima de Souza Cascardo wrote:
On Mon, Jul 13, 2015 at 09:02:17AM -0700, Ben Pfaff wrote:
On Tue, Jul 07, 2015 at 03:38:51PM -0300, Thadeu Lima de Souza Cascardo
wrote:
When there is no vport for a given netdev, dpif_netlink_vport_get might
On Tue, Jul 14, 2015 at 07:45:42AM +, Pritesh Kothari (pritkoth) wrote:
On Jul 13, 2015, at 9:40 PM, Ben Pfaff b...@nicira.com wrote:
On Tue, Jul 14, 2015 at 12:34:14AM +, Pritesh Kothari (pritkoth) wrote:
How about automating this using travis and gerrit, so no commit gets in
v1-v2:
- The first path I took with this tried to model a provider network
as an OVN logical switch. This patch takes a different approach
suggested by Ben Pfaff where each connection to a provider network
is modeled as a 2-port OVN logical switch. More details below.
- This
Add a new OVN configuration entry in the Open_vSwitch database called
ovn-bridge-mappings. This allows the configuration of mappings
between a physical network name and an OVS bridge that provides
connectivity to that network.
For example, if you wanted to configure physnet1 to map to br-eth0
This file used mixed indentation. Fix it up to be consistent.
Signed-off-by: Russell Bryant rbry...@redhat.com
---
ovn/ovn-sb.xml | 74 +-
1 file changed, 37 insertions(+), 37 deletions(-)
diff --git a/ovn/ovn-sb.xml b/ovn/ovn-sb.xml
A recent patch added a new OVN configuration option called
ovn-bridge-mappings. Create some bridges and configure
ovn-bridge-mappings to make testing this part of OVN easier.
This setup will not interfere with anything unless you try to create
networks that make use of it.
Signed-off-by:
While parsing the OVN bridge mapping configuration, ensure that patch
ports exist between the OVN integration bridge and the physical
network bridge. If they do not exist, create them automatically.
Signed-off-by: Russell Bryant rbry...@redhat.com
---
ovn/controller/ovn-controller.c | 107
When getting the initial configuration, go ahead and store br_int in
the controller context. This is already done in the main loop, but a
future patch wants to use br_int from the controller context sooner in
the code.
Signed-off-by: Russell Bryant rbry...@redhat.com
---
A recent patch added type and options columns to the Logical_Port
table in OVN_Northbound. This patch allows you to get and set those
columns with ovn-nbctl.
Signed-off-by: Russell Bryant rbry...@redhat.com
---
ovn/ovn-nbctl.8.xml | 24 ++--
ovn/ovn-nbctl.c | 111
Add a method to determine of two smaps are equal (have the exact same
set of key-value pairs).
Signed-off-by: Russell Bryant rbry...@redhat.com
---
lib/smap.c | 34 ++
lib/smap.h | 2 ++
2 files changed, 36 insertions(+)
diff --git a/lib/smap.c b/lib/smap.c
The logic in parents_equal() and tags_equal() is useful for other
columns, so convert them into more generic functions that can be
reused.
Signed-off-by: Russell Bryant rbry...@redhat.com
---
ovn/northd/ovn-northd.c | 31 ---
1 file changed, 20 insertions(+), 11
We have started discussing the use of the logical port abstraction in
OVN to represent special types of connections into an OVN logical
switch. This patch proposes some schema updates to reflect these
special types of logical ports. A logical port can have a type and
a set of options specific to
On Thu, Jul 16, 2015 at 06:06:09PM -0400, Russell Bryant wrote:
This file used mixed indentation. Fix it up to be consistent.
Signed-off-by: Russell Bryant rbry...@redhat.com
Applied to master, thanks!
___
dev mailing list
dev@openvswitch.org
On Thu, Jul 16, 2015 at 4:03 PM, Jarno Rajahalme jrajaha...@nicira.com
wrote:
Alex,
Not a review, but a comment: Isn’t it unnecessary to use atomics when the
refcount is protected with a mutex?
That's very true, now atomics is unnecessary.
Also, in general, a thread should not release
On Jul 16, 2015, at 4:43 PM, Ben Pfaff b...@nicira.com wrote:
On Thu, Jul 16, 2015 at 03:15:53PM -0700, Jarno Rajahalme wrote:
MSVC does not like zero sized arrays in structs. Hence, remove the
'values' member from struct miniflow and add back the getters
miniflow_values() and
On Tue, Jul 14, 2015 at 11:57:09AM -0700, Gurucharan Shetty wrote:
MSVC 2013 update 4 was released in Nov 2014. Its release notes
says that it has fixed the problem wherein using designated
initializers to initialize unions within structs would
fail to compile.
Using designated initializers
On Wed, Jul 15, 2015 at 04:40:03PM +0300, Gal Sagie wrote:
On Wed, Jul 15, 2015 at 1:08 AM, Ben Pfaff b...@nicira.com wrote:
A router needs one IP address per port, not just one for the whole router.
Reported-by: Gal Sagie gal.sa...@gmail.com
Reported-at:
These features are not yet designed in OpenStack API, for Liberty (the next
release) its going to be only
bandwidth limit, but they certainly will be added to the next release as
there is a high demand for them.
I can start trying to propose them, but i think at this point its a waste
of time if
On Thu, Jul 16, 2015 at 01:12:34PM -0700, Jarno Rajahalme wrote:
I think the right fix here is to eliminate the ‘values’ field and
reintroduce the miniflow_get_values() getter. I’ll post a patch to
that effect as part of the v3 of the rest of the series.
Good idea. I've often found that
On 07/16/2015 04:23 PM, Gal Sagie wrote:
Similar to logical port, its convinent to use a 'name'
field for the logical router port
Signed-off-by: Gal Sagie gal.sagie@g gal.sa...@huawei.commail.com
This still isn't quite right. ;-)
--
Russell Bryant
On Wed, Jul 15, 2015 at 10:18:29PM -0700, Joe Stringer wrote:
This code attempts to first check whether another error was detected for
the string it is parsing, then if it's not at the end of the tokens,
report an error. However, 'errorp' is always a valid pointer to a
'char *', so the first
On Wed, Jul 15, 2015 at 3:48 PM, Ben Pfaff b...@nicira.com wrote:
On Mon, Jul 13, 2015 at 08:22:39PM -0700, Alex Wang wrote:
This commit adds ovn-sbctl to ovn family by using the db-ctl-base
library.
As a side effect, we move the ovn-nbctl/ovn-sbctl related files
into ovn/utilities.
Hi Ben,
According Open vSwitch Manual:
ovs-vswitchd can perform all configured bridging and switching locally, or it
can be configured to communicate with one or more external Open#8208;Flow
controllers. The switch is typically configured to connect to a primary
controller that takes
This commit adds the pipeline module to ovn-controller-vtep. The
module will scan through the Binding table, and create the
Ucast_Macs_Remote in vtep database for each logical port's MACs in
the same logical datapath (in ovn-sb database). Also the module will
create the physical locators to each
This commit adds the gateway module to ovn-controller-vtep. The
module will register the physical switches to ovnsb as chassis and
constantly update the vtep_logical_switches column in Chassis table.
Limitation:
- Do not support reading multiple tunnel ips of physical switch.
Signed-off-by:
On Fri, Jul 10, 2015 at 04:19:24PM +0200, Thomas Graf wrote:
From: Pravin Shelar pshe...@nicira.com
Removes all of the OVS specific GRE code and makes OVS use a
GRE net_device.
Signed-off-by: Pravin B Shelar pshe...@nicira.com
[snip]
@@ -115,6 +117,8 @@ static bool log_ecn_error = true;
On Wed, Jul 15, 2015 at 03:53:27PM -0700, Jesse Gross wrote:
On Wed, Jul 15, 2015 at 11:35 AM, Ben Pfaff b...@nicira.com wrote:
On Mon, Jul 13, 2015 at 02:53:56PM -0700, Jesse Gross wrote:
There are several implementations of functions that parse/format
flags and their binary
On Thu, Jul 16, 2015 at 01:35:14AM +, Alin Serdean wrote:
Hi Jarno,
This is breaking the windows build
(https://ci.appveyor.com/project/blp/ovs/build/1.0.459):
lib/classifier.c(1355) : error C2229: struct 'unnamed-tag' has an illegal
zero-sized array.
The problem is the following
This commit adds the binding module to ovn-controller-vtep. The
module will scan through the Binding table in ovnsb. If there is
a binding for a logical port in the vtep gateway chassis's
vtep_logical_switches map, sets the binding's chassis column to the
vtep gateway chassis.
Signed-off-by:
In a gateway like the VTEP L2 gateway, physical vlans belonging to
the same logical network form a logical switch. Each logical switch
has a dedicated tunnel key and will keep records of all MACs learned
from the owned vlans. So user can just send packet to a logical
switch and the gateway will
This commit adds ovn-sbctl to ovn family by using the db-ctl-base
library.
As a side effect, we move the ovn-nbctl/ovn-sbctl related files
into ovn/utilities.
Signed-off-by: Alex Wang al...@nicira.com
Acked-by: Ben Pfaff b...@nicira.com
---
V3-V4:
- mention ovn-sbctl should never be used in
This commit lays down the foundation for a new controller in OVN, the
ovn-controller-vtep, for controlling the vtep enabled gateways.
Limitation:
- How to make controller-vtep connect to OVN_SB?
my understanding is that vtep should not know the location of ovnsb.
but there is no
Hi Ben,
Thanks for the review.
All the comments have been incorporated in Version 3 of the patch.
I am not having any issues with make check. All the test cases in
testsuite are executing successfully on the latest master code.
Thanks
Niti Rohilla
On Wed, Jul 15, 2015 at 9:49 PM, Ben Pfaff
On 07/14/2015 06:08 PM, Ben Pfaff wrote:
A router needs one IP address per port, not just one for the whole router.
Reported-by: Gal Sagie gal.sa...@gmail.com
Reported-at: http://openvswitch.org/pipermail/dev/2015-July/057242.html
Signed-off-by: Ben Pfaff b...@nicira.com
---
On 07/16/2015 03:56 AM, Alex Wang wrote:
In a gateway like the VTEP L2 gateway, physical vlans belonging to
the same logical network form a logical switch. Each logical switch
has a dedicated tunnel key and will keep records of all MACs learned
from the owned vlans. So user can just send
Similar to logical port, its convinent to use a 'name'
field for the logical router port
Signed-off-by: Gal Sagie gal.sa...@huawei.com
---
ovn/ovn-nb.ovsschema | 1 +
ovn/ovn-nb.xml | 9 +
2 files changed, 10 insertions(+)
diff --git a/ovn/ovn-nb.ovsschema b/ovn/ovn-nb.ovsschema
On 07/16/2015 03:56 AM, Alex Wang wrote:
+static void
+usage(void)
+{
+printf(\
+%s: ovs-vswitchd management utility\n\
+\n\
+for debugging and testing only, never use it in production\n\
+\n\
+usage: %s [OPTIONS] COMMAND [ARG...]\n\
+\n\
+SouthBound DB commands:\n\
+ show
On 07/16/2015 03:56 AM, Alex Wang wrote:
This commit lays down the foundation for a new controller in OVN, the
ovn-controller-vtep, for controlling the vtep enabled gateways.
Limitation:
- How to make controller-vtep connect to OVN_SB?
my understanding is that vtep should not know
On 07/16/2015 07:51 AM, Gal Sagie wrote:
Similar to logical port, its convinent to use a 'name'
field for the logical router port
Signed-off-by: Gal Sagie gal.sa...@huawei.com
The patch seems fine to me, so:
Acked-by: Russell Bryant rbry...@redhat.com
However, your Signed-off-by email
On 07/16/15 at 05:59pm, Simon Horman wrote:
On Fri, Jul 10, 2015 at 04:19:24PM +0200, Thomas Graf wrote:
static void ipgre_tap_setup(struct net_device *dev)
{
ether_setup(dev);
- dev-netdev_ops = gre_tap_netdev_ops;
dev-priv_flags |= IFF_LIVE_ADDR_CHANGE;
On 07/16/2015 01:09 AM, Alex Wang wrote:
I'd like to see and try out your change. At the same time, I do want to
have
my vtep controller code reviewed. I think I'll first adopt your
suggestion and
rename the column to vtep_logical_switches and repost the series to
have it
reviewed.
On Wed, Jul 15, 2015 at 10:52 PM, Joe Stringer joestrin...@nicira.com wrote:
On 15 July 2015 at 17:41, Jesse Gross je...@nicira.com wrote:
On Wed, Jul 15, 2015 at 5:31 PM, Joe Stringer joestrin...@nicira.com wrote:
On 15 July 2015 at 15:53, Jesse Gross je...@nicira.com wrote:
On Wed, Jul 15,
On Mon, Jul 13, 2015 at 11:48 PM, Andy Zhou az...@nicira.com wrote:
Instead of require user to declare a global variable, pass the value
via ctl_init().
Signed-off-by: Andy Zhou az...@nicira.com
---
lib/db-ctl-base.c | 34 +-
lib/db-ctl-base.h | 17
Thx for making it consistent,
Acked-by: Alex Wang al...@nicira.com
On Mon, Jul 13, 2015 at 11:48 PM, Andy Zhou az...@nicira.com wrote:
all_commands currently is some times accessed directly, some times via
a function call. Change to always access it directly.
Signed-off-by: Andy Zhou
Could you extend the proposal to include those features then? At the
moment it is a skeleton.
On Thu, Jul 16, 2015 at 02:11:12AM +0300, Gal Sagie wrote:
Yes, there are currently plans for marking (DSCP) and flow classification
profiles.
And other ideas like connections limiting.
On Wed,
From: Niti niti.rohi...@tcs.com
This patch adds support for Openflow1.4 set/get asynchronous configuration
messages. OpenVSwitch already supports set/get asynchronous configuration
messages for Openflow1.3. In this patch OFPT_SET_ASYNC_CONFIG message
allows the controllers to set the
On Thu, Jul 16, 2015 at 6:25 AM, Russell Bryant rbry...@redhat.com wrote:
On 07/16/2015 03:56 AM, Alex Wang wrote:
In a gateway like the VTEP L2 gateway, physical vlans belonging to
the same logical network form a logical switch. Each logical switch
has a dedicated tunnel key and will
On Thu, Jul 16, 2015 at 6:41 AM, Russell Bryant rbry...@redhat.com wrote:
On 07/16/2015 03:56 AM, Alex Wang wrote:
This commit lays down the foundation for a new controller in OVN, the
ovn-controller-vtep, for controlling the vtep enabled gateways.
Limitation:
- How to make
On Thu, Jul 16, 2015 at 08:49:44AM -0400, Russell Bryant wrote:
On 07/16/2015 07:51 AM, Gal Sagie wrote:
Similar to logical port, its convinent to use a 'name'
field for the logical router port
Signed-off-by: Gal Sagie gal.sa...@huawei.com
The patch seems fine to me, so:
Acked-by:
On Thu, Jul 16, 2015 at 6:38 AM, Russell Bryant rbry...@redhat.com wrote:
On 07/16/2015 03:56 AM, Alex Wang wrote:
+static void
+usage(void)
+{
+printf(\
+%s: ovs-vswitchd management utility\n\
+\n\
+for debugging and testing only, never use it in production\n\
+\n\
+usage:
On Thu, Jul 16, 2015 at 6:56 AM, Russell Bryant rbry...@redhat.com wrote:
On 07/16/2015 01:09 AM, Alex Wang wrote:
I'd like to see and try out your change. At the same time, I do want to
have
my vtep controller code reviewed. I think I'll first adopt your
suggestion and
rename the
Thx for the cleanup,
I found myself wondering the same question,
Acked-by: Alex Wang al...@nicira.com
On Mon, Jul 13, 2015 at 11:48 PM, Andy Zhou az...@nicira.com wrote:
It is not clear why show command needs its own command table.
Remove it for simplicity.
Signed-off-by: Andy Zhou
On Thu, Jun 25, 2015 at 7:46 AM, Daniele Di Proietto
diproiet...@vmware.com wrote:
DPDK with vhost-user doesn't require libfuse, so we shouldn't link OVS
with libfuse unless DPDK is built with vhost-cuse support.
CC: Rapelly, Varun vrape...@sonusnet.com
Signed-off-by: Daniele Di Proietto
On Wed, Jul 15, 2015 at 10:18:26PM -0700, Joe Stringer wrote:
This check in parse_ofp_str__() attempted to detect inconsistencies
between matches and actions, or inconsistencies within the actions. In
this case, ofpacts_check() would effectively zero the usable_protocols
and return 0 (ie, OK).
I sent out a V4 series here:
http://openvswitch.org/pipermail/dev/2015-July/057570.html
On Thu, Jul 16, 2015 at 1:43 PM, Ben Pfaff b...@nicira.com wrote:
On Wed, Jul 15, 2015 at 11:02:29PM -0700, Alex Wang wrote:
On Wed, Jul 15, 2015 at 3:48 PM, Ben Pfaff b...@nicira.com wrote:
On Mon,
Won't work with changing [0] to [].
We need to shift the [0] to be the last member of a struct. I have no idea
about the implications that it will have but something like would work:
diff --git a/lib/classifier.c b/lib/classifier.c
index 2ed8697..9f58d43 100644
--- a/lib/classifier.c
+++
I think the right fix here is to eliminate the ‘values’ field and reintroduce
the miniflow_get_values() getter. I’ll post a patch to that effect as part of
the v3 of the rest of the series.
Jarno
On Jul 16, 2015, at 12:18 PM, Alin Serdean aserd...@cloudbasesolutions.com
wrote:
Won't
On Mon, Jun 22, 2015 at 10:38:42AM -0700, Jesse Gross wrote:
On Mon, Jun 22, 2015 at 10:11 AM, Ben Pfaff b...@nicira.com wrote:
On Thu, Jun 11, 2015 at 09:43:58AM -0700, Neil McKee wrote:
If new optional attribute OVS_USERSPACE_ATTR_ACTIONS is added to an
OVS_ACTION_ATTR_USERSPACE action,
On Thu, Jul 16, 2015 at 12:55 PM, Ben Pfaff b...@nicira.com wrote:
On Tue, Jul 14, 2015 at 07:45:42AM +, Pritesh Kothari (pritkoth) wrote:
On Jul 13, 2015, at 9:40 PM, Ben Pfaff b...@nicira.com wrote:
On Tue, Jul 14, 2015 at 12:34:14AM +, Pritesh Kothari (pritkoth)
wrote:
On Thu, Jul 16, 2015 at 09:47:19PM +0300, Gal Sagie wrote:
If i understand your concern right, you worry that the NB schema is getting
too big.
I can't think of a better way to do this both use full and smaller, if you
feel that we don't want to support
QoS at this point, thats fine. but this
On Wed, Jul 15, 2015 at 10:18:27PM -0700, Joe Stringer wrote:
OVS will exit if the allocations in this function fail, so this check is
pointless.
Found by MIT STACK analyzer.
Signed-off-by: Joe Stringer joestrin...@nicira.com
Acked-by: Ben Pfaff b...@nicira.com
Similar to logical port, its convinent to use a 'name'
field for the logical router port
Signed-off-by: Gal Sagie gal.sagie@g gal.sa...@huawei.commail.com
---
ovn/ovn-nb.ovsschema | 1 +
ovn/ovn-nb.xml | 9 +
2 files changed, 10 insertions(+)
diff --git a/ovn/ovn-nb.ovsschema
On Wed, Jul 15, 2015 at 11:02:29PM -0700, Alex Wang wrote:
On Wed, Jul 15, 2015 at 3:48 PM, Ben Pfaff b...@nicira.com wrote:
On Mon, Jul 13, 2015 at 08:22:39PM -0700, Alex Wang wrote:
This commit adds ovn-sbctl to ovn family by using the db-ctl-base
library.
As a side effect, we
On Wed, Jul 15, 2015 at 03:47:33PM -0700, Andy Zhou wrote:
Private variable perf_counters needs to be initialized before
use. Otherwise, perf_counter_init() call will cause crashes.
Signed-off-by: Andy Zhou az...@nicira.com
Acked-by: Ben Pfaff b...@nicira.com
On Wed, Jul 15, 2015 at 09:40:08PM -0700, Alex Wang wrote:
On Wed, Jul 15, 2015 at 1:10 PM, Ben Pfaff b...@nicira.com wrote:
On Mon, Jul 13, 2015 at 08:22:38PM -0700, Alex Wang wrote:
In a gateway like the VTEP L2 gateway, physical vlans belonging to
the same logical network form a
On Wed, Jul 15, 2015 at 10:18:28PM -0700, Joe Stringer wrote:
In several places, br is dereferenced just before a check such as
if (br ...). These checks may be optimised out, and they provide no
additional safety, so this patch removes them.
Found by MIT STACK analyzer.
Signed-off-by:
Tests pass for me now with v3, so please push these when you are
satisfied with them. Thank you!
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
On Mon, Jul 06, 2015 at 05:41:38PM -0400, Mussar, Gary wrote:
Fix detection of vhost_cuse in dpdk rte_config.h
Dpdk allows users to create a config that includes other config files and
then override values.
Eg.
defconfig_x86_64-native_vhost_cuse-linuxapp-gcc:
On Wed, Jul 15, 2015 at 04:36:58PM +0300, Gal Sagie wrote:
The change looks good to me.
Thanks, I applied this to master.
The question i have is how will public network be configured/connected to
the logical router in OVN?
(In neutron router can only be connected to one public network)
This
It has been observed that some DPDK device (e.g intel xl710) report an
high number of queues but make some of them available only for special
functions (SRIOV). Therefore the queues will be counted in
rte_eth_dev_info_get(), but rte_eth_tx_queue_setup() will fail.
This commit works around the
netdev_dpdk_set_multiq() should not set the number of configured rxq
and txq if the driver initialization fails (meaning that the driver
failed to setup the queues). Otherwise, on a subsequent call to
netdev_dpdk_set_multiq(), the code may believe that the queues have
already been setup and
I will re-send with gal.sa...@gmail.com
On Thu, Jul 16, 2015 at 7:07 PM, Ben Pfaff b...@nicira.com wrote:
On Thu, Jul 16, 2015 at 08:49:44AM -0400, Russell Bryant wrote:
On 07/16/2015 07:51 AM, Gal Sagie wrote:
Similar to logical port, its convinent to use a 'name'
field for the logical
On Tue, Jul 14, 2015 at 10:09:41PM -0400, Aaron Conole wrote:
For some time, it seems that the distcheck make target has not functioned
properly. This patch works around the automake imposed distcheck read-only
directory, as well as signaling to travis-ci that sudo is required (which is
Alex,
Not a review, but a comment: Isn’t it unnecessary to use atomics when the
refcount is protected with a mutex?
Also, in general, a thread should not release the last reference if other
threads can still find the object. Why does this happen here?
Jarno
On Jul 16, 2015, at 3:48 PM,
On Thu, Jul 16, 2015 at 06:06:12PM -0400, Russell Bryant wrote:
While parsing the OVN bridge mapping configuration, ensure that patch
ports exist between the OVN integration bridge and the physical
network bridge. If they do not exist, create them automatically.
Signed-off-by: Russell
On Jul 16, 2015, at 4:33 PM, Ben Pfaff b...@nicira.com wrote:
On Thu, Jul 16, 2015 at 03:15:52PM -0700, Jarno Rajahalme wrote:
I've seen core files appear and then be automatically removed as the
test case was successful. Such success is highly doubtful, so fail
the test cases if any core
On Jul 16, 2015, at 4:44 PM, Ben Pfaff b...@nicira.com wrote:
On Thu, Jul 16, 2015 at 03:15:54PM -0700, Jarno Rajahalme wrote:
Use two maps in miniflow to allow for expansion of struct flow past
512 bytes. We now have one map for tunnel related fields, and another
for the rest of the
In ovs 2.1, the unref functions for ipfix and sflow are called
in ofproto/ofproto-dpif-upcall.c without the protection of
any lock. If the unprotected call of unref function removes
the last reference, and at the same time, other thread is
trying to ref the same struct, the race could cause
On Thu, Jul 16, 2015 at 03:15:52PM -0700, Jarno Rajahalme wrote:
I've seen core files appear and then be automatically removed as the
test case was successful. Such success is highly doubtful, so fail
the test cases if any core files exist at the end of the test.
Signed-off-by: Jarno
While running ovn-controller under valgrind, I noticed several
variations of the same warning. Switching this allocation from malloc
to zalloc resolves it.
For reference, some examples from valgrind are:
==8487== Conditional jump or move depends on uninitialised value(s)
==8487==at
Sorry for this very delayed reply,
I think I found the issue, in branch 2.1.*
Will send out a fix soon,
At the same time, it branch = 2.3 do not have this issue due to the use of
ovs-rcu... And since branch 2.3 is the LTS branch, I would really recommend
you to switch to 2.3+
Thanks,
Alex
Introduce a new logical port type called localnet. A logical port
with this type also has an option called network_name. A localnet
logical port represents a connection to a locally accessible network.
ovn-controller will use the ovn-bridge-mappings configuration to
figure out which patch port
I've seen core files appear and then be automatically removed as the
test case was successful. Such success is highly doubtful, so fail
the test cases if any core files exist at the end of the test.
Signed-off-by: Jarno Rajahalme jrajaha...@nicira.com
---
tests/ofproto-macros.at |6 +-
MSVC does not like zero sized arrays in structs. Hence, remove the
'values' member from struct miniflow and add back the getters
miniflow_values() and miniflow_get_values().
Signed-off-by: Jarno Rajahalme jrajaha...@nicira.com
---
lib/classifier-private.h | 19 ++-
Use two maps in miniflow to allow for expansion of struct flow past
512 bytes. We now have one map for tunnel related fields, and another
for the rest of the packet metadata and actual packet header fields.
This split has the benefit that for non-tunneled packets the overhead
should be minimal.
On Thu, Jul 16, 2015 at 1:36 PM, Ben Pfaff b...@nicira.com wrote:
On Mon, Jun 22, 2015 at 10:38:42AM -0700, Jesse Gross wrote:
On Mon, Jun 22, 2015 at 10:11 AM, Ben Pfaff b...@nicira.com wrote:
On Thu, Jun 11, 2015 at 09:43:58AM -0700, Neil McKee wrote:
If new optional attribute
On Thu, Jul 16, 2015 at 03:15:54PM -0700, Jarno Rajahalme wrote:
Use two maps in miniflow to allow for expansion of struct flow past
512 bytes. We now have one map for tunnel related fields, and another
for the rest of the packet metadata and actual packet header fields.
This split has the
On Thu, Jul 16, 2015 at 06:06:10PM -0400, Russell Bryant wrote:
Add a new OVN configuration entry in the Open_vSwitch database called
ovn-bridge-mappings. This allows the configuration of mappings
between a physical network name and an OVS bridge that provides
connectivity to that network.
On Thu, Jul 16, 2015 at 7:52 AM, Thomas Graf tg...@suug.ch wrote:
On 07/16/15 at 05:59pm, Simon Horman wrote:
On Fri, Jul 10, 2015 at 04:19:24PM +0200, Thomas Graf wrote:
static void ipgre_tap_setup(struct net_device *dev)
{
ether_setup(dev);
- dev-netdev_ops =
On Jul 16, 2015, at 1:21 PM, Ben Pfaff b...@nicira.com wrote:
On Thu, Jul 16, 2015 at 01:12:34PM -0700, Jarno Rajahalme wrote:
I think the right fix here is to eliminate the ‘values’ field and
reintroduce the miniflow_get_values() getter. I’ll post a patch to
that effect as part of the v3
I posted a v3 of the patch series that includes a fix for this. However, I
don’t have the MSVC environment set up, so I’d be grateful if you could test it
out.
Thanks,
Jarno
On Jul 15, 2015, at 5:16 PM, Gurucharan Shetty shet...@nicira.com wrote:
tomorrow is fine, thank Jarno!
On
On Thu, Jul 16, 2015 at 04:55:11PM -0700, Ben Pfaff wrote:
On Thu, Jul 16, 2015 at 06:06:10PM -0400, Russell Bryant wrote:
Add a new OVN configuration entry in the Open_vSwitch database called
ovn-bridge-mappings. This allows the configuration of mappings
between a physical network name
Use two maps in miniflow to allow for expansion of struct flow past
512 bytes. We now have one map for tunnel related fields, and another
for the rest of the packet metadata and actual packet header fields.
This split has the benefit that for non-tunneled packets the overhead
should be minimal.
On Thu, Jul 16, 2015 at 04:12:25PM -0700, Pravin Shelar wrote:
On Thu, Jul 16, 2015 at 1:36 PM, Ben Pfaff b...@nicira.com wrote:
On Mon, Jun 22, 2015 at 10:38:42AM -0700, Jesse Gross wrote:
On Mon, Jun 22, 2015 at 10:11 AM, Ben Pfaff b...@nicira.com wrote:
On Thu, Jun 11, 2015 at
On Thu, Jul 16, 2015 at 03:15:53PM -0700, Jarno Rajahalme wrote:
MSVC does not like zero sized arrays in structs. Hence, remove the
'values' member from struct miniflow and add back the getters
miniflow_values() and miniflow_get_values().
Signed-off-by: Jarno Rajahalme jrajaha...@nicira.com
96 matches
Mail list logo
