Skip to content

Commit

Permalink
KVM: selftests: Shorten the test args in memslot_modification_stress_…
Browse files Browse the repository at this point in the history
…test.c

Change test args memslot_modification_delay and nr_memslot_modifications
to delay and nr_iterations for simplicity.

Signed-off-by: Vipin Sharma <vipinsh@google.com>
Suggested-by: Sean Christopherson <seanjc@google.com>
Reviewed-by: Sean Christopherson <seanjc@google.com>
Link: https://lore.kernel.org/r/20221103191719.1559407-6-vipinsh@google.com
Signed-off-by: Sean Christopherson <seanjc@google.com>
  • Loading branch information
Vipin Sharma authored and Sean Christopherson committed Nov 16, 2022
1 parent 69a62e2 commit c15bdeb
Showing 1 changed file with 8 additions and 10 deletions.
18 changes: 8 additions & 10 deletions tools/testing/selftests/kvm/memslot_modification_stress_test.c
Original file line number Diff line number Diff line change
Expand Up @@ -87,8 +87,8 @@ static void add_remove_memslot(struct kvm_vm *vm, useconds_t delay,
}

struct test_params {
useconds_t memslot_modification_delay;
uint64_t nr_memslot_modifications;
useconds_t delay;
uint64_t nr_iterations;
bool partition_vcpu_memory_access;
};

Expand All @@ -107,8 +107,7 @@ static void run_test(enum vm_guest_mode mode, void *arg)

pr_info("Started all vCPUs\n");

add_remove_memslot(vm, p->memslot_modification_delay,
p->nr_memslot_modifications);
add_remove_memslot(vm, p->delay, p->nr_iterations);

run_vcpus = false;

Expand Down Expand Up @@ -144,9 +143,8 @@ int main(int argc, char *argv[])
int max_vcpus = kvm_check_cap(KVM_CAP_MAX_VCPUS);
int opt;
struct test_params p = {
.memslot_modification_delay = 0,
.nr_memslot_modifications =
DEFAULT_MEMSLOT_MODIFICATION_ITERATIONS,
.delay = 0,
.nr_iterations = DEFAULT_MEMSLOT_MODIFICATION_ITERATIONS,
.partition_vcpu_memory_access = true
};

Expand All @@ -158,8 +156,8 @@ int main(int argc, char *argv[])
guest_modes_cmdline(optarg);
break;
case 'd':
p.memslot_modification_delay = atoi_paranoid(optarg);
TEST_ASSERT(p.memslot_modification_delay >= 0,
p.delay = atoi_paranoid(optarg);
TEST_ASSERT(p.delay >= 0,
"A negative delay is not supported.");
break;
case 'b':
Expand All @@ -175,7 +173,7 @@ int main(int argc, char *argv[])
p.partition_vcpu_memory_access = false;
break;
case 'i':
p.nr_memslot_modifications = atoi_paranoid(optarg);
p.nr_iterations = atoi_paranoid(optarg);
break;
case 'h':
default:
Expand Down

0 comments on commit c15bdeb

Please sign in to comment.