linux-kernel - [PATCH] KVM: arm64:Use #include <linux/cpufeature.h> instead of <asm/cpufeature.h> trailing whitespace braces {} are not necessary for any arm of this statement

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 13 Jul 2023 11:03:44 +0800
From:   chenqingyun001@...suo.com
To:     maz@...nel.org, oliver.upton@...ux.dev, catalin.marinas@....com,
        will@...nel.org
Cc:     kvmarm@...ts.linux.dev, linux-kernel@...r.kernel.org,
        james.morse@....com, suzuki.poulose@....com, yuzenghui@...wei.com
Subject: [PATCH] KVM: arm64:Use #include <linux/cpufeature.h> instead of
 <asm/cpufeature.h>        trailing whitespace     braces {} are not necessary for
 any arm of this statement

include <linux/cpufeature.h> is a generic
Having extra spaces or tabs has no real effect
If there is only one statement in the if branch,
curly braces {} can be omitted

Signed-off-by: Qingyun Chen <chenqingyun001@...suo.com>
---
  arch/arm64/kvm/reset.c | 8 ++++----
  1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm64/kvm/reset.c b/arch/arm64/kvm/reset.c
index b5dee8e57e77..4e6305dd1909 100644
--- a/arch/arm64/kvm/reset.c
+++ b/arch/arm64/kvm/reset.c
@@ -19,7 +19,7 @@

  #include <kvm/arm_arch_timer.h>

-#include <asm/cpufeature.h>
+#include <linux/cpufeature.h>
  #include <asm/cputype.h>
  #include <asm/fpsimd.h>
  #include <asm/ptrace.h>
@@ -122,7 +122,7 @@ static int kvm_vcpu_finalize_sve(struct kvm_vcpu 
*vcpu)
          kfree(buf);
          return ret;
      }
-
+
      vcpu->arch.sve_state = buf;
      vcpu_set_flag(vcpu, VCPU_SVE_FINALIZED);
      return 0;
@@ -308,9 +308,9 @@ int kvm_reset_vcpu(struct kvm_vcpu *vcpu)

      switch (vcpu->arch.target) {
      default:
-        if (vcpu_el1_is_32bit(vcpu)) {
+        if (vcpu_el1_is_32bit(vcpu))
              pstate = VCPU_RESET_PSTATE_SVC;
-        } else if (vcpu_has_nv(vcpu)) {
+        else if (vcpu_has_nv(vcpu)) {
              pstate = VCPU_RESET_PSTATE_EL2;
          } else {
              pstate = VCPU_RESET_PSTATE_EL1;