[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 04/10] default devices: parallel port.
From: |
Gerd Hoffmann |
Subject: |
[Qemu-devel] [PATCH 04/10] default devices: parallel port. |
Date: |
Tue, 24 Nov 2009 09:58:38 +0100 |
Qemu creates a default parallel port for you in case you didn't specify
one on the command line. Right now this is tied to the '-parallel
<chardev>' command line switch, which in turn causes trouble if you are
creating your parallel port via '-device isa-parallel,<props>'.
This patch adds a variable default_parallel which says whenever a default
parallel port should be added. It is enabled by default. It is cleared
when qemu finds '-parallel' or '-device isa-parallel' 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 474e688..ca0af9d 100644
--- a/vl.c
+++ b/vl.c
@@ -273,12 +273,14 @@ static QEMUBootSetHandler *boot_set_handler;
static void *boot_set_opaque;
static int default_serial = 1;
+static int default_parallel = 1;
static struct {
const char *driver;
int *flag;
} default_list[] = {
- { .driver = "isa-serial", .flag = &default_serial },
+ { .driver = "isa-serial", .flag = &default_serial },
+ { .driver = "isa-parallel", .flag = &default_parallel },
};
static int default_driver_check(QemuOpts *opts, void *opaque)
@@ -4568,6 +4570,7 @@ struct device_config {
DEV_USB, /* -usbdevice */
DEV_BT, /* -bt */
DEV_SERIAL, /* -serial */
+ DEV_PARALLEL, /* -parallel */
} type;
const char *cmdline;
QTAILQ_ENTRY(device_config) next;
@@ -4643,6 +4646,28 @@ static int serial_parse(const char *devname)
return 0;
}
+static int parallel_parse(const char *devname)
+{
+ static int index = 0;
+ char label[32];
+
+ if (strcmp(devname, "none") == 0)
+ return 0;
+ if (index == MAX_PARALLEL_PORTS) {
+ fprintf(stderr, "qemu: too many parallel ports\n");
+ exit(1);
+ }
+ snprintf(label, sizeof(label), "parallel%d", index);
+ parallel_hds[index] = qemu_chr_open(label, devname, NULL);
+ if (!parallel_hds[index]) {
+ fprintf(stderr, "qemu: could not open parallel device '%s': %s\n",
+ devname, strerror(errno));
+ return -1;
+ }
+ index++;
+ return 0;
+}
+
int main(int argc, char **argv, char **envp)
{
const char *gdbstub_dev = NULL;
@@ -4661,8 +4686,6 @@ int main(int argc, char **argv, char **envp)
CharDriverState *monitor_hds[MAX_MONITOR_DEVICES];
const char *monitor_devices[MAX_MONITOR_DEVICES];
int monitor_device_index;
- const char *parallel_devices[MAX_PARALLEL_PORTS];
- int parallel_device_index;
const char *virtio_consoles[MAX_VIRTIO_CONSOLES];
int virtio_console_index;
const char *loadvm = NULL;
@@ -4730,11 +4753,6 @@ int main(int argc, char **argv, char **envp)
cyls = heads = secs = 0;
translation = BIOS_ATA_TRANSLATION_AUTO;
- parallel_devices[0] = "vc:80Cx24C";
- for(i = 1; i < MAX_PARALLEL_PORTS; i++)
- parallel_devices[i] = NULL;
- parallel_device_index = 0;
-
for(i = 0; i < MAX_VIRTIO_CONSOLES; i++)
virtio_consoles[i] = NULL;
virtio_console_index = 0;
@@ -5202,12 +5220,8 @@ int main(int argc, char **argv, char **envp)
virtio_console_index++;
break;
case QEMU_OPTION_parallel:
- if (parallel_device_index >= MAX_PARALLEL_PORTS) {
- fprintf(stderr, "qemu: too many parallel ports\n");
- exit(1);
- }
- parallel_devices[parallel_device_index] = optarg;
- parallel_device_index++;
+ add_device_config(DEV_PARALLEL, optarg);
+ default_parallel = 0;
break;
case QEMU_OPTION_loadvm:
loadvm = optarg;
@@ -5483,14 +5497,16 @@ int main(int argc, char **argv, char **envp)
if (display_type == DT_NOGRAPHIC) {
if (default_serial)
add_device_config(DEV_SERIAL, "stdio");
- if (parallel_device_index == 0)
- parallel_devices[0] = "null";
+ if (default_parallel)
+ add_device_config(DEV_PARALLEL, "null");
if (strncmp(monitor_devices[0], "vc", 2) == 0) {
monitor_devices[0] = "stdio";
}
} else {
if (default_serial)
add_device_config(DEV_SERIAL, "vc:80Cx24C");
+ if (default_parallel)
+ add_device_config(DEV_PARALLEL, "vc:80Cx24C");
}
#ifndef _WIN32
@@ -5700,20 +5716,8 @@ int main(int argc, char **argv, char **envp)
if (foreach_device_config(DEV_SERIAL, serial_parse) < 0)
exit(1);
-
- for(i = 0; i < MAX_PARALLEL_PORTS; i++) {
- const char *devname = parallel_devices[i];
- if (devname && strcmp(devname, "none")) {
- char label[32];
- snprintf(label, sizeof(label), "parallel%d", i);
- parallel_hds[i] = qemu_chr_open(label, devname, NULL);
- if (!parallel_hds[i]) {
- fprintf(stderr, "qemu: could not open parallel device '%s':
%s\n",
- devname, strerror(errno));
- 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];
--
1.6.2.5
- [Qemu-devel] [PATCH 00/10] default devices: qdev integration., Gerd Hoffmann, 2009/11/24
- [Qemu-devel] [PATCH 01/10] chardev: move greeting into vc backend., Gerd Hoffmann, 2009/11/24
- [Qemu-devel] [PATCH 02/10] vc: colorize chardev title line with blue background., Gerd Hoffmann, 2009/11/24
- [Qemu-devel] [PATCH 03/10] default devices: core code & serial lines., Gerd Hoffmann, 2009/11/24
- [Qemu-devel] [PATCH 06/10] zap serial_monitor_mux, Gerd Hoffmann, 2009/11/24
- [Qemu-devel] [PATCH 08/10] default devices: add global cmd line option., Gerd Hoffmann, 2009/11/24
- [Qemu-devel] [PATCH 07/10] default devices: vga adapter., Gerd Hoffmann, 2009/11/24
- [Qemu-devel] [PATCH 04/10] default devices: parallel port.,
Gerd Hoffmann <=
- [Qemu-devel] [PATCH 05/10] default devices: qemu monitor., Gerd Hoffmann, 2009/11/24
- [Qemu-devel] [PATCH 10/10] default devices: drives, Gerd Hoffmann, 2009/11/24
- [Qemu-devel] [PATCH 09/10] default devices: network, Gerd Hoffmann, 2009/11/24
- Re: [Qemu-devel] [PATCH 00/10] default devices: qdev integration., Markus Armbruster, 2009/11/24