qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH v4 03/23] qdev: Drop ID matching from qtree path


From: Jan Kiszka
Subject: Re: [Qemu-devel] [PATCH v4 03/23] qdev: Drop ID matching from qtree paths
Date: Wed, 23 Jun 2010 12:17:55 +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

Markus Armbruster wrote:
> [cc: kraxel]
> 
> Jan Kiszka <address@hidden> writes:
> 
>> From: Jan Kiszka <address@hidden>
>>
>> Device IDs may conflict with device names or aliases. From now on we
>> only accept them outside qtree paths. This also makes dumping IDs in
>> qbus_list_dev/bus obsolete.
>>
>> Signed-off-by: Jan Kiszka <address@hidden>
> 
> I don't like this at all.
> 
> 1. Specific problem:
> 
>    With the current code, multiple devices with the same driver work
>    only if I take care: addressing by driver name gets me only the
>    first, so I better set suitable IDs.
> 
>    With your patch, multiple devices with the same driver don't work, no
>    matter what I do.

[ you already found how this is resolved ]

> 
> 2. General principle:
> 
>    When I set an ID, I want the system to accept that ID in all contexts
>    where it makes sense.  Ambiguity created by badly chosen IDs is *my*
>    problem.

I disagree. IMO, QEMU should support the user to avoid such subtle
shadowing.

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]