public inbox for passt-dev@passt.top
 help / color / mirror / code / Atom feed
From: David Gibson <david@gibson.dropbear.id.au>
To: Stefano Brivio <sbrivio@redhat.com>, passt-dev@passt.top
Cc: David Gibson <david@gibson.dropbear.id.au>
Subject: [PATCH 16/16] inany: Extend inany_from_af to easily set unspecified addresses
Date: Mon, 29 Jan 2024 15:35:57 +1100	[thread overview]
Message-ID: <20240129043557.823451-17-david@gibson.dropbear.id.au> (raw)
In-Reply-To: <20240129043557.823451-1-david@gibson.dropbear.id.au>

inany_from_af() can be used to initialise a union inany_addr from either
an IPv4 or IPv6 address.  If we want to set it to the unspecified IPv4
or IPv6 address we can do that, but need to explicitly pass the correct
address version.  Make this easier by allowing it to interpret a NULL
address as the unspecified address of the appropriate family.

We only have one trivial use for this at present, but it will have more
uses in future.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
---
 inany.h | 10 ++++++++--
 tcp.c   |  2 +-
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/inany.h b/inany.h
index 063545b7..70237821 100644
--- a/inany.h
+++ b/inany.h
@@ -92,11 +92,17 @@ static inline void inany_from_af(union inany_addr *aa,
 				 sa_family_t af, const void *addr)
 {
 	if (af == AF_INET6) {
-		aa->a6 = *((struct in6_addr *)addr);
+		if (addr)
+			aa->a6 = *((struct in6_addr *)addr);
+		else
+			aa->a6 = in6addr_any;
 	} else if (af == AF_INET) {
 		memset(&aa->v4mapped.zero, 0, sizeof(aa->v4mapped.zero));
 		memset(&aa->v4mapped.one, 0xff, sizeof(aa->v4mapped.one));
-		aa->v4mapped.a4 = *((struct in_addr *)addr);
+		if (addr)
+			aa->v4mapped.a4 = *((struct in_addr *)addr);
+		else
+			aa->v4mapped.a4 = in4addr_any;
 	} else {
 		/* Not valid to call with other address families */
 		ASSERT(0);
diff --git a/tcp.c b/tcp.c
index 6c9edbe1..1326584e 100644
--- a/tcp.c
+++ b/tcp.c
@@ -883,7 +883,7 @@ static void tcp_rtt_dst_check(const struct tcp_tap_conn *conn,
 	low_rtt_dst[hole++] = conn->faddr;
 	if (hole == LOW_RTT_TABLE_SIZE)
 		hole = 0;
-	inany_from_af(low_rtt_dst + hole, AF_INET6, &in6addr_any);
+	inany_from_af(low_rtt_dst + hole, AF_INET6, NULL);
 #else
 	(void)conn;
 	(void)tinfo;
-- 
@@ -883,7 +883,7 @@ static void tcp_rtt_dst_check(const struct tcp_tap_conn *conn,
 	low_rtt_dst[hole++] = conn->faddr;
 	if (hole == LOW_RTT_TABLE_SIZE)
 		hole = 0;
-	inany_from_af(low_rtt_dst + hole, AF_INET6, &in6addr_any);
+	inany_from_af(low_rtt_dst + hole, AF_INET6, NULL);
 #else
 	(void)conn;
 	(void)tinfo;
-- 
2.43.0


  parent reply	other threads:[~2024-01-29  4:36 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-29  4:35 [PATCH 00/16] More flow table preliminaries: address handling improvements David Gibson
2024-01-29  4:35 ` [PATCH 01/16] treewide: Use sa_family_t for address family variables David Gibson
2024-01-29  4:35 ` [PATCH 02/16] tcp, udp: Don't precompute port remappings in epoll references David Gibson
2024-01-29  4:35 ` [PATCH 03/16] flow: Add helper to determine a flow's protocol David Gibson
2024-01-29  4:35 ` [PATCH 04/16] tcp_splice: Simplify clean up logic David Gibson
2024-01-29  4:35 ` [PATCH 05/16] inany: Helper to test for IPv4 or IPv6 loopback address David Gibson
2024-01-29  4:35 ` [PATCH 06/16] tcp, tcp_splice: Helpers for getting sockets from the pools David Gibson
2024-01-29  4:35 ` [PATCH 07/16] tcp_splice: More specific variable names in new splice path David Gibson
2024-01-29  4:35 ` [PATCH 08/16] tcp_splice: Fix incorrect parameter comment for tcp_splice_connect() David Gibson
2024-01-29  4:35 ` [PATCH 09/16] tcp_splice: Merge tcp_splice_new() into its caller David Gibson
2024-01-29  4:35 ` [PATCH 10/16] tcp_splice: Improve error reporting on connect path David Gibson
2024-01-29  4:35 ` [PATCH 11/16] inany: Add inany_ntop() helper David Gibson
2024-01-29  4:35 ` [PATCH 12/16] tcp_splice: Improve logic deciding when to splice David Gibson
2024-01-29  4:35 ` [PATCH 13/16] util: Provide global constants for IPv4 loopback and unspecified address David Gibson
2024-01-29  4:35 ` [PATCH 14/16] inany: Introduce union sockaddr_inany David Gibson
2024-01-29  4:35 ` [PATCH 15/16] tcp, tcp_splice: Better construction of IPv4 or IPv6 sockaddrs David Gibson
2024-01-29  4:35 ` David Gibson [this message]
2024-01-29  9:02 ` [PATCH 00/16] More flow table preliminaries: address handling improvements David Gibson

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=20240129043557.823451-17-david@gibson.dropbear.id.au \
    --to=david@gibson.dropbear.id.au \
    --cc=passt-dev@passt.top \
    --cc=sbrivio@redhat.com \
    /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.
Code repositories for project(s) associated with this public inbox

	https://passt.top/passt

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for IMAP folder(s).