From patchwork Wed Nov 2 23:27:32 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vipin Sharma X-Patchwork-Id: 14563 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:6687:0:0:0:0:0 with SMTP id l7csp208914wru; Wed, 2 Nov 2022 16:45:27 -0700 (PDT) X-Google-Smtp-Source: AMsMyM5aBHG8ibxt9z20riMx+WtNkZ/VX9MTpxl21SymFERpAw8L2M7P9VnEJxD+eodqQtPAEgDi X-Received: by 2002:a50:950d:0:b0:463:298a:448c with SMTP id u13-20020a50950d000000b00463298a448cmr21710449eda.153.1667432727275; Wed, 02 Nov 2022 16:45:27 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1667432727; cv=none; d=google.com; s=arc-20160816; b=IkgM3F6DFEzqdpF7GXrcaI6WrcDVWeOOHRawSvsGGI89Z/ZmjZvhjuBOlcKAnApxJS EDho+kJ6nbsCtJAHexu6SaJW1bdEhi5HOAEUnhVFbExMXU9CdURJwkGHXeeNnEWKcxnn /4/ZhugIYnY6DuPsaD1TvgSqAx2+KDi/MJS09ovSqltpS2GTZhmoLto4myyrQnSlAF2A UXkNG7HbzcrrOi7xrfFaNqHbAPhEiIMqz66EZkJFBZx9ap7AYsZi/GIJf0I8itLa6yKL tCPOh6MZNPKyXMtokVFfCe4vmifOUCjdpX6uy63k30l+JERtU+bUZ6abpFtXo9phQW6Z StQw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:from:subject:message-id:references :mime-version:in-reply-to:date:dkim-signature; bh=sdCA7mcVjAUWc0YMUhoHrXJZD+ZYpCbmWTqjMXRG8IY=; b=WmesFkzJxJvX3Pbq3UsR6BJQzI0c8HdhNgvN1URnrxyz0Zy40Dt/gbC0EDRLcVfrIt r9rpmyVvTZdMBI16CdhALzAmqGaTuxj6+z0dSzcvT673SQJOhlLMN7+srj0/HiPcbeDr QZiF64qWCU9m1x2vJIdQiVrZ7KoqSerH8yVPzfFQOJCpwvk54kviV4ytikYp1rUBdF6I fLxPVyEP9XE4Zx61FlE2y2rLHZmeka8DiJtX8O+tLbWhXDmExzz6suGSMTza9ehN6aFu PIoiIZ8HTViZAf6OqZe2qRgnUtbbBYB4R4cafzgLoNuSvJo/ukPMBXctuZLo9gK0TRWe Ugcw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@google.com header.s=20210112 header.b=PGaFr+qG; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id s5-20020a056402520500b0045ca15afe1csi20237684edd.553.2022.11.02.16.45.02; Wed, 02 Nov 2022 16:45:27 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@google.com header.s=20210112 header.b=PGaFr+qG; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231624AbiKBXgy (ORCPT + 99 others); Wed, 2 Nov 2022 19:36:54 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53180 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231396AbiKBXg1 (ORCPT ); Wed, 2 Nov 2022 19:36:27 -0400 Received: from mail-pj1-x1049.google.com (mail-pj1-x1049.google.com [IPv6:2607:f8b0:4864:20::1049]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6181D64E0 for ; Wed, 2 Nov 2022 16:28:25 -0700 (PDT) Received: by mail-pj1-x1049.google.com with SMTP id b1-20020a17090a10c100b0020da29fa5e5so85790pje.2 for ; Wed, 02 Nov 2022 16:28:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:from:to:cc:subject:date:message-id:reply-to; bh=sdCA7mcVjAUWc0YMUhoHrXJZD+ZYpCbmWTqjMXRG8IY=; b=PGaFr+qGgiFZsuqNJ4r0qGcqe0peI7+c4i53VjKhst54w0URdIJKXzZNrE1Ly93vHm ZUbVyccc+uf8FdIO/NLMK+zN19LS2ILfB8gdsaXOgRqmNf5gNf8Sq41ErU5US7FBhDnN IbJDzyfjQaKFpkHkPl3bYwopRjVjLACAqEC0DNB29g1Ge5y9OBvBYWx5FgfCeRj47bTd uP0JA8MOwOO3U1UGREYehl5yuws8MbqIXcCuXKo+lp0r7KUXTVAPpiUkXP49Oc4FpCms bJYwo+JQ3/cJgnCgmhkiynV8huNUX8jXmwm0fx3NQvdXEGtKtu0iZB7J+Ljry5Hp06qq yEJg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=sdCA7mcVjAUWc0YMUhoHrXJZD+ZYpCbmWTqjMXRG8IY=; b=s5z3+ULMvN782tjsjGE35o4jCaaPV5SC6OhzOPyhs3qlNakcaAUoxs8ufwXfKL7J7X vpT41/CY4BL7Uc79sUgiV+PGjBGkFkAMLi1/3MW1u0wBtibaN6+llmTT91oovjjXVIWq YetP3tI3DUVchIw0vtsi6QY7QCtciihQZKpIL95JetJZ6pF3BeTRRTzjH+vQo9EopGZ/ m8DzLetCEZAakIo/rp4KLjAXhAOkIuj8d84sCni1ndRCznEL9xtU7TUXzS3OuWhSGUu2 eeQIpczFqdNBe9j5gXKu+WwYaGQAUwXCA76OQDXXfuiJKy8JDeo9I7ISnYiPzn1BhLlR ue0w== X-Gm-Message-State: ACrzQf0w6U0J7rGHDkOjvBupGlujDXMLiLEYbirCw5HBlax/pHKt4N0J IX87pyaJUH997RSu8+Ka+aQDT7bLfKMo X-Received: from vipin.c.googlers.com ([fda3:e722:ac3:cc00:7f:e700:c0a8:479f]) (user=vipinsh job=sendgmr) by 2002:a17:903:2342:b0:181:bc30:b02f with SMTP id c2-20020a170903234200b00181bc30b02fmr27154262plh.30.1667431664230; Wed, 02 Nov 2022 16:27:44 -0700 (PDT) Date: Wed, 2 Nov 2022 16:27:32 -0700 In-Reply-To: <20221102232737.1351745-1-vipinsh@google.com> Mime-Version: 1.0 References: <20221102232737.1351745-1-vipinsh@google.com> X-Mailer: git-send-email 2.38.1.273.g43a17bfeac-goog Message-ID: <20221102232737.1351745-3-vipinsh@google.com> Subject: [PATCH v8 2/7] KVM: selftests: Put command line options in alphabetical order in dirty_log_perf_test From: Vipin Sharma To: seanjc@google.com, pbonzini@redhat.com, dmatlack@google.com Cc: andrew.jones@linux.dev, wei.w.wang@intel.com, kvm@vger.kernel.org, linux-kernel@vger.kernel.org, Vipin Sharma X-Spam-Status: No, score=-9.6 required=5.0 tests=BAYES_00,DKIMWL_WL_MED, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS,USER_IN_DEF_DKIM_WL autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1748429939600040736?= X-GMAIL-MSGID: =?utf-8?q?1748429939600040736?= There are 13 command line options and they are not in any order. Put them in alphabetical order to make it easy to add new options. No functional change intended. Signed-off-by: Vipin Sharma Reviewed-by: Wei Wang Reviewed-by: Sean Christopherson --- .../selftests/kvm/dirty_log_perf_test.c | 36 ++++++++++--------- 1 file changed, 19 insertions(+), 17 deletions(-) diff --git a/tools/testing/selftests/kvm/dirty_log_perf_test.c b/tools/testing/selftests/kvm/dirty_log_perf_test.c index 56e08da3a87f..5bb6954b2358 100644 --- a/tools/testing/selftests/kvm/dirty_log_perf_test.c +++ b/tools/testing/selftests/kvm/dirty_log_perf_test.c @@ -406,50 +406,52 @@ int main(int argc, char *argv[]) guest_modes_append_default(); - while ((opt = getopt(argc, argv, "eghi:p:m:nb:f:v:os:x:")) != -1) { + while ((opt = getopt(argc, argv, "b:ef:ghi:m:nop:s:v:x:")) != -1) { switch (opt) { + case 'b': + guest_percpu_mem_size = parse_size(optarg); + break; case 'e': /* 'e' is for evil. */ run_vcpus_while_disabling_dirty_logging = true; break; + case 'f': + p.wr_fract = atoi(optarg); + TEST_ASSERT(p.wr_fract >= 1, + "Write fraction cannot be less than one"); + break; case 'g': dirty_log_manual_caps = 0; break; + case 'h': + help(argv[0]); + break; case 'i': p.iterations = atoi(optarg); break; - case 'p': - p.phys_offset = strtoull(optarg, NULL, 0); - break; case 'm': guest_modes_cmdline(optarg); break; case 'n': perf_test_args.nested = true; break; - case 'b': - guest_percpu_mem_size = parse_size(optarg); + case 'o': + p.partition_vcpu_memory_access = false; break; - case 'f': - p.wr_fract = atoi(optarg); - TEST_ASSERT(p.wr_fract >= 1, - "Write fraction cannot be less than one"); + case 'p': + p.phys_offset = strtoull(optarg, NULL, 0); + break; + case 's': + p.backing_src = parse_backing_src_type(optarg); break; case 'v': nr_vcpus = atoi(optarg); TEST_ASSERT(nr_vcpus > 0 && nr_vcpus <= max_vcpus, "Invalid number of vcpus, must be between 1 and %d", max_vcpus); break; - case 'o': - p.partition_vcpu_memory_access = false; - break; - case 's': - p.backing_src = parse_backing_src_type(optarg); - break; case 'x': p.slots = atoi(optarg); break; - case 'h': default: help(argv[0]); break;