Re: [Qemu-devel] [PATCH] Reorganize option rom (+linux kernel) loading.

[Blue Swirl]

On Thu, Oct 1, 2009 at 12:11 PM, Gerd Hoffmann <address@hidden> wrote:
>  Hi,
>
>> Nice idea. The implementation seems to be buggy, I only ever see one
>> rom
>
> On !pc platforms only elf roms will show up.
>
>> and for example sparc-test crashes when issuing 'info roms'.
>
> Huh?  Works for me.  Details please.  sparc is one of the archs I've
> actually tested:

OK, V2 seems to work, except that the command line is not passed:
NET: Registered protocol family 1
NET: Registered protocol family 17
Kernel panic - not syncing: VFS: Unable to mount root fs on unknown-block(1,0)
 <0>Press L1-A to return to the boot prom

>> Perhaps 'info roms' could also check whether the rom has been changed?
>
> Point being?

Could be useful information. Maybe not.

>> On most platforms roms are really read only memories.
>
> Yep, I've noticed on a quick glimpse.  So in that case writing roms on reset
> isn't very useful because the guest shouldn't be able to modify them in the
> first place.  It doesn't harm though, except that we keep a copy of the roms
> around for no reason.  Maybe make restore-on-reset opt-in?

Or make writability opt-in feature?