[libvirt] [PATCH 05/10] conf: Introduce new attribute for device address format
pbonzini at redhat.com
Fri Feb 17 17:03:04 UTC 2012
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.
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