From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [PATCH 07/12] xen/event: Add reference counting to event channels Date: Fri, 16 Dec 2011 15:35:00 -0400 Message-ID: <20111216193459.GA26802@andromeda.dapyr.net> 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> <4EE8FE60.30602@tycho.nsa.gov> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <4EE8FE60.30602@tycho.nsa.gov> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Daniel De Graaf Cc: Ian.Campbell@citrix.com, xen-devel@lists.xensource.com, david.vrabel@citrix.com, JBeulich@suse.com, Konrad Rzeszutek Wilk List-Id: xen-devel@lists.xenproject.org On Wed, Dec 14, 2011 at 02:52:00PM -0500, Daniel De Graaf wrote: > 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 Yup. I see it now. Thx > 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. > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xensource.com > http://lists.xensource.com/xen-devel