[LTP] [PATCH v2 1/2] Add SAFE_MPROTECT() macro
Petr Vorel
pvorel@suse.cz
Mon Mar 4 17:30:12 CET 2024
> Hi!
> > > Signed-off-by: Andrea Cervesato <andrea.cervesato@suse.com>
> > > ---
> > > include/tst_safe_macros.h | 5 +++++
> > > lib/safe_macros.c | 18 ++++++++++++++++++
> > > 2 files changed, 23 insertions(+)
> > Although Cyril acked this I wonder why we are adding it into the old API.
> > Shouldn't it be added only into tst_safe_macros.c?
> To be honest I do not care that much as long as we do not expose the
> functions into the headers for old API. We will have to convert the
> safe_macros.c with coccinelle once all tests are converted and one more
> function in there does not really matter that much.
OTOH we have tst_* file, moving it there (+ remove 'm' from functions) would be
trivial (and less confusing). Sure, finally coccinelle will be used.
Kind regards,
Petr
More information about the ltp
mailing list