diff options
author | Laurent Vivier | 2022-10-21 11:09:07 +0200 |
---|---|---|
committer | Jason Wang | 2022-10-28 07:28:52 +0200 |
commit | d63ef17bfcba1126df6bfcb7bca64c96ac4a8d99 (patch) | |
tree | 5e84e81b619740690bc691f76daa7caa34859070 /net | |
parent | net: introduce convert_host_port() (diff) | |
download | qemu-d63ef17bfcba1126df6bfcb7bca64c96ac4a8d99.tar.gz qemu-d63ef17bfcba1126df6bfcb7bca64c96ac4a8d99.tar.xz qemu-d63ef17bfcba1126df6bfcb7bca64c96ac4a8d99.zip |
net: remove the @errp argument of net_client_inits()
The only caller passes &error_fatal, so use this directly in the function.
It's what we do for -blockdev, -device, and -object.
Suggested-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
Diffstat (limited to 'net')
-rw-r--r-- | net/net.c | 20 |
1 files changed, 7 insertions, 13 deletions
@@ -1566,27 +1566,21 @@ out: return ret; } -int net_init_clients(Error **errp) +void net_init_clients(void) { net_change_state_entry = qemu_add_vm_change_state_handler(net_vm_change_state_handler, NULL); QTAILQ_INIT(&net_clients); - if (qemu_opts_foreach(qemu_find_opts("netdev"), - net_init_netdev, NULL, errp)) { - return -1; - } - - if (qemu_opts_foreach(qemu_find_opts("nic"), net_param_nic, NULL, errp)) { - return -1; - } + qemu_opts_foreach(qemu_find_opts("netdev"), net_init_netdev, NULL, + &error_fatal); - if (qemu_opts_foreach(qemu_find_opts("net"), net_init_client, NULL, errp)) { - return -1; - } + qemu_opts_foreach(qemu_find_opts("nic"), net_param_nic, NULL, + &error_fatal); - return 0; + qemu_opts_foreach(qemu_find_opts("net"), net_init_client, NULL, + &error_fatal); } int net_client_parse(QemuOptsList *opts_list, const char *optarg) |