[LTP] [PATCH 1/4] syscalls/sync01: Remove it

Xiao Yang ice_yangxiao@163.com
Fri Nov 6 17:14:22 CET 2020


On 11/6/20 8:36 PM, Cyril Hrubis wrote:
> Hi!
>> Ping. I think this patchset is simple.
> Great cleanup, pushed, thanks.

Hi Cyril, Petr


I have a doubt after reading Xu's patch[1] and Martin's patch[2]:

1) Xu removed sync01 because sync() always return 0.

2) On error, normal syscall always return -1 but Martin added a check 
for undefined return value(i.e. negative value except -1).

Is it necessary to check undefined return value?


Patches:

[1]: [PATCH 1/4] syscalls/sync01: Remove it

[2]: [PATCH 03/19] Unify error handling in lib/tst_safe_timerfd.c


Best Regards,

Xiao Yang

>



More information about the ltp mailing list