All of lore.kernel.org
 help / color / mirror / Atom feed
From: Cyril Hrubis <chrubis@suse.cz>
To: Cui Bixuan <cuibixuan@huawei.com>
Cc: ltp-list@lists.sourceforge.net, zhuyanpeng@huawei.com,
	zhanyongming@huawei.com
Subject: Re: [LTP] [PATCH] open/open12: compile error
Date: Mon, 18 May 2015 13:56:27 +0200	[thread overview]
Message-ID: <20150518115627.GF4104@rei> (raw)
In-Reply-To: <55559C6A.9090001@huawei.com>

Hi!
> The 'MS_STRICTATIME' flag is not defined in old kernel.Now add
                                                  ^
						  libc

The system headers are supplied by glibc.

> it into include/lapi/mount.h and include it in testcase source.
> 
> Signed-off-by: Cui Bixuan <cuibixuan@huawei.com>
> ---
> Cat /usr/include/x86_64-linux-gnu/sys/mount.h in Ubuntu14 and find
> 'MS_STRICTATIME' flag:
> MS_STRICTATIME = 1 << 24,     /* Always perform atime updates.  */
> 
> So add it into include/lapi/mount.h
> And thank you for your advice,My idea is wrong.
> 
>  include/lapi/mount.h                    |   26 ++++++++++++++++++++++++++
>  testcases/kernel/syscalls/open/open12.c |    1 +
>  2 files changed, 27 insertions(+), 0 deletions(-)
>  create mode 100644 include/lapi/mount.h
> 
> diff --git a/include/lapi/mount.h b/include/lapi/mount.h
> new file mode 100644
> index 0000000..c71a638
> --- /dev/null
> +++ b/include/lapi/mount.h
> @@ -0,0 +1,26 @@
> +/*
> + * Copyright (c) 2014 Cyril Hrubis <chrubis@suse.cz>

              ^
	      Should be your name and current year. :)

> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License as
> + * published by the Free Software Foundation; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it would be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write the Free Software Foundation,
> + * Inc.,  51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
> + */
> +
> +#ifndef __MOUNT_H__
> +#define __MOUNT_H__
> +
> +#ifndef MS_STRICTATIME
> +#define MS_STRICTATIME  1 << 24
> +#endif
> +
> +#endif /* __MOUNT_H__ */
> diff --git a/testcases/kernel/syscalls/open/open12.c b/testcases/kernel/syscalls/open/open12.c
> index e665368..af3fab3 100644
> --- a/testcases/kernel/syscalls/open/open12.c
> +++ b/testcases/kernel/syscalls/open/open12.c
> @@ -31,6 +31,7 @@
>  #include "test.h"
>  #include "safe_macros.h"
>  #include "lapi/fcntl.h"
> +#include "lapi/mount.h"
> 
>  #define MNTPOINT	"mntpoint"
>  #define TEST_FILE	MNTPOINT"/test_file"
> -- 
> 1.6.0.2

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
One dashboard for servers and applications across Physical-Virtual-Cloud 
Widest out-of-the-box monitoring support with 50+ applications
Performance metrics, stats and reports that give you Actionable Insights
Deep dive visibility with transaction tracing using APM Insight.
http://ad.doubleclick.net/ddm/clk/290420510;117567292;y
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

  parent reply	other threads:[~2015-05-18 11:57 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-04  6:34 [LTP] [PATCH] performing test on loop device if noatime or relatime specified Han Pingtian
2015-05-04 11:53 ` Cyril Hrubis
2015-05-06 13:55   ` Han Pingtian
2015-05-07 12:48     ` Cyril Hrubis
2015-05-08  1:41       ` Han Pingtian
2015-05-12 12:50     ` Cui Bixuan
2015-05-12 13:48       ` Cyril Hrubis
     [not found]         ` <55530639.2070806@huawei.com>
2015-05-13  8:14           ` Cyril Hrubis
     [not found]             ` <55559C6A.9090001@huawei.com>
2015-05-18 11:56               ` Cyril Hrubis [this message]
     [not found]         ` <55530742.3030707@huawei.com>
2015-05-13  9:03           ` [LTP] [PATCH] open/open12: Check the kernel version for 'MS_STRICTATIME' Cyril Hrubis
     [not found]             ` <5555A746.4080006@huawei.com>
2015-05-18 11:52               ` Cyril Hrubis
     [not found]                 ` <5559DDD9.4090807@huawei.com>
2015-05-27 14:20                   ` [LTP] [PATCH 2/2 v2] " Cyril Hrubis

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150518115627.GF4104@rei \
    --to=chrubis@suse.cz \
    --cc=cuibixuan@huawei.com \
    --cc=ltp-list@lists.sourceforge.net \
    --cc=zhanyongming@huawei.com \
    --cc=zhuyanpeng@huawei.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.