From: kbuild test robot <lkp@intel.com> To: Peter Xu <peterx@redhat.com>, linux-kernel@vger.kernel.org, kvm@vger.kernel.org Cc: kbuild-all@lists.01.org, clang-built-linux@googlegroups.com, Vitaly Kuznetsov <vkuznets@redhat.com>, Alex Williamson <alex.williamson@redhat.com>, "Michael S . Tsirkin" <mst@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, "Dr . David Alan Gilbert" <dgilbert@redhat.com>, Christophe de Dinechin <dinechin@redhat.com>, Jason Wang <jasowang@redhat.com>, Kevin Tian <kevin.tian@intel.com>, peterx@redhat.com Subject: Re: [PATCH v9 07/14] KVM: Don't allocate dirty bitmap if dirty ring is enabled Date: Wed, 27 May 2020 03:21:45 +0800 [thread overview] Message-ID: <202005270340.nWreGtw0%lkp@intel.com> (raw) In-Reply-To: <20200523225659.1027044-8-peterx@redhat.com> [-- Attachment #1: Type: text/plain, Size: 2257 bytes --] Hi Peter, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on vhost/linux-next] [also build test WARNING on linus/master v5.7-rc7] [cannot apply to kvm/linux-next tip/auto-latest linux/master next-20200526] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system. BTW, we also suggest to use '--base' option to specify the base tree in git format-patch, please see https://stackoverflow.com/a/37406982] url: https://github.com/0day-ci/linux/commits/Peter-Xu/KVM-Dirty-ring-interface/20200524-070926 base: https://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git linux-next config: x86_64-allyesconfig (attached as .config) compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project 3393cc4cebf9969db94dc424b7a2b6195589c33b) reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # install x86_64 cross compiling tool for clang build # apt-get install binutils-x86-64-linux-gnu # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 If you fix the issue, kindly add following tag as appropriate Reported-by: kbuild test robot <lkp@intel.com> All warnings (new ones prefixed by >>, old ones prefixed by <<): >> arch/x86/kvm/mmu/mmu.c:1280:10: warning: expression which evaluates to zero treated as a null pointer constant of type 'struct kvm_memory_slot *' [-Wnon-literal-null-conversion] return false; ^~~~~ 1 warning generated. vim +1280 arch/x86/kvm/mmu/mmu.c 1269 1270 static struct kvm_memory_slot * 1271 gfn_to_memslot_dirty_bitmap(struct kvm_vcpu *vcpu, gfn_t gfn, 1272 bool no_dirty_log) 1273 { 1274 struct kvm_memory_slot *slot; 1275 1276 slot = kvm_vcpu_gfn_to_memslot(vcpu, gfn); 1277 if (!slot || slot->flags & KVM_MEMSLOT_INVALID) 1278 return NULL; 1279 if (no_dirty_log && kvm_slot_dirty_track_enabled(slot)) > 1280 return false; 1281 1282 return slot; 1283 } 1284 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 73533 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kbuild test robot <lkp@intel.com> To: kbuild-all@lists.01.org Subject: Re: [PATCH v9 07/14] KVM: Don't allocate dirty bitmap if dirty ring is enabled Date: Wed, 27 May 2020 03:21:45 +0800 [thread overview] Message-ID: <202005270340.nWreGtw0%lkp@intel.com> (raw) In-Reply-To: <20200523225659.1027044-8-peterx@redhat.com> [-- Attachment #1: Type: text/plain, Size: 2314 bytes --] Hi Peter, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on vhost/linux-next] [also build test WARNING on linus/master v5.7-rc7] [cannot apply to kvm/linux-next tip/auto-latest linux/master next-20200526] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system. BTW, we also suggest to use '--base' option to specify the base tree in git format-patch, please see https://stackoverflow.com/a/37406982] url: https://github.com/0day-ci/linux/commits/Peter-Xu/KVM-Dirty-ring-interface/20200524-070926 base: https://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git linux-next config: x86_64-allyesconfig (attached as .config) compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project 3393cc4cebf9969db94dc424b7a2b6195589c33b) reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # install x86_64 cross compiling tool for clang build # apt-get install binutils-x86-64-linux-gnu # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 If you fix the issue, kindly add following tag as appropriate Reported-by: kbuild test robot <lkp@intel.com> All warnings (new ones prefixed by >>, old ones prefixed by <<): >> arch/x86/kvm/mmu/mmu.c:1280:10: warning: expression which evaluates to zero treated as a null pointer constant of type 'struct kvm_memory_slot *' [-Wnon-literal-null-conversion] return false; ^~~~~ 1 warning generated. vim +1280 arch/x86/kvm/mmu/mmu.c 1269 1270 static struct kvm_memory_slot * 1271 gfn_to_memslot_dirty_bitmap(struct kvm_vcpu *vcpu, gfn_t gfn, 1272 bool no_dirty_log) 1273 { 1274 struct kvm_memory_slot *slot; 1275 1276 slot = kvm_vcpu_gfn_to_memslot(vcpu, gfn); 1277 if (!slot || slot->flags & KVM_MEMSLOT_INVALID) 1278 return NULL; 1279 if (no_dirty_log && kvm_slot_dirty_track_enabled(slot)) > 1280 return false; 1281 1282 return slot; 1283 } 1284 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org [-- Attachment #2: config.gz --] [-- Type: application/gzip, Size: 73533 bytes --]
next prev parent reply other threads:[~2020-05-26 19:25 UTC|newest] Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-05-23 22:56 [PATCH v9 00/14] KVM: Dirty ring interface Peter Xu 2020-05-23 22:56 ` [PATCH v9 01/14] KVM: X86: Change parameter for fast_page_fault tracepoint Peter Xu 2020-05-23 22:56 ` [PATCH v9 02/14] KVM: Cache as_id in kvm_memory_slot Peter Xu 2020-05-23 22:56 ` [PATCH v9 03/14] KVM: X86: Don't track dirty for KVM_SET_[TSS_ADDR|IDENTITY_MAP_ADDR] Peter Xu 2020-05-23 22:56 ` [PATCH v9 04/14] KVM: Pass in kvm pointer into mark_page_dirty_in_slot() Peter Xu 2020-05-23 22:56 ` [PATCH v9 05/14] KVM: X86: Implement ring-based dirty memory tracking Peter Xu 2020-05-25 2:48 ` kbuild test robot 2020-05-25 2:48 ` kbuild test robot 2020-05-25 15:26 ` Peter Xu 2020-05-25 15:26 ` Peter Xu 2020-05-25 14:54 ` kbuild test robot 2020-05-25 14:54 ` kbuild test robot 2020-05-25 15:33 ` Peter Xu 2020-05-25 15:33 ` Peter Xu 2020-05-23 22:56 ` [PATCH v9 06/14] KVM: Make dirty ring exclusive to dirty bitmap log Peter Xu 2020-05-23 22:56 ` [PATCH v9 07/14] KVM: Don't allocate dirty bitmap if dirty ring is enabled Peter Xu 2020-05-26 15:05 ` kbuild test robot 2020-05-26 15:05 ` kbuild test robot 2020-05-26 15:21 ` Peter Xu 2020-05-26 15:21 ` Peter Xu 2020-05-26 19:21 ` kbuild test robot [this message] 2020-05-26 19:21 ` kbuild test robot 2020-05-27 12:53 ` kbuild test robot 2020-05-27 12:53 ` kbuild test robot 2020-05-23 22:56 ` [PATCH v9 08/14] KVM: selftests: Always clear dirty bitmap after iteration Peter Xu 2020-05-23 22:56 ` [PATCH v9 09/14] KVM: selftests: Sync uapi/linux/kvm.h to tools/ Peter Xu 2020-05-23 22:56 ` [PATCH v9 10/14] KVM: selftests: Use a single binary for dirty/clear log test Peter Xu 2020-05-23 22:56 ` [PATCH v9 11/14] KVM: selftests: Introduce after_vcpu_run hook for dirty " Peter Xu 2020-05-23 22:56 ` [PATCH v9 12/14] KVM: selftests: Add dirty ring buffer test Peter Xu 2020-05-23 23:06 ` [PATCH v9 13/14] KVM: selftests: Let dirty_log_test async for dirty ring test Peter Xu 2020-05-23 23:06 ` [PATCH v9 14/14] KVM: selftests: Add "-c" parameter to dirty log test Peter Xu 2020-05-23 23:14 ` [PATCH v9 00/14] KVM: Dirty ring interface Peter Xu 2020-05-25 20:22 [PATCH v9 07/14] KVM: Don't allocate dirty bitmap if dirty ring is enabled kbuild test robot
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=202005270340.nWreGtw0%lkp@intel.com \ --to=lkp@intel.com \ --cc=alex.williamson@redhat.com \ --cc=clang-built-linux@googlegroups.com \ --cc=dgilbert@redhat.com \ --cc=dinechin@redhat.com \ --cc=jasowang@redhat.com \ --cc=kbuild-all@lists.01.org \ --cc=kevin.tian@intel.com \ --cc=kvm@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=mst@redhat.com \ --cc=pbonzini@redhat.com \ --cc=peterx@redhat.com \ --cc=vkuznets@redhat.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.