-
Notifications
You must be signed in to change notification settings - Fork 176
Closed
Description
When using vmstoragemodel to specify the stroagemodel for a diskfile, it will be replaced by 'scsi' for a ppc64 or ppc64le hypervisor.
# lsdef testvm -i vmstoragemodel
Object name: testvm
vmstoragemodel=virtio
<disk type='file' device='disk'>
<driver name='qemu' type='qcow2' cache='none'/>
<source file='/var/lib/libvirt/images/testvm.sda.qcow2'/>
<backingStore/>
<target dev='sda' bus='scsi'/> ==> Shall be dev='vda' and bus='virtio'
<alias name='scsi0-0-0-0'/>
<address type='drive' controller='0' bus='0' target='0' unit='0'/>
</disk>
It is wrongly modified within our code when it detect the hypervisor is a ppc64/ppc64le model. Changing to 'ppc64$' only will work.
if ($confdata->{$hyp}->{cpumodel} and $confdata->{$hyp}->{cpumodel} =~ /ppc64/i) {
$confdata->{vm}->{$node}->[0]->{storagemodel} = "scsi";
}
It is approved that 'virtio' also works for rhev4.x on ppc64le hypervisor, so need more test for vm related functions.