WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

[Xen-devel] Re: [PATCH v3] libvchan: interdomain communications library

To: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Subject: [Xen-devel] Re: [PATCH v3] libvchan: interdomain communications library
From: Daniel De Graaf <dgdegra@xxxxxxxxxxxxx>
Date: Thu, 01 Sep 2011 13:46:13 -0400
Cc: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <Stefano.Stabellini@xxxxxxxxxxxxx>, "konrad.wilk@xxxxxxxxxx" <konrad.wilk@xxxxxxxxxx>
Delivery-date: Thu, 01 Sep 2011 10:46:46 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <20063.47441.576537.375177@xxxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Organization: National Security Agency
References: <20055.42803.979775.531468@xxxxxxxxxxxxxxxxxxxxxxxx> <1314643714-28350-1-git-send-email-dgdegra@xxxxxxxxxxxxx> <1314700361.10283.135.camel@xxxxxxxxxxxxxxxxxxxxxx> <4E5E88CF.7090408@xxxxxxxxxxxxx> <1314894519.28989.143.camel@xxxxxxxxxxxxxxxxxxxxxx> <4E5FB72B.7010200@xxxxxxxxxxxxx> <20063.47441.576537.375177@xxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:5.0) Gecko/20110707 Thunderbird/5.0
On 09/01/2011 12:56 PM, Ian Jackson wrote:
> Daniel De Graaf writes ("Re: [PATCH v3] libvchan: interdomain communications 
> library"):
>> Making this change has exposed an interesting ordering dependency in the
>> notify API under Linux: the file descriptor for gntdev or gntalloc must be
>> less than the file descriptor for evtchn in order for the event channel to
>> still be active when the unmap occurs on a crash. The init functions of
>> libvchan do open the files in the proper order for this to happen.
> 
> Wow, that's pretty crazy.  Surely the gnt* fd should have an internal
> reference to the evtchn fd ?
> 
> Ian.
> 

The gnt* drivers will need to be changed to both find and take such a
reference; currently, they only refer to the port. This will probably
add a dependency from the gnt* module on evtchn; I'll look at what is
actually required to hold the event channel open when I make the patch.


-- 
Daniel De Graaf
National Security Agency

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

<Prev in Thread] Current Thread [Next in Thread>