All Virtuozzo development lists (kernel + QEMU)
 help / color / mirror / Atom feed
From: Konstantin Khorenko <khorenko@virtuozzo.com>
Subject: [Devel] [PATCH RHEL10 COMMIT] ms/net, pidfd: report EINVAL for ESRCH
Date: Mon, 24 Aug 2026 14:05:15 +0200	[thread overview]
Message-ID: <202608241205.67OC5FqW440373@hci8VM.finist.virtuozzo.com> (raw)
In-Reply-To: <20260824120457.440337-1-khorenko@virtuozzo.com>

The commit is pushed to "branch-rh10-6.12.0-211.39.1.16.x.vz10-ovz" and will appear at git at bitbucket.org:openvz/vzkernel.git
after rh10-6.12.0-211.39.1.16.8.vz10
------>
commit 24567fd12e301d424740a02cfbf67f761cb6b74d
Author: Christian Brauner <brauner@kernel.org>
Date:   Wed Apr 16 22:05:40 2025 +0200

    ms/net, pidfd: report EINVAL for ESRCH
    
    dbus-broker relies -EINVAL being returned to indicate ESRCH in [1].
    This causes issues for some workloads as reported in [2].
    Paper over it until this is fixed in userspace.
    
    Link: https://lore.kernel.org/20250416-gegriffen-tiefbau-70cfecb80ac8 at brauner
    Link: https://github.com/bus1/dbus-broker/blob/5d34d91b138fc802a016aa68c093eb81ea31139c/src/util/sockopt.c#L241 [1]
    Link: https://lore.kernel.org/20250415223454.GA1852104 at ax162 [2]
    Tested-by: Nathan Chancellor <nathan@kernel.org>
    Signed-off-by: Christian Brauner <brauner@kernel.org>
    
    (cherry picked from commit b590c928cca7bdc7fd580d52e42bfdc3ac5eeacb)
    https://virtuozzo.atlassian.net/browse/VSTOR-142602
    Feature: fix ms/net
    Signed-off-by: Konstantin Khorenko <khorenko@virtuozzo.com>
---
 net/core/sock.c | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/net/core/sock.c b/net/core/sock.c
index 7152c2f70792..5281899aaafd 100644
--- a/net/core/sock.c
+++ b/net/core/sock.c
@@ -1914,8 +1914,16 @@ int sk_getsockopt(struct sock *sk, int level, int optname,
 
 		pidfd = pidfd_prepare(peer_pid, 0, &pidfd_file);
 		put_pid(peer_pid);
-		if (pidfd < 0)
+		if (pidfd < 0) {
+			/*
+			 * dbus-broker relies on -EINVAL being returned
+			 * to indicate ESRCH. Paper over it until this
+			 * is fixed in userspace.
+			 */
+			if (pidfd == -ESRCH)
+				pidfd = -EINVAL;
 			return pidfd;
+		}
 
 		if (copy_to_sockptr(optval, &pidfd, len) ||
 		    copy_to_sockptr(optlen, &len, sizeof(int))) {

      reply	other threads:[~2026-08-24 12:05 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-24 12:04 [Devel] [PATCH vz10] " Konstantin Khorenko
2026-08-24 12:05 ` Konstantin Khorenko [this message]

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=202608241205.67OC5FqW440373@hci8VM.finist.virtuozzo.com \
    --to=khorenko@virtuozzo.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.