[LTP] [PATCH v2 2/2] Add shmat04 SysV IPC bug reproducer
Petr Vorel
pvorel@suse.cz
Mon Mar 4 15:32:23 CET 2024
> Hi!
> > > Hmm, it's kind of ugly how we wrap the macro here like that...
> > > What about we instead add debugging messages to all the SAFE_MACROS()?
> > > Given that we added TDEBUG flag recently we can do soemthing as:
> > > tst_res_(TDEBUG, file, lineno, "mprotect(%p, %d, %s)",
> > > addr, size, prot_to_str(prot));
> > > To the SAFE_MPROTECT() and get the verbose output for free with verbose
> > > flag passed to the test.
> > > We can do that with all SAFE_MACROS() then we do not have to print most
> > > of the messages in this test...
> > Is this comment related with the previous patch of the set?
> Not at all, I'm just complaining that we are adding debuging print to
> the test itself when it would be much cleaner to put it into the test
> library instead.
+1, setting patchset as changes requested, please send v3.
Kind regards,
Petr
More information about the ltp
mailing list