From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756585AbbGTXB5 (ORCPT ); Mon, 20 Jul 2015 19:01:57 -0400 Received: from mga02.intel.com ([134.134.136.20]:39968 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753249AbbGTXBz (ORCPT ); Mon, 20 Jul 2015 19:01:55 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.15,511,1432623600"; d="scan'208";a="767953294" Subject: [PATCH] x86, mm: add parenthesis for TLB tracepoint size calculation To: dave@sr71.net Cc: dave.hansen@linux.intel.com, x86@kernel.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org From: Dave Hansen Date: Mon, 20 Jul 2015 16:01:53 -0700 Message-Id: <20150720230153.9E834081@viggo.jf.intel.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Dave Hansen flush_tlb_info->flush_start/end are both normal virtual addresses. When calculating 'nr_pages' (only used for the tracepoint), I neglected to put parenthesis in. Thanks to David Koufaty for pointing this out. Signed-off-by: Dave Hansen Cc: x86@kernel.org Cc: linux-kernel@vger.kernel.org Cc: stable@vger.kernel.org --- b/arch/x86/mm/tlb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -puN arch/x86/mm/tlb.c~tlb-parens arch/x86/mm/tlb.c --- a/arch/x86/mm/tlb.c~tlb-parens 2015-07-20 15:37:33.951272224 -0700 +++ b/arch/x86/mm/tlb.c 2015-07-20 15:37:33.955272406 -0700 @@ -117,7 +117,7 @@ static void flush_tlb_func(void *info) } else { unsigned long addr; unsigned long nr_pages = - f->flush_end - f->flush_start / PAGE_SIZE; + (f->flush_end - f->flush_start) / PAGE_SIZE; addr = f->flush_start; while (addr < f->flush_end) { __flush_tlb_single(addr); _