The message was undeliverable due to the following reason(s):
Your message was not delivered because the destination server was
unreachable within the allowed queue period. The amount of time
a message is queued before it is returned depends on local configura-
tion parameters.
Most likely there
The original message was included as attachment
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
Account for kernels which provide udp_sock_create() in an
insufficient version.
Avoids the following error when inserting openvswitch.ko on
respective kernels:
openvswitch: exports duplicate symbol udp_sock_create (owned by udp_tunnel)
Fixes: eb6eebd28 ("datapath: Account for "udp: Add udp_sock_
On 03/25/15 at 02:48pm, Pravin Shelar wrote:
> Thanks for the patch. I saw couple of issues:-
> I saw following warning due to missing symbol.
> WARNING: "lockdep_ovsl_is_held"
> [/home/pravin/ovs/w8/datapath/linux/vport-gre.ko] undefined!
Thanks for catching! Fixed.
> Command to reload kernel da
Return module reference before invoking the respective vport
->destroy() function. This is needed as ovs_vport_del() is not
invoked inside an RCU read side critical section so the kfree
can occur immediately before returning to ovs_vport_del().
Returning the module reference before ->destroy() is
Bonjour Mlle, Mme et Mr.
Vous aviez besoin d’un financement, d’un prêt immobilière, vous êtes interdit
bancaire, fiché, vous aviez quelques soucis financières pour monter vos
projets? Je suis un particulier financier octroyant des prêts allant de 3.000 à
600.000 € à des conditions abordable,si
Bonjour Mlle, Mme et Mr.
Vous aviez besoin d’un financement, d’un prêt immobilière, vous êtes interdit
bancaire, fiché, vous aviez quelques soucis financières pour monter vos
projets? Je suis un particulier financier octroyant des prêts allant de 3.000 à
600.000 € à des conditions abordable,si
We manually rmmod the loaded vports as using modprobe -r
only works if the modules are available through modules.dep
Upon restart, the vports get automatically reloaded when the
respective vports are added.
Reported-by: Pravin Shelar
Signed-off-by: Thomas Graf
---
utilities/ovs-ctl.in | 4
This patch changes the behaviour in case the configure argument: --with-debug
was specified.
Currently the optimization flag in the case of debugging is the following:
https://msdn.microsoft.com/en-us/library/f9534wye.aspx
which does not fully disable optimization, that is why it was changed with
This patch changes the behaviour in case the configure argument: --with-debug
was specified.
Currently the optimization flag in the case of debugging is the following:
https://msdn.microsoft.com/en-us/library/f9534wye.aspx
which does not fully disable optimization, that is why it was changed with
On Mon, Mar 30, 2015 at 3:27 AM, Thomas Graf wrote:
> Account for kernels which provide udp_sock_create() in an
> insufficient version.
>
> Avoids the following error when inserting openvswitch.ko on
> respective kernels:
>
> openvswitch: exports duplicate symbol udp_sock_create (owned by udp_tunn
On Sun, Mar 29, 2015 at 9:27 PM, yinpeijun wrote:
> From: caochengrong
>
> tcp flows with gso between two VMs in diffrent host,
> go through vxlan tunnel, cause kernel crash.
>
> Signed-off-by: caochengrong
> Signed-off-by: Arika Chen
What OVS and host kernel version is this for? I don't think
On Thu, Mar 19, 2015 at 07:31:06AM -0700, Gurucharan Shetty wrote:
> The design was come up after inputs and discussions with multiple
> people, including (in alphabetical order) Aaron Rosen, Ben Pfaff,
> Ganesan Chandrashekhar, Justin Pettit, Russell Bryant and Somik Behera.
>
> Signed-off-by: Gu
On Mon, Mar 30, 2015 at 4:57 AM, Thomas Graf wrote:
> Return module reference before invoking the respective vport
> ->destroy() function. This is needed as ovs_vport_del() is not
> invoked inside an RCU read side critical section so the kfree
> can occur immediately before returning to ovs_vport_
On 03/30/2015 01:45 PM, Ben Pfaff wrote:
> On Thu, Mar 19, 2015 at 07:31:06AM -0700, Gurucharan Shetty wrote:
>> The design was come up after inputs and discussions with multiple
>> people, including (in alphabetical order) Aaron Rosen, Ben Pfaff,
>> Ganesan Chandrashekhar, Justin Pettit, Russell B
>> Other than the one suggestion I sent in a separate email,
>> Acked-by: Ben Pfaff
>
> FWIW, same here. I like it. Thanks!
>
> Acked-by: Russell Bryant
>
>> Thank you so much for being proactive about figuring out how containers
>> fit into the system! It ought to save us a lot of trouble
On Mon, Mar 30, 2015 at 11:31 AM, Gurucharan Shetty wrote:
>>> Other than the one suggestion I sent in a separate email,
>>> Acked-by: Ben Pfaff
>>
>> FWIW, same here. I like it. Thanks!
>>
>> Acked-by: Russell Bryant
>>
>>> Thank you so much for being proactive about figuring out how cont
On 03/30/2015 02:39 PM, Gurucharan Shetty wrote:
> On Mon, Mar 30, 2015 at 11:31 AM, Gurucharan Shetty
> wrote:
Other than the one suggestion I sent in a separate email,
Acked-by: Ben Pfaff
>>>
>>> FWIW, same here. I like it. Thanks!
>>>
>>> Acked-by: Russell Bryant
>>>
Tha
Acked-by: Ethan Jackson
Merged, thanks.
On Fri, Mar 27, 2015 at 9:29 AM, Daniele Di Proietto
wrote:
> Packets for which an upcall has failed (lost packets) must be deleted.
> We also need to count them as MISS and LOST.
>
> Signed-off-by: Daniele Di Proietto
> ---
> lib/dpif-netdev.c | 3 +++
> Since statistics updates might require locking (in future commits)
> grouping them will reduce the locking overhead.
Do they actually require locking in future commits or is this speculative?
Adding locks for these stats seems like a pretty significant step
backwards, have you measured the over
You're not going to like my review of this patch =)
So the code is very high quality, though I haven't read it super
closely yet. My main question, is if this is really the right
approach to handle stats in the DPDK fast path in general.
So here's my question: why not just use atomic variables f
The indication to calculate the GRE checksum is currently the port
config rather than the tunnel flow. Currently there is a one to one
mapping between the two so there is no difference. However, the
kernel datapath must use the flow and it is also potentially more
flexible, so this switches how we
The IP TTL is currently omitted in the extracted tunnel information
that is stored in the flow for userspace tunneling. This includes it
so that the same logic used by the kernel also applies.
Signed-off-by: Jesse Gross
---
lib/netdev-vport.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/l
The GRE checksum is a 16 bit field stored in a 32 bit option (the
rest is reserved). The current code treats the checksum as a 32-bit
field and places it in the right place for little endian systems but
not big endian. This fixes the problem by storing the 16 bit field
directly.
Signed-off-by: Jes
As OVS adds userspace support for being the endpoint in protocols
like tunnels, it will need to be able to calculate pseudoheaders
as part of the checksum calculation.
Signed-off-by: Jesse Gross
---
lib/packets.c | 14 ++
lib/packets.h | 1 +
tests/library.at | 2 +-
tests
Currently, the userspace VXLAN implementation contains the code
for generating and parsing both the UDP and VXLAN headers. This
pulls out the UDP portion for better layering and to make it
easier to support additional UDP based tunnels and features.
Signed-off-by: Jesse Gross
---
lib/netdev-vpor
This adds basic userspace dataplane support for the Geneve
tunneling protocol. The rest of userspace only has the ability
to handle Geneve without options and this follows that pattern
for the time being. However, when the rest of userspace is updated
it should be easy to extend the dataplane as we
Kernel based OVS recently added the ability to support checksums
for UDP based tunnels (Geneve and VXLAN). This adds similar support
for the userspace datapath to bring feature parity.
Signed-off-by: Jesse Gross
---
lib/netdev-vport.c | 37 ++---
lib/odp-uti
On receive, the userspace GRE code doesn't check the protocol
field. Since OVS only understands Ethernet packets, this adds a
check that the inner protocol is Ethernet and discards other types
of packets.
Signed-off-by: Jesse Gross
---
lib/netdev-vport.c | 4
1 file changed, 4 insertions(+)
Currently when printing a userspace tunnel action for VXLAN, the
VNI is treated as a 32 bit field rather than 24 bit. Even if this
is the representation that we use internally, we should still show
the right VNI to avoid confusing people.
Signed-off-by: Jesse Gross
---
lib/odp-util.c |
Acked-by: Pritesh Kothari
> On Mar 30, 2015, at 3:14 PM, Jesse Gross wrote:
>
> The IP TTL is currently omitted in the extracted tunnel information
> that is stored in the flow for userspace tunneling. This includes it
> so that the same logic used by the kernel also applies.
>
> Signed-off-by
should a test case be added to catch this ?
Regards,
Pritesh
> On Mar 30, 2015, at 3:14 PM, Jesse Gross wrote:
>
> +if (greh->protocol != htons(ETH_TYPE_TEB)) {
> +return -EINVAL;
> +}
> +
___
dev mailing list
dev@openvswitch.org
http
This series implements a prototype of using Header Space Analysis (HSA)
for OVS OpenFlow table analysis. The implementation allows users to find
all possible output ports with the input header format reachable from a
specified input port. It also allows users to check if there is any loop
formed
This commit adds function that allows the appending of one list
content to the other. Also, it adds functions which allows list
to be sorted.
Signed-off-by: Alex Wang
---
lib/list.h| 67 +
tests/library.at |2 +-
tests/test-list.c | 106 ++
For conducting Header Space Analysis (HSA), we convert the wildcarded
OpenFlow flow represented by 'struct match' into an encoded byte array.
To further save memory, we use a sparse array to represent such byte
array in the same way as 'struct miniflow'. So, this commit implements
the structs and
This commit adds functions that check if the 'struct flow_wildcards'
member is fully masked (exact-match) or fully unmasked (don't care
the entire field at all). Also, this commit adds a function to
shape a less-masked flow field by a more-masked flow field.
Signed-off-by: Alex Wang
---
lib/flo
This commit implements a prototype of Header Space Analysis of the
OVS OpenFlow table.
What It Does
1. Dump all OpenFlow rules from a specified bridge.
2. Generate all-unmasked header space with specified input port number
and start looking up from default starting table.
3.
LGTM
Acked-by: Pritesh Kothari
> On Mar 30, 2015, at 3:14 PM, Jesse Gross wrote:
>
> The GRE checksum is a 16 bit field stored in a 32 bit option (the
> rest is reserved). The current code treats the checksum as a 32-bit
> field and places it in the right place for little endian systems but
>
LGTM
Acked-by: Pritesh Kothari
will get to rest of series as time permits.
> On Mar 30, 2015, at 3:14 PM, Jesse Gross wrote:
>
> The indication to calculate the GRE checksum is currently the port
> config rather than the tunnel flow. Currently there is a one to one
> mapping between the two s
On 03/30/15 at 09:29am, Jesse Gross wrote:
> On Mon, Mar 30, 2015 at 3:27 AM, Thomas Graf wrote:
> > Account for kernels which provide udp_sock_create() in an
> > insufficient version.
> >
> > Avoids the following error when inserting openvswitch.ko on
> > respective kernels:
> >
> > openvswitch:
This patch fixes handling of logical switch and logical port names
that look like a UUID. Previously, if the argument looked like a
UUID, it would only do a UUID lookup for a match. After this change,
if the UUID lookup fails, it will treat it as a name and look it up
that way.
Signed-off-by: Ru
Hi,
Can anyone please let me know if Centos 7.1 (3.10.0-229) kernel supported in
master branch? I'm currently based off branch-2.3. If this is already
available, what would be the commit hash(es) to be picked up?
Thanks,
Sabya
___
dev mailing list
>> tcp flows with gso between two VMs in diffrent host, go through vxlan
>> tunnel, cause kernel crash.
>>
>> Signed-off-by: caochengrong
>> Signed-off-by: Arika Chen
>
>What OVS and host kernel version is this for? I don't think this should be
>necessary, at least on master. vxlan_xmit_skb() c
The syntax of matching expressions supported here is documented in ovn(5),
which I've made available (along with the rest of the OVS docs for this
branch) at http://benpfaff.org/~blp/dist-docs, e.g. specifically
http://benpfaff.org/~blp/dist-docs/ovn.5.pdf
Ben Pfaff (5):
meta-flow: Add convenien
This makes access to the least-significant bits more convenient. This
commit simplifies a few existing cases; later commits will make more use
of this feature.
Signed-off-by: Ben Pfaff
---
lib/learn.c | 6 ++
lib/meta-flow.h | 16
2 files changed, 18 insertions(+), 4 d
To be used in upcoming commits.
Signed-off-by: Ben Pfaff
---
lib/util.c | 139 +
lib/util.h | 8 +++-
2 files changed, 139 insertions(+), 8 deletions(-)
diff --git a/lib/util.c b/lib/util.c
index bcf7700..3293ca4 100644
--- a/lib/uti
I'm determined not to let the terrible style of pseudo-parsing we have in
OVS leak into OVN. Here's the first step.
Signed-off-by: Ben Pfaff
---
ovn/TODO | 5 -
ovn/automake.mk| 3 +
ovn/lex.c | 697 +
ovn/lex.h
To be first used in upcoming commits.
Signed-off-by: Ben Pfaff
---
lib/meta-flow.c | 81 -
lib/meta-flow.h | 13 +
2 files changed, 93 insertions(+), 1 deletion(-)
diff --git a/lib/meta-flow.c b/lib/meta-flow.c
index 8058e07..f8103
On Mon, Mar 30, 2015 at 10:07:04PM -0400, Russell Bryant wrote:
> This patch fixes handling of logical switch and logical port names
> that look like a UUID. Previously, if the argument looked like a
> UUID, it would only do a UUID lookup for a match. After this change,
> if the UUID lookup fails
On Mon, Mar 30, 2015 at 03:46:26PM -0700, Alex Wang wrote:
> This commit adds function that allows the appending of one list
> content to the other. Also, it adds functions which allows list
> to be sorted.
>
> Signed-off-by: Alex Wang
I always like new basic algorithms!
> +/* Appends 'src''s
It's probably a good idea although it requires a different type of
test compared with what we already have since the GRE protocol doesn't
appear in the flow. Here's what I came up with:
diff --git a/tests/tunnel-push-pop.at b/tests/tunnel-push-pop.at
index 6e1c0c1..ee17a2f 100644
--- a/tests/tunne
I am trying to edit the Ip source and destination address in
void ovs_dp_process_received_packet(struct vport *p, struct sk_buff *skb)
in datapath.c file. I am able to edit mac headers successfully using
memcpy(eth_hdr(skb), str, 2)
// str contains my target data
I tried a similar approach wit
Thx a lot for the suggested changes~, I really did it the hard way!
The changes are simple and clean! really like it,
Adopt them all,
Thanks,
Alex Wang,
On Mon, Mar 30, 2015 at 9:19 PM, Ben Pfaff wrote:
> On Mon, Mar 30, 2015 at 03:46:26PM -0700, Alex Wang wrote:
> > This commit adds functio
53 matches
Mail list logo
