[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 30/30] target-sparc: fix up niagara machine
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v2 30/30] target-sparc: fix up niagara machine |
Date: |
Fri, 27 Jan 2017 15:27:26 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 |
On 27/01/2017 15:06, Markus Armbruster wrote:
>> Is rom_add_file_fixed supposed to mark the memory region read-only? Or
>> should it be created read-only before calling rom_add_file_fixed?
> I'm not familiar with the memory API, but the fact that you're getting
> RW suggests that you need to mark read-only yourself. Suggest to
> examine similar code elsewhere for additional clues.
>
>>> The monkey-patching of dinfo->is_default is to silence the "Orphaned
>>> drive without device" warning. Naughty.
>>>
>>> This use of -drive if=pflash is highly unorthodox. To suggest better
>>> ways, we need to understand the role of "the virtual ramdisk". Can you
>>> explain? How does it work on a physical system?
>> AFAIK except for simulators it's only used on FPGA boards. It's what
>> the comment says: a kind of initrd, but it resides outside of the
>> partition RAM. So, it is a RAM region pre-filled with data in the way
>> not visible to the guest.
>>
>> I would have used an -initrd option instead of -pflash, but it
>> required a -kernel option, and there is no sensible way to use the
>> -kernel with Solaris.
>> If QEMU had something like a nvram-disk - it would have worked here the best.
>
> Have you considered pressing -bios into service? -option-rom? -object
> memory-backend-file? Paolo, any recommendations?
Yeah, I think -bios is closest. By the way, where do I get all the .rom
files for the Niagara board? "make check" complains about them.
Paolo
- [Qemu-devel] [PATCH v2 22/30] target-sparc: allow 256M sized pages, (continued)
- [Qemu-devel] [PATCH v2 22/30] target-sparc: allow 256M sized pages, Artyom Tarasenko, 2017/01/11
- [Qemu-devel] [PATCH v2 23/30] target-sparc: implement auto-demapping for UA2005 CPUs, Artyom Tarasenko, 2017/01/11
- [Qemu-devel] [PATCH v2 24/30] target-sparc: add more registers to dump_mmu, Artyom Tarasenko, 2017/01/11
- [Qemu-devel] [PATCH v2 27/30] target-sparc: add ST_BLKINIT_ ASIs for UA2005+ CPUs, Artyom Tarasenko, 2017/01/11
- [Qemu-devel] [PATCH v2 25/30] target-sparc: implement UA2005 ASI_MMU (0x21), Artyom Tarasenko, 2017/01/11
- [Qemu-devel] [PATCH v2 28/30] target-sparc: implement sun4v RTC, Artyom Tarasenko, 2017/01/11
- [Qemu-devel] [PATCH v2 30/30] target-sparc: fix up niagara machine, Artyom Tarasenko, 2017/01/11
[Qemu-devel] [PATCH v2 29/30] target-sparc: move common cpu initialisation routines to sparc64.c, Artyom Tarasenko, 2017/01/11