[libvirt] [PATCH 1/2] Add VSCSI bus type and VSCSI controller type for pseries guest.
Eric Blake
eblake at redhat.com
Wed Jun 6 22:45:37 UTC 2012
On 06/04/2012 09:16 AM, Cole Robinson wrote:
>>> This isn't exactly what I had in mind when I recommended a libvirt patch.
>>> Change libvirt to add:
>>>
>>> <controller type='scsi' index='0'>
>>> <address type='spapr-vio'/>
>>> </controller>
>>>
>>> when the guest XML has
>>>
>>> <disk type='file' device='disk'>
>>> <driver name='qemu' type='raw'/>
>>> <source file='/path/file'/>
>>> <target dev='sda' bus='scsi'/>
>>> <address type='spapr-vio'/>
>>> </disk>
>>>
>>> and the guest XML doesn't already have spapr-vio<controller>.
>>
>> Thanks a lot for your suggestion. :)
>>
>> I see the libvirt will always set the address type of the disk as
>> "drive". There is one drive address structure in libvirt, which
>> records the controller, bus and unit.
It sounds like we need <disk> address structure to support more than
just drive addressing, but to instead support a union of addressing
based on the type='' attribute.
>>
>> the address can be specified in the target.
>>
>> <disk type='file' device='disk'>
>> <driver name='qemu' type='raw'/>
>> <source file='/path/file'/>
>> <target dev='sda' bus='scsi' address='spapr-vio'/>
No, I don't think you want address added to <target>, but...
>> <address type='drive' controller='0' bus='0' unit='0' />
you instead want <address type='spapr-vio'/> to be something you add to
your XML to force libvirt to then populate the remaining attributes and
auto-instantiate the controller.
>> </disk>
>>
>
> That sounds okay to me, bug maybe eblake or danpb should sign off on it
> conceptually, since it's a new XML element.
>
> - Cole
>
>
--
Eric Blake eblake at redhat.com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120606/6eca4255/attachment-0001.sig>
More information about the libvir-list
mailing list