| Message ID | 1305133877-4163-1-git-send-email-glommer@redhat.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On 2011-05-11 19:11, 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. That means bios output is written to the virtual uart? Sounds interesting. Where do we find such an option rom? Alternatively, what about adding serial console support to SeaBIOS? I've seen real BIOSes that allow driving both VGA and serial in parallel. That could be controlled via -boot and some fwcfg, just like we request the boot menu today. Jan
----- Original Message ----- > From: "Jan Kiszka" <jan.kiszka@web.de> > To: "Glauber Costa" <glommer@redhat.com> > Cc: kvm@vger.kernel.org, aliguori@us.ibm.com, mtosatti@redhat.com, qemu-devel@nongnu.org, avi@redhat.com > Sent: Wednesday, May 11, 2011 5:00:10 PM > Subject: Re: [PATCH] Add an isa device for SGA > On 2011-05-11 19:11, 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. > > That means bios output is written to the virtual uart? Sounds > interesting. Where do we find such an option rom? code.google.com/p/sgabios Plan is to include it in the set of roms distributed by QEMU. > Alternatively, what about adding serial console support to SeaBIOS? > I've > seen real BIOSes that allow driving both VGA and serial in parallel. > That could be controlled via -boot and some fwcfg, just like we > request > the boot menu today. Possible, cleaner, but much more work. I think using sgabios give us what we need for very little additional cost. If someone would be willing to do the work in SeaBIOS, that would be another story. > > Jan -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Glauber Costa <glommer@redhat.com> writes: > 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. > > Signed-off-by: Glauber Costa <glommer@redhat.com> > --- > Makefile.target | 2 +- > hw/pc.c | 2 ++ > hw/pc.h | 3 +++ > hw/sga.c | 49 +++++++++++++++++++++++++++++++++++++++++++++++++ > 4 files changed, 55 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..56c3887 100644 > --- a/hw/pc.c > +++ b/hw/pc.c > @@ -1096,6 +1096,8 @@ void pc_vga_init(PCIBus *pci_bus) > isa_vga_init(); > } > } > + > + isa_sga_init(); Please do this the qdev way (untested): if (display_type == DT_NOGRAPHIC) { dev = qdev_create(NULL, "sga"); qdev_init_nofail(dev); } > } > > static void cpu_request_exit(void *opaque, int irq, int level) > diff --git a/hw/pc.h b/hw/pc.h > index 1291e2d..a00e054 100644 > --- a/hw/pc.h > +++ b/hw/pc.h > @@ -219,6 +219,9 @@ int isa_vga_mm_init(target_phys_addr_t vram_base, > void pci_cirrus_vga_init(PCIBus *bus); > void isa_cirrus_vga_init(void); > > +/* serial graphics */ > +void isa_sga_init(void); > + > /* ne2000.c */ > static inline bool isa_ne2000_init(int base, int irq, NICInfo *nd) > { > diff --git a/hw/sga.c b/hw/sga.c > new file mode 100644 > index 0000000..411191b > --- /dev/null > +++ b/hw/sga.c > @@ -0,0 +1,49 @@ > +#include "pci.h" > +#include "pc.h" > +#include "loader.h" > +#include "sysemu.h" > + > +#define SGABIOS_FILENAME "sgabios.bin" > + > +typedef struct ISAGAState { > + ISADevice dev; > +} ISASGAState; > + > +/* We can have both -device, and the initfn called, so better > + * avoid to have the rom loaded twice */ > +static void deploy_rom(void) > +{ > + static int rom_deployed = 0; > + > + if (!rom_deployed++) { > + rom_add_vga(SGABIOS_FILENAME); > + } > +} Is rom_deployed still needed with isa_sga_init() gone? > + > +static int isa_cirrus_vga_initfn(ISADevice *dev) > +{ > + deploy_rom(); > + > + return 0; > +} > + > +void isa_sga_init(void) > +{ > + if (display_type == DT_NOGRAPHIC) { > + deploy_rom(); > + } > +} > + > +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); -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 05/11/11 19:11, 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. How about adding a qdev property to isa-serial instead? cheers, Gerd -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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..56c3887 100644 --- a/hw/pc.c +++ b/hw/pc.c @@ -1096,6 +1096,8 @@ void pc_vga_init(PCIBus *pci_bus) isa_vga_init(); } } + + isa_sga_init(); } static void cpu_request_exit(void *opaque, int irq, int level) diff --git a/hw/pc.h b/hw/pc.h index 1291e2d..a00e054 100644 --- a/hw/pc.h +++ b/hw/pc.h @@ -219,6 +219,9 @@ int isa_vga_mm_init(target_phys_addr_t vram_base, void pci_cirrus_vga_init(PCIBus *bus); void isa_cirrus_vga_init(void); +/* serial graphics */ +void isa_sga_init(void); + /* ne2000.c */ static inline bool isa_ne2000_init(int base, int irq, NICInfo *nd) { diff --git a/hw/sga.c b/hw/sga.c new file mode 100644 index 0000000..411191b --- /dev/null +++ b/hw/sga.c @@ -0,0 +1,49 @@ +#include "pci.h" +#include "pc.h" +#include "loader.h" +#include "sysemu.h" + +#define SGABIOS_FILENAME "sgabios.bin" + +typedef struct ISAGAState { + ISADevice dev; +} ISASGAState; + +/* We can have both -device, and the initfn called, so better + * avoid to have the rom loaded twice */ +static void deploy_rom(void) +{ + static int rom_deployed = 0; + + if (!rom_deployed++) { + rom_add_vga(SGABIOS_FILENAME); + } +} + +static int isa_cirrus_vga_initfn(ISADevice *dev) +{ + deploy_rom(); + + return 0; +} + +void isa_sga_init(void) +{ + if (display_type == DT_NOGRAPHIC) { + deploy_rom(); + } +} + +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);
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. Signed-off-by: Glauber Costa <glommer@redhat.com> --- Makefile.target | 2 +- hw/pc.c | 2 ++ hw/pc.h | 3 +++ hw/sga.c | 49 +++++++++++++++++++++++++++++++++++++++++++++++++ 4 files changed, 55 insertions(+), 1 deletions(-) create mode 100644 hw/sga.c