> >>THis looks also problematic. What if someone sets hotplug=0? This also
> >>use wrong values if the VM is migrated?
> 
> I manage this inside deviceplug|unplug.
> (I remove the pending device at the end)
> 
> 
> sub vm_deviceplug {
> ....
>    return 1 if !$conf->{hotplug};
> ...
>    #delete pending device after hotplug
>     if($conf->{pending}->{$deviceid}){
>         $conf->{$deviceid} = $optvalue;
>         delete $conf->{pending}->{$deviceid};
>         PVE::QemuServer::update_config_nolock($vmid, $conf, 1);
>     }
> }

What is someone wants:

hotplug: 0

Then you want to keep changes in [PENDING], without applying them.

_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to