From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Michael S. Tsirkin" Subject: Re: [PATCH 3/3] eventfd: add internal reference counting to fix notifier race conditions Date: Fri, 26 Jun 2009 14:23:31 +0300 Message-ID: <20090626112331.GA4950@redhat.com> References: <20090619183534.31118.30934.stgit@dev.haskins.net> <200906241255.54709.rusty@rustcorp.com.au> <200906252112.24730.rusty@rustcorp.com.au> <4A43B4A6.8050001@novell.com> <20090625182636.GA3239@redhat.com> <4A43C4C5.4040404@novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: avi@redhat.com, Davide Libenzi , Rusty Russell , kvm@vger.kernel.org, Linux Kernel Mailing List , paulmck@linux.vnet.ibm.com, Ingo Molnar To: Gregory Haskins Return-path: Received: from mx2.redhat.com ([66.187.237.31]:48107 "EHLO mx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752825AbZFZLYL (ORCPT ); Fri, 26 Jun 2009 07:24:11 -0400 Content-Disposition: inline In-Reply-To: <4A43C4C5.4040404@novell.com> Sender: kvm-owner@vger.kernel.org List-ID: On Thu, Jun 25, 2009 at 02:41:09PM -0400, Gregory Haskins wrote: > Michael S. Tsirkin wrote: > > On Thu, Jun 25, 2009 at 01:32:22PM -0400, Gregory Haskins wrote: > > > >> Please take a close look at it and consider for merging, if you would. > >> > > > > With all the incremental patching, I kind of lost track of what the > > complete file would look like. Is there a git tree I could pull? > > > > > > Ask and ye shall receive :) > > git pull > ssh://master.kernel.org/pub/scm/linux/kernel/git/ghaskins/linux-2.6-hacks.git > kvm/irqfd > > If you dont have a kernel.org account, it should rsync to the > gitweb/git: server soon, in which case > s|ssh://master|git://git should get you the proper path. > > Note that there is one additional race that I found that is not included > in that tree in the slow-work infrastructure. You can find my latest > submission to David Howells to resolve that issue here: > > http://patchwork.kernel.org/patch/32287/ > > Thanks Michael! > -Greg > Yes, I saw that. I agree your patch solves that issue. -- MST