From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paolo Bonzini Subject: Re: KVM: HAVE_KVM_ARCH_DIRTY_LOG_PROTECT? Date: Thu, 22 Jan 2015 16:32:37 +0100 Message-ID: <54C11815.8070501@redhat.com> References: <1421918381.13638.43.camel@x220> <54C101B9.1050804@redhat.com> <1421938175.13638.58.camel@x220> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: Mario Smarduch , Valentin Rothberg , Christoffer Dall , Gleb Natapov , kvm@vger.kernel.org, linux-kernel@vger.kernel.org To: Paul Bolle Return-path: Received: from mx1.redhat.com ([209.132.183.28]:46938 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752061AbbAVPcw (ORCPT ); Thu, 22 Jan 2015 10:32:52 -0500 In-Reply-To: <1421938175.13638.58.camel@x220> Sender: kvm-owner@vger.kernel.org List-ID: On 22/01/2015 15:49, Paul Bolle wrote: >> > Ah, there are two Kconfig symbols added by mistake. >> > >> > +config HAVE_KVM_ARCH_DIRTY_LOG_PROTECT >> > + bool >> > + >> > +config KVM_GENERIC_DIRTYLOG_READ_PROTECT >> > + bool > This one is actually used (so my 800 line perl monster didn't bark): > $ git grep -n KVM_GENERIC_DIRTYLOG_READ_PROTECT next-20150122 > next-20150122:arch/arm/kvm/Kconfig:27: select KVM_GENERIC_DIRTYLOG_READ_PROTECT > next-20150122:arch/arm64/kvm/Kconfig:30: select KVM_GENERIC_DIRTYLOG_READ_PROTECT > next-20150122:arch/x86/kvm/Kconfig:42: select KVM_GENERIC_DIRTYLOG_READ_PROTECT > next-20150122:virt/kvm/Kconfig:47:config KVM_GENERIC_DIRTYLOG_READ_PROTECT > next-20150122:virt/kvm/kvm_main.c:998:#ifdef CONFIG_KVM_GENERIC_DIRTYLOG_READ_PROTECT > Yes, the mistake is adding two symbols instead of one. :) Paolo