All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Matlack <dmatlack@google.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: kvm@vger.kernel.org, Andrew Jones <drjones@redhat.com>,
	Ben Gardon <bgardon@google.com>,
	Axel Rasmussen <axelrasmussen@google.com>,
	Yanan Wang <wangyanan55@huawei.com>,
	David Matlack <dmatlack@google.com>
Subject: [PATCH v2 0/3] KVM: selftests: Small fixes for dirty_log_perf_test
Date: Fri, 17 Sep 2021 17:36:54 +0000	[thread overview]
Message-ID: <20210917173657.44011-1-dmatlack@google.com> (raw)

This series fixes 2 bugs in dirty_log_perf_test:
 - Incorrect interleaving of help messages for -s and -x (patch 2)
 - Buffer overflow when using multiple slots (patch 3)

Both bugs were introduced by commit 609e6202ea5f ("KVM: selftests:
Support multiple slots in dirty_log_perf_test").

Patch 1 is a small tangentially related cleanup to use a consistent
flag for the backing source across all selftests.

v2:
 - Add Ben and Andrew's SOB to patches 1 and 2
 - Delete stray newline in patch 2 [Andrew]
 - Make print_available_backing_src_types static [Andrew]
 - Create a separate dirty bitmap per slot [Andrew, Ben]

v1: https://lore.kernel.org/kvm/20210915213034.1613552-1-dmatlack@google.com/

David Matlack (3):
  KVM: selftests: Change backing_src flag to -s in demand_paging_test
  KVM: selftests: Refactor help message for -s backing_src
  KVM: selftests: Create a separate dirty bitmap per slot

 .../selftests/kvm/access_tracking_perf_test.c |  6 +-
 .../selftests/kvm/demand_paging_test.c        | 13 ++--
 .../selftests/kvm/dirty_log_perf_test.c       | 63 +++++++++++++------
 .../testing/selftests/kvm/include/test_util.h |  4 +-
 .../selftests/kvm/kvm_page_table_test.c       |  7 +--
 tools/testing/selftests/kvm/lib/test_util.c   | 17 +++--
 6 files changed, 69 insertions(+), 41 deletions(-)

-- 
2.33.0.464.g1972c5931b-goog


             reply	other threads:[~2021-09-17 17:37 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-17 17:36 David Matlack [this message]
2021-09-17 17:36 ` [PATCH v2 1/3] KVM: selftests: Change backing_src flag to -s in demand_paging_test David Matlack
2021-09-17 17:36 ` [PATCH v2 2/3] KVM: selftests: Refactor help message for -s backing_src David Matlack
2021-09-17 17:36 ` [PATCH v2 3/3] KVM: selftests: Create a separate dirty bitmap per slot David Matlack
2021-09-20 12:05   ` Andrew Jones
2021-09-22 13:10 ` [PATCH v2 0/3] KVM: selftests: Small fixes for dirty_log_perf_test Paolo Bonzini

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=20210917173657.44011-1-dmatlack@google.com \
    --to=dmatlack@google.com \
    --cc=axelrasmussen@google.com \
    --cc=bgardon@google.com \
    --cc=drjones@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=pbonzini@redhat.com \
    --cc=wangyanan55@huawei.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.