[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 03/15] net: Improve -net nic error reporting
From: |
Markus Armbruster |
Subject: |
[Qemu-devel] [PATCH v2 03/15] net: Improve -net nic error reporting |
Date: |
Fri, 15 May 2015 13:58:51 +0200 |
When -net nic fails, it first reports a specific error, then a generic
one, like this:
$ qemu-system-x86_64 -net nic,netdev=nonexistent
qemu-system-x86_64: -net nic,netdev=nonexistent: netdev 'nonexistent' not
found
qemu-system-x86_64: -net nic,netdev=nonexistent: Device 'nic' could not be
initialized
Convert net_init_nic() to Error to get rid of the unwanted second
error message.
While there, tidy up an Overcapitalized Error Message.
Signed-off-by: Markus Armbruster <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
---
net/net.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/net/net.c b/net/net.c
index 3295741..0fe5b8b 100644
--- a/net/net.c
+++ b/net/net.c
@@ -742,7 +742,6 @@ int qemu_find_nic_model(NICInfo *nd, const char * const
*models,
static int net_init_nic(const NetClientOptions *opts, const char *name,
NetClientState *peer, Error **errp)
{
- /* FIXME error_setg(errp, ...) on failure */
int idx;
NICInfo *nd;
const NetLegacyNicOptions *nic;
@@ -752,7 +751,7 @@ static int net_init_nic(const NetClientOptions *opts, const
char *name,
idx = nic_get_free_idx();
if (idx == -1 || nb_nics >= MAX_NICS) {
- error_report("Too Many NICs");
+ error_setg(errp, "too many NICs");
return -1;
}
@@ -763,7 +762,7 @@ static int net_init_nic(const NetClientOptions *opts, const
char *name,
if (nic->has_netdev) {
nd->netdev = qemu_find_netdev(nic->netdev);
if (!nd->netdev) {
- error_report("netdev '%s' not found", nic->netdev);
+ error_setg(errp, "netdev '%s' not found", nic->netdev);
return -1;
}
} else {
@@ -780,19 +779,20 @@ static int net_init_nic(const NetClientOptions *opts,
const char *name,
if (nic->has_macaddr &&
net_parse_macaddr(nd->macaddr.a, nic->macaddr) < 0) {
- error_report("invalid syntax for ethernet address");
+ error_setg(errp, "invalid syntax for ethernet address");
return -1;
}
if (nic->has_macaddr &&
is_multicast_ether_addr(nd->macaddr.a)) {
- error_report("NIC cannot have multicast MAC address (odd 1st byte)");
+ error_setg(errp,
+ "NIC cannot have multicast MAC address (odd 1st byte)");
return -1;
}
qemu_macaddr_default_if_unset(&nd->macaddr);
if (nic->has_vectors) {
if (nic->vectors > 0x7ffffff) {
- error_report("invalid # of vectors: %"PRIu32, nic->vectors);
+ error_setg(errp, "invalid # of vectors: %"PRIu32, nic->vectors);
return -1;
}
nd->nvectors = nic->vectors;
--
1.9.3
- [Qemu-devel] [PATCH v2 00/15] net: Improve error reporting, Markus Armbruster, 2015/05/15
- [Qemu-devel] [PATCH v2 01/15] net: Improve error message for -net hubport a bit, Markus Armbruster, 2015/05/15
- [Qemu-devel] [PATCH v2 05/15] tap: net_tap_fd_init() can't fail, drop dead error handling, Markus Armbruster, 2015/05/15
- [Qemu-devel] [PATCH v2 03/15] net: Improve -net nic error reporting,
Markus Armbruster <=
- [Qemu-devel] [PATCH v2 04/15] net/dump: Improve -net/host_net_add dump error reporting, Markus Armbruster, 2015/05/15
- [Qemu-devel] [PATCH v2 14/15] tap: Finish conversion of tap_open() to Error, Markus Armbruster, 2015/05/15
- [Qemu-devel] [PATCH v2 06/15] tap: Improve -netdev/netdev_add/-net/... bridge error reporting, Markus Armbruster, 2015/05/15
- [Qemu-devel] [PATCH v2 02/15] net: Permit incremental conversion of init functions to Error, Markus Armbruster, 2015/05/15
- [Qemu-devel] [PATCH v2 11/15] tap-linux: Convert tap_open() to Error, Markus Armbruster, 2015/05/15
- [Qemu-devel] [PATCH v2 07/15] tap: Convert tap_set_sndbuf() to Error, Markus Armbruster, 2015/05/15
- [Qemu-devel] [PATCH v2 09/15] tap: Convert launch_script() to Error, Markus Armbruster, 2015/05/15
- [Qemu-devel] [PATCH v2 12/15] tap-bsd: Convert tap_open() to Error, Markus Armbruster, 2015/05/15
- [Qemu-devel] [PATCH v2 08/15] tap: Convert net_init_tap_one() to Error, Markus Armbruster, 2015/05/15