[libvirt] [PATCH v2] storage: Allow runtime detection of scrub

Eric Blake eblake at redhat.com
Wed Feb 15 22:45:28 UTC 2012


On 02/15/2012 06:37 AM, Michal Privoznik wrote:
> Currently, if scrub (used for wiping algorithms) is not present
> at compile time, we don't support any other wiping algorithms than
> zeroing, even if it was installed later. Switch to runtime detection
> instead.
> ---
> diff to v1:
> -Add BuildRequires to spec file
> 
>  configure.ac                 |   30 ++++--------------------------
>  libvirt.spec.in              |    3 +++
>  src/storage/storage_driver.c |    4 ----
>  3 files changed, 7 insertions(+), 30 deletions(-)

> +++ b/libvirt.spec.in
> @@ -513,6 +513,9 @@ BuildRequires: nfs-utils
>  # Fedora build root suckage
>  BuildRequires: gawk
>  
> +# For storage wiping with different algorithms
> +BuildRequires: scrub
> +

I'm wondering if this should be protected by a conditional, such as the
%if %{with_storage_fs} conditional a few lines earlier.  Then again,
that example a few lines earlier was for $UMOUNT, used only in
src/storage/storage_backend_fs.c, which is conditional, while your use
of $SCRUB is in src/storage/storage_driver.c which is unconditionally
built.  So I think you're okay.

ACK.

-- 
Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120215/67357e8c/attachment-0001.sig>


More information about the libvir-list mailing list