Please ignor this patch set.
在 2017/5/1 0:15, Michael Qiu 写道:
From: Michael Qiu
Currently, OVS only support DPDK single segment mbuf,
it could lead problems, like a large non-DPDK source
packet transmit to dpdk port.
Also, OVS doesn't copy enough info in mbuf when do
packet copy.
At the sam
From: Michael Qiu
Currently, one packet is only copied to one segment
in function dpdk_do_tx_copy(), this could be an issue
when a jumbo frame comes, especially for multiple segments.
This patch calculate the segment number needed by the packet and
copy the data to different segments.
Signed-o
From: Michael Qiu
When doing packet clone, if packet source is from DPDK driver,
multi-segment must be considered, and copy the segment's
data one by one.
Signed-off-by: Michael Qiu
---
lib/dp-packet.c | 27 ---
1 file changed, 24 insertions(+), 3 deletions(-)
diff --g
From: Michael Qiu
When a packet is from DPDK source, and it contains
multiple segments, data_len is not equal to the
packet size. This patch fix this issue.
Signed-off-by: Michael Qiu
Signed-off-by: Marcin Ksiadz
Signed-off-by: Mark Kavanagh
Signed-off-by: Przemyslaw Lal
Signed-off-by: Yuanh
Please ignor this patch set
在 2017/5/1 0:15, Michael Qiu 写道:
From: Michael Qiu
Currently, OVS only support DPDK single segment mbuf,
it could lead problems, like a large non-DPDK source
packet transmit to dpdk port.
Also, OVS doesn't copy enough info in mbuf when do
packet copy.
At the same
From: Michael Qiu
Currently, when doing packet copy, lots of DPDK mbuf's info
will be missed, like packet type, ol_flags, etc.
Those information is very important for DPDK to do
packets processing.
Signed-off-by: Michael Qiu
---
lib/dp-packet.c | 3 +++
lib/netdev-dpdk.c | 4
2 files ch
From: Michael Qiu
Currently, OVS only support DPDK single segment mbuf,
it could lead problems, like a large non-DPDK source
packet transmit to dpdk port.
Also, OVS doesn't copy enough info in mbuf when do
packet copy.
At the same time, vlan and tunnelling packet's DPDK
offloads, for example TS
From: Michael Qiu
When building with DPDK, and using xmalloc() to get a new packet,
field mbuf of the packet will not be initialized, but it's very important for
DPDK port when copying the data to DPDK mbuf, because if ol_flags
and other info are random values, DPDK driver may hang.
Signed-off-b
On Tue, May 2, 2017 at 5:21 AM, Ben Pfaff wrote:
> On Fri, Apr 21, 2017 at 10:32:57AM +0900, YAMAMOTO Takashi wrote:
> > NetBSD implementation of wc command outputs extra whitespaces
> > like the following. Tweak the test to success on such environments.
> >
> > % echo hoge|wc -l|hexdump -C
On Mon, May 1, 2017 at 5:12 PM, Ben Pfaff wrote:
> On Mon, May 01, 2017 at 03:39:32PM -0700, Mickey Spiegel wrote:
> > On Sun, Apr 30, 2017 at 4:22 PM, Ben Pfaff wrote:
> >
> > > Without this support, ovn-trace is not very useful with OpenStack,
> which
> > > uses connection tracking extensively
On Mon, May 1, 2017 at 11:19 AM, Ben Pfaff wrote:
>
> On Tue, Apr 18, 2017 at 11:12:14AM -0700, Han Zhou wrote:
> > In commit c1bfdd9d it disables probe when not needed, but commit
> > 715038b6 updates ovn-controller probe interval for OVNSB DB
> > periodically according to ovn-remote-probe-interv
On Mon, May 01, 2017 at 03:39:32PM -0700, Mickey Spiegel wrote:
> On Sun, Apr 30, 2017 at 4:22 PM, Ben Pfaff wrote:
>
> > Without this support, ovn-trace is not very useful with OpenStack, which
> > uses connection tracking extensively.
> >
>
> I scanned the patch set briefly, not what I would c
On Mon, May 01, 2017 at 10:55:55PM +, Bodireddy, Bhanuprakash wrote:
> >> > This patch is aimed at achieving Fastpath Service Assurance in
> >> > OVS-DPDK deployments. This commit adds support for monitoring the
> >> > packet processing cores(pmd thread cores) by dispatching heartbeats
> >> > a
>There two ways to go with the design.
>
>1) Make it generic, so that it is not so PMD specific, as it is now.
>2) If it stays PMD specific, make it stronger; right now, the health check is
>limited – it detects that a PMD thread is proceeding or not.
>For something like DPDK, I don’t think that wi
>> > This patch is aimed at achieving Fastpath Service Assurance in
>> > OVS-DPDK deployments. This commit adds support for monitoring the
>> > packet processing cores(pmd thread cores) by dispatching heartbeats
>> > at regular intervals. Incase of heartbeat miss the failure shall be
>> > detected
> From: "Ben Pfaff"
> To: "Lance Richardson"
> Cc: d...@openvswitch.org
> Sent: Monday, 1 May, 2017 5:57:43 PM
> Subject: Re: [ovs-dev] [PATCH v2 1/3] ovsdb: add xml equivalents of remote
> man page fragments
>
> On Mon, Apr 24, 2017 at 02:57:11PM -0400, Lance Richardson wrote:
> > Add XML equi
On Sun, Apr 30, 2017 at 4:22 PM, Ben Pfaff wrote:
> Without this support, ovn-trace is not very useful with OpenStack, which
> uses connection tracking extensively.
>
I scanned the patch set briefly, not what I would call a full review but
quick sanity checking. The only issue that I saw is desc
On Mon, May 1, 2017 at 2:44 PM, Ben Pfaff wrote:
>
> On Fri, Apr 21, 2017 at 06:55:28PM -0700, Han Zhou wrote:
> > As discussed in [1], what the incremental processing code
> > actually accomplished was that the ovn-controller would
> > be "quiet" and not burn CPU when things weren't changing.
> >
On Mon, Apr 24, 2017 at 02:57:11PM -0400, Lance Richardson wrote:
> Add XML equivalents for remote-active.man and remote-passive.man
> for inclusion by man pages using XML format.
>
> Signed-off-by: Lance Richardson
Thanks a lot! I really appreciate updating the documentation. No one
reads the
On Tue, Apr 25, 2017 at 02:06:25PM +0100, Kevin Traynor wrote:
> On 04/24/2017 07:50 PM, Ben Pfaff wrote:
> > On Mon, Apr 24, 2017 at 06:48:34PM +0100, Kevin Traynor wrote:
> >> Using dpdk-socket-mem to allocate memory for some NUMA nodes
> >> but leaving blank for subsequent ones is equivalent of
On Tue, Apr 25, 2017 at 02:04:44PM +0100, Kevin Traynor wrote:
> Using dpdk-socket-mem to allocate memory for some NUMA nodes
> but leaving blank for subsequent ones is equivalent of assigning
> 0 MB memory to those subsequent nodes. Document this behavior.
>
> Signed-off-by: Kevin Traynor
Thank
On Mon, Apr 24, 2017 at 10:21:18AM +0200, majop...@redhat.com wrote:
> From: Miguel Angel Ajo
>
> The intergateway monitoring covers host failure well, but
> it doesn't cover path failure which is a more complicated
> problem.
>
> By this change I don't mean we should implement something
> to co
On Fri, Apr 21, 2017 at 06:55:28PM -0700, Han Zhou wrote:
> As discussed in [1], what the incremental processing code
> actually accomplished was that the ovn-controller would
> be "quiet" and not burn CPU when things weren't changing.
> This patch set recreates this state by calculating whether
>
On Fri, Apr 21, 2017 at 06:55:27PM -0700, Han Zhou wrote:
> When there are in-flight msgs being sent to OVS, ofctrl_put will
> skip, which makes all the flows computed in that main loop
> iteration useless. To avoid the wasted CPU cycles, a check is added
> before lflow/physical flow run in each it
On Mon, May 01, 2017 at 04:45:56PM -0400, Aaron Conole wrote:
> Ben Pfaff writes:
> ...
> > I think it'd be even better to include measurements in one of the commit
> > messages, because those are available in the repository after the
> > patches are applied. It's harder to find cover letters bec
On Mon, May 01, 2017 at 04:39:30PM -0400, Aaron Conole wrote:
> Ben Pfaff writes:
>
> > On Mon, May 01, 2017 at 04:14:09PM -0400, Aaron Conole wrote:
> >> A common way of expressing 'raise to the power of' when authoring
> >> comments uses **. This is currently getting caught by the pointer
> >>
On Thu, Apr 27, 2017 at 05:18:04PM -0400, Aaron Conole wrote:
> Bhanuprakash Bodireddy writes:
>
> > This patch is aimed at achieving Fastpath Service Assurance in
> > OVS-DPDK deployments. This commit adds support for monitoring the packet
> > processing cores(pmd thread cores) by dispatching he
Hi Ben,
Ben Pfaff writes:
...
> I think it'd be even better to include measurements in one of the commit
> messages, because those are available in the repository after the
> patches are applied. It's harder to find cover letters because they're
> only on the mailing list.
One thing that other
On Wed, Apr 26, 2017 at 10:38:08AM -0700, Han Zhou wrote:
> ct_zones is not really used in lflow_run, so remove the dependency.
>
> Signed-off-by: Han Zhou
Thanks for noticing! I wonder how this came about.
I applied this to master.
___
dev mailing l
Ben Pfaff writes:
> On Mon, May 01, 2017 at 04:14:09PM -0400, Aaron Conole wrote:
>> A common way of expressing 'raise to the power of' when authoring
>> comments uses **. This is currently getting caught by the pointer
>> spacing warning. So, catch it here.
>>
>> Reported-by: Lance Richardson
On Mon, May 01, 2017 at 04:14:09PM -0400, Aaron Conole wrote:
> A common way of expressing 'raise to the power of' when authoring
> comments uses **. This is currently getting caught by the pointer
> spacing warning. So, catch it here.
>
> Reported-by: Lance Richardson
> Signed-off-by: Aaron Co
On Mon, May 01, 2017 at 04:14:05PM -0400, Aaron Conole wrote:
> All of the checks are now part of the new 'check' framework.
>
> Signed-off-by: Aaron Conole
> ---
> utilities/checkpatch.py | 41 ++---
> 1 file changed, 26 insertions(+), 15 deletions(-)
>
> di
On Mon, May 01, 2017 at 04:01:12PM -0400, Aaron Conole wrote:
> Ben Pfaff writes:
>
> > On Mon, May 01, 2017 at 03:44:37PM -0400, Aaron Conole wrote:
> >> Developers wishing to add checks to checkpatch sift through an adhoc mess,
> >> currently. The process goes something like:
> >> 1. Figure ou
On Fri, Apr 21, 2017 at 10:32:57AM +0900, YAMAMOTO Takashi wrote:
> NetBSD implementation of wc command outputs extra whitespaces
> like the following. Tweak the test to success on such environments.
>
> % echo hoge|wc -l|hexdump -C
> 20 20 20 20 20 20 20 31 0a
A common way of expressing 'raise to the power of' when authoring
comments uses **. This is currently getting caught by the pointer
spacing warning. So, catch it here.
Reported-by: Lance Richardson
Signed-off-by: Aaron Conole
---
utilities/checkpatch.py | 2 +-
1 file changed, 1 insertion(+),
Filenames that come from the hunks match include the git-ified 'b/'
prefix, which makes jumping to the error file that much harder. This
patch corrects that by simply skipping those bytes.
Signed-off-by: Aaron Conole
---
utilities/checkpatch.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion
Other utilities (notoriously the linux kernel's checkpatch.pl) have a more
standardized form for printing file and lines. With this change, the
template used to print gains two enhancements:
1. Color
2. Conformance with the kernel's version of checkpatch.pl
Signed-off-by: Aaron Conole
---
utili
Occasionally, characters will be sent which violate the
ascii decoder's sense of propriety. In fact, in-tree there are
a few such files (ex: tests/atlocal.in), and they cause an
exception to be raised when they are encountered.
Set the policy to ignore these cases. This means these bytes are
omi
All of the checks are now part of the new 'check' framework.
Signed-off-by: Aaron Conole
---
utilities/checkpatch.py | 41 ++---
1 file changed, 26 insertions(+), 15 deletions(-)
diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py
index 30c0a3e..9d
With the new framework, print_line can be moved out to the checks
framework.
Signed-off-by: Aaron Conole
---
utilities/checkpatch.py | 13 +
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py
index b3b833b..30c0a3e 100755
-
Developers wishing to add checks to checkpatch sift through an adhoc mess,
currently. The process goes something like:
1. Figure out what to test in the patch
2. Write some code, quickly, that checks for that condition
3. Look through the statemachine to find where the check should go
4. ignore pa
The following series refactors checkpatch to make
file-type specific checks. This lets checkpatch have
finer grained checks, and should reduce the amount
of false positives.
Additionally, I've incorporated an older request to print
files in the more standard `filename:lineno: ` form that
most edi
Acked-by: Jarno Rajahalme
> On May 1, 2017, at 12:58 PM, Joe Stringer wrote:
>
> This was being logged using xlate_strerror(), but the return code is
> actually an errno code. Use ovs_strerror() instead.
>
> Fixes: dd0dc9eda0e0 ("revalidator: Reuse xlate_ukey from deletion.")
> Signed-off-by:
Acked-by: Jarno Rajahalme
> On May 1, 2017, at 12:58 PM, Joe Stringer wrote:
>
> If there is no active ukey for a particular datapath flow, and it is
> dumped from the datapath, then the revalidator threads will assemble a
> ukey based on the datapath flow. This will allow tracking of the stats
Ben Pfaff writes:
> On Mon, May 01, 2017 at 03:44:37PM -0400, Aaron Conole wrote:
>> Developers wishing to add checks to checkpatch sift through an adhoc mess,
>> currently. The process goes something like:
>> 1. Figure out what to test in the patch
>> 2. Write some code, quickly, that checks fo
If there is no active ukey for a particular datapath flow, and it is
dumped from the datapath, then the revalidator threads will assemble a
ukey based on the datapath flow. This will allow tracking of the stats
for proper attribution, and future validation of the flow.
However, until now when crea
This was being logged using xlate_strerror(), but the return code is
actually an errno code. Use ovs_strerror() instead.
Fixes: dd0dc9eda0e0 ("revalidator: Reuse xlate_ukey from deletion.")
Signed-off-by: Joe Stringer
---
ofproto/ofproto-dpif-upcall.c | 4 ++--
1 file changed, 2 insertions(+), 2
On Mon, May 01, 2017 at 03:44:37PM -0400, Aaron Conole wrote:
> Developers wishing to add checks to checkpatch sift through an adhoc mess,
> currently. The process goes something like:
> 1. Figure out what to test in the patch
> 2. Write some code, quickly, that checks for that condition
> 3. Look
Corresponding Business Partnership needed. Kindly get back to me urgently.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
A common way of expressing 'raise to the power of' when authoring
comments uses **. This is currently getting caught by the pointer
spacing warning. So, catch it here.
Reported-by: Lance Richardson
Signed-off-by: Aaron Conole
---
utilities/checkpatch.py | 2 +-
1 file changed, 1 insertion(+),
Filenames that come from the hunks match include the git-ified 'b/'
prefix, which makes jumping to the error file that much harder. This
patch corrects that by simply skipping those bytes.
Signed-off-by: Aaron Conole
---
utilities/checkpatch.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion
Other utilities (notoriously the linux kernel's checkpatch.pl) have a more
standardized form for printing file and lines. With this change, the
template used to print gains two enhancements:
1. Color
2. Conformance with the kernel's version of checkpatch.pl
Signed-off-by: Aaron Conole
---
utili
Occasionally, characters will be sent which violate the
ascii decoder's sense of propriety. In fact, in-tree there are
a few such files (ex: tests/atlocal.in), and they cause an
exception to be raised when they are encountered.
Set the policy to ignore these cases. This means these bytes are
omi
All of the checks are now part of the new 'check' framework.
Signed-off-by: Aaron Conole
---
utilities/checkpatch.py | 41 ++---
1 file changed, 26 insertions(+), 15 deletions(-)
diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py
index 239ce69..68
With the new framework, print_line can be moved out to the checks
framework.
Signed-off-by: Aaron Conole
---
utilities/checkpatch.py | 13 +
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py
index 4ee5670..239ce69 100755
-
Developers wishing to add checks to checkpatch sift through an adhoc mess,
currently. The process goes something like:
1. Figure out what to test in the patch
2. Write some code, quickly, that checks for that condition
3. Look through the statemachine to find where the check should go
4. ignore pa
The following series refactors checkpatch to make
file-type specific checks. This lets checkpatch have
finer grained checks, and should reduce the amount
of false positives.
Additionally, I've incorporated an older request to print
files in the more standard `filename:lineno: ` form that
most edi
On Thu, Apr 20, 2017 at 03:25:13PM +0100, mweglicx wrote:
> Implementation of IPFix counters which hold
> total values measured since metering process startup.
>
> v2: Patch is reapplied for a review because
> it hasn't been correctly marked in patchwork.
>
> Signed-off-by: Michal Weglicki
Than
Ben Pfaff writes:
> On Wed, Apr 26, 2017 at 11:54:52AM -0400, Aaron Conole wrote:
>> The following series refactors checkpatch to make
>> file-type specific checks. This lets checkpatch have
>> finer grained checks, and should reduce the amount
>> of false positives.
>>
>> Additionally, I've in
Thanks for pointing that out; it was on my todo list from before and I
forgot about it.
On 5/1/17, 10:54 AM, "ovs-dev-boun...@openvswitch.org on behalf of Jarno
Rajahalme" wrote:
Would be nice to have a commit message, with the motivation for the change.
Jarno
> On Mar 2
On Thu, Apr 20, 2017 at 05:46:58PM +0530, Satyavalli Rama wrote:
> OpenFlow 1.5 introduces the Extensible Statistics (OXS) by redefining the
> existing flow entry statistics with OXS Fields.
> This Patch provide implementation for OXS fields encoding in TLV format.
Thanks a lot for working on th
On Tue, Apr 18, 2017 at 11:12:14AM -0700, Han Zhou wrote:
> In commit c1bfdd9d it disables probe when not needed, but commit
> 715038b6 updates ovn-controller probe interval for OVNSB DB
> periodically according to ovn-remote-probe-interval config, and sets
> it to DEFAULT_PROBE_INTERVAL_MSEC if no
On Mon, May 01, 2017 at 01:51:35PM -0400, Aaron Conole wrote:
> Ben Pfaff writes:
>
> > On Mon, Apr 17, 2017 at 01:59:49PM -0400, Aaron Conole wrote:
> >> The install documentation guided users to manually start/stop
> >> daemons. This is good information to have, but with the
> >> existence of
On Wed, Apr 26, 2017 at 11:54:52AM -0400, Aaron Conole wrote:
> The following series refactors checkpatch to make
> file-type specific checks. This lets checkpatch have
> finer grained checks, and should reduce the amount
> of false positives.
>
> Additionally, I've incorporated an older request
On Thu, Apr 27, 2017 at 02:36:18PM -0700, Joe Stringer wrote:
> On 13 April 2017 at 13:47, Eric Garver wrote:
> > This series adds support for the creation of tunnels using the rtnetlink
> > interface. This will open the possibility for new features and flags on
> > those
> > vports without the n
On Mon, Apr 17, 2017 at 07:04:12PM +0300, Bogdan Ilisei wrote:
> Package: openvswitch-datapath-dkms
> Version: 1.4.2+git20120612-9.1~deb7u1.1
> Severity: grave
> Justification: renders package unusable
>
> Dear Maintainer,
>
> Installing openvswitch-datapath-dkms (OR openvswitch-datapath-source)
Would be nice to have a commit message, with the motivation for the change.
Jarno
> On Mar 24, 2017, at 2:15 AM, Darrell Ball wrote:
>
> Signed-off-by: Darrell Ball
> Acked-by: Flavio Leitner
> ---
> lib/conntrack.c | 58 +++--
> 1 file cha
Ben Pfaff writes:
> On Mon, Apr 17, 2017 at 01:59:49PM -0400, Aaron Conole wrote:
>> The install documentation guided users to manually start/stop
>> daemons. This is good information to have, but with the
>> existence of ovs-ctl, is probably not the best way to start
>> guiding new users of ovs
On Mon, Apr 17, 2017 at 01:59:49PM -0400, Aaron Conole wrote:
> The install documentation guided users to manually start/stop
> daemons. This is good information to have, but with the
> existence of ovs-ctl, is probably not the best way to start
> guiding new users of ovs.
>
> Suggest that users
Thank you! I applied this to master.
On Thu, Apr 27, 2017 at 09:56:45AM -0700, Yi-Hung Wei wrote:
> Thanks for the patch. I cross-check this patch with 6dc5374435cc
> ("Enable OpenFlow 1.0, 1.1, 1.2, and 1.3 by default."). It LGTM.
>
> Acked-by: Yi-Hung Wei
>
> On Mon, Apr 24, 2017 at 11:49 AM
On Thu, Apr 27, 2017 at 09:37:17AM -0700, Ben Pfaff wrote:
> "ovn-nbctl show" indented every line of output by at least 4 spaces, which
> needlessly wastes horizontal space. This drops 4 spaces of indent from
> each line of output.
>
> Signed-off-by: Ben Pfaff
This is part of the 27-patch serie
Hemant, does this fix the problem you reported?
On Fri, Apr 14, 2017 at 09:14:55PM -0700, Ben Pfaff wrote:
> The check for rte_config.h in acinclude.m4 used AC_CHECK_FILE, but this
> macro is intended to check for a file on the host system, not the build
> system, which means that it fails uncondi
On Sat, Apr 29, 2017 at 06:30:59AM -0700, William Tu wrote:
> Under Linux, when users create bridge named "default" or "all", although
> ovs-vsctl fails but vswitchd in the background will keep retrying it,
> causing the systemd-udev to reach 100% cpu utilization. The patch prevents
> any attempt t
There's a bit of a problem with flake8 and release branches. The
release branches mainly stay the same, but flake8 marches on, and the
result is that building an old branch, e.g. branch-2.6, with modern
flake8 causes errors that break the build.
I can think of several ways to deal with this:
Add ping test to verify the behavior of mpls_push/pop actions. In this
test, we use the resubmit action to trigger recirulation for making sure
the flow key is revalidated after mpls_push/pop. This test depends on
commit 5ba0c107c51e ("datapath: Fix ovs_flow_key_update()") to behave
correctly.
Sig
Upstream commit:
commit 6f56f6186c18e3fd54122b73da68e870687b8c59
Author: Yi-Hung Wei
Date: Thu Mar 30 12:36:03 2017 -0700
ovs_flow_key_update() is called when the flow key is invalid, and it is
used to update and revalidate the flow key. Commit 329f45bc4f19
("openvswitch
Upstream commit:
commit c66549ffd05831abf6cf19ce0571ad868e39
Author: Jiri Benc
Date: Wed Oct 5 15:01:57 2016 +0200
openvswitch: correctly fragment packet with mpls headers
If mpls headers were pushed to a defragmented packet, the refragmentation no
longer works corr
Upstream commit:
commit f7d49bce8e741e1e6aa14ce4db1b6cea7e4be4e8
Author: Jiri Benc
Date: Fri Sep 30 19:08:05 2016 +0200
openvswitch: mpls: set network header correctly on key extract
After the 48d2ab609b6b ("net: mpls: Fixups for GSO"), MPLS handling in
openvswitch was
Sure, I will rebase the patch set and post the v4.
Thanks,
-Yi-Hung
On Mon, May 1, 2017 at 3:42 AM, Simon Horman wrote:
> Hi Yi-Hung,
>
> I'm having a little trouble following the status of this patchset.
> Would it be possible to repost v3 (or post v4 if that is pending) with:
>
> * A cover le
This patch backports the following two upstream commits to fix MPLS GSO in
ovs datapath. Starting from upstream commit 48d2ab609b6b ("net: mpls: Fixups
for GSO"), the mpls_gso kernel module relies on the fact that
skb_network_header() points to the mpls header and skb_inner_network_header()
points
This series backports the upstream mpls fixes and add a system traffic
test for mpls actions.
v2:
- rebase to master
- seperate the first patch to 3 patches
v3:
- rebase to master
- rename HAVE_MPLS_HDR to MPLS_HEADER_IS_L3
- add comments about the mpls_hdr() backport
v4:
- rebase to
Thanks! Applied to master.
On Fri, Apr 28, 2017 at 04:50:30PM -0700, Jarno Rajahalme wrote:
> Acked-by: Jarno Rajahalme
>
> > On Mar 20, 2017, at 10:10 PM, Ben Pfaff wrote:
> >
> > Until now, the Travis build for Mac OS X has been configured to ignore
> > format specifier warnings. These war
On 28 April 2017 at 17:02, Jarno Rajahalme wrote:
> A commit message would be nice.
>
> Acked-by: Jarno Rajahalme
Thanks, I wrote a commit message and applied to master.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/lis
Add support for specifying rbac "role" when setting remote
connection configuration in the southbound database.
Prior to this change, usage examples included:
ovn-sbctl set-connection ptcp:6642
ovn-sbctl set-connection pssl:6642 \
read-only ptcp: \
Add rbac "roles" and "permissions" tables to ovn southbound
database schema, add support to ovn-northd for managing these
tables.
Signed-off-by: Lance Richardson
---
ovn/northd/ovn-northd.c| 190 +
ovn/ovn-architecture.7.xml | 155 +
Add suport for ovsdb RBAC (role-based access control). This includes:
- Support for "RBAC_Role" table. A db schema containing a table
by this name will enable role-based access controls using
this table for RBAC role configuration.
The "RBAC_Role" table has one row per role, wit
For some applications, it is desirable to have full control of
the common name field in generated certificates. Add a command-line
option to suppress appending " id:" to the user-
specified name.
Signed-off-by: Lance Richardson
---
utilities/ovs-pki.8.in | 9 +
utilities/ovs-pki.in |
Move local db access functions to a new file and make give them
global scope so they can be included in the ovsdb library and used
by other ovsdb library functions.
Signed-off-by: Lance Richardson
---
ovsdb/automake.mk| 4 +-
ovsdb/ovsdb-server.c | 187 +++--
Track authenticated stream peer ID. For SSL connections, the
authenticated ID is the CN (Common Name) field extracted from
the peer's SSL certificate.
Signed-off-by: Lance Richardson
---
lib/stream-provider.h | 1 +
lib/stream-ssl.c | 44
lib/st
This series implements role-based access control infrastructure for
ovsdb-server, and uses that infrastructure to apply role-based access
controls to the OVN_Southbound database. This implementation follows
the outline discussed at:
https://mail.openvswitch.org/pipermail/ovs-dev/2017-March/32
Hi Yi-Hung,
I'm having a little trouble following the status of this patchset.
Would it be possible to repost v3 (or post v4 if that is pending) with:
* A cover letter including:
- An overview of the patchset
- An overview of the changes between v2 and v3
(and v4 is there is one). Changes
91 matches
Mail list logo