Hi Ankur,
Couple of comments, otherwise LGTM.
Thanks
Numan
On Wed, Sep 4, 2019 at 12:45 AM Ankur Sharma
wrote:
> During E-W routing for vlan backed networks, we replace router port
> mac with chassis mac, when packet leaves the source hypervisor.
>
> As a result, the destination VM (on
On 5 Sep 2019, at 18:21, William Tu wrote:
Did some more testing and with PVP I see a performance decrease, with
physical to physical I see an increase.
Tests are performed with a port redirect open flow rule on an ixgbe
(Xeon E5-2690 v4 2.60GHz):
On 02.09.2019 14:27, Ilya Maximets wrote:
>
> Version 2:
> * Removed redundant OVS_UNUSED.
> * 'ovs_numa_thread_setaffinity_core' reimplemented via
> 'ovs_numa_thread_setaffinity_dump' to reduce code duplication.
> * Added Ack from William to patches #2 and #3 as they wasn't changed.
>
On 13.08.2019 19:37, Ilya Maximets wrote:
> This is highly useful to see on which core PMD is running by
> only looking at the thread name. Thread Id still allows to
> distinguish different threads running on the same core over the time:
>
>|dpif_netdev(pmd-c10/id:53)|DBG|Creating 2. subtable
On 05.09.2019 17:44, Eelco Chaudron wrote:
> See inlines below, and will sent a v2 early next week.
>
> On 5 Sep 2019, at 14:40, Ilya Maximets wrote:
>
>> Hi Eelco,
>> Thanks for the patch! Looks reasonable.
>>
>> One comment is that it's better to explicitly initialize the
>> flag in
On Fri, Sep 6, 2019 at 3:24 PM Ilya Maximets wrote:
>
> On 06.09.2019 14:26, David Marchand wrote:
> > No need to duplicate and null-terminate the passed buffer.
> > We can directly give it to the vlog subsystem using a dynamic precision
> > in the format string.
> >
> > Signed-off-by: David
On 05.09.2019 16:21, Kevin Traynor wrote:
> The comment was incorrectly updated. Fix it to the
> correct value of 32 pad bytes.
>
> /* --- cacheline 5 boundary (320 bytes) --- */
> union {
> struct {
> struct netdev_stats stats; /* 320 336 */
> /*
On 06.09.2019 14:26, David Marchand wrote:
> No need to duplicate and null-terminate the passed buffer.
> We can directly give it to the vlog subsystem using a dynamic precision
> in the format string.
>
> Signed-off-by: David Marchand
> ---
Thanks for a patch. Looks good at a first glance.
I pushed this series to master.
On 9/3/19 10:22 AM, Lorenzo Bianconi wrote:
Introduce the capability to associate a meter to each controller event
type in order to not overload the pinctrl thread under heavy load.
For each event type we need to define a naming convention for the related
meter:
On 05.09.2019 9:58, Sriram Vatala wrote:
> Hi Ilya,
> Thanks a lot for the explanation. As per your suggestion, I will move all the
> counters (including 'tx_retries')to some structure and place a pointer to it
> in netdev_dpdk structure so that the padding size will not vary with the
>
On 6 Sep 2019, at 13:26, David Marchand wrote:
> No need to duplicate and null-terminate the passed buffer.
> We can directly give it to the vlog subsystem using a dynamic precision
> in the format string.
>
> Signed-off-by: David Marchand
Change looks good to me!
Acked-by: Eelco Chaudron
Signed-off-by: Justin Pettit
---
NEWS | 3 +++
configure.ac | 2 +-
debian/changelog | 6 ++
3 files changed, 10 insertions(+), 1 deletion(-)
diff --git a/NEWS b/NEWS
index f177d7efc181..7c45f561cb0d 100644
--- a/NEWS
+++ b/NEWS
@@ -1,3 +1,6 @@
+v2.11.4 - xx xxx
Signed-off-by: Justin Pettit
---
NEWS | 3 ++-
debian/changelog | 2 +-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/NEWS b/NEWS
index 8f84f68a4dfe..f177d7efc181 100644
--- a/NEWS
+++ b/NEWS
@@ -1,5 +1,6 @@
-v2.11.3 - xx xxx
+v2.11.3 - 06 Sep 2019
Bleep bloop. Greetings Justin Pettit, I am a robot and I have tried out your
patch.
Thanks for your contribution.
I encountered some error that I wasn't expecting. See the details below.
git-am:
Failed to merge in the changes.
Patch failed at 0001 Set release date for 2.11.3.
The copy of the
Katar Petroleum
Doha, Katar
Grüße an dich, Schatz.
Haben Sie die vorherige Mail erhalten, die ich Ihnen gesendet habe?
Ich warte auf Ihre Antwort so schnell wie möglich.
Freundliche Grüße,
Eng. Salem Mohd H.A. Al-Mohannadi
Manager Finanzen und Konto,
Aus Katar Petroleum.
Signed-off-by: Justin Pettit
---
NEWS | 3 ++-
debian/changelog | 2 +-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/NEWS b/NEWS
index 46c751571482..8015901d63d9 100644
--- a/NEWS
+++ b/NEWS
@@ -1,5 +1,6 @@
-v2.10.4 - xx xxx
+v2.10.4 - 06 Sep 2019
Signed-off-by: Justin Pettit
---
NEWS | 3 +++
configure.ac | 2 +-
debian/changelog | 6 ++
3 files changed, 10 insertions(+), 1 deletion(-)
diff --git a/NEWS b/NEWS
index 8015901d63d9..1756f605a657 100644
--- a/NEWS
+++ b/NEWS
@@ -1,3 +1,6 @@
+v2.10.5 - xx xxx
Hi:
Adding correct ovs-discuss ML. I did get a chance to take a look on it a
bit. I think this is the bug in 4.4.0-104-generic kernel on ubuntu 16.04 as
its being discussed on ubuntu forum
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1711407 where it can
be hit all of a sudden as per the
Hi,
Would you be interested in an email lead list of Supply Chain Executives? We
can help you reach out to.
Title includes:
? VP of Supply Chain
? Director of Supply Chain
? Supply Chain Manager
? Purchasing Manager
? Purchasing Director
? Procurement Manager
? Procurement
Hi Lorenzo, I only have one finding down below
On 9/6/19 11:00 AM, Lorenzo Bianconi wrote:
Add empty load balancer controller_event support to logical router
pipeline. Update northd documentation even for logical switch pipeline
Signed-off-by: Lorenzo Bianconi
---
Changes since v1:
- rebase
Bleep bloop. Greetings Justin Pettit, I am a robot and I have tried out your
patch.
Thanks for your contribution.
I encountered some error that I wasn't expecting. See the details below.
git-am:
Failed to merge in the changes.
Patch failed at 0001 Set release date for 2.10.4.
The copy of the
On Fri, Sep 06, 2019 at 10:16:24AM -0700, Justin Pettit wrote:
> Signed-off-by: Justin Pettit
Seems fine.
For both patches:
Acked-by: Ben Pfaff
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> On Sep 5, 2019, at 12:38 PM, Ben Pfaff wrote:
>
> All of these 2.5 to 2.8 release series look good to me.
>
> Acked-by: Ben Pfaff
Thanks. I pushed these to all the appropriate branches.
--Justin
___
dev mailing list
d...@openvswitch.org
On Fri, Sep 06, 2019 at 10:16:59AM -0700, Justin Pettit wrote:
> Signed-off-by: Justin Pettit
Seems fine.
For both patches:
Acked-by: Ben Pfaff
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
This document serves to provide an explanation for how OVN will remain
compatible with OVS. It provides instructions for OVN contributors for
how to maintain compatibility even across older versions of OVS when
possible.
Note that the document currently makes reference to some non-existent
items.
Hi there,
Hope you're doing great.
Would you like to target accounts of below technologies in order to boost your
sales and marketing goals in this quarter?
We can help you provide the data intelligence of:
*Microsoft Azure Users
*AWS Users
*
On Fri, Sep 06, 2019 at 10:33:03AM -0400, Mark Michelson wrote:
> OVN is separated into its own repo. This commit removes the OVN source,
> OVN tests, and OVN documentation. It also removes mentions of OVN from
> most documentation. The only place where OVN has been left is in
> changelogs/NEWS,
> On Sep 6, 2019, at 4:52 PM, Yifeng Sun wrote:
>
> Looks good to me, thanks.
>
> Reviewed-by: Yifeng Sun
Thanks for the quick review. I pushed this to master and branch-2.12.
--Justin
___
dev mailing list
d...@openvswitch.org
Signed-off-by: Justin Pettit
---
Documentation/faq/releases.rst | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/faq/releases.rst b/Documentation/faq/releases.rst
index 8daa23bb2d0c..8c29e32efa71 100644
--- a/Documentation/faq/releases.rst
+++ b/Documentation/faq/releases.rst
@@
Looks good to me, thanks.
Reviewed-by: Yifeng Sun
On Fri, Sep 6, 2019 at 4:33 PM Justin Pettit wrote:
>
> Signed-off-by: Justin Pettit
> ---
> Documentation/faq/releases.rst | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/faq/releases.rst
Good finding. So it is a known kernel bug and it won't get fixed in that
kernel version. :(
>From OVS point of view, the problem is, if such bug exists in kernel, it
prevents ovs-vswitchd to be killed (even by SIGKILL) because the process is
in system call which got blocked because of the kernel
On 05.09.2019 18:18, Ben Pfaff wrote:
> On Tue, Sep 03, 2019 at 06:07:26PM +0300, Ilya Maximets wrote:
>> Useful for tracking where the uninitialized memory came from.
>> Report example:
>>
>> Thread 13 revalidator11:
>> Conditional jump or move depends on uninitialised value(s)
>> at
No need to duplicate and null-terminate the passed buffer.
We can directly give it to the vlog subsystem using a dynamic precision
in the format string.
Signed-off-by: David Marchand
---
lib/dpdk.c | 12 +---
1 file changed, 5 insertions(+), 7 deletions(-)
diff --git a/lib/dpdk.c
Looks good, Lorenzo.
Acked-by: Mark Michelson
On 9/3/19 10:22 AM, Lorenzo Bianconi wrote:
Introduce the capability to associate a meter to each controller event
type in order to not overload the pinctrl thread under heavy load.
For each event type we need to define a naming convention for the
> On Sep 5, 2019, at 4:40 PM, aginwala wrote:
>
> Hi Justin:
>
> Please back-port the above patches to branch-2.10 too as it fails there too.
Done. Thanks.
--Justin
___
dev mailing list
d...@openvswitch.org
On Fri, Sep 6, 2019 at 1:14 AM Eelco Chaudron wrote:
>
>
>
> On 5 Sep 2019, at 18:21, William Tu wrote:
>
> >> Did some more testing and with PVP I see a performance decrease, with
> >> physical to physical I see an increase.
> >> Tests are performed with a port redirect open flow rule on an
Add empty load balancer controller_event support to logical router
pipeline. Update northd documentation even for logical switch pipeline
Signed-off-by: Lorenzo Bianconi
---
Changes since v1:
- rebase on-top of current ovn master branch
---
northd/ovn-northd.8.xml | 10
37 matches
Mail list logo