[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [FOR 0.12 PATCH v3 20/21] default devices: virtio consoles.
From: |
Gerd Hoffmann |
Subject: |
[Qemu-devel] [FOR 0.12 PATCH v3 20/21] default devices: virtio consoles. |
Date: |
Mon, 7 Dec 2009 13:42:52 +0100 |
This patch adds a variable default_virtcon which says whenever a default
virtio console should be added. It is disabled by default, followup
patch will enable it for s390. It is cleared when qemu finds
'-virtiocon', '-device virtio-console-s390' or '-device
virtio-console-pci' on the command line.
Signed-off-by: Gerd Hoffmann <address@hidden>
---
vl.c | 64 ++++++++++++++++++++++++++++++++++------------------------------
1 files changed, 34 insertions(+), 30 deletions(-)
diff --git a/vl.c b/vl.c
index 9949886..298ac73 100644
--- a/vl.c
+++ b/vl.c
@@ -270,6 +270,7 @@ static void *boot_set_opaque;
static int default_serial = 1;
static int default_parallel = 1;
+static int default_virtcon = 1;
static int default_monitor = 1;
static int default_vga = 1;
static int default_drive = 1;
@@ -280,6 +281,8 @@ static struct {
} default_list[] = {
{ .driver = "isa-serial", .flag = &default_serial },
{ .driver = "isa-parallel", .flag = &default_parallel },
+ { .driver = "virtio-console-pci", .flag = &default_virtcon },
+ { .driver = "virtio-console-s390", .flag = &default_virtcon },
{ .driver = "VGA", .flag = &default_vga },
{ .driver = "Cirrus VGA", .flag = &default_vga },
{ .driver = "QEMUware SVGA", .flag = &default_vga },
@@ -4701,10 +4704,11 @@ static void monitor_parse(const char *optarg, const
char *mode)
struct device_config {
enum {
- DEV_USB, /* -usbdevice */
- DEV_BT, /* -bt */
- DEV_SERIAL, /* -serial */
- DEV_PARALLEL, /* -parallel */
+ DEV_USB, /* -usbdevice */
+ DEV_BT, /* -bt */
+ DEV_SERIAL, /* -serial */
+ DEV_PARALLEL, /* -parallel */
+ DEV_VIRTCON, /* -virtioconsole */
} type;
const char *cmdline;
QTAILQ_ENTRY(device_config) next;
@@ -4780,6 +4784,28 @@ static int parallel_parse(const char *devname)
return 0;
}
+static int virtcon_parse(const char *devname)
+{
+ static int index = 0;
+ char label[32];
+
+ if (strcmp(devname, "none") == 0)
+ return 0;
+ if (index == MAX_VIRTIO_CONSOLES) {
+ fprintf(stderr, "qemu: too many virtio consoles\n");
+ exit(1);
+ }
+ snprintf(label, sizeof(label), "virtcon%d", index);
+ virtcon_hds[index] = qemu_chr_open(label, devname, NULL);
+ if (!virtcon_hds[index]) {
+ fprintf(stderr, "qemu: could not open virtio console '%s': %s\n",
+ devname, strerror(errno));
+ return -1;
+ }
+ index++;
+ return 0;
+}
+
int main(int argc, char **argv, char **envp)
{
const char *gdbstub_dev = NULL;
@@ -4795,8 +4821,6 @@ int main(int argc, char **argv, char **envp)
QemuOpts *hda_opts = NULL, *opts;
int optind;
const char *r, *optarg;
- const char *virtio_consoles[MAX_VIRTIO_CONSOLES];
- int virtio_console_index;
const char *loadvm = NULL;
QEMUMachine *machine;
const char *cpu_model;
@@ -4862,10 +4886,6 @@ int main(int argc, char **argv, char **envp)
cyls = heads = secs = 0;
translation = BIOS_ATA_TRANSLATION_AUTO;
- for(i = 0; i < MAX_VIRTIO_CONSOLES; i++)
- virtio_consoles[i] = NULL;
- virtio_console_index = 0;
-
for (i = 0; i < MAX_NODES; i++) {
node_mem[i] = 0;
node_cpumask[i] = 0;
@@ -5324,12 +5344,8 @@ int main(int argc, char **argv, char **envp)
}
break;
case QEMU_OPTION_virtiocon:
- if (virtio_console_index >= MAX_VIRTIO_CONSOLES) {
- fprintf(stderr, "qemu: too many virtio consoles\n");
- exit(1);
- }
- virtio_consoles[virtio_console_index] = optarg;
- virtio_console_index++;
+ add_device_config(DEV_VIRTCON, optarg);
+ default_virtcon = 0;
break;
case QEMU_OPTION_parallel:
add_device_config(DEV_PARALLEL, optarg);
@@ -5832,20 +5848,8 @@ int main(int argc, char **argv, char **envp)
exit(1);
if (foreach_device_config(DEV_PARALLEL, parallel_parse) < 0)
exit(1);
-
- for(i = 0; i < MAX_VIRTIO_CONSOLES; i++) {
- const char *devname = virtio_consoles[i];
- if (devname && strcmp(devname, "none")) {
- char label[32];
- snprintf(label, sizeof(label), "virtcon%d", i);
- virtcon_hds[i] = qemu_chr_open(label, devname, NULL);
- if (!virtcon_hds[i]) {
- fprintf(stderr, "qemu: could not open virtio console '%s':
%s\n",
- devname, strerror(errno));
- exit(1);
- }
- }
- }
+ if (foreach_device_config(DEV_VIRTCON, virtcon_parse) < 0)
+ exit(1);
module_call_init(MODULE_INIT_DEVICE);
--
1.6.5.2
- [Qemu-devel] [FOR 0.12 PATCH v3 12/21] default devices: drives, (continued)
- [Qemu-devel] [FOR 0.12 PATCH v3 12/21] default devices: drives, Gerd Hoffmann, 2009/12/07
- [Qemu-devel] [FOR 0.12 PATCH v3 08/21] zap serial_monitor_mux, Gerd Hoffmann, 2009/12/07
- [Qemu-devel] [FOR 0.12 PATCH v3 09/21] default devices: vga adapter., Gerd Hoffmann, 2009/12/07
- [Qemu-devel] [FOR 0.12 PATCH v3 07/21] default devices: qemu monitor., Gerd Hoffmann, 2009/12/07
- [Qemu-devel] [FOR 0.12 PATCH v3 10/21] default devices: add global cmd line option., Gerd Hoffmann, 2009/12/07
- [Qemu-devel] [FOR 0.12 PATCH v3 15/21] chardev: make chardevs specified in config file work., Gerd Hoffmann, 2009/12/07
- [Qemu-devel] [FOR 0.12 PATCH v3 06/21] default devices: parallel port., Gerd Hoffmann, 2009/12/07
- [Qemu-devel] [FOR 0.12 PATCH v3 13/21] qdev: make compat stuff more generic, Gerd Hoffmann, 2009/12/07
- [Qemu-devel] [FOR 0.12 PATCH v3 20/21] default devices: virtio consoles.,
Gerd Hoffmann <=
- [Qemu-devel] [FOR 0.12 PATCH v3 16/21] un-static qemu_chr_parse_compat(), Gerd Hoffmann, 2009/12/07
- [Qemu-devel] [FOR 0.12 PATCH v3 17/21] rework -monitor handling, switch to QemuOpts, Gerd Hoffmann, 2009/12/07
- [Qemu-devel] [FOR 0.12 PATCH v3 21/21] Set default console to virtio on S390x, Gerd Hoffmann, 2009/12/07
- [Qemu-devel] [FOR 0.12 PATCH v3 19/21] add -qmp convinience switch, Gerd Hoffmann, 2009/12/07
- [Qemu-devel] [FOR 0.12 PATCH v3 14/21] qdev: add command line option to set global defaults for properties., Gerd Hoffmann, 2009/12/07