From nobody Mon Sep 15 18:08:19 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 AAD2FC46467 for ; Tue, 10 Jan 2023 21:52:28 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230259AbjAJVw1 (ORCPT ); Tue, 10 Jan 2023 16:52:27 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52928 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231654AbjAJVwY (ORCPT ); Tue, 10 Jan 2023 16:52:24 -0500 Received: from mail-pl1-x649.google.com (mail-pl1-x649.google.com [IPv6:2607:f8b0:4864:20::649]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5018D4435F for ; Tue, 10 Jan 2023 13:52:23 -0800 (PST) Received: by mail-pl1-x649.google.com with SMTP id u2-20020a17090341c200b00192bc565119so9213691ple.16 for ; Tue, 10 Jan 2023 13:52:23 -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=l82ntIFd7MfJAb7UiGfLmOAfwMt4TTGXS4HxL44y3uE=; b=Uz9zoI5PIuJEA3ZBY0NKTCrxWY1gEr3XrAkrjbc0mNIWWEKtyjhjkJbdAWwFHsA5mz 42BBPxGHZOuY1lZXGJ8yo7K4veyW+s+7BDy00DwsX81abu/bJLJvQTyDLFNksvvnqHrY Ss0cd47ymETSaSkxU2VtFh/UunHB1akrwCg4KcTsRV+2KAcpx45/1Y45D5+bQX2Oi+Ew DDiPDHUQINNAXGRYONFlkS8VHXXSzrHUhusni7eGl2JPO9fpeGLmsKNNdgV/gDwacsWh LFXCZ0NI7yjO/A5Kbrs8iQUInwWmRt3pU/ufQNBFCS03eoIWa8DmPo6pKlGhZ4/5MxtT WA4A== 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=l82ntIFd7MfJAb7UiGfLmOAfwMt4TTGXS4HxL44y3uE=; b=3NksegOAkK7+Qk68WKiTG08o9tWIl63bLXIuFrkp0SaHtLLpZoVGw9Ka/s8oc2iBwo 1FRj9mak+xIo2CYZ43YLEAMx1oJQmeqWkcTo5rk69KYISDr9JmtYAwYO4MrqSCJVO9nQ u5Ul0GdN9cLuAWlnINaN7yV0pfsRst0Uw05i6jhjA/vVl2IWGQPY344NBAG+wKB7YBpt DOIPHl8hPIO1izbPUSvU2qY93Lt/6MWv3cIXrEIHlZK8ez0XvpRRRY6JxK3QEU788uVO FdzARIMQ2R79rzjv4k25OR2rA92+wUf2HadIZFHrf0M6VeL4NbPraePj/6d8JKWy1qLO r/qQ== X-Gm-Message-State: AFqh2kpB39h2A7ZxWTZdZBTP2SCn9h44Wlp/JTf0XQQ2L61OjysaQzLj fKRx6i1m91aRrZdCHb56ZloaifO5mb9/ X-Google-Smtp-Source: AMrXdXstAIFHK82xxA5xKyMtQGZenpxd6q1WUBPtqRilf0ydCgzQse3nEmc5uf3aIs86K21xX17SjCa3bzB9 X-Received: from vipin.c.googlers.com ([fda3:e722:ac3:cc00:7f:e700:c0a8:479f]) (user=vipinsh job=sendgmr) by 2002:aa7:82cb:0:b0:578:3ec0:f264 with SMTP id f11-20020aa782cb000000b005783ec0f264mr5821763pfn.22.1673387542828; Tue, 10 Jan 2023 13:52:22 -0800 (PST) Date: Tue, 10 Jan 2023 13:52:03 -0800 Mime-Version: 1.0 X-Mailer: git-send-email 2.39.0.314.g84b9a713c41-goog Message-ID: <20230110215203.4144627-1-vipinsh@google.com> Subject: [Patch] 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 used 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. Signed-off-by: Vipin Sharma Reported-by: Ben Gardon Reviewed-by: Ben Gardon --- 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