From owner-cvs-src@FreeBSD.ORG Fri Apr 11 08:12:37 2008 Return-Path: Delivered-To: cvs-src@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 10E38106566C; Fri, 11 Apr 2008 08:12:37 +0000 (UTC) (envelope-from rwatson@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 07E858FC12; Fri, 11 Apr 2008 08:12:37 +0000 (UTC) (envelope-from rwatson@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.1/8.14.1) with ESMTP id m3B8CaoQ064026; Fri, 11 Apr 2008 08:12:36 GMT (envelope-from rwatson@repoman.freebsd.org) Received: (from rwatson@localhost) by repoman.freebsd.org (8.14.1/8.14.1/Submit) id m3B8CaXQ064025; Fri, 11 Apr 2008 08:12:36 GMT (envelope-from rwatson) Message-Id: <200804110812.m3B8CaXQ064025@repoman.freebsd.org> From: Robert Watson Date: Fri, 11 Apr 2008 08:12:36 +0000 (UTC) To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org X-FreeBSD-CVS-Branch: RELENG_7 Cc: Subject: cvs commit: src/sys/kern kern_proc.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 11 Apr 2008 08:12:37 -0000 rwatson 2008-04-11 08:12:36 UTC FreeBSD src repository Modified files: (Branch: RELENG_7) sys/kern kern_proc.c Log: Merge kern_proc.c:1.259 from HEAD to RELENG_7: Return ESRCH when a kernel stack is queried on a process in execve() -- p_candebug() will return EAGAIN which, if the other process never leaves execve(), will result in the sysctl spinning and never returning to userspace. Processes should always eventually leave execve(), but spinning in kernel while we wait is bad for countless reasons, and particularly harmful if execve() itself is deadlocked. Possibly we should return another error, or return a marker indicating the thread is in execve() so it can be reported that way in userspace. Reported by: kris Revision Changes Path 1.252.2.7 +2 -1 src/sys/kern/kern_proc.c