unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#73733] [PATCH] gnu: lsof: fix build in Linux 6.9
@ 2024-10-10 13:14 Ekaitz Zarraga
  2024-10-10 14:50 ` bug#73733: " Efraim Flashner
  0 siblings, 1 reply; 2+ messages in thread
From: Ekaitz Zarraga @ 2024-10-10 13:14 UTC (permalink / raw)
  To: 73733; +Cc: Ekaitz Zarraga

This commit adds the patch that fixes:
<https://github.com/lsof-org/lsof/issues/317>

The issue was triggering the following error in the `test` phase that
resulted in a build failure in Linux 6.9 or higher:

    FAIL: lib/dialects/linux/tests/case-20-pidfd-pid.bash

* gnu/packages/lsof.scm(patches): Add `lsof-compat-linux-6.9.patch`.
* gnu/packages/patches/lsof-compat-linux-6.9.patch: New file.

Change-Id: I98b895ecaf1b952bfd33ea7ae3af70b1c39a0a94
---
 gnu/packages/lsof.scm                         |  4 +-
 .../patches/lsof-compat-linux-6.9.patch       | 58 +++++++++++++++++++
 2 files changed, 61 insertions(+), 1 deletion(-)
 create mode 100644 gnu/packages/patches/lsof-compat-linux-6.9.patch

diff --git a/gnu/packages/lsof.scm b/gnu/packages/lsof.scm
index 7d849c80b5..d76ef585c3 100644
--- a/gnu/packages/lsof.scm
+++ b/gnu/packages/lsof.scm
@@ -46,7 +46,9 @@ (define-public lsof
               (file-name (git-file-name name version))
               (sha256
                (base32
-                "1v32407al4j0hhcph95lv4xvr9h012lii29iyq41iwj39zwfavax"))))
+                "1v32407al4j0hhcph95lv4xvr9h012lii29iyq41iwj39zwfavax"))
+              ;; FIXES: https://github.com/lsof-org/lsof/issues/317
+              (patches (search-patches "lsof-compat-linux-6.9.patch"))))
     (build-system gnu-build-system)
     (native-inputs (list automake
                          autoconf
diff --git a/gnu/packages/patches/lsof-compat-linux-6.9.patch b/gnu/packages/patches/lsof-compat-linux-6.9.patch
new file mode 100644
index 0000000000..419cae927a
--- /dev/null
+++ b/gnu/packages/patches/lsof-compat-linux-6.9.patch
@@ -0,0 +1,58 @@
+From c1678e3f6e4b4d984cb3078b7bf0c9e24bedb8ca Mon Sep 17 00:00:00 2001
+From: Jiajie Chen <c@jia.je>
+Date: Fri, 17 May 2024 15:22:35 +0800
+Subject: [PATCH] [linux] Maintain original output for pidfd in linux 6.9
+
+---
+ 00DIST                     |  3 +++
+ lib/dialects/linux/dproc.c | 13 +++++++++++++
+ 2 files changed, 16 insertions(+)
+
+diff --git a/00DIST b/00DIST
+index d5a0349..9a29611 100644
+--- a/00DIST
++++ b/00DIST
+@@ -5605,6 +5605,9 @@ Supplement	Regenerated the 4.04 distribution to correct a non-
+ 
+ 		[linux] Do not embed kernel version in CFLAGS (#314)
+ 
++		[linux] Linux 6.9 changed the pidfs appearence in procfs. Try
++		to maintain original output in lsof (#317)
++
+ 
+ Vic Abell <abe@purdue.edu>
+ July 14, 2018
+diff --git a/lib/dialects/linux/dproc.c b/lib/dialects/linux/dproc.c
+index cb6bb64..3a7a120 100644
+--- a/lib/dialects/linux/dproc.c
++++ b/lib/dialects/linux/dproc.c
+@@ -1273,6 +1273,10 @@ static int process_id(struct lsof_context *ctx, /* context */
+ #endif     /* defined(HASEPTOPTS) */
+                 if (rest && rest[0] == '[' && rest[1] == 'p')
+                     fdinfo_mask |= FDINFO_PID;
++                else if (Lf->ntype == N_REGLR && rest && *rest && strcmp(pbuf, "pidfd") == 0) {
++                    // https://github.com/lsof-org/lsof/issues/317
++                    fdinfo_mask |= FDINFO_PID;
++                }
+ 
+                 if ((av = get_fdinfo(ctx, pathi, fdinfo_mask, &fi)) &
+                     FDINFO_POS) {
+@@ -1338,6 +1342,15 @@ static int process_id(struct lsof_context *ctx, /* context */
+                     Lf->sf |= SELPTYINFO;
+                 }
+ #endif /* defined(HASEPTOPTS) && defined(HASPTYEPT) */
++                else if (Lf->ntype == N_REGLR && rest && *rest && Lf->nm &&
++                         strcmp(Lf->nm, "pidfd") == 0) {
++                    // https://github.com/lsof-org/lsof/issues/317
++                    // pidfd since Linux 6.9 becomes a regular file:
++                    // /proc/PID/fd/FD -> pidfd:[INODE]
++                    (void)snpf(rest, sizeof(pbuf) - (rest - pbuf),
++                                "[pidfd:%d]", fi.pid);
++                    enter_nm(ctx, rest);
++                }
+ 
+                 if (Lf->sf)
+                     link_lfile(ctx);
+-- 
+2.45.2
+

base-commit: f5aff5b3972b9a3342b1ce807f7ed68289cce74d
-- 
2.46.0





^ permalink raw reply related	[flat|nested] 2+ messages in thread

* bug#73733: [PATCH] gnu: lsof: fix build in Linux 6.9
  2024-10-10 13:14 [bug#73733] [PATCH] gnu: lsof: fix build in Linux 6.9 Ekaitz Zarraga
@ 2024-10-10 14:50 ` Efraim Flashner
  0 siblings, 0 replies; 2+ messages in thread
From: Efraim Flashner @ 2024-10-10 14:50 UTC (permalink / raw)
  To: Ekaitz Zarraga; +Cc: 73733-done

[-- Attachment #1: Type: text/plain, Size: 268 bytes --]

Thanks for the patch. I added a line to gnu/local.mk.

-- 
Efraim Flashner   <efraim@flashner.co.il>   רנשלפ םירפא
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-10-10 14:53 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-10 13:14 [bug#73733] [PATCH] gnu: lsof: fix build in Linux 6.9 Ekaitz Zarraga
2024-10-10 14:50 ` bug#73733: " Efraim Flashner

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).