[ovs-dev] [PATCH ovn] RFC ovn-northd: bypass ct for allow ACLs

2021-02-01 Thread Ihar Hrachyshka
NB: For now, the patch is for discussion only, pending performance measurements. For allow ACLs, bypass connection tracking by avoiding setting ct hints for matching traffic. Avoid sending all traffic to ct when a stateful ACL is present. Before the patch, this unnecessarily hit performance when

[ovs-dev] 答复: 答复: [PATCH] netdev-dpdk: fix incorrect shinfo initialization

2021-02-01 Thread 杨燚
Thanks Ilya, net_tap PMD is handling tap device on host side, so it can leverage vnet header to do TSO/GSO, maybe net_pmd authors don't know how to do this, from source code, tap fd isn't enabled vnet header and TSO. -邮件原件- 发件人: Ilya Maximets [mailto:i.maxim...@ovn.org] 发送时间:

Re: [ovs-dev] [PATCH ovn v9 1/7] ovs-sandbox: Print and check results of *ctl commands.

2021-02-01 Thread Ben Pfaff
On Fri, Dec 11, 2020 at 01:36:13PM +0100, Dumitru Ceara wrote: > On 12/2/20 7:50 AM, Ben Pfaff wrote: > > Otherwise when something hangs or goes wrong, it's not obvious what > > happened. > > > > Signed-off-by: Ben Pfaff > > --- > > Looks good to me, thanks! > > Acked-by: Dumitru Ceara >

Re: [ovs-dev] [PATCH] rhel: Add option to enable AF_XDP on rpm package

2021-02-01 Thread Yifeng Sun
LGTM, thanks. Tested-by: Yifeng Sun Reviewed-by: Yifeng Sun On Thu, Jan 21, 2021 at 10:57 AM Yi-Hung Wei wrote: > This patch adds an RPMBUILD_OPT so that user can enable > AF_XDP support in the rpm package by: > > $ make rpm-fedora RPMBUILD_OPT="--with afxdp" > > Signed-off-by: Yi-Hung Wei

Re: [ovs-dev] [PATCH] doc: Add 'faq' about datapath-id

2021-02-01 Thread Ben Pfaff
On Mon, Feb 01, 2021 at 08:15:32PM +0800, Tao YunXiang wrote: > This patch populate faq about datapath-id. > > Author: Tao YunXiang There's no "Author:" tag. We use Git to track commit authorship. I deleted this. > Signed-off-by: Tao YunXiang I edited the answer to be better suited for the

Re: [ovs-dev] [PATCH branch-2.11] python: Add 'six' to list of install requirements

2021-02-01 Thread Ilya Maximets
On 10/27/20 8:07 PM, Thomas Neuman wrote: > From ba3b444d8c182ab7ff2e007c8960e30706089faa Mon Sep 17 00:00:00 2001 > From: Thomas Neuman > Date: Tue, 27 Oct 2020 17:39:27 + > Subject: [PATCH branch-2.11] python: Add 'six' to list of install requirements > > Fixes: 73eb682edb This should be:

[ovs-dev] [PATCH ovn] Documentation: Update release schedule to include 2022.

2021-02-01 Thread Mark Michelson
Signed-off-by: Mark Michelson --- Documentation/internals/release-process.rst | 20 +--- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/Documentation/internals/release-process.rst b/Documentation/internals/release-process.rst index 875e01554..f37c09e51 100644 ---

[ovs-dev] Reminder: OVN 21.03 soft freeze is 5 February

2021-02-01 Thread Mark Michelson
Hi all, I sent an email a week ago announcing the upcoming soft freeze of OVN 21.03 on 5 February. As a reminder, all new feature patches need to be posted by this date in order to be considered for 21.03. After the soft freeze, there is a two week period where patches may be reviewed and

Re: [ovs-dev] 答复: [PATCH] netdev-dpdk: fix incorrect shinfo initialization

2021-02-01 Thread Ilya Maximets
On 10/28/20 1:35 AM, Yi Yang (杨燚)-云服务集团 wrote: > -邮件原件- > 发件人: dev [mailto:ovs-dev-boun...@openvswitch.org] 代表 Flavio Leitner > 发送时间: 2020年10月27日 21:08 > 收件人: Ilya Maximets > 抄送: yang_y...@163.com; ovs-dev@openvswitch.org; olivier.m...@6wind.com > 主题: Re: [ovs-dev] [PATCH] netdev-dpdk:

Re: [ovs-dev] [dpdk-dev] [PATCH v4 00/44] net/virtio: Virtio PMD rework

2021-02-01 Thread Maxime Coquelin
On 2/1/21 2:42 PM, Ilya Maximets wrote: > On 2/1/21 2:16 PM, Maxime Coquelin wrote: >> Hi Ilya, >> >> On 2/1/21 2:03 PM, Ilya Maximets wrote: >>> CC: ovs-dev >>> >>> On 2/1/21 2:00 PM, Ilya Maximets wrote: On 1/26/21 11:15 AM, Maxime Coquelin wrote: > > Only functionnal change in

Re: [ovs-dev] [PATCH] lib/tc: add ICMP type and code match offload

2021-02-01 Thread Simon Horman
On Thu, Jan 28, 2021 at 03:51:08PM +0200, Roi Dayan wrote: > From: Maor Dickman > > Add TC offload support for classifying ICMPv4/6 type and code. > > $ ovs-appctl dpctl/add-flow 'recirc_id(0),in_port(3),eth(),\ > eth_type(0x0800),ipv4(proto=1),icmp(type=9,code=0)' 2 > > $ ovs-appctl

Re: [ovs-dev] [dpdk-dev] [PATCH v4 00/44] net/virtio: Virtio PMD rework

2021-02-01 Thread Maxime Coquelin
Hi Ilya, On 2/1/21 2:03 PM, Ilya Maximets wrote: > CC: ovs-dev > > On 2/1/21 2:00 PM, Ilya Maximets wrote: >> On 1/26/21 11:15 AM, Maxime Coquelin wrote: >>> >>> Only functionnal change in this second part is making the >>> Vhost-user server mode blocking at init time, as long as >>> a client is

Re: [ovs-dev] [PATCH] github: Don't fail the job if 'apt update' failed.

2021-02-01 Thread Aaron Conole
Ilya Maximets writes: > On 2/1/21 3:08 PM, Aaron Conole wrote: >> Ilya Maximets writes: >> >>> Some repositories that are enabled in GHA are not stable and lead >>> to 'apt update' failures: >>> >>> E: The repository >>> 'https://apt.postgresql.org/pub/repos/apt bionic-pgdg Release' >>>

Re: [ovs-dev] [PATCH] vswitchd.conf.db.5: Fix supported IPsec tunnels

2021-02-01 Thread Ilya Maximets
On 1/15/21 3:29 PM, Mark Gray wrote: > 'ovs-monitor-ipsec' does not support 'ip6gre' tunnels. > > Fixes: 22c5eafb6efa ("ipsec: reintroduce IPsec support for tunneling") > Signed-off-by: Mark Gray > --- > vswitchd/vswitch.xml | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff

Re: [ovs-dev] [PATCH] github: Don't fail the job if 'apt update' failed.

2021-02-01 Thread Ilya Maximets
On 2/1/21 12:56 PM, Ilya Maximets wrote: > Some repositories that are enabled in GHA are not stable and lead > to 'apt update' failures: > > E: The repository > 'https://apt.postgresql.org/pub/repos/apt bionic-pgdg Release' > no longer has a Release file. > > This causes the job

Re: [ovs-dev] [PATCH] github: Don't fail the job if 'apt update' failed.

2021-02-01 Thread Ilya Maximets
On 2/1/21 3:08 PM, Aaron Conole wrote: > Ilya Maximets writes: > >> Some repositories that are enabled in GHA are not stable and lead >> to 'apt update' failures: >> >> E: The repository >> 'https://apt.postgresql.org/pub/repos/apt bionic-pgdg Release' >> no longer has a Release

Re: [ovs-dev] [PATCH v6 0/4] Add PMD auto load balance parameters.

2021-02-01 Thread Kevin Traynor
On 01/02/2021 15:41, Kevin Traynor wrote: > On 21/12/2020 13:10, Kevin Traynor wrote: >> v6: >> - Split patches to add log to existing interval param as a seperate fix first >> - Fix reference in vswitchd.xml >> - Kept Acks as minor changes >> >> v5: >> - Make naming more consistent in

Re: [ovs-dev] [PATCH v6 0/4] Add PMD auto load balance parameters.

2021-02-01 Thread Kevin Traynor
On 21/12/2020 13:10, Kevin Traynor wrote: > v6: > - Split patches to add log to existing interval param as a seperate fix first > - Fix reference in vswitchd.xml > - Kept Acks as minor changes > > v5: > - Make naming more consistent in vswitchd.xml/NEWS/commit msgs > - Added Co-Authored-By > >

Re: [ovs-dev] [PATCH] github: Don't fail the job if 'apt update' failed.

2021-02-01 Thread David Marchand
On Mon, Feb 1, 2021 at 3:22 PM Ilya Maximets wrote: > On 2/1/21 3:12 PM, David Marchand wrote: > > Do you want to send the fix for DPDK? > > Sorry, I have no much time for it right now. And since this blocks CI, > this needs to be fixed ASAP, so it's better if you or someone else will > prepare

Re: [ovs-dev] [PATCH] github: Don't fail the job if 'apt update' failed.

2021-02-01 Thread Ilya Maximets
On 2/1/21 3:12 PM, David Marchand wrote: > On Mon, Feb 1, 2021 at 12:56 PM Ilya Maximets wrote: >> >> Some repositories that are enabled in GHA are not stable and lead >> to 'apt update' failures: >> >> E: The repository >> 'https://apt.postgresql.org/pub/repos/apt bionic-pgdg Release' >>

Re: [ovs-dev] [PATCH] github: Don't fail the job if 'apt update' failed.

2021-02-01 Thread David Marchand
On Mon, Feb 1, 2021 at 12:56 PM Ilya Maximets wrote: > > Some repositories that are enabled in GHA are not stable and lead > to 'apt update' failures: > > E: The repository > 'https://apt.postgresql.org/pub/repos/apt bionic-pgdg Release' > no longer has a Release file. > > This causes

Re: [ovs-dev] [PATCH] github: Don't fail the job if 'apt update' failed.

2021-02-01 Thread Aaron Conole
Ilya Maximets writes: > Some repositories that are enabled in GHA are not stable and lead > to 'apt update' failures: > > E: The repository > 'https://apt.postgresql.org/pub/repos/apt bionic-pgdg Release' > no longer has a Release file. > > This causes the job failure. > In most

Re: [ovs-dev] [dpdk-dev] [PATCH v4 00/44] net/virtio: Virtio PMD rework

2021-02-01 Thread Ilya Maximets
On 2/1/21 2:16 PM, Maxime Coquelin wrote: > Hi Ilya, > > On 2/1/21 2:03 PM, Ilya Maximets wrote: >> CC: ovs-dev >> >> On 2/1/21 2:00 PM, Ilya Maximets wrote: >>> On 1/26/21 11:15 AM, Maxime Coquelin wrote: Only functionnal change in this second part is making the Vhost-user server

Re: [ovs-dev] [PATCH RESEND] dpif-netlink: fix using uninitialized info.tc_modify_flow_deleted in out label

2021-02-01 Thread 0-day Robot
Bleep bloop. Greetings wangyunjian, 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: WARNING: Unexpected sign-offs from developers who are not authors or co-authors or committers:

Re: [ovs-dev] [PATCH] doc: Add 'faq' about datapath-id

2021-02-01 Thread 0-day Robot
Bleep bloop. Greetings Tao YunXiang, 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: WARNING: Line has trailing whitespace #23 FILE: Documentation/faq/configuration.rst:285: Q:

Re: [ovs-dev] [dpdk-dev] [PATCH v4 00/44] net/virtio: Virtio PMD rework

2021-02-01 Thread Ilya Maximets
CC: ovs-dev On 2/1/21 2:00 PM, Ilya Maximets wrote: > On 1/26/21 11:15 AM, Maxime Coquelin wrote: >> >> Only functionnal change in this second part is making the >> Vhost-user server mode blocking at init time, as long as >> a client is not connected. The goal of this change is to >> make the

[ovs-dev] [PATCH RESEND] dpif-netlink: fix using uninitialized info.tc_modify_flow_deleted in out label

2021-02-01 Thread wangyunjian
From: Yunjian Wang Before info.tc_modify_flow_deleted is assigned a value, error processing of other statements goes to the out label. In the out label, the uninitialized variant is used for condition determination, which may cause uncertain behavior. Fixes: 65b84d4a32bd ("dpif-netlink: avoid

[ovs-dev] [PATCH] doc: Add 'faq' about datapath-id

2021-02-01 Thread Tao YunXiang
This patch populate faq about datapath-id. Author: Tao YunXiang Signed-off-by: Tao YunXiang --- Documentation/faq/configuration.rst | 21 + 1 file changed, 21 insertions(+) diff --git a/Documentation/faq/configuration.rst b/Documentation/faq/configuration.rst index

[ovs-dev] [PATCH v2] ovs-lib: Remove second parameter for stop_daemon

2021-02-01 Thread wangyunjian
From: Yunjian Wang The stop_daemon no longer takes second argument after commit f3e24610ea18 ("Remove OVN."). Fixes: f3e24610ea18 ("Remove OVN.") Signed-off-by: Yunjian Wang --- utilities/ovs-lib.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utilities/ovs-lib.in

[ovs-dev] [PATCH] github: Don't fail the job if 'apt update' failed.

2021-02-01 Thread Ilya Maximets
Some repositories that are enabled in GHA are not stable and lead to 'apt update' failures: E: The repository 'https://apt.postgresql.org/pub/repos/apt bionic-pgdg Release' no longer has a Release file. This causes the job failure. In most cases we don't really need any packages from

Re: [ovs-dev] [PATCHv2] netdev-afxdp: updated documentation

2021-02-01 Thread Ilya Maximets
On 12/7/20 9:03 PM, Toms Atteka wrote: > If system is missing ethtool, test fails silently. > > Signed-off-by: Toms Atteka > --- > Documentation/intro/install/afxdp.rst | 1 + > 1 file changed, 1 insertion(+) Thanks! Applied to master. Best regards, Ilya Maximets.