lkml.org 
[lkml]   [2019]   [Nov]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH RFC 05/15] KVM: Make dirty ring exclusive to dirty bitmap log
    Date
    There's no good reason to use both the dirty bitmap logging and the
    new dirty ring buffer to track dirty bits. We should be able to even
    support both of them at the same time, but it could complicate things
    which could actually help little. Let's simply make it the rule
    before we enable dirty ring on any arch, that we don't allow these two
    interfaces to be used together.

    The big world switch would be KVM_CAP_DIRTY_LOG_RING capability
    enablement. That's where we'll switch from the default dirty logging
    way to the dirty ring way. As long as kvm->dirty_ring_size is setup
    correctly, we'll once and for all switch to the dirty ring buffer mode
    for the current virtual machine.

    Signed-off-by: Peter Xu <peterx@redhat.com>
    ---
    Documentation/virt/kvm/api.txt | 7 +++++++
    virt/kvm/kvm_main.c | 12 ++++++++++++
    2 files changed, 19 insertions(+)

    diff --git a/Documentation/virt/kvm/api.txt b/Documentation/virt/kvm/api.txt
    index fa622c9a2eb8..9f72ca1fd3e4 100644
    --- a/Documentation/virt/kvm/api.txt
    +++ b/Documentation/virt/kvm/api.txt
    @@ -5487,3 +5487,10 @@ with the exit reason set to KVM_EXIT_DIRTY_LOG_FULL, and the
    KVM_RUN ioctl will return -EINTR. Once that happens, userspace
    should pause all the vcpus, then harvest all the dirty pages and
    rearm the dirty traps. It can unpause the guest after that.
    +
    +NOTE: the KVM_CAP_DIRTY_LOG_RING capability and the new ioctl
    +KVM_RESET_DIRTY_RINGS are exclusive to the existing KVM_GET_DIRTY_LOG
    +interface. After enabling KVM_CAP_DIRTY_LOG_RING with an acceptable
    +dirty ring size, the virtual machine will switch to the dirty ring
    +tracking mode, and KVM_GET_DIRTY_LOG, KVM_CLEAR_DIRTY_LOG ioctls will
    +stop working.
    diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c
    index 8642c977629b..782127d11e9d 100644
    --- a/virt/kvm/kvm_main.c
    +++ b/virt/kvm/kvm_main.c
    @@ -1236,6 +1236,10 @@ int kvm_get_dirty_log(struct kvm *kvm,
    unsigned long n;
    unsigned long any = 0;

    + /* Dirty ring tracking is exclusive to dirty log tracking */
    + if (kvm->dirty_ring_size)
    + return -EINVAL;
    +
    as_id = log->slot >> 16;
    id = (u16)log->slot;
    if (as_id >= KVM_ADDRESS_SPACE_NUM || id >= KVM_USER_MEM_SLOTS)
    @@ -1293,6 +1297,10 @@ int kvm_get_dirty_log_protect(struct kvm *kvm,
    unsigned long *dirty_bitmap;
    unsigned long *dirty_bitmap_buffer;

    + /* Dirty ring tracking is exclusive to dirty log tracking */
    + if (kvm->dirty_ring_size)
    + return -EINVAL;
    +
    as_id = log->slot >> 16;
    id = (u16)log->slot;
    if (as_id >= KVM_ADDRESS_SPACE_NUM || id >= KVM_USER_MEM_SLOTS)
    @@ -1364,6 +1372,10 @@ int kvm_clear_dirty_log_protect(struct kvm *kvm,
    unsigned long *dirty_bitmap;
    unsigned long *dirty_bitmap_buffer;

    + /* Dirty ring tracking is exclusive to dirty log tracking */
    + if (kvm->dirty_ring_size)
    + return -EINVAL;
    +
    as_id = log->slot >> 16;
    id = (u16)log->slot;
    if (as_id >= KVM_ADDRESS_SPACE_NUM || id >= KVM_USER_MEM_SLOTS)
    --
    2.21.0
    \
     
     \ /
      Last update: 2019-11-29 22:36    [W:4.193 / U:0.200 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site