<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 17, 2018 at 9:50 PM, Jan Stancek <span dir="ltr"><<a href="mailto:jstancek@redhat.com" target="_blank">jstancek@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">OFD commands require 64-bit argument (struct flock64). Until<br>
glibc commit 06ab719d30b0 ("Fix Linux fcntl OFD locks for<br>
non-LFS architectures (BZ#20251)") we relied on glibc passing<br>
arg directly to syscall.<br>
<br>
This creates problem for 32-bit version of the test, because old<br>
glibc is passing arg directly, while new one is casting it to<br>
struct flock.<br>
<br>
We could add a configure check for glibc version, but that may<br>
not help with other libc libraries.<br>
<br>
We could do a runtime check that exploits non-zero l_pid returning<br>
EINVAL. This however complicates SAFE_FCNTL macro substantially.<br>
<br>
This patch changes 32-bit version of test to use syscall directly.<br>
<br>
Signed-off-by: Jan Stancek <<a href="mailto:jstancek@redhat.com">jstancek@redhat.com</a>><br></span></blockquote><div><br></div><div class="gmail_default" style="font-size:small">Reviewed-by: Li Wang <<a href="mailto:liwang@redhat.com">liwang@redhat.com</a>></div></div><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div>
</div></div></div>