Re: [Xen-devel] [PATCH 0/7] Optionally add libvchan to stubdom

[Marek Marczykowski <marmarek@xxxxxxxxxxxxxxxxxxxxxx>]

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@xxxxxxxxxxxxxxxxxxxxxx> 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: signature.asc
Description: OpenPGP digital signature

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel