qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 00/14] qdev: assign unique names to all devices


From: Jan Kiszka
Subject: Re: [Qemu-devel] [PATCH 00/14] qdev: assign unique names to all devices (part 1)
Date: Tue, 20 Sep 2011 10:32:53 +0200
User-agent: Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666

On 2011-09-19 16:24, Kevin Wolf wrote:
> Am 19.09.2011 16:05, schrieb Anthony Liguori:
>> On 09/19/2011 02:26 AM, Jan Kiszka wrote:
>>> On 2011-09-16 20:03, Anthony Liguori wrote:
>>>> So this is a simplification that I plan on running with.  For now, I think 
>>>> this
>>>> series is the right next step because it gives us a path name for the name
>>>> (although different syntax) and let's us enforce that all devices has a
>>>> canonical path.
>>>
>>> For something that changes lots of devices and, at the same time, is
>>> going to be removed again, I'm hesitating to call it the right direction.
>>>
>>> A right step would be, IMHO, to introduce a generic introspectable
>>> device link so that parent devices can reference their children and a
>>> visitor can derive a child's relative name from that link name. Then
>>> make sure this link type is consistently used.
>>>
>>> I really dislike this focusing on assigning names internally and using
>>> them in QEMU-internal APIs. They should just fall out of the core when
>>> external interaction is required.
>>
>> I thought a lot about this over the weekend and decided that I should go in 
>> a 
>> different direction based on this discussion.
>> [...]
> 
> Sounds like a good direction to me. (At least until someone brings up
> the details :-))

Yeah. Maybe we should keep this at concept level to avoid getting into
disagreement again. ;)

Jan

-- 
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux



reply via email to

[Prev in Thread] Current Thread [Next in Thread]