From: Vitaly Kuznetsov <vkuznets@redhat.com>
To: Tim Deegan <tim@xen.org>
Cc: Wei Liu <wei.liu2@citrix.com>, Andrew Jones <drjones@redhat.com>,
Julien Grall <julien.grall@linaro.org>,
Keir Fraser <keir@xen.org>,
Ian Campbell <ian.campbell@citrix.com>,
Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
Andrew Cooper <andrew.cooper3@citrix.com>,
Ian Jackson <ian.jackson@eu.citrix.com>,
Olaf Hering <olaf@aepfle.de>,
David Vrabel <david.vrabel@citrix.com>,
Jan Beulich <jbeulich@suse.com>,
xen-devel@lists.xenproject.org,
Daniel De Graaf <dgdegra@tycho.nsa.gov>
Subject: Re: [PATCH RFC 4/4] xen: arch-specific hooks for domain_soft_reset()
Date: Mon, 22 Jun 2015 11:44:18 +0200 [thread overview]
Message-ID: <87616g9ix9.fsf@vitty.brq.redhat.com> (raw)
In-Reply-To: <20150604141953.GH74656@deinos.phlegethon.org> (Tim Deegan's message of "Thu, 4 Jun 2015 15:19:53 +0100")
Tim Deegan <tim@xen.org> writes:
> At 15:35 +0200 on 03 Jun (1433345722), Vitaly Kuznetsov wrote:
>> x86-specific hook cleans up the pirq-emuirq mappings and replaces the
>> shared_info frame with an empty page to support subsequent
>> XENMAPSPACE_shared_info call.
>
> That's a bit roundabout. I think we might be better off allocating a
> new shared-info page and abandoning the old one as an ordinary guest
> RAM page.
>
I've tried looking into such approach and there is an issue: shared_info
page is being allocated from Xen heap and in case it's built with
CONFIG_SEPARATE_XENHEAP we can't just reassign the page to the domain's
pagelist or we'll open a possibility for a domain to move itself
entirely to Xen's heap by e.g looping over all its gfns, doing
XENMAPSPACE_shared_info and perforting soft reset.
--
Vitaly
next prev parent reply other threads:[~2015-06-22 9:44 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-03 13:35 [PATCH RFC 0/4] 'reset everything' approach to PVHVM guest kexec Vitaly Kuznetsov
2015-06-03 13:35 ` [PATCH RFC 1/4] xen: evtchn: make evtchn_reset() ready for soft reset Vitaly Kuznetsov
2015-06-04 14:05 ` Tim Deegan
2015-06-04 15:19 ` David Vrabel
2015-06-04 15:47 ` Tim Deegan
2015-06-05 8:52 ` Jan Beulich
2015-06-05 8:58 ` Ian Campbell
2015-06-05 9:07 ` Jan Beulich
2015-06-08 14:10 ` Jan Beulich
2015-06-08 15:05 ` Vitaly Kuznetsov
2015-06-03 13:35 ` [PATCH RFC 2/4] xen: grant_table: implement grant_table_soft_reset() Vitaly Kuznetsov
2015-06-04 14:11 ` Tim Deegan
2015-06-04 15:22 ` David Vrabel
2015-06-04 15:44 ` Tim Deegan
2015-06-08 14:26 ` Jan Beulich
2015-06-08 14:58 ` Vitaly Kuznetsov
2015-06-08 15:35 ` Jan Beulich
2015-06-03 13:35 ` [PATCH RFC 3/4] xen: implement SCHEDOP_soft_reset Vitaly Kuznetsov
2015-06-08 14:31 ` Jan Beulich
2015-06-22 16:00 ` Vitaly Kuznetsov
2015-06-22 16:06 ` Jan Beulich
2015-06-22 16:24 ` Vitaly Kuznetsov
2015-06-23 7:13 ` Jan Beulich
2015-06-23 12:10 ` Vitaly Kuznetsov
2015-06-23 12:52 ` Jan Beulich
2015-06-03 13:35 ` [PATCH RFC 4/4] xen: arch-specific hooks for domain_soft_reset() Vitaly Kuznetsov
2015-06-04 14:19 ` Tim Deegan
2015-06-22 9:44 ` Vitaly Kuznetsov [this message]
2015-06-25 9:57 ` Tim Deegan
2015-06-08 15:32 ` Jan Beulich
2015-06-08 15:59 ` Vitaly Kuznetsov
2015-06-04 14:22 ` [PATCH RFC 0/4] 'reset everything' approach to PVHVM guest kexec Tim Deegan
2015-06-08 15:38 ` Ian Jackson
2015-06-08 15:53 ` Wei Liu
2015-06-08 17:43 ` Wei Liu
2015-06-09 8:19 ` Dave Scott
2015-06-09 9:29 ` Wei Liu
2015-06-09 9:38 ` Wei Liu
2015-06-09 10:02 ` Dave Scott
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=87616g9ix9.fsf@vitty.brq.redhat.com \
--to=vkuznets@redhat.com \
--cc=andrew.cooper3@citrix.com \
--cc=david.vrabel@citrix.com \
--cc=dgdegra@tycho.nsa.gov \
--cc=drjones@redhat.com \
--cc=ian.campbell@citrix.com \
--cc=ian.jackson@eu.citrix.com \
--cc=jbeulich@suse.com \
--cc=julien.grall@linaro.org \
--cc=keir@xen.org \
--cc=olaf@aepfle.de \
--cc=stefano.stabellini@eu.citrix.com \
--cc=tim@xen.org \
--cc=wei.liu2@citrix.com \
--cc=xen-devel@lists.xenproject.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.