CC: [email protected]
In-Reply-To: <[email protected]>
References: <[email protected]>
TO: David Matlack <[email protected]>
TO: [email protected]
CC: Ben Gardon <[email protected]>
CC: Joerg Roedel <[email protected]>
CC: Jim Mattson <[email protected]>
CC: Wanpeng Li <[email protected]>
CC: Vitaly Kuznetsov <[email protected]>
CC: Sean Christopherson <[email protected]>
CC: Paolo Bonzini <[email protected]>
CC: Junaid Shahid <[email protected]>
CC: Andrew Jones <[email protected]>

Hi David,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on kvm/queue]
[also build test WARNING on vhost/linux-next linus/master v5.14-rc1 
next-20210714]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    
https://github.com/0day-ci/linux/commits/David-Matlack/KVM-x86-mmu-Fast-page-fault-support-for-the-TDP-MMU/20210714-061057
base:   https://git.kernel.org/pub/scm/virt/kvm/kvm.git queue
:::::: branch date: 13 hours ago
:::::: commit date: 13 hours ago
config: x86_64-rhel-8.3-kselftests (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce:
        # apt-get install sparse
        # sparse version: v0.6.3-341-g8af24329-dirty
        # 
https://github.com/0day-ci/linux/commit/b49595e7d2c56cc13bb1d6de2ab41002178b26ec
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review 
David-Matlack/KVM-x86-mmu-Fast-page-fault-support-for-the-TDP-MMU/20210714-061057
        git checkout b49595e7d2c56cc13bb1d6de2ab41002178b26ec
        # save the attached .config to linux build tree
        make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir 
ARCH=x86_64 SHELL=/bin/bash

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <[email protected]>


sparse warnings: (new ones prefixed by >>)
>> arch/x86/kvm/mmu/mmu.c:687:13: sparse: sparse: context imbalance in 
>> 'walk_shadow_page_lockless_begin' - different lock contexts for basic block
   arch/x86/kvm/mmu/mmu.c: note: in included file (through 
include/linux/rbtree.h, include/linux/mm_types.h, arch/x86/kvm/irq.h):
   include/linux/rcupdate.h:709:9: sparse: sparse: context imbalance in 
'walk_shadow_page_lockless_end' - unexpected unlock
   arch/x86/kvm/mmu/mmu.c:4382:57: sparse: sparse: cast truncates bits from 
constant value (ffffff33 becomes 33)
   arch/x86/kvm/mmu/mmu.c:4384:56: sparse: sparse: cast truncates bits from 
constant value (ffffff0f becomes f)
   arch/x86/kvm/mmu/mmu.c:4386:57: sparse: sparse: cast truncates bits from 
constant value (ffffff55 becomes 55)

vim +/walk_shadow_page_lockless_begin +687 arch/x86/kvm/mmu/mmu.c

f160c7b7bb322b arch/x86/kvm/mmu.c     Junaid Shahid  2016-12-06  686  
c2a2ac2b563ccc arch/x86/kvm/mmu.c     Xiao Guangrong 2011-07-12 @687  static 
void walk_shadow_page_lockless_begin(struct kvm_vcpu *vcpu)
c2a2ac2b563ccc arch/x86/kvm/mmu.c     Xiao Guangrong 2011-07-12  688  {
b49595e7d2c56c arch/x86/kvm/mmu/mmu.c David Matlack  2021-07-13  689    if 
(is_tdp_mmu(vcpu->arch.mmu)) {
b49595e7d2c56c arch/x86/kvm/mmu/mmu.c David Matlack  2021-07-13  690            
kvm_tdp_mmu_walk_lockless_begin();
b49595e7d2c56c arch/x86/kvm/mmu/mmu.c David Matlack  2021-07-13  691            
return;
b49595e7d2c56c arch/x86/kvm/mmu/mmu.c David Matlack  2021-07-13  692    }
b49595e7d2c56c arch/x86/kvm/mmu/mmu.c David Matlack  2021-07-13  693  
c142786c629118 arch/x86/kvm/mmu.c     Avi Kivity     2012-05-14  694    /*
c142786c629118 arch/x86/kvm/mmu.c     Avi Kivity     2012-05-14  695     * 
Prevent page table teardown by making any free-er wait during
c142786c629118 arch/x86/kvm/mmu.c     Avi Kivity     2012-05-14  696     * 
kvm_flush_remote_tlbs() IPI to all active vcpus.
c142786c629118 arch/x86/kvm/mmu.c     Avi Kivity     2012-05-14  697     */
c142786c629118 arch/x86/kvm/mmu.c     Avi Kivity     2012-05-14  698    
local_irq_disable();
36ca7e0a577aa6 arch/x86/kvm/mmu.c     Lan Tianyu     2016-03-13  699  
c142786c629118 arch/x86/kvm/mmu.c     Avi Kivity     2012-05-14  700    /*
c142786c629118 arch/x86/kvm/mmu.c     Avi Kivity     2012-05-14  701     * Make 
sure a following spte read is not reordered ahead of the write
c142786c629118 arch/x86/kvm/mmu.c     Avi Kivity     2012-05-14  702     * to 
vcpu->mode.
c142786c629118 arch/x86/kvm/mmu.c     Avi Kivity     2012-05-14  703     */
36ca7e0a577aa6 arch/x86/kvm/mmu.c     Lan Tianyu     2016-03-13  704    
smp_store_mb(vcpu->mode, READING_SHADOW_PAGE_TABLES);
c2a2ac2b563ccc arch/x86/kvm/mmu.c     Xiao Guangrong 2011-07-12  705  }
c2a2ac2b563ccc arch/x86/kvm/mmu.c     Xiao Guangrong 2011-07-12  706  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]

Attachment: .config.gz
Description: application/gzip

_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to