b-idl: Preserve references for deleted rows.")
Reported-at: https://issues.redhat.com/browse/FDP-193
Signed-off-by: Mike Pattrick
---
lib/ovsdb-idl.c | 13 +++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/lib/ovsdb-idl.c b/lib/ovsdb-idl.c
index 634fbb56d..77f1a6db4 10
From: Dumitru Ceara
Add a check for inserting and then deleting rows referenced by weak
references by the same idl client. Verify that insertion and deletion
events are recorded.
Reported-at: https://issues.redhat.com/browse/FDP-193
Signed-off-by: Dumitru Ceara
Co-authored-by: Mike Pattrick
On Tue, Dec 12, 2023 at 7:45 AM Dumitru Ceara wrote:
>
> On 12/12/23 05:04, Mike Pattrick wrote:
> > In the case of a weak reference, clearing all change_seqno's can delete
> > useful information. Instead of clearing all seqno's when removing
> > track_node
From: Dumitru Ceara
Add a check for inserting and then deleting rows referenced by weak
references by the same idl client. Verify that insertion and deletion
events are recorded.
Reported-at: https://issues.redhat.com/browse/FDP-193
Signed-off-by: Dumitru Ceara
Co-authored-by: Mike Pattrick
b-idl: Preserve references for deleted rows.")
Reported-at: https://issues.redhat.com/browse/FDP-193
Signed-off-by: Mike Pattrick
---
v2: Removed a redundant case.
Signed-off-by: Mike Pattrick
---
lib/ovsdb-idl.c | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/l
and geneve tso,packets will drop. Front-end devices can close offload
features by ethtool also.
Signed-off-by: Dexia Li
Signed-off-by: Mike Pattrick
---
lib/dp-packet.c | 41 +++-
lib/dp-packet.h | 216
lib/dpif-netdev.c | 4
as the traffic produced by this test gives
incorrect values in the vnet header.
Signed-off-by: Mike Pattrick
---
lib/dp-packet.h | 39 ---
lib/dpif-netdev-extract-avx512.c | 8 ++
lib/flow.c | 12 +++--
lib/netdev-linux.c
On Fri, Dec 15, 2023 at 8:01 AM Eelco Chaudron wrote:
>
>
>
> On 11 Dec 2023, at 16:39, Mike Pattrick wrote:
>
> > Currently dp_netdev_upcall() resolves checksums on all packets, but this
> > isn't strictly needed. The checksums will be resolved before
> >
and geneve tso,packets will drop. Front-end devices can close offload
features by ethtool also.
Signed-off-by: Dexia Li
Co-authored-by: Mike Pattrick
Signed-off-by: Mike Pattrick
---
v9: Rebased patch
---
lib/dp-packet.c | 41 +++-
lib/dp-packet.h | 216
as the traffic produced by this test gives
incorrect values in the vnet header.
Signed-off-by: Mike Pattrick
---
v10:
- Software TCP checksums now support encapsulated TSO case
- Redundant inner offset code was removed
---
lib/dp-packet.h | 49
On Mon, Dec 18, 2023 at 5:04 AM Eelco Chaudron wrote:
>
>
>
> On 15 Dec 2023, at 19:22, Mike Pattrick wrote:
>
> > On Fri, Dec 15, 2023 at 8:01 AM Eelco Chaudron wrote:
> >>
> >>
> >>
> >> On 11 Dec 2023, at 16:39, Mike Pattrick wrote:
&
On Fri, Dec 15, 2023 at 8:05 PM Ilya Maximets wrote:
>
> On 12/11/23 16:39, Mike Pattrick wrote:
> > Test that netdev-dummy is able to send and recieve segment offloaded
> > packets.
> >
> > Signed-off-by: Mike Pattrick
> >
> > ---
> > v
patch explicitly sets the appropriate flags when the segmentation
flag is removed, to guarantee that packets always end up with correct
checksums.
Signed-off-by: Mike Pattrick
---
v3: Moved logic from ofproto-dpif-upcall to dp-packet
---
lib/dp-packet.h | 16 ++--
1 file change
Test that netdev-dummy is able to send and receive segment offloaded
packets.
Signed-off-by: Mike Pattrick
---
v2: Fix clang build error: mutex needed to access netdev_dummy members
v3: Remove use of tcpdump, hexdump, and otherwise clean up test
---
lib/netdev-dummy.c | 32
These patches apply cleanly on the current master branch for me. I
will try to resubmit.
-M
On Mon, Dec 18, 2023 at 4:21 PM 0-day Robot wrote:
>
> Bleep bloop. Greetings Mike Pattrick, I am a robot and I have tried out your
> patch.
> Thanks for your contribution.
>
>
Test that netdev-dummy is able to send and receive segment offloaded
packets.
Signed-off-by: Mike Pattrick
---
v2: Fix clang build error: mutex needed to access netdev_dummy members
v3: Remove use of tcpdump, hexdump, and otherwise clean up test
Note, somehow the v3 patch reported as corrupted
patch explicitly sets the appropriate flags when the segmentation
flag is removed, to guarantee that packets always end up with correct
checksums.
Signed-off-by: Mike Pattrick
---
v3: Moved logic from ofproto-dpif-upcall to dp-packet
Signed-off-by: Mike Pattrick
---
lib/dp-packet.h
patch explicitly sets the appropriate flags when the segmentation
flag is removed, to guarantee that packets always end up with correct
checksums.
Signed-off-by: Mike Pattrick
---
v3: Moved logic from ofproto-dpif-upcall to dp-packet
Signed-off-by: Mike Pattrick
---
lib/dp-packet.h
Test that netdev-dummy is able to send and receive segment offloaded
packets.
Signed-off-by: Mike Pattrick
---
v2: Fix clang build error: mutex needed to access netdev_dummy members
v3: Remove use of tcpdump, hexdump, and otherwise clean up test
v4: Wrapped long lines in test
---
lib/netdev
and geneve tso,packets will drop. Front-end devices can close offload
features by ethtool also.
Signed-off-by: Dexia Li
Co-authored-by: Mike Pattrick
Signed-off-by: Mike Pattrick
---
v9: Rebased patch
---
lib/dp-packet.c | 41 +++-
lib/dp-packet.h | 216
as the traffic produced by this test gives
incorrect values in the vnet header.
Signed-off-by: Mike Pattrick
---
v11:
- Corrected logic in netdev-linux vnet prepend code
v10:
- Software TCP checksums now support encapsulated TSO case
- Redundant inner offset code was removed
Signed-off-by
citly
> unshared.
>
> Also making ovsdb_datum_subtract() call conditional, so the
> issue can be hit by existing unit tests.
>
> Fixes: 485ac63d10f8 ("ovsdb: Add lazy-copy support for ovsdb_datum objects.")
> Signed-off-by: Il
uot;)
> Fixes: b2712d026eae ("ovsdb: transaction: Use diffs for strong reference
> counting.")
> Signed-off-by: Ilya Maximets
Acked-by: Mike Pattrick
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
nit_empty(&added);
> +ovsdb_datum_init_empty(&removed);
Nit: Why init these here if they will be overwritten in
ovsdb_datum_added_removed and ovsdb_datum_clone? Couldn't this be
included in an else?
Otherwise, looks good!
Acked-b
not live long, we keep them around only while transaction
> is alive, so should not affect memory consumption.
>
> Users for this data will be added in later commits.
>
> Signed-off-by: Ilya Maximets
Acked-by: Mike Pattrick
___
On Sun, Dec 17, 2023 at 9:03 PM Ilya Maximets wrote:
>
> In case the difference between 'old' and 'new' rows is readily
> available, it can be used to construct added/removed datums
> instead. Diffs are typically much smaller than the column
> itself. This change more than doubles the performanc
On Wed, Dec 13, 2023 at 8:05 PM Ilya Maximets wrote:
>
> It's almost impossible to debug test failures without them.
>
> Signed-off-by: Ilya Maximets
Acked-by: Mike Pattrick
___
dev mailing list
d...@openvswitch.org
https://mai
On Wed, Dec 13, 2023 at 8:05 PM Ilya Maximets wrote:
>
> It's easier to analyze failures when the lines that are different
> are shown next to each other.
>
> Signed-off-by: Ilya Maximets
Acked-by: Mike Pattrick
__
adic test failures.
> Should not affect performance in real-world cases as the actual
> outgoing message is still not sorted.
>
> Signed-off-by: Ilya Maximets
Acked-by: Mike Pattrick
___
dev mailing list
d...@openvswitch.org
https://mail.openvs
ff-by: Ilya Maximets
Acked-by: Mike Pattrick
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
On Wed, Dec 13, 2023 at 8:05 PM Ilya Maximets wrote:
>
> It's not a big problem, but it would be nice to ensure that
> the backup database cannot be locally converted.
>
> Fixes: e51879e99b3e ("ovsdb: Make OVSDB backup sever read only")
> Signed-off-by: Ilya M
a database itself. Will be used
> later for running active and backup databases in a single process.
>
> Marking the _Server database as read only is not necessary, because
> modifications of internal databases are not allowed anyway, but it
> doesn't hurt.
>
> Signed-off
On Fri, Jan 5, 2024 at 9:30 AM Ilya Maximets wrote:
>
> On 12/29/23 17:28, Mike Pattrick wrote:
> > On Sun, Dec 17, 2023 at 9:03 PM Ilya Maximets wrote:
> >>
> >> While reassessing weak references the code attempts to collect added
> >> and removed atoms
On Fri, Jan 5, 2024 at 9:40 AM Ilya Maximets wrote:
>
> On 1/1/24 07:37, Mike Pattrick wrote:
> > On Sun, Dec 17, 2023 at 9:03 PM Ilya Maximets wrote:
> >>
> >> In case the difference between 'old' and 'new' rows is readily
> >>
-by: Frode Nordahl
That looks correct to me.
Acked-by: Mike Pattrick
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
a new structure. This
> is done in order for different callers to have their own
> default values and only update them with what is provided
> by the user explicitly. For example, replication and relay
> have different default probe intervals.
>
> Signed-o
> Signed-off-by: Ilya Maximets
Thanks for the updated comment.
Acked-by: Mike Pattrick
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
On Sat, Jan 13, 2024 at 4:21 PM Ilya Maximets wrote:
>
> On 12/19/23 15:47, Mike Pattrick wrote:
> > When OVS needs to fallback on the software TSO implementation to segment
> > a packet, it currently doesn't guarantee that IP and TCP checksum
> > offload flags are
On Sat, Jan 13, 2024 at 4:47 PM Ilya Maximets wrote:
>
> On 12/19/23 15:47, Mike Pattrick wrote:
> > Test that netdev-dummy is able to send and receive segment offloaded
> > packets.
> >
> > Signed-off-by: Mike Pattrick
> > ---
> > v2: Fix clang build
patch explicitly sets the appropriate flags when the segmentation
flag is removed, to guarantee that packets always end up with correct
checksums.
Signed-off-by: Mike Pattrick
---
v3: Moved logic from ofproto-dpif-upcall to dp-packet
v5: Corrected indectionation, and removed a redundant flag from the
Test that netdev-dummy is able to send and receive segment offloaded
packets.
Signed-off-by: Mike Pattrick
---
v2: Fix clang build error: mutex needed to access netdev_dummy members
v3: Remove use of tcpdump, hexdump, and otherwise clean up test
v4: Wrapped long lines in test
v5:
- Implemented
Correct the length field in IPv6 packets when applying software fallback
GSO. Previosuly the field retained an IPv4 header size, which was
incorrect.
Signed-off-by: Mike Pattrick
---
lib/dp-packet-gso.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/dp-packet-gso.c b
Test that netdev-dummy is able to send and receive segment offloaded
packets.
Signed-off-by: Mike Pattrick
---
v2: Fix clang build error: mutex needed to access netdev_dummy members
v3: Remove use of tcpdump, hexdump, and otherwise clean up test
v4: Wrapped long lines in test
v5:
- Implemented
Correct the length field in IPv6 packets when applying software fallback
GSO. Previosuly the field retained an IPv4 header size, which was
incorrect.
Signed-off-by: Mike Pattrick
Fixes: 8b5fe2dc6080 ("userspace: Add Generic Segmentation Offloading.")
---
v6: include ip6 extensions
-
patch explicitly sets the appropriate flags when the segmentation
flag is removed, to guarantee that packets always end up with correct
checksums.
Signed-off-by: Mike Pattrick
Fixes: 8b5fe2dc6080 ("userspace: Add Generic Segmentation Offloading.")
---
v3: Moved logic from ofproto-dpif-upca
On Sat, Jan 13, 2024 at 6:51 PM Ilya Maximets wrote:
>
> On 12/27/23 19:55, Mike Pattrick wrote:
> > This patch enables most of the tunnel tests in the testsuite, and adds a
> > large TCP transfer to a vxlan and geneve test to verify TSO
> > functionality. Some additional
as the traffic produced by this test gives
incorrect values in the vnet header.
Signed-off-by: Mike Pattrick
---
v10:
- Software TCP checksums now support encapsulated TSO case
- Redundant inner offset code was removed
v11:
- Corrected logic in netdev-linux vnet prepend code
v12:
- Submitted
and geneve tso,packets will drop. Front-end devices can close offload
features by ethtool also.
Signed-off-by: Dexia Li
Co-authored-by: Mike Pattrick
Signed-off-by: Mike Pattrick
---
v9: Rebased patch
v12:
- Sent in by Dexia
v13:
- Corrected formatting and comment/function naming consistency
On Wed, Jan 17, 2024 at 5:09 AM Dexia Li wrote:
>
>
>
> On 1/14/24 00:34, Ilya Maximets wrote:
> > On 12/27/23 19:55, Mike Pattrick wrote:
> >> From: Dexia Li
> >>
> >> For userspace datapath, this patch provides vxlan and geneve tunnel tso.
>
n inline function and calling into the full send prepare if it
passes, but that change didn't yield any noticeable performance
improvement over the current solution.
Acked-by: Mike Pattrick
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
led in the single function that is causing all of
the errors.
Signed-off-by: Mike Pattrick
---
include/openvswitch/compiler.h | 11 +++
lib/ovs-rcu.c | 1 +
2 files changed, 12 insertions(+)
diff --git a/include/openvswitch/compiler.h b/include/openvswitch/compiler.h
index
=),protoinfo=(state=)
+tcp,orig=(src=10.1.1.4,dst=20.22.166.15,sport=,dport=),reply=(src=20.22.166.15,dst=10.1.1.4,sport=,dport=),protoinfo=(state=)
Those are github IP's, don't know how github traffic got into the tests.
-M
On Fri, Jan 19, 2024 at 11:13 AM Mike Pattrick wrote:
>
ned-off-by: Mike Pattrick
---
lib/dp-packet.h | 10 --
lib/packets.c | 6 +++---
2 files changed, 11 insertions(+), 5 deletions(-)
diff --git a/lib/dp-packet.h b/lib/dp-packet.h
index 939bec5c8..81b01f331 100644
--- a/lib/dp-packet.h
+++ b/lib/dp-packet.h
@@ -537,7 +537,7 @@ dp_packe
On Mon, Jan 22, 2024 at 1:47 PM Ilya Maximets wrote:
>
> On 1/22/24 18:51, Mike Pattrick wrote:
> > The OVN test suite identified a bug in dp_packet_ol_send_prepare() where
> > a double encapsulated BFD packet would trigger a seg fault. This
> > happened because w
On Mon, Jan 22, 2024 at 4:10 PM Ilya Maximets wrote:
>
> On 1/22/24 21:33, Mike Pattrick wrote:
> > On Mon, Jan 22, 2024 at 1:47 PM Ilya Maximets wrote:
> >>
> >> On 1/22/24 18:51, Mike Pattrick wrote:
> >>> The OVN test suite identified a bug in dp_pac
c6080 ("userspace: Add Generic Segmentation Offloading.")
Fixes: 85bcbbed839a ("userspace: Enable tunnel tests with TSO.")
Reported-by: Dumitru Ceara
Reported-at: https://issues.redhat.com/browse/FDP-300
Signed-off-by: Mike Pattrick
---
lib/dp-packet.h | 4 +++-
1 file ch
On Tue, Jan 23, 2024 at 7:41 AM Ilya Maximets wrote:
>
> On 1/22/24 23:24, Mike Pattrick wrote:
> > On Mon, Jan 22, 2024 at 4:10 PM Ilya Maximets wrote:
> >>
> >> On 1/22/24 21:33, Mike Pattrick wrote:
> >>> On Mon, Jan 22, 2024 at 1:47 PM Ilya Maxim
When sending packets that are flagged as requiring segmentation to an
interface that doens't support this feature, send the packet to the TSO
software fallback instead of dropping it.
Signed-off-by: Mike Pattrick
---
lib/dp-packet-gso.c | 73 +
l
On Wed, Jan 24, 2024 at 1:14 PM Mike Pattrick wrote:
>
> When sending packets that are flagged as requiring segmentation to an
> interface that doens't support this feature, send the packet to the TSO
> software fallback instead of dropping it.
>
> Signed-off-by: Mike Pat
Previously the BFD packet creation code did not appropriately set
offsets or flags. This contributed to issues involving encapsulation and
the TSO code.
Signed-off-by: Mike Pattrick
Fixes: ccc096898c46 ("bfd: Implement Bidirectional Forwarding Detection.")
Signed-off-by: Mike Pattrick
Include inner offsets in functions where l3 and l4 offsets are either
modified or checked.
Signed-off-by: Mike Pattrick
Fixes: 084c8087292c ("userspace: Support VXLAN and GENEVE TSO.")
---
lib/dp-packet.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/lib/dp-
Currently the monitor will call dp_packet_clear() on the dp_packet that
is shared amongst BFD, LLDP, and CFM. However, all of these packets are
created with eth_compose(), which already calls dp_packet_clear().
Signed-off-by: Mike Pattrick
---
ofproto/ofproto-dpif-monitor.c | 3 ---
1 file
le as long as
the flags of the packet are correct.
Fixes: 85bcbbed839a ("userspace: Enable tunnel tests with TSO.")
Signed-off-by: Mike Pattrick
---
lib/dp-packet.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/dp-packet.h b/lib/dp-packet.h
index dceb701e8..8
orted-at: https://issues.redhat.com/browse/FDP-300
Signed-off-by: Mike Pattrick
---
lib/dp-packet.h | 3 +++
lib/packets.c | 3 ---
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/lib/dp-packet.h b/lib/dp-packet.h
index 939bec5c8..dceb701e8 100644
--- a/lib/dp-packet.h
+++ b/lib/dp-pac
le as long as
the flags of the packet are correct.
Fixes: 85bcbbed839a ("userspace: Enable tunnel tests with TSO.")
Signed-off-by: Mike Pattrick
---
v2: Corrected patch subject
---
lib/dp-packet.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/dp-packet.h b/
Previously the BFD packet creation code did not appropriately set
offsets or flags. This contributed to issues involving encapsulation and
the TSO code.
Fixes: ccc096898c46 ("bfd: Implement Bidirectional Forwarding Detection.")
Signed-off-by: Mike Pattrick
---
v2: Corrected formatting
Currently the monitor will call dp_packet_clear() on the dp_packet that
is shared amongst BFD, LLDP, and CFM. However, all of these packets are
created with eth_compose(), which already calls dp_packet_clear().
Signed-off-by: Mike Pattrick
---
ofproto/ofproto-dpif-monitor.c | 3 ---
1 file
Include inner offsets in functions where l3 and l4 offsets are either
modified or checked.
Fixes: 084c8087292c ("userspace: Support VXLAN and GENEVE TSO.")
Signed-off-by: Mike Pattrick
---
v2:
- Prints out new offsets in autovalidator
- Extends resize_l2 change to avx512
Signed-of
On Wed, Jan 31, 2024 at 10:04 AM David Marchand
wrote:
>
> On Tue, Jan 30, 2024 at 11:15 PM Mike Pattrick wrote:
> >
> > Include inner offsets in functions where l3 and l4 offsets are either
> > modified or checked.
> >
> > Fixes: 084c8087292c (&quo
, returning immediately. One notable change in
functionality is not performing the RBAC check in this condition, as no
mutation actually takes place.
Reported-by: Terry Wilson
Reported-at: https://issues.redhat.com/browse/FDP-359
Signed-off-by: Mike Pattrick
---
ovsdb/execution.c | 9
Include inner offsets in functions where l3 and l4 offsets are either
modified or checked.
Fixes: 084c8087292c ("userspace: Support VXLAN and GENEVE TSO.")
Signed-off-by: Mike Pattrick
---
v2:
- Prints out new offsets in autovalidator
- Extends resize_l2 change to avx512
v3:
-
Currently the monitor will call dp_packet_clear() on the dp_packet that
is shared amongst BFD, LLDP, and CFM. However, all of these packets are
created with eth_compose(), which already calls dp_packet_clear().
Reviewed-by: David Marchand
Signed-off-by: Mike Pattrick
---
ofproto/ofproto-dpif
t;bfd: Implement Bidirectional Forwarding Detection.")
Signed-off-by: Mike Pattrick
---
v2: Corrected formatting, and just calculate checksum up front
v3: Extended patch comment
---
lib/bfd.c | 23 +++
1 file changed, 11 insertions(+), 12 deletions(-)
diff --git a/lib/bfd.c b
le as long as
the flags of the packet are correct.
Fixes: 85bcbbed839a ("userspace: Enable tunnel tests with TSO.")
Reviewed-by: David Marchand
Signed-off-by: Mike Pattrick
---
v2: Corrected patch subject
---
lib/dp-packet.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
le as long as
the flags of the packet are correct.
Fixes: 85bcbbed839a ("userspace: Enable tunnel tests with TSO.")
Reviewed-by: David Marchand
Signed-off-by: Mike Pattrick
---
v2: Corrected patch subject
---
lib/dp-packet.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
t;bfd: Implement Bidirectional Forwarding Detection.")
Signed-off-by: Mike Pattrick
---
v2: Corrected formatting, and just calculate checksum up front
v3: Extended patch comment
---
lib/bfd.c | 23 +++
1 file changed, 11 insertions(+), 12 deletions(-)
diff --git a/lib/bfd.c b
Include inner offsets in functions where l3 and l4 offsets are either
modified or checked.
Fixes: 084c8087292c ("userspace: Support VXLAN and GENEVE TSO.")
Signed-off-by: Mike Pattrick
---
v2:
- Prints out new offsets in autovalidator
- Extends resize_l2 change to avx512
v3:
-
Currently the monitor will call dp_packet_clear() on the dp_packet that
is shared amongst BFD, LLDP, and CFM. However, all of these packets are
created with eth_compose(), which already calls dp_packet_clear().
Reviewed-by: David Marchand
Signed-off-by: Mike Pattrick
---
ofproto/ofproto-dpif
When sending packets that are flagged as requiring segmentation to an
interface that doens't support this feature, send the packet to the TSO
software fallback instead of dropping it.
Signed-off-by: Mike Pattrick
---
Note: Previously this patch failed gitlab ci, however, I was not ab
tunnel not checksum offloading properly.
Fixes: 3337e6d91c5b ("userspace: Enable L4 checksum offloading by default.")
Reported-by: David Marchand
Signed-off-by: Mike Pattrick
---
lib/netdev-linux.c | 15 ---
1 file changed, 12 insertions(+), 3 deletions(-)
diff --git a/
h TSO.")
Signed-off-by: Mike Pattrick
---
lib/netdev-linux.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/netdev-linux.c b/lib/netdev-linux.c
index 7a156cc28..51517854b 100644
--- a/lib/netdev-linux.c
+++ b/lib/netdev-linux.c
@@ -7191,8 +7191,8 @@ netdev_li
Previously a gap existed in the tunnel system tests where only ICMP and
TCP traffic was tested. However, the code paths using for UDP traffic is
different then either of those and should also be tested.
Signed-off-by: Mike Pattrick
---
tests/system-traffic.at | 50
On Mon, Feb 12, 2024 at 7:52 AM Ilya Maximets wrote:
>
> On 2/12/24 09:13, Mike Pattrick wrote:
> > When sending packets that are flagged as requiring segmentation to an
> > interface that doens't support this feature, send the packet to the TSO
> > software fa
When sending packets that are flagged as requiring segmentation to an
interface that doens't support this feature, send the packet to the TSO
software fallback instead of dropping it.
Signed-off-by: Mike Pattrick
---
v2: Start the ncat listener before pings, so the socket has a better
chan
h TSO.")
Signed-off-by: Mike Pattrick
---
lib/netdev-linux.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/netdev-linux.c b/lib/netdev-linux.c
index 7a156cc28..51517854b 100644
--- a/lib/netdev-linux.c
+++ b/lib/netdev-linux.c
@@ -7191,8 +7191,8 @@ netdev_li
de an appropriate check for ncat support. That check was
added to these tests.
Signed-off-by: Mike Pattrick
---
v2: Start the ncat listener before pings, so the socket has a better
chance of being ready to accept connections when the ncat client starts.
Signed-off-by: Mike Pattrick
---
tests/s
tunnel not checksum offloading properly.
Fixes: 3337e6d91c5b ("userspace: Enable L4 checksum offloading by default.")
Reported-by: David Marchand
Signed-off-by: Mike Pattrick
---
lib/netdev-linux.c | 15 ---
1 file changed, 12 insertions(+), 3 deletions(-)
diff --git a/
On Mon, Feb 12, 2024 at 10:04 AM Ilya Maximets wrote:
>
> On 2/12/24 15:19, Mike Pattrick wrote:
> > On Mon, Feb 12, 2024 at 7:52 AM Ilya Maximets wrote:
> >>
> >> On 2/12/24 09:13, Mike Pattrick wrote:
> >>> When sending packets that are flagged as re
On Tue, Feb 13, 2024 at 2:42 PM Ilya Maximets wrote:
>
> It's not an executable file.
>
> Signed-off-by: Ilya Maximets
Acked-by: Mike Pattrick
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
cate right after its generation. So, there is no point in
> creating it with restricted permissions in the first place.
>
> Fixes: 99e5e05db37a ("ovs-pki: Create private keys with restricted
> permissions.")
> Signed-off-by: Ilya Maximets
Acked-by: Mike Pattrick
__
Note, this failed github CI with the following error message:
../../tests/testsuite: line 4034: ./atconfig: No such file or directory
This appears to be a false negative.
-M
On Mon, Feb 12, 2024 at 2:50 PM Mike Pattrick wrote:
>
> Previously if an OVS configuration nested multiple lay
On Wed, Feb 14, 2024 at 12:09 PM David Marchand
wrote:
>
> Hello Mike,
>
> On Mon, Feb 12, 2024 at 8:50 PM Mike Pattrick wrote:
> >
> > Previously if an OVS configuration nested multiple layers of UDP tunnels
> > like VXLAN or GENEVE on top of each other through ne
tunnel not checksum offloading properly.
Fixes: 85bcbbed839a ("userspace: Enable tunnel tests with TSO.")
Reported-by: David Marchand
Signed-off-by: Mike Pattrick
---
nb: The first patch was removed from the v2 version of this series,
which was not explicitly a bug fix.
---
lib/netd
tests with TSO.")
Reported-by: David Marchand
Signed-off-by: Mike Pattrick
---
lib/dp-packet.h| 18 +-
lib/netdev-linux.c | 9 +
2 files changed, 22 insertions(+), 5 deletions(-)
diff --git a/lib/dp-packet.h b/lib/dp-packet.h
index 802d3f385..770ddc1b9 100644
--- a/lib/d
de an appropriate check for ncat support. That check was
added to these tests.
Signed-off-by: Mike Pattrick
---
tests/system-traffic.at | 118 +---
1 file changed, 111 insertions(+), 7 deletions(-)
diff --git a/tests/system-traffic.at b/tests/system-traffic.at
VXLAN and GENEVE TSO.")
Reported-by: David Marchand
Signed-off-by: Mike Pattrick
---
lib/dp-packet.h | 19 +--
lib/netdev-native-tnl.c | 10 --
lib/packets.c | 8
3 files changed, 29 insertions(+), 8 deletions(-)
diff --git a/lib/dp-pa
On Thu, Feb 15, 2024 at 4:59 PM Mike Pattrick wrote:
>
> Previously some packets were excluded from the tunnel mark if they
> weren't L4. However, this causes problems with multi encapsulated
> packets like arp.
>
> Due to these flags being set, additional checks a
tunnel not checksum offloading properly.
Fixes: 85bcbbed839a ("userspace: Enable tunnel tests with TSO.")
Reported-by: David Marchand
Signed-off-by: Mike Pattrick
---
nb: The first patch was removed from the v2 version of this series,
which was not explicitly a bug fix.
---
lib/netd
tests with TSO.")
Reported-by: David Marchand
Signed-off-by: Mike Pattrick
---
lib/dp-packet.h| 18 +-
lib/netdev-linux.c | 9 +
2 files changed, 22 insertions(+), 5 deletions(-)
diff --git a/lib/dp-packet.h b/lib/dp-packet.h
index 802d3f385..770ddc1b9 100644
--- a/lib/d
de an appropriate check for ncat support. That check was
added to these tests.
Signed-off-by: Mike Pattrick
---
tests/system-traffic.at | 118 +---
1 file changed, 111 insertions(+), 7 deletions(-)
diff --git a/tests/system-traffic.at b/tests/system-traffic.at
101 - 200 of 782 matches
Mail list logo