qemu-block
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-block] [PATCH 1/1] block: add gluster ifdef guard checks for S


From: Eric Blake
Subject: Re: [Qemu-block] [PATCH 1/1] block: add gluster ifdef guard checks for SEEK_DATA/SEEK_HOLE support
Date: Fri, 7 Oct 2016 09:20:54 -0500
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0

On 10/06/2016 10:53 PM, Jeff Cody wrote:
> Add checks to see if the system compiling QEMU has support for
> SEEK_HOLE/SEEK_DATA.  If the system does not, we will flag that seek
> data is unsupported in gluster.
> 
> Note: this is not a check on whether the gluster server itself supports
> SEEK_DATA (that is already done during runtime), but rather if the
> compilation environment supports SEEK_DATA.
> 
> Signed-off-by: Jeff Cody <address@hidden>
> ---
> 
> Note: this patch is untested on older systems that do not support SEEK_DATA
> (e.g.. RHEL6).  This won't be pulled into my tree until it is verified.
> 

  CC      block/gluster.o
block/gluster.c: In function ‘qemu_gluster_test_seek’:
block/gluster.c:676: warning: unused variable ‘eof’
block/gluster.c: In function ‘find_allocation’:
block/gluster.c:1188: warning: unused variable ‘offs’

but compilation succeeded in spite of the warning.  Would be nice to
clean that up for -Werror users.

Tested-by: Eric Blake <address@hidden>
Reviewed-by: Eric Blake <address@hidden>

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

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]