From: Numan Siddique
Presently if the user wants to start OVN db servers as separate containers,
'ovn-ctl'
script is not useful as '--detach' option is passed when ovsdb-servers are
started.
If the container command is 'ovn-ctl start_nb_ovsdb', the container exits as
soon as
ovn-ctl exits.
T
On 20/11/2017 14:30, Simon Horman wrote:
On Sun, Nov 19, 2017 at 08:45:19AM +0200, Roi Dayan wrote:
On 16/11/2017 18:29, Simon Horman wrote:
On Wed, Oct 25, 2017 at 02:24:15PM +0300, Roi Dayan wrote:
On 27/09/2017 12:08, Simon Horman wrote:
On Mon, Sep 25, 2017 at 04:31:42PM +0300, Paul
On 11/20/17, 2:53 PM, "Jan Scheurich" wrote:
Thanks, Darrel, for the quick patch.
I have one major concern (see below).
> > This code doesn't care across packets. It simply always sets
> > CS_ESTABLISHED and CS_REPLY_DIR when ctx->reply is true.
> >
> >
This patch enhances dpif-netdev-perf to detect iterations with
suspicious statistics according to the following criteria:
- iteration lasts longer than US_THR microseconds (default 250).
This can be used to capture events where a PMD is blocked or
interrupted for such a period of time that the
Friendly reminder to test and/or review this patch series presented in the OVS
conference talk "Troubleshooting the OVS Data Plane".
The run-time performance of PMDs is often difficult to understand and
trouble-shoot. The existing PMD statistics counters only provide a coarse
grained average pi
This patch instruments the dpif-netdev datapath to record detailed
statistics of what is happening in every iteration of a PMD thread.
The covered metrics per iteration are:
- cycles
- packets
- (rx) batches
- packets/batch
- max. vhostuser qlen
- upcalls
- cycles spent in upcalls
T
Add module dpif-netdev-perf to host all PMD performance-related
data structures and functions in dpif-netdev. Refactor the PMD
stats handling in dpif-netdev and delegate whatever possible into
the new module, using clean interfaces to shield dpif-netdev from
the implementation details. Accordingly,
This change adds three new options to the Northbound
Logical_Router_Port's ipv6_ra_configs option:
* send_periodic: If set to "true", then OVN will send periodic router
advertisements out of this router port.
* max_interval: The maximum amount of time to wait between sending
periodic router advert
When this flag is set, then a multicast packet that would normally be
delivered to ports on multiple hypervisors is only delivered to ports
on the local hypervisor.
The primary known use case for this is when multicast packets originate
from ovn-controller. Multiple ovn-controllers will be attempt
This patch series adds support for sending periodic router
advertisements from OVN. The patch is divided into two commits
* Commit 1 introduces a method to ensure that multicast packets are sent
only to targets on the local hypervisor
* Commit 2 adds the ability to actually send the periodic RAs
Decap functions for tunneling protocols do not compute
the packet header offsets correctly when there is a VLAN
tag in the L2 header. This results in incorrect checksum
computation causing the packet to be dropped.
This patch adds support to account for the VLAN tag in the
packet if its present, a
On Mon, Nov 20, 2017 at 11:48 AM Mark Michelson wrote:
> On Tue, Nov 14, 2017 at 2:31 AM Numan Siddique
> wrote:
>
>>
>> A few comments inline, otherwise LGTM.
>>
>> Thanks
>> Numan
>>
>>
>> On Mon, Nov 13, 2017 at 8:50 AM, Mark Michelson
>> wrote:
>>
>>> This change adds three new options to t
Thanks, Darrel, for the quick patch.
I have one major concern (see below).
> > This code doesn't care across packets. It simply always sets
> > CS_ESTABLISHED and CS_REPLY_DIR when ctx->reply is true.
> >
> > Did I misunderstand something?
> >
> > There are Two separate ‘if’ conditio
On Mon, 20 Nov 2017 08:48:10 -0800
Ben Pfaff wrote:
> On Mon, Nov 20, 2017 at 11:42:43AM -0200, Flavio Leitner wrote:
> > On Sun, 19 Nov 2017 10:58:11 -0800 Ben Pfaff wrote:
> > > The other technical issue here is about whether && and || should be
> > > at the beginning or end of a line. The
Darrell Ball writes:
> On 11/20/17, 10:02 AM, "Aaron Conole" wrote:
>
> Darrell Ball writes:
>
> > On 11/20/17, 9:43 AM, "Aaron Conole" wrote:
> >
> > Darrell Ball writes:
> >
> > > On 11/20/17, 7:46 AM, "ovs-dev-boun...@openvswitch.org on behalf
> o
(NOTE: This is a resend - I fat-fingered the ovs email. Apologies to
those who got duplicates).
This email is meant to summarize some of the discussions we had at OvS
conference.
The interest in the userspace conntrack is heating up. That's a good
thing, but it means that we'll probably have so
Darrell Ball writes:
> On 11/20/17, 7:19 AM, "ovs-dev-boun...@openvswitch.org on behalf of
> Aaron Conole" acon...@redhat.com> wrote:
>
> Hi Darrell,
>
> Darrell Ball writes:
>
> > Presently, alg processing is enabled by default to exercise testing.
> > This is similar
On 11/20/17, 10:02 AM, "Aaron Conole" wrote:
Darrell Ball writes:
> On 11/20/17, 9:43 AM, "Aaron Conole" wrote:
>
> Darrell Ball writes:
>
> > On 11/20/17, 7:46 AM, "ovs-dev-boun...@openvswitch.org on behalf of
> > Aaron Conole" > acon..
Nothing in the OVS tree uses Perl any longer, so remove the dependency.
Signed-off-by: Ben Pfaff
---
Documentation/intro/install/general.rst | 8
Makefile.am | 2 --
configure.ac| 6 --
tests/atlocal.in|
Perl is unfashionable and Python is more widely available and understood,
so this commit converts one of the OVS uses of Perl into Python.
Signed-off-by: Ben Pfaff
---
tests/atlocal.in | 8 +++-
tests/ofproto.at | 38 --
2 files changed, 23 insertions(+),
Perl is unfashionable and Python is more widely available and understood,
so this commit converts one of the OVS uses of Perl into Python.
Signed-off-by: Ben Pfaff
---
ovn/automake.mk | 4 +-
ovsdb/dot2pic| 155 +++
vswitchd/automake
Perl is unfashionable and Python is more widely available and understood,
so this commit converts one of the OVS uses of Perl into Python.
Signed-off-by: Ben Pfaff
---
Makefile.am| 6 ++---
build-aux/automake.mk | 1 +
build-aux/sodepends.pl | 70 --
Perl is unfashionable and Python is more widely available and understood,
so this commit converts one of the OVS uses of Perl into Python.
Signed-off-by: Ben Pfaff
---
Makefile.am| 4 ++--
build-aux/automake.mk | 3 ++-
build-aux/soexpand.pl | 40 -
Perl is unfashionable and Python is more widely available and understood,
so this commit converts one of the OVS uses of Perl into Python.
Signed-off-by: Ben Pfaff
---
Makefile.am | 4 ++--
build-aux/automake.mk | 3 ++-
build-aux/dpdkstrip.pl | 35 --
Perl is unfashionable and Python is more widely available and understood,
so this commit converts one of the OVS uses of Perl into Python.
Signed-off-by: Ben Pfaff
---
tests/automake.mk | 2 +-
tests/flowgen.pl | 253 --
tests/flowgen.py |
Perl is unfashionable and Python is more widely available and understood,
so this commit converts one of the OVS uses of Perl into Python.
Signed-off-by: Ben Pfaff
---
tests/automake.mk| 5 +-
tests/ofproto-macros.at | 4 +-
tests/ofproto.at | 24 +++
Perl is unfashionable and Python is more widely available and understood,
so this commit converts one of the OVS uses of Perl into Python.
Signed-off-by: Ben Pfaff
---
tests/automake.mk | 2 +-
tests/ovsdb-monitor-sort.pl | 52 ---
tests/ovsdb-monitor-sort.py |
This series gets rid of all usage of Perl in the tree, converting
it to Python instead.
v1->v2:
* Minor fixes from Aaron Conole.
* Break soexpand and sodepends common code into library (suggested by
Aaron Conole)
Ben Pfaff (9):
tests: Convert uuidfilt utility from Perl to Python.
t
On Thu, Nov 16, 2017 at 09:32:18AM -0500, Aaron Conole wrote:
> Ben Pfaff writes:
>
> > Perl is unfashionable and Python is more widely available and understood,
> > so this commit converts one of the OVS uses of Perl into Python.
> >
> > Signed-off-by: Ben Pfaff
> > +label =
On Thu, Nov 16, 2017 at 09:35:48AM -0500, Aaron Conole wrote:
> Ben Pfaff writes:
>
> > Perl is unfashionable and Python is more widely available and understood,
> > so this commit converts one of the OVS uses of Perl into Python.
> >
> > Signed-off-by: Ben Pfaff
> > +# This is copied out of the
Darrell Ball writes:
> On 11/20/17, 9:43 AM, "Aaron Conole" wrote:
>
> Darrell Ball writes:
>
> > On 11/20/17, 7:46 AM, "ovs-dev-boun...@openvswitch.org on behalf of
> > Aaron Conole" > acon...@redhat.com> wrote:
> >
> > Darrell Ball writes:
> >
>
On Thu, Nov 16, 2017 at 09:40:06AM -0500, Aaron Conole wrote:
> Ben Pfaff writes:
>
> > Perl is unfashionable and Python is more widely available and understood,
> > so this commit converts one of the OVS uses of Perl into Python.
> >
> > Signed-off-by: Ben Pfaff
> > +def find_file(include_dirs,
On Thu, Nov 16, 2017 at 09:42:20AM -0500, Aaron Conole wrote:
> Ben Pfaff writes:
>
> > Perl is unfashionable and Python is more widely available and understood,
> > so this commit converts one of the OVS uses of Perl into Python.
> >
> > Signed-off-by: Ben Pfaff
> > +def find_file(include_dirs,
On Tue, Nov 14, 2017 at 2:31 AM Numan Siddique wrote:
>
> A few comments inline, otherwise LGTM.
>
> Thanks
> Numan
>
>
> On Mon, Nov 13, 2017 at 8:50 AM, Mark Michelson
> wrote:
>
>> This change adds three new options to the Northbound
>> Logical_Router_Port's ipv6_ra_configs option:
>>
>> * se
On 11/20/17, 9:43 AM, "Aaron Conole" wrote:
Darrell Ball writes:
> On 11/20/17, 7:46 AM, "ovs-dev-boun...@openvswitch.org on behalf of
> Aaron Conole" acon...@redhat.com> wrote:
>
> Darrell Ball writes:
>
> > Presently, the userpace connection tr
Darrell Ball writes:
> On 11/20/17, 7:46 AM, "ovs-dev-boun...@openvswitch.org on behalf of
> Aaron Conole" acon...@redhat.com> wrote:
>
> Darrell Ball writes:
>
> > Presently, the userpace connection tracker 'established' packet
> > state diverges from the kernel and this patch
On 11/20/17, 7:19 AM, "ovs-dev-boun...@openvswitch.org on behalf of Aaron
Conole" wrote:
Hi Darrell,
Darrell Ball writes:
> Presently, alg processing is enabled by default to exercise testing.
> This is similar to kernels before 4.7. The recommended default
> be
Hi Yipeng and Sameh,
As discussed on the OVS conference, I have just sent my Datapath Flow Cache
patch to the ML:
https://mail.openvswitch.org/pipermail/ovs-dev/2017-November/341066.html
http://patchwork.ozlabs.org/patch/839645/
I would be happy if you could test it in your setup and compare the
So far the netdev datapath uses an 8K EMC to speed up the lookup of
frequently used flows by comparing the parsed packet headers against
the miniflow of a cached flow, using 13 bits of the packet RSS hash
as index. The EMC is too small for many applications with 100K or more
parallel packet flows s
When ofm is not referenced by xc_entry, we should release its
resources by calling ofproto_flow_mod_uninit because no one is
going to use it in this function.
Signed-off-by: Yifeng Sun
---
ofproto/ofproto-dpif-xlate.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/ofproto/ofproto-dpif-xla
If there is an error after ovsdb_rbac_insert, 'row' is leaked.
So move the existing ovsdb_row_destroy to the function end.
Signed-off-by: Yifeng Sun
---
ovsdb/execution.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/ovsdb/execution.c b/ovsdb/execution.c
index 1ebe7a772e4
On Fri, Nov 17, 2017 at 07:27:51PM +, Stokes, Ian wrote:
> The following changes since commit 1ae83bb20677b42d63dbb2140fa8ed3144c6260f:
>
> netdev-tc-offloads: Add support for action set (2017-11-16 08:10:29 -0800)
>
> are available in the git repository at:
>
> https://github.com/istoke
On 11/20/17, 7:46 AM, "ovs-dev-boun...@openvswitch.org on behalf of Aaron
Conole" wrote:
Darrell Ball writes:
> Presently, the userpace connection tracker 'established' packet
> state diverges from the kernel and this patch brings them in line.
> The behavior is now that
On Thu, Nov 09, 2017 at 10:15:31AM +, Vishal Deep Ajmera wrote:
> Adding the break statement for CLONE action case.
>
> Signed-off-by: Vishal Deep Ajmera
Thanks, applied to master.
I adjusted the commit message to reflect that it doesn't fix an existing
bug, only makes future bugs less like
On Mon, Nov 20, 2017 at 11:42:43AM -0200, Flavio Leitner wrote:
> On Sun, 19 Nov 2017 10:58:11 -0800 Ben Pfaff wrote:
> > The other technical issue here is about whether && and || should be
> > at the beginning or end of a line. The coding style document,
> > intentionally, does not take an expli
Yes, of course. I didn't get much work done last week with ovscon
going.
On Mon, Nov 20, 2017 at 09:11:41PM +0530, Numan Siddique wrote:
> A gentle ping. Ben, is it possible to have a look at this patch set -
> https://patchwork.ozlabs.org/project/openvswitch/list/?series=12481.
>
> Thanks
> Num
>
>Bhanuprakash Bodireddy writes:
>
>> pkt_metadata_prefetch_init() is used to prefetch the packet metadata
>> before initializing the metadata in pkt_metadata_init(). This is done
>> for every packet in userspace datapath and is performance critical.
>>
>> Commit 99fc16c0 prefetches only cachline
Bhanuprakash Bodireddy writes:
> pkt_metadata_prefetch_init() is used to prefetch the packet metadata
> before initializing the metadata in pkt_metadata_init(). This is done
> for every packet in userspace datapath and is performance critical.
>
> Commit 99fc16c0 prefetches only cachline0 and cac
Darrell Ball writes:
> Presently, the userpace connection tracker 'established' packet
> state diverges from the kernel and this patch brings them in line.
> The behavior is now that 'established' is only possible after a
> reply packet is seen.
> The previous behavior is hard to notice when rule
A gentle ping. Ben, is it possible to have a look at this patch set -
https://patchwork.ozlabs.org/project/openvswitch/list/?series=12481.
Thanks
Numan
On Wed, Nov 8, 2017 at 2:30 PM, Numan Siddique wrote:
>
>
> On Wed, Nov 8, 2017 at 9:36 AM, Numan Siddique
> wrote:
>
>>
>>
>> On Wed, Nov 8,
Hi Darrell,
Darrell Ball writes:
> Presently, alg processing is enabled by default to exercise testing.
> This is similar to kernels before 4.7. The recommended default
> behavior in the kernel is to only process algs if a helper is
> supplied in a conntrack rule. The behavior is changed to ma
Hi Darrell,
Darrell Ball writes:
> Upcoming requirements for new algs make it necessary to split out
> alg helper more cleanly.
>
> Signed-off-by: Darrell Ball
> ---
Thanks for jumping on this so quickly.
I think this and 3/4 should be an independent series. They don't have
much to do with 1
Hello there,
I would like to know if you are interested in acquiring CHEF Software Users
List.
Information fields: Names, Title, Email, Phone, Company Name, Company URL,
Company physical address, SIC Code, Industry and Company Size (Revenue and
Employee).
Let me know if you are interested and
On Sat, 18 Nov 2017 19:41:44 -0500
Eric Garver wrote:
> This supports using the ct_clear action in the kernel datapath. To
> preserve compatibility with current ct_clear behavior on old kernels, we
> only pass this action down to the datapath if a probe reveals the
> datapath actually supports it.
Hi Ben,
On Sun, 19 Nov 2017 10:58:11 -0800
Ben Pfaff wrote:
> On Fri, Nov 17, 2017 at 03:51:34PM -0200, Flavio Leitner wrote:
> > No functional change, just fixing coding style.
> >
> > Signed-off-by: Flavio Leitner
>
> I see from the thread that there's some controversy here. I think tha
On Sun, 19 Nov 2017 11:18:25 -0800
Ben Pfaff wrote:
[...]
> I tend to take this patch as a kind of bug report against the coding
> style document. The right response to that might be to explain why the
> coding style document is wrong and the history behind it, and to thank
> the submitter (thank
On Wed, Nov 8, 2017 at 1:00 PM, Mark Kavanagh wrote:
> This commit adds support for DPDK v17.11:
> - minor updates to accomodate DPDK API changes
> - update references to DPDK version in Documentation
> - update DPDK version in travis' linux-build script
>
> Signed-off-by: Mark Kavanagh
> diff
On Sun, Nov 19, 2017 at 08:45:19AM +0200, Roi Dayan wrote:
>
>
> On 16/11/2017 18:29, Simon Horman wrote:
> > On Wed, Oct 25, 2017 at 02:24:15PM +0300, Roi Dayan wrote:
> > >
> > >
> > > On 27/09/2017 12:08, Simon Horman wrote:
> > > > On Mon, Sep 25, 2017 at 04:31:42PM +0300, Paul Blakey wrote
Hello
Still waiting for your reply
Regards
Susan
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Hi,
2nd attempt. Does anyone have any comments on this add-on to upcalls?
Thanx
Manu
On 17/11/17, 11:17 AM, "Manohar Krishnappa Chidambaraswamy"
wrote:
Hi,
Does anyone have any comments on this add-on to upcalls?
Thanx
Manu
On 10/11/17, 7:06 PM, "ovs-dev-bou
60 matches
Mail list logo
