Coverity points out that the "d &&" is redundant.
Fixes: c9476c4ad72 ("x86: don't override INVALID_M2P_ENTRY with
SHARED_M2P_ENTRY")
Signed-off-by: Andrew Cooper <[email protected]>
---
CC: Jan Beulich <[email protected]>
CC: Roger Pau Monné <[email protected]>
CC: Wei Liu <[email protected]>
---
xen/arch/x86/x86_64/mm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/xen/arch/x86/x86_64/mm.c b/xen/arch/x86/x86_64/mm.c
index 0d1aadbfce..4c0a3a275c 100644
--- a/xen/arch/x86/x86_64/mm.c
+++ b/xen/arch/x86/x86_64/mm.c
@@ -1336,7 +1336,7 @@ void set_gpfn_from_mfn(unsigned long mfn, unsigned long
pfn)
{
const struct domain *d = page_get_owner(mfn_to_page(_mfn(mfn)));
- if ( d && (d == dom_cow) )
+ if ( d == dom_cow )
entry = SHARED_M2P_ENTRY;
}
--
2.11.0