[pve-devel] [PATCH v2 qemu-server 3/5] rename foreach_writable_storage to foreach_storage_used_by_vm

Dietmar Maurer dietmar at proxmox.com
Mon May 15 17:30:08 CEST 2017


Are you sure this name is better? The function does not really
return all storages used by the VM, because it skips storages
used in 'read-only' mode (CD roms). AFAIK that is why it's called
'foreach_writable_storage'.


> -sub foreach_writable_storage {
> +sub foreach_storage_used_by_vm {




More information about the pve-devel mailing list