From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933129AbaHYOq3 (ORCPT ); Mon, 25 Aug 2014 10:46:29 -0400 Received: from mx1.redhat.com ([209.132.183.28]:13391 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933032AbaHYOqU (ORCPT ); Mon, 25 Aug 2014 10:46:20 -0400 From: Jiri Olsa To: linux-kernel@vger.kernel.org Cc: Jiri Olsa , Andi Kleen , Arnaldo Carvalho de Melo , Corey Ashford , David Ahern , Frederic Weisbecker , Ingo Molnar , "Jen-Cheng(Tommy) Huang" , Namhyung Kim , Paul Mackerras , Peter Zijlstra , Stephane Eranian Subject: [PATCH 1/9] perf: Remove redundant parent context check from context_equiv Date: Mon, 25 Aug 2014 16:45:35 +0200 Message-Id: <1408977943-16594-2-git-send-email-jolsa@kernel.org> In-Reply-To: <1408977943-16594-1-git-send-email-jolsa@kernel.org> References: <1408977943-16594-1-git-send-email-jolsa@kernel.org> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org As described in commit: 1f9a7268c67f perf: Do not allow optimized switch for non-cloned events we dont allow optimized switch for non cloned contexts. There's no need now to check if one of the context is parent of the other in context_equiv function. Cc: Andi Kleen Cc: Arnaldo Carvalho de Melo Cc: Corey Ashford Cc: David Ahern Cc: Frederic Weisbecker Cc: Ingo Molnar Cc: Jen-Cheng(Tommy) Huang Cc: Namhyung Kim Cc: Paul Mackerras Cc: Peter Zijlstra Cc: Stephane Eranian Signed-off-by: Jiri Olsa --- kernel/events/core.c | 8 -------- 1 file changed, 8 deletions(-) diff --git a/kernel/events/core.c b/kernel/events/core.c index d8cb4d21a346..4ad4ba2bc106 100644 --- a/kernel/events/core.c +++ b/kernel/events/core.c @@ -2257,14 +2257,6 @@ static int context_equiv(struct perf_event_context *ctx1, if (ctx1->pin_count || ctx2->pin_count) return 0; - /* If ctx1 is the parent of ctx2 */ - if (ctx1 == ctx2->parent_ctx && ctx1->generation == ctx2->parent_gen) - return 1; - - /* If ctx2 is the parent of ctx1 */ - if (ctx1->parent_ctx == ctx2 && ctx1->parent_gen == ctx2->generation) - return 1; - /* * If ctx1 and ctx2 have the same parent; we flatten the parent * hierarchy, see perf_event_init_context(). -- 1.8.3.1