Linux Netfilter development
 help / color / mirror / Atom feed
* [PATCH net-next] selftests: netfilter: nft_queue.sh: fix warnings with socat 1.8.0.0
@ 2024-11-04 14:28 Florian Westphal
  2024-11-05 10:41 ` Antonio Ojea
  2024-11-06  2:20 ` patchwork-bot+netdevbpf
  0 siblings, 2 replies; 3+ messages in thread
From: Florian Westphal @ 2024-11-04 14:28 UTC (permalink / raw)
  To: netdev; +Cc: netfilter-devel, Florian Westphal

Updated to a more recent socat release and saw this:
 socat E xioopen_ipdgram_listen(): unknown address family 0
 socat W address is opened in read-write mode but only supports read-only

First error is avoided via pf=ipv4 option, second one via -u
(unidirectional) mode.

Signed-off-by: Florian Westphal <fw@strlen.de>
---
 tools/testing/selftests/net/netfilter/nft_queue.sh | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/tools/testing/selftests/net/netfilter/nft_queue.sh b/tools/testing/selftests/net/netfilter/nft_queue.sh
index a9d109fcc15c..785e3875a6da 100755
--- a/tools/testing/selftests/net/netfilter/nft_queue.sh
+++ b/tools/testing/selftests/net/netfilter/nft_queue.sh
@@ -512,10 +512,10 @@ EOF
 	:> "$TMPFILE1"
 	:> "$TMPFILE2"
 
-	timeout 10 ip netns exec "$ns2" socat UDP-LISTEN:12345,fork OPEN:"$TMPFILE1",trunc &
+	timeout 10 ip netns exec "$ns2" socat UDP-LISTEN:12345,fork,pf=ipv4 OPEN:"$TMPFILE1",trunc &
 	local rpid1=$!
 
-	timeout 10 ip netns exec "$ns3" socat UDP-LISTEN:12345,fork OPEN:"$TMPFILE2",trunc &
+	timeout 10 ip netns exec "$ns3" socat UDP-LISTEN:12345,fork,pf=ipv4 OPEN:"$TMPFILE2",trunc &
 	local rpid2=$!
 
 	ip netns exec "$nsrouter" ./nf_queue -q 12 -d 1000 &
@@ -528,8 +528,8 @@ EOF
 	# Send two packets, one should end up in ns1, other in ns2.
 	# This is because nfqueue will delay packet for long enough so that
 	# second packet will not find existing conntrack entry.
-	echo "Packet 1" | ip netns exec "$ns1" socat STDIN UDP-DATAGRAM:10.6.6.6:12345,bind=0.0.0.0:55221
-	echo "Packet 2" | ip netns exec "$ns1" socat STDIN UDP-DATAGRAM:10.6.6.6:12345,bind=0.0.0.0:55221
+	echo "Packet 1" | ip netns exec "$ns1" socat -u STDIN UDP-DATAGRAM:10.6.6.6:12345,bind=0.0.0.0:55221
+	echo "Packet 2" | ip netns exec "$ns1" socat -u STDIN UDP-DATAGRAM:10.6.6.6:12345,bind=0.0.0.0:55221
 
 	busywait 10000 output_files_written "$TMPFILE1" "$TMPFILE2"
 
-- 
2.45.2


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH net-next] selftests: netfilter: nft_queue.sh: fix warnings with socat 1.8.0.0
  2024-11-04 14:28 [PATCH net-next] selftests: netfilter: nft_queue.sh: fix warnings with socat 1.8.0.0 Florian Westphal
@ 2024-11-05 10:41 ` Antonio Ojea
  2024-11-06  2:20 ` patchwork-bot+netdevbpf
  1 sibling, 0 replies; 3+ messages in thread
From: Antonio Ojea @ 2024-11-05 10:41 UTC (permalink / raw)
  To: Florian Westphal; +Cc: netdev, netfilter-devel

On Mon, 4 Nov 2024 at 14:30, Florian Westphal <fw@strlen.de> wrote:
>
> Updated to a more recent socat release and saw this:
>  socat E xioopen_ipdgram_listen(): unknown address family 0
>  socat W address is opened in read-write mode but only supports read-only
>
> First error is avoided via pf=ipv4 option, second one via -u
> (unidirectional) mode.
>
>

Yeah, I hit that in  7e37e0e , I ended adding the "-4" and "-6" flags
because it was easier to me to reuse the test code, but this LGTM

> # socat 1.8.0 has a bug that requires to specify the IP family to bind (fixed in 1.8.0.1)

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH net-next] selftests: netfilter: nft_queue.sh: fix warnings with socat 1.8.0.0
  2024-11-04 14:28 [PATCH net-next] selftests: netfilter: nft_queue.sh: fix warnings with socat 1.8.0.0 Florian Westphal
  2024-11-05 10:41 ` Antonio Ojea
@ 2024-11-06  2:20 ` patchwork-bot+netdevbpf
  1 sibling, 0 replies; 3+ messages in thread
From: patchwork-bot+netdevbpf @ 2024-11-06  2:20 UTC (permalink / raw)
  To: Florian Westphal; +Cc: netdev, netfilter-devel

Hello:

This patch was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Mon,  4 Nov 2024 15:28:18 +0100 you wrote:
> Updated to a more recent socat release and saw this:
>  socat E xioopen_ipdgram_listen(): unknown address family 0
>  socat W address is opened in read-write mode but only supports read-only
> 
> First error is avoided via pf=ipv4 option, second one via -u
> (unidirectional) mode.
> 
> [...]

Here is the summary with links:
  - [net-next] selftests: netfilter: nft_queue.sh: fix warnings with socat 1.8.0.0
    https://git.kernel.org/netdev/net-next/c/a84e8c05f583

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-11-06  2:20 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-11-04 14:28 [PATCH net-next] selftests: netfilter: nft_queue.sh: fix warnings with socat 1.8.0.0 Florian Westphal
2024-11-05 10:41 ` Antonio Ojea
2024-11-06  2:20 ` patchwork-bot+netdevbpf

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox