Hi Roger,
On 4/30/24 10:34 AM, Roger Pau Monne wrote:
> None of the implementations support set_foreign_p2m_entry() yet, neither they
> have a p2m walk in domain_relinquish_resources() in order to remove the
> foreign
> mappings from the p2m and thus drop the extra refcounts.
>
> Adjust the arch helpers to return false and introduce a comment that clearly
> states it is not only taking extra refcounts that's needed, but also dropping
> them on domain teardown.
>
> Fixes: 4988704e00d8 ('xen/riscv: introduce p2m.h')
> Fixes: 4a2f68f90930 ('xen/ppc: Define minimal stub headers required for full
> build')
> Signed-off-by: Roger Pau Monné <[email protected]>
> ---
This makes sense to me. This stub implementation was definitely an
oversight on my part.
Acked-by: Shawn Anastasio <[email protected]>
Thanks,
Shawn
