public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
* [LTP] [PATCH v2 0/1] syscalls/madvise: Handle zero page poisoning
@ 2017-04-03 11:01 Richard Palethorpe
  0 siblings, 0 replies; only message in thread
From: Richard Palethorpe @ 2017-04-03 11:01 UTC (permalink / raw)
  To: ltp

Hello Metan,

>This is getting a bit out of hand, can we change this into a function
>instead?
>
>static const char *maptype(int mtype)
>{
>	switch (mtype) {
>	case MAP_SHARED:
>		return "MAP_SHARED";
>	...
>	default:
>		return "???";
>	}
>}

I have done that in this second version, I suppose the conditional expression
was a bit weird and probably not as compiler friendly. I also joined a few
lines together and added some braces because check-patch.pl now asks me to.

Richard Palethorpe (1):
  syscalls/madvise: Handle zero page poisoning

 testcases/kernel/syscalls/madvise/madvise07.c | 53 ++++++++++++++++++++++-----
 1 file changed, 43 insertions(+), 10 deletions(-)

-- 
2.12.0

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2017-04-03 11:01 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-04-03 11:01 [LTP] [PATCH v2 0/1] syscalls/madvise: Handle zero page poisoning Richard Palethorpe

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox