linux-kernel - Re: [PATCH] perf: disable mmap2 support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20131009193322.GA3794@infradead.org>
Date:	Wed, 9 Oct 2013 16:33:22 -0300
From:	Arnaldo Carvalho de Melo <acme@...hat.com>
To:	Stephane Eranian <eranian@...gle.com>
Cc:	linux-kernel@...r.kernel.org, mingo@...e.hu, peterz@...radead.org,
	dsahern@...il.com, ak@...ux.intel.com, jolsa@...hat.com,
	hughd@...gle.com, keescook@...omium.org, akpm@...ux-foundation.org
Subject: Re: [PATCH] perf: disable mmap2 support

Em Wed, Oct 09, 2013 at 06:17:43PM +0200, Stephane Eranian escreveu:
> This patch disables the mmap2 record format support
> for now. The support needs a bit more work to cover
> VM_CLONE cases.
> 
> The patch leaves attr->mmap2 defined, but returns
> an error if it is set. That ensures no PERF_RECORD_MMAP2
> record can be generated. Yet it minimizes the reverts in
> the perf tool.

Would be interesting to have a simple new tool that would detect all the
mmaps that are shared in a given perf.data file, using the
PERF_RECORD_MMAP2 information, and that tool should come together with
the reworked kernel ABI, showing how it is used.

A 'perf test' entry, based on the test programs you guys are using to
test this functionality would be even better, showing that all cases
you've been striving to cover are in fact covered by the new ABI.

- Arnaldo
 
> 
> Signed-off-by: Stephane Eranian <eranian@...gle.com>
> ---
>  include/uapi/linux/perf_event.h |   18 +-----------------
>  kernel/events/core.c            |    4 ++++
>  2 files changed, 5 insertions(+), 17 deletions(-)
> 
> diff --git a/include/uapi/linux/perf_event.h b/include/uapi/linux/perf_event.h
> index da48837..8074b29 100644
> --- a/include/uapi/linux/perf_event.h
> +++ b/include/uapi/linux/perf_event.h
> @@ -682,23 +682,7 @@ enum perf_event_type {
>  	PERF_RECORD_SAMPLE			= 9,
>  
>  	/*
> -	 * The MMAP2 records are an augmented version of MMAP, they add
> -	 * maj, min, ino numbers to be used to uniquely identify each mapping
> -	 *
> -	 * struct {
> -	 *	struct perf_event_header	header;
> -	 *
> -	 *	u32				pid, tid;
> -	 *	u64				addr;
> -	 *	u64				len;
> -	 *	u64				pgoff;
> -	 *	u32				maj;
> -	 *	u32				min;
> -	 *	u64				ino;
> -	 *	u64				ino_generation;
> -	 *	char				filename[];
> -	 * 	struct sample_id		sample_id;
> -	 * };
> +	 * reserved, do not use
>  	 */
>  	PERF_RECORD_MMAP2			= 10,
>  
> diff --git a/kernel/events/core.c b/kernel/events/core.c
> index c716385..d7152a6 100644
> --- a/kernel/events/core.c
> +++ b/kernel/events/core.c
> @@ -6782,6 +6782,10 @@ static int perf_copy_attr(struct perf_event_attr __user *uattr,
>  	if (attr->read_format & ~(PERF_FORMAT_MAX-1))
>  		return -EINVAL;
>  
> +	/* disable for now */
> +	if (attr->mmap2)
> +		return -EINVAL;
> +
>  	if (attr->sample_type & PERF_SAMPLE_BRANCH_STACK) {
>  		u64 mask = attr->branch_sample_type;
>  
> -- 
> 1.7.10.4
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/