[libvirt] [PATCH 05/10] conf: Introduce new attribute for device address format
Osier Yang
jyang at redhat.com
Fri Feb 17 16:54:20 UTC 2012
On 2012年02月18日 00:41, Paolo Bonzini wrote:
> On 02/17/2012 06:18 PM, Osier Yang wrote:
>> + if ((bus == VIR_DOMAIN_DISK_BUS_SCSI)&&
>> + (controllerModel != VIR_DOMAIN_CONTROLLER_MODEL_SCSI_LSILOGIC)) {
>> + virBufferAsprintf(buf, " controller='%d' bus='%d' target='%d' unit='%d'",
>> + info->addr.drive.controller,
>> + info->addr.drive.bus,
>> + info->addr.drive.target,
>> + info->addr.drive.unit);
>> + } else {
>> + virBufferAsprintf(buf, " controller='%d' bus='%d' unit='%d'",
>> + info->addr.drive.controller,
>> + info->addr.drive.bus,
>> + info->addr.drive.unit);
>> + }
>
> 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.
Osier
More information about the libvir-list
mailing list