Thanks, Numan and Mark. I applied this to master and branch-2.10.
On Thu, Sep 20, 2018 at 05:11:36PM -0400, Mark Michelson wrote:
> Thanks for the assist, Numan!
>
> Acked-by: Mark Michelson
>
> On 09/18/2018 01:29 PM, nusid...@redhat.com wrote:
> >From: Numan Siddique
> >
> >The commit
On Thu, Sep 20, 2018 at 12:32:57PM -0700, Guru Shetty wrote:
> On Thu, 20 Sep 2018 at 11:34, Ben Pfaff wrote:
>
> > Since the time that support for IP fragments was introduce, the OVS
> > functions that format flows have used "nw_frag", but the ones that parse
> > flows have expected "ip_frag".
On Thu, Sep 20, 2018 at 02:22:55AM -0700, Gurucharan Shetty wrote:
> The description of 'ip_frag' is now available
> in 'man ovs-fields'
>
> Signed-off-by: Gurucharan Shetty
Acked-by: Ben Pfaff
___
dev mailing list
d...@openvswitch.org
Thanks for the assist, Numan!
Acked-by: Mark Michelson
On 09/18/2018 01:29 PM, nusid...@redhat.com wrote:
From: Numan Siddique
The commit "6f01617442" added the documenation for the DHCPv4 option
252 in the wrong section. This patch fixes it.
Fixes: 6f01617442 ("ovn: Add DHCP support for
Hi Numan,
I understand the code and the use case pretty well here. There's another
open issue from OpenStack regarding SR-IOV:
https://bugzilla.redhat.com/show_bug.cgi?id=1613384
With this patch, this will mean that an external port will be set up to
represent the VF, and a localnet port
LGTM
Acked-by: Mark Michelson
On 09/20/2018 10:46 AM, Lorenzo Bianconi wrote:
Add CT_LB action to ovn-trace utility in order to fix the
following ovn-trace error if a load balancer rule is added to
OVN configuration
ct_next(ct_state=est|trk /* default (use --ct to customize) */) {
The description of 'ip_frag' is now available
in 'man ovs-fields'
Signed-off-by: Gurucharan Shetty
---
utilities/ovs-ofctl.8.in | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/utilities/ovs-ofctl.8.in b/utilities/ovs-ofctl.8.in
index 4850f4a..d6b6d28 100644
---
On Thu, 20 Sep 2018 at 11:34, Ben Pfaff wrote:
> Since the time that support for IP fragments was introduce, the OVS
> functions that format flows have used "nw_frag", but the ones that parse
> flows have expected "ip_frag". Obviously this is a bug and it's a surprise
> that it's gone so long
In post-install in kmod fedora spec file, the variables storing
different parts of kernel version numbers are renamed. The condition
check to run ovs-kmod-manage.sh for RHEL 7.2 and 7.4 uses the older
variables.
Fixes c3570519ecaf (rhel: add 4.4 kernel in kmod build with mulitple
versions,
This patch fixes the path for ovs-kmod-manage.sh script in the
openvswitch-kmod RPM in fedora spec file. Currently the path prefix is
hard coded to /usr/share. Use %{_datadir} instead.
Fixes 22c33c303932 (rhel: support kmod build against mulitple kernel
versions, fedora)
Signed-off-by: Martin Xu
Since the time that support for IP fragments was introduce, the OVS
functions that format flows have used "nw_frag", but the ones that parse
flows have expected "ip_frag". Obviously this is a bug and it's a surprise
that it's gone so long without anyone reporting the problem. This fixes
it and
Bleep bloop. Greetings Harry van Haaren, 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.
checkpatch:
ERROR: Improper whitespace around control block
#79 FILE:
On 9/18/2018 2:39 PM, Yifeng Sun wrote:
The test 'truncate and output to gre tunnel' is broken on certain kernels
where OVS kernel module and upstream GRE module can't co-exist. This
patch creates a test that doesn't depend on upstream GRE module but
provides the same testing.
The replaced test
This allows plugging-in of different subtable hash-lookup-verify
routines, and allows special casing of those functions based on
known context (eg: # of bits set) of the specific subtable.
Signed-off-by: Harry van Haaren
---
lib/dpif-netdev.c | 127 +++---
This commit moves some data-structures to be available
in the dpif-netdev.h header. This allows specific implementations
of the subtable lookup function to include just that header
file, and not require that the code exists in dpif-netdev.c
Signed-off-by: Harry van Haaren
---
lib/dpif-netdev.c
This commit splits the generic hash-lookup-verify
function to its own file. In doing so, we must move
some MACRO definitions to dpif-netdev.h
Signed-off-by: Harry van Haaren
---
I've checkpatched and see some "Whitespace around Macros" errors,
however the current code more readable than the
On 09/20/2018 07:32 AM, YueHaibing wrote:
> The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
> which is a typedef for an enum type, so make sure the implementation in
> this driver has returns 'netdev_tx_t' value, and change the function
> return type to netdev_tx_t.
>
From: YueHaibing
Date: Thu, 20 Sep 2018 20:32:44 +0800
> The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
> which is a typedef for an enum type, so make sure the implementation in
> this driver has returns 'netdev_tx_t' value, and change the function
> return type to
Thanks! I applied this on master and branch-2.10.
Alin.
> -Mesaj original-
> De la: ovs-dev-boun...@openvswitch.org boun...@openvswitch.org> În numele Anand Kumar
> Trimis: Thursday, September 20, 2018 5:30 AM
> Către: Alin Gabriel Serdean ; d...@openvswitch.org
> Subiect: Re: [ovs-dev]
> -Original Message-
> From: Stephen Hemminger
> Sent: Thursday, September 20, 2018 10:44 AM
> To: YueHaibing
> Cc: da...@davemloft.net; dmitry.tarnya...@lockless.no;
> w...@grandegger.com; m...@pengutronix.de; michal.si...@xilinx.com;
> hswee...@visionengravers.com;
On Tue, 18 Sep 2018 at 01:25, Ben Pfaff wrote:
> On Mon, Sep 17, 2018 at 02:54:10PM +0530, nusid...@redhat.com wrote:
> > From: Numan Siddique
> >
> > When a child vlan interface is created inside a VM, the below kernel
> message
> > is seen and IPv6 doesn't work on that interface.
> >
> > [
Add CT_LB action to ovn-trace utility in order to fix the
following ovn-trace error if a load balancer rule is added to
OVN configuration
ct_next(ct_state=est|trk /* default (use --ct to customize) */) {
*** ct_lb action not implemented;
};
Add '--lb_dst' option in order to specify
On Thu, 20 Sep 2018 20:33:01 +0800
YueHaibing wrote:
> The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
> which is a typedef for an enum type, so make sure the implementation in
> this driver has returns 'netdev_tx_t' value, and change the function
> return type to
> -Original Message-
> From: YueHaibing
> Sent: Thursday, September 20, 2018 8:33 AM
> To: da...@davemloft.net; dmitry.tarnya...@lockless.no;
> w...@grandegger.com; m...@pengutronix.de; michal.si...@xilinx.com;
> hswee...@visionengravers.com; madalin.bu...@nxp.com;
>
On Thu, Sep 20, 2018 at 08:32:57PM +0800, YueHaibing wrote:
> The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
> which is a typedef for an enum type, so make sure the implementation in
> this driver has returns 'netdev_tx_t' value, and change the function
> return type to
On Wed, Sep 19, 2018 at 02:47:03PM +0200, Matteo Croce wrote:
> When using the kernel datapath, OVS allocates a pool of sockets to handle
> netlink events. The number of sockets is: ports * n-handler-threads, where
> n-handler-threads is user configurable and defaults to 3/4*number of cores.
>
>
0-day Robot writes:
> Bleep bloop. Greetings YueHaibing, 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.
Whoops - sorry for the noise. I will add patches to netdev to the list
of
On Tue, Sep 18, 2018 at 12:03:11PM +0200, Simon Horman wrote:
> On Tue, Sep 18, 2018 at 09:36:18AM +0100, Pieter Jansen van Vuuren wrote:
> > This series adds match and action support for TC offload of
> > geneve tunnel with options.
> >
> > Pieter Jansen van Vuuren (2):
> > lib/tc: add geneve
Bleep bloop. Greetings YueHaibing, 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:
fatal: sha1 information is lacking or useless
(drivers/net/ethernet/micrel/ks8695net.c).
Repository
The MAC address of the default gateway (the local dvr gateway) ages out of ovs
and then craziness happens. DVR then breaks and packets egress through the L3
agents. If it egresses though the active L3 bad stuff happens. The active L3
either generates "no route to host" or forwards the packet
Hi Ophir,
I see no significant code changes. At least there are no changes
related to my comments for v2.
Best regards, Ilya Maximets.
> 1. Enable compilation and linkage with dpdk 18.08.0
> The following dpdk commits which were introduced after dpdk 17.11.x
> require OVS updates to accommodate
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
> Hi Lorenzo,
>
Hi Mark,
thx for the review
> I only had one finding on this, which I mentioned below. My only other
> concern is that this could use some documentation. If you can document this
> in the ovn-trace manpage, I think we'll be good.
>
ack, will do in v2
> On 09/18/2018 11:27 AM,
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, but the implementation in this
driver returns an 'int'.
Found by coccinelle.
Signed-off-by: YueHaibing
---
drivers/net/ethernet/faraday/ftgmac100.c | 4 ++--
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing
---
Forgot to mention that I'm using OVS 2.7.2.
Thanks,
Arun.
On Thu, Sep 20, 2018 at 5:08 PM Arunkumar Rg wrote:
> Hi,
>
> I have a ovsdb-client using ovsdb-idl to get updates from ovsdb-server. In
> my client, I track(ovsdb_idl_track_add_column) column of a specific table
> in the DB.
>
> The
Hi,
I have a ovsdb-client using ovsdb-idl to get updates from ovsdb-server. In
my client, I track(ovsdb_idl_track_add_column) column of a specific table
in the DB.
The issue I'm facing is - in the client, the sequence of updates that I get
is different from the sequence of updates that actually
I applied the patch on master and branch-2.10.
Alin.
> -Mesaj original-
> De la: ovs-dev-boun...@openvswitch.org boun...@openvswitch.org> În numele Alin Gabriel Serdean
> Trimis: Thursday, September 20, 2018 1:23 AM
> Către: Anand Kumar
> Cc: d...@openvswitch.org
> Subiect: Re:
Minutes for 19th September.
Short meeting. Moving back to monthly.
Next meeting 17th October 2018 16:00 UTC.
Attendees: Aaron, Eelco, Ian, Frikkie, Pieter, Kevin.
===
GENERAL
===
- DPDK userspace
-- Some talks related to OVS. Most slides are here
https://dpdkuserspace2018.sched.com/
-
Cher(e) client(e)Nous tenons à vous informer que de nouveaux messages
sont disponibles dans votre espace client en ligne.
Allez à mon espace client.
NB: ce message est automatique merci de ne pas y répondre.Cordialement.Votre
Service clients.
58 matches
Mail list logo