git: 80626f34ee98 - main - kern_proc.c: make kern.proc.osrel atomic

The branch main has been updated by kib:

URL: https://cgit.FreeBSD.org/src/commit/?id=80626f34ee985671bb8c60ee986b89587b7a1511

commit 80626f34ee985671bb8c60ee986b89587b7a1511
Author:     Konstantin Belousov <kib@FreeBSD.org>
AuthorDate: 2026-06-15 17:28:15 +0000
Commit:     Konstantin Belousov <kib@FreeBSD.org>
CommitDate: 2026-06-21 11:46:53 +0000

    kern_proc.c: make kern.proc.osrel atomic
    
    Reviewed by:    markj
    Tested by:      pho
    Sponsored by:   The FreeBSD Foundation
    MFC after:      1 week
    Differential revision:  https://reviews.freebsd.org/D57497
---
 sys/kern/kern_proc.c | 45 ++++++++++++++++++++++-----------------------
 1 file changed, 22 insertions(+), 23 deletions(-)

diff --git a/sys/kern/kern_proc.c b/sys/kern/kern_proc.c
index a42d528cc9c1..17b4effde030 100644
--- a/sys/kern/kern_proc.c
+++ b/sys/kern/kern_proc.c
@@ -3121,39 +3121,38 @@ sysctl_kern_proc_osrel(SYSCTL_HANDLER_ARGS)
 	int *name = (int *)arg1;
 	u_int namelen = arg2;
 	struct proc *p;
-	int flags, error, osrel;
+	int flags, error, old_osrel, osrel;
 
 	if (namelen != 1)
 		return (EINVAL);
 
-	if (req->newptr != NULL && req->newlen != sizeof(osrel))
-		return (EINVAL);
-
-	flags = PGET_HOLD | PGET_NOTWEXIT;
-	if (req->newptr != NULL)
+	flags = PGET_NOTWEXIT;
+	if (req->newptr != NULL) {
+		if (req->newlen != sizeof(osrel))
+			return (EINVAL);
+		error = SYSCTL_IN(req, &osrel, sizeof(osrel));
+		if (error != 0)
+			return (error);
+		if (osrel < 0)
+			return (EINVAL);
 		flags |= PGET_CANDEBUG;
-	else
+	} else {
 		flags |= PGET_CANSEE;
+	}
 	error = pget((pid_t)name[0], flags, &p);
 	if (error != 0)
 		return (error);
-
-	error = SYSCTL_OUT(req, &p->p_osrel, sizeof(p->p_osrel));
-	if (error != 0)
-		goto errout;
-
-	if (req->newptr != NULL) {
-		error = SYSCTL_IN(req, &osrel, sizeof(osrel));
-		if (error != 0)
-			goto errout;
-		if (osrel < 0) {
-			error = EINVAL;
-			goto errout;
-		}
-		p->p_osrel = osrel;
+	if ((p->p_flag & P_INEXEC) != 0) {
+		error = EBUSY;
+	} else {
+		old_osrel = p->p_osrel;
+		if (req->newptr != NULL)
+			p->p_osrel = osrel;
 	}
-errout:
-	PRELE(p);
+	PROC_UNLOCK(p);
+
+	if (error == 0)
+		error = SYSCTL_OUT(req, &old_osrel, sizeof(old_osrel));
 	return (error);
 }