From mboxrd@z Thu Jan 1 00:00:00 1970 From: Radim =?utf-8?B?S3LEjW3DocWZ?= Subject: Re: [PATCH 0/2] page track add notifier type track_flush_slot Date: Wed, 12 Oct 2016 22:52:10 +0200 Message-ID: <20161012205209.GD16406@potion> References: <1475998904-13456-1-git-send-email-xiaoguang.chen@intel.com> <9c7b2357-9e36-1990-30a9-0442b6c337cf@redhat.com> <666d0e58-945c-b026-677a-db336bf1c913@linux.intel.com> <6c161c5f-cff1-13af-af39-ab9095237a67@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Xiao Guangrong , Xiaoguang Chen , kvm@vger.kernel.org, guangrong.xiao@intel.com, jike.song@intel.com To: Paolo Bonzini Return-path: Received: from mx1.redhat.com ([209.132.183.28]:56034 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934182AbcJLUwN (ORCPT ); Wed, 12 Oct 2016 16:52:13 -0400 Content-Disposition: inline In-Reply-To: <6c161c5f-cff1-13af-af39-ab9095237a67@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: 2016-10-11 10:55+0200, Paolo Bonzini: > On 11/10/2016 04:43, Xiao Guangrong wrote: >> On 10/11/2016 01:06 AM, Paolo Bonzini wrote: >>> On 09/10/2016 09:41, Xiaoguang Chen wrote: >>>> The seires is to add a new notifer type track_flush_slot for page track. >>>> By using this notifer type when a memory slot is being moved or removed >>>> users of page track can be notified. >>>> >>>> This notifier type is needed by KVMGT to sync up its shadow page table >>>> when memory slot is being moved or removed. >>>> >>>> Xiaoguang Chen (2): >>>> KVM: page track: add a new notifier type: track_flush_slot >>>> KVM: MMU: apply page track notifier type track_flush_slot >>>> >>>> arch/x86/include/asm/kvm_page_track.h | 9 +++++++++ >>>> arch/x86/kvm/mmu.c | 7 +++++++ >>>> arch/x86/kvm/page_track.c | 25 +++++++++++++++++++++++++ >>>> arch/x86/kvm/x86.c | 2 +- >>>> 4 files changed, 42 insertions(+), 1 deletion(-) >>>> >>> >>> Hi, >>> >>> the two patches should be squashed for bisectability (alternatively, do >>> not remove the call to kvm_mmu_invalidate_zap_all_pages in patch 1, and >>> only drop it in patch 2). >> >> Indeed. >> >> Sorry for the carelessness, we will adapt these patches. > > We can squash them too, it's easy. I did that and applied to kvm/queue, thanks. (Please check whether the result is agreeable.)