From: Oleksii Kurochko <oleksii.kurochko@gmail.com>
To: Jan Beulich <jbeulich@suse.com>
Cc: "Romain Caritey" <Romain.Caritey@microchip.com>,
"Andrew Cooper" <andrew.cooper3@citrix.com>,
"Anthony PERARD" <anthony.perard@vates.tech>,
"Michal Orzel" <michal.orzel@amd.com>,
"Julien Grall" <julien@xen.org>,
"Roger Pau Monné" <roger.pau@citrix.com>,
"Stefano Stabellini" <sstabellini@kernel.org>,
xen-devel@lists.xenproject.org
Subject: Re: [PATCH v2 10/11] xen/riscv: add definition of guest RAM banks
Date: Mon, 6 Apr 2026 17:43:16 +0200 [thread overview]
Message-ID: <912263a2-8dff-414e-bea4-64428ce2ec36@gmail.com> (raw)
In-Reply-To: <0e37083a-42ab-4c99-83fc-b77f519394b9@suse.com>
On 4/1/26 5:10 PM, Jan Beulich wrote:
> On 01.04.2026 16:53, Oleksii Kurochko wrote:
>>
>>
>> On 4/1/26 4:22 PM, Jan Beulich wrote:
>>> On 01.04.2026 15:57, Oleksii Kurochko wrote:
>>>> On 4/1/26 8:17 AM, Jan Beulich wrote:
>>>>> On 31.03.2026 18:14, Oleksii Kurochko wrote:
>>>>>> On 3/30/26 5:51 PM, Jan Beulich wrote:
>>>>>>> On 23.03.2026 17:29, Oleksii Kurochko wrote:
>>>>>>>> The dom0less solution uses defined RAM banks as compile-time constants,
>>>>>>>> so introduce macros to describe guest RAM banks.
>>>>>>>>
>>>>>>>> The reason for 2 banks is that there is typically always a use case for
>>>>>>>> low memory under 4 GB, but the bank under 4 GB ends up being small because
>>>>>>>> there are other things under 4 GB it can conflict with (interrupt
>>>>>>>> controller, PCI BARs, etc.).
>>>>>>> Fixed layouts like the one you suggest come with (potentially severe)
>>>>>>> downsides. For example, what if more than 2Gb of MMIO space are needed
>>>>>>> for non-64-bit BARs?
>>>>>> It looks where usually RAM on RISC-V boards start, so I expect that 2gb
>>>>>> before RAM start is enough for MMIO space.
>>>>> Likely in the common case. Board designers aren't constrained by this,
>>>>> though (aiui). Whereas you set in stone a single, fixed layout.
>>>>>
>>>>> Arm maintainers - since a similar fixed layout is used there iirc,
>>>>> could you chime in here, please?
>>>>>
>>>>>> Answering your question it will be an issue or it will also use some
>>>>>> space before banks, no?
>>>>> I fear I don't understand what you're trying to tell me.
>>>> I meant that there is also some space between banks and pretty big which
>>>> could be used for MMIO which could be used for non-64-bit BARs.
>>> I don't follow: Bank 0 extends to 4G. There's no space above it, below
>>> bank 1, which could be use for non-64-bit BARs.
>>
>> So we have two banks:
>> bank[0] -> [0x80000000, 0x100000000)
>> bank[1] -> [0x0200000000, 10000000000)
>>
>> So i think we have some space between them [0x100000000, 0x0200000000)
>> -> 4gb to be used for non-64-bit BARs.
>
> But a non-64-bit BAR need to be assigned an address below 0x100000000?
Right, I had in mind that RV32 uses for guest Sv32x4 which could
translate 34-bit GPA into 34-bit MPA and automatically applied that to
32-bit BAR...
I can keep first 4gb for MMIO purpose and start bank[0] at 4gb as 34 MPA
address space is more then enough to cover reserved 2gb of bank[0] after
4gb.
~ Oleksii
next prev parent reply other threads:[~2026-04-06 15:43 UTC|newest]
Thread overview: 47+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-23 16:29 [PATCH v2 00/11] RISCV: enable DOMAIN_BUILD_HELPERS Oleksii Kurochko
2026-03-23 16:29 ` [PATCH v2 01/11] xen/riscv: implement get_page_from_gfn() Oleksii Kurochko
2026-03-26 13:50 ` Jan Beulich
2026-03-30 13:40 ` Oleksii Kurochko
2026-03-30 14:04 ` Jan Beulich
2026-03-23 16:29 ` [PATCH v2 02/11] xen: return proper type for guest access functions Oleksii Kurochko
2026-03-26 13:56 ` Jan Beulich
2026-03-23 16:29 ` [PATCH v2 03/11] xen/riscv: implement copy_to_guest_phys() Oleksii Kurochko
2026-03-30 14:24 ` Jan Beulich
2026-03-23 16:29 ` [PATCH v2 04/11] xen/dom0less: rename kernel_zimage_probe() to kernel_image_probe() Oleksii Kurochko
2026-03-23 16:29 ` [PATCH v2 05/11] xen/riscv: add kernel loading support Oleksii Kurochko
2026-03-30 14:47 ` Jan Beulich
[not found] ` <05b1bc67-bbed-412e-881e-a3fb2c2d873b@gmail.com>
2026-03-31 15:14 ` Jan Beulich
[not found] ` <a0efb7a6-4854-4fe5-bbf4-2561f25d7133@gmail.com>
2026-03-31 15:56 ` Jan Beulich
2026-03-23 16:29 ` [PATCH v2 06/11] xen: move declaration of fw_unreserved_regions() to common header Oleksii Kurochko
2026-03-23 16:29 ` [PATCH v2 07/11] xen: move domain_use_host_layout() to common code Oleksii Kurochko
2026-03-30 15:13 ` Jan Beulich
[not found] ` <57581b7d-cb9f-444c-9321-63b2fc3d09f0@gmail.com>
2026-03-31 15:53 ` Jan Beulich
2026-03-31 16:32 ` Oleksii Kurochko
2026-03-31 19:49 ` Oleksii Kurochko
2026-04-01 5:59 ` Jan Beulich
2026-04-01 14:44 ` Oleksii Kurochko
2026-04-01 5:58 ` Jan Beulich
2026-04-01 14:38 ` Oleksii Kurochko
2026-04-01 14:42 ` Jan Beulich
2026-03-23 16:29 ` [PATCH v2 08/11] xen: rename p2m_ipa_bits to p2m_gpa_bits Oleksii Kurochko
2026-03-30 15:16 ` Jan Beulich
2026-03-23 16:29 ` [PATCH v2 09/11] xen/riscv: introduce p2m_gpa_bits Oleksii Kurochko
2026-03-30 15:34 ` Jan Beulich
2026-03-31 16:02 ` Oleksii Kurochko
2026-04-01 6:07 ` Jan Beulich
2026-04-01 13:50 ` Oleksii Kurochko
2026-04-01 13:57 ` Jan Beulich
2026-03-23 16:29 ` [PATCH v2 10/11] xen/riscv: add definition of guest RAM banks Oleksii Kurochko
2026-03-30 15:51 ` Jan Beulich
2026-03-31 16:14 ` Oleksii Kurochko
2026-04-01 6:17 ` Jan Beulich
2026-04-01 13:57 ` Oleksii Kurochko
2026-04-01 14:22 ` Jan Beulich
2026-04-01 14:53 ` Oleksii Kurochko
2026-04-01 15:10 ` Jan Beulich
2026-04-06 15:43 ` Oleksii Kurochko [this message]
2026-04-07 6:23 ` Jan Beulich
2026-04-07 8:54 ` Oleksii Kurochko
2026-04-07 9:09 ` Jan Beulich
2026-04-07 9:19 ` Oleksii Kurochko
2026-03-23 16:29 ` [PATCH v2 11/11] xen/riscv: enable DOMAIN_BUILD_HELPERS Oleksii Kurochko
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=912263a2-8dff-414e-bea4-64428ce2ec36@gmail.com \
--to=oleksii.kurochko@gmail.com \
--cc=Romain.Caritey@microchip.com \
--cc=andrew.cooper3@citrix.com \
--cc=anthony.perard@vates.tech \
--cc=jbeulich@suse.com \
--cc=julien@xen.org \
--cc=michal.orzel@amd.com \
--cc=roger.pau@citrix.com \
--cc=sstabellini@kernel.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.