Re: [PATCH] KVM: arm: get rid of unused arg in cpu_init_hyp_mode()
On 2019-11-21 07:15, linmiaohe wrote: From: Miaohe Lin As arg dummy is not really needed, there's no need to pass NULL when calling cpu_init_hyp_mode(). So clean it up. Signed-off-by: Miaohe Lin --- virt/kvm/arm/arm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/virt/kvm/arm/arm.c b/virt/kvm/arm/arm.c index 86c6aa1cb58e..a5470f1b1a19 100644 --- a/virt/kvm/arm/arm.c +++ b/virt/kvm/arm/arm.c @@ -1315,7 +1315,7 @@ long kvm_arch_vm_ioctl(struct file *filp, } } -static void cpu_init_hyp_mode(void *dummy) +static void cpu_init_hyp_mode(void) { phys_addr_t pgd_ptr; unsigned long hyp_stack_ptr; @@ -1349,7 +1349,7 @@ static void cpu_hyp_reinit(void) if (is_kernel_in_hyp_mode()) kvm_timer_init_vhe(); else - cpu_init_hyp_mode(NULL); + cpu_init_hyp_mode(); kvm_arm_init_debug(); Applied, thanks. M. -- Jazz is not dead. It just smells funny... ___ kvmarm mailing list kvmarm@lists.cs.columbia.edu https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
Re: [PATCH] KVM: arm: get rid of unused arg in cpu_init_hyp_mode()
On 30/11/2019 07:20, linmiaohe wrote: >> From: Miaohe Lin >> >> As arg dummy is not really needed, there's no need to pass NULL when calling >> cpu_init_hyp_mode(). So clean it up. It looks like the requirement for this dummy arg was removed in 67f691976662 ("arm64: kvm: allows kvm cpu hotplug"). FWIW: Reviewed-by: Steven Price >> Signed-off-by: Miaohe Lin >> --- >> virt/kvm/arm/arm.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/virt/kvm/arm/arm.c b/virt/kvm/arm/arm.c index >> 86c6aa1cb58e..a5470f1b1a19 100644 >> --- a/virt/kvm/arm/arm.c >> +++ b/virt/kvm/arm/arm.c >> @@ -1315,7 +1315,7 @@ long kvm_arch_vm_ioctl(struct file *filp, >> } >> } >> >> -static void cpu_init_hyp_mode(void *dummy) >> +static void cpu_init_hyp_mode(void) >> { >> phys_addr_t pgd_ptr; >> unsigned long hyp_stack_ptr; >> @@ -1349,7 +1349,7 @@ static void cpu_hyp_reinit(void) >> if (is_kernel_in_hyp_mode()) >> kvm_timer_init_vhe(); >> else >> -cpu_init_hyp_mode(NULL); >> +cpu_init_hyp_mode(); >> >> kvm_arm_init_debug(); >> > friendly ping ... > ___ kvmarm mailing list kvmarm@lists.cs.columbia.edu https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
Re: [PATCH] KVM: arm: get rid of unused arg in cpu_init_hyp_mode()
>From: Miaohe Lin > >As arg dummy is not really needed, there's no need to pass NULL when calling >cpu_init_hyp_mode(). So clean it up. > >Signed-off-by: Miaohe Lin >--- > virt/kvm/arm/arm.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >diff --git a/virt/kvm/arm/arm.c b/virt/kvm/arm/arm.c index >86c6aa1cb58e..a5470f1b1a19 100644 >--- a/virt/kvm/arm/arm.c >+++ b/virt/kvm/arm/arm.c >@@ -1315,7 +1315,7 @@ long kvm_arch_vm_ioctl(struct file *filp, > } > } > >-static void cpu_init_hyp_mode(void *dummy) >+static void cpu_init_hyp_mode(void) > { > phys_addr_t pgd_ptr; > unsigned long hyp_stack_ptr; >@@ -1349,7 +1349,7 @@ static void cpu_hyp_reinit(void) > if (is_kernel_in_hyp_mode()) > kvm_timer_init_vhe(); > else >- cpu_init_hyp_mode(NULL); >+ cpu_init_hyp_mode(); > > kvm_arm_init_debug(); > friendly ping ... ___ kvmarm mailing list kvmarm@lists.cs.columbia.edu https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
[PATCH] KVM: arm: get rid of unused arg in cpu_init_hyp_mode()
From: Miaohe Lin As arg dummy is not really needed, there's no need to pass NULL when calling cpu_init_hyp_mode(). So clean it up. Signed-off-by: Miaohe Lin --- virt/kvm/arm/arm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/virt/kvm/arm/arm.c b/virt/kvm/arm/arm.c index 86c6aa1cb58e..a5470f1b1a19 100644 --- a/virt/kvm/arm/arm.c +++ b/virt/kvm/arm/arm.c @@ -1315,7 +1315,7 @@ long kvm_arch_vm_ioctl(struct file *filp, } } -static void cpu_init_hyp_mode(void *dummy) +static void cpu_init_hyp_mode(void) { phys_addr_t pgd_ptr; unsigned long hyp_stack_ptr; @@ -1349,7 +1349,7 @@ static void cpu_hyp_reinit(void) if (is_kernel_in_hyp_mode()) kvm_timer_init_vhe(); else - cpu_init_hyp_mode(NULL); + cpu_init_hyp_mode(); kvm_arm_init_debug(); -- 2.19.1 ___ kvmarm mailing list kvmarm@lists.cs.columbia.edu https://lists.cs.columbia.edu/mailman/listinfo/kvmarm