From: Marek Marczykowski <marmarek@invisiblethingslab.com>
To: George Dunlap <george.dunlap@eu.citrix.com>
Cc: Stefano Stabellini <Stefano.Stabellini@eu.citrix.com>,
Ian Jackson <Ian.Jackson@eu.citrix.com>,
Ian Campbell <Ian.Campbell@citrix.com>,
"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
Subject: Re: [PATCH 0/7] Optionally add libvchan to stubdom
Date: Fri, 26 Apr 2013 17:02:01 +0200 [thread overview]
Message-ID: <517A96E9.1030108@invisiblethingslab.com> (raw)
In-Reply-To: <517A95E5.3080209@eu.citrix.com>
[-- Attachment #1.1: Type: text/plain, Size: 1984 bytes --]
On 26.04.2013 16:57, George Dunlap wrote:
> On 26/04/13 15:56, Marek Marczykowski wrote:
>> On 26.04.2013 16:55, George Dunlap wrote:
>>> On Fri, Apr 26, 2013 at 3:45 PM, Marek Marczykowski
>>> <marmarek@invisiblethingslab.com> wrote:
>>>> This patch series enhances stubdomain with libvchan. It is disabled by
>>>> default,
>>>> but can be easily enabled in mini-os.cfg when needed.
>>>> In order to do so, some parts of minios needs to be extended:
>>>> - xenstore (to support transactions)
>>>> - libxc - to support xc_gntshr
>>>>
>>>> This series assumes "libxc: fix xc_gntshr_munmap" already applied.
>>>>
>>>> One unsolved problem is implementation of xc_gntshr_share_page_notify. On
>>>> Linux,
>>>> there is kernel, which can notify remote in case of server process death. In
>>>> stubdom there is no such place - if exit() is called anywhere, the whole
>>>> stubdom is terminated. Also the common case is stubdom destruction (the case
>>>> for ioemu stubdom - it is destroyed by toolstack at domain shutdown), I don't
>>>> know if stubdom even support clean shutdown.
>>>> Anyway I can implement some notification (which needs some additional info
>>>> stored by minios version of gnttab), just for clean libxenvchan_close().
>>>> But it
>>>> is rather rare case.
>>>> Some workaround can be checking for domain state at client side, but IMHO it
>>>> isn't the best way (although still used in original Qubes OS vchan version).
>>>> Any ideas?
>>>>
>>>> Patch 2 is independent - should be also useful for normal Linux case.
>>> I assume your targeting this for 4.4?
>> Probably yes, I assume it is too late for 4.3, right?
>
> Unless you've got a really amazing justification, yes. :-)
Hmm... let me try: "it is disable in default build". But it isn't entirely
true, especially not for patch no 2.
Anyway I'll be happy to hear some comments on this.
--
Best Regards / Pozdrawiam,
Marek Marczykowski
Invisible Things Lab
[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 553 bytes --]
[-- Attachment #2: Type: text/plain, Size: 126 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
next prev parent reply other threads:[~2013-04-26 15:02 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-26 14:45 [PATCH 0/7] Optionally add libvchan to stubdom Marek Marczykowski
2013-04-21 1:46 ` [PATCH 2/7] libvchan: create xenstore entries in one transaction Marek Marczykowski
2013-04-26 18:04 ` David Vrabel
2013-04-28 22:43 ` Marek Marczykowski
2013-04-22 1:44 ` [PATCH 1/7] minios: enhance xenstore available for stubdoms Marek Marczykowski
2013-04-28 22:34 ` Samuel Thibault
2013-04-22 1:50 ` [PATCH 3/7] libvchan: remove unnecessary includes Marek Marczykowski
2013-04-22 2:10 ` [PATCH 6/7] stubdom: make libvchan available in stubdom Marek Marczykowski
2013-04-28 22:46 ` Samuel Thibault
2013-04-23 3:16 ` [PATCH 4/7] minios: add gnttab_find_grant_of_page Marek Marczykowski
2013-04-28 22:41 ` Samuel Thibault
2013-04-28 22:51 ` Marek Marczykowski
2013-04-23 3:17 ` [PATCH 5/7] libxc: implement gntshr for minios Marek Marczykowski
2013-04-28 22:44 ` Samuel Thibault
2013-04-28 23:17 ` Marek Marczykowski
2013-04-28 23:22 ` Samuel Thibault
2013-04-23 3:25 ` [PATCH 7/7] libvchan: do not use xc_gntshr_share_page_notify in Mini-OS Marek Marczykowski
2013-04-28 22:48 ` Samuel Thibault
2013-04-28 23:00 ` Marek Marczykowski
2013-04-26 14:55 ` [PATCH 0/7] Optionally add libvchan to stubdom George Dunlap
2013-04-26 14:56 ` Marek Marczykowski
2013-04-26 14:57 ` George Dunlap
2013-04-26 15:02 ` Marek Marczykowski [this message]
2013-04-26 15:03 ` Ian Campbell
2013-04-26 16:41 ` Daniel De Graaf
2013-04-28 22:52 ` Samuel Thibault
2013-04-28 23:08 ` Marek Marczykowski
2013-04-30 21:22 ` Samuel Thibault
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=517A96E9.1030108@invisiblethingslab.com \
--to=marmarek@invisiblethingslab.com \
--cc=Ian.Campbell@citrix.com \
--cc=Ian.Jackson@eu.citrix.com \
--cc=Stefano.Stabellini@eu.citrix.com \
--cc=george.dunlap@eu.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.