From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel De Graaf Subject: Re: [PATCH 06/18] lib{xc, xl}: Seed grant tables with xenstore and console grants Date: Thu, 12 Jan 2012 12:32:26 -0500 Message-ID: <4F0F192A.5010503@tycho.nsa.gov> References: <1326302490-19428-1-git-send-email-dgdegra@tycho.nsa.gov> <1326302490-19428-7-git-send-email-dgdegra@tycho.nsa.gov> <1326362398.17210.227.camel@zakaz.uk.xensource.com> <4F0EF816.6030206@tycho.nsa.gov> <20239.5794.29197.280475@mariner.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20239.5794.29197.280475@mariner.uk.xensource.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: Ian Jackson Cc: "xen-devel@lists.xensource.com" , Alex Zeffertt , Ian Campbell , Diego Ongaro List-Id: xen-devel@lists.xenproject.org On 01/12/2012 12:21 PM, Ian Jackson wrote: > Daniel De Graaf writes ("Re: [Xen-devel] [PATCH 06/18] lib{xc, xl}: Seed grant tables with xenstore and console grants"): >> The previous patch series used a domid file named similarly to a pid file >> to identify the location of xenstored and xenconsoled; this method would >> allow the TODO to be resolved. I think a better solution is to refer to the >> xenstore/xenconsole domains by name instead of domid, and set the names in >> a configuration file (/etc/xen/xl.conf?). This would also replace the >> xenstore_dom/console_dom parameters in patch #5. > > Clearly this isn't a configuration parameter; it's a runtime value. > You don't store pids in config files. I was suggesting storing the domain name in the config file, which wouldn't be a runtime value. However, I think it should be possible to place this in xenstore, and that will probably be better as that's where you would expect things like this to be. > If this can't go in xenstore itself then a file in /var/run/xen would > do just fine. > > Ian. >