From owner-cvs-all@FreeBSD.ORG Wed Sep 10 21:30:49 2008 Return-Path: Delivered-To: cvs-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 817251065693; Wed, 10 Sep 2008 21:30:49 +0000 (UTC) (envelope-from jhb@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 6A4738FC17; Wed, 10 Sep 2008 21:30:49 +0000 (UTC) (envelope-from jhb@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.3/8.14.3) with ESMTP id m8ALUn8c099686; Wed, 10 Sep 2008 21:30:49 GMT (envelope-from jhb@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.3/8.14.3/Submit) id m8ALUnV7099685; Wed, 10 Sep 2008 21:30:49 GMT (envelope-from jhb@repoman.freebsd.org) Message-Id: <200809102130.m8ALUnV7099685@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to jhb@repoman.freebsd.org using -f From: John Baldwin Date: Wed, 10 Sep 2008 21:30:35 +0000 (UTC) To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org X-FreeBSD-CVS-Branch: RELENG_6 Cc: Subject: cvs commit: src/sys/kern subr_sleepqueue.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 10 Sep 2008 21:30:49 -0000 jhb 2008-09-10 21:30:35 UTC FreeBSD src repository Modified files: (Branch: RELENG_6) sys/kern subr_sleepqueue.c Log: SVN rev 182920 on 2008-09-10 21:30:35Z by jhb Remove an assertion that is no longer correct with the addition of sx_sleep(). During sx_sleep(), we drop the sleepqueue chain lock after putting the thread on the queue but before checking for signals while we release the sx lock. If another thread wakes up the thread that is about to go to sleep during that window, then TDF_SINTR won't be set when the current thread checks for signals. Thus, it isn't accurate to always assert that that flag is set here anymore. Reported by: peter Approved by: re (kib) Revision Changes Path 1.18.2.10 +0 -1 src/sys/kern/subr_sleepqueue.c