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@xxxxxxxxxx +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list