From patchwork Fri Aug 9 12:14:45 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Leo Yan X-Patchwork-Id: 13758741 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id C3B8DC3DA4A for ; Fri, 9 Aug 2024 12:20:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: MIME-Version:References:In-Reply-To:Message-Id:Date:Subject:Cc:To:From: Reply-To:Content-Type:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=RI69gYO9BN2OeA66it4DPa1TEplZJ7GIr6VmyOdPikk=; b=EP/BDRez8WLmjmbXIk6Dro/GBn 9dARX90pWBeXZzBxssigZ4X2XeRnj9bwY2j0v4iVp1eUVTKy6xe2xZvBeYkZVEcZ/1gWA0VBxI7bM 4+Kw/ja4sNzhVuvT5Cz62I6JiuIzbnHfnT3Sog9B8uWiqXd7e3e2uCahg437BUk9FEDGSQo42DnfB CpNEI2YK0DSNpeng0lG1fxdKXmN1uYIqUvyzG6+r8WPIxesmg6OnLxpRkuK6gz5GtuJhm2An0zotC F9wmK1GcE8kbyl4SfPxVMENsmbHM06qo/pwAK014SadkGr51tcNU+grZDGTmsgkgYrsdR/qD6w4jz Ed5EfTjQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1scObI-0000000BAvi-05ZC; Fri, 09 Aug 2024 12:20:08 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1scOWU-0000000BA98-08RE for linux-arm-kernel@lists.infradead.org; Fri, 09 Aug 2024 12:16:26 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 8253516F8; Fri, 9 Aug 2024 05:15:35 -0700 (PDT) Received: from e132581.cambridge.arm.com (e132581.arm.com [10.2.76.71]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id D8D383F6A8; Fri, 9 Aug 2024 05:15:07 -0700 (PDT) From: Leo Yan To: Peter Zijlstra , Arnaldo Carvalho de Melo , Ingo Molnar , Namhyung Kim , Ian Rogers , Adrian Hunter , "Liang, Kan" , Suzuki K Poulose , Mike Leach , James Clark , linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org, coresight@lists.linaro.org, linux-arm-kernel@lists.infradead.org Cc: Leo Yan Subject: [PATCH v4 6/8] perf auxtrace: Bails out after finding the event for the map index Date: Fri, 9 Aug 2024 13:14:45 +0100 Message-Id: <20240809121447.2078084-7-leo.yan@arm.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20240809121447.2078084-1-leo.yan@arm.com> References: <20240809121447.2078084-1-leo.yan@arm.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240809_051510_132406_7F8274C8 X-CRM114-Status: GOOD ( 13.14 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org After finding the corresponding event for the passed buffer index, it is safe to say the found event has been used. Then, the tool can check the event status and bails out if it has been disabled. Signed-off-by: Leo Yan --- tools/perf/util/auxtrace.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tools/perf/util/auxtrace.c b/tools/perf/util/auxtrace.c index c970024f16db..ac6e2e428a66 100644 --- a/tools/perf/util/auxtrace.c +++ b/tools/perf/util/auxtrace.c @@ -688,15 +688,15 @@ int auxtrace_record__read_finish(struct auxtrace_record *itr, int idx) if (evsel__is_aux_event(evsel)) { int cpu_map_idx; - if (evsel->disabled) - return 0; - cpu_map_idx = evlist__find_cpu_map_idx(itr->evlist, evsel, idx); /* No map is found in per CPU mmap */ if (cpu_map_idx == -ENOENT) return cpu_map_idx; + if (evsel->disabled) + return 0; + if (cpu_map_idx >= 0) return evlist__enable_event_idx(evsel, 1, cpu_map_idx); else