[LTP] [PATCH v3 3/7] lib/tst_safe_pidfd: Add SAFE_PIDFD_OPEN macro
    Cyril Hrubis 
    chrubis@suse.cz
       
    Fri Feb 25 16:59:38 CET 2022
    
    
  
Hi!
> Just avoid a big tst_safe_macros.h like tst_safe_sysv_ipc.h does.
> Also the pidfd related syscalls seems no libc wrapper.
> 
> Based on the above two points, I started a new header. Also if you want 
> to merge it into tst_safe_macros.h, I will do(I don't have objection to 
> add this into tst_safe_macros.h ).
I would have just added the definitions to the tst_safe_macros.h, but
it's not a big deal.
Otherwise the actuall code looks good, with the header merged to tst_safe_macros.h:
Reviewed-by: Cyril Hrubis <chrubis@suse.cz>
-- 
Cyril Hrubis
chrubis@suse.cz
    
    
More information about the ltp
mailing list