Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1350628

Re: [PATCH 00/12] KVM: MMU: cleanup around kvm_sync_page, and a few micro-optimizations

Path csiph.com!feeder.erje.net!2.eu.feeder.erje.net!news2.arglkargh.de!news.mixmin.net!aioe.org!gothmog.csi.it!bofh.it!news.nic.it!robomod
From Paolo Bonzini <pbonzini@redhat.com>
Newsgroups linux.kernel
Subject Re: [PATCH 00/12] KVM: MMU: cleanup around kvm_sync_page, and a few micro-optimizations
Date Fri, 04 Mar 2016 22:50:03 +0100
Message-ID <r95LJ-4GI-29@gated-at.bofh.it> (permalink)
References <r5Hwe-543-5@gated-at.bofh.it>
Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:subject:to:references:cc:from:message-id:date:user-agent :mime-version:in-reply-to:content-transfer-encoding; bh=yMdgqnBknsaIU0R1GTcFdDIi8KSk3ZTuAX5e3BgY0VM=; b=Chpe6G96sbhsJQ/DmTdUjPZsTXvhfl78CRuJWwReG0P9NoCE5pqaLfvhDBsAfbd8Xm f9486lB1DHDMDRMnhpLqq5BBUbH7AuOgnVayeiU8rgL0G4g0OcHnVG1/czJvQailVGir MwHzIwZtIhRVkdFrZNv+AXX3gDSmpJ1KJOd60Z9gvJUteXix2NHweF8gRg8hzfkEKBTt iLbBWuIPFjcrepC7NYegSohyrb+YjB+/LAh36s/VOCaWwpR5m09csHA3VxKjEeEqq6i1 SmfLPuhRf+CpJ4pX6Me9VMi+D0QcZfrjRVqxC2wghaB1P3jGoObH3Dd8MFGiMs8bJVhu qpRw==
X-Google-Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:sender:subject:to:references:cc:from:message-id :date:user-agent:mime-version:in-reply-to:content-transfer-encoding; bh=yMdgqnBknsaIU0R1GTcFdDIi8KSk3ZTuAX5e3BgY0VM=; b=TMjZFzZRuY+YGT9BhucqOWHOoHKn/XuOZMLRFT4xYXcKqb0o6IXXBoL7iLbRY2000t Mn3T6iH3domXVe2irBrPxr9W6Dn3FXCVAVfSX7A/hedDpgA/FXQPlHO570cpdxPhwd0K G5Mdz5QP5KRi67JO0k2Rm8+tqLaXQj2lNJlpjAfP2/u3AmKxyDhODNEiYxg9cyCbb3qc 1hX1En+Wwp91oYsvCG2zM239wRXK2VIei097YpglcZ1IvXdiHrWbovckC6vZsaFS7pOo 37oJdqMsrSabQf2wcGR05xuzZTsopeTTlda7LoWY16w4KxJO5m5dHGGXPH0w1QJuUEBy m6uA==
X-Gm-Message-State AD7BkJKB6UGku00SFCkJIVKAZDBB+9v1078N2Zc6XRDnZfWbohi7o/d7iDos3QqrQVBj8Q==
X-Received by 10.194.71.46 with SMTP id r14mr12173725wju.100.1457127797469; Fri, 04 Mar 2016 13:43:17 -0800 (PST)
User-Agent Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0
MIME-Version 1.0
Content-Type text/plain; charset=windows-1252
Content-Transfer-Encoding 7bit
Sender robomod@news.nic.it
List-ID <linux-kernel.vger.kernel.org>
X-Mailing-List linux-kernel@vger.kernel.org
Approved robomod@news.nic.it
Lines 53
Organization linux.* mail to news gateway
X-Original-Cc yoshikawa_takuya_b1@lab.ntt.co.jp, guangrong.xiao@linux.intel.com, mtosatti@redhat.com
X-Original-Date Fri, 4 Mar 2016 22:43:14 +0100
X-Original-Message-ID <56DA0172.5030406@redhat.com>
X-Original-References <1456319873-34182-1-git-send-email-pbonzini@redhat.com>
X-Original-Sender linux-kernel-owner@vger.kernel.org
Xref csiph.com linux.kernel:1350628

Show key headers only | View raw



On 24/02/2016 14:17, Paolo Bonzini wrote:
> This series started from looking at mmu_unsync_walk for the ubsan thread.
> Patches 1 and 2 are the result of the discussions in that thread.
> 
> Patches 3 to 9 do more cleanups in __kvm_sync_page and its callers.
> Among other changes, it removes kvm_sync_page_transient and avoids
> duplicate code between __kvm_sync_page and kvm_sync_pages.
> 
> I stopped where I had questions about the existing kvm_mmu_get_page
> code (see patch 8 for the question).  However perhaps more cleanups
> are possible, also thanks to Takuya's work on that function and
> link_shadow_page.
> 
> Patches 10 to 12 are just micro-optimizations.
> 
> Guangrong, it would be great if you took a look since you know this part
> of KVM very well.
> 
> I have tested this series minus patch 9, and it survived installation
> of various Linux and Windows guests with EPT disabled.  Of course before
> committing the patches I will retest with patch 9 included.
> 
> Paolo
> 
> Paolo Bonzini (11):
>   KVM: MMU: Fix ubsan warnings
>   KVM: MMU: introduce kvm_mmu_flush_or_zap
>   KVM: MMU: move TLB flush out of __kvm_sync_page
>   KVM: MMU: use kvm_sync_page in kvm_sync_pages
>   KVM: MMU: cleanup __kvm_sync_page and its callers
>   KVM: MMU: invert return value of FNAME(sync_page) and *kvm_sync_page*
>   KVM: MMU: move zap/flush to kvm_mmu_get_page
>   KVM: MMU: coalesce zapping page after mmu_sync_children
>   KVM: mark memory barrier with smp_mb__after_atomic
>   KVM: MMU: simplify last_pte_bitmap
>   KVM: MMU: micro-optimize gpte_access
> 
> Xiao Guangrong (1):
>   KVM: MMU: check kvm_mmu_pages and mmu_page_path indices
> 
>  arch/x86/include/asm/kvm_host.h |   6 +-
>  arch/x86/kvm/mmu.c              | 216 ++++++++++++++++++++++------------------
>  arch/x86/kvm/paging_tmpl.h      |  11 +-
>  virt/kvm/kvm_main.c             |   2 +-
>  4 files changed, 126 insertions(+), 109 deletions(-)
> 

I've pushed patches 1 and 2 to kvm/next now; I'll repost the others next
Monday since there were very small conflicts with the page tracking series.

Paolo

Back to linux.kernel | Previous | Next | Find similar | Unroll thread


Thread

Re: [PATCH 00/12] KVM: MMU: cleanup around kvm_sync_page, and a few  micro-optimizations Paolo Bonzini <pbonzini@redhat.com> - 2016-03-04 22:50 +0100

csiph-web