From patchwork Fri Sep 6 18:07:22 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Peter Maydell X-Patchwork-Id: 826006 Delivered-To: patch@linaro.org Received: by 2002:adf:a345:0:b0:367:895a:4699 with SMTP id d5csp934752wrb; Fri, 6 Sep 2024 11:08:29 -0700 (PDT) X-Forwarded-Encrypted: i=2; AJvYcCXOH79DpWd7hoxKR/ulawmCwjidTuCiaqRGX2VqPpkqqz2ZaXEGQuzIsWC5DzGPAkkbcvWWsA==@linaro.org X-Google-Smtp-Source: AGHT+IHy8KUwRD4jJ6V020yt72bNoaFGtYOPnR9RttX0iIbsgeQwlv5TJzdVQCLwxVFESWhyMlas X-Received: by 2002:a05:6214:3d9e:b0:6c4:b278:e04f with SMTP id 6a1803df08f44-6c528528254mr46504826d6.44.1725646108918; Fri, 06 Sep 2024 11:08:28 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1725646108; cv=none; d=google.com; s=arc-20240605; b=GMls62WcXhSfjDtXujB3C5Bk7GYW5gcU0yT4yZSEkQf6Ths+q/bmVUhHhWiuovFOiO 8aCQ32M4thV6iiBFL9fQDeO6mJj6S/w2h1gQGhfu2FIOYnfc1fcs0A7bYvZJdSl11Jft 3UhjDpKeCPFKHVVrI/Rq54ThneacewUlROTsxsT3S9iU8XJ2nwJes9sNk4Kvj56NDb2G SC5u1nviPzyZyQjaLrj4dGvjmABDhxaJp0RSLry94pO1/VKk2uS+1FIYu6sEw/QkXd9v LDt3ap0Du7V/kxhdhe8KAxiCzW3+MZ7TGyD2iGA1UVAjQONByJVRhoSn4oCRebAn/Xzm DI7g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20240605; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:content-transfer-encoding :mime-version:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=eLZYP9oZiGwTAwn0IB4n2i9sG23m9F19T+BjlVlZE4E=; fh=1sLZY1WOc/ZrJb9QQy+LKcwsCOPd0n5eRmkkJg5dC2c=; b=bM/hE3S151bio4aQrN+mxgWfPT/3Cd5Hn0x+ay3oBeYpihiasW2gt063Fwk7Z6aZPj AsJVpJhiiuOb75OTE9vInBOLAIQMgSBovVq1xD/tE1muZxs/CcpWFvJuZY/M+Dsquqwg SKS0SruxVTj6quhwjL8HttUtuBwv8seD/n1y6YKMX4UbW5ddxm83VB7SEate+ezuC+MI +cCj7K6dOwmZkOMo5b/kGeXL7Thqgvp4Y9fp/fql86WA2Z6LyxeXrLojAgiZFbAanWbw 9o7+eTx94iRg04MEmHot9apc2HeCy8N2tY5XX0D3RGyMRhFjBVR1g3Hb/KEcq10f6lcC +Mxg==; dara=google.com ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=wet083Pl; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org; dara=neutral header.i=@linaro.org Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id 6a1803df08f44-6c5306ae3c7si8116286d6.564.2024.09.06.11.08.28 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Fri, 06 Sep 2024 11:08:28 -0700 (PDT) Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=wet083Pl; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org; dara=neutral header.i=@linaro.org Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1smdN9-0002Fy-Nv; Fri, 06 Sep 2024 14:07:52 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1smdMp-0001id-2o for qemu-devel@nongnu.org; Fri, 06 Sep 2024 14:07:34 -0400 Received: from mail-wr1-x435.google.com ([2a00:1450:4864:20::435]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1smdMm-000563-S2 for qemu-devel@nongnu.org; Fri, 06 Sep 2024 14:07:30 -0400 Received: by mail-wr1-x435.google.com with SMTP id ffacd0b85a97d-374c1e5fe79so1429262f8f.1 for ; Fri, 06 Sep 2024 11:07:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1725646046; x=1726250846; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=eLZYP9oZiGwTAwn0IB4n2i9sG23m9F19T+BjlVlZE4E=; b=wet083Pls+8muELKY+MIOyL6uy73mETXvfXhwOJcY9tZ81T0nRpFRf1WbxLdaHCyKI LQH0JwNrYBaicEMR5NfaYT0r33boKcKlJO+rIi2uOh1DB/AkOaqIq+R5K30iRgX+RHG2 HcgtROxEsS2X4dscarBgzLl0wk4gDxfnHDpJPqUdmuej+u3Ftadf866X4QXuv7f+rTZ4 whd5zbPES0T3MN8DnY+Fy61XivdCY22S7ZqREgw41B60eGUAgPX60aNiaCSdE3cgR02A CZKH4zWyDrdieyr1sNlCAgtYqluOzEtqnDBsEUrVJ4ka5GjWWcjhiomQwsLpoOAyhAHA VXSA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1725646046; x=1726250846; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=eLZYP9oZiGwTAwn0IB4n2i9sG23m9F19T+BjlVlZE4E=; b=bp8edsoUO83Q4/36SCHBUIvqu8flfZyjOGJAQcZasojW7JJhFmiTU+YGEY44Vk5BfI Dmay/0DD900zPo23zzxsAuV3qNCmVusNZzJ0dUvEmDA27AhPSxkHzS6W3MfVlVZQbWYB J9QXtdNTkgGWCDD5CTKYP8I3RxfPwXDmNdfTcHb6ZAvpkrmMO7gI6ad0jAGPM84W/W8v LUUMt6ZFVl8bC0Kqa5T6Azb/eaOdQtAZq6hrk5ZWb3hpFNHGGPHeaB8pWp8PVx+bolO0 vLFx6f7vFDzzafDJxDe5iR2lILaaV5Lr1o9AnIOQwOa0v2I0d6uELoh0hBnT3NW3QJqI Gb/Q== X-Gm-Message-State: AOJu0YyhyPm33BuSDtSz5wUWi6rFiEF5z926oFqbgIePOK7TNF7U838x yvcH7QyhumhxFCF4S5RLuOb/uKnsTfCOLD5lJzBlNnmPGsq2ZNtTMh4J/SC4W+JgWqNlHBTqpVa k X-Received: by 2002:a5d:670a:0:b0:374:c61a:69b8 with SMTP id ffacd0b85a97d-378895c9439mr2267106f8f.15.1725646046005; Fri, 06 Sep 2024 11:07:26 -0700 (PDT) Received: from orth.archaic.org.uk (orth.archaic.org.uk. [2001:8b0:1d0::2]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-37891657a2csm441860f8f.95.2024.09.06.11.07.25 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 06 Sep 2024 11:07:25 -0700 (PDT) From: Peter Maydell To: qemu-devel@nongnu.org Cc: Paolo Bonzini , Thomas Huth , =?utf-8?q?Alex_Benn=C3=A9e?= , =?utf-8?q?Philippe_M?= =?utf-8?q?athieu-Daud=C3=A9?= Subject: [PATCH 1/2] scripts/mtest2make: Honour MESON_TESTTHREADS if it is set Date: Fri, 6 Sep 2024 19:07:22 +0100 Message-Id: <20240906180723.503855-2-peter.maydell@linaro.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20240906180723.503855-1-peter.maydell@linaro.org> References: <20240906180723.503855-1-peter.maydell@linaro.org> MIME-Version: 1.0 Received-SPF: pass client-ip=2a00:1450:4864:20::435; envelope-from=peter.maydell@linaro.org; helo=mail-wr1-x435.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: qemu-devel-bounces+patch=linaro.org@nongnu.org 'meson test' has two ways to set how many tests to run in parallel: * the command line --num-processes option * the environment variable MESON_TESTTHREADS If both are given, then it prefers the command line option over the environment variable. This is an unfortunate order of precedence, because it's frequently handy to be able to override the default by setting an environment variable at the top level of a build or CI invocation. Fortunately, we can flip the order for our purposes, because we always invoke 'meson test' indirectly. We currently invoke 'meson test' passing it a --num-processes argument based on whatever the '-j' option to make was. Adjust the logic in mtest2make so that if MESON_TESTTHREADS is set then we honour that, and don't override it. Signed-off-by: Peter Maydell --- scripts/mtest2make.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/scripts/mtest2make.py b/scripts/mtest2make.py index eb01a05ddbd..6de0044134c 100644 --- a/scripts/mtest2make.py +++ b/scripts/mtest2make.py @@ -32,7 +32,11 @@ def names(self, base): ifneq ($(SPEED), quick) .mtestargs += --setup $(SPEED) endif +# If the user set MESON_TESTTHREADS then don't override that preference by +# passing --num-processes to 'meson test' +ifeq ($(MESON_TESTTHREADS),) .mtestargs += $(subst -j,--num-processes , $(filter-out -j, $(lastword -j1 $(filter -j%, $(MAKEFLAGS))))) +endif .check.mtestargs = $(MTESTARGS) $(.mtestargs) $(if $(V),--verbose,--print-errorlogs) .bench.mtestargs = $(MTESTARGS) $(.mtestargs) --benchmark --verbose''')