All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.