[Xen-changelog] [xen master] x86/shadow: adjust mask shadow_audit_tables() passes to hash_foreach()

[patchbot@xxxxxxx]

commit 88d07de868419224898508852e0c2553aa8d2e40
Author:     Jan Beulich <jbeulich@xxxxxxxx>
AuthorDate: Thu Jan 29 13:42:20 2015 +0000
Commit:     Tim Deegan <tim@xxxxxxx>
CommitDate: Thu Jan 29 13:44:49 2015 +0000

    x86/shadow: adjust mask shadow_audit_tables() passes to hash_foreach()
    
    It so far having been ~1 made most of the code preceding the call
    pointless, but I assume this wasn't meant to be that way. Also replace
    the remaining hard coded ~1 with an expression documenting the
    intention a little better.
    
    Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
    
    Adjust again to use SHF_page_type_mask, at Jan's suggestion.
    
    Signed-off-by: Tim Deegan <tim@xxxxxxx>
---
 xen/arch/x86/mm/shadow/common.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/xen/arch/x86/mm/shadow/common.c b/xen/arch/x86/mm/shadow/common.c
index 7e6aeeb..64c5a28 100644
--- a/xen/arch/x86/mm/shadow/common.c
+++ b/xen/arch/x86/mm/shadow/common.c
@@ -3761,7 +3761,7 @@ void shadow_audit_tables(struct vcpu *v)
 #endif
 
     if ( SHADOW_AUDIT & SHADOW_AUDIT_ENTRIES_FULL )
-        mask = ~1; /* Audit every table in the system */
+        mask = SHF_page_type_mask; /* Audit every table in the system */
     else 
     {
         /* Audit only the current mode's tables */
@@ -3776,7 +3776,7 @@ void shadow_audit_tables(struct vcpu *v)
         }
     }
 
-    hash_foreach(v, ~1, callbacks, _mfn(INVALID_MFN));
+    hash_foreach(v, mask, callbacks, _mfn(INVALID_MFN));
 }
 
 #endif /* Shadow audit */
--
generated by git-patchbot for /home/xen/git/xen.git#master

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog