kvm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Radim Krčmář" <rkrcmar@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: Xiao Guangrong <guangrong.xiao@linux.intel.com>,
	Xiaoguang Chen <xiaoguang.chen@intel.com>,
	kvm@vger.kernel.org, guangrong.xiao@intel.com,
	jike.song@intel.com
Subject: Re: [PATCH 0/2] page track add notifier type track_flush_slot
Date: Wed, 12 Oct 2016 22:52:10 +0200	[thread overview]
Message-ID: <20161012205209.GD16406@potion> (raw)
In-Reply-To: <6c161c5f-cff1-13af-af39-ab9095237a67@redhat.com>

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.)

      reply	other threads:[~2016-10-12 20:52 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-09  7:41 [PATCH 0/2] page track add notifier type track_flush_slot Xiaoguang Chen
2016-10-09  7:41 ` [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot Xiaoguang Chen
2016-10-09  8:31   ` Neo Jia
2016-10-09  8:56     ` Chen, Xiaoguang
2016-10-10 17:06     ` Paolo Bonzini
2016-10-10 18:01       ` Neo Jia
2016-10-10 18:32         ` Paolo Bonzini
2016-10-11  2:39           ` Xiao Guangrong
2016-10-11  8:54             ` Paolo Bonzini
2016-10-11  9:21               ` Xiao Guangrong
2016-10-11  9:47                 ` Paolo Bonzini
2016-10-14 10:37                   ` Jike Song
2016-10-14 10:43                     ` Paolo Bonzini
2016-10-14 12:26                       ` Jike Song
2016-10-14 14:41                     ` [Qemu-devel] " Alex Williamson
2016-10-14 14:46                       ` Alex Williamson
2016-10-14 16:35                         ` [Qemu-devel] " Neo Jia
2016-10-14 16:51                           ` Alex Williamson
2016-10-14 22:19                             ` Neo Jia
2016-10-17 16:02                               ` Alex Williamson
2016-10-18 12:38                                 ` Jike Song
2016-10-18 14:59                                   ` Alex Williamson
2016-10-19  2:32                                     ` Jike Song
2016-10-19  5:45                                       ` Xiao Guangrong
2016-10-19 11:56                                         ` Paolo Bonzini
2016-10-19 13:39                                           ` Xiao Guangrong
2016-10-19 14:14                                             ` Paolo Bonzini
2016-10-20  1:48                                               ` [Qemu-devel] " Xiao Guangrong
2016-10-20 17:06                                                 ` Paolo Bonzini
2016-10-20 17:19                                                   ` Xiao, Guangrong
2016-10-21  2:47                                                     ` [Qemu-devel] " Jike Song
2016-10-26 13:44                                                   ` Jike Song
2016-10-26 14:45                                                     ` Paolo Bonzini
2016-10-29  4:07                                                       ` Jike Song
2016-10-19 13:56                                       ` Eric Blake
2016-10-24  6:32                                         ` [Qemu-devel] " Jike Song
2016-10-12 20:48   ` Radim Krčmář
2016-10-09  7:41 ` [PATCH 2/2] KVM: MMU: apply page track notifier type track_flush_slot Xiaoguang Chen
2016-10-10 17:06 ` [PATCH 0/2] page track add " Paolo Bonzini
2016-10-11  2:43   ` Xiao Guangrong
2016-10-11  8:55     ` Paolo Bonzini
2016-10-12 20:52       ` Radim Krčmář [this message]

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=20161012205209.GD16406@potion \
    --to=rkrcmar@redhat.com \
    --cc=guangrong.xiao@intel.com \
    --cc=guangrong.xiao@linux.intel.com \
    --cc=jike.song@intel.com \
    --cc=kvm@vger.kernel.org \
    --cc=pbonzini@redhat.com \
    --cc=xiaoguang.chen@intel.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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).