Hi Xenia, On 17/10/2022 19:32, Xenia Ragiadakou wrote: > > > Currently the pa_range_info for the 52-bit pa range advertizes that the > p2m root table consists of 8 concatenated tables at level 3, which does > not make much sense. I think the current code advertises 8 concatenated tables at level -1 (sl0=3 -> root_level=-1) which is obviously incorrect, but the commit msg should be updated. Funnily enough p2m_root_level is unsigned so it would lead to overflow (p2m_root_level would end up with (1 << 32) - 1 instead of -1).
> In order to support the 52-bit pa size with 4KB granule, the p2m root > table needs to be configured either as a single table at level -1 or > as 16 concatenated tables at level 0. > Since, currently there is not support for level -1, set the > root_order and sl0 fields of the corresponding pa_range_info according > to the second approach. > > Signed-off-by: Xenia Ragiadakou <burzalod...@gmail.com> Reviewed-by: Michal Orzel <michal.or...@amd.com> ~Michal
