mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Philipp Zabel <p.zabel@pengutronix.de>
To: ptxdist@pengutronix.de
Cc: Philipp Zabel <p.zabel@pengutronix.de>
Subject: [ptxdist] [PATCH] igt-gpu-tools: version bump 1.30 -> 2.0
Date: Tue, 13 May 2025 09:39:26 +0200	[thread overview]
Message-ID: <20250513073926.3134132-1-p.zabel@pengutronix.de> (raw)

Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
 ...e-procps-ng-4.0.5-PIDS_VAL-takes-3-a.patch | 66 -------------------
 patches/igt-gpu-tools-1.30/series             |  4 --
 rules/igt-gpu-tools.make                      |  9 ++-
 3 files changed, 7 insertions(+), 72 deletions(-)
 delete mode 100644 patches/igt-gpu-tools-1.30/0001-lib-igt_aux-since-procps-ng-4.0.5-PIDS_VAL-takes-3-a.patch
 delete mode 100644 patches/igt-gpu-tools-1.30/series

diff --git a/patches/igt-gpu-tools-1.30/0001-lib-igt_aux-since-procps-ng-4.0.5-PIDS_VAL-takes-3-a.patch b/patches/igt-gpu-tools-1.30/0001-lib-igt_aux-since-procps-ng-4.0.5-PIDS_VAL-takes-3-a.patch
deleted file mode 100644
index 5dd5d5801e03..000000000000
--- a/patches/igt-gpu-tools-1.30/0001-lib-igt_aux-since-procps-ng-4.0.5-PIDS_VAL-takes-3-a.patch
+++ /dev/null
@@ -1,66 +0,0 @@
-From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
-Date: Fri, 3 Jan 2025 23:26:28 +0100
-Subject: [PATCH] lib/igt_aux: since procps-ng 4.0.5, PIDS_VAL() takes 3
- arguments, not 4
-
-Unfortunately, the API of procps-ng has changed between procps-ng
-4.0.4 and procps-ng 4.0.5, with the PIDS_VAL() macro changing in
-upstream commit
-https://gitlab.com/procps-ng/procps/-/commit/967fdcfb06e20aad0f31073474cb94545c9bdea5
-"library: expunge that 'info' parameter in 'VAL' macros"
-
-This API breakage has been reported upstream as of
-https://gitlab.com/procps-ng/procps/-/issues/366
-but in the mean time, the build of igt-gpu-tools breaks with:
-
-../lib/igt_aux.c:1358:78: error: macro "PIDS_VAL" passed 4 arguments, but takes just 3
-
-We fix this by detecting the version of the libproc2 library, and
-adjusting accordingly how PIDS_VAL() gets used.
-
-Cc: DeX77 <dex@dragonslave.de>
-Cc: Jim Warner <james.warner@comcast.net>
-Closes: https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/issues/164
-Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
-Reviewed-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
----
- lib/igt_aux.c | 7 +++++++
- meson.build   | 3 +++
- 2 files changed, 10 insertions(+)
-
-diff --git a/lib/igt_aux.c b/lib/igt_aux.c
-index 3407cc4f2aef..f5bf48da642b 100644
---- a/lib/igt_aux.c
-+++ b/lib/igt_aux.c
-@@ -1358,10 +1358,17 @@ static bool get_process_ids(struct igt_process *prcs)
- 	prcs->comm = NULL;
- 	prcs->stack = procps_pids_get(prcs->info, PIDS_FETCH_TASKS_ONLY);
- 	if (prcs->stack) {
-+#if defined(HAVE_LIBPROC2_POST_4_0_5_API)
-+		prcs->tid = PIDS_VAL(EU_PID, s_int, prcs->stack);
-+		prcs->euid = PIDS_VAL(EU_EUID, s_int, prcs->stack);
-+		prcs->egid = PIDS_VAL(EU_EGID, s_int, prcs->stack);
-+		prcs->comm = PIDS_VAL(EU_CMD, str, prcs->stack);
-+#else
- 		prcs->tid = PIDS_VAL(EU_PID, s_int, prcs->stack, prcs->info);
- 		prcs->euid = PIDS_VAL(EU_EUID, s_int, prcs->stack, prcs->info);
- 		prcs->egid = PIDS_VAL(EU_EGID, s_int, prcs->stack, prcs->info);
- 		prcs->comm = PIDS_VAL(EU_CMD, str, prcs->stack, prcs->info);
-+#endif
- 	}
- #endif
- 	return prcs->tid != 0;
-diff --git a/meson.build b/meson.build
-index ed2a79d5a2d9..2f663dc03737 100644
---- a/meson.build
-+++ b/meson.build
-@@ -139,6 +139,9 @@ if libprocps.found()
-   config.set('HAVE_LIBPROCPS', 1)
- elif libproc2.found()
-   config.set('HAVE_LIBPROC2', 1)
-+  if libproc2.version().version_compare('>= 4.0.5')
-+    config.set('HAVE_LIBPROC2_POST_4_0_5_API', 1)
-+  endif
- else
-   error('Either libprocps or libproc2 is required')
- endif
diff --git a/patches/igt-gpu-tools-1.30/series b/patches/igt-gpu-tools-1.30/series
deleted file mode 100644
index 67bd0bbce463..000000000000
--- a/patches/igt-gpu-tools-1.30/series
+++ /dev/null
@@ -1,4 +0,0 @@
-# generated by git-ptx-patches
-#tag:base --start-number 1
-0001-lib-igt_aux-since-procps-ng-4.0.5-PIDS_VAL-takes-3-a.patch
-# 1ccf3af41d04e1f9360b59bb14131a6f  - git-ptx-patches magic
diff --git a/rules/igt-gpu-tools.make b/rules/igt-gpu-tools.make
index 3f963ea31718..1097fd2f7875 100644
--- a/rules/igt-gpu-tools.make
+++ b/rules/igt-gpu-tools.make
@@ -14,8 +14,8 @@ PACKAGES-$(PTXCONF_IGT_GPU_TOOLS) += igt-gpu-tools
 #
 # Paths and names
 #
-IGT_GPU_TOOLS_VERSION	:= 1.30
-IGT_GPU_TOOLS_MD5	:= 10706e68a57f464408d309ae74bec5e2
+IGT_GPU_TOOLS_VERSION	:= 2.0
+IGT_GPU_TOOLS_MD5	:= dfd4dd415c3b3e53ddf4d13f4164a62b
 IGT_GPU_TOOLS		:= igt-gpu-tools-$(IGT_GPU_TOOLS_VERSION)
 IGT_GPU_TOOLS_SUFFIX	:= tar.xz
 IGT_GPU_TOOLS_URL	:= $(call ptx/mirror, XORG, individual/app/$(IGT_GPU_TOOLS).$(IGT_GPU_TOOLS_SUFFIX))
@@ -95,6 +95,7 @@ $(STATEDIR)/igt-gpu-tools.targetinstall:
 
 ifdef PTXCONF_IGT_GPU_TOOLS_INTEL
 	@$(call install_lib, igt-gpu-tools, 0, 0, 0644, libi915_perf)
+	@$(call install_lib, igt-gpu-tools, 0, 0, 0644, libxe_oa)
 
 	@$(call install_copy, igt-gpu-tools, 0, 0, 0755, -, /usr/bin/i915-perf-configs)
 	@$(call install_copy, igt-gpu-tools, 0, 0, 0755, -, /usr/bin/i915-perf-control)
@@ -133,6 +134,10 @@ ifdef PTXCONF_IGT_GPU_TOOLS_INTEL
 	@$(call install_copy, igt-gpu-tools, 0, 0, 0755, -, /usr/bin/intel_stepping)
 	@$(call install_copy, igt-gpu-tools, 0, 0, 0755, -, /usr/bin/intel_vbt_decode)
 	@$(call install_copy, igt-gpu-tools, 0, 0, 0755, -, /usr/bin/intel_watermark)
+	@$(call install_copy, igt-gpu-tools, 0, 0, 0755, -, /usr/bin/xe-perf-configs)
+	@$(call install_copy, igt-gpu-tools, 0, 0, 0755, -, /usr/bin/xe-perf-control)
+	@$(call install_copy, igt-gpu-tools, 0, 0, 0755, -, /usr/bin/xe-perf-reader)
+	@$(call install_copy, igt-gpu-tools, 0, 0, 0755, -, /usr/bin/xe-perf-recorder)
 
 	@$(call install_tree, igt-gpu-tools, 0, 0, -, /usr/share/igt-gpu-tools/registers)
 endif
-- 
2.39.5




                 reply	other threads:[~2025-05-13  7:39 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250513073926.3134132-1-p.zabel@pengutronix.de \
    --to=p.zabel@pengutronix.de \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox