From patchwork Sun Dec 27 09:27:44 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Salvatore Bonaccorso X-Patchwork-Id: 352538 Delivered-To: patch@linaro.org Received: by 2002:a02:85a7:0:0:0:0:0 with SMTP id d36csp8930029jai; Sun, 27 Dec 2020 01:30:03 -0800 (PST) X-Google-Smtp-Source: ABdhPJworTT/l7zHj2elSDlOLDXPSUp3FoFW7KvKTUSp0LHFN3J7CYmDsLCar+I7Hb7mg4dPQacH X-Received: by 2002:a17:906:3f8d:: with SMTP id b13mr24442115ejj.464.1609061403052; Sun, 27 Dec 2020 01:30:03 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1609061403; cv=none; d=google.com; s=arc-20160816; b=N+Pea+nuYF1vtG772Bpc+CDJ7aJ1TK/1L3Bp/xobG2PeubdZ+nHa82c/562/wP5tcE imwRoVSUppPwUxGMkMydpC6lnKCY/TB7A07uTI0SbnlMNmY7AZXjhje5d9XvHNKyWUon Zn2+9uJUlZY2kwOBbjxThxrcNLVcHmzCO+uUMJCVVTvDWQgw9EnY8+A6UfsxRodiTjXF dgxAA+8kplqnBNyUj9cbWM1ptNSL4jvi1RWgogstaUHhnwIGc3Y5LRhGEqQdp0xrWrMw wOeQ4r08cgJv7sLvYFh+aCAgyfBmXe27A/Rbw0c+my3JKNpslPNx0MMw3qwKNq9gxaA4 P32w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from:sender :dkim-signature; bh=dqnzdkn07KMk7S0gm0jW9zryguHE8iY7/Za7LU/Hb8U=; b=jIIaAKPzmi4L6JYzkTpZDrGP1plSMW6E6RhLTdt8VN+8YIo0ue4HPTHamse33eXkLw sEDNNpw0SRDXLPEFHEjUO3HX6Xt1NvosPUdu6WIMana4MoiK285NPXxtCOcO4YkgJCei g4GPwpI6RsHldvQaMNxsOcT4fmWrvooa66aWX9gWz1aDo+O9Jrg5tAhpz1ikNuC02JXx fJuVMuwWxQYKfpKDDdCUV2NZuOZU/yP60Z0ykqRMwTsKdy5zpnHZL63Wt/EOqDA7doTG VXpn2LLiGuGrOA/B/qGdE8vXcPzgFjQW74nzjIkESyZ8LyrPT0JIWcGb0ctpxi9XJ9Bv pvmg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=Zsfbs5Nh; spf=pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id j15si19497269edv.474.2020.12.27.01.30.02; Sun, 27 Dec 2020 01:30:03 -0800 (PST) Received-SPF: pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=Zsfbs5Nh; spf=pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726116AbgL0J30 (ORCPT + 14 others); Sun, 27 Dec 2020 04:29:26 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49606 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726019AbgL0J3Y (ORCPT ); Sun, 27 Dec 2020 04:29:24 -0500 Received: from mail-ej1-x62a.google.com (mail-ej1-x62a.google.com [IPv6:2a00:1450:4864:20::62a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BFD20C061799 for ; Sun, 27 Dec 2020 01:28:43 -0800 (PST) Received: by mail-ej1-x62a.google.com with SMTP id b9so10754958ejy.0 for ; Sun, 27 Dec 2020 01:28:43 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=dqnzdkn07KMk7S0gm0jW9zryguHE8iY7/Za7LU/Hb8U=; b=Zsfbs5NhxzqPAMA5Z7tIbprk6ILGq1J3bdU5GFIAN5Hze8Mdl0nTyPpiB98GprKF57 geVivrGn+/BG+fyn5Ab/DI2IId0uxhnz1xIQYAhNi4m9gJgMH5TyjrsFlZs6dSO31ykN eDBL/lRrl/kZcBhb1VzBFQ4JbzlN/l1vQUlEAVtZKq+8wa155k89KFu7nQGUeszRMvdD 0AQ1EkhAbkQxVxqo8oyjCw7zSf8T0nLMfS371CgbLg1ew4+/trYE5+IwpVzgJiHZA2wx UvXiql4tyd3/a+xEbTEect/qXfnBVUkho9EIBVA/MPd8JNKqzSE3JB7beia153sN4gEW 3qoQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:from:to:cc:subject:date:message-id :in-reply-to:references:mime-version:content-transfer-encoding; bh=dqnzdkn07KMk7S0gm0jW9zryguHE8iY7/Za7LU/Hb8U=; b=o9wYjslF4xDTWbslRhy1ZrFx4h8MhXuQNdYNuzd20HCYT9GUBNdKhF5hqUkHhawmOG 7lFSmdRElp0oQPjWV9Afcf2XjVEHGa0OxjpwjWg+CcIqJ6Sjod0Ys/L3/SHbjjqSoZ53 QLo1aZUBwgcy2FgVEzQr9aEF3yIaMGWHberBKCpUuxjW7L7ESxZIQHeGoouSUyrB6Jr1 lHGbPA0aHAr0lUPOihKDvjuK+P1P65NXxh3FpvIIWL276cQ3tfJsiVsSpOtiVo/lXKWK qgm956xMUAC3xV3KhDWTRNg7lZsvYifLTvXnfr/KDUiEGbACzekGzG/kbZPBv6PDqH0r fFeA== X-Gm-Message-State: AOAM531kClAurOALTRT/gQO04O3apHh4oiCgMajT3Sh/DZTS+BR67DXy oqvj6xbBtIVjRJksZRAz7iI= X-Received: by 2002:a17:906:958e:: with SMTP id r14mr38624227ejx.319.1609061322488; Sun, 27 Dec 2020 01:28:42 -0800 (PST) Received: from eldamar (80-218-24-251.dclient.hispeed.ch. [80.218.24.251]) by smtp.gmail.com with ESMTPSA id e1sm14603394edk.51.2020.12.27.01.28.41 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 27 Dec 2020 01:28:41 -0800 (PST) Sender: Salvatore Bonaccorso From: Salvatore Bonaccorso To: stable@vger.kernel.org Cc: Greg Kroah-Hartman , Sasha Levin , Leo Yan , Mathieu Poirier , Alexander Shishkin , Jiri Olsa , Mike Leach , Namhyung Kim , Robert Walker , Suzuki K Poulouse , coresight ml , linux-arm-kernel@lists.infradead.org, Arnaldo Carvalho de Melo , Salvatore Bonaccorso Subject: [PATCH 1/2] perf cs-etm: Change tuple from traceID-CPU# to traceID-metadata Date: Sun, 27 Dec 2020 10:27:44 +0100 Message-Id: <20201227092745.447945-2-carnil@debian.org> X-Mailer: git-send-email 2.30.0.rc2 In-Reply-To: <20201227092745.447945-1-carnil@debian.org> References: <20201227092745.447945-1-carnil@debian.org> MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org From: Leo Yan commit 95c6fe970a0160cb770c5dce9f80311b42d030c0 upstream. If packet processing wants to know the packet is bound with which ETM version, it needs to access metadata to decide that based on metadata magic number; but we cannot simply to use CPU logic ID number as index to access metadata sequential array, especially when system have hotplugged off CPUs, the metadata array are only allocated for online CPUs but not offline CPUs, so the CPU logic number doesn't match with its index in the array. This patch is to change tuple from traceID-CPU# to traceID-metadata, thus it can use the tuple to retrieve metadata pointer according to traceID. For safe accessing metadata fields, this patch provides helper function cs_etm__get_cpu() which is used to return CPU number according to traceID; cs_etm_decoder__buffer_packet() is the first consumer for this helper function. Signed-off-by: Leo Yan Reviewed-by: Mathieu Poirier Cc: Alexander Shishkin Cc: Jiri Olsa Cc: Mike Leach Cc: Namhyung Kim Cc: Robert Walker Cc: Suzuki K Poulouse Cc: coresight ml Cc: linux-arm-kernel@lists.infradead.org Link: http://lkml.kernel.org/r/20190129122842.32041-6-leo.yan@linaro.org Signed-off-by: Arnaldo Carvalho de Melo [Salvatore Bonaccorso: Adjust for context changes in tools/perf/util/cs-etm-decoder/cs-etm-decoder.c] Signed-off-by: Salvatore Bonaccorso --- .../perf/util/cs-etm-decoder/cs-etm-decoder.c | 8 +++--- tools/perf/util/cs-etm.c | 26 ++++++++++++++----- tools/perf/util/cs-etm.h | 9 ++++++- 3 files changed, 31 insertions(+), 12 deletions(-) -- 2.30.0.rc2 diff --git a/tools/perf/util/cs-etm-decoder/cs-etm-decoder.c b/tools/perf/util/cs-etm-decoder/cs-etm-decoder.c index 938def6d0bb9..f540037eb705 100644 --- a/tools/perf/util/cs-etm-decoder/cs-etm-decoder.c +++ b/tools/perf/util/cs-etm-decoder/cs-etm-decoder.c @@ -278,14 +278,12 @@ cs_etm_decoder__buffer_packet(struct cs_etm_decoder *decoder, enum cs_etm_sample_type sample_type) { u32 et = 0; - struct int_node *inode = NULL; + int cpu; if (decoder->packet_count >= MAX_BUFFER - 1) return OCSD_RESP_FATAL_SYS_ERR; - /* Search the RB tree for the cpu associated with this traceID */ - inode = intlist__find(traceid_list, trace_chan_id); - if (!inode) + if (cs_etm__get_cpu(trace_chan_id, &cpu) < 0) return OCSD_RESP_FATAL_SYS_ERR; et = decoder->tail; @@ -296,7 +294,7 @@ cs_etm_decoder__buffer_packet(struct cs_etm_decoder *decoder, decoder->packet_buffer[et].sample_type = sample_type; decoder->packet_buffer[et].exc = false; decoder->packet_buffer[et].exc_ret = false; - decoder->packet_buffer[et].cpu = *((int *)inode->priv); + decoder->packet_buffer[et].cpu = cpu; decoder->packet_buffer[et].start_addr = CS_ETM_INVAL_ADDR; decoder->packet_buffer[et].end_addr = CS_ETM_INVAL_ADDR; diff --git a/tools/perf/util/cs-etm.c b/tools/perf/util/cs-etm.c index 7b5e15cc6b71..5cde3956e19a 100644 --- a/tools/perf/util/cs-etm.c +++ b/tools/perf/util/cs-etm.c @@ -91,6 +91,20 @@ static int cs_etm__update_queues(struct cs_etm_auxtrace *etm); static int cs_etm__process_timeless_queues(struct cs_etm_auxtrace *etm, pid_t tid, u64 time_); +int cs_etm__get_cpu(u8 trace_chan_id, int *cpu) +{ + struct int_node *inode; + u64 *metadata; + + inode = intlist__find(traceid_list, trace_chan_id); + if (!inode) + return -EINVAL; + + metadata = inode->priv; + *cpu = (int)metadata[CS_ETM_CPU]; + return 0; +} + static void cs_etm__packet_dump(const char *pkt_string) { const char *color = PERF_COLOR_BLUE; @@ -230,7 +244,7 @@ static void cs_etm__free(struct perf_session *session) cs_etm__free_events(session); session->auxtrace = NULL; - /* First remove all traceID/CPU# nodes for the RB tree */ + /* First remove all traceID/metadata nodes for the RB tree */ intlist__for_each_entry_safe(inode, tmp, traceid_list) intlist__remove(traceid_list, inode); /* Then the RB tree itself */ @@ -1316,9 +1330,9 @@ int cs_etm__process_auxtrace_info(union perf_event *event, 0xffffffff); /* - * Create an RB tree for traceID-CPU# tuple. Since the conversion has - * to be made for each packet that gets decoded, optimizing access in - * anything other than a sequential array is worth doing. + * Create an RB tree for traceID-metadata tuple. Since the conversion + * has to be made for each packet that gets decoded, optimizing access + * in anything other than a sequential array is worth doing. */ traceid_list = intlist__new(NULL); if (!traceid_list) { @@ -1384,8 +1398,8 @@ int cs_etm__process_auxtrace_info(union perf_event *event, err = -EINVAL; goto err_free_metadata; } - /* All good, associate the traceID with the CPU# */ - inode->priv = &metadata[j][CS_ETM_CPU]; + /* All good, associate the traceID with the metadata pointer */ + inode->priv = metadata[j]; } /* diff --git a/tools/perf/util/cs-etm.h b/tools/perf/util/cs-etm.h index 37f8d48179ca..fb5fc6538b7f 100644 --- a/tools/perf/util/cs-etm.h +++ b/tools/perf/util/cs-etm.h @@ -53,7 +53,7 @@ enum { CS_ETMV4_PRIV_MAX, }; -/* RB tree for quick conversion between traceID and CPUs */ +/* RB tree for quick conversion between traceID and metadata pointers */ struct intlist *traceid_list; #define KiB(x) ((x) * 1024) @@ -69,6 +69,7 @@ static const u64 __perf_cs_etmv4_magic = 0x4040404040404040ULL; #ifdef HAVE_CSTRACE_SUPPORT int cs_etm__process_auxtrace_info(union perf_event *event, struct perf_session *session); +int cs_etm__get_cpu(u8 trace_chan_id, int *cpu); #else static inline int cs_etm__process_auxtrace_info(union perf_event *event __maybe_unused, @@ -76,6 +77,12 @@ cs_etm__process_auxtrace_info(union perf_event *event __maybe_unused, { return -1; } + +static inline int cs_etm__get_cpu(u8 trace_chan_id __maybe_unused, + int *cpu __maybe_unused) +{ + return -1; +} #endif #endif From patchwork Sun Dec 27 09:27:45 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Salvatore Bonaccorso X-Patchwork-Id: 352537 Delivered-To: patch@linaro.org Received: by 2002:a02:85a7:0:0:0:0:0 with SMTP id d36csp8930033jai; Sun, 27 Dec 2020 01:30:03 -0800 (PST) X-Google-Smtp-Source: ABdhPJwE2AbOkdOipOV0Jo3prtspb4pHBL2jewMvubib8ARB3n2lLjtBj5EemcN2ctESRC+wTRE8 X-Received: by 2002:a17:907:546:: with SMTP id wk6mr36938381ejb.238.1609061403450; Sun, 27 Dec 2020 01:30:03 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1609061403; cv=none; d=google.com; s=arc-20160816; b=x09nqzmVG0l60ZkIFCtti+q5fgRpOpJHRN7n3FudvUipLvyeZI1zb9XEHYIECMJLe6 JFMO77RQLW1uMKF3b0JPgykknbkU9Eyo0IY/xwlcGVfysuoL8p3kihKcrfIj9yu6S43r x0ykML3fiX6y5ZJEI+5ejs4Soqs5MCZgtqt1g51W23f1L4HCtMCnVcUVlLiFg560pSI5 13GlnARvhS8t5PuBAMJPxjZPgH1zUinLwoQeHhi7etu0r/xq3OVkq4BTrYtl7IhYPUTH j1gaSwk8zmHEUIMehNpu9MUL+VEGifRGiQK5PFD2UAYkUKK24fpYmS0WAAhKN0TWT3CI tTCw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from:sender :dkim-signature; bh=b3XfiZGTmdtILp+unm6rSSuThTHGpNqTBR1owWhke04=; b=q/PO/FkIUBeDWwK75dGRTisY/ZcU0TBKPz913/EYO4b+z//k4clpvnIk2offH03XfJ jR+z7Pky3PwfwlxUQuui174ScQN5N160RMW5z3HkijRg4BtcvfkWEEoLDuKLCCouaN2E i4sEZNN19W33hIs7wg8YdD0BJRCCHjbzFQlryx4kkRt8AWuxvJ16JMVN+CQ3EzVZ7Auo EQj6IaIVsv8KWavf4S7UZa69Upu01FvQOYRO+9ISpwYYl7g2jIAb7EFEpQtTHKCr8dmn GcXR7zHWVuU126no0QjdnZXPN3qQP/67R8sRIXNOY24IyJS1xrDkZv75xerNPAkbch2E 1nTA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=G0s73267; spf=pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id j15si19497269edv.474.2020.12.27.01.30.03; Sun, 27 Dec 2020 01:30:03 -0800 (PST) Received-SPF: pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=G0s73267; spf=pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726117AbgL0J32 (ORCPT + 14 others); Sun, 27 Dec 2020 04:29:28 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49614 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726019AbgL0J31 (ORCPT ); Sun, 27 Dec 2020 04:29:27 -0500 Received: from mail-ej1-x62a.google.com (mail-ej1-x62a.google.com [IPv6:2a00:1450:4864:20::62a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 93A63C061794 for ; Sun, 27 Dec 2020 01:28:46 -0800 (PST) Received: by mail-ej1-x62a.google.com with SMTP id g20so10749257ejb.1 for ; Sun, 27 Dec 2020 01:28:46 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=b3XfiZGTmdtILp+unm6rSSuThTHGpNqTBR1owWhke04=; b=G0s73267c0w0axAQajWTEYK2o7gF7BNjzv4WbWX6fD13vVy3UMCkFVEv5EErPCY+so R6VY3PeLQcQs1GoIKMHgEih9fGp5qQAnVZR3XFvCdMCFFFXrrBLNty7zm1f/8zIXReTX H1/NLhmUV/lIQ27UfD+oO+3KatCWg6NOiA5I/NMh5p+FmOO+xKqGxDaMnvf+vBUwNCsP CmdxMHdx3II2YmoDlt7PaKiYmwa20Nxwk+UR+5XcB9Uli70SvvENmGxed+UTJhiik64I ybD/LZhymP6TD+QogBUPyjb0Yzu3dEU4HJDVweE0XApm5Ju/VwuDE+H/fTaoa0tPvbtx aILQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:from:to:cc:subject:date:message-id :in-reply-to:references:mime-version:content-transfer-encoding; bh=b3XfiZGTmdtILp+unm6rSSuThTHGpNqTBR1owWhke04=; b=jl5lu3ttsHNBoRk96C1RR5ooze9awtlsVxhamom/8GyjeqiTZUQaamvahzq0mMF/kS HV8LxtC4iFBz6reKz1Rkcz8dGcjTQISTXpMsfJVFz2cB9nAWGHH1gCcu1I+Yua/xvb7e Ebj+qRZDuvjHZh2wRwpcAeCsHa6hk/cOXdQYbSVamcUu92XsgPGc4/6KGS3mtmFlyIDP NOgM2fXHPCYc4B+QXDeOmYyc1AuONA0v3EDleYCr/JMw1+jVBb0NvshnKTrI3eRLg2/E 3O2ImhbHSSf+WBKznHgN4JDZ3zhvi+LMg7BaUrWa+PRm/RcZFb+JxH+Uzn4/vGK+bnjv HJAw== X-Gm-Message-State: AOAM531lE/MYB/yZbpqszrBRmQRgysKQyFy9XAKChjydol782zF2h+QA 8ca8iE0MnNYsQFa3oRZFPfA= X-Received: by 2002:a17:906:705:: with SMTP id y5mr36419497ejb.428.1609061325403; Sun, 27 Dec 2020 01:28:45 -0800 (PST) Received: from eldamar (80-218-24-251.dclient.hispeed.ch. [80.218.24.251]) by smtp.gmail.com with ESMTPSA id c16sm9905934ejk.91.2020.12.27.01.28.44 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 27 Dec 2020 01:28:44 -0800 (PST) Sender: Salvatore Bonaccorso From: Salvatore Bonaccorso To: stable@vger.kernel.org Cc: Greg Kroah-Hartman , Sasha Levin , Leo Yan , Thomas Backlund , Mathieu Poirier , Mike Leach , Alexander Shishkin , Jiri Olsa , Mark Rutland , Namhyung Kim , Peter Zijlstra , Suzuki Poulouse , Tor Jeremiassen , linux-arm-kernel@lists.infradead.org, Arnaldo Carvalho de Melo , Salvatore Bonaccorso Subject: [PATCH 2/2] perf cs-etm: Move definition of 'traceid_list' global variable from header file Date: Sun, 27 Dec 2020 10:27:45 +0100 Message-Id: <20201227092745.447945-3-carnil@debian.org> X-Mailer: git-send-email 2.30.0.rc2 In-Reply-To: <20201227092745.447945-1-carnil@debian.org> References: <20201227092745.447945-1-carnil@debian.org> MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org From: Leo Yan commit 168200b6d6ea0cb5765943ec5da5b8149701f36a upstream. The variable 'traceid_list' is defined in the header file cs-etm.h, if multiple C files include cs-etm.h the compiler might complaint for multiple definition of 'traceid_list'. To fix multiple definition error, move the definition of 'traceid_list' into cs-etm.c. Fixes: cd8bfd8c973e ("perf tools: Add processing of coresight metadata") Reported-by: Thomas Backlund Signed-off-by: Leo Yan Reviewed-by: Mathieu Poirier Reviewed-by: Mike Leach Tested-by: Mike Leach Tested-by: Thomas Backlund Cc: Alexander Shishkin Cc: Jiri Olsa Cc: Mark Rutland Cc: Namhyung Kim Cc: Peter Zijlstra Cc: Suzuki Poulouse Cc: Tor Jeremiassen Cc: linux-arm-kernel@lists.infradead.org Link: http://lore.kernel.org/lkml/20200505133642.4756-1-leo.yan@linaro.org Signed-off-by: Arnaldo Carvalho de Melo Signed-off-by: Salvatore Bonaccorso --- tools/perf/util/cs-etm.c | 3 +++ tools/perf/util/cs-etm.h | 3 --- 2 files changed, 3 insertions(+), 3 deletions(-) -- 2.30.0.rc2 diff --git a/tools/perf/util/cs-etm.c b/tools/perf/util/cs-etm.c index 5cde3956e19a..3275b8dc9344 100644 --- a/tools/perf/util/cs-etm.c +++ b/tools/perf/util/cs-etm.c @@ -87,6 +87,9 @@ struct cs_etm_queue { struct cs_etm_packet *packet; }; +/* RB tree for quick conversion between traceID and metadata pointers */ +static struct intlist *traceid_list; + static int cs_etm__update_queues(struct cs_etm_auxtrace *etm); static int cs_etm__process_timeless_queues(struct cs_etm_auxtrace *etm, pid_t tid, u64 time_); diff --git a/tools/perf/util/cs-etm.h b/tools/perf/util/cs-etm.h index fb5fc6538b7f..97c3152f5bfd 100644 --- a/tools/perf/util/cs-etm.h +++ b/tools/perf/util/cs-etm.h @@ -53,9 +53,6 @@ enum { CS_ETMV4_PRIV_MAX, }; -/* RB tree for quick conversion between traceID and metadata pointers */ -struct intlist *traceid_list; - #define KiB(x) ((x) * 1024) #define MiB(x) ((x) * 1024 * 1024)