From nobody Thu Dec 18 04:49:37 2025 Received: from mail-pl1-f202.google.com (mail-pl1-f202.google.com [209.85.214.202]) (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 9C0511F30D1 for ; Mon, 24 Mar 2025 22:35:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.202 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742855712; cv=none; b=LNWDfpnfb10++9Q5TdDGLWrhj6Ev7H+J+OBCWQFJRkszOn1GLcIr+/0HojTfHR8lL6A/9eE2ZzMl6jBebnfZ73/TRi0RqpCOvLJr8f8gtD15Idmb+k5tiICdyqop/BqURFN8swRpPSZsoXYMa/ijtjuZQFc2LabQsnGCs340plE= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742855712; c=relaxed/simple; bh=jq5PjJ/ugYrx4xcRpoE6fLhSyd7TK8mfub7jd04bhO0=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Content-Type; b=UNcfTTCUWuhxaBQAN65GOXwtXZg7Q/EcVuDc1p3e+zzJt/xPUnliGKMFCR/TMIzr7R6COnQdKUV883WSFw7Ye+SEPrNCUPj7p7ak4ZYqvZsxKGbjZyYCuhzZntgpEIDoKdVOS08OSZJ5RlqiI00Avxeir59rhzCrmX6VRxxEmqE= 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=STSKn9zx; arc=none smtp.client-ip=209.85.214.202 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="STSKn9zx" Received: by mail-pl1-f202.google.com with SMTP id d9443c01a7336-225505d1ca5so77962735ad.2 for ; Mon, 24 Mar 2025 15:35:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1742855709; x=1743460509; darn=vger.kernel.org; h=to:from:subject:message-id:references:mime-version:in-reply-to:date :from:to:cc:subject:date:message-id:reply-to; bh=Hi9qV/98s3GAaLJGCI97xHdsmqnmvPcBtjZEurAgWmA=; b=STSKn9zx6+srrxS9qyjWdUA0eBvYKEvCd8GOYBRTp/d/v/gVi3lj7xAK1UaZo7Jrq3 YnmWdwTsaBq0xDA2+hJ4o4i2Kjt9iqC2ETNadjwLrsDDTSHU8wSud42r3Z4Cc59rNzjB mJHUlmayxvP9wwSaahj1FjW6mXD6A3TWVP20xF1kC+zWMhpKbZyesk5bFSbMh2SbnuPf 0b3E/7lLUmZuolVUmAxNJHfUyFYjPORT2QbSutBIt1gbJ2ljnVKz9fUNKlK/A2JhCJC5 P+DxTo99NZXOlVcw8Zp88CS2iimY4ZX44neE3XgFq3lkRNUaVi+7ReFosF/EOKfyjVBO rbRw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1742855709; x=1743460509; h=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; bh=Hi9qV/98s3GAaLJGCI97xHdsmqnmvPcBtjZEurAgWmA=; b=u1yG88XamMXtDzQmz0IcYWFskLNuOm5V7zyEcWqkto1u5j2cCIo+vv55w4Jq6ciML8 o6zC4fPuFZlvv5U+7CB9RihnvL9NdhFQQ959DzwVBM/dHYXpiQKwgr66JZ27HFwt3nQ8 96Ar9NiwvHpVBt1zIIdnGQxEpHREbXHGcnVQV6grtGUcG9eQIG6mM6oW7zJPS7p2kG8i Ccms9f8G2GvbKqr7wWXp8vkiBQ4UL2t3GBOQgF8+/b/rWzopyM0KvABJXBzFBDspix8u G0tax1WDna90QPxRIHIghTzlTq/4pA1TlKsx667x0K875hHmWIflNRJBVoFflBraegJT C4rQ== X-Forwarded-Encrypted: i=1; AJvYcCWWKjhC4TpkQKwxYgzPyEFTSG7rAWNYKrvTD7Ki3EtzPjgdAlALpj42vEsDMsMq0ppvEJl+9tN6BrSz4WM=@vger.kernel.org X-Gm-Message-State: AOJu0Yy0wU8RJd1huK0UXzQ046PWz1GQaemTTmHD5tCE5rokSbmMpUZB SorBjsib43SgwiippvqCs3+o/voq9GoSARQQ41/BU+DnAxsMnooJDzFqySk+xNoKYRpFUT0KJ3m pQS3x3w== X-Google-Smtp-Source: AGHT+IGFxmafsPpLyD17+00wez3sX1EFe8IKkFijnkrwpG/ZYSYE6tSXfbTvjzfIcsV17fEt155JaIjS1hm6 X-Received: from pgbdo14.prod.google.com ([2002:a05:6a02:e8e:b0:af2:48c3:360f]) (user=irogers job=prod-delivery.src-stubby-dispatcher) by 2002:a05:6a20:3d84:b0:1f5:7f2e:5c3c with SMTP id adf61e73a8af0-1fe42f0920bmr20802083637.1.1742855708807; Mon, 24 Mar 2025 15:35:08 -0700 (PDT) Date: Mon, 24 Mar 2025 15:34:44 -0700 In-Reply-To: <20250324223452.208081-1-irogers@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20250324223452.208081-1-irogers@google.com> X-Mailer: git-send-email 2.49.0.395.g12beb8f557-goog Message-ID: <20250324223452.208081-7-irogers@google.com> Subject: [PATCH v1 06/14] perf intel-tpebs: Inline get_perf_record_args From: Ian Rogers To: Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Namhyung Kim , Mark Rutland , Alexander Shishkin , Jiri Olsa , Ian Rogers , Adrian Hunter , Kan Liang , Weilin Wang , James Clark , Xu Yang , John Garry , Howard Chu , Levi Yun , Dominique Martinet , linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Code is short enough to be inlined and there are no error cases when made inline. Make the implicit NULL pointer at the end of the argv explicit. Move the fixed number of arguments before the variable number of arguments. Correctly size the argv allocation and zero when feeing to avoid a dangling pointer. Signed-off-by: Ian Rogers --- tools/perf/util/intel-tpebs.c | 75 +++++++++++++---------------------- 1 file changed, 28 insertions(+), 47 deletions(-) diff --git a/tools/perf/util/intel-tpebs.c b/tools/perf/util/intel-tpebs.c index e5ca58599b5f..3141b7d2e94f 100644 --- a/tools/perf/util/intel-tpebs.c +++ b/tools/perf/util/intel-tpebs.c @@ -46,72 +46,53 @@ struct tpebs_retire_lat { double val; }; =20 -static int get_perf_record_args(const char **record_argv, char buf[], - const char *cpumap_buf) +static int evsel__tpebs_start_perf_record(struct evsel *evsel, int control= _fd[], int ack_fd[]) { - struct tpebs_retire_lat *e; - int i =3D 0; + const char **record_argv; + int tpebs_event_size =3D 0, i =3D 0, ret; + char control_fd_buf[32]; + char cpumap_buf[50]; + struct tpebs_retire_lat *t; + + list_for_each_entry(t, &tpebs_results, nd) + tpebs_event_size++; =20 - pr_debug("tpebs: Prepare perf record for retire_latency\n"); + record_argv =3D malloc((10 + 2 * tpebs_event_size) * sizeof(*record_argv)= ); + if (!record_argv) + return -ENOMEM; =20 record_argv[i++] =3D "perf"; record_argv[i++] =3D "record"; record_argv[i++] =3D "-W"; record_argv[i++] =3D "--synth=3Dno"; - record_argv[i++] =3D buf; =20 - if (!cpumap_buf) { - pr_err("tpebs: Require cpumap list to run sampling\n"); - return -ECANCELED; - } - /* Use -C when cpumap_buf is not "-1" */ - if (strcmp(cpumap_buf, "-1")) { + scnprintf(control_fd_buf, sizeof(control_fd_buf), "--control=3Dfd:%d,%d", + control_fd[0], ack_fd[1]); + record_argv[i++] =3D control_fd_buf; + + record_argv[i++] =3D "-o"; + record_argv[i++] =3D PERF_DATA; + + if (!perf_cpu_map__is_any_cpu_or_is_empty(evsel->evlist->core.user_reques= ted_cpus)) { + cpu_map__snprint(evsel->evlist->core.user_requested_cpus, cpumap_buf, + sizeof(cpumap_buf)); record_argv[i++] =3D "-C"; record_argv[i++] =3D cpumap_buf; } =20 - list_for_each_entry(e, &tpebs_results, nd) { + list_for_each_entry(t, &tpebs_results, nd) { record_argv[i++] =3D "-e"; - record_argv[i++] =3D e->name; + record_argv[i++] =3D t->name; } - - record_argv[i++] =3D "-o"; - record_argv[i++] =3D PERF_DATA; - - return 0; -} - -static int evsel__tpebs_start_perf_record(struct evsel *evsel, int control= _fd[], int ack_fd[]) -{ - const char **record_argv; - size_t tpebs_event_size =3D 0; - int ret; - char buf[32]; - char cpumap_buf[50]; - struct tpebs_retire_lat *t; - - cpu_map__snprint(evsel->evlist->core.user_requested_cpus, cpumap_buf, - sizeof(cpumap_buf)); - - scnprintf(buf, sizeof(buf), "--control=3Dfd:%d,%d", control_fd[0], ack_fd= [1]); - - list_for_each_entry(t, &tpebs_results, nd) - tpebs_event_size++; - - record_argv =3D calloc(12 + 2 * tpebs_event_size, sizeof(char *)); - if (!record_argv) - return -ENOMEM; - - ret =3D get_perf_record_args(record_argv, buf, cpumap_buf); - if (ret) - goto out; + record_argv[i++] =3D NULL; + assert(i =3D=3D 10 + 2 * tpebs_event_size || i =3D=3D 8 + 2 * tpebs_event= _size); + /* Note, no workload given so system wide is implied. */ =20 assert(tpebs_cmd.pid =3D=3D 0); tpebs_cmd.argv =3D record_argv; tpebs_cmd.out =3D -1; ret =3D start_command(&tpebs_cmd); -out: - free(record_argv); + zfree(&tpebs_cmd.argv); return ret; } =20 --=20 2.49.0.395.g12beb8f557-goog