diff mbox series

net: Reinstate '-net nic, model=help' output as documented in man page

Message ID 20240709123444.6933-1-dwmw2@infradead.org (mailing list archive)
State New, archived
Headers show
Series net: Reinstate '-net nic, model=help' output as documented in man page | expand

Commit Message

David Woodhouse July 9, 2024, 12:34 p.m. UTC 
From: David Woodhouse <dwmw@amazon.co.uk>

While refactoring the NIC initialization code, I broke '-net nic,model=help'
which no longer outputs a list of available NIC models.

Fixes: 2cdeca04adab ("net: report list of available models according to platform")
Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
---
 net/net.c | 25 ++++++++++++++++++++++---
 1 file changed, 22 insertions(+), 3 deletions(-)

Comments

Michael Tokarev July 9, 2024, 12:46 p.m. UTC | #1 
09.07.2024 15:34, David Woodhouse wrote:
> From: David Woodhouse <dwmw@amazon.co.uk>
> 
> While refactoring the NIC initialization code, I broke '-net nic,model=help'
> which no longer outputs a list of available NIC models.
> 
> Fixes: 2cdeca04adab ("net: report list of available models according to platform")
> Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Cc: qemu-stable@nongnu.org
Reviewed-by: Michael Tokarev <mjt@tls.msk.ru>

Thanks,

/mjt

> ---
>   net/net.c | 25 ++++++++++++++++++++++---
>   1 file changed, 22 insertions(+), 3 deletions(-)
> 
> diff --git a/net/net.c b/net/net.c
> index 6938da05e0..2eb8bc9c0b 100644
> --- a/net/net.c
> +++ b/net/net.c
> @@ -1139,6 +1139,21 @@ NICInfo *qemu_find_nic_info(const char *typename, bool match_default,
>       return NULL;
>   }
>   
> +static bool is_nic_model_help_option(const char *model)
> +{
> +    if (model && is_help_option(model)) {
> +        /*
> +         * Trigger the help output by instantiating the hash table which
> +         * will gather tha available models as they get registered.
> +         */
> +        if (!nic_model_help) {
> +            nic_model_help = g_hash_table_new_full(g_str_hash, g_str_equal,
> +                                                   g_free, NULL);
> +        }
> +        return true;
> +    }
> +    return false;
> +}
>   
>   /* "I have created a device. Please configure it if you can" */
>   bool qemu_configure_nic_device(DeviceState *dev, bool match_default,
> @@ -1722,6 +1737,12 @@ void net_check_clients(void)
>   
>   static int net_init_client(void *dummy, QemuOpts *opts, Error **errp)
>   {
> +    const char *model = qemu_opt_get_del(opts, "model");
> +
> +    if (is_nic_model_help_option(model)) {
> +        return 0;
> +    }
> +
>       return net_client_init(opts, false, errp);
>   }
>   
> @@ -1778,9 +1799,7 @@ static int net_param_nic(void *dummy, QemuOpts *opts, Error **errp)
>       memset(ni, 0, sizeof(*ni));
>       ni->model = qemu_opt_get_del(opts, "model");
>   
> -    if (!nic_model_help && !g_strcmp0(ni->model, "help")) {
> -        nic_model_help = g_hash_table_new_full(g_str_hash, g_str_equal,
> -                                               g_free, NULL);
> +    if (is_nic_model_help_option(ni->model)) {
>           return 0;
>       }
>
David Woodhouse July 24, 2024, 12:01 p.m. UTC | #2 
On Tue, 2024-07-09 at 15:46 +0300, Michael Tokarev wrote:
> 09.07.2024 15:34, David Woodhouse wrote:
> > From: David Woodhouse <dwmw@amazon.co.uk>
> > 
> > While refactoring the NIC initialization code, I broke '-net nic,model=help'
> > which no longer outputs a list of available NIC models.
> > 
> > Fixes: 2cdeca04adab ("net: report list of available models according to platform")
> > Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
> 
> Cc: qemu-stable@nongnu.org
> Reviewed-by: Michael Tokarev <mjt@tls.msk.ru>

Ping?
Jason Wang July 26, 2024, 2:25 a.m. UTC | #3 
On Wed, Jul 24, 2024 at 8:02 PM David Woodhouse <dwmw2@infradead.org> wrote:
>
> On Tue, 2024-07-09 at 15:46 +0300, Michael Tokarev wrote:
> > 09.07.2024 15:34, David Woodhouse wrote:
> > > From: David Woodhouse <dwmw@amazon.co.uk>
> > >
> > > While refactoring the NIC initialization code, I broke '-net nic,model=help'
> > > which no longer outputs a list of available NIC models.
> > >
> > > Fixes: 2cdeca04adab ("net: report list of available models according to platform")
> > > Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
> >
> > Cc: qemu-stable@nongnu.org
> > Reviewed-by: Michael Tokarev <mjt@tls.msk.ru>
>
> Ping?

Queued.

Thanks
diff mbox series

Patch

diff --git a/net/net.c b/net/net.c
index 6938da05e0..2eb8bc9c0b 100644
--- a/net/net.c
+++ b/net/net.c
@@ -1139,6 +1139,21 @@  NICInfo *qemu_find_nic_info(const char *typename, bool match_default,
     return NULL;
 }
 
+static bool is_nic_model_help_option(const char *model)
+{
+    if (model && is_help_option(model)) {
+        /*
+         * Trigger the help output by instantiating the hash table which
+         * will gather tha available models as they get registered.
+         */
+        if (!nic_model_help) {
+            nic_model_help = g_hash_table_new_full(g_str_hash, g_str_equal,
+                                                   g_free, NULL);
+        }
+        return true;
+    }
+    return false;
+}
 
 /* "I have created a device. Please configure it if you can" */
 bool qemu_configure_nic_device(DeviceState *dev, bool match_default,
@@ -1722,6 +1737,12 @@  void net_check_clients(void)
 
 static int net_init_client(void *dummy, QemuOpts *opts, Error **errp)
 {
+    const char *model = qemu_opt_get_del(opts, "model");
+
+    if (is_nic_model_help_option(model)) {
+        return 0;
+    }
+
     return net_client_init(opts, false, errp);
 }
 
@@ -1778,9 +1799,7 @@  static int net_param_nic(void *dummy, QemuOpts *opts, Error **errp)
     memset(ni, 0, sizeof(*ni));
     ni->model = qemu_opt_get_del(opts, "model");
 
-    if (!nic_model_help && !g_strcmp0(ni->model, "help")) {
-        nic_model_help = g_hash_table_new_full(g_str_hash, g_str_equal,
-                                               g_free, NULL);
+    if (is_nic_model_help_option(ni->model)) {
         return 0;
     }