On 30/10/2019 13:02, Alexandru Stefan ISAILA wrote:
> Originally the gfn and altp2m_idx are assigned from the a.u.mem_access union.
> This works because it's the same memory used. This patch addresses this
> issue by changing the mem_access union with the suppress_ve union for
> consistency.
>
> Signed-off-by: Alexandru Isaila <aisa...@bitdefender.com>
Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com>
CC Juergen. This wants backporting, so should be included in 4.13
~Andrew
> ---
> xen/arch/x86/hvm/hvm.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/xen/arch/x86/hvm/hvm.c b/xen/arch/x86/hvm/hvm.c
> index e4c0425330..06a7b40107 100644
> --- a/xen/arch/x86/hvm/hvm.c
> +++ b/xen/arch/x86/hvm/hvm.c
> @@ -4685,8 +4685,8 @@ static int do_altp2m_op(
> rc = -EINVAL;
> else
> {
> - gfn_t gfn = _gfn(a.u.mem_access.gfn);
> - unsigned int altp2m_idx = a.u.mem_access.view;
> + gfn_t gfn = _gfn(a.u.suppress_ve.gfn);
> + unsigned int altp2m_idx = a.u.suppress_ve.view;
> bool suppress_ve = a.u.suppress_ve.suppress_ve;
>
> rc = p2m_set_suppress_ve(d, gfn, suppress_ve, altp2m_idx);
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
