On Tue, Jul 30, 2024 at 6:18 AM Sergiy Kibrik <sergiy_kib...@epam.com> wrote: > > Explicitly check whether altp2m is on for domain when getting altp2m index. > If explicit call to altp2m_active() always returns false, DCE will remove > call to altp2m_vcpu_idx(). > > p2m_get_mem_access() expects 0 as altp2m_idx parameter when altp2m not active > or not supported, so 0 is a fallback value then. > > The puspose of that is later to be able to disable altp2m support and
"purpose" misspelled here > exclude its code from the build completely, when not supported by target > platform (as of now it's supported for VT-d only). Probably wanted to say "VT-x only" here > > Also all other calls to altp2m_vcpu_idx() are guarded by altp2m_active(), so > this change puts usage of this routine in line with the rest of code. > > Signed-off-by: Sergiy Kibrik <sergiy_kib...@epam.com> With the commit message fixed (probably could be done when applied): Acked-by: Tamas K Lengyel <ta...@tklengyel.com>
