Re: [PATCH v1] core: mount xenfs, ignore proc-xen.mount (#6442, #6662)

From: Olaf Hering <olaf@aepfle.de>
To: Wei Liu <wei.liu2@citrix.com>
Cc: systemd-devel@lists.freedesktop.org,
	Vasilis Liaskovitis <vliaskovitis@suse.com>,
	Jan Beulich <JBeulich@suse.com>,
	xen-devel@lists.xen.org
Subject: Re: [PATCH v1] core: mount xenfs, ignore proc-xen.mount (#6442, #6662)
Date: Fri, 1 Dec 2017 13:38:48 +0100	[thread overview]
Message-ID: <20171201133848.140d2063.olaf@aepfle.de> (raw)
In-Reply-To: <20171201122924.pz7tcv3wb3fyssdr@citrix.com>

[-- Attachment #1.1: Type: text/plain, Size: 864 bytes --]

Am Fri, 1 Dec 2017 12:29:24 +0000
schrieb Wei Liu <wei.liu2@citrix.com>:

> But Olaf needs to know if some of the services like xenconsoled or
> xenstored should be started, and if some of the special file systems
> should be mounted, right? Those aren't tied to hardware in anyway. In my
> view that's the responsibility of the toolstack control domain.

No, I did not intent to make use of ConditionVirtualization= in the
xen*.service files in tools/hotplug/Linux/. That variable can not be used
for this purpose, and the patch would not change that.

In case you refer to the "proc-xen.mount" change from a few days/weeks ago,
this was all about avoiding the error when xenfs becomes an "API filesystem".
With this suggested change the existing "proc-xen.mount units would not fail
anymore because /proc/xen is added to the ignore list.

Olaf

[-- Attachment #1.2: Digitale Signatur von OpenPGP --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 157 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel