public inbox for ltp@lists.linux.it
 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox