[PATCH] Fix Linux fcntl OFD locks on unsupported kernels

[Adhemerval Zanella]

On 27/07/2018 10:27, Florian Weimer wrote:
> On 07/27/2018 03:12 PM, Adhemerval Zanella wrote:
>> This patch make the OFD tests return unsupported if kernel does not
>> support OFD locks (it was added on 3.15).
>>
>> Checked on a ia64-linux-gnu with Linux 3.14.
>>
>> Â Â Â Â * sysdeps/unix/sysv/linux/tst-ofdlocks.c: Return unsupported if
>> Â Â Â Â kernel does not support OFD locks.
>> Â Â Â Â * sysdeps/unix/sysv/linux/tst-ofdlocks-compat.c: Likewise.
>> ---
>>   ChangeLog                                     | 6 ++++++
>> Â  sysdeps/unix/sysv/linux/tst-ofdlocks-compat.c | 7 ++++++-
>>   sysdeps/unix/sysv/linux/tst-ofdlocks.c        | 7 ++++++-
>> Â  3 files changed, 18 insertions(+), 2 deletions(-)
>>
>> diff --git a/sysdeps/unix/sysv/linux/tst-ofdlocks-compat.c b/sysdeps/unix/sysv/linux/tst-ofdlocks-compat.c
>> index 03c4abf..8da5a0b 100644
>> --- a/sysdeps/unix/sysv/linux/tst-ofdlocks-compat.c
>> +++ b/sysdeps/unix/sysv/linux/tst-ofdlocks-compat.c
>> @@ -62,7 +62,12 @@ do_test (void)
>>       .l_start  = (off64_t)INT32_MAX + 1024,
>>       .l_len    = 1024,
>> Â Â Â  };
>> -Â  TEST_VERIFY_EXIT (fcntl (temp_fd, F_OFD_SETLKW, &lck64) == 0);
>> +Â  int ret = fcntl (temp_fd, F_OFD_SETLKW, &lck64);
>> +Â  if (ret == -1 && errno == EINVAL)
>> +    /* OFD locks are only available on Linux 3.15.  */
>> +Â Â Â  FAIL_UNSUPPORTED ("fcntl (F_OFD_SETLKW) not supported)");
> 
> Should use %m.

This does not add much information, but it does not hurt either.

> 
>> Â Â Â  /* Open file description locks placed through the same open file description
>> Â Â Â Â Â Â  (either by same file descriptor or a duplicated one created by fork,
>> diff --git a/sysdeps/unix/sysv/linux/tst-ofdlocks.c b/sysdeps/unix/sysv/linux/tst-ofdlocks.c
>> index bd345e9..66c7856 100644
>> --- a/sysdeps/unix/sysv/linux/tst-ofdlocks.c
>> +++ b/sysdeps/unix/sysv/linux/tst-ofdlocks.c
>> @@ -46,7 +46,12 @@ do_test (void)
>>       .l_start  = (off64_t)INT32_MAX + 1024,
>>       .l_len    = 1024,
>> Â Â Â  };
>> -Â  TEST_VERIFY_EXIT (fcntl64 (temp_fd, F_OFD_SETLKW, &lck64) == 0);
>> +Â  int ret = fcntl64 (temp_fd, F_OFD_SETLKW, &lck64);
>> +Â  if (ret == -1 && errno == EINVAL)
>> +    /* OFD locks are only available on Linux 3.15.  */
>> +Â Â Â  FAIL_UNSUPPORTED ("fcntl (F_OFD_SETLKW) not supported)");
> 
> Likewise.
> 
> This could use TEST_COMPARE (twice).
> 
>> +Â  TEST_VERIFY_EXIT (ret == 0);
> 
> Thanks,
> Florian

Patch updated below:

diff --git a/sysdeps/unix/sysv/linux/tst-ofdlocks-compat.c b/sysdeps/unix/sysv/linux/tst-ofdlocks-compat.c
index 03c4abf..9930a50 100644
--- a/sysdeps/unix/sysv/linux/tst-ofdlocks-compat.c
+++ b/sysdeps/unix/sysv/linux/tst-ofdlocks-compat.c
@@ -62,7 +62,12 @@ do_test (void)
     .l_start  = (off64_t)INT32_MAX + 1024,
     .l_len    = 1024,
   };
-  TEST_VERIFY_EXIT (fcntl (temp_fd, F_OFD_SETLKW, &lck64) == 0);
+  int ret = fcntl (temp_fd, F_OFD_SETLKW, &lck64);
+  if (ret == -1 && errno == EINVAL)
+    /* OFD locks are only available on Linux 3.15.  */
+    FAIL_UNSUPPORTED ("fcntl (F_OFD_SETLKW) not supported): %m");
+
+  TEST_COMPARE (ret, 0);
 
   /* Open file description locks placed through the same open file description
      (either by same file descriptor or a duplicated one created by fork,
diff --git a/sysdeps/unix/sysv/linux/tst-ofdlocks.c b/sysdeps/unix/sysv/linux/tst-ofdlocks.c
index bd345e9..1aa5c2a 100644
--- a/sysdeps/unix/sysv/linux/tst-ofdlocks.c
+++ b/sysdeps/unix/sysv/linux/tst-ofdlocks.c
@@ -46,7 +46,12 @@ do_test (void)
     .l_start  = (off64_t)INT32_MAX + 1024,
     .l_len    = 1024,
   };
-  TEST_VERIFY_EXIT (fcntl64 (temp_fd, F_OFD_SETLKW, &lck64) == 0);
+  int ret = fcntl64 (temp_fd, F_OFD_SETLKW, &lck64);
+  if (ret == -1 && errno == EINVAL)
+    /* OFD locks are only available on Linux 3.15.  */
+    FAIL_UNSUPPORTED ("fcntl (F_OFD_SETLKW) not supported): %m");
+
+  TEST_COMPARE (ret, 0);
 
   /* Open file description locks placed through the same open file description
      (either by same file descriptor or a duplicated one created by fork,