From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel De Graaf Subject: Re: [PATCH v2 1/2] xenconsoled: use grant references instead of map_foreign_range Date: Thu, 17 Jan 2013 12:15:28 -0500 Message-ID: <50F831B0.8050707@tycho.nsa.gov> References: <1357838913-29968-1-git-send-email-dgdegra@tycho.nsa.gov> <1358441447.13856.94.camel@zakaz.uk.xensource.com> <50F82DC8.4060704@tycho.nsa.gov> <1358442356.13856.99.camel@zakaz.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1358442356.13856.99.camel@zakaz.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell Cc: Stefano Stabellini , Ian Jackson , "xen-devel@lists.xen.org" List-Id: xen-devel@lists.xenproject.org On 01/17/2013 12:05 PM, Ian Campbell wrote: > On Thu, 2013-01-17 at 16:58 +0000, Daniel De Graaf wrote: >> On 01/17/2013 11:50 AM, Ian Campbell wrote: >>> On Thu, 2013-01-10 at 17:28 +0000, Daniel De Graaf wrote: >>>> This allows xenconsoled to be run in a domain other than dom0 if set up >>>> correctly - for libxl, the xenstore path /tool/xenconsoled/domid >>>> specifies the domain containing xenconsoled. >>> >>> Have I missed the patch which writes that path? >>> >> >> There is currently no patch which writes to that path; this would be done >> by the scripts that launch the domain that xenconsoled resides in. I don't >> think having xenconsoled write its domain ID to that path is useful - it >> would require waiting for the xenconsoled domain and daemon to start before >> building other guests, whereas writing the path from dom0 allows setting >> everything up first and letting xenconsoled catch up later. > > That makes sense. > > I have a follow up question though. Have I missed the patch which reads > this path ;-) > > Ian. > That was done in the same patch that reads xenstored's path, and it was put in quite a while ago - the code is in tools/libxl/libxl_dom.c -- Daniel De Graaf National Security Agency