[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qdev: obey no_user
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH] qdev: obey no_user |
Date: |
Mon, 07 Jan 2013 23:10:20 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/17.0 Thunderbird/17.0 |
Am 07.01.2013 21:16, schrieb Peter Maydell:
> On 7 January 2013 20:12, Anthony Liguori <address@hidden> wrote:
>> Peter Maydell <address@hidden> writes:
>>> It seems to me like arbitrarily allowing the monitor to construct
>>> no-user devices isn't really the right way to attack the problem
>>> of "allow complete machine construction by management tools"...
>>
>> There is no such thing as a 'no-user' device. It's a silly distinction
>> that has never had a consistent meaning.
>
> Then let's just rip that flag out completely.
That's a bad idea, given that we are about to make the CPU a device.
So yes, there are devices that are not meant to be instantiated more
than once (e.g., sclp-console). And there are devices that are not meant
to be instantiated by the user at all.
BTW this patch exposed that there is silly code in this function: When
the bus is NULL, it is initialized as the SysBus. But we all know that
SysBus is not hotplug-capable in the first place, so we can never
device_add on SysBus.
Regards,
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
- Re: [Qemu-devel] [PATCH] qdev: obey no_user, Christian Borntraeger, 2013/01/07
- Re: [Qemu-devel] [PATCH] qdev: obey no_user, Andreas Färber, 2013/01/07
- Re: [Qemu-devel] [PATCH] qdev: obey no_user, Anthony Liguori, 2013/01/07
- Re: [Qemu-devel] [PATCH] qdev: obey no_user, Peter Maydell, 2013/01/07
- Re: [Qemu-devel] [PATCH] qdev: obey no_user, Anthony Liguori, 2013/01/07
- Re: [Qemu-devel] [PATCH] qdev: obey no_user, Peter Maydell, 2013/01/07
- Re: [Qemu-devel] [PATCH] qdev: obey no_user, Anthony Liguori, 2013/01/07
- Re: [Qemu-devel] [PATCH] qdev: obey no_user,
Andreas Färber <=
- Re: [Qemu-devel] [PATCH] qdev: obey no_user, Peter Maydell, 2013/01/07
- Re: [Qemu-devel] [PATCH] qdev: obey no_user, Andreas Färber, 2013/01/07
- Re: [Qemu-devel] [PATCH] qdev: obey no_user, Anthony Liguori, 2013/01/07
Re: [Qemu-devel] [PATCH] qdev: obey no_user, Christian Borntraeger, 2013/01/07