diff mbox series

KVM: PPC: code cleanup for kvmppc_book3s_irqprio_deliver

Message ID 20240124093647.479176-1-chentao@kylinos.cn
State New
Headers
Series KVM: PPC: code cleanup for kvmppc_book3s_irqprio_deliver |

Commit Message

Kunwu Chan Jan. 24, 2024, 9:36 a.m. UTC 
  This part was commented from commit 2f4cf5e42d13 ("Add book3s.c")
in about 14 years before.
If there are no plans to enable this part code in the future,
we can remove this dead code.

Signed-off-by: Kunwu Chan <chentao@kylinos.cn>
---
 arch/powerpc/kvm/book3s.c | 3 ---
 1 file changed, 3 deletions(-)

Comments

Christophe Leroy Jan. 24, 2024, 1:01 p.m. UTC | #1 
Le 24/01/2024 à 10:36, Kunwu Chan a écrit :
> This part was commented from commit 2f4cf5e42d13 ("Add book3s.c")
> in about 14 years before.
> If there are no plans to enable this part code in the future,
> we can remove this dead code.
> 
> Signed-off-by: Kunwu Chan <chentao@kylinos.cn>
> ---
>   arch/powerpc/kvm/book3s.c | 3 ---
>   1 file changed, 3 deletions(-)
> 
> diff --git a/arch/powerpc/kvm/book3s.c b/arch/powerpc/kvm/book3s.c
> index 8acec144120e..c2f50e04eec8 100644
> --- a/arch/powerpc/kvm/book3s.c
> +++ b/arch/powerpc/kvm/book3s.c
> @@ -360,9 +360,6 @@ static int kvmppc_book3s_irqprio_deliver(struct kvm_vcpu *vcpu,
>   		break;
>   	}
>   
> -#if 0
> -	printk(KERN_INFO "Deliver interrupt 0x%x? %x\n", vec, deliver);
> -#endif

Please also remove one of the two blank lines.

>   
>   	if (deliver)
>   		kvmppc_inject_interrupt(vcpu, vec, 0);
Markus Elfring Jan. 24, 2024, 5:04 p.m. UTC | #2 
> If there are no plans to enable this part code in the future,

Will the word combination “code part” become preferred for
a subsequent change description?


> we can remove this dead code.

And omit another blank line accordingly?

Regards,
Markus
Kunwu Chan Jan. 25, 2024, 8:36 a.m. UTC | #3 
On 2024/1/24 21:01, Christophe Leroy wrote:
> 
> 
> Le 24/01/2024 à 10:36, Kunwu Chan a écrit :
>> This part was commented from commit 2f4cf5e42d13 ("Add book3s.c")
>> in about 14 years before.
>> If there are no plans to enable this part code in the future,
>> we can remove this dead code.
>>
>> Signed-off-by: Kunwu Chan <chentao@kylinos.cn>
>> ---
>>    arch/powerpc/kvm/book3s.c | 3 ---
>>    1 file changed, 3 deletions(-)
>>
>> diff --git a/arch/powerpc/kvm/book3s.c b/arch/powerpc/kvm/book3s.c
>> index 8acec144120e..c2f50e04eec8 100644
>> --- a/arch/powerpc/kvm/book3s.c
>> +++ b/arch/powerpc/kvm/book3s.c
>> @@ -360,9 +360,6 @@ static int kvmppc_book3s_irqprio_deliver(struct kvm_vcpu *vcpu,
>>    		break;
>>    	}
>>    
>> -#if 0
>> -	printk(KERN_INFO "Deliver interrupt 0x%x? %x\n", vec, deliver);
>> -#endif
> 
> Please also remove one of the two blank lines.
Thanks for your reply. I've send the v2 patch:
https://lore.kernel.org/all/20240125082637.532826-1-chentao@kylinos.cn/
https://lore.kernel.org/all/20240125083348.533883-1-chentao@kylinos.cn/
>>    
>>    	if (deliver)
>>    		kvmppc_inject_interrupt(vcpu, vec, 0);
diff mbox series

Patch

diff --git a/arch/powerpc/kvm/book3s.c b/arch/powerpc/kvm/book3s.c
index 8acec144120e..c2f50e04eec8 100644
--- a/arch/powerpc/kvm/book3s.c
+++ b/arch/powerpc/kvm/book3s.c
@@ -360,9 +360,6 @@  static int kvmppc_book3s_irqprio_deliver(struct kvm_vcpu *vcpu,
 		break;
 	}
 
-#if 0
-	printk(KERN_INFO "Deliver interrupt 0x%x? %x\n", vec, deliver);
-#endif
 
 	if (deliver)
 		kvmppc_inject_interrupt(vcpu, vec, 0);