From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [PATCH] libxl_pci: check that host device is assignable before adding to the domain Date: Tue, 17 Jan 2012 16:12:45 -0400 Message-ID: <20120117201245.GA25805@andromeda.dapyr.net> References: <301cc006677f645bf4cb.1326753363@mnetdjm4.mageenet.host> <1326793645.14689.75.camel@zakaz.uk.xensource.com> <1326809782.14689.114.camel@zakaz.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: djmagee@mageenet.net Cc: xen-devel@lists.xensource.com, Ian Jackson , Ian Campbell , Stefano Stabellini List-Id: xen-devel@lists.xenproject.org On Tue, Jan 17, 2012 at 09:24:39AM -0500, djmagee@mageenet.net wrote: > > -----Original Message----- > > From: Ian Campbell [mailto:Ian.Campbell@citrix.com] > > Sent: Tuesday, January 17, 2012 9:16 AM > > To: djmagee@mageenet.net > > Cc: xen-devel@lists.xensource.com; Ian Jackson; Stefano Stabellini > > Subject: RE: [Xen-devel] [PATCH] libxl_pci: check that host device is > > assignable before adding to the domain > > > > Please don't top post. > > > > On Tue, 2012-01-17 at 14:05 +0000, djmagee@mageenet.net wrote: > > > That was my first thought, but this was also the first thing in libxl > > > I touched and wasn't sure what would happen if I ended up nesting > > > GC_INITs. If it's safe to do then I can call > > > libxl_device_pci_list_assignable, otherwise I'd have to pull the meat > > > out and put it in another function. What's the best way to do it? > > > > You can use libxl__gc_owner(gc) (or the helpful CTX macro) to get a ctx > > from a gc which you use to call an externally visible function from > > within libxl -- it'll do the right thing (TM). > > Cool, I'll send an updated patch along today. You might want to check for 'xen-pciback', 'pciback' and 'pci-stub' as they are valid. > > Doug > > > > > Ian. > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xensource.com > http://lists.xensource.com/xen-devel