On 06/24/2015 05:06 AM, Pravin Shelar wrote:
On Tue, Jun 23, 2015 at 2:51 PM, Jesse Gross wrote:
On Mon, Jun 22, 2015 at 8:08 PM, Pravin Shelar wrote:
On Fri, Jun 19, 2015 at 11:24 AM, Daniele Di Proietto
wrote:
On 18/06/2015 23:57, "Traynor, Kevin" wrote:
-Original Message-
> -Original Message-
> From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Panu Matilainen
> Sent: Wednesday, June 24, 2015 9:33 AM
> To: Pravin Shelar; Jesse Gross
> Cc: dev@openvswitch.org; Flavio Leitner
> Subject: Re: [ovs-dev] [PATCH] dpif-netdev: Check for PKT_RX_RSS_HASH fla
Hi Ben,
Thanks, my query has been resolved and the code is updated in version 10
--"[PATCH v10] ovs-ofctl:Implementation of eviction on the basis of Importance"
sent on 17-June-2015.
Sending a new patch version 11, rebased with the latest master.
Thanks and Regards,
Saloni Jain
Tata Consultancy
From: Saloni Jain
This commit enables the eviction mechanism on the basis of importance as
per the openflow specification 1.4.
"ovs-ofctl -O OpenFlow14 mod-table evict"
Enable eviction on of . Eviction adds a mechanism
enabling the switch to automatically eliminate entries of lower
importance
Bug found in OvsPartialCopyToMultipleNBLs function in the cleanup part of
the code. Before completing the current NBL (newNbl) the NEXT link for the
following NBL (firstNbl) was broken, instead of the current one (newNbl).
This patch is to be applied both on master and on 2.4 branch.
Signed-off-b
On 06/23/2015 06:56 PM, Ben Pfaff wrote:
> On Tue, Jun 23, 2015 at 11:58:25PM +0200, Salvatore Orlando wrote:
>> I'm afraid I have to start bike shedding on this thread too.
>> Apparently that's what I do best.
>
> These are important clarifications, not bikeshedding.
>
> More below (with lots of
>When using multiple PMDs and numerous ports, a performance gain
>may be achieved in some use cases by pinning a PMD/port to a
>particular (set of) core(s).
>
>This patch provides a summary of the switch's port/core affinities
>each time that the status of the switch's ports is modified.
>Based on
On Tue, Jun 23, 2015 at 6:11 PM, Alin Serdean
wrote:
> How about the following:
>
> Before running the unit tests make sure to add the pthread libraries to your
> PATH environment variable.
The above is already part of the documentation with the following wording:
You should add the pthread-win32
You can look at "commit 5e65e080ad4d57e" for more information on the
exact problem with taskkill.
One option is to check for tskill and if it does not exist, use taskkill.
On Tue, Jun 23, 2015 at 6:37 PM, Alin Serdean
wrote:
> Think I found why tskill is missing on my system. It is part of the r
[resending to the ovs-dev as I sent my original reply to Russell only]
Comments inline
Salvatore
On 24 June 2015 at 16:15, Russell Bryant wrote:
> On 06/23/2015 06:56 PM, Ben Pfaff wrote:
> > On Tue, Jun 23, 2015 at 11:58:25PM +0200, Salvatore Orlando wrote:
> >> I'm afraid I have to start bik
> On Jun 24, 2015, at 3:56 AM, Sorin Vinturis
> wrote:
>
> Bug found in OvsPartialCopyToMultipleNBLs function in the cleanup part of
> the code. Before completing the current NBL (newNbl) the NEXT link for the
> following NBL (firstNbl) was broken, instead of the current one (newNbl).
>
> This
Signed-off-by: Daniele Di Proietto
---
lib/netdev-dpdk.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
index 3af1ee7..c03ff3d 100644
--- a/lib/netdev-dpdk.c
+++ b/lib/netdev-dpdk.c
@@ -93,8 +93,8 @@ BUILD_ASSERT_DECL((MAX_NB_M
A vhost device, being a virtual interface, doesn't provide a valid RSS
hash. As a workaround we set the value to 0, which suggests the
datapath to recompute the hash in software.
Reported-by: Dongjun
CC: Traynor, Kevin
CC: Flavio Leitner
Signed-off-by: Daniele Di Proietto
---
lib/netdev-dpdk
On 24/06/2015 09:47, "Traynor, Kevin" wrote:
>
>
>> -Original Message-
>
>> From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Panu
>>Matilainen
>
>> Sent: Wednesday, June 24, 2015 9:33 AM
>
>> To: Pravin Shelar; Jesse Gross
>
>> Cc: dev@openvswitch.org; Flavio Leitner
>
>> Sub
On 24/06/2015 02:48, "Pravin Shelar" wrote:
>On Tue, Jun 23, 2015 at 11:42 AM, Ben Pfaff wrote:
>> Do you two have an opinion on this? If DPDK support is pretty solid now
>> then it makes sense to apply this to master and backport it to
>> branch-2.4.
>
>Personally I would like to have better
ofproto_enable_eviction() and ofproto_disable_eviction() require
ofproto_mutex (and they were even annotated that way, though not on their
prototypes but only at definition), but it wasn't being held. This fixes
the problem.
Found by inspection.
Signed-off-by: Ben Pfaff
---
ofproto/ofproto.c |
OpenFlow provides no means to change the importance of a flow after it is
added, so make it immutable so that it is reasonable to read it without
locking.
Signed-off-by: Ben Pfaff
---
ofproto/ofproto-provider.h | 2 +-
ofproto/ofproto.c | 5 ++---
2 files changed, 3 insertions(+), 4 del
Signed-off-by: Ben Pfaff
---
lib/ofp-util.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/ofp-util.c b/lib/ofp-util.c
index 89359c1..c1b2394 100644
--- a/lib/ofp-util.c
+++ b/lib/ofp-util.c
@@ -1317,7 +1317,7 @@ ofputil_decode_hello_bitmap(const struct
ofp_hello_elem_he
The "importance" field is considered before flow timeout because I figure
that if you set the importance, you think it's important.
Signed-off-by: Ben Pfaff
Co-authored-by: Saloni Jain
Signed-off-by: Saloni Jain
---
NEWS | 1 +
ofproto/ofproto.c| 40 +++
Signed-off-by: Ben Pfaff
Co-authored-by: Saloni Jain
Signed-off-by: Saloni Jain
---
NEWS| 1 +
include/openflow/openflow-1.4.h | 10 +++
lib/learning-switch.c | 2 +
lib/ofp-msgs.h | 10 +++
lib/ofp-print.c | 39 +
OpenFlow 1.4 introduces the ability to turn on flow table eviction with an
OFPT_TABLE_MOD message specifying OFPTC_EVICTION. It also adds related
machinery to other messages that mention OFPTC_* fields. This commit adds
support for the new feature, implementing it as a second, parallel way to
ena
I reworked this into a patch series, starting here:
http://openvswitch.org/pipermail/dev/2015-June/056771.html
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
On Wed, Jun 24, 2015 at 10:56:55AM +, Sorin Vinturis wrote:
> Bug found in OvsPartialCopyToMultipleNBLs function in the cleanup part of
> the code. Before completing the current NBL (newNbl) the NEXT link for the
> following NBL (firstNbl) was broken, instead of the current one (newNbl).
>
> T
I guess this solution is OK with me for 2.4. Nithin (or anyone), do you
agree that it's the best solution for now?
On Fri, Jun 19, 2015 at 04:41:47PM +, Alin Serdean wrote:
> To do it the proper way yes.
>
> I really hate to do it this way but given the timeframe for 2.4 we could nuke
> it
On Fri, Jun 19, 2015 at 04:14:44PM +, Nithin Raju wrote:
>
> > On Jun 19, 2015, at 8:59 AM, Alin Serdean
> > wrote:
> >
> > I would rather leave them the way they are until we fix the actual problem
> > (disable from userspace, treat the dpif_probe_feature correctly)
>
> Sure, let’s wait
On Fri, Jun 19, 2015 at 04:13:15PM -0700, Jesse Gross wrote:
> Until now, all fields that OVS can match against have been fixed
> size (variable length headers can be skipped during parsing but
> the match is fixed). However, Geneve options can vary in size
> so we must not require the size of thes
On Fri, Jun 19, 2015 at 04:13:16PM -0700, Jesse Gross wrote:
> Currently we treat the entire NXM/OXM header, including length,
> as an ID to define a field. However, this does not allow for
> multiple lengths of a particular field.
>
> If a field has been marked as variable, we should ignore the l
NXM/OXM headers as represented in this file are 64-bit long and the low
bits are essentially constant (almost always 0) so using hash_int(),
which takes an uint32_t, is going to be a useless hash function. This
commit fixes the problem.
Found by inspection.
CC: Jesse Gross
Signed-off-by: Ben Pf
On Wed, Jun 24, 2015 at 11:13 AM, Ben Pfaff wrote:
> On Fri, Jun 19, 2015 at 04:13:16PM -0700, Jesse Gross wrote:
>> Currently we treat the entire NXM/OXM header, including length,
>> as an ID to define a field. However, this does not allow for
>> multiple lengths of a particular field.
>>
>> If a
On Wed, Jun 24, 2015 at 11:20:20AM -0700, Jesse Gross wrote:
> On Wed, Jun 24, 2015 at 11:13 AM, Ben Pfaff wrote:
> > While reading nxm_field_by_header() I realized that we've got a terrible
> > existing mistake in our hashing. Ugh. I'll send a fix, which we'll want
> > to pre-apply (and maybe ba
On Wed, Jun 24, 2015 at 11:17 AM, Ben Pfaff wrote:
> NXM/OXM headers as represented in this file are 64-bit long and the low
> bits are essentially constant (almost always 0) so using hash_int(),
> which takes an uint32_t, is going to be a useless hash function. This
> commit fixes the problem.
>
On Wed, Jun 24, 2015 at 11:21:51AM -0700, Jesse Gross wrote:
> On Wed, Jun 24, 2015 at 11:17 AM, Ben Pfaff wrote:
> > NXM/OXM headers as represented in this file are 64-bit long and the low
> > bits are essentially constant (almost always 0) so using hash_int(),
> > which takes an uint32_t, is goi
On Fri, Jun 19, 2015 at 04:13:17PM -0700, Jesse Gross wrote:
> This adds support for receiving variable length fields encoded in
> NXM/OXM and mapping them into OVS internal structures. In order
> for this to make sense, we need to define some semantics:
>
> There are three lengths that matter in
On Fri, Jun 19, 2015 at 04:13:18PM -0700, Jesse Gross wrote:
> Currently when an NXM field is encoded, the caller must specify
> the length of the data being provided. However, this data is
> always placed into a field of standard length. In order to
> support variable length options, the length fi
Signed-off-by: Ben Pfaff
---
lib/nx-match.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/nx-match.c b/lib/nx-match.c
index 3dcee5d..e9de0fd 100644
--- a/lib/nx-match.c
+++ b/lib/nx-match.c
@@ -1150,7 +1150,7 @@ oxm_put_field_array(struct ofpbuf *b, const struct
field_a
On Wed, Jun 24, 2015 at 11:38 AM, Ben Pfaff wrote:
> On Fri, Jun 19, 2015 at 04:13:17PM -0700, Jesse Gross wrote:
>> This adds support for receiving variable length fields encoded in
>> NXM/OXM and mapping them into OVS internal structures. In order
>> for this to make sense, we need to define som
On Wed, Jun 24, 2015 at 11:46 AM, Ben Pfaff wrote:
> Signed-off-by: Ben Pfaff
> ---
> lib/nx-match.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Acked-by: Jesse Gross
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman
On Fri, Jun 19, 2015 at 04:13:19PM -0700, Jesse Gross wrote:
> It is technically correct to send the entire maximum length of
> a field when it is variable length. However, it is awkward to
> do so and not what one would naively expect. Since receivers will
> internally zero-extend fields, we can d
On Fri, Jun 19, 2015 at 04:13:20PM -0700, Jesse Gross wrote:
> When reading in hex strings that form NXM fields, we don't need to
> enforce size constraints if the fields are variable length.
> Instead, we can set the header size based on the string length.
>
> Signed-off-by: Jesse Gross
I think
On Wed, Jun 24, 2015 at 12:12 PM, Ben Pfaff wrote:
> On Fri, Jun 19, 2015 at 04:13:19PM -0700, Jesse Gross wrote:
>> It is technically correct to send the entire maximum length of
>> a field when it is variable length. However, it is awkward to
>> do so and not what one would naively expect. Since
I'm wondering if we could fail this call in the datapath level by examining the
DPIF_FP_PROBE bit in the flags.
If it is too hard we can still live with user mode code change.
Thanks,
Eitan
-Original Message-
From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Ben Pfaff
Sent: Wed
On Fri, Jun 19, 2015 at 04:13:21PM -0700, Jesse Gross wrote:
> In order to work with Geneve options, we need to maintain a mapping
> table between an option (defined by ) and
> an NXM field that can be operated on for the purposes of matches,
> actions, etc. This mapping must be explicitly specifie
On Fri, Jun 19, 2015 at 04:13:22PM -0700, Jesse Gross wrote:
> Tunnel metadata can be substantially larger than our existing fields
> (up to 124 bytes in a single Geneve option) so this extends the size
> of the data that we can handle with metaflow fields. This also
> breaks a few tests that assum
On Wed, Jun 24, 2015 at 12:26 PM, Ben Pfaff wrote:
> On Fri, Jun 19, 2015 at 04:13:20PM -0700, Jesse Gross wrote:
>> When reading in hex strings that form NXM fields, we don't need to
>> enforce size constraints if the fields are variable length.
>> Instead, we can set the header size based on the
On Fri, Jun 19, 2015 at 04:13:23PM -0700, Jesse Gross wrote:
> Sometimes we need to look at flow fields to understand how to parse
> an attribute. However, masks don't have this information - just the
> mask on the field. We already use the translated flow structure for
> this purpose but this isn'
On Mon, Jun 22, 2015 at 9:58 AM, Ben Pfaff wrote:
> On Wed, Jun 10, 2015 at 06:37:59PM -0700, Andy Zhou wrote:
>> Currently, JSON cache is always flushed whenever the monitor receives
>> a transaction. This patch improves the JSON cache efficiency by only
>> flush the JSON cache when a transaction
On Wed, Jun 24, 2015 at 1:11 PM, Ben Pfaff wrote:
> On Fri, Jun 19, 2015 at 04:13:21PM -0700, Jesse Gross wrote:
> In parse_ofp_geneve_table_mod_str(), I'd prefer to use %i instead of %x,
> because one of my long-term goals for OVS is to switch away from this
> awful scanf() stuff to a proper pars
On Fri, Jun 19, 2015 at 04:13:21PM -0700, Jesse Gross wrote:
> In order to work with Geneve options, we need to maintain a mapping
> table between an option (defined by ) and
> an NXM field that can be operated on for the purposes of matches,
> actions, etc. This mapping must be explicitly specifie
On Wed, Jun 24, 2015 at 1:17 PM, Ben Pfaff wrote:
> On Fri, Jun 19, 2015 at 04:13:22PM -0700, Jesse Gross wrote:
>> Tunnel metadata can be substantially larger than our existing fields
>> (up to 124 bytes in a single Geneve option) so this extends the size
>> of the data that we can handle with me
On Wed, Jun 24, 2015 at 1:20 PM, Ben Pfaff wrote:
> On Fri, Jun 19, 2015 at 04:13:23PM -0700, Jesse Gross wrote:
>> Sometimes we need to look at flow fields to understand how to parse
>> an attribute. However, masks don't have this information - just the
>> mask on the field. We already use the tr
On Wed, Jun 24, 2015 at 1:45 PM, Ben Pfaff wrote:
> On Fri, Jun 19, 2015 at 04:13:21PM -0700, Jesse Gross wrote:
>> In order to work with Geneve options, we need to maintain a mapping
>> table between an option (defined by ) and
>> an NXM field that can be operated on for the purposes of matches,
When doing native tunneling, we look at packets destined to the
local port to see if they match tunnel protocols that we should
intercept. The criteria are IP protocol, destination UDP port, etc.
However, we also look at the source IP address of the packets. This
should be a function of the port-b
On Wed, Jun 24, 2015 at 01:52:46PM -0700, Jesse Gross wrote:
> On Wed, Jun 24, 2015 at 1:45 PM, Ben Pfaff wrote:
> > On Fri, Jun 19, 2015 at 04:13:21PM -0700, Jesse Gross wrote:
> >> In order to work with Geneve options, we need to maintain a mapping
> >> table between an option (defined by ) and
On Fri, Jun 19, 2015 at 04:13:24PM -0700, Jesse Gross wrote:
> The current support for Geneve in OVS is exactly equivalent to VXLAN:
> it is possible to set and match on the VNI but not on any options
> contained in the header. This patch enables the use of options.
>
> The goal for Geneve support
On Fri, Jun 19, 2015 at 04:13:25PM -0700, Jesse Gross wrote:
> The addition of Geneve metadata requires a large amount of additional
> space to handle the maximum set of options. In most cases, this is
> not a big deal since it is only temporary storage on the stack or
> can be automatically stripp
On Wed, Jun 24, 2015 at 11:59:03AM -0700, Jesse Gross wrote:
> On Wed, Jun 24, 2015 at 11:46 AM, Ben Pfaff wrote:
> > Signed-off-by: Ben Pfaff
> > ---
> > lib/nx-match.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
>
> Acked-by: Jesse Gross
Thanks, applied.
__
Hi Sumit. Sorry it took me a while to review this--I'm catching up as
best I can.
Can I have a Signed-off-by for this commit? CONTRIBUTING.md says this
about Signed-off-by:
Signed-off-by: Author Name
Informally, this indicates that Author Name is the author or
submitter of
On Sat, Jun 20, 2015 at 05:09:55PM -0700, Ben Pfaff wrote:
> From: Sumit Garg
>
> In other words, when stream_or_pstream_needs_probes()
> return non-zero, the probes are gettting disabled as
> the probe interval is getting set to zero. This leads
> to incorrect behavior such that probes are:
>
>
On Mon, Jun 22, 2015 at 02:15:48AM -0700, Gurucharan Shetty wrote:
> The ability to move IP address and routes between two interfaces
> is useful when we want to make a physical interface (say eth0)
> as a port of OVS bridge (say breth0) with all its IP address and route
> information transferred t
On Mon, Jun 22, 2015 at 02:15:49AM -0700, Gurucharan Shetty wrote:
> Signed-off-by: Gurucharan Shetty
Is this intended exclusively for use with Docker, or is it meant to be
more generic? Is the documentation just the Docker installation
instructions in the following patch?
__
On Mon, Jun 22, 2015 at 02:15:50AM -0700, Gurucharan Shetty wrote:
> Docker committed experimental support for multi-host
> networking yesterday. This commit adds a driver that
> works with that experimental support. Since Docker
> code is not part of any official Docker releases yet,
> this patch
On Mon, Jun 22, 2015 at 04:44:28PM -0700, Alex Wang wrote:
> On Mon, Jun 22, 2015 at 3:58 PM, Alex Wang wrote:
>
> >
> >
> > On Mon, Jun 22, 2015 at 3:42 PM, Ben Pfaff wrote:
> >
> >> Originally only the OVN documentation used the XML format, but now it's
> >> used outside the ovn directory (ini
On Wed, Jun 24, 2015 at 07:49:08AM -0700, Gurucharan Shetty wrote:
> You can look at "commit 5e65e080ad4d57e" for more information on the
> exact problem with taskkill.
>
> One option is to check for tskill and if it does not exist, use taskkill.
Here's a concept for that, it's probably buggy bec
On Wed, Jun 24, 2015 at 07:43:03AM -0700, Gurucharan Shetty wrote:
> On Tue, Jun 23, 2015 at 6:11 PM, Alin Serdean
> wrote:
> > How about the following:
> >
> > Before running the unit tests make sure to add the pthread libraries to your
> > PATH environment variable.
> The above is already part o
On Tue, Jun 23, 2015 at 08:11:22AM -0700, Sabyasachi Sengupta wrote:
> Extend bfd to save wall clock time of the last flap in bfd_forwarding__,
> and display it throught bfd/show. This information is also exported out
> to ovsdb in bfd_get_status.
>
> Signed-off-by: Sabyasachi Sengupta
This seem
On Tue, Jun 23, 2015 at 02:22:08PM -0400, Russell Bryant wrote:
> This patch adds a new column to the Logical_Port table of the
> OVN_Northbound database called 'enabled'. The purpose is to allow a
> port to be administratively enabled or disabled. It is sometimes
> useful to keep a port and its
On Tue, Jun 23, 2015 at 03:18:57PM -0400, Russell Bryant wrote:
> On 06/23/2015 02:22 PM, Russell Bryant wrote:
> > While working on OpenStack Neutron integration for OVN, I came across a
> > small
> > feature gap. The Neutron API supports setting a port as administratively
> > down.
> > One way
On Tue, Jun 23, 2015 at 02:22:09PM -0400, Russell Bryant wrote:
> This patch adds support for getting and setting the 'enabled' column
> for logical ports using ovn-nbctl.
>
> Signed-off-by: Russell Bryant
Applied, thanks!
___
dev mailing list
dev@open
On Tue, Jun 23, 2015 at 05:03:14PM -0300, Thadeu Lima de Souza Cascardo wrote:
> Identation was one extra level at ofproto_unixctl_mcast_snooping_show.
>
> Signed-off-by: Thadeu Lima de Souza Cascardo
Applied, thanks!
___
dev mailing list
dev@openvswit
On Tue, Jun 23, 2015 at 05:03:15PM -0300, Thadeu Lima de Souza Cascardo wrote:
> Use IPv6 internally for storing multicast addresses. IPv4 addresses are
> translated to their IPv4-mapped equivalent.
>
> Signed-off-by: Thadeu Lima de Souza Cascardo
From Clang:
../lib/mcast-snooping.c:93:23:
On Tue, Jun 23, 2015 at 05:03:16PM -0300, Thadeu Lima de Souza Cascardo wrote:
> Add support for MLDv1 and MLDv2. The behavior is not that different from
> IGMP. Packets to all-hosts address and queries are always flooded,
> reports go to routers, routers are added when a query is observed, and
> a
On Tue, Jun 23, 2015 at 08:16:41PM +, Nithin Raju wrote:
> > On Jun 23, 2015, at 12:42 PM, Ben Pfaff wrote:
> >
> > On Thu, Jun 18, 2015 at 09:05:32PM +, Nithin Raju wrote:
> >>> On Jun 18, 2015, at 1:57 PM, Alin Serdean
> >>> wrote:
> >>>
> >>> I have no issue with it if it is tested.
>
> > Acked-by: Alex Wang
>
> Thanks.
>
> I doubt this patch was related to the problem, because it only moves
> text around without changing it.
>
Yeah, it is unrelated,
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/d
Hi Ben,
It's a little unconventional for us to use a wall clock time for this.
I'd be more inclined to report it as "N seconds ago" or "N ms ago". Any
particular reason to use a wall clock time?
I've seen that all BFD other outputs use "now -/+" convention, but just that
I thought wall cloc
+1.
We still need to add the description for the SYSTEM one or put in another patch
using the setx command (https://support.microsoft.com/en-us/kb/195050).
Alin.
-Mesaj original-
De la: Ben Pfaff [mailto:b...@nicira.com]
Trimis: Thursday, June 25, 2015 3:22 AM
Către: Gurucharan Shetty
On Wed, Jun 24, 2015 at 3:57 PM, Ben Pfaff wrote:
> On Fri, Jun 19, 2015 at 04:13:24PM -0700, Jesse Gross wrote:
>> The current support for Geneve in OVS is exactly equivalent to VXLAN:
>> it is possible to set and match on the VNI but not on any options
>> contained in the header. This patch enab
On Wed, Jun 24, 2015 at 06:34:05PM -0700, Jesse Gross wrote:
> On Wed, Jun 24, 2015 at 3:57 PM, Ben Pfaff wrote:
> > On Fri, Jun 19, 2015 at 04:13:24PM -0700, Jesse Gross wrote:
> > I like the implementation approach used in ULONG_FOR_EACH_1 better than
> > the one in PRESENT_OPT_FOR_EACH, since t
This reduces the user burden for running "make check".
Signed-off-by: Ben Pfaff
---
I can't test this; whoever reviews it ought to.
m4/openvswitch.m4 | 6 --
tests/automake.mk | 2 +-
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/m4/openvswitch.m4 b/m4/openvswitch.m4
index
It's a step forward, then. I sent out an official patch, I'd appreciate
it if anyone out there is willing to test it:
http://openvswitch.org/pipermail/dev/2015-June/056833.html
On Thu, Jun 25, 2015 at 01:19:14AM +, Alin Serdean wrote:
> +1.
>
> We still need to add the description fo
On Wed, Jun 24, 2015 at 2:55 PM, Jesse Gross wrote:
> When doing native tunneling, we look at packets destined to the
> local port to see if they match tunnel protocols that we should
> intercept. The criteria are IP protocol, destination UDP port, etc.
>
> However, we also look at the source IP a
Hi Ben,
On Tue, Jun 23, 2015 at 11:38:56AM -0700, Ben Pfaff wrote:
> Groups were not destroyed until after lots of other important bridge
> data had been destroyed, including the connection manager. There was an
> indirect dependency on the connection manager for bridge destruction
> because dest
On Tue, Jun 23, 2015 at 11:39:45AM -0700, Ben Pfaff wrote:
> On Wed, Jun 17, 2015 at 02:22:32PM +0900, Simon Horman wrote:
> > As per the testcase included in this patch it has been observed
> > that ovs-vswtichd may segfault when deleting a bridge.
> >
> > Analysing the output of valgrind and gdb
82 matches
Mail list logo
