linux-kernel - arch/arm64/kvm/vmid.c:62 flush_context() warn: inconsistent indenting

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>] [day] [month] [year] [list]

Message-ID: <202206020819.HlyEM8rW-lkp@intel.com>
Date:   Thu, 2 Jun 2022 08:18:33 +0800
From:   kernel test robot <lkp@...el.com>
To:     Julien Grall <julien.grall@....com>
Cc:     kbuild-all@...ts.01.org, linux-kernel@...r.kernel.org,
        Marc Zyngier <maz@...nel.org>,
        Shameer Kolothum <shameerali.kolothum.thodi@...wei.com>
Subject: arch/arm64/kvm/vmid.c:62 flush_context() warn: inconsistent indenting

Hi Julien,

First bad commit (maybe != root cause):

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   8171acb8bc9b33f3ed827f0615b24f7a06495cd0
commit: 3248136b3637e1671e4fa46e32e2122f9ec4bc3d KVM: arm64: Align the VMID allocation with the arm64 ASID
date:   4 months ago
config: arm64-randconfig-m031-20220530 (https://download.01.org/0day-ci/archive/20220602/202206020819.HlyEM8rW-lkp@intel.com/config)
compiler: aarch64-linux-gcc (GCC) 11.3.0

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@...el.com>

smatch warnings:
arch/arm64/kvm/vmid.c:62 flush_context() warn: inconsistent indenting

vim +62 arch/arm64/kvm/vmid.c

417838392f2e657 Shameer Kolothum 2021-11-22  34  
417838392f2e657 Shameer Kolothum 2021-11-22  35  #define vmid_gen_match(vmid) \
417838392f2e657 Shameer Kolothum 2021-11-22  36  	(!(((vmid) ^ atomic64_read(&vmid_generation)) >> kvm_arm_vmid_bits))
417838392f2e657 Shameer Kolothum 2021-11-22  37  
417838392f2e657 Shameer Kolothum 2021-11-22  38  static void flush_context(void)
417838392f2e657 Shameer Kolothum 2021-11-22  39  {
417838392f2e657 Shameer Kolothum 2021-11-22  40  	int cpu;
417838392f2e657 Shameer Kolothum 2021-11-22  41  	u64 vmid;
417838392f2e657 Shameer Kolothum 2021-11-22  42  
417838392f2e657 Shameer Kolothum 2021-11-22  43  	bitmap_clear(vmid_map, 0, NUM_USER_VMIDS);
417838392f2e657 Shameer Kolothum 2021-11-22  44  
417838392f2e657 Shameer Kolothum 2021-11-22  45  	for_each_possible_cpu(cpu) {
417838392f2e657 Shameer Kolothum 2021-11-22  46  		vmid = atomic64_xchg_relaxed(&per_cpu(active_vmids, cpu), 0);
417838392f2e657 Shameer Kolothum 2021-11-22  47  
417838392f2e657 Shameer Kolothum 2021-11-22  48  		/* Preserve reserved VMID */
417838392f2e657 Shameer Kolothum 2021-11-22  49  		if (vmid == 0)
417838392f2e657 Shameer Kolothum 2021-11-22  50  			vmid = per_cpu(reserved_vmids, cpu);
417838392f2e657 Shameer Kolothum 2021-11-22  51  		__set_bit(vmid2idx(vmid), vmid_map);
417838392f2e657 Shameer Kolothum 2021-11-22  52  		per_cpu(reserved_vmids, cpu) = vmid;
417838392f2e657 Shameer Kolothum 2021-11-22  53  	}
417838392f2e657 Shameer Kolothum 2021-11-22  54  
417838392f2e657 Shameer Kolothum 2021-11-22  55  	/*
417838392f2e657 Shameer Kolothum 2021-11-22  56  	 * Unlike ASID allocator, we expect less frequent rollover in
417838392f2e657 Shameer Kolothum 2021-11-22  57  	 * case of VMIDs. Hence, instead of marking the CPU as
417838392f2e657 Shameer Kolothum 2021-11-22  58  	 * flush_pending and issuing a local context invalidation on
417838392f2e657 Shameer Kolothum 2021-11-22  59  	 * the next context-switch, we broadcast TLB flush + I-cache
417838392f2e657 Shameer Kolothum 2021-11-22  60  	 * invalidation over the inner shareable domain on rollover.
417838392f2e657 Shameer Kolothum 2021-11-22  61  	 */
417838392f2e657 Shameer Kolothum 2021-11-22 @62  	 kvm_call_hyp(__kvm_flush_vm_context);
417838392f2e657 Shameer Kolothum 2021-11-22  63  }
417838392f2e657 Shameer Kolothum 2021-11-22  64  

:::::: The code at line 62 was first introduced by commit
:::::: 417838392f2e657ee25cc30e373ff4c35a0faa90 KVM: arm64: Introduce a new VMID allocator for KVM

:::::: TO: Shameer Kolothum <shameerali.kolothum.thodi@...wei.com>
:::::: CC: Marc Zyngier <maz@...nel.org>

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp