From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olaf Hering Subject: Re: [PATCH] tools/hotplug: update systemd dependency to use service instead of socket Date: Fri, 5 Dec 2014 09:28:44 +0100 Message-ID: <20141205082844.GA643@aepfle.de> References: <1417534763-25053-1-git-send-email-olaf@aepfle.de> <1417535095.29004.4.camel@citrix.com> <20141202184240.GI32385@laptop.dumpdata.com> <20141203165410.GA2819@laptop.dumpdata.com> <20141204074756.GA4505@aepfle.de> <20141205021508.GB19424@andromeda.dapyr.net> <20141205074733.GA29524@aepfle.de> <20141205080307.GA31016@aepfle.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20141205080307.GA31016@aepfle.de> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Konrad Rzeszutek Wilk Cc: Wei Liu , Ian Campbell , Stefano Stabellini , Ian Jackson , xen-devel@lists.xen.org, M A Young List-Id: xen-devel@lists.xenproject.org On Fri, Dec 05, Olaf Hering wrote: > So looking again at > tools/hotplug/Linux/systemd/var-lib-xenstored.mount.in it seems that it > happens to work for me because XENSTORED_MOUNT_CTX is set within that > file. So if something happens to need a different value for > XENSTORED_MOUNT_CTX it has to be provided in the to-be-created config > file: EnvironmentFile=-@CONFIG_DIR@/@CONFIG_LEAF_DIR@/xenstored > This config file is not part of xen. And I wonder why a new config file has to be created, instead of just reusing the existing tools/hotplug/Linux/init.d/sysconfig.xencommons.in? I will send out a few patches to adjust the EnvironmentFile handling. Its just the question if a configure --with-selinux-mount-context=VAL is needed. Olaf