[libvirt] [PATCH 05/10] conf: Introduce new attribute for device address format
jyang at redhat.com
Fri Feb 17 17:11:14 UTC 2012
On 2012年02月18日 01:03, Paolo Bonzini wrote:
> On 02/17/2012 05:54 PM, Osier Yang wrote:
>>> Yeah, I definitely misunderstood. Here it is correct to print the
>>> target for all models.
>> Do you mean it's correct check the model here? I think the "lsilogic"
>> model won't want to see the "target". It's complete useless.
> It's true that it's useless. However, there is no reason not to print
> it; so is "bus", for example. Also, even for non-SCSI disks you need
> to print the target. You may omit it if it is zero, but I'm not sure
> it brings any benefit to do so.
Finally I understand you, yeah, it's no harm to print the target. Except
we need another patch to change lots of the tests.
> By the way, you need to test for "target == 0" and print an error also
> when creating an IDE or AHCI disk.
More information about the libvir-list