From owner-cvs-all Wed Apr 4 23:50:14 2001 Delivered-To: cvs-all@freebsd.org Received: from gratis.grondar.za (grouter.grondar.za [196.7.18.65]) by hub.freebsd.org (Postfix) with ESMTP id 63C9337B422; Wed, 4 Apr 2001 23:50:02 -0700 (PDT) (envelope-from mark@grondar.za) Received: from grondar.za (root@gratis.grondar.za [196.7.18.133]) by gratis.grondar.za (8.11.1/8.11.1) with ESMTP id f356nrH26443; Thu, 5 Apr 2001 08:49:56 +0200 (SAST) (envelope-from mark@grondar.za) Message-Id: <200104050649.f356nrH26443@gratis.grondar.za> To: John Baldwin Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern kern_sig.c References: <200104021726.f32HQqE97399@freefall.freebsd.org> In-Reply-To: <200104021726.f32HQqE97399@freefall.freebsd.org> ; from John Baldwin "Mon, 02 Apr 2001 10:26:52 MST." Date: Thu, 05 Apr 2001 08:51:11 +0200 From: Mark Murray Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG Things are looking very good! :-) M > jhb 2001/04/02 10:26:51 PDT > > Modified files: > sys/kern kern_sig.c > Log: > - Move the second stop() of process 'p' in issignal() to be after we send > SIGCHLD to our parent process. Otherwise, we could block while obtaining > the process lock for our parent process and switch out while we were > in SSTOP. Even worse, when we try to resume from the mutex being blocked > on our p_stat will be SRUN, not SSTOP. > - Fix a comment above stop() to indicate that it requires that the proc lock > be held, not a proctree lock. > > Reported by: markm > Sleuthing by: jake > > Revision Changes Path > 1.113 +3 -4 src/sys/kern/kern_sig.c > > -- Mark Murray Warning: this .sig is umop ap!sdn To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message