Re: [Xen-devel] [PATCH v2] x86/ept: pass correct level to p2m_entry_modify

[Jan Beulich <JBeulich@xxxxxxxx>]

On 03.07.2019 13:33, Roger Pau Monne wrote:
> EPT differs from NPT and shadow when translating page orders to levels
> in the physmap page tables. EPT page tables level for order 0 pages is
> 0, while NPT and shadow instead use 1, ie: EPT page tables levels
> starts at 0 while NPT and shadow starts at 1.
> 
> Fix the p2m_entry_modify call in atomic_write_ept_entry to always add
> one to the level, in order to match NPT and shadow usage.
> 
> While there also add a check to ensure p2m_entry_modify is never
> called with level == 0. That should allow to catch future errors
> related to the level parameter.
> 
> Fixes: c7a4c088ad1c ('x86/mm: split p2m ioreq server pages special handling 
> into helper')
> Signed-off-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>

Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel