linux-kernel - Re: [PATCH v4 3/7] x86: mm/gup: add gup trace points

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] [day] [month] [year] [list]

Message-ID: <56689CDF.8060708@linaro.org>
Date:	Wed, 9 Dec 2015 13:27:59 -0800
From:	"Shi, Yang" <yang.shi@...aro.org>
To:	Steven Rostedt <rostedt@...dmis.org>
Cc:	akpm@...ux-foundation.org, mingo@...hat.com,
	linux-kernel@...r.kernel.org, linux-mm@...ck.org,
	linaro-kernel@...ts.linaro.org,
	Thomas Gleixner <tglx@...utronix.de>,
	"H. Peter Anvin" <hpa@...or.com>, x86@...nel.org
Subject: Re: [PATCH v4 3/7] x86: mm/gup: add gup trace points

On 12/9/2015 1:07 PM, Steven Rostedt wrote:
> On Wed,  9 Dec 2015 09:29:20 -0800
> Yang Shi <yang.shi@...aro.org> wrote:
>
>> Cc: Thomas Gleixner <tglx@...utronix.de>
>> Cc: Ingo Molnar <mingo@...hat.com>
>> Cc: "H. Peter Anvin" <hpa@...or.com>
>> Cc: x86@...nel.org
>> Signed-off-by: Yang Shi <yang.shi@...aro.org>
>> ---
>>   arch/x86/mm/gup.c | 7 +++++++
>>   1 file changed, 7 insertions(+)
>>
>> diff --git a/arch/x86/mm/gup.c b/arch/x86/mm/gup.c
>> index ae9a37b..a96bcb7 100644
>> --- a/arch/x86/mm/gup.c
>> +++ b/arch/x86/mm/gup.c
>> @@ -12,6 +12,9 @@
>>
>>   #include <asm/pgtable.h>
>>
>> +#define CREATE_TRACE_POINTS
>> +#include <trace/events/gup.h>>
>
> First off, does the above even compile?
>
> Second, you already created the tracepoints in mm/gup.c, why are you
> creating them here again? CREATE_TRACE_POINTS must be defined only once
> per events/.h file.

Sorry for that. The typo was introduced by git amend without running 
test build. The multiple definition of trace points was not caught by my 
shaky test script.

Will fix them soon.

Thanks,
Yang

>
> -- Steve
>
>> +
>>   static inline pte_t gup_get_pte(pte_t *ptep)
>>   {
>>   #ifndef CONFIG_X86_PAE
>> @@ -270,6 +273,8 @@ int __get_user_pages_fast(unsigned long start, int nr_pages, int write,
>>   					(void __user *)start, len)))
>>   		return 0;
>>
>> +	trace_gup_get_user_pages_fast(start, nr_pages);
>> +
>>   	/*
>>   	 * XXX: batch / limit 'nr', to avoid large irq off latency
>>   	 * needs some instrumenting to determine the common sizes used by
>> @@ -373,6 +378,8 @@ int get_user_pages_fast(unsigned long start, int nr_pages, int write,
>>   	} while (pgdp++, addr = next, addr != end);
>>   	local_irq_enable();
>>
>> +	trace_gup_get_user_pages_fast(start, nr_pages);
>> +
>>   	VM_BUG_ON(nr != (end - start) >> PAGE_SHIFT);
>>   	return nr;
>>
>

--
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/