[PATCH v2] nptl: Add sendmmsg and recvmmsg cancellation tests
Adhemerval Zanella
adhemerval.zanella@linaro.org
Mon Jun 13 16:40:00 GMT 2016
On 13/06/2016 11:48, Carlos O'Donell wrote:
> On 06/10/2016 04:35 PM, Adhemerval Zanella wrote:
>> Changes from previous version:
>>
>> - Added separated tests for both sendmmsg and recvmmsg.
>>
>> --
>>
>> This patch adds cancellation tests for both sendmmsg and recvmmsg
>> syscalls. Since for some system configuration (x86_64/i686 on
>> older kernels and non-Linux platforms), the tests are added as
>> two independent that report as unsupported if the syscall is not
>> presented.
>>
>> Both new tests uses the already tst-cancel4.c code, which as moved
>> to a common tst-cancel4-common{.c,h} files.
>>
>> Tested on x86_64 and i686.
>>
>> * nptl/Makefile (test): Add tst-cancel4_1 and tst-cancel4_2.
>> * nptl/tst-cancel4-common.c: New file.
>> * nptl/tst-cancel4-common.h: Likewise.
>> * nptl/tst-cancel4.c: Move common definitions to
>> tst-cancel4-common.{c,h} file.
>> * nptl/tst-cancel4_1.c: New test.
>> * nptl/tst-cancel4_2.c: New test.
>
> This looks good to me.
>
> OK to commit if you tested the UNSUPPORTED case by either hacking
> out the syscall wrapper to return ENOSYS or running on a system
> that didn't have sendmmsg.
>
> I think hakcing sysdeps/unix/sysv/linux/sendmmsg.c to return ENOSYS,
> recompile, recheck, should be enough to test UNSUPPORTED was wired
> up properly.
That's what I did in fact.
>
> Cheers,
> Carlos.
>
More information about the Libc-alpha
mailing list