[libvirt] [PATCH 2/3] virfiletest: Test virFileInData iff SEEK_HOLE is defined

Michal Privoznik mprivozn at redhat.com
Fri May 19 11:31:41 UTC 2017


On 05/19/2017 12:36 PM, Peter Krempa wrote:
> On Thu, May 18, 2017 at 15:46:45 +0200, Michal Privoznik wrote:
>> Yet another place where we need to wrap code in
>> HAVE_DECL_SEEK_HOLE block.
>>
>> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
>> ---
>>  tests/virfiletest.c | 36 +++++++++++++++++++++++-------------
>>  1 file changed, 23 insertions(+), 13 deletions(-)
>>
>> diff --git a/tests/virfiletest.c b/tests/virfiletest.c
>> index a93bee01a..d6a526c13 100644
>> --- a/tests/virfiletest.c
>> +++ b/tests/virfiletest.c
>> @@ -123,7 +123,11 @@ static int
>>  makeSparseFile(const off_t offsets[],
>>                 const bool startData);
>>  
>> -#ifdef __linux__
>> +static bool
>> +holesSupported(void);
>> +
>> +#if HAVE_DECL_SEEK_HOLE
> 
> Also this stubbed out code is now dependant on HAVE_DECL_SEEK_HOLE, but
> the next patch includes file necessary for FALLOC_FL_PUNCH_HOLE only if
> __linux__ is defined. So you probably want  HAVE_DECL_SEEK_HOLE &&
> __linux__
> 

Okay.

Michal




More information about the libvir-list mailing list