[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.12 1/2] net: Remove the legacy "-net chann
From: |
Samuel Thibault |
Subject: |
Re: [Qemu-devel] [PATCH for-2.12 1/2] net: Remove the legacy "-net channel" parameter |
Date: |
Mon, 11 Dec 2017 22:53:13 +0100 |
User-agent: |
NeoMutt/20170113 (1.7.2) |
Thomas Huth, on jeu. 07 déc. 2017 19:02:34 +0100, wrote:
> It has never been documented, so hardly anybody knows about this
> parameter, and it is marked as deprecated since QEMU v2.6.
> Time to let it go now.
>
> Signed-off-by: Thomas Huth <address@hidden>
Reviewed-by: Samuel Thibault <address@hidden>
> ---
> include/net/slirp.h | 2 --
> net/net.c | 7 -------
> net/slirp.c | 34 ----------------------------------
> qemu-doc.texi | 5 -----
> 4 files changed, 48 deletions(-)
>
> diff --git a/include/net/slirp.h b/include/net/slirp.h
> index 64b795c..0c98e46 100644
> --- a/include/net/slirp.h
> +++ b/include/net/slirp.h
> @@ -36,8 +36,6 @@ void hmp_hostfwd_remove(Monitor *mon, const QDict *qdict);
>
> int net_slirp_redir(const char *redir_str);
>
> -int net_slirp_parse_legacy(QemuOptsList *opts_list, const char *optarg, int
> *ret);
> -
> int net_slirp_smb(const char *exported_dir);
>
> void hmp_info_usernet(Monitor *mon, const QDict *qdict);
> diff --git a/net/net.c b/net/net.c
> index 39ef546..7425857 100644
> --- a/net/net.c
> +++ b/net/net.c
> @@ -1565,13 +1565,6 @@ int net_init_clients(void)
>
> int net_client_parse(QemuOptsList *opts_list, const char *optarg)
> {
> -#if defined(CONFIG_SLIRP)
> - int ret;
> - if (net_slirp_parse_legacy(opts_list, optarg, &ret)) {
> - return ret;
> - }
> -#endif
> -
> if (!qemu_opts_parse_noisily(opts_list, optarg, true)) {
> return -1;
> }
> diff --git a/net/slirp.c b/net/slirp.c
> index 318a26e..cb8ca23 100644
> --- a/net/slirp.c
> +++ b/net/slirp.c
> @@ -956,37 +956,3 @@ int net_init_slirp(const Netdev *netdev, const char
> *name,
>
> return ret;
> }
> -
> -int net_slirp_parse_legacy(QemuOptsList *opts_list, const char *optarg, int
> *ret)
> -{
> - if (strcmp(opts_list->name, "net") != 0 ||
> - strncmp(optarg, "channel,", strlen("channel,")) != 0) {
> - return 0;
> - }
> -
> - error_report("The '-net channel' option is deprecated. "
> - "Please use '-netdev user,guestfwd=...' instead.");
> -
> - /* handle legacy -net channel,port:chr */
> - optarg += strlen("channel,");
> -
> - if (QTAILQ_EMPTY(&slirp_stacks)) {
> - struct slirp_config_str *config;
> -
> - config = g_malloc(sizeof(*config));
> - pstrcpy(config->str, sizeof(config->str), optarg);
> - config->flags = SLIRP_CFG_LEGACY;
> - config->next = slirp_configs;
> - slirp_configs = config;
> - *ret = 0;
> - } else {
> - Error *err = NULL;
> - *ret = slirp_guestfwd(QTAILQ_FIRST(&slirp_stacks), optarg, 1, &err);
> - if (*ret < 0) {
> - error_report_err(err);
> - }
> - }
> -
> - return 1;
> -}
> -
> diff --git a/qemu-doc.texi b/qemu-doc.texi
> index db2351c..982cab5 100644
> --- a/qemu-doc.texi
> +++ b/qemu-doc.texi
> @@ -2459,11 +2459,6 @@ The ``-smb /some/dir'' argument is now a synonym for
> setting
> the ``-netdev user,smb=/some/dir'' argument instead. The new
> syntax allows different settings to be provided per NIC.
>
> address@hidden -net channel (since 2.6.0)
> -
> -The ``--net channel,ARGS'' argument is now a synonym for setting
> -the ``-netdev user,guestfwd=ARGS'' argument instead.
> -
> @subsection -net vlan (since 2.9.0)
>
> The ``-net vlan=NN'' argument is partially replaced with the
> --
> 1.8.3.1
>
--
Samuel
R: Parce que ça renverse bêtement l'ordre naturel de lecture!
Q: Mais pourquoi citer en fin d'article est-il si effroyable?
R: Citer en fin d'article
Q: Quelle est la chose la plus désagréable sur les groupes de news?
- [Qemu-devel] [PATCH for-2.12 0/2] Remove deprecated net parameters, Thomas Huth, 2017/12/07
- [Qemu-devel] [PATCH for-2.12 1/2] net: Remove the legacy "-net channel" parameter, Thomas Huth, 2017/12/07
- Re: [Qemu-devel] [PATCH for-2.12 1/2] net: Remove the legacy "-net channel" parameter,
Samuel Thibault <=
- [Qemu-devel] [PATCH for-2.12 2/2] net: Remove the deprecated -tftp, -bootp, -redir and -smb options, Thomas Huth, 2017/12/07
- Re: [Qemu-devel] [PATCH for-2.12 2/2] net: Remove the deprecated -tftp, -bootp, -redir and -smb options, Samuel Thibault, 2017/12/11
- Re: [Qemu-devel] [PATCH for-2.12 2/2] net: Remove the deprecated -tftp, -bootp, -redir and -smb options, Peter Maydell, 2017/12/11
- Re: [Qemu-devel] [PATCH for-2.12 2/2] net: Remove the deprecated -tftp, -bootp, -redir and -smb options, Thomas Huth, 2017/12/14
- Re: [Qemu-devel] [PATCH for-2.12 2/2] net: Remove the deprecated -tftp, -bootp, -redir and -smb options, Peter Maydell, 2017/12/14
- Re: [Qemu-devel] [PATCH for-2.12 2/2] net: Remove the deprecated -tftp, -bootp, -redir and -smb options, Thomas Huth, 2017/12/14