[PATCH] x86/xen: make xen_pvmmu_arch_setup() static

Jason Yan posted 1 patch 3 years, 11 months ago
Failed in applying to current master (apply log)
arch/x86/xen/setup.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] x86/xen: make xen_pvmmu_arch_setup() static
Posted by Jason Yan 3 years, 11 months ago
Fix the following sparse warning:

arch/x86/xen/setup.c:998:12: warning: symbol 'xen_pvmmu_arch_setup' was not
declared. Should it be static?

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Jason Yan <yanaijie@huawei.com>
---
 arch/x86/xen/setup.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/x86/xen/setup.c b/arch/x86/xen/setup.c
index 33b0e20df7fc..1a2d8a50dac4 100644
--- a/arch/x86/xen/setup.c
+++ b/arch/x86/xen/setup.c
@@ -985,7 +985,7 @@ void xen_enable_syscall(void)
 #endif /* CONFIG_X86_64 */
 }
 
-void __init xen_pvmmu_arch_setup(void)
+static void __init xen_pvmmu_arch_setup(void)
 {
 	HYPERVISOR_vm_assist(VMASST_CMD_enable, VMASST_TYPE_4gb_segments);
 	HYPERVISOR_vm_assist(VMASST_CMD_enable, VMASST_TYPE_writable_pagetables);
-- 
2.17.2


Re: [PATCH] x86/xen: make xen_pvmmu_arch_setup() static
Posted by Jürgen Groß 3 years, 11 months ago
On 08.04.20 04:46, Jason Yan wrote:
> Fix the following sparse warning:
> 
> arch/x86/xen/setup.c:998:12: warning: symbol 'xen_pvmmu_arch_setup' was not
> declared. Should it be static?
> 
> Reported-by: Hulk Robot <hulkci@huawei.com>
> Signed-off-by: Jason Yan <yanaijie@huawei.com>

Reviewed-by: Juergen Gross <jgross@suse.com>


Juergen