[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader |
Date: |
Wed, 21 Sep 2016 17:53:29 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0 |
On 21/09/2016 17:46, Alistair Francis wrote:
>> > 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?
> I see your point, but I kind of think it makes sense that everything
> uses the same command line argument.
>
> I image it would be very confusing if we have -device and -object.
> Then when you want to add something you will need to figure out if it
> is a device or an object? How do you know which one is which?
>
> I agree that technically it isn't a device but I think this is still
> clear what you are trying to do.
I think -device is okay for something that isn't a "backend" but is
directly guest-visible.
Paolo
- [Qemu-devel] [PATCH v11 7/8] generic-loader: Add a generic loader, (continued)
- [Qemu-devel] [PATCH v11 7/8] generic-loader: Add a generic loader, Alistair Francis, 2016/09/20
- [Qemu-devel] [PATCH v11 1/8] loader: Allow ELF loader to auto-detect the ELF arch, Alistair Francis, 2016/09/20
- [Qemu-devel] [PATCH v11 6/8] loader: Add AddressSpace loading support to targphys, Alistair Francis, 2016/09/20
- [Qemu-devel] [PATCH v11 2/8] loader: Use the specified MemoryRegion, Alistair Francis, 2016/09/20
- [Qemu-devel] [PATCH v11 3/8] loader: Allow a custom AddressSpace when loading ROMs, Alistair Francis, 2016/09/20
- [Qemu-devel] [PATCH v11 8/8] docs: Add a generic loader explanation document, Alistair Francis, 2016/09/20
- Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader, Peter Maydell, 2016/09/20
- Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader, Markus Armbruster, 2016/09/21
- Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader, Alistair Francis, 2016/09/21
- Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader,
Paolo Bonzini <=
- 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, Markus Armbruster, 2016/09/27
- Re: [Qemu-devel] [PATCH v11 0/8] Add a generic loader, Alistair Francis, 2016/09/27