From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f72.google.com (mail-pj1-f72.google.com [209.85.216.72]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id AD3214A9D78 for ; Wed, 2 Sep 2026 15:49:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.72 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788364158; cv=none; b=Kdyq8obnlK2+QzQGsXH4TwbdPLe3ikE+OdcQWRbN8/0kiVLElVATB4VO6263AzkTKCf6m7JJvTZRR9572UctrguY2+f7QpwgiNxzU1RvMyl0IPf7FmPvXsoQhJnL3vEf29bfBYZ3r6Ej00G8H6VR5Mz4t3EQadAsAwvl1xffF78= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788364158; c=relaxed/simple; bh=0rHOZoRobN/My/uaUo/taCIKk/nrLMB8OuXHYhWB9Q8=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Content-Type; b=aWIA5BDMs6md+DV0mH/Qj3PolOXd0WEN3fIUAzkuzTXPFoiGWMjkN7F5p+CbRbBFlSDaMIfbHvagGu4HpmsU5D7zbW8DVGuM75ofTPoC61rMSbn5dpHd0rRfssrVZQNLW6ab2jPWYvP2KjoyaSQoN+ceMWstwKThGdJnlNAmVwM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--irogers.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=pYJ8FKex; arc=none smtp.client-ip=209.85.216.72 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--irogers.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="pYJ8FKex" Received: by mail-pj1-f72.google.com with SMTP id 98e67ed59e1d1-398d292eac1so2503909a91.1 for ; Wed, 02 Sep 2026 08:49:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20251104; t=1788364154; x=1788968954; darn=vger.kernel.org; h=content-type:to:from:subject:message-id:references:mime-version :in-reply-to:date:from:to:cc:subject:date:message-id:reply-to :content-type; bh=HlO9J/OYs/VWmsx5/JTVY1hRLPz5HlVbyPwG8syJOco=; b=pYJ8FKexkoISdSpA7BHKigvVZVNZAfQnDLEpBSmWswDuN9AdrPQ8pdf+ZoPhIzK95X jC/CZ+ztIBQ2xQC6PF2Xy1RSM5dKCVOU2Nq4UiZIt/J5wForY+yGcmg6GtUFC7MGh/6U SBG3HBrWkn4IYsPjsqAz/4EC7muuSdDgha/BDXJAa9KJEbYVT/3XLn/VFHHECpgvb0S4 ZCka2e19NM0f18mXSbSea1SOKpp2cprJGcRvVWR2OUnEbeZr2q1q/gxX2bI33sOSgQcT 2dkyN4eGdGOngJbXqS4zfjfRkOOTyhdKhIVN/CegbKgnHbAu5xL8wOCVZ9sNXooFLfcX SXNQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1788364154; x=1788968954; h=content-type:to:from:subject:message-id:references:mime-version :in-reply-to:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=HlO9J/OYs/VWmsx5/JTVY1hRLPz5HlVbyPwG8syJOco=; b=THXwM51XGxeh78BOMjMtN/AzENcG80sZtCtoGS1y4pH34jxutynkdhCSIRhyIcghQJ 1lY9Wn9rdhv71QEg7WnqRy0LWvwyJ6YySMas9MJnOQxzbFOYgeAQicjFGtt6j34/7SBL +fF6ANM7dYdndfWm6skRFUt7J3B39nmrRelHVj3kjFRtYCj4mZBltK8vM4+wYt8rnpAD m0h4QK1Z8Gw397YXAHtOd6n3wHoAV5qej+3RVS7mffOmbLq6iGjSsOiR8UVKoe89H/tY zLZrLJkKgtWztt+U2bAtKVZkbbM91XDrUYILBoqwAZBAJzFpW4xxnlcthqW+n+bT3cHA lUNQ== X-Forwarded-Encrypted: i=1; AKwUvBy16T0GhwsY7C310UGne8+1b7CflSNEzAuiXsE08aPTrfUeYoEnLVjauJRmmgC33YTh1eYvN43bBLmAa+nrBoLN@vger.kernel.org X-Gm-Message-State: AFuF++mqHljlSHfzGe3sFkNaUgGRXdv9GbI8oERt1YoIF0sXY1V3NTHN Y7PfVyejtstyC/cvxweL4+VQjGbwymHLoOgoi8/Ln7Fw8nIRfVKG7O9eovsT2fpYMv6Te86pmz7 cOk/QEzGeIQ== X-Received: from dltt5.prod.google.com ([2002:a05:701a:c965:b0:141:a317:d828]) (user=irogers job=prod-delivery.src-stubby-dispatcher) by 2002:a17:90b:524c:b0:390:8361:a532 with SMTP id 98e67ed59e1d1-39aedfb8ebcmr8928932a91.7.1788364153348; Wed, 02 Sep 2026 08:49:13 -0700 (PDT) Date: Wed, 2 Sep 2026 08:48:58 -0700 In-Reply-To: <20260902154858.2078885-1-irogers@google.com> Precedence: bulk X-Mailing-List: linux-perf-users@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20260902154858.2078885-1-irogers@google.com> X-Mailer: git-send-email 2.55.0.966.g6673acef38-goog Message-ID: <20260902154858.2078885-7-irogers@google.com> Subject: [PATCH v3 6/6] perf hist: Remove keysym.h include from hist.h From: Ian Rogers To: Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Namhyung Kim , Jiri Olsa , Ian Rogers , Adrian Hunter , James Clark , Tianyou Li , Jiebin Sun , Dapeng Mi , Eric Biggers , Thomas Richter , Stephen Brennan , Swapnil Sapkal , linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org Content-Type: text/plain; charset="UTF-8" hist.h isn't related to UI and so remove the UI inclusion. Fix the transitive dependency issues this exposes. Signed-off-by: Ian Rogers --- tools/perf/builtin-annotate.c | 8 ++++++++ tools/perf/builtin-c2c.c | 1 + tools/perf/builtin-kvm.c | 1 + tools/perf/builtin-report.c | 1 + tools/perf/builtin-top.c | 1 + tools/perf/ui/browsers/annotate.c | 1 + tools/perf/ui/browsers/hists.c | 1 + tools/perf/ui/browsers/scripts.c | 1 + tools/perf/ui/libslang.h | 2 ++ tools/perf/util/hist.h | 6 ------ 10 files changed, 17 insertions(+), 6 deletions(-) diff --git a/tools/perf/builtin-annotate.c b/tools/perf/builtin-annotate.c index f208ff513238..a4c014a1d28f 100644 --- a/tools/perf/builtin-annotate.c +++ b/tools/perf/builtin-annotate.c @@ -24,6 +24,7 @@ #include "arch/common.h" #include "builtin.h" #include "perf.h" +#include "ui/keysyms.h" #include "ui/progress.h" #include "ui/ui.h" #include "util/annotate-data.h" @@ -46,6 +47,13 @@ #include "util/tool.h" #include "util/util.h" +#ifndef HAVE_SLANG_SUPPORT +#undef K_LEFT +#define K_LEFT -1000 +#undef K_RIGHT +#define K_RIGHT -2000 +#endif + struct perf_annotate { struct perf_tool tool; struct perf_session *session; diff --git a/tools/perf/builtin-c2c.c b/tools/perf/builtin-c2c.c index 715b75d42f2a..6b64c0d6b24f 100644 --- a/tools/perf/builtin-c2c.c +++ b/tools/perf/builtin-c2c.c @@ -49,6 +49,7 @@ #include "thread.h" #include "tool.h" #include "ui/browsers/hists.h" +#include "ui/keysyms.h" #include "ui/progress.h" #include "ui/ui.h" #include "util/annotate.h" diff --git a/tools/perf/builtin-kvm.c b/tools/perf/builtin-kvm.c index b9f6cb71c0a0..c5e82c078848 100644 --- a/tools/perf/builtin-kvm.c +++ b/tools/perf/builtin-kvm.c @@ -27,6 +27,7 @@ #include "util/kvm-stat.h" #include "util/util.h" #include "ui/browsers/hists.h" +#include "ui/keysyms.h" #include "ui/progress.h" #include "ui/ui.h" #include "util/string2.h" diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c index 60d1f166629e..6f4ad5fa536d 100644 --- a/tools/perf/builtin-report.c +++ b/tools/perf/builtin-report.c @@ -51,6 +51,7 @@ #include "util/units.h" #include "util/unwind.h" #include "util/util.h" // perf_tip() +#include "ui/keysyms.h" #include "ui/ui.h" #include "ui/progress.h" #include "util/block-info.h" diff --git a/tools/perf/builtin-top.c b/tools/perf/builtin-top.c index 570410599f1b..44ba67e53cf1 100644 --- a/tools/perf/builtin-top.c +++ b/tools/perf/builtin-top.c @@ -51,6 +51,7 @@ #include "util/intlist.h" #include "util/parse-branch-options.h" #include "arch/common.h" +#include "ui/keysyms.h" #include "ui/ui.h" #include "util/debug.h" diff --git a/tools/perf/ui/browsers/annotate.c b/tools/perf/ui/browsers/annotate.c index e47a46775089..61ffdc52933d 100644 --- a/tools/perf/ui/browsers/annotate.c +++ b/tools/perf/ui/browsers/annotate.c @@ -1,6 +1,7 @@ // SPDX-License-Identifier: GPL-2.0 #include "../browser.h" #include "../helpline.h" +#include "../keysyms.h" #include "../ui.h" #include "../../util/annotate.h" #include "../../util/debug.h" diff --git a/tools/perf/ui/browsers/hists.c b/tools/perf/ui/browsers/hists.c index c15874a491b4..f62cb2d534ed 100644 --- a/tools/perf/ui/browsers/hists.c +++ b/tools/perf/ui/browsers/hists.c @@ -35,6 +35,7 @@ #include "../browsers/hists.h" #include "../helpline.h" +#include "../keysyms.h" #include "../util.h" #include "../ui.h" #include "map.h" diff --git a/tools/perf/ui/browsers/scripts.c b/tools/perf/ui/browsers/scripts.c index 1e8c2c2f952d..94cc1f427c96 100644 --- a/tools/perf/ui/browsers/scripts.c +++ b/tools/perf/ui/browsers/scripts.c @@ -7,6 +7,7 @@ #include "../../util/session.h" #include "../../util/symbol.h" #include "../browser.h" +#include "../keysyms.h" #include "../libslang.h" #include "config.h" #include diff --git a/tools/perf/ui/libslang.h b/tools/perf/ui/libslang.h index 6722561e0458..dd6b9f192155 100644 --- a/tools/perf/ui/libslang.h +++ b/tools/perf/ui/libslang.h @@ -15,7 +15,9 @@ #define ENABLE_SLFUTURE_CONST 1 #define ENABLE_SLFUTURE_VOID 1 +#ifdef HAVE_SLANG_SUPPORT #include +#endif #define SL_KEY_UNTAB 0x1000 diff --git a/tools/perf/util/hist.h b/tools/perf/util/hist.h index b830cbe7f95b..b30375a203e7 100644 --- a/tools/perf/util/hist.h +++ b/tools/perf/util/hist.h @@ -714,7 +714,6 @@ struct block_hist { #define NO_ADDR 0 #ifdef HAVE_SLANG_SUPPORT -#include "../ui/keysyms.h" void attr_to_script(char *buf, struct perf_event_attr *attr); int __hist_entry__tui_annotate(struct hist_entry *he, struct map_symbol *ms, @@ -787,11 +786,6 @@ static inline int block_hists_tui_browse(struct block_hist *bh __maybe_unused, { return 0; } - -#define K_LEFT -1000 -#define K_RIGHT -2000 -#define K_SWITCH_INPUT_DATA -3000 -#define K_RELOAD -4000 #endif unsigned int hists__sort_list_width(struct hists *hists); -- 2.55.0.966.g6673acef38-goog