Re: [LTP] [RFC PATCH] tst_checkpoint_destroy

[Cyril Hrubis <chrubis@suse.cz>]

Hi!
> Ok, we can do it this way.
> 
> One question regarding the typedef of futex_t. As of now the futex_t 
> type is defined in two places:
>   * testcases/kernel/syscalls/futex/futextest.h
>   * lib/tst_checkpoint.h
> 
> If I put unmap() in tst_rmdir(), I will add futex_t to tst_tmpdir.c 
> either, so it will be the third occurrence.
> 
> It seems it would be good to move the typedef of futex_t into a common 
> header.
> 
> I'm unsure which one to use :) Or just create include/lapi/futex.h?

Header in lapi sounds good, I should have probably done so when I was
rewriting the checkpoint interface.

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
One dashboard for servers and applications across Physical-Virtual-Cloud 
Widest out-of-the-box monitoring support with 50+ applications
Performance metrics, stats and reports that give you Actionable Insights
Deep dive visibility with transaction tracing using APM Insight.
http://ad.doubleclick.net/ddm/clk/290420510;117567292;y
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list