flush_vcpu static function here is only used in one place which is just below where it is defined. Inline the function to reduce the noise and clarify what we are doing.
No functional change. Signed-off-by: Teddy Astie <[email protected]> --- xen/arch/x86/mm/hap/hap.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/xen/arch/x86/mm/hap/hap.c b/xen/arch/x86/mm/hap/hap.c index 2f69ff9c7b..407c80afab 100644 --- a/xen/arch/x86/mm/hap/hap.c +++ b/xen/arch/x86/mm/hap/hap.c @@ -721,11 +721,6 @@ static pagetable_t cf_check hap_update_cr3(struct vcpu *v, bool noflush) return pagetable_null(); } -static bool flush_vcpu(const struct vcpu *v, const unsigned long *vcpu_bitmap) -{ - return !vcpu_bitmap || test_bit(v->vcpu_id, vcpu_bitmap); -} - /* Flush TLB of selected vCPUs. NULL for all. */ static bool cf_check flush_tlb(const unsigned long *vcpu_bitmap) { @@ -742,7 +737,7 @@ static bool cf_check flush_tlb(const unsigned long *vcpu_bitmap) { unsigned int cpu; - if ( !flush_vcpu(v, vcpu_bitmap) ) + if ( vcpu_bitmap && !test_bit(v->vcpu_id, vcpu_bitmap) ) continue; hvm_asid_flush_vcpu(v); -- 2.51.0 -- Teddy Astie | Vates XCP-ng Developer XCP-ng & Xen Orchestra - Vates solutions web: https://vates.tech
