From owner-svn-src-all@FreeBSD.ORG Thu May 12 10:56:00 2011 Return-Path: Delivered-To: svn-src-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 08509106566B; Thu, 12 May 2011 10:56:00 +0000 (UTC) (envelope-from jhb@FreeBSD.org) Received: from cyrus.watson.org (cyrus.watson.org [65.122.17.42]) by mx1.freebsd.org (Postfix) with ESMTP id CF4368FC08; Thu, 12 May 2011 10:55:59 +0000 (UTC) Received: from bigwig.baldwin.cx (66.111.2.69.static.nyinternet.net [66.111.2.69]) by cyrus.watson.org (Postfix) with ESMTPSA id 6CFBF46B03; Thu, 12 May 2011 06:55:59 -0400 (EDT) Received: from John-Baldwins-Macbook-Pro.local (unknown [24.114.252.233]) by bigwig.baldwin.cx (Postfix) with ESMTPSA id CB3228A027; Thu, 12 May 2011 06:55:58 -0400 (EDT) Message-ID: <4DCBBCBE.5020004@FreeBSD.org> Date: Thu, 12 May 2011 06:55:58 -0400 From: John Baldwin User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.2.17) Gecko/20110414 Thunderbird/3.1.10 MIME-Version: 1.0 To: Stanislav Sedov , neel@FreeBSD.org References: <201105091734.p49HY0P3006180@svn.freebsd.org> <20110512024956.996cd973.stas@FreeBSD.org> <4DCBB9EE.8070809@FreeBSD.org> <20110512035522.e42b379c.stas@FreeBSD.org> In-Reply-To: <20110512035522.e42b379c.stas@FreeBSD.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.2.6 (bigwig.baldwin.cx); Thu, 12 May 2011 06:55:59 -0400 (EDT) X-Mailman-Approved-At: Thu, 12 May 2011 11:43:48 +0000 Cc: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org, Jung-uk Kim , Andriy Gapon Subject: Re: svn commit: r221703 - in head/sys: amd64/include i386/include x86/isa x86/x86 X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 12 May 2011 10:56:00 -0000 On 5/12/11 6:55 AM, Stanislav Sedov wrote: > On Thu, 12 May 2011 13:43:58 +0300 > Andriy Gapon mentioned: > >> >> Theory: >> - smp_rv_waiters[2] becomes equal to smp_rv_ncpus >> - [at least] one slave CPU is still in the last call to cpu_spinwait() in >> smp_rendezvous_action() >> - master CPU notices that the condition is true, exits smp_rendezvous_cpus() and >> calls it again >> - the slave CPU is still in spinwait >> - the master CPU resets smp_rv_waiters[2] to zero >> - the slave CPU exits spinwait, see smp_rv_waiters[2] with zero value >> - endless loop >> > > That might explain it. > Do you have a patch for me to try? > > Thanks! > The NetApp folks working on BHyVe also ran into this. They have a fix that I think sounds reasonable which is to add a generation count to the smp rendezvous "structure" and have waiting CPUs stop waiting if the generation count changes. -- John Baldwin