This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] configure: check for perf_event.h version
- From: Pedro Alves <palves at redhat dot com>
- To: Markus Metzger <markus dot t dot metzger at intel dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Thu, 06 Aug 2015 14:48:34 +0100
- Subject: Re: [PATCH] configure: check for perf_event.h version
- Authentication-results: sourceware.org; auth=none
- References: <1438866405-22616-1-git-send-email-markus dot t dot metzger at intel dot com>
On 08/06/2015 02:06 PM, Markus Metzger wrote:
> Intel(R) Processor Trace support requires a recent linux/perf_event.h header.
>
> When GDB is built on an older system, Intel(R) Processor Trace will not be
> available and there is no indication in the configure and build log as to
> what went wrong.
>
> Check for a compatible linux/perf_event.h at configure-time.
> diff --git a/gdb/configure.ac b/gdb/configure.ac
> index 905c27b..d867e85 100644
> --- a/gdb/configure.ac
> +++ b/gdb/configure.ac
> @@ -1252,6 +1252,20 @@ if test "${with_intel_pt}" = no; then
> AC_MSG_WARN([Intel(R) Processor Trace support disabled; some features may be unavailable.])
> HAVE_LIBIPT=no
> else
> + AC_PREPROC_IFELSE(AC_LANG_SOURCE([[
> +#include <linux/perf_event.h>
> +#ifdef PERF_ATTR_SIZE_VER5
> +# error
> +#endif
Can you explain what kind of symbol PERF_ATTR_SIZE_VER5 is?
>From the patch, I understand that that is something that is _not_
defined in the perf versions that are supposedly supported?
(otherwise, I'd expect an #ifndef instead.)
What about these? :
nat/linux-btrace.c:722:#if defined (PERF_ATTR_SIZE_VER5)
nat/linux-btrace.c:739:#endif /* defined (PERF_ATTR_SIZE_VER5) */
nat/linux-btrace.c:759:#if defined (PERF_ATTR_SIZE_VER5)
nat/linux-btrace.c:872:#else /* !defined (PERF_ATTR_SIZE_VER5) */
nat/linux-btrace.c:881:#endif /* !defined (PERF_ATTR_SIZE_VER5) */
Thanks,
Pedro Alves