Reported-at:
https://mail.openvswitch.org/pipermail/ovs-dev/2021-November/389256.html
Regards,
Vladislav Odintsov
On 23 Nov 2021, at 20:37, Vladislav Odintsov
mailto:odiv...@gmail.com>> wrote:
OVN requested-tnl-key VXLAN checks affect tunnel_key in
datapath_binding record only on datapath
BFD entries are updated and their ->ref field is set to 'true' when
static routes are parsed, within build_lflows(), in the 'en_lflow' I-P
node. Therefore we cannot clean up BFD entries in 'en_northd' which is
an input of 'en_lflow'.
To fix this we now move all BFD handling in the 'en_lflow'
Hi, Flavio.
On 11/23/21 15:32, Flavio Leitner wrote:
On Mon, Nov 22, 2021 at 03:46:28PM -0300, Murilo Opsfelder Araújo wrote:
Hi, Ilya Maximets.
On 11/19/21 13:23, Ilya Maximets wrote:
On 11/18/21 22:19, David Christensen wrote:
On 11/18/21 11:56 AM, Murilo Opsfelder Araújo wrote:
On
By default perform memory trimming 30 seconds after the lflow cache
utilization has dropped. Otherwise, idle ovn-controller processes might
wastefully fail to return unused memory to the system. The timeout is
configurable through the 'ovn-trim-timeout-ms' external_id.
Reported-at:
On Mon, Nov 22, 2021 at 03:46:28PM -0300, Murilo Opsfelder Araújo wrote:
> Hi, Ilya Maximets.
>
> On 11/19/21 13:23, Ilya Maximets wrote:
> > On 11/18/21 22:19, David Christensen wrote:
> > >
> > >
> > > On 11/18/21 11:56 AM, Murilo Opsfelder Araújo wrote:
> > > > On 11/16/21 19:31, Ilya
On Tue, Nov 16, 2021 at 09:54:54PM +0100, Ilya Maximets wrote:
> On 10/25/21 16:36, Michael Santana wrote:
> > In the case that a client disables jsonrpc probes the client would fail
> > to detect if the connection to the server has dropped. To workaround
> > such case TCP keepalive is enabled.
>
OVN requested-tnl-key VXLAN checks affect tunnel_key in
datapath_binding record only on datapath tunnel key assigning.
If a dp tnl key was already assigned and VXLAN-enabled chassis
is added after that, the value would never be changed.
The test didn't check the VXLAN-enabled chassis was created
Bleep bloop. Greetings Vladislav Odintsov, 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:
error: sha1 information is lacking or useless
LGTM!
Acked-by: Flavio Fernandes
On Fri, Nov 5, 2021 at 2:24 PM Terry Wilson wrote:
> ovsdb-tool join-cluster requires a remote addr, so the existing
> code that tried to join a cluster without it when there was an
> existing $DB_FILE would fail.
>
> Instead, if we are trying to specifically
OVN requested-tnl-key VXLAN checks affect tunnel_key in
datapath_binding record only on datapath tunnel key assigning.
If a dp tnl key was already assigned and VXLAN-enabled chassis
is added after that, the value would never be changed.
The test didn't check the VXLAN-enabled chassis was created
In ovn-ic enabled setups, which also use VXLAN encapsulation
(for instance, for VTEP-switches) interconnected transit switches
(datapaths) were not configured after the patch from "Fixes" tag.
This was due to the fact that ovn-ic utilises tunnel key for
transit switches > OVN_VXLAN_MIN_MULTICAST.
When a packet requests l4 csum offload, the l4 csum field includes
a pseudo csum instead of a full csum. The pseudo csum doesn't do
the final one's complement, and it must be updated differently
than a full csum.
Signed-off-by: Jun Xiao
---
lib/packets.c | 12
1 file changed, 12
When a packet requests l4 csum offload, the l4 csum field includes
a pseudo csum instead of a full csum. The pseudo csum doesn't do
the final one's complement, and it must be updated differently
than a full csum.
Signed-off-by: Jun Xiao
---
lib/packets.c | 12
1 file changed, 12
LGTM!
Acked-by: Flavio Fernandes
> -- Forwarded message -
> From: Terry Wilson mailto:twil...@redhat.com>>
> Date: Fri, Nov 5, 2021 at 10:14 AM
> Subject: [ovs-dev] [PATCH v2] python: idl: Avoid pre-allocating column
> defaults
> To: mailto:d...@openvswitch.org>>
>
>
>
On 23/11/2021 15:13, David Marchand wrote:
On Thu, Nov 18, 2021 at 4:31 PM Kevin Traynor wrote:
@@ -1583,6 +1584,13 @@ netdev_dpdk_get_xstat_name(struct netdev_dpdk *dev,
uint64_t id)
return dev->rte_xstats_names[id].name;
}
+/* We filter out everything except per rxq/txq basic
On Thu, Nov 18, 2021 at 4:31 PM Kevin Traynor wrote:
> > @@ -1583,6 +1584,13 @@ netdev_dpdk_get_xstat_name(struct netdev_dpdk *dev,
> > uint64_t id)
> > return dev->rte_xstats_names[id].name;
> > }
> >
> > +/* We filter out everything except per rxq/txq basic stats, and dropped,
> > + *
DPIF AVX512 optimisations currently rely on DPDK availability while
they can be used without DPDK.
Besides, checking for availability of some isa only has to be done once
and won't change while a OVS process runs.
Resolve isa availability in constructors by using a simplified query
based on cpuid
OVS commit 066741d9c5ca ("ovsdb-idl: Add memory report function.")
enhanced the IDL to track memory usage. Use it in ovn-controller for
the Southbound DB and local OVS DB IDLs.
Signed-off-by: Dumitru Ceara
---
controller/ovn-controller.c | 2 ++
1 file changed, 2 insertions(+)
diff --git
The DPDK unit test only runs if vfio or igb_uio kernel module is loaded:
on systems with only mlx5, this test is always skipped.
Besides, the test tries to grab the first device listed by dpdk-devbind.py,
regardless of the PCI device status regarding kmod binding.
Remove dependency on this DPDK
net_pcap is not always available in DPDK (like, in a dev
environment when you forgot to install the libpcap-devel).
On the other hand, OVS already has its own way to inject packets into a
bridge. Let's make use of it.
This solution is slower than net/pcap DPDK, so lower the number of
expected
Move EAL logs and commonly ignored logs to a common macro.
Remove obsolete ones (like i40e [1] and timer [2] logs).
Extend regex on hugepage logs since a check on hugepages availability is
already present on OVS side, and as a consequence, we don't care about
the warnings on availability for
On Tue, Nov 23, 2021 at 2:16 PM David Marchand
wrote:
> Did you have a chance to look at my patch?
>
> Now that I tested it with sse4.2 enabled, I can see the "fuzzy" test
> fails because of a timeout: the test expects 100k packet in 30s.
> I wonder why 100k random data is better than 1k like for
It seems that on slow system with high concurrency and cpu contention
time/warp is not accurate enough for the ALB unit tests with the minimum
time/warp that was used to hit an amount of events. This results in some
intermittent test failures.
As those tests are just waiting for a certain amount
The next log line number should be updated to ensure that the
anticipated log has occurred again after more time has passed.
Fixes: a83a406096e9 ("dpif-netdev: Sync PMD ALB state with user commands.")
Signed-off-by: Kevin Traynor
---
tests/alb.at | 1 +
1 file changed, 1 insertion(+)
diff
Hello Ilya,
On Fri, Nov 19, 2021 at 4:45 PM Ilya Maximets wrote:
>
> On 11/18/21 10:16, Maxime Coquelin wrote:
> > Hi David,
> >
> > On 9/27/21 15:57, David Marchand wrote:
> >> net_pcap is not always available in DPDK (like, in a dev
> >> environment when you forgot to install the
Thanks Numan!
regards,
Vladislav Odintsov
> On 23 Nov 2021, at 00:35, Numan Siddique wrote:
>
> On Fri, Nov 19, 2021 at 11:07 AM Vladislav Odintsov
> wrote:
>>
>> v8 -> v9:
>> - Fix documentation errors.
>> - Fix commit messages.
>>
>> v7 -> v8:
>> - Updated manpages with and commit
Thanks for the patch!
Tested-by: Vladislav Odintsov
Regards,
Vladislav Odintsov
> On 22 Nov 2021, at 18:23, Ilya Maximets wrote:
>
> While removing flows, removal itself is deferred, so classifier changes
> performed already from the RCU thread. This way every deferred removal
> triggers
I’ve re-checked and it really stopped growing after a couple of iterations.
Thanks again for the patches.
Regards,
Vladislav Odintsov
> On 23 Nov 2021, at 02:03, Ilya Maximets wrote:
>
> On 11/22/21 20:54, Vladislav Odintsov wrote:
>> Hi Ilya,
>>
>> I’ve tested both patches, the problem with
Modify ci linux build script to use the latest DPDK stable release 19.11.10.
Modify Documentation to use the latest DPDK stable release 19.11.10.
Update NEWS file to reflect the latest DPDK stable release 19.11.10.
FAQ is updated to reflect the latest DPDK for each OVS branch.
Signed-off-by:
Modify ci linux build script to use the latest DPDK stable release 19.11.10.
Modify Documentation to use the latest DPDK stable release 19.11.10.
Update NEWS file to reflect the latest DPDK stable release 19.11.10.
FAQ is updated to reflect the latest DPDK for each OVS branch.
Signed-off-by:
On 11/23/21 3:07 AM, Numan Siddique wrote:
> On Mon, Nov 22, 2021 at 11:03 AM Mark Michelson wrote:
>>
>> Acked-by: Mark Michelson
>
Thanks, Mark!
>
> Thanks Dumitru and Mark.
>
> I applied this patch to the main branch with the below changes as the
> patch needed some
> updates in the
31 matches
Mail list logo