[libvirt] [PATCH v2 0/7] storage:dir: ploop volumes support

[Maxim Nestratov]

08.02.2016 16:04, Olga Krishtal пишет:
> In-Reply-To:
>
> v2:
> - fixed memory leak
> - chenged the return value of all helper functions to 0/-1.
> Now check for success is smth like that: vir****Ploop() < 0
> - fixed some identation issues.
>

Overall looks good, except minor issues in some patches. Also, I wonder 
if we should add some check to ensure ploop binary existence?
This would make user experience much better. For instance, instead of 
unconditionally returning virStorageBackendCreatePloop in case of ploop 
volumes in virStorageBackendGetBuildVolFromFunction we could check for 
ploop external tool and report an error message that it wasn't detected 
and it is required to be installed for PLOOP storage pools.

Maxim Nestratov