Re: [RFC PATCH 2/4] xen/arm: Move make_hypervisor_node()

[Orzel, Michal]

On 21/06/2025 17:11, Koichiro Den wrote:
> Even though make_hypervisor_node() does not rely on the /reserved-memory
> instantiation when calling find_unused_regions() (the wrapper introduced
> in the previous commit), the next but one commit will use it for PV time
Unless for specific reasons, you should not rely on which commit gets committed
first. Hypothetically there could be a different patch committed between this
and the next one in your series. That's why it's not a good practice to mention
order of commits in the commit message.

> shared regions, in addition to the existing extended regions.
> 
> Move it as a prerequisite for the commit after next.
This one is particularly not useful.

> 
> No functional changes intended.
> 
> Signed-off-by: Koichiro Den <d...@valinux.co.jp>

Other than that, LGTM.

~Michal