[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 30/30] target-sparc: fix up niagara machine
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PULL 30/30] target-sparc: fix up niagara machine |
Date: |
Mon, 23 Jan 2017 14:24:40 +0000 |
On 23 January 2017 at 14:10, Artyom Tarasenko <address@hidden> wrote:
> On Mon, Jan 23, 2017 at 1:40 PM, Peter Maydell <address@hidden> wrote:
>> I see that 'make check' now warns:
>> GTESTER check-qtest-sparc64
>> Could not open option rom 'nvram1': No such file or directory
>> Could not open option rom '1up-md.bin': No such file or directory
>> Could not open option rom '1up-hv.bin': No such file or directory
>> Could not open option rom 'reset.bin': No such file or directory
>> Could not open option rom 'q.bin': No such file or directory
>> Could not open option rom 'openboot.bin': No such file or directory
>>
>> (though the tests still pass).
>>
>> Could we either ship these images in pc-bios if they're
>> necessary, or not complain that they don't exist if they're
>> not necessary, please?
>
> I wonder what would be the best option here. The images are
> necessary, so the last option - not complaining - can be misleading
> for a user.
If they're actually necessary then perhaps we should refuse
to start entirely?
> Concerning shipping them.
> Pros:
> - the images are obviously freely distributable (the link above).
> - the corresponding source code was open-sourced by Sun under various
> licenses (GPL for hypervisor, BSD for openboot).
> Cons:
> - there is no exact tag the the OpenSPARC source tree which would
> correspond to the binaries.
> - building them is tricky, because it requires Solaris 9 / SPARC.
>
> What do you think would be a better option?
One thing we could do is only warn if !qtest_enabled().
We do this for some other boards that otherwise fail entirely
when their BIOS image is not present. This is sufficient for
the qtest checks which don't actually try to run code on the
guest, but merely interact with it via the qtest protocol.
We do ship some other ROMs that are only buildable on the
right host hardware, so it's not impossible, but I don't know
the details of our rules about what we put in pc-bios/.
thanks
-- PMM
- [Qemu-devel] [PULL 21/30] target-sparc: simplify ultrasparc_tsb_pointer, (continued)
- [Qemu-devel] [PULL 21/30] target-sparc: simplify ultrasparc_tsb_pointer, Artyom Tarasenko, 2017/01/18
- [Qemu-devel] [PULL 22/30] target-sparc: allow 256M sized pages, Artyom Tarasenko, 2017/01/18
- [Qemu-devel] [PULL 24/30] target-sparc: add more registers to dump_mmu, Artyom Tarasenko, 2017/01/18
- [Qemu-devel] [PULL 25/30] target-sparc: implement UA2005 ASI_MMU (0x21), Artyom Tarasenko, 2017/01/18
- [Qemu-devel] [PULL 23/30] target-sparc: implement auto-demapping for UA2005 CPUs, Artyom Tarasenko, 2017/01/18
- [Qemu-devel] [PULL 27/30] target-sparc: add ST_BLKINIT_ ASIs for UA2005+ CPUs, Artyom Tarasenko, 2017/01/18
- [Qemu-devel] [PULL 28/30] target-sparc: implement sun4v RTC, Artyom Tarasenko, 2017/01/18
- [Qemu-devel] [PULL 30/30] target-sparc: fix up niagara machine, Artyom Tarasenko, 2017/01/18
- Re: [Qemu-devel] [PULL 30/30] target-sparc: fix up niagara machine, Jakub Jermář, 2017/01/27
[Qemu-devel] [PULL 29/30] target-sparc: move common cpu initialisation routines to sparc64.c, Artyom Tarasenko, 2017/01/18
Re: [Qemu-devel] [PULL 00/30] target-sparc sun4v support, Peter Maydell, 2017/01/19