Message ID | 20171103185246.5999-1-gimcuan@gmail.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On 03.11.2017 19:52, Gimcuan Hui wrote: > The kvm_mmu_clear_all_pte_masks interface is only used by kvm_mmu_module_init > locally, and does not need to be called by other module, make it static. > > This patch cleans up sparse warning: > symbol 'kvm_mmu_clear_all_pte_masks' was not declared. Should it be static? > > Signed-off-by: Gimcuan Hui <gimcuan@gmail.com> > --- > arch/x86/kvm/mmu.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/x86/kvm/mmu.c b/arch/x86/kvm/mmu.c > index 7a69cf053711..6b247d64926e 100644 > --- a/arch/x86/kvm/mmu.c > +++ b/arch/x86/kvm/mmu.c > @@ -367,7 +367,7 @@ void kvm_mmu_set_mask_ptes(u64 user_mask, u64 accessed_mask, > } > EXPORT_SYMBOL_GPL(kvm_mmu_set_mask_ptes); > > -void kvm_mmu_clear_all_pte_masks(void) > +static void kvm_mmu_clear_all_pte_masks(void) > { > shadow_user_mask = 0; > shadow_accessed_mask = 0; > Reviewed-by: David Hildenbrand <david@redhat.com>
On 03/11/2017 19:52, Gimcuan Hui wrote: > The kvm_mmu_clear_all_pte_masks interface is only used by kvm_mmu_module_init > locally, and does not need to be called by other module, make it static. > > This patch cleans up sparse warning: > symbol 'kvm_mmu_clear_all_pte_masks' was not declared. Should it be static? > > Signed-off-by: Gimcuan Hui <gimcuan@gmail.com> > --- > arch/x86/kvm/mmu.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/x86/kvm/mmu.c b/arch/x86/kvm/mmu.c > index 7a69cf053711..6b247d64926e 100644 > --- a/arch/x86/kvm/mmu.c > +++ b/arch/x86/kvm/mmu.c > @@ -367,7 +367,7 @@ void kvm_mmu_set_mask_ptes(u64 user_mask, u64 accessed_mask, > } > EXPORT_SYMBOL_GPL(kvm_mmu_set_mask_ptes); > > -void kvm_mmu_clear_all_pte_masks(void) > +static void kvm_mmu_clear_all_pte_masks(void) > { > shadow_user_mask = 0; > shadow_accessed_mask = 0; > Queued, thanks. Paolo
diff --git a/arch/x86/kvm/mmu.c b/arch/x86/kvm/mmu.c index 7a69cf053711..6b247d64926e 100644 --- a/arch/x86/kvm/mmu.c +++ b/arch/x86/kvm/mmu.c @@ -367,7 +367,7 @@ void kvm_mmu_set_mask_ptes(u64 user_mask, u64 accessed_mask, } EXPORT_SYMBOL_GPL(kvm_mmu_set_mask_ptes); -void kvm_mmu_clear_all_pte_masks(void) +static void kvm_mmu_clear_all_pte_masks(void) { shadow_user_mask = 0; shadow_accessed_mask = 0;
The kvm_mmu_clear_all_pte_masks interface is only used by kvm_mmu_module_init locally, and does not need to be called by other module, make it static. This patch cleans up sparse warning: symbol 'kvm_mmu_clear_all_pte_masks' was not declared. Should it be static? Signed-off-by: Gimcuan Hui <gimcuan@gmail.com> --- arch/x86/kvm/mmu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)