Instance PID is already deleted in the OvsCleanupPacketQueue function.
Signed-off-by: Sorin Vinturis
Acked-by: Nithin Raju
---
v2: Rebased the patch.
v3: Added ack.
---
datapath-windows/ovsext/User.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/datapath-windows/ovsext/User.c b/data
Hello.
I need to do some slight modification to the code and being able to register
when a packet_in message is sent to controller due to a "no match".
What files should I look for this?
Thank you!
Miguel Antonio
___
On Fri, Apr 24, 2015 at 03:34:50PM -0700, Ben Pfaff wrote:
> v1->v2: Add patch 5.
> v2->v3: Break patch 5 into a series of 6 patches, each of which actually
> makes sense and is ready for review.
I've applied patches 1 through 6 to the ovn branch. I'll repost the
rest of the patches, by request
On Tue, Apr 28, 2015 at 05:36:50PM -0700, Justin Pettit wrote:
>
> > On Apr 28, 2015, at 5:21 PM, Ben Pfaff wrote:
> >
> > On Tue, Apr 28, 2015 at 03:53:03PM -0400, Russell Bryant wrote:
> >>
> >> The code here looks correct and I also tested it. I was just wondering
> >> if you could comment
On Tue, Apr 28, 2015 at 05:20:13PM -0700, Justin Pettit wrote:
>
> > On Apr 24, 2015, at 3:34 PM, Ben Pfaff wrote:
> >
> > When packets travel among nodes in OVN over tunnels, a tunnel key value is
> > needed to convey the logical port to which the packet is destined. This
> > commit adds a tun
> On Apr 28, 2015, at 5:21 PM, Ben Pfaff wrote:
>
> On Tue, Apr 28, 2015 at 03:53:03PM -0400, Russell Bryant wrote:
>>
>> The code here looks correct and I also tested it. I was just wondering
>> if you could comment on the choice of a 16 bit integer here instead of a
>> UUID. My guess is tha
On Tue, Apr 28, 2015 at 03:53:03PM -0400, Russell Bryant wrote:
> On 04/24/2015 06:34 PM, Ben Pfaff wrote:
> > When packets travel among nodes in OVN over tunnels, a tunnel key value is
> > needed to convey the logical port to which the packet is destined. This
> > commit adds a tunnel_key column
> On Apr 24, 2015, at 3:34 PM, Ben Pfaff wrote:
>
> When packets travel among nodes in OVN over tunnels, a tunnel key value is
> needed to convey the logical port to which the packet is destined. This
> commit adds a tunnel_key column to the Bindings table and adds code to
> ovn-northd to assig
On Tue, Apr 28, 2015 at 03:34:31PM -0400, Russell Bryant wrote:
> On 04/24/2015 06:34 PM, Ben Pfaff wrote:
> > These functions will have their first users in upcoming commits.
> >
> > Signed-off-by: Ben Pfaff
> > ---
> > ovn/lib/lex.c | 28
> > ovn/lib/lex.h |2
On Tue, Apr 28, 2015 at 02:57:25PM -0400, Russell Bryant wrote:
> On 04/24/2015 06:34 PM, Ben Pfaff wrote:
> > This is what I'm in the midst of implementing, and what ovn-northd uses.
> >
> > Reported-by: Justin Pettit
> > Signed-off-by: Ben Pfaff
> > ---
> > ovn/northd/ovn-northd.c | 11 ++--
Learn how to treat obesity at initiatory stage effectively! Get rid of ugly fat
forever!
I started last sunday and I am aready down two sizes.
Everything for your health needs In our store!
http://zzb.bz/7dPkm
___
dev mailing list
dev@openvswitch.org
h
On Tue, Apr 28, 2015 at 04:48:43PM -0700, Justin Pettit wrote:
>
> > On Apr 28, 2015, at 4:48 PM, Ben Pfaff wrote:
> >
> > On Tue, Apr 28, 2015 at 02:40:45PM -0700, Justin Pettit wrote:
> >>
> >>> On Apr 24, 2015, at 3:34 PM, Ben Pfaff wrote:
> >>>
> >>>resubmit;
> >>> -execut
Think twice before swallowing another hamburger. Declare war on excess kilos!
Get slim by summer starting TODAY with this
Free 4-12 items for you with Any Purchase
http://zzb.bz/B9clc
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailm
> On Apr 28, 2015, at 4:48 PM, Ben Pfaff wrote:
>
> On Tue, Apr 28, 2015 at 02:40:45PM -0700, Justin Pettit wrote:
>>
>>> On Apr 24, 2015, at 3:34 PM, Ben Pfaff wrote:
>>>
>>>resubmit;
>>> -execute next logical datapath table as subroutine
>>> +
>>> + Executes the
On Tue, Apr 28, 2015 at 02:40:45PM -0700, Justin Pettit wrote:
>
> > On Apr 24, 2015, at 3:34 PM, Ben Pfaff wrote:
> >
> > resubmit;
> > -execute next logical datapath table as subroutine
> > +
> > + Executes the next logical datapath table as a subroutine.
> > +
>
On Mon, Apr 27, 2015 at 2:04 PM, Pravin B Shelar wrote:
> diff --git a/datapath/linux/compat/include/net/stt.h
> b/datapath/linux/compat/include/net/stt.h
> new file mode 100644
> index 000..96181fd
> --- /dev/null
> +++ b/datapath/linux/compat/include/net/stt.h
> +#define stt_sock_add rpl_st
On Mon, Apr 27, 2015 at 10:14:51PM -0700, Justin Pettit wrote:
> This creates a tunnel to each Chassis's specified Encaps. In the
> future, we may want to limit this to only Chassis that share a logical
> datapath on the local system.
>
> Signed-off-by: Justin Pettit
I had a little bit of a men
Change the documentation for the possible vaues of WDK.
Signed-off-by: Alin Gabriel Serdean
---
INSTALL.Windows.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/INSTALL.Windows.md b/INSTALL.Windows.md
index 78af0a1..2402b5c 100644
--- a/INSTALL.Windows.md
+++ b/INSTALL.Wind
Add testsuite.tmp.orig to .gitignore
Signed-off-by: Alin Gabriel Serdean
---
v3: Respin
v2: Respin
.gitignore | 1 +
1 file changed, 1 insertion(+)
diff --git a/.gitignore b/.gitignore
index e37a690..8f0d41f 100644
--- a/.gitignore
+++ b/.gitignore
@@ -66,3 +66,4 @@ _debian
odp-netlink.h
OvsD
> On Apr 24, 2015, at 3:34 PM, Ben Pfaff wrote:
>
> Without a logical_datapath, ovn-controller won't be able to tell which
> logical datapath a logical port belongs to, thus it won't be able to
> implement logical flows related to the logical port.
>
> Signed-off-by: Ben Pfaff
Acked-by: Justi
This commit is a POC on how to build the installer from the command line.
One could issue the following commands to build the binaries, driver and
installer.
./boot.sh;./configure CC=./build-aux/cccl LD="`which link`" LIBS="-lws2_32" \
--prefix="C:/openvswitch/usr" --localstatedir="C:/openvswit
Change the documentation for the possible vaues of WDK.
Signed-off-by: Alin Gabriel Serdean
---
INSTALL.Windows.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/INSTALL.Windows.md b/INSTALL.Windows.md
index 78af0a1..2402b5c 100644
--- a/INSTALL.Windows.md
+++ b/INSTALL.Wind
Add testsuite.tmp.orig to .gitignore
Signed-off-by: Alin Gabriel Serdean
---
v3: Respin
v2: Respin
.gitignore | 1 +
1 file changed, 1 insertion(+)
diff --git a/.gitignore b/.gitignore
index e37a690..8f0d41f 100644
--- a/.gitignore
+++ b/.gitignore
@@ -66,3 +66,4 @@ _debian
odp-netlink.h
OvsD
This commit is a POC on how to build the installer from the command line.
One could issue the following commands to build the binaries, driver and
installer.
./boot.sh;./configure CC=./build-aux/cccl LD="`which link`" LIBS="-lws2_32" \
--prefix="C:/openvswitch/usr" --localstatedir="C:/openvswit
From: Edwin Chiu
In XenServer 6.5, multiple kernel packages with different
rpm versions can have the same uname. So, it is not
necessary for openvswitch kernel module to require the
exact rpm version. Instead, the kernel module package
should check the uname version.
This commit will add a new
Signed-off-by: Justin Pettit
---
ovn/ovn-sb.xml |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/ovn/ovn-sb.xml b/ovn/ovn-sb.xml
index be876b8..19ed455 100644
--- a/ovn/ovn-sb.xml
+++ b/ovn/ovn-sb.xml
@@ -228,7 +228,7 @@
- The logical datapath to which
Change the documentation for the possible vaues of WDK.
Signed-off-by: Alin Gabriel Serdean
---
INSTALL.Windows.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/INSTALL.Windows.md b/INSTALL.Windows.md
index 78af0a1..2402b5c 100644
--- a/INSTALL.Windows.md
+++ b/INSTALL.Wind
Add testsuite.tmp.orig to .gitignore
Signed-off-by: Alin Gabriel Serdean
---
v3: Respin
v2: Respin
.gitignore | 1 +
1 file changed, 1 insertion(+)
diff --git a/.gitignore b/.gitignore
index e37a690..8f0d41f 100644
--- a/.gitignore
+++ b/.gitignore
@@ -66,3 +66,4 @@ _debian
odp-netlink.h
OvsD
This commit is a POC on how to build the installer from the command line.
One could issue the following commands to build the binaries, driver and
installer.
./boot.sh;./configure CC=./build-aux/cccl LD="`which link`" LIBS="-lws2_32" \
--prefix="C:/openvswitch/usr" --localstatedir="C:/openvswit
> On Apr 24, 2015, at 3:34 PM, Ben Pfaff wrote:
>
> These functions will have their first users in upcoming commits.
>
> Signed-off-by: Ben Pfaff
> ---
> ovn/lib/lex.c | 28
> ovn/lib/lex.h |2 ++
> 2 files changed, 30 insertions(+)
Acked-by: Justin Pettit
-
> On Apr 24, 2015, at 3:34 PM, Ben Pfaff wrote:
>
> Reported-by: Justin Pettit
> Signed-off-by: Ben Pfaff
> ---
> ovn/ovn-nb.xml | 18 +++---
> 1 file changed, 15 insertions(+), 3 deletions(-)
Acked-by: Justin Pettit
--Justin
___
de
On Mon, Apr 27, 2015 at 10:14:50PM -0700, Justin Pettit wrote:
> Limit the supported tunnel types to Geneve and STT on hypervisors, and
> VXLAN, Geneve, and STT on gateways. This commit adds "DESIGN.md" that
> describes the reasoning.
>
> Signed-off-by: Justin Pettit
I like the content, but did
Acked-by: Justin Pettit
--Justin
> On Apr 24, 2015, at 3:34 PM, Ben Pfaff wrote:
>
> The implementation of port_security was partial, and the specification was
> vague. This reduces both to what was actually meant to be supported at
> this point. It shouldn't be hard to fully implement port
On Mon, Apr 27, 2015 at 10:14:49PM -0700, Justin Pettit wrote:
> This makes it more consistent with other IDs in the system.
>
> Signed-off-by: Justin Pettit
No objection.
Acked-by: Ben Pfaff
___
dev mailing list
dev@openvswitch.org
http://openvswitc
Acked-by: Justin Pettit
--Justin
> On Apr 24, 2015, at 3:34 PM, Ben Pfaff wrote:
>
> Otherwise ARP and other broadcast protocols would be problematic.
>
> Reported-by: Justin Pettit
> Signed-off-by: Ben Pfaff
> ---
> ovn/northd/ovn-northd.c |3 +++
> ovn/ovn-nb.xml |3 ++-
>
On Mon, Apr 27, 2015 at 10:14:48PM -0700, Justin Pettit wrote:
> Tunnels create a lot of interfaces without an "external-id:iface-id", so
> logging each interface missing one (even at debug) seems unnecessary.
>
> Signed-off-by: Justin Pettit
Is it worthwhile to retain the log message for interf
On Mon, Apr 27, 2015 at 10:14:47PM -0700, Justin Pettit wrote:
> Signed-off-by: Justin Pettit
Wouldn't it be better to support this? I agree with documenting current
behavior but I'd prefer to fix it in the long term (i.e. leave in the
XXX comment).
__
On Mon, Apr 27, 2015 at 10:14:46PM -0700, Justin Pettit wrote:
> The integration bridge will be needed by other components soon, so make
> it part of the general ovn-controller context.
>
> Signed-off-by: Justin Pettit
This solves a problem I had too, and I like your solution better.
Thanks.
It
> On Apr 24, 2015, at 3:34 PM, Ben Pfaff wrote:
>
> resubmit;
> -execute next logical datapath table as subroutine
> +
> + Executes the next logical datapath table as a subroutine.
> +
Should we indicate that it's the next higher version table number? Resubmi
In this patch, we make changes to usersapce as well as
kernel datapath on hyperv to make it more netlink socket
like. Previously, the kernel datapath did not distinguish
between "transport errors" and other errors. Netlink
semantics dictate that netlink functions should only
return an error only in
On Thu, Apr 23, 2015 at 1:37 PM, Sorin Vinturis
wrote:
> Instance PID is already deleted in the OvsCleanupPacketQueue function.
>
> v2: Rebased the patch.
Please put it below '---'
>
> Signed-off-by: Sorin Vinturis
Applied, thanks!
> ---
> datapath-windows/ovsext/User.c | 6 --
> 1 file chan
On 04/24/2015 06:34 PM, Ben Pfaff wrote:
> Without a logical_datapath, ovn-controller won't be able to tell which
> logical datapath a logical port belongs to, thus it won't be able to
> implement logical flows related to the logical port.
>
> Signed-off-by: Ben Pfaff
The code looks good and I v
> On Apr 23, 2015, at 1:37 PM, Sorin Vinturis
> wrote:
>
> Instance PID is already deleted in the OvsCleanupPacketQueue function.
>
> v2: Rebased the patch.
>
> Signed-off-by: Sorin Vinturis
> ---
> datapath-windows/ovsext/User.c | 6 --
> 1 file changed, 6 deletions(-)
>
> diff --git a/d
On 04/28/2015 01:14 AM, Justin Pettit wrote:
> Tunnels create a lot of interfaces without an "external-id:iface-id", so
> logging each interface missing one (even at debug) seems unnecessary.
>
> Signed-off-by: Justin Pettit
> ---
> ovn/controller/bindings.c |1 -
> 1 files changed, 0 insert
On 04/28/2015 01:14 AM, Justin Pettit wrote:
> Signed-off-by: Justin Pettit
> ---
> ovn/controller/ovn-controller.8.xml |5 +
> ovn/controller/ovn-controller.c |6 --
> 2 files changed, 9 insertions(+), 2 deletions(-)
Acked-by: Russell Bryant
--
Russell Bryant
On 04/28/2015 01:14 AM, Justin Pettit wrote:
> The integration bridge will be needed by other components soon, so make
> it part of the general ovn-controller context.
>
> Signed-off-by: Justin Pettit
> ---
> ovn/controller/bindings.c | 33 ++--
> ovn/controller/o
On 04/24/2015 06:34 PM, Ben Pfaff wrote:
> When packets travel among nodes in OVN over tunnels, a tunnel key value is
> needed to convey the logical port to which the packet is destined. This
> commit adds a tunnel_key column to the Bindings table and adds code to
> ovn-northd to assign a unique t
On 04/24/2015 06:34 PM, Ben Pfaff wrote:
> These functions will have their first users in upcoming commits.
>
> Signed-off-by: Ben Pfaff
> ---
> ovn/lib/lex.c | 28
> ovn/lib/lex.h |2 ++
> 2 files changed, 30 insertions(+)
>
> diff --git a/ovn/lib/lex.h b/ov
On 04/24/2015 06:34 PM, Ben Pfaff wrote:
> Reported-by: Justin Pettit
> Signed-off-by: Ben Pfaff
> ---
> ovn/ovn-nb.xml | 18 +++---
> 1 file changed, 15 insertions(+), 3 deletions(-)
Acked-by: Russell Bryant
--
Russell Bryant
___
dev
On 04/24/2015 06:34 PM, Ben Pfaff wrote:
> The implementation of port_security was partial, and the specification was
> vague. This reduces both to what was actually meant to be supported at
> this point. It shouldn't be hard to fully implement port security when we
> have a clear specification.
On 04/24/2015 06:34 PM, Ben Pfaff wrote:
> Otherwise ARP and other broadcast protocols would be problematic.
>
> Reported-by: Justin Pettit
> Signed-off-by: Ben Pfaff
I tested from your ovs-reviews ovn branch and the additional Pipeline
entry is there and makes sense to me.
Acked-by: Russell B
On 04/24/2015 06:34 PM, Ben Pfaff wrote:
> This is what I'm in the midst of implementing, and what ovn-northd uses.
>
> Reported-by: Justin Pettit
> Signed-off-by: Ben Pfaff
> ---
> ovn/northd/ovn-northd.c | 11 ++
> ovn/ovn-sb.xml | 89
> --
This commit is a POC on how to build the installer from the command line.
One could issue the following commands to build the binaries, driver and
installer.
./boot.sh;./configure CC=./build-aux/cccl LD="`which link`" LIBS="-lws2_32" \
--prefix="C:/openvswitch/usr" --localstatedir="C:/openvswit
Add testsuite.tmp.orig to .gitignore
Signed-off-by: Alin Gabriel Serdean
---
.gitignore | 1 +
1 file changed, 1 insertion(+)
diff --git a/.gitignore b/.gitignore
index e37a690..8f0d41f 100644
--- a/.gitignore
+++ b/.gitignore
@@ -66,3 +66,4 @@ _debian
odp-netlink.h
OvsDpInterface.h
/.vagran
It will be a new life for you and your lady
Feel like a man once again with these amazing pills
WE have got what you are looking for!
Free tablets only this week!
http://zzb.bz/l3P2E
World Genuine (Pfizer)
___
dev mailing list
dev@openvswitch.org
637874743609301
Content-Type: text/plain;
Content-Transfer-Encoding: 7Bit
Take the pills and become a much better lover
The fun you have between the sheets will blow your mind
Sale this week!
Get FREE pills now!
http://zzb.bz/AbFXB
Here we sell World Branded tablets
637874743609301--
In this patch, we make changes to usersapce as well as
kernel datapath on hyperv to make it more netlink socket
like. Previously, the kernel datapath did not distinguish
between "transport errors" and other errors. Netlink
semantics dictate that netlink functions should only
return an error only in
hi Sorin,
Thanks so much for the review.
Let me address your comments and then send out the updated patch. If I missed
anything, pls. let me know.
> UINT8 version;
> -UINT8 pad;
> +UINT8 pad1;
> UINT16 maxAttr;
> +UINT16 pad2;
> SV: Do we need the new 'pad' members, 'pad1' &
I now look better than I did in my teens!
Why wouldn't you want to try this.
This month we sell most popular pharmacy products at lowest prices ever seen!
http://zzb.bz/4IggP
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listin
Hi Nithin,
The patch looks good to me. I have some minor comments. Please see inline.
Acked-by: Sorin Vinturis
Thanks,
Sorin
-Original Message-
From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Nithin Raju
Sent: Saturday, 18 April, 2015 00:37
To: dev@openvswitch.org
Subject:
60 matches
Mail list logo