From: Andrew Cooper Date: Fri, 21 Jun 2024 20:57:59 +0000 (+0100) Subject: x86/pagewalk: Address MISRA R8.3 violation in guest_walk_tables() X-Git-Tag: 4.19.0-rc1~20 X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=b14dae96c07ef27cc7f8107ddaa16989e9ab024b;p=xen.git x86/pagewalk: Address MISRA R8.3 violation in guest_walk_tables() Commit 4c5d78a10dc8 ("x86/pagewalk: Re-implement the pagetable walker") intentionally renamed guest_walk_tables()'s 'pfec' parameter to 'walk' because it's not a PageFault Error Code, despite the name of some of the constants passed in. Sadly the constants-cleanup I've been meaning to do since then still hasn't come to pass. Update the declaration to match, to placate MISRA. Fixes: 4c5d78a10dc8 ("x86/pagewalk: Re-implement the pagetable walker") Signed-off-by: Andrew Cooper Reviewed-by: Stefano Stabellini Release-Acked-by: Oleksii Kurochko --- diff --git a/xen/arch/x86/include/asm/guest_pt.h b/xen/arch/x86/include/asm/guest_pt.h index bc312343cd..7b0c9b005c 100644 --- a/xen/arch/x86/include/asm/guest_pt.h +++ b/xen/arch/x86/include/asm/guest_pt.h @@ -422,7 +422,7 @@ static inline unsigned int guest_walk_to_page_order(const walk_t *gw) bool guest_walk_tables(const struct vcpu *v, struct p2m_domain *p2m, - unsigned long va, walk_t *gw, uint32_t pfec, + unsigned long va, walk_t *gw, uint32_t walk, gfn_t top_gfn, mfn_t top_mfn, void *top_map); /* Pretty-print the contents of a guest-walk */