Re: [PATCH i-g-t v2] igt_core: add igt_constructor

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Em Qua, 2016-12-21 às 15:50 -0500, Lyude escreveu:
> This is a simple macro for executing a block of code at the beginning
> of
> intel-gpu-tools, before any tests have been ran. Useful for
> initialization of global resources used in IGT libraries.

IGT doesn't compile anymore here. Reverting this patch fixes it for me.

On a brand new git clone, if I just run autogen.sh, I get this:

./configure: line 6319: syntax error near unexpected token
`constructor'
./configure: line 6319: `AX_GCC_FUNC_ATTRIBUTE(constructor)'

The gcc/autoconf/etc versions are from Fedora 24.

I'm not sure what I'm missing here...

> 
> Signed-off-by: Lyude <lyude@xxxxxxxxxx>
> 
> Changes since v1:
>  - Add the line number into the name of the constructor function so
> that
>    multiple constructors may be used per-file.
> ---
>  configure.ac   |  1 +
>  lib/igt_core.h | 13 +++++++++++++
>  2 files changed, 14 insertions(+)
> 
> diff --git a/configure.ac b/configure.ac
> index fa31a52..411bfab 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -65,6 +65,7 @@
> AC_CHECK_TYPES([sighandler_t],[],[],[AC_INCLUDES_DEFAULT
>  #include <signal.h>])
>  AC_CHECK_FUNCS([swapctl])
>  AC_CHECK_FUNCS([asprintf])
> +AX_GCC_FUNC_ATTRIBUTE([constructor])
>  
>  dnl Check for POSIX timers
>  AC_CHECK_FUNCS(timer_create, [], [
> diff --git a/lib/igt_core.h b/lib/igt_core.h
> index 661a634..b669de3 100644
> --- a/lib/igt_core.h
> +++ b/lib/igt_core.h
> @@ -281,6 +281,19 @@ void igt_simple_init_parse_opts(int *argc, char
> **argv,
>  	} \
>  	static void igt_tokencat(__real_main, __LINE__)(void) \
>  
> +/**
> + * igt_constructor:
> + *
> + * Convenience macro to run the provided code block when igt first
> starts,
> + * before any tests have been run. This should be used for parts of
> the igt
> + * library that require initialization of objects with global
> context.
> + *
> + * This code block will be executed exactly once.
> + */
> +#define igt_constructor \
> +	__attribute__((constructor)) \
> +	static void igt_tokencat(__igt_constructor_l,
> __LINE__)(void)
> +
>  __attribute__((format(printf, 1, 2)))
>  void igt_skip(const char *f, ...) __attribute__((noreturn));
>  __attribute__((format(printf, 5, 6)))
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux