[Qemu-devel] [PATCH v3] Add an isa device for SGA

[Qemu-devel] [PATCH v3] Add an isa device for SGA

[Glauber Costa]

This patch adds a dummy legacy ISA device whose responsibility is to
deploy sgabios, an option rom for a serial graphics adapter.
The proposal is that this device is always-on when -nographics,
but can otherwise be enable in any setup when -device sga is used.

[v2: suggestions on qdev by Markus ]
[v3: cleanups and documentation, per list suggestions ]

Signed-off-by: Glauber Costa <glommer@redhat.com>
---
 Makefile.target |    2 +-
 hw/pc.c         |    9 ++++++++
 hw/sga.c        |   56 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 66 insertions(+), 1 deletions(-)
 create mode 100644 hw/sga.c

diff --git a/Makefile.target b/Makefile.target
index fdbdc6c..004ea7e 100644
--- a/Makefile.target
+++ b/Makefile.target
@@ -224,7 +224,7 @@ obj-$(CONFIG_KVM) += ivshmem.o
 # Hardware support
 obj-i386-y += vga.o
 obj-i386-y += mc146818rtc.o i8259.o pc.o
-obj-i386-y += cirrus_vga.o apic.o ioapic.o piix_pci.o
+obj-i386-y += cirrus_vga.o sga.o apic.o ioapic.o piix_pci.o
 obj-i386-y += vmport.o
 obj-i386-y += device-hotplug.o pci-hotplug.o smbios.o wdt_ib700.o
 obj-i386-y += extboot.o
diff --git a/hw/pc.c b/hw/pc.c
index 8d351ba..5a8e00a 100644
--- a/hw/pc.c
+++ b/hw/pc.c
@@ -1096,6 +1096,15 @@ void pc_vga_init(PCIBus *pci_bus)
             isa_vga_init();
         }
     }
+
+    /*
+     * sga does not suppress normal vga output. So a machine can have both a
+     * vga card and sga manually enabled. Output will be seen on both.
+     * For nographic case, sga is enabled at all times
+     */
+    if (display_type == DT_NOGRAPHIC) {
+        isa_create_simple("sga");
+    }
 }
 
 static void cpu_request_exit(void *opaque, int irq, int level)
diff --git a/hw/sga.c b/hw/sga.c
new file mode 100644
index 0000000..7ef750a
--- /dev/null
+++ b/hw/sga.c
@@ -0,0 +1,56 @@
+/*
+ * QEMU dummy ISA device for loading sgabios option rom.
+ *
+ * Copyright (c) 2011 Glauber Costa, Red Hat Inc.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+ * THE SOFTWARE.
+ *
+ * sgabios code originally available at code.google.com/p/sgabios
+ *
+ */
+#include "pci.h"
+#include "pc.h"
+#include "loader.h"
+#include "sysemu.h"
+
+#define SGABIOS_FILENAME "sgabios.bin"
+
+typedef struct ISAGAState {
+    ISADevice dev;
+} ISASGAState;
+
+static int isa_cirrus_vga_initfn(ISADevice *dev)
+{
+    rom_add_vga(SGABIOS_FILENAME);
+    return 0;
+}
+
+static ISADeviceInfo sga_info = {
+    .qdev.name    = "sga",
+    .qdev.desc    = "Serial Graphics Adapter",
+    .qdev.size    = sizeof(ISASGAState),
+    .init         = isa_cirrus_vga_initfn,
+};
+
+static void sga_register(void)
+{
+      isa_qdev_register(&sga_info);
+}
+
+device_init(sga_register);
-- 
1.7.4.2

Re: [Qemu-devel] [PATCH v3] Add an isa device for SGA

[Anthony Liguori]

On 05/16/2011 01:45 PM, Glauber Costa wrote:
> This patch adds a dummy legacy ISA device whose responsibility is to
> deploy sgabios, an option rom for a serial graphics adapter.
> The proposal is that this device is always-on when -nographics,
> but can otherwise be enable in any setup when -device sga is used.
>
> [v2: suggestions on qdev by Markus ]
> [v3: cleanups and documentation, per list suggestions ]
>
> Signed-off-by: Glauber Costa<glommer@redhat.com>

Applied.  But I'd like to figure out what to do about sgabios.bin.  I 
think we should ship a copy.

Regards,

Anthony Liguori

> ---
>   Makefile.target |    2 +-
>   hw/pc.c         |    9 ++++++++
>   hw/sga.c        |   56 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
>   3 files changed, 66 insertions(+), 1 deletions(-)
>   create mode 100644 hw/sga.c
>
> diff --git a/Makefile.target b/Makefile.target
> index fdbdc6c..004ea7e 100644
> --- a/Makefile.target
> +++ b/Makefile.target
> @@ -224,7 +224,7 @@ obj-$(CONFIG_KVM) += ivshmem.o
>   # Hardware support
>   obj-i386-y += vga.o
>   obj-i386-y += mc146818rtc.o i8259.o pc.o
> -obj-i386-y += cirrus_vga.o apic.o ioapic.o piix_pci.o
> +obj-i386-y += cirrus_vga.o sga.o apic.o ioapic.o piix_pci.o
>   obj-i386-y += vmport.o
>   obj-i386-y += device-hotplug.o pci-hotplug.o smbios.o wdt_ib700.o
>   obj-i386-y += extboot.o
> diff --git a/hw/pc.c b/hw/pc.c
> index 8d351ba..5a8e00a 100644
> --- a/hw/pc.c
> +++ b/hw/pc.c
> @@ -1096,6 +1096,15 @@ void pc_vga_init(PCIBus *pci_bus)
>               isa_vga_init();
>           }
>       }
> +
> +    /*
> +     * sga does not suppress normal vga output. So a machine can have both a
> +     * vga card and sga manually enabled. Output will be seen on both.
> +     * For nographic case, sga is enabled at all times
> +     */
> +    if (display_type == DT_NOGRAPHIC) {
> +        isa_create_simple("sga");
> +    }
>   }
>
>   static void cpu_request_exit(void *opaque, int irq, int level)
> diff --git a/hw/sga.c b/hw/sga.c
> new file mode 100644
> index 0000000..7ef750a
> --- /dev/null
> +++ b/hw/sga.c
> @@ -0,0 +1,56 @@
> +/*
> + * QEMU dummy ISA device for loading sgabios option rom.
> + *
> + * Copyright (c) 2011 Glauber Costa, Red Hat Inc.
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a copy
> + * of this software and associated documentation files (the "Software"), to deal
> + * in the Software without restriction, including without limitation the rights
> + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
> + * copies of the Software, and to permit persons to whom the Software is
> + * furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice shall be included in
> + * all copies or substantial portions of the Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
> + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
> + * THE SOFTWARE.
> + *
> + * sgabios code originally available at code.google.com/p/sgabios
> + *
> + */
> +#include "pci.h"
> +#include "pc.h"
> +#include "loader.h"
> +#include "sysemu.h"
> +
> +#define SGABIOS_FILENAME "sgabios.bin"
> +
> +typedef struct ISAGAState {
> +    ISADevice dev;
> +} ISASGAState;
> +
> +static int isa_cirrus_vga_initfn(ISADevice *dev)
> +{
> +    rom_add_vga(SGABIOS_FILENAME);
> +    return 0;
> +}
> +
> +static ISADeviceInfo sga_info = {
> +    .qdev.name    = "sga",
> +    .qdev.desc    = "Serial Graphics Adapter",
> +    .qdev.size    = sizeof(ISASGAState),
> +    .init         = isa_cirrus_vga_initfn,
> +};
> +
> +static void sga_register(void)
> +{
> +      isa_qdev_register(&sga_info);
> +}
> +
> +device_init(sga_register);

Re: [Qemu-devel] [PATCH v3] Add an isa device for SGA

[Glauber Costa]

On 06/07/2011 04:17 PM, Anthony Liguori wrote:
> On 05/16/2011 01:45 PM, Glauber Costa wrote:
>> This patch adds a dummy legacy ISA device whose responsibility is to
>> deploy sgabios, an option rom for a serial graphics adapter.
>> The proposal is that this device is always-on when -nographics,
>> but can otherwise be enable in any setup when -device sga is used.
>>
>> [v2: suggestions on qdev by Markus ]
>> [v3: cleanups and documentation, per list suggestions ]
>>
>> Signed-off-by: Glauber Costa<glommer@redhat.com>
>
> Applied. But I'd like to figure out what to do about sgabios.bin. I
> think we should ship a copy.

Agree.

> Regards,
>
> Anthony Liguori
>
>> ---
>> Makefile.target | 2 +-
>> hw/pc.c | 9 ++++++++
>> hw/sga.c | 56 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
>> 3 files changed, 66 insertions(+), 1 deletions(-)
>> create mode 100644 hw/sga.c
>>
>> diff --git a/Makefile.target b/Makefile.target
>> index fdbdc6c..004ea7e 100644
>> --- a/Makefile.target
>> +++ b/Makefile.target
>> @@ -224,7 +224,7 @@ obj-$(CONFIG_KVM) += ivshmem.o
>> # Hardware support
>> obj-i386-y += vga.o
>> obj-i386-y += mc146818rtc.o i8259.o pc.o
>> -obj-i386-y += cirrus_vga.o apic.o ioapic.o piix_pci.o
>> +obj-i386-y += cirrus_vga.o sga.o apic.o ioapic.o piix_pci.o
>> obj-i386-y += vmport.o
>> obj-i386-y += device-hotplug.o pci-hotplug.o smbios.o wdt_ib700.o
>> obj-i386-y += extboot.o
>> diff --git a/hw/pc.c b/hw/pc.c
>> index 8d351ba..5a8e00a 100644
>> --- a/hw/pc.c
>> +++ b/hw/pc.c
>> @@ -1096,6 +1096,15 @@ void pc_vga_init(PCIBus *pci_bus)
>> isa_vga_init();
>> }
>> }
>> +
>> + /*
>> + * sga does not suppress normal vga output. So a machine can have both a
>> + * vga card and sga manually enabled. Output will be seen on both.
>> + * For nographic case, sga is enabled at all times
>> + */
>> + if (display_type == DT_NOGRAPHIC) {
>> + isa_create_simple("sga");
>> + }
>> }
>>
>> static void cpu_request_exit(void *opaque, int irq, int level)
>> diff --git a/hw/sga.c b/hw/sga.c
>> new file mode 100644
>> index 0000000..7ef750a
>> --- /dev/null
>> +++ b/hw/sga.c
>> @@ -0,0 +1,56 @@
>> +/*
>> + * QEMU dummy ISA device for loading sgabios option rom.
>> + *
>> + * Copyright (c) 2011 Glauber Costa, Red Hat Inc.
>> + *
>> + * Permission is hereby granted, free of charge, to any person
>> obtaining a copy
>> + * of this software and associated documentation files (the
>> "Software"), to deal
>> + * in the Software without restriction, including without limitation
>> the rights
>> + * to use, copy, modify, merge, publish, distribute, sublicense,
>> and/or sell
>> + * copies of the Software, and to permit persons to whom the Software is
>> + * furnished to do so, subject to the following conditions:
>> + *
>> + * The above copyright notice and this permission notice shall be
>> included in
>> + * all copies or substantial portions of the Software.
>> + *
>> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
>> EXPRESS OR
>> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
>> MERCHANTABILITY,
>> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT
>> SHALL
>> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES
>> OR OTHER
>> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
>> ARISING FROM,
>> + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
>> DEALINGS IN
>> + * THE SOFTWARE.
>> + *
>> + * sgabios code originally available at code.google.com/p/sgabios
>> + *
>> + */
>> +#include "pci.h"
>> +#include "pc.h"
>> +#include "loader.h"
>> +#include "sysemu.h"
>> +
>> +#define SGABIOS_FILENAME "sgabios.bin"
>> +
>> +typedef struct ISAGAState {
>> + ISADevice dev;
>> +} ISASGAState;
>> +
>> +static int isa_cirrus_vga_initfn(ISADevice *dev)
>> +{
>> + rom_add_vga(SGABIOS_FILENAME);
>> + return 0;
>> +}
>> +
>> +static ISADeviceInfo sga_info = {
>> + .qdev.name = "sga",
>> + .qdev.desc = "Serial Graphics Adapter",
>> + .qdev.size = sizeof(ISASGAState),
>> + .init = isa_cirrus_vga_initfn,
>> +};
>> +
>> +static void sga_register(void)
>> +{
>> + isa_qdev_register(&sga_info);
>> +}
>> +
>> +device_init(sga_register);
>