From: Oleksandr <olekstysh@gmail.com>
To: Jan Beulich <jbeulich@suse.com>
Cc: "Oleksandr Tyshchenko" <oleksandr_tyshchenko@epam.com>,
"Daniel De Graaf" <dgdegra@tycho.nsa.gov>,
"Daniel P. Smith" <dpsmith@apertussolutions.com>,
"Ian Jackson" <iwj@xenproject.org>, "Wei Liu" <wl@xen.org>,
"George Dunlap" <george.dunlap@citrix.com>,
"Stefano Stabellini" <sstabellini@kernel.org>,
"Volodymyr Babchuk" <Volodymyr_Babchuk@epam.com>,
"Roger Pau Monné" <roger.pau@citrix.com>,
"Bertrand Marquis" <bertrand.marquis@arm.com>,
"Wei Chen" <Wei.Chen@arm.com>,
"Andrew Cooper" <andrew.cooper3@citrix.com>,
xen-devel@lists.xenproject.org, "Julien Grall" <julien@xen.org>
Subject: Re: [RFC PATCH] xen/memory: Introduce a hypercall to provide unallocated space
Date: Wed, 4 Aug 2021 22:18:49 +0300 [thread overview]
Message-ID: <090ffc19-92fd-5ef9-99d5-affcfdc28ad2@gmail.com> (raw)
In-Reply-To: <7d79a197-a126-2eed-3198-c20e63c1eece@suse.com>
On 03.08.21 15:53, Jan Beulich wrote:
Hi, Jan
Thank you for the input.
> On 30.07.2021 18:13, Oleksandr wrote:
>> Well, if new hypercall and, what is more, "the querying hypervisor at
>> runtime to find unused space" model itself is not welcome, I am ok,
>> let's try to create a working system,
>> may we please find a common ground to move this forward (at least on Arm
>> for now, the foreign mapping is the most important question).
>>
>> I got the proposed idea in general, but I haven't connected all dots
>> yet, some points need clarification.
>>
>> 1. The safe range must be defined/allocated in advance and must remain
>> const during the runtime. The safe range must be chosen by the toolstack.
>> [For the initial implementation we can start with some large value
>> (128GB) as discussed above]
>>
>> Questions:
>>
>> - Do we need to inform Xen about that range (via domain_create
>> hypercall, etc)?
>> - What will be in charge of guaranteeing the safety of that range at
>> runtime (reject new mapping requests with possible overlaps, etc), Xen,
>> toolstack or both?
> Well, what other entity than Xen could enforce this? (By implication,
> the answer to the earlier question can imo only be "yes", unless it's
> Xen itself to establish the region.)
Indeed, agree.
>
>> - Where that range should be located in guest address space, should that
>> range be the same for all domains (how GUEST_GNTTAB_BASE(SIZE) for example)
>> or it should be calculated based on actual guest_ram_base(size) for a
>> particular domain?
> The default size may well be fixed or amount-of-memory-dependent, but
> I think there will need to be a way to enlarge the region for guests
> with particular needs.
Well, but why we couldn't just make a large chunk by default which would
satisfy all guests, as it was mentioned earlier in this thread "as it
doesn't consume resource when not being used"
to avoid an extra configuration option, etc?
>
>> - What about a safe range the Dom0 can use itself? Xen should choose it
>> for Dom0 the same way how toolstack chooses it for other domains, correct?
>>
>> 2. The safe range must be provided to domain using the firmware table.
>> [We can start with the DTB and leave ACPI unimplemented for now,
>> assuming we will be able to solve open questions as discussed above]
>>
>> Questions:
>>
>> - Do we need distinguish between foreign and grant mappings at the
>> domain side at all? Can the same safe range be used for all types of
>> mapping?
> Like Stefano I don't think so.
Agree.
--
Regards,
Oleksandr Tyshchenko
next prev parent reply other threads:[~2021-08-04 19:19 UTC|newest]
Thread overview: 52+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-28 16:18 [RFC PATCH] xen/memory: Introduce a hypercall to provide unallocated space Oleksandr Tyshchenko
2021-07-28 17:06 ` Oleksandr
2021-07-28 17:19 ` Andrew Cooper
2021-07-28 17:27 ` Julien Grall
2021-07-28 19:00 ` Andrew Cooper
2021-07-28 19:53 ` Julien Grall
2021-07-30 16:13 ` Oleksandr
2021-07-30 23:57 ` Stefano Stabellini
2021-08-02 19:12 ` Oleksandr
2021-08-04 20:56 ` Oleksandr
2021-08-04 22:00 ` Julien Grall
2021-08-05 14:52 ` Oleksandr
2021-08-05 17:25 ` Julien Grall
2021-08-05 20:48 ` Oleksandr
2021-08-06 0:20 ` Stefano Stabellini
2021-08-06 18:03 ` Oleksandr
2021-08-13 23:49 ` Oleksandr
2021-08-06 0:30 ` Stefano Stabellini
2021-08-07 17:03 ` Oleksandr
2021-08-09 15:42 ` Julien Grall
2021-08-09 18:24 ` Oleksandr
2021-08-09 20:45 ` Julien Grall
2021-08-09 21:18 ` Oleksandr
2021-08-10 16:28 ` Julien Grall
2021-08-10 17:03 ` Oleksandr
2021-08-17 17:53 ` Julien Grall
2021-08-17 17:54 ` Julien Grall
2021-08-27 20:34 ` Oleksandr
2021-08-10 6:34 ` Wei Chen
2021-08-10 11:58 ` Oleksandr
2021-08-10 16:21 ` Julien Grall
2021-08-10 16:49 ` Oleksandr
2021-08-09 14:51 ` Julien Grall
2021-08-09 17:14 ` Oleksandr
2021-08-09 17:18 ` Julien Grall
2021-08-09 17:49 ` Oleksandr
2021-08-13 21:45 ` Oleksandr
2021-08-03 12:53 ` Jan Beulich
2021-08-04 19:18 ` Oleksandr [this message]
2021-08-05 5:58 ` Jan Beulich
2021-08-05 15:10 ` Oleksandr
2021-08-03 12:49 ` Jan Beulich
2021-08-03 12:53 ` Julien Grall
2021-08-17 17:59 ` Julien Grall
2021-08-05 15:03 ` Daniel P. Smith
2021-08-05 15:59 ` Oleksandr
2021-08-05 16:37 ` Daniel P. Smith
2021-08-05 21:56 ` Oleksandr
2021-08-06 6:09 ` Jan Beulich
2021-08-06 15:08 ` Daniel P. Smith
2021-09-07 8:53 ` Henry Wang
2021-09-07 21:34 ` Oleksandr
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=090ffc19-92fd-5ef9-99d5-affcfdc28ad2@gmail.com \
--to=olekstysh@gmail.com \
--cc=Volodymyr_Babchuk@epam.com \
--cc=Wei.Chen@arm.com \
--cc=andrew.cooper3@citrix.com \
--cc=bertrand.marquis@arm.com \
--cc=dgdegra@tycho.nsa.gov \
--cc=dpsmith@apertussolutions.com \
--cc=george.dunlap@citrix.com \
--cc=iwj@xenproject.org \
--cc=jbeulich@suse.com \
--cc=julien@xen.org \
--cc=oleksandr_tyshchenko@epam.com \
--cc=roger.pau@citrix.com \
--cc=sstabellini@kernel.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.