[CRIU] [PATCH] test/zdtm: fix compilation of maps02.c
Andrey Vagin
avagin at parallels.com
Fri Oct 3 03:03:55 PDT 2014
On Fri, Oct 03, 2014 at 12:49:02PM +0400, Pavel Emelyanov wrote:
> On 10/02/2014 05:12 PM, Nicolas Dichtel wrote:
> > Error was:
> > maps02.c: In function ‘main’:
> > maps02.c:57:74: error: ‘MADV_DONTDUMP’ undeclared (first use in this function)
> > maps02.c:57:74: note: each undeclared identifier is reported only once for each function it appears in
> > make: *** [maps02] Error 1
> > ERROR: fail to start /home/root/criu/test/zdtm/live/static/maps02
> >
> > Signed-off-by: Nicolas Dichtel <nicolas.dichtel at 6wind.com>
>
> Andrey, please, review the patch.
>
> > ---
> > test/zdtm/live/static/maps02.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/test/zdtm/live/static/maps02.c b/test/zdtm/live/static/maps02.c
> > index 336044fea672..b9fb27bb4c45 100644
> > --- a/test/zdtm/live/static/maps02.c
> > +++ b/test/zdtm/live/static/maps02.c
> > @@ -1,5 +1,6 @@
> > #include <sys/mman.h>
> > #include "zdtmtst.h"
> > +#include <asm-generic/mman-common.h>
asm-generic/mman-common.h can't be used directly.
Looks like you use old headers. MADV_DONTDUMP was added recently.
commit accb61fe7bb0f5c2a4102239e4981650f9048519
Author: Jason Baron <jbaron at redhat.com>
Date: Fri Mar 23 15:02:51 2012 -0700
coredump: add VM_NODUMP, MADV_NODUMP, MADV_CLEAR_NODUMP
Look at my headers:
[root at avagin-fc19-cr include]# grep -r "define MADV_DONTDUMP" .
./bits/mman-linux.h:# define MADV_DONTDUMP 16
./asm-generic/mman-common.h:#define MADV_DONTDUMP 16
./bits/mman.h:#include <bits/mman-linux.h>
./sys/mman.h:#include <bits/mman.h>
./asm-generic/mman.h:#include <asm-generic/mman-common.h>
./asm/mman.h:#include <asm-generic/mman.h>
./linux/mman.h:#include <asm/mman.h>
You can try to include ./linux/mman.h or use smth like this:
#ifndef MADV_DONTDUMP
#define MADV_DONTDUMP 16
#endif
> >
> > const char *test_doc = "Test shared memory with advises";
> > const char *test_author = "Cyrill Gorcunov <gorcunov at openvz.org>";
> >
>
More information about the CRIU
mailing list