Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750987AbdHRHvd (ORCPT ); Fri, 18 Aug 2017 03:51:33 -0400 Received: from mx1.redhat.com ([209.132.183.28]:57772 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750739AbdHRHvb (ORCPT ); Fri, 18 Aug 2017 03:51:31 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 852008553D Authentication-Results: ext-mx04.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx04.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=david@redhat.com Subject: Re: [PATCH 1/3] KVM: x86: simplify ept_misconfig To: Paolo Bonzini , linux-kernel@vger.kernel.org, kvm@vger.kernel.org Cc: wanpeng.li@hotmail.com, rkrcmar@redhat.com References: <1502987818-24065-1-git-send-email-pbonzini@redhat.com> <1502987818-24065-2-git-send-email-pbonzini@redhat.com> From: David Hildenbrand Organization: Red Hat GmbH Message-ID: <49fe94d6-d085-204e-452e-100ca6c672ca@redhat.com> Date: Fri, 18 Aug 2017 09:51:27 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <1502987818-24065-2-git-send-email-pbonzini@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.28]); Fri, 18 Aug 2017 07:51:31 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 635 Lines: 24 On 17.08.2017 18:36, Paolo Bonzini wrote: > Calling handle_mmio_page_fault() has been unnecessary since commit > e9ee956e311d ("KVM: x86: MMU: Move handle_mmio_page_fault() call to > kvm_mmu_page_fault()", 2016-02-22). > > handle_mmio_page_fault() can now be made static. > > Signed-off-by: Paolo Bonzini > --- > v1->v2: make the function static. > > arch/x86/kvm/mmu.c | 19 ++++++++++++++++++- > arch/x86/kvm/mmu.h | 17 ----------------- > arch/x86/kvm/vmx.c | 13 +++---------- > 3 files changed, 21 insertions(+), 28 deletions(-) Reviewed-by: David Hildenbrand -- Thanks, David