From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel De Graaf Subject: Re: [PATCH 07/12] xen/event: Add reference counting to event channels Date: Wed, 14 Dec 2011 14:52:00 -0500 Message-ID: <4EE8FE60.30602@tycho.nsa.gov> References: <20111128152031.GB9655@andromeda.dapyr.net> <1322498951-4392-1-git-send-email-dgdegra@tycho.nsa.gov> <1322498951-4392-8-git-send-email-dgdegra@tycho.nsa.gov> <20111214185618.GC24598@phenom.dumpdata.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20111214185618.GC24598@phenom.dumpdata.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Konrad Rzeszutek Wilk Cc: xen-devel@lists.xensource.com, david.vrabel@citrix.com, JBeulich@suse.com, Ian.Campbell@citrix.com List-Id: xen-devel@lists.xenproject.org On 12/14/2011 01:56 PM, Konrad Rzeszutek Wilk wrote: > On Mon, Nov 28, 2011 at 11:49:06AM -0500, Daniel De Graaf wrote: >> Event channels exposed to userspace by the evtchn module may be used by >> other modules in an asynchronous manner, which requires that reference >> counting be used to prevent the event channel from being closed before >> the signals are delivered. >> >> The reference count on new event channels defaults to -1 which indicates >> the event channel is not referenced outside the kernel; evtchn_get fails >> if called on such an event channel. The event channels made visible to >> userspace by evtchn have a normal reference count. > > So I've 7 through 12 in my branch now. (7,8 and 9 are in the #linux-next > and the 10-12 are in the #testing). > > The other 1-6 you have might need to be parceled out (especially the netback > one which usually goes through David Miller). Also I have to double-check - > but did somebody review those 1-6 patches? > The last review I've seen of these patches is this thread: http://lists.xen.org/archives/html/xen-devel/2011-10/threads.html#01484 I'll post a v3 for those in a bit incorporating the comments in that thread; the netback one was (unofficially?) acked by David Miller. I don't see which of the other patches need to be parceled out as the files they touch look like they are still being committed with only Xen developer sign-offs.