Re: [LTP] [PATCH v5 1/2] lib/cloner.c: add more args

[Zeng L. <zen...@cn...>]

On Wed, 2014-01-08 at 06:55 -0500, Jan Stancek wrote:
> 
> ----- Original Message -----
> > From: "Zeng Linggang" <zen...@cn...>
> > To: ch...@su...
> > Cc: "Jan Stancek" <jst...@re...>, "ltp-list" <ltp...@li...>
> > Sent: Wednesday, 8 January, 2014 12:41:37 PM
> > Subject: Re: [LTP] [PATCH v5 1/2] lib/cloner.c: add more args
> > I run it in RHEL5.0GA, RHEL6.0GA, 7.0Beta
> > and do not find these messages.
> > Could you tell me which distributions it is.
> > 
> > > So this needs an autoconf check and few ifdefs.
> > > 
> > > And as we are about to create release this needs to be either fixed
> > > ASAP or we will have to revert it for the release.
> > > 
> > 
> > I will send a patch as soon as possible.
> 
> Thanks, I started looking into it few moments ago, so I'll just send
> what I have in case it can save you some time:
> 

OK. Thank you very much.

Best regards,
Zeng

> diff --git a/configure.ac b/configure.ac
> index 4af7662..e564ee5 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -171,5 +171,6 @@ LTP_CHECK_FS_IOC_FLAGS
>  LTP_CHECK_MREMAP_FIXED
>  LTP_CHECK_KERNEL_DEVEL
>  LTP_CHECK_XFS_QUOTACTL
> +LTP_CHECK_CLONE7ARGS
>  
>  AC_OUTPUT
> diff --git a/m4/ltp-clone7args.m4 b/m4/ltp-clone7args.m4
> new file mode 100644
> index 0000000..df1ec37
> --- /dev/null
> +++ b/m4/ltp-clone7args.m4
> @@ -0,0 +1,36 @@
> +dnl
> +dnl Copyright (c) Linux Test Project, 2013
> +dnl
> +dnl This program is free software;  you can redistribute it and/or modify
> +dnl it under the terms of the GNU General Public License as published by
> +dnl the Free Software Foundation; either version 2 of the License, or
> +dnl (at your option) any later version.
> +dnl
> +dnl This program is distributed in the hope that it will be useful,
> +dnl but WITHOUT ANY WARRANTY;  without even the implied warranty of
> +dnl MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See
> +dnl the GNU General Public License for more details.
> +dnl
> +dnl You should have received a copy of the GNU General Public License
> +dnl along with this program;  if not, write to the Free Software
> +dnl Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
> +dnl
> +
> +dnl
> +dnl LTP_CHECK_CLONE7ARGS
> +dnl ----------------------------
> +dnl
> +AC_DEFUN([LTP_CHECK_CLONE7ARGS],[
> +AH_TEMPLATE(HAVE_CLONE7ARGS,
> +[Define to 1 if clone() supports 7 arguments.])
> +AC_MSG_CHECKING([for CLONE7ARGS])
> +AC_TRY_LINK([#define _GNU_SOURCE
> +                #include <sched.h>
> +                #include <stdlib.h>],
> +                [
> +               #if !defined(__ia64__)
> +               clone(NULL, NULL, 0, NULL, NULL, NULL, NULL);
> +               #endif
> +               ],
> +               AC_DEFINE(HAVE_CLONE7ARGS) AC_MSG_RESULT(yes), AC_MSG_RESULT(no))
> +])
> 
> Regards,
> Jan
> 
> > 
> > Thanks and best regards,
> > Zeng
> > 
> > 
> >