From: "Roger Pau Monné" <roger.pau@citrix.com>
To: Jan Beulich <jbeulich@suse.com>,
Andrew Cooper <andrew.cooper3@citrix.com>
Cc: Wei Liu <wl@xen.org>, Tim Deegan <tim@xen.org>,
"xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org>
Subject: Re: [PATCH v3 2/9] x86/PV: properly set shadow allocation for Dom0
Date: Wed, 22 Sep 2021 16:25:30 +0200 [thread overview]
Message-ID: <YUs82i7S0fd8r1uX@MacBook-Air-de-Roger.local> (raw)
In-Reply-To: <8c1ab98a-8fde-3d9e-5250-c5a40af70453@suse.com>
On Wed, Sep 22, 2021 at 03:50:25PM +0200, Jan Beulich wrote:
> On 22.09.2021 15:31, Andrew Cooper wrote:
> > On 21/09/2021 08:17, Jan Beulich wrote:
> >> @@ -933,7 +934,18 @@ int __init dom0_construct_pv(struct doma
> >> #ifdef CONFIG_SHADOW_PAGING
> >> if ( opt_dom0_shadow )
> >> {
> >> + bool preempted;
> >> +
> >> printk("Switching dom0 to using shadow paging\n");
> >> +
> >> + nr_pt_pages = dom0_paging_pages(d, nr_pages);
> >> +
> >> + do {
> >> + preempted = false;
> >> + shadow_set_allocation(d, nr_pt_pages, &preempted);
> >> + process_pending_softirqs();
> >> + } while ( preempted );
> >
> > This is still broken.
> >
> > The loop setting the shadow allocation needs to be outside of this
> > conditional, because it is not related to early activation of the l1tf
> > tasklet.
>
> Well, I'm not sure what to say. On v1 you already said so. But then you
> didn't care to reply to me responding:
>
> "Are you suggesting to set up a (perhaps large) shadow pool just in
> case we need to enable shadow mode on Dom0? And all of this memory
> to then remain unused in the majority of cases?
>
> Plus even if so, I'd view this as a 2nd, independent step, largely
> orthogonal to the handling of "dom0=shadow". If somebody really
> wanted that, I think this should be driven by an explicit setting
> of the shadow pool size, indicating the admin is willing to waste
> the memory.
Maybe an acceptable compromise would be to allocate the pool if
opt_dom0_shadow || opt_pv_l1tf_hwdom?
opt_pv_l1tf_hwdom is not enabled by default, so an admin opting to
enable it should also be willing to reserve the memory it requires in
case it needs activating during runtime.
Roger.
next prev parent reply other threads:[~2021-09-22 14:25 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-21 7:15 [PATCH v3 0/9] x86/PVH: Dom0 building adjustments Jan Beulich
2021-09-21 7:16 ` [PATCH v3 1/9] x86/PVH: improve Dom0 memory size calculation Jan Beulich
2021-09-22 11:59 ` Roger Pau Monné
2021-09-29 10:53 ` Jan Beulich
2021-09-21 7:17 ` [PATCH v3 2/9] x86/PV: properly set shadow allocation for Dom0 Jan Beulich
2021-09-22 13:01 ` Roger Pau Monné
2021-09-22 13:31 ` Andrew Cooper
2021-09-22 13:50 ` Jan Beulich
2021-09-22 14:25 ` Roger Pau Monné [this message]
2021-09-22 14:28 ` Jan Beulich
2021-09-21 7:17 ` [PATCH v3 3/9] x86/PVH: permit more physdevop-s to be used by Dom0 Jan Beulich
2021-09-22 14:22 ` Roger Pau Monné
2021-09-24 12:18 ` Jan Beulich
2021-09-21 7:18 ` [PATCH v3 4/9] x86/PVH: provide VGA console info to Dom0 Jan Beulich
2021-09-22 15:01 ` Roger Pau Monné
2021-09-22 17:03 ` Andrew Cooper
2021-09-23 9:58 ` Jan Beulich
2021-09-23 9:46 ` Jan Beulich
2021-09-23 13:22 ` Roger Pau Monné
2021-09-21 7:19 ` [PATCH v3 5/9] x86/PVH: actually show Dom0's register state from debug key '0' Jan Beulich
2021-09-22 15:48 ` Roger Pau Monné
2021-09-23 10:21 ` Jan Beulich
2021-09-23 14:27 ` Roger Pau Monné
2021-09-21 7:19 ` [PATCH v3 6/9] x86/HVM: convert hvm_virtual_to_linear_addr() to be remote-capable Jan Beulich
2021-09-23 8:09 ` Roger Pau Monné
2021-09-23 10:34 ` Jan Beulich
2021-09-23 14:28 ` Roger Pau Monné
2021-09-21 7:20 ` [PATCH v3 7/9] x86/PVH: actually show Dom0's stacks from debug key '0' Jan Beulich
2021-09-23 10:31 ` Roger Pau Monné
2021-09-23 10:38 ` Roger Pau Monné
2021-09-23 10:47 ` Jan Beulich
2021-09-23 14:43 ` Roger Pau Monné
2021-09-21 7:20 ` [PATCH v3 8/9] x86/HVM: skip offline vCPU-s when dumping VMCBs/VMCSes Jan Beulich
2021-09-23 8:23 ` Roger Pau Monné
2021-09-23 11:27 ` Jan Beulich
2021-09-23 14:46 ` Roger Pau Monné
2021-09-21 7:21 ` [PATCH v3 9/9] x86/P2M: relax permissions of PVH Dom0's MMIO entries Jan Beulich
2021-09-23 11:10 ` Roger Pau Monné
2021-09-23 11:32 ` Jan Beulich
2021-09-23 11:54 ` Roger Pau Monné
2021-09-23 12:15 ` Jan Beulich
2021-09-23 15:15 ` Roger Pau Monné
2021-09-23 15:22 ` Jan Beulich
2021-09-23 15:32 ` Roger Pau Monné
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=YUs82i7S0fd8r1uX@MacBook-Air-de-Roger.local \
--to=roger.pau@citrix.com \
--cc=andrew.cooper3@citrix.com \
--cc=jbeulich@suse.com \
--cc=tim@xen.org \
--cc=wl@xen.org \
--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.