OpenBIOS May 2012

openbios@openbios.org

10 participants
42 discussions

Re: [OpenBIOS] [PATCH] Sparc32: avoid problems with SMP table

by Andreas Färber

Am 06.05.2012 20:45, schrieb Blue Swirl: > Check CPU ID if we're running on boot CPU. If so, > don't use SMP table information since it may contain > uninitialized or garbage values. > > Signed-off-by: Blue Swirl <blauwirbel(a)gmail.com> > --- > arch/sparc32/entry.S | 11 +++++++++++ > 1 file changed, 11 insertions(+) I've still not got around to testing this on ppc yet, but on x86_64 valgrind was happy. sparc64 seems to have a similar problem though according to valgrind: ==22736== Thread 3: ==22736== Conditional jump or move depends on uninitialised value(s) ==22736== at 0x2C8050: compute_all_sub (cc_helper.c:37) ==22736== by 0x2C84A8: helper_compute_psr (cc_helper.c:470) ==22736== by 0x9769447: ??? ==22736== Uninitialised value was created by a heap allocation ==22736== at 0x4C27CE8: memalign (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) ==22736== by 0x4C27D97: posix_memalign (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) ==22736== by 0x256061: qemu_memalign (oslib-posix.c:93) ==22736== by 0x256109: qemu_vmalloc (oslib-posix.c:126) ==22736== by 0x2D1EF6: qemu_ram_alloc_from_ptr (exec.c:2664) ==22736== by 0x2F5FF6: memory_region_init_ram (memory.c:954) ==22736== by 0x303EFD: ram_init1 (sun4u.c:711) ==22736== by 0x268E1E: qdev_init (qdev.c:151) ==22736== by 0x268F5C: qdev_init_nofail (qdev.c:258) ==22736== by 0x304620: sun4uv_init (sun4u.c:729) ==22736== by 0x304F1F: sun4u_init (sun4u.c:932) ==22736== by 0x1846D8: main (vl.c:3519) ==22736== ==22736== Conditional jump or move depends on uninitialised value(s) ==22736== at 0x2C7F92: compute_all_sub_xcc (cc_helper.c:60) ==22736== by 0x2C84C2: helper_compute_psr (cc_helper.c:473) ==22736== by 0x9769447: ??? ==22736== Uninitialised value was created by a heap allocation ==22736== at 0x4C27CE8: memalign (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) ==22736== by 0x4C27D97: posix_memalign (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) ==22736== by 0x256061: qemu_memalign (oslib-posix.c:93) ==22736== by 0x256109: qemu_vmalloc (oslib-posix.c:126) ==22736== by 0x2D1EF6: qemu_ram_alloc_from_ptr (exec.c:2664) ==22736== by 0x2F5FF6: memory_region_init_ram (memory.c:954) ==22736== by 0x303EFD: ram_init1 (sun4u.c:711) ==22736== by 0x268E1E: qdev_init (qdev.c:151) ==22736== by 0x268F5C: qdev_init_nofail (qdev.c:258) ==22736== by 0x304620: sun4uv_init (sun4u.c:729) ==22736== by 0x304F1F: sun4u_init (sun4u.c:932) ==22736== by 0x1846D8: main (vl.c:3519) ==22736== ==22736== Conditional jump or move depends on uninitialised value(s) ==22736== at 0x2C8050: compute_all_sub (cc_helper.c:37) ==22736== by 0x2C84A8: helper_compute_psr (cc_helper.c:470) ==22736== by 0x97C39BC: ??? ==22736== Uninitialised value was created by a heap allocation ==22736== at 0x4C27CE8: memalign (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) ==22736== by 0x4C27D97: posix_memalign (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) ==22736== by 0x256061: qemu_memalign (oslib-posix.c:93) ==22736== by 0x256109: qemu_vmalloc (oslib-posix.c:126) ==22736== by 0x2D1EF6: qemu_ram_alloc_from_ptr (exec.c:2664) ==22736== by 0x2F5FF6: memory_region_init_ram (memory.c:954) ==22736== by 0x303EFD: ram_init1 (sun4u.c:711) ==22736== by 0x268E1E: qdev_init (qdev.c:151) ==22736== by 0x268F5C: qdev_init_nofail (qdev.c:258) ==22736== by 0x304620: sun4uv_init (sun4u.c:729) ==22736== by 0x304F1F: sun4u_init (sun4u.c:932) ==22736== by 0x1846D8: main (vl.c:3519) ==22736== ==22736== Conditional jump or move depends on uninitialised value(s) ==22736== at 0x2C7F92: compute_all_sub_xcc (cc_helper.c:60) ==22736== by 0x2C84C2: helper_compute_psr (cc_helper.c:473) ==22736== by 0x97C39BC: ??? ==22736== Uninitialised value was created by a heap allocation ==22736== at 0x4C27CE8: memalign (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) ==22736== by 0x4C27D97: posix_memalign (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) ==22736== by 0x256061: qemu_memalign (oslib-posix.c:93) ==22736== by 0x256109: qemu_vmalloc (oslib-posix.c:126) ==22736== by 0x2D1EF6: qemu_ram_alloc_from_ptr (exec.c:2664) ==22736== by 0x2F5FF6: memory_region_init_ram (memory.c:954) ==22736== by 0x303EFD: ram_init1 (sun4u.c:711) ==22736== by 0x268E1E: qdev_init (qdev.c:151) ==22736== by 0x268F5C: qdev_init_nofail (qdev.c:258) ==22736== by 0x304620: sun4uv_init (sun4u.c:729) ==22736== by 0x304F1F: sun4u_init (sun4u.c:932) ==22736== by 0x1846D8: main (vl.c:3519) ==22736== ==22736== Conditional jump or move depends on uninitialised value(s) ==22736== at 0x2C83AF: compute_all_logic (cc_helper.c:37) ==22736== by 0x2C84A8: helper_compute_psr (cc_helper.c:470) ==22736== by 0x97C3A67: ??? ==22736== Uninitialised value was created by a heap allocation ==22736== at 0x4C27CE8: memalign (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) ==22736== by 0x4C27D97: posix_memalign (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) ==22736== by 0x256061: qemu_memalign (oslib-posix.c:93) ==22736== by 0x256109: qemu_vmalloc (oslib-posix.c:126) ==22736== by 0x2D1EF6: qemu_ram_alloc_from_ptr (exec.c:2664) ==22736== by 0x2F5FF6: memory_region_init_ram (memory.c:954) ==22736== by 0x303EFD: ram_init1 (sun4u.c:711) ==22736== by 0x268E1E: qdev_init (qdev.c:151) ==22736== by 0x268F5C: qdev_init_nofail (qdev.c:258) ==22736== by 0x304620: sun4uv_init (sun4u.c:729) ==22736== by 0x304F1F: sun4u_init (sun4u.c:932) ==22736== by 0x1846D8: main (vl.c:3519) ==22736== ==22736== Conditional jump or move depends on uninitialised value(s) ==22736== at 0x2C8441: compute_all_logic_xcc (cc_helper.c:60) ==22736== by 0x2C84C2: helper_compute_psr (cc_helper.c:473) ==22736== by 0x97C3A67: ??? ==22736== Uninitialised value was created by a heap allocation ==22736== at 0x4C27CE8: memalign (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) ==22736== by 0x4C27D97: posix_memalign (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) ==22736== by 0x256061: qemu_memalign (oslib-posix.c:93) ==22736== by 0x256109: qemu_vmalloc (oslib-posix.c:126) ==22736== by 0x2D1EF6: qemu_ram_alloc_from_ptr (exec.c:2664) ==22736== by 0x2F5FF6: memory_region_init_ram (memory.c:954) ==22736== by 0x303EFD: ram_init1 (sun4u.c:711) ==22736== by 0x268E1E: qdev_init (qdev.c:151) ==22736== by 0x268F5C: qdev_init_nofail (qdev.c:258) ==22736== by 0x304620: sun4uv_init (sun4u.c:729) ==22736== by 0x304F1F: sun4u_init (sun4u.c:932) ==22736== by 0x1846D8: main (vl.c:3519) ==22736== ==22736== Conditional jump or move depends on uninitialised value(s) ==22736== at 0x977207F: ??? ==22736== Uninitialised value was created by a heap allocation ==22736== at 0x4C27CE8: memalign (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) ==22736== by 0x4C27D97: posix_memalign (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) ==22736== by 0x256061: qemu_memalign (oslib-posix.c:93) ==22736== by 0x256109: qemu_vmalloc (oslib-posix.c:126) ==22736== by 0x2D1EF6: qemu_ram_alloc_from_ptr (exec.c:2664) ==22736== by 0x2F5FF6: memory_region_init_ram (memory.c:954) ==22736== by 0x303EFD: ram_init1 (sun4u.c:711) ==22736== by 0x268E1E: qdev_init (qdev.c:151) ==22736== by 0x268F5C: qdev_init_nofail (qdev.c:258) ==22736== by 0x304620: sun4uv_init (sun4u.c:729) ==22736== by 0x304F1F: sun4u_init (sun4u.c:932) ==22736== by 0x1846D8: main (vl.c:3519) ==22736== Andreas -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

9 years, 5 months

Re: [OpenBIOS] [PATCH] Identify virtio-net devices

by Artyom Tarasenko

Ping? Can anyone apply this before the 1.1 release of QEMU? On 5/19/12, Artyom Tarasenko <atar4qemu(a)gmail.com> wrote: > Identify virtio network device in PCI probe. > > Signed-off-by: Artyom Tarasenko <atar4qemu(a)gmail.com> > --- > drivers/pci_database.c | 8 ++++++++ > include/drivers/pci.h | 1 + > 2 files changed, 9 insertions(+), 0 deletions(-) > > diff --git a/drivers/pci_database.c b/drivers/pci_database.c > index 1d2958c..197f27e 100644 > --- a/drivers/pci_database.c > +++ b/drivers/pci_database.c > @@ -129,6 +129,14 @@ static const pci_dev_t eth_devices[] = { > NULL, "ethernet", > }, > { > + /* Virtio-network controller */ > + PCI_VENDOR_ID_REDHAT_QUMRANET, PCI_DEVICE_ID_VIRTIO_NET, > + NULL, "virtio-net", NULL, > + "pci1af4,1000\0pci1af4,1000\0pciclass,020000\0", > + 0, 0, 0, > + virtio_config_cb, NULL, > + }, > + { > 0xFFFF, 0xFFFF, > NULL, NULL, NULL, NULL, > -1, -1, -1, > diff --git a/include/drivers/pci.h b/include/drivers/pci.h > index 4c996c0..d3b290a 100644 > --- a/include/drivers/pci.h > +++ b/include/drivers/pci.h > @@ -206,6 +206,7 @@ extern const pci_arch_t *arch; > #define PCI_DEVICE_ID_QEMU_VGA 0x1111 > > #define PCI_VENDOR_ID_REDHAT_QUMRANET 0x1af4 > +#define PCI_DEVICE_ID_VIRTIO_NET 0x1000 > #define PCI_DEVICE_ID_VIRTIO_BLOCK 0x1001 > > #define PCI_VENDOR_ID_INTEL 0x8086 > -- > 1.7.1 > > -- Regards, Artyom Tarasenko solaris/sparc under qemu blog: http://tyom.blogspot.com/search/label/qemu

9 years, 5 months

Re: [OpenBIOS] adding eepom node to sparc64

by Artyom Tarasenko

On Mon, May 21, 2012 at 7:10 PM, Blue Swirl <blauwirbel(a)gmail.com> wrote: > On Sun, May 20, 2012 at 10:23 PM, Mark Cave-Ayland > <mark.cave-ayland(a)ilande.co.uk> wrote: >> On 19/05/12 18:21, Blue Swirl wrote: >> >>> On Sat, May 19, 2012 at 4:38 PM, Artyom Tarasenko<atar4qemu(a)gmail.com> >>> wrote: >>>> >>>> Both Linux and MilaX complain about the missing eeprom / rtc-m48t59 node. >>>> Since QEMU actually has this device I tried to add it, but not sure >>>> where is it located. >> >> >> Oooh this will be interesting as Milax currently fails when it fails to >> detect this device - fixing it should get Solaris to boot further. This has >> also been on my list for a while, but so much to do, not enough time... >> >> >>>> In sun4u.c it's initialized like this: >>>> >>>> nvram = m48t59_init_isa(isa_bus, 0x0074, NVRAM_SIZE, 59); >>>> >>>> Is 0x74 the offset to the ISA bus? Then it's wired not as on Ultra-5: >>>> >>>> Node 0xf00662d8 >>>> address: fffba000 >>>> reg: 00000014.00000000.00002000 >>>> model: 'mk48t59' >>>> name: 'eeprom' >>>> >>>> Here it looks like it had to be aligned to a beginning of a page, >>>> which makes sense because of the large size... >>>> So, where QEMU's 0x74 are coming from? >>> >>> >>> It's the I/O port used by PPC. Looking at the reg, it should be using >>> MMIO at address 0 instead. >> >> >> FWIW I've just had a discussion on qemu-devel and the outcome was that the >> <device>_init() functions are generally just shortcuts to the most common >> "wiring" of the peripherals into a machine. So I suspect you'll need to copy >> the _init() function as sun4_m48t59_init() and customise as required. > > We're now using m48t59_init_isa(), we could instead use m48t59_init() > with zero ports. But this would create a SysBusDevice behind Ebus > which is behind PCI, so PCI BAR changes would not propagate. Yes, I rather thought about adding the mem_base parameter to the m48t59_init_isa() function, similar to m48t59_init() . -- Regards, Artyom Tarasenko solaris/sparc under qemu blog: http://tyom.blogspot.com/search/label/qemu

9 years, 5 months

Re: [OpenBIOS] adding eepom node to sparc64

by Mark Cave-Ayland

On 19/05/12 18:21, Blue Swirl wrote: > On Sat, May 19, 2012 at 4:38 PM, Artyom Tarasenko<atar4qemu(a)gmail.com> wrote: >> Both Linux and MilaX complain about the missing eeprom / rtc-m48t59 node. >> Since QEMU actually has this device I tried to add it, but not sure >> where is it located. Oooh this will be interesting as Milax currently fails when it fails to detect this device - fixing it should get Solaris to boot further. This has also been on my list for a while, but so much to do, not enough time... >> In sun4u.c it's initialized like this: >> >> nvram = m48t59_init_isa(isa_bus, 0x0074, NVRAM_SIZE, 59); >> >> Is 0x74 the offset to the ISA bus? Then it's wired not as on Ultra-5: >> >> Node 0xf00662d8 >> address: fffba000 >> reg: 00000014.00000000.00002000 >> model: 'mk48t59' >> name: 'eeprom' >> >> Here it looks like it had to be aligned to a beginning of a page, >> which makes sense because of the large size... >> So, where QEMU's 0x74 are coming from? > > It's the I/O port used by PPC. Looking at the reg, it should be using > MMIO at address 0 instead. FWIW I've just had a discussion on qemu-devel and the outcome was that the <device>_init() functions are generally just shortcuts to the most common "wiring" of the peripherals into a machine. So I suspect you'll need to copy the _init() function as sun4_m48t59_init() and customise as required. HTH, Mark.

9 years, 5 months

[PATCH] Replace 'Qemu' by 'QEMU'

by Stefan Weil

The official product name is written 'QEMU'. Signed-off-by: Stefan Weil <sw(a)weilnetz.de> --- arch/ppc/qemu/start.S | 2 +- arch/ppc/qemu/tree.fs | 2 +- arch/sparc32/ldscript | 2 +- arch/sparc64/ldscript | 2 +- drivers/pci_database.c | 2 +- drivers/sbus.c | 6 +++--- 6 files changed, 8 insertions(+), 8 deletions(-) diff --git a/arch/ppc/qemu/start.S b/arch/ppc/qemu/start.S index aa582e2..77ced18 100644 --- a/arch/ppc/qemu/start.S +++ b/arch/ppc/qemu/start.S @@ -701,7 +701,7 @@ _GLOBAL(flush_icache_range): isync blr - /* Get RAM size from Qemu configuration device */ + /* Get RAM size from QEMU configuration device */ #define CFG_ADDR 0xf0000510 #define FW_CFG_RAM_SIZE 0x03 diff --git a/arch/ppc/qemu/tree.fs b/arch/ppc/qemu/tree.fs index e4115ea..1b9c293 100644 --- a/arch/ppc/qemu/tree.fs +++ b/arch/ppc/qemu/tree.fs @@ -1,4 +1,4 @@ -\ Qemu specific initialization code +\ QEMU specific initialization code \ \ This program is free software; you can redistribute it and/or \ modify it under the terms of the GNU General Public License diff --git a/arch/sparc32/ldscript b/arch/sparc32/ldscript index 32c9445..b543c15 100644 --- a/arch/sparc32/ldscript +++ b/arch/sparc32/ldscript @@ -1,7 +1,7 @@ OUTPUT_FORMAT(elf32-sparc) OUTPUT_ARCH(sparc) -/* Qemu ELF loader can't handle very complex files, so we put ELFBoot +/* QEMU ELF loader can't handle very complex files, so we put ELFBoot info to rodata and put initctx to data.*/ ENTRY(trap_table) diff --git a/arch/sparc64/ldscript b/arch/sparc64/ldscript index 11167c0..d62ab78 100644 --- a/arch/sparc64/ldscript +++ b/arch/sparc64/ldscript @@ -1,7 +1,7 @@ OUTPUT_FORMAT(elf64-sparc) OUTPUT_ARCH(sparc:v9) -/* Qemu ELF loader can't handle very complex files, so we put ELFBoot +/* QEMU ELF loader can't handle very complex files, so we put ELFBoot info to rodata and put initctx to data.*/ ENTRY(trap_table) diff --git a/drivers/pci_database.c b/drivers/pci_database.c index ac47621..0983da3 100644 --- a/drivers/pci_database.c +++ b/drivers/pci_database.c @@ -176,7 +176,7 @@ static const pci_dev_t vga_devices[] = { }, { PCI_VENDOR_ID_QEMU, PCI_DEVICE_ID_QEMU_VGA, - NULL, "QEMU,VGA", "Qemu VGA", "VGA\0", + NULL, "QEMU,VGA", "QEMU VGA", "VGA\0", 0, 0, 0, NULL, NULL, }, diff --git a/drivers/sbus.c b/drivers/sbus.c index 1107c1d..3708a5e 100644 --- a/drivers/sbus.c +++ b/drivers/sbus.c @@ -429,7 +429,7 @@ ob_macio_init(unsigned int slot, uint64_t base, unsigned long offset) static void sbus_probe_slot_ss5(unsigned int slot, uint64_t base) { - // OpenBIOS and Qemu don't know how to do Sbus probing + // OpenBIOS and QEMU don't know how to do Sbus probing switch(slot) { case 3: // SUNW,tcx ob_tcx_init(slot, "/iommu/sbus/SUNW,tcx"); @@ -451,7 +451,7 @@ sbus_probe_slot_ss5(unsigned int slot, uint64_t base) static void sbus_probe_slot_ss10(unsigned int slot, uint64_t base) { - // OpenBIOS and Qemu don't know how to do Sbus probing + // OpenBIOS and QEMU don't know how to do Sbus probing switch(slot) { case 2: // SUNW,tcx ob_tcx_init(slot, "/iommu/sbus/SUNW,tcx"); @@ -469,7 +469,7 @@ sbus_probe_slot_ss10(unsigned int slot, uint64_t base) static void sbus_probe_slot_ss600mp(unsigned int slot, uint64_t base) { - // OpenBIOS and Qemu don't know how to do Sbus probing + // OpenBIOS and QEMU don't know how to do Sbus probing switch(slot) { case 2: // SUNW,tcx ob_tcx_init(slot, "/iommu/sbus/SUNW,tcx"); -- 1.7.9

9 years, 5 months

[commit] r1060 - in trunk/openbios-devel: arch/ppc/qemu arch/sparc32 arch/sparc64 drivers

by repository service

Author: afaerber Date: Sun May 20 18:07:35 2012 New Revision: 1060 URL: http://tracker.coreboot.org/trac/openbios/changeset/1060 Log: Replace 'Qemu' by 'QEMU' The official product name is written 'QEMU'. Signed-off-by: Stefan Weil <sw(a)weilnetz.de> Signed-off-by: Andreas Färber <afaerber(a)suse.de> Modified: trunk/openbios-devel/arch/ppc/qemu/start.S trunk/openbios-devel/arch/ppc/qemu/tree.fs trunk/openbios-devel/arch/sparc32/ldscript trunk/openbios-devel/arch/sparc64/ldscript trunk/openbios-devel/drivers/sbus.c Modified: trunk/openbios-devel/arch/ppc/qemu/start.S ============================================================================== --- trunk/openbios-devel/arch/ppc/qemu/start.S Sat May 19 12:38:26 2012 (r1059) +++ trunk/openbios-devel/arch/ppc/qemu/start.S Sun May 20 18:07:35 2012 (r1060) @@ -701,7 +701,7 @@ isync blr - /* Get RAM size from Qemu configuration device */ + /* Get RAM size from QEMU configuration device */ #define CFG_ADDR 0xf0000510 #define FW_CFG_RAM_SIZE 0x03 Modified: trunk/openbios-devel/arch/ppc/qemu/tree.fs ============================================================================== --- trunk/openbios-devel/arch/ppc/qemu/tree.fs Sat May 19 12:38:26 2012 (r1059) +++ trunk/openbios-devel/arch/ppc/qemu/tree.fs Sun May 20 18:07:35 2012 (r1060) @@ -1,4 +1,4 @@ -\ Qemu specific initialization code +\ QEMU specific initialization code \ \ This program is free software; you can redistribute it and/or \ modify it under the terms of the GNU General Public License Modified: trunk/openbios-devel/arch/sparc32/ldscript ============================================================================== --- trunk/openbios-devel/arch/sparc32/ldscript Sat May 19 12:38:26 2012 (r1059) +++ trunk/openbios-devel/arch/sparc32/ldscript Sun May 20 18:07:35 2012 (r1060) @@ -1,7 +1,7 @@ OUTPUT_FORMAT(elf32-sparc) OUTPUT_ARCH(sparc) -/* Qemu ELF loader can't handle very complex files, so we put ELFBoot +/* QEMU ELF loader can't handle very complex files, so we put ELFBoot info to rodata and put initctx to data.*/ ENTRY(trap_table) Modified: trunk/openbios-devel/arch/sparc64/ldscript ============================================================================== --- trunk/openbios-devel/arch/sparc64/ldscript Sat May 19 12:38:26 2012 (r1059) +++ trunk/openbios-devel/arch/sparc64/ldscript Sun May 20 18:07:35 2012 (r1060) @@ -1,7 +1,7 @@ OUTPUT_FORMAT(elf64-sparc) OUTPUT_ARCH(sparc:v9) -/* Qemu ELF loader can't handle very complex files, so we put ELFBoot +/* QEMU ELF loader can't handle very complex files, so we put ELFBoot info to rodata and put initctx to data.*/ ENTRY(trap_table) Modified: trunk/openbios-devel/drivers/sbus.c ============================================================================== --- trunk/openbios-devel/drivers/sbus.c Sat May 19 12:38:26 2012 (r1059) +++ trunk/openbios-devel/drivers/sbus.c Sun May 20 18:07:35 2012 (r1060) @@ -429,7 +429,7 @@ static void sbus_probe_slot_ss5(unsigned int slot, uint64_t base) { - // OpenBIOS and Qemu don't know how to do Sbus probing + // OpenBIOS and QEMU don't know how to do Sbus probing switch(slot) { case 3: // SUNW,tcx ob_tcx_init(slot, "/iommu/sbus/SUNW,tcx"); @@ -451,7 +451,7 @@ static void sbus_probe_slot_ss10(unsigned int slot, uint64_t base) { - // OpenBIOS and Qemu don't know how to do Sbus probing + // OpenBIOS and QEMU don't know how to do Sbus probing switch(slot) { case 2: // SUNW,tcx ob_tcx_init(slot, "/iommu/sbus/SUNW,tcx"); @@ -469,7 +469,7 @@ static void sbus_probe_slot_ss600mp(unsigned int slot, uint64_t base) { - // OpenBIOS and Qemu don't know how to do Sbus probing + // OpenBIOS and QEMU don't know how to do Sbus probing switch(slot) { case 2: // SUNW,tcx ob_tcx_init(slot, "/iommu/sbus/SUNW,tcx");

9 years, 5 months

Re: [OpenBIOS] adding eepom node to sparc64

by Blue Swirl

On Sat, May 19, 2012 at 4:38 PM, Artyom Tarasenko <atar4qemu(a)gmail.com> wrote: > Both Linux and MilaX complain about the missing eeprom / rtc-m48t59 node. > Since QEMU actually has this device I tried to add it, but not sure > where is it located. > In sun4u.c it's initialized like this: > > nvram = m48t59_init_isa(isa_bus, 0x0074, NVRAM_SIZE, 59); > > Is 0x74 the offset to the ISA bus? Then it's wired not as on Ultra-5: > > Node 0xf00662d8 > address: fffba000 > reg: 00000014.00000000.00002000 > model: 'mk48t59' > name: 'eeprom' > > Here it looks like it had to be aligned to a beginning of a page, > which makes sense because of the large size... > So, where QEMU's 0x74 are coming from? It's the I/O port used by PPC. Looking at the reg, it should be using MMIO at address 0 instead. > > -- > Regards, > Artyom Tarasenko > > solaris/sparc under qemu blog: http://tyom.blogspot.com/search/label/qemu

9 years, 5 months

adding eepom node to sparc64

by Artyom Tarasenko

Both Linux and MilaX complain about the missing eeprom / rtc-m48t59 node. Since QEMU actually has this device I tried to add it, but not sure where is it located. In sun4u.c it's initialized like this: nvram = m48t59_init_isa(isa_bus, 0x0074, NVRAM_SIZE, 59); Is 0x74 the offset to the ISA bus? Then it's wired not as on Ultra-5: Node 0xf00662d8 address: fffba000 reg: 00000014.00000000.00002000 model: 'mk48t59' name: 'eeprom' Here it looks like it had to be aligned to a beginning of a page, which makes sense because of the large size... So, where QEMU's 0x74 are coming from? -- Regards, Artyom Tarasenko solaris/sparc under qemu blog: http://tyom.blogspot.com/search/label/qemu

9 years, 5 months

[PATCH] Identify virtio-net devices

by Artyom Tarasenko

Identify virtio network device in PCI probe. Signed-off-by: Artyom Tarasenko <atar4qemu(a)gmail.com> --- drivers/pci_database.c | 8 ++++++++ include/drivers/pci.h | 1 + 2 files changed, 9 insertions(+), 0 deletions(-) diff --git a/drivers/pci_database.c b/drivers/pci_database.c index 1d2958c..197f27e 100644 --- a/drivers/pci_database.c +++ b/drivers/pci_database.c @@ -129,6 +129,14 @@ static const pci_dev_t eth_devices[] = { NULL, "ethernet", }, { + /* Virtio-network controller */ + PCI_VENDOR_ID_REDHAT_QUMRANET, PCI_DEVICE_ID_VIRTIO_NET, + NULL, "virtio-net", NULL, + "pci1af4,1000\0pci1af4,1000\0pciclass,020000\0", + 0, 0, 0, + virtio_config_cb, NULL, + }, + { 0xFFFF, 0xFFFF, NULL, NULL, NULL, NULL, -1, -1, -1, diff --git a/include/drivers/pci.h b/include/drivers/pci.h index 4c996c0..d3b290a 100644 --- a/include/drivers/pci.h +++ b/include/drivers/pci.h @@ -206,6 +206,7 @@ extern const pci_arch_t *arch; #define PCI_DEVICE_ID_QEMU_VGA 0x1111 #define PCI_VENDOR_ID_REDHAT_QUMRANET 0x1af4 +#define PCI_DEVICE_ID_VIRTIO_NET 0x1000 #define PCI_DEVICE_ID_VIRTIO_BLOCK 0x1001 #define PCI_VENDOR_ID_INTEL 0x8086 -- 1.7.1

9 years, 5 months

[commit] r1059 - trunk/openbios-devel/drivers

by repository service

Author: blueswirl Date: Sat May 19 12:38:26 2012 New Revision: 1059 URL: http://tracker.coreboot.org/trac/openbios/changeset/1059 Log: Sparc64: fix interrupt properties Add interrupt-map and interrupt-mask properties to host and ebus bridges and sprinkle interrupt properties to various devices. Based on hacks by Artyom Tarasenko. Signed-off-by: Blue Swirl <blauwirbel(a)gmail.com> Modified: trunk/openbios-devel/drivers/ide.c trunk/openbios-devel/drivers/pc_serial.c trunk/openbios-devel/drivers/pci.c trunk/openbios-devel/drivers/pci_database.c trunk/openbios-devel/drivers/pci_database.h Modified: trunk/openbios-devel/drivers/ide.c ============================================================================== --- trunk/openbios-devel/drivers/ide.c Sat May 19 12:38:24 2012 (r1058) +++ trunk/openbios-devel/drivers/ide.c Sat May 19 12:38:26 2012 (r1059) @@ -1432,6 +1432,8 @@ props[0]=14; props[1]=0; set_property(dnode, "interrupts", (char *)&props, 2*sizeof(props[0])); +#elif defined(CONFIG_SPARC64) + set_int_property(get_cur_dev(), "interrupts", 0); #endif props[0] = __cpu_to_be32(chan->io_regs[0]); Modified: trunk/openbios-devel/drivers/pc_serial.c ============================================================================== --- trunk/openbios-devel/drivers/pc_serial.c Sat May 19 12:38:24 2012 (r1058) +++ trunk/openbios-devel/drivers/pc_serial.c Sat May 19 12:38:26 2012 (r1059) @@ -190,6 +190,9 @@ fword("encode+"); push_str("reg"); fword("property"); +#if defined(CONFIG_SPARC64) + set_int_property(get_cur_dev(), "interrupts", 1); +#endif aliases = find_dev("/aliases"); set_property(aliases, "ttya", nodebuff, strlen(nodebuff) + 1); Modified: trunk/openbios-devel/drivers/pci.c ============================================================================== --- trunk/openbios-devel/drivers/pci.c Sat May 19 12:38:24 2012 (r1058) +++ trunk/openbios-devel/drivers/pci.c Sat May 19 12:38:26 2012 (r1059) @@ -401,6 +401,21 @@ props[PCI_INT_MAP_PCI_INT] = 0x7; set_property(dev, "interrupt-map-mask", (char *)props, 4 * sizeof(props[0])); +#elif defined(CONFIG_SPARC64) + phandle_t dev = get_cur_dev(); + uint32_t props[5]; + + props[0] = 0x000001fe; + props[1] = 0x020003f8; + props[2] = 1; + props[3] = find_dev("/"); + props[4] = 0x2b; + set_property(dev, "interrupt-map", (char *)props, 5 * sizeof(props[0])); + + props[0] = 0x000001ff; + props[1] = 0xffffffff; + props[2] = 3; + set_property(dev, "interrupt-map-mask", (char *)props, 3 * sizeof(props[0])); #endif } @@ -522,6 +537,14 @@ return 0; } +int virtio_config_cb(const pci_config_t *config) +{ +#if defined(CONFIG_SPARC64) + set_int_property(get_cur_dev(), "interrupts", 0); +#endif + return 0; +} + int ide_config_cb2 (const pci_config_t *config) { ob_ide_init(config->path, @@ -756,6 +779,21 @@ int ebus_config_cb(const pci_config_t *config) { #ifdef CONFIG_DRIVER_EBUS + phandle_t dev = get_cur_dev(); + uint32_t props[5]; + + props[0] = 0x000001fe; + props[1] = 0x020003f8; + props[2] = 1; + props[3] = find_dev("/"); + props[4] = 0x2b; + set_property(dev, "interrupt-map", (char *)props, 5 * sizeof(props[0])); + + props[0] = 0x000001ff; + props[1] = 0xffffffff; + props[2] = 3; + set_property(dev, "interrupt-map-mask", (char *)props, 3 * sizeof(props[0])); + #ifdef CONFIG_DRIVER_FLOPPY ob_floppy_init(config->path, "fdthree", 0x3f0ULL, 0); #endif Modified: trunk/openbios-devel/drivers/pci_database.c ============================================================================== --- trunk/openbios-devel/drivers/pci_database.c Sat May 19 12:38:24 2012 (r1058) +++ trunk/openbios-devel/drivers/pci_database.c Sat May 19 12:38:26 2012 (r1059) @@ -52,7 +52,7 @@ NULL, "virtio-scsi", NULL, "pci1af4,1001\0pci1af4,1001\0pciclass,01018f\0", 0, 0, 0, - NULL, NULL, + virtio_config_cb, NULL, }, { 0xFFFF, 0xFFFF, Modified: trunk/openbios-devel/drivers/pci_database.h ============================================================================== --- trunk/openbios-devel/drivers/pci_database.h Sat May 19 12:38:24 2012 (r1058) +++ trunk/openbios-devel/drivers/pci_database.h Sat May 19 12:38:26 2012 (r1059) @@ -28,6 +28,7 @@ const void *private; }; +extern int virtio_config_cb(const pci_config_t *config); extern int ide_config_cb2(const pci_config_t *config); extern int eth_config_cb(const pci_config_t *config); extern int macio_heathrow_config_cb(const pci_config_t *config);

9 years, 5 months