[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader |
Date: |
Wed, 21 Sep 2016 16:54:48 +0100 |
User-agent: |
Mutt/1.7.0 (2016-08-17) |
On Wed, Sep 21, 2016 at 08:05:23AM +0200, Markus Armbruster wrote:
> Alistair Francis <address@hidden> writes:
>
> > This work is based on the original work by Li Guang with extra
> > features added by Peter C and myself.
> >
> > The idea of this loader is to allow the user to load multiple images
> > or values into QEMU at startup.
> >
> > Memory values can be loaded like this: -device
> > loader,addr=0xfd1a0104,data=0x8000000e,data-len=4
> >
> > Images can be loaded like this: -device
> > loader,file=./images/u-boot.elf,cpu=0
>
> I know it's way too late for design questions, but the thought just
> occured to me: -device gives you what you need without defining yet
> another command line option (good!), but is it appropriate? It's not
> exactly a device... Would -object be a better fit? I honestly don't
> know. Paolo?
Yes, -object would be better. Basically -device is used to create stuff
which is related to guest ABI, while -object is used to create anything
else, which is basically host integration stuff.
Loading firmeware images is clearly host integration, so belongs in
-object
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
- Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader, (continued)
- Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader, Paolo Bonzini, 2016/09/21
- Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader, Markus Armbruster, 2016/09/22
- Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader, Paolo Bonzini, 2016/09/22
- Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader, Markus Armbruster, 2016/09/22
- Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader, Peter Maydell, 2016/09/22
- Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader, Markus Armbruster, 2016/09/23
- Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader, Paolo Bonzini, 2016/09/23
- Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader, Markus Armbruster, 2016/09/27
- Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader, Paolo Bonzini, 2016/09/27
- Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader, Peter Maydell, 2016/09/27
Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader,
Daniel P. Berrange <=