Re: [PATCH] x86: Function missing integer return value

From: Maciej S. Szmigiero
Date: Thu May 05 2022 - 09:38:03 EST


On 5.05.2022 13:36, Juergen Gross wrote:
On 05.05.22 13:32, Li kunyu wrote:
This function may need to return a value

Signed-off-by: Li kunyu <kunyu@xxxxxxxxxxxx>
---
  arch/x86/kvm/mmu/mmu.c | 2 ++
  1 file changed, 2 insertions(+)

diff --git a/arch/x86/kvm/mmu/mmu.c b/arch/x86/kvm/mmu/mmu.c
index 64a2a7e2be90..68f33b932f94 100644
--- a/arch/x86/kvm/mmu/mmu.c
+++ b/arch/x86/kvm/mmu/mmu.c
@@ -6500,6 +6500,8 @@ static int kvm_nx_lpage_recovery_worker(struct kvm *kvm, uintptr_t data)
          kvm_recover_nx_lpages(kvm);
      }
+
+    return 0;

This statement is not reachable, so the patch is adding unneeded dead
code only.

Maybe some static checker isn't smart enough to figure this out.

In this case it would probably be better to also change:
if (kthread_should_stop())
return 0;

into:
if (kthread_should_stop())
break;

so the newly introduced code isn't dead.

Juergen

Thanks,
Maciej