All of lore.kernel.org
 help / color / mirror / Atom feed
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 12:04:55 +0100	[thread overview]
Message-ID: <20171201120455.6e8877fb.olaf@aepfle.de> (raw)
In-Reply-To: <20171201102146.m27cpsgm2jjcomgl@citrix.com>


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

Am Fri, 1 Dec 2017 10:21:46 +0000
schrieb Wei Liu <wei.liu2@citrix.com>:

> In Olaf's case, he cares about knowing whether the domain runs the
> controlling toolstack, he doesn't care about if it is the hardware
> domain or not, so my conclusion was using that flag was wrong.

I think this is not entirely accurate. Right now the term "dom0" is 
a mix of "has access to host (IO) hardware" and "runs the toolstack".

ConditionVirtualization= today lacks such details as well.
"xen" means domU, and "none" is dom0, simply to handle "dom0" like "native"
so that all services that want access to "host hardware" can start.

One could argue that passing a PCI device to a domU may also require
.service files to manage that PCI device in some way. The specifc case
which triggered all the suggested changes was smartd, which is not
supposed to run in "VMs". If a SATA card is provided to a domU it may
be a good idea to monitor the attached disks as well.

So in some way Jan is correct with his suggestion to use XENFEAT_dom0
instead of "control_d". I will do some research about when it became available
and update the patch description.

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

  reply	other threads:[~2017-12-01 11:04 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-29 15:38 [PATCH v1] core: mount xenfs, ignore proc-xen.mount (#6442, #6662) Olaf Hering
2017-11-29 15:46 ` Jan Beulich
2017-11-29 15:54   ` Olaf Hering
2017-11-29 16:01     ` Jan Beulich
     [not found]     ` <5A1EE7F40200007800193394@prv-mh.provo.novell.com>
2017-11-29 16:07       ` Olaf Hering
2017-11-29 16:24         ` Jan Beulich
     [not found]         ` <5A1EED6702000078001933EF@prv-mh.provo.novell.com>
2017-11-30  8:23           ` Olaf Hering
     [not found]           ` <20171130082355.GC3073@aepfle.de>
2017-11-30  8:35             ` Jan Beulich
     [not found]             ` <5A1FD0F10200007800193657@prv-mh.provo.novell.com>
2017-12-01 10:21               ` Wei Liu
2017-12-01 11:04                 ` Olaf Hering [this message]
2017-12-01 17:06                   ` [systemd-devel] " Lennart Poettering
2017-12-01 11:39                 ` Jan Beulich
     [not found]                 ` <5A214D820200007800193BFB@prv-mh.provo.novell.com>
2017-12-01 11:48                   ` Wei Liu
2017-12-01 12:11                     ` Jan Beulich
     [not found]                     ` <5A2155110200007800193C60@prv-mh.provo.novell.com>
2017-12-01 12:15                       ` Wei Liu
2017-12-01 12:23                         ` Jan Beulich
     [not found]                         ` <5A2157C40200007800193C90@prv-mh.provo.novell.com>
2017-12-01 12:29                           ` Wei Liu
2017-12-01 12:38                             ` Olaf Hering
2017-12-01 13:00                               ` Wei Liu
2017-12-01 15:49                                 ` Olaf Hering
     [not found]                                 ` <20171201154901.GA13177@aepfle.de>
2017-12-01 15:59                                   ` Wei Liu
2017-12-01 16:52                             ` Jan Beulich
2017-11-29 17:48 ` [systemd-devel] " Lennart Poettering

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20171201120455.6e8877fb.olaf@aepfle.de \
    --to=olaf@aepfle.de \
    --cc=JBeulich@suse.com \
    --cc=systemd-devel@lists.freedesktop.org \
    --cc=vliaskovitis@suse.com \
    --cc=wei.liu2@citrix.com \
    --cc=xen-devel@lists.xen.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.