From nobody Mon Sep 15 16:18:31 2025 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0EB6FC46467 for ; Wed, 11 Jan 2023 18:34:17 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229511AbjAKSeP (ORCPT ); Wed, 11 Jan 2023 13:34:15 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39542 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229623AbjAKSeM (ORCPT ); Wed, 11 Jan 2023 13:34:12 -0500 Received: from mail-yb1-xb49.google.com (mail-yb1-xb49.google.com [IPv6:2607:f8b0:4864:20::b49]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 64C9713D2A for ; Wed, 11 Jan 2023 10:34:11 -0800 (PST) Received: by mail-yb1-xb49.google.com with SMTP id k18-20020a25e812000000b0077cc9ab9dd9so16828673ybd.8 for ; Wed, 11 Jan 2023 10:34:11 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=cc:to:from:subject:message-id:mime-version:date:from:to:cc:subject :date:message-id:reply-to; bh=9KIwZTDSAmIppW3arQta89dBIMqNXF0skn62yduf4ZQ=; b=kV0DACr/rMJL2hpaEWN48WWzxdmEyQ0AC0c60/Ewbaxmpiq5TIs95p82ilfqdeC37y B2oI9ku51dzp4rQOImG6HlFD6HTrsAAF5NGk0kfv0Kahgs8FBxJzFLyoDao2N/SI4xRX d+zEQR3NkU9Vz77KEfcWoOW4RAjpHLaXicTtxJx8XxgYdrYZMJ46bxZInqvJh58NxSeV ntPbtTHhqOhYrVFkZw0ZygJw7NQWraxPLUPxTAFLd2X529uP5NiFygcpoDj9AxwiSMiw f8KjApR0+QuglrZZHB4zG65nPI9sa7DJfcPOjgFq2mtTLkxAslwKct+GeOUe44gD7XPV yqww== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:from:subject:message-id:mime-version:date:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=9KIwZTDSAmIppW3arQta89dBIMqNXF0skn62yduf4ZQ=; b=e0yGkJNzmAEUGQVF/9eu+Iy5vipNIztXb9MGNzqI9etYFCovhhGrtda+j3sT23qTcf YduYNQ6AucbCX/qzdI+R9m1gAe+XAeivIDqT1KfTyPlcIYpMcGjEVhUjUri+nktkO6Zp HSzVurQgaJGJYqi4dVkZ91IxaFYIb53g+Rvg3N+qXuXDnpvFnCoVVPnPJkCnk09hXYjd iO7HqNdGq9Zxq61YnFSaCTUnl0lOMwLKn+Zx02bqmzTN9j4JuQsd0Rwa0l3HulaVO1MO oe5XB3zJ2OvOA98GwLfUmesc9mYRZyv+QoYO1Anun6NWLkcDGL+lhvR0Yl008rQvF8p3 Ka/Q== X-Gm-Message-State: AFqh2kra5kEX8iVqr/mWkwEn6aAu9ss7L8zRgvT4VTpHP+5p6YwsYWLy KFaD8kD8C5TaAsUmtbVdCxoOpLYBJ1D1 X-Google-Smtp-Source: AMrXdXvQBrek+QKvBaZd+mrabbLY7khNP7e9pQGNB6+CMZGEjPvfFHANzOZ3Aj+gMUE4UqCdKSimwoCRc0fK X-Received: from vipin.c.googlers.com ([fda3:e722:ac3:cc00:7f:e700:c0a8:479f]) (user=vipinsh job=sendgmr) by 2002:a25:cecb:0:b0:7c1:3a0:e48d with SMTP id x194-20020a25cecb000000b007c103a0e48dmr388967ybe.183.1673462050697; Wed, 11 Jan 2023 10:34:10 -0800 (PST) Date: Wed, 11 Jan 2023 10:34:08 -0800 Mime-Version: 1.0 X-Mailer: git-send-email 2.39.0.314.g84b9a713c41-goog Message-ID: <20230111183408.104491-1-vipinsh@google.com> Subject: [Patch v2] KVM: selftests: Make reclaim_period_ms input always be positive From: Vipin Sharma To: seanjc@google.com, pbonzini@redhat.com, bgardon@google.com Cc: kvm@vger.kernel.org, linux-kernel@vger.kernel.org, Vipin Sharma Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" reclaim_period_ms use to be positive only but the commit 0001725d0f9b ("KVM: selftests: Add atoi_positive() and atoi_non_negative() for input validation") incorrectly changed it to non-negative validation. Change validation to allow only positive input. Fixes: 0001725d0f9b ("KVM: selftests: Add atoi_positive() and atoi_non_nega= tive() for input validation") Signed-off-by: Vipin Sharma Reported-by: Ben Gardon Reviewed-by: Ben Gardon Reviewed-by: Sean Christopherson --- v2: - Added Fixes and Reviewed-by tags v1: https://lore.kernel.org/lkml/20230110215203.4144627-1-vipinsh@google.co= m/ tools/testing/selftests/kvm/x86_64/nx_huge_pages_test.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/testing/selftests/kvm/x86_64/nx_huge_pages_test.c b/tool= s/testing/selftests/kvm/x86_64/nx_huge_pages_test.c index ea0978f22db8..251794f83719 100644 --- a/tools/testing/selftests/kvm/x86_64/nx_huge_pages_test.c +++ b/tools/testing/selftests/kvm/x86_64/nx_huge_pages_test.c @@ -241,7 +241,7 @@ int main(int argc, char **argv) while ((opt =3D getopt(argc, argv, "hp:t:r")) !=3D -1) { switch (opt) { case 'p': - reclaim_period_ms =3D atoi_non_negative("Reclaim period", optarg); + reclaim_period_ms =3D atoi_positive("Reclaim period", optarg); break; case 't': token =3D atoi_paranoid(optarg); --=20 2.39.0.314.g84b9a713c41-goog