[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 19/20] net: vl: Move default_net to vl.c
From: |
Jason Wang |
Subject: |
[Qemu-devel] [PULL 19/20] net: vl: Move default_net to vl.c |
Date: |
Mon, 23 May 2016 10:14:01 +0800 |
From: Eduardo Habkost <address@hidden>
All handling of defaults (default_* variables) is inside vl.c,
move default_net there too, so we can more easily refactor that
code later.
Reviewed-by: Paolo Bonzini <address@hidden>
Signed-off-by: Eduardo Habkost <address@hidden>
Signed-off-by: Jason Wang <address@hidden>
---
include/net/net.h | 1 -
net/net.c | 23 -----------------------
vl.c | 24 +++++++++++++++++++++++-
3 files changed, 23 insertions(+), 25 deletions(-)
diff --git a/include/net/net.h b/include/net/net.h
index 129d46b..422bc3e 100644
--- a/include/net/net.h
+++ b/include/net/net.h
@@ -183,7 +183,6 @@ struct NICInfo {
extern int nb_nics;
extern NICInfo nd_table[MAX_NICS];
-extern int default_net;
extern const char *host_net_devices[];
/* from net.c */
diff --git a/net/net.c b/net/net.c
index 0ad6217..1680b68 100644
--- a/net/net.c
+++ b/net/net.c
@@ -76,8 +76,6 @@ const char *host_net_devices[] = {
NULL,
};
-int default_net = 1;
-
/***********************************************************/
/* network device redirectors */
@@ -1415,18 +1413,6 @@ void net_check_clients(void)
NetClientState *nc;
int i;
- /* Don't warn about the default network setup that you get if
- * no command line -net or -netdev options are specified. There
- * are two cases that we would otherwise complain about:
- * (1) board doesn't support a NIC but the implicit "-net nic"
- * requested one
- * (2) CONFIG_SLIRP not set, in which case the implicit "-net nic"
- * sets up a nic that isn't connected to anything.
- */
- if (default_net) {
- return;
- }
-
net_hub_check_clients();
QTAILQ_FOREACH(nc, &net_clients, next) {
@@ -1483,14 +1469,6 @@ int net_init_clients(void)
{
QemuOptsList *net = qemu_find_opts("net");
- if (default_net) {
- /* if no clients, we use a default config */
- qemu_opts_set(net, NULL, "type", "nic", &error_abort);
-#ifdef CONFIG_SLIRP
- qemu_opts_set(net, NULL, "type", "user", &error_abort);
-#endif
- }
-
net_change_state_entry =
qemu_add_vm_change_state_handler(net_vm_change_state_handler, NULL);
@@ -1521,7 +1499,6 @@ int net_client_parse(QemuOptsList *opts_list, const char
*optarg)
return -1;
}
- default_net = 0;
return 0;
}
diff --git a/vl.c b/vl.c
index a43a3ba..ccd4a4f 100644
--- a/vl.c
+++ b/vl.c
@@ -211,6 +211,7 @@ static int default_floppy = 1;
static int default_cdrom = 1;
static int default_sdcard = 1;
static int default_vga = 1;
+static int default_net = 1;
static struct {
const char *driver;
@@ -3284,11 +3285,13 @@ int main(int argc, char **argv, char **envp)
fd_bootchk = 0;
break;
case QEMU_OPTION_netdev:
+ default_net = 0;
if (net_client_parse(qemu_find_opts("netdev"), optarg) == -1) {
exit(1);
}
break;
case QEMU_OPTION_net:
+ default_net = 0;
if (net_client_parse(qemu_find_opts("net"), optarg) == -1) {
exit(1);
}
@@ -4405,6 +4408,14 @@ int main(int argc, char **argv, char **envp)
/* clean up network at qemu process termination */
atexit(&net_cleanup);
+ if (default_net) {
+ QemuOptsList *net = qemu_find_opts("net");
+ qemu_opts_set(net, NULL, "type", "nic", &error_abort);
+#ifdef CONFIG_SLIRP
+ qemu_opts_set(net, NULL, "type", "user", &error_abort);
+#endif
+ }
+
if (net_init_clients() < 0) {
exit(1);
}
@@ -4549,7 +4560,18 @@ int main(int argc, char **argv, char **envp)
/* Did we create any drives that we failed to create a device for? */
drive_check_orphaned();
- net_check_clients();
+ /* Don't warn about the default network setup that you get if
+ * no command line -net or -netdev options are specified. There
+ * are two cases that we would otherwise complain about:
+ * (1) board doesn't support a NIC but the implicit "-net nic"
+ * requested one
+ * (2) CONFIG_SLIRP not set, in which case the implicit "-net nic"
+ * sets up a nic that isn't connected to anything.
+ */
+ if (!default_net) {
+ net_check_clients();
+ }
+
if (boot_once) {
qemu_boot_set(boot_once, &error_fatal);
--
2.7.4
- [Qemu-devel] [PULL 09/20] net: Add macros for MAC address tracing, (continued)
- [Qemu-devel] [PULL 09/20] net: Add macros for MAC address tracing, Jason Wang, 2016/05/22
- [Qemu-devel] [PULL 10/20] vmxnet3: Use common MAC address tracing macros, Jason Wang, 2016/05/22
- [Qemu-devel] [PULL 12/20] rtl8139: Move more TCP definitions to common header, Jason Wang, 2016/05/22
- [Qemu-devel] [PULL 14/20] vmxnet3: Use pci_dma_* API instead of cpu_physical_memory_*, Jason Wang, 2016/05/22
- [Qemu-devel] [PULL 11/20] net_pkt: Name vmxnet3 packet abstractions more generic, Jason Wang, 2016/05/22
- [Qemu-devel] [PULL 13/20] net_pkt: Extend packet abstraction as required by e1000e functionality, Jason Wang, 2016/05/22
- [Qemu-devel] [PULL 15/20] e1000_regs: Add definitions for Intel 82574-specific bits, Jason Wang, 2016/05/22
- [Qemu-devel] [PULL 16/20] e1000: Move out code that will be reused in e1000e, Jason Wang, 2016/05/22
- [Qemu-devel] [PULL 18/20] e1000e: Introduce qtest for e1000e device, Jason Wang, 2016/05/22
- [Qemu-devel] [PULL 20/20] net/net: Add SocketReadState for reuse codes, Jason Wang, 2016/05/22
- [Qemu-devel] [PULL 19/20] net: vl: Move default_net to vl.c,
Jason Wang <=
- [Qemu-devel] [PULL 17/20] net: Introduce e1000e device emulation, Jason Wang, 2016/05/22
- Re: [Qemu-devel] [PULL 00/20] Net patches, Peter Maydell, 2016/05/23