Re: [PATCH v1] net: add per-connection guestfwd unix support

[Samuel Thibault <samuel.thibault@gnu.org>]

Gavin Li, le ven. 06 nov. 2026 23:42:25 -0500, a ecrit:
> For -netdev user,guestfwd=..., add support for unix-per-connection: in
> addition to the currently supported tcp: and cmd:. With
> unix-per-connection:, a new connection is opened to the given unix
> socket.  This is an improvement over the current unix: support, which
> does not support 1:1 guest:host connections.
> 
> Also update the minimum libslirp version to 4.2.0 (released in 2020) as
> it adds support for the slirp_add_unix() function.
> 
> Signed-off-by: Gavin Li <gavin.li@samsara.com>

Acked-by: Samuel Thibault <samuel.thibault@ens-lyon.org>

Samuel

> ---
>  meson.build     | 2 +-
>  net/slirp.c     | 6 ++++++
>  qemu-options.hx | 9 +++++++++
>  3 files changed, 16 insertions(+), 1 deletion(-)
> 
> diff --git a/meson.build b/meson.build
> index d7bcb581c1..6567ede5d9 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -1289,7 +1289,7 @@ endif
>  slirp = not_found
>  if not get_option('slirp').auto() or have_system
>    slirp = dependency('slirp', required: get_option('slirp'),
> -                     method: 'pkg-config')
> +                     method: 'pkg-config', version: '>=4.2.0')
>    # slirp < 4.7 is incompatible with CFI support in QEMU.  This is because
>    # it passes function pointers within libslirp as callbacks for timers.
>    # When using a system-wide shared libslirp, the type information for the
> diff --git a/net/slirp.c b/net/slirp.c
> index 451c0fd1ba..b0b03b2bf5 100644
> --- a/net/slirp.c
> +++ b/net/slirp.c
> @@ -1159,6 +1159,12 @@ static int slirp_guestfwd(SlirpState *s, const char *config_str, Error **errp)
>                         "forwarding rule '%s'", config_str);
>              return -1;
>          }
> +    } else if (g_str_has_prefix(p, "unix-per-connection:")) {
> +        if (slirp_add_unix(s->slirp, &p[20], &server, port) < 0) {
> +            error_setg(errp, "Conflicting/invalid host:port in guest "
> +                       "forwarding rule '%s'", config_str);
> +            return -1;
> +        }
>      } else {
>          Error *err = NULL;
>          /*
> diff --git a/qemu-options.hx b/qemu-options.hx
> index 5fe9d73eaa..062cd39aa3 100644
> --- a/qemu-options.hx
> +++ b/qemu-options.hx
> @@ -3553,6 +3553,15 @@ SRST
>              # and connect the TCP stream to its stdin/stdout
>              |qemu_system| -nic  'user,id=n1,guestfwd=tcp:10.0.2.100:1234-cmd:netcat 10.10.1.1 4321'
>  
> +    ``guestfwd=[tcp]:server:port-unix-per-connection:path``
> +        With ``unix-per-connection:``, a new connection is opened to the
> +        given unix socket per TCP connection made by the guest. The unix socket
> +        need not exist at startup time.
> +
> +        .. parsed-literal::
> +
> +            |qemu_system| -nic user,guestfwd=tcp:10.0.2.100:1234-unix-per-connection:/tmp/socket
> +
>  ``-netdev tap,id=id[,fd=h][,ifname=name][,script=file][,downscript=dfile][,br=bridge][,helper=helper]``
>      Configure a host TAP network backend with ID id.
>  
> 
> base-commit: 71f25c75417d31c214ba2cd26b273c114fc99987
> -- 
> 2.55.0
> 

-- 
Samuel
 PS> Salut ! J'ai un sujet de philo à vous soumettre : "Suffit-il
 PS> d'observer pour connaître" Idées + plan Merçi
 Oui, ya qu'a t'observer pour connaître le fait que tu es une feignasse.
 -+- FF in: Guide du Neuneu d'Usenet - Neuneu fait de la philo -+-