diff --git a/xen/arch/x86/mm/shadow/common.c b/xen/arch/x86/mm/shadow/common.c index ed698fa90b..b167cc3ce2 100644 --- a/xen/arch/x86/mm/shadow/common.c +++ b/xen/arch/x86/mm/shadow/common.c @@ -126,7 +126,7 @@ void shadow_vcpu_init(struct vcpu *v) } #if SHADOW_AUDIT -int shadow_audit_enable = 0; +bool __read_mostly shadow_audit_enable; static void cf_check shadow_audit_key(unsigned char key) { @@ -884,7 +884,7 @@ static void sh_hash_audit_bucket(struct domain *d, int bucket) struct page_info *sp, *x; if ( !(SHADOW_AUDIT & (SHADOW_AUDIT_HASH|SHADOW_AUDIT_HASH_FULL)) || - !SHADOW_AUDIT_ENABLE ) + !shadow_audit_enable ) return; sp = d->arch.paging.shadow.hash_table[bucket]; @@ -953,7 +953,7 @@ static void sh_hash_audit(struct domain *d) { int i; - if ( !(SHADOW_AUDIT & SHADOW_AUDIT_HASH_FULL) || !SHADOW_AUDIT_ENABLE ) + if ( !(SHADOW_AUDIT & SHADOW_AUDIT_HASH_FULL) || !shadow_audit_enable ) return; for ( i = 0; i < SHADOW_HASH_BUCKETS; i++ ) @@ -2619,7 +2619,7 @@ void shadow_audit_tables(struct vcpu *v) unsigned int mask; if ( !(SHADOW_AUDIT & (SHADOW_AUDIT_ENTRIES | SHADOW_AUDIT_ENTRIES_FULL)) || - !SHADOW_AUDIT_ENABLE ) + !shadow_audit_enable ) return; if ( SHADOW_AUDIT & SHADOW_AUDIT_ENTRIES_FULL ) diff --git a/xen/arch/x86/mm/shadow/multi.c b/xen/arch/x86/mm/shadow/multi.c index 2990cee869..1b0477ed2b 100644 --- a/xen/arch/x86/mm/shadow/multi.c +++ b/xen/arch/x86/mm/shadow/multi.c @@ -316,7 +316,7 @@ static void sh_audit_gw(struct vcpu *v, const walk_t *gw) struct domain *d = v->domain; mfn_t smfn; - if ( !(SHADOW_AUDIT_ENABLE) ) + if ( !shadow_audit_enable ) return; #if GUEST_PAGING_LEVELS >= 4 /* 64-bit only... */ diff --git a/xen/arch/x86/mm/shadow/private.h b/xen/arch/x86/mm/shadow/private.h index 8c8e29a021..40b02421ff 100644 --- a/xen/arch/x86/mm/shadow/private.h +++ b/xen/arch/x86/mm/shadow/private.h @@ -33,11 +33,10 @@ #ifdef NDEBUG #define SHADOW_AUDIT 0 -#define SHADOW_AUDIT_ENABLE 0 +#define shadow_audit_enable false #else #define SHADOW_AUDIT 0x15 /* Basic audit of all */ -#define SHADOW_AUDIT_ENABLE shadow_audit_enable -extern int shadow_audit_enable; +extern bool shadow_audit_enable; #endif /******************************************************************************