From: Ilya Maximets <i.maximets@ovn.org>
To: Minxi Hou <houminxi@gmail.com>, netdev@vger.kernel.org
Cc: aconole@redhat.com, echaudro@redhat.com, davem@davemloft.net,
edumazet@google.com, kuba@kernel.org, pabeni@redhat.com,
horms@kernel.org, shuah@kernel.org, dev@openvswitch.org,
linux-kselftest@vger.kernel.org, i.maximets@ovn.org
Subject: Re: [PATCH net-next v4 0/2] selftests/net/openvswitch: SCTP tests
Date: Wed, 9 Sep 2026 17:03:15 +0200 [thread overview]
Message-ID: <34457ca4-73dc-45ff-90f7-f3fe3c62f2f2@ovn.org> (raw)
In-Reply-To: <20260909122521.2687193-1-houminxi@gmail.com>
On 9/9/26 2:25 PM, Minxi Hou wrote:
> This series extends the Open vSwitch SCTP selftests: patch 1 adds the
> IPv6 flow-key test (mirroring the merged IPv4 test), and patch 2 adds an
> SCTP NAT test exercising port translation across conntrack.
>
> v3 -> v4:
> - test_sctp_connect_v6 now also asserts that installing a keyless
> ipv6(proto=132) flow is refused with EINVAL, pinning the reject side
> of the match_validate() sctp() key requirement (pointed out in
> review: only the accept side was exercised, so a regression dropping
> the requirement would have passed unnoticed). The refusal is checked
> to be errno 22 specifically, not a flow-string parse error.
> - Wrap a few over-80-column lines in the NAT test; checkpatch strict
> is now clean for the series.
> - Rebased onto 548b86839f7f.
>
> Signed-off-by: Minxi Hou <houminxi@gmail.com>
>
> Minxi Hou (2):
> selftests/net/openvswitch: add SCTP flow key test over IPv6
> selftests/net/openvswitch: add SCTP NAT test with port translation
>
> .../testing/selftests/net/openvswitch/config | 1 +
> .../selftests/net/openvswitch/openvswitch.sh | 201 ++++++++++++++++++
> 2 files changed, 202 insertions(+)
Beside the Aaron's comment on v3, please, avoid re-naming patches and
the series while sending new versions. This confuses both the automation
and humans trying to follow the history of the change.
Also, keep the full changelog from previous versions, and include links
to lore for the previous versions.
BTW, it seems like due to a volume of patches and the speed you're
re-sending them over and over, email providers started to reject your
submissions again due to spam detection. For example, while I'm on
CC list for this patch set, it didn't reach my inbox so far. So, once
again, please, try not to have too many patches on the list at the same
time and try to avoid re-spinning them too often.
Best regards, Ilya Maximets.
prev parent reply other threads:[~2026-09-09 15:03 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-09 12:25 [PATCH net-next v4 0/2] selftests/net/openvswitch: SCTP tests Minxi Hou
2026-09-09 12:25 ` [PATCH net-next v4 1/2] selftests/net/openvswitch: add SCTP flow key test over IPv6 Minxi Hou
2026-09-10 12:27 ` netdev-bot+sashiko
2026-09-09 12:25 ` [PATCH net-next v4 2/2] selftests/net/openvswitch: add SCTP NAT test with port translation Minxi Hou
2026-09-10 12:27 ` netdev-bot+sashiko
2026-09-09 15:03 ` Ilya Maximets [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=34457ca4-73dc-45ff-90f7-f3fe3c62f2f2@ovn.org \
--to=i.maximets@ovn.org \
--cc=aconole@redhat.com \
--cc=davem@davemloft.net \
--cc=dev@openvswitch.org \
--cc=echaudro@redhat.com \
--cc=edumazet@google.com \
--cc=horms@kernel.org \
--cc=houminxi@gmail.com \
--cc=kuba@kernel.org \
--cc=linux-kselftest@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=shuah@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox