From patchwork Tue Nov 8 09:23:00 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Alex_Benn=C3=A9e?= X-Patchwork-Id: 622641 Delivered-To: patch@linaro.org Received: by 2002:a17:522:c983:b0:460:3032:e3c4 with SMTP id kr3csp2806041pvb; Tue, 8 Nov 2022 01:24:26 -0800 (PST) X-Google-Smtp-Source: AMsMyM5Jcrkr2/SU2q9Klpk8RZD5RnU6kS5CN8pSpXf/Uplh2lEDu6JKbQsjtFLSia9tVI/d99Ur X-Received: by 2002:a05:6214:2528:b0:4b8:2025:5f6e with SMTP id gg8-20020a056214252800b004b820255f6emr48488045qvb.88.1667899466788; Tue, 08 Nov 2022 01:24:26 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1667899466; cv=none; d=google.com; s=arc-20160816; b=qhCJCtteOYwTwMKYSKQKZCCRyUkR65EgziSVOqA6OSdjjhuTeZxV6OYSKowJR3lb6o pt2AjPjOSwx/JiW25twxiqi1Ly86mCQYY25ufpHpe9MRmzcD7bmuXliHUgI5AbwPnT1N FOc5eM+zzfr+S1IkGJX8l8Q2I929hwhSICBEkRn+BJ/JTTE1fugE3S7OCVdKi6bNQKe+ zAXcml52pgg1E40cNah5JbYEivLcunnHJ+1REMFBiaB0FPvObfNSFjZ/tv765OlUSTY0 Jy4H2M4JIPsyVjpXBgcn4gARYLVoeRcREGW5N5DrIGpBRieGerzZhhv93dFrI1BfCiuw GszQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; 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=XsIUPL4URppAX7w6NGrUdBT/BUmCRhwxzbv6AwfzKCA=; b=0wIb2v6yGgo+zpzKtIeHhJrMwBtCPjsBQd9LlkQmtt5YyOzFDTRARrgjqIMjTAk7ih /5Af85F4CQbYn1l6rUf3vObjl5EfJfopyL6xgYhz93vlrQc1zPEsjgT7FSLLX3w3v7Aj Z1JqHcdUnQwH+zTOVsBCgzTtpLXZaJZWWxJhsNceMyDXabikqdT2PbXJHffYDD4U15Mv Ovkoj95gHVKQmhm2b+4MG6esErEvRHkD2uI1/6OHT16RvZbdOWr5UA8kghJqQ3ezRoPf te8N5n4BeOGn6SBmofXDgyNprlQKcHhnEUjQeIhBRWliOlZUXjD4m4zdkxxzUstPzhHK Jmpw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=W5CvJjaZ; 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 Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id e12-20020a0562140d8c00b004bb812cbf27si6002895qve.64.2022.11.08.01.24.26 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Tue, 08 Nov 2022 01:24:26 -0800 (PST) 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=W5CvJjaZ; 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 Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1osKpC-0007Da-Ar; Tue, 08 Nov 2022 04:23:18 -0500 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 1osKp7-0007CK-5X for qemu-devel@nongnu.org; Tue, 08 Nov 2022 04:23:13 -0500 Received: from mail-wm1-x32f.google.com ([2a00:1450:4864:20::32f]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1osKp4-0005mb-VR for qemu-devel@nongnu.org; Tue, 08 Nov 2022 04:23:12 -0500 Received: by mail-wm1-x32f.google.com with SMTP id m7-20020a05600c090700b003cf8a105d9eso8650722wmp.5 for ; Tue, 08 Nov 2022 01:23:10 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; 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=XsIUPL4URppAX7w6NGrUdBT/BUmCRhwxzbv6AwfzKCA=; b=W5CvJjaZeWufGA+5W/omDak7/5f8CoqAIkChf0pWDiIq05dkeNceeKUEtRJCdYB7FN N3pAUJyRsOr3E+dRfcwsxblg9SWi6j7+EWxKUFRnonQ1xOQ0u6Kt7SS0JHvcD1jrxluq n5uTWcaInCx1zopeeqULNVcpOnErP8Ng01922eA+fskQsIZhY7dg20SlYehVMdeE7OVw O3nR7BYItRRgTrwRdwatYXjNYLa1wGfiPAndWHzsfDOy2QnNUECVlxYwpK47Wz1xHgjr i3AtjX6jZ7lIwQb95ms4Zakfy6UcopoItVnjzwXJZrVXWfrtsgOgeN6E1wWdOcOiUJVH Ig9w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; 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=XsIUPL4URppAX7w6NGrUdBT/BUmCRhwxzbv6AwfzKCA=; b=e/orVsSXxXsquZ2Bg4aJpIm9Zzc4+Hl8z0dIZhPVFTC4RQSb6eDcs8np/Xm6RMq6tl PJadb3kz6bHzW6m65i/vSqljOskOppNEonCTdd/y55KGHivW5plfNyv4VPYOYJszlReb 136+MTKny6fedLhX+zj/VD737/IbxZrNO8JpQ7SSncl06VGWqv8UE/YsXWBWC9s/EPhY zGYz57KbmHAD50OJYZPSGa/aWiUCLqZqmw7Q/FqoO7k3lX9f3e9UVyO6aYYdr6RMbTgc INAS3qB2AISWkKsnG5Za5cBNx6IfXRz8tSHUgcAe+TcRee5gU0MBZ9Da5VSElR/p2WtT R5XQ== X-Gm-Message-State: ACrzQf0Js0EWcFOlv21yaMTiGEwfE9Mt680teRd8GzIkJA1fVu3VEPeD 3o0Dvg5R/8yp/lwv4o6PTgKabQ== X-Received: by 2002:a05:600c:4e45:b0:3cf:3e69:9351 with SMTP id e5-20020a05600c4e4500b003cf3e699351mr47101397wmq.2.1667899389460; Tue, 08 Nov 2022 01:23:09 -0800 (PST) Received: from zen.linaroharston ([185.81.254.11]) by smtp.gmail.com with ESMTPSA id c2-20020a5d4f02000000b002366553eca7sm9519221wru.83.2022.11.08.01.23.08 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 08 Nov 2022 01:23:08 -0800 (PST) Received: from zen.lan (localhost [127.0.0.1]) by zen.linaroharston (Postfix) with ESMTP id 3CE1F1FFB8; Tue, 8 Nov 2022 09:23:08 +0000 (GMT) From: =?utf-8?q?Alex_Benn=C3=A9e?= To: qemu-devel@nongnu.org Cc: fam@euphon.net, berrange@redhat.com, f4bug@amsat.org, aurelien@aurel32.net, pbonzini@redhat.com, stefanha@redhat.com, crosa@redhat.com, Stefan Weil , =?utf-8?q?Alex_Benn=C3=A9e?= , Thomas Huth Subject: [PATCH v1 1/9] Run docker probe only if docker or podman are available Date: Tue, 8 Nov 2022 09:23:00 +0000 Message-Id: <20221108092308.1717426-2-alex.bennee@linaro.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20221108092308.1717426-1-alex.bennee@linaro.org> References: <20221108092308.1717426-1-alex.bennee@linaro.org> MIME-Version: 1.0 Received-SPF: pass client-ip=2a00:1450:4864:20::32f; envelope-from=alex.bennee@linaro.org; helo=mail-wm1-x32f.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 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 From: Stefan Weil The docker probe uses "sudo -n" which can cause an e-mail with a security warning each time when configure is run. Therefore run docker probe only if either docker or podman are available. That avoids the problematic "sudo -n" on build environments which have neither docker nor podman installed. Fixes: c4575b59155e2e00 ("configure: store container engine in config-host.mak") Signed-off-by: Stefan Weil Message-Id: <20221030083510.310584-1-sw@weilnetz.de> Signed-off-by: Alex Bennée --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 66928692b0..26c7bc5154 100755 --- a/configure +++ b/configure @@ -1780,7 +1780,7 @@ fi # functions to probe cross compilers container="no" -if test $use_containers = "yes"; then +if test $use_containers = "yes" && (has "docker" || has "podman"); then case $($python "$source_path"/tests/docker/docker.py probe) in *docker) container=docker ;; podman) container=podman ;; From patchwork Tue Nov 8 09:23:01 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Alex_Benn=C3=A9e?= X-Patchwork-Id: 622640 Delivered-To: patch@linaro.org Received: by 2002:a17:522:c983:b0:460:3032:e3c4 with SMTP id kr3csp2805841pvb; Tue, 8 Nov 2022 01:24:02 -0800 (PST) X-Google-Smtp-Source: AMsMyM4cBCwXZIFB9Kt2rt6kcRTRNvy5M/NYcJdAJNrR5yabFmbNEwcwn1PE44OqUGslRwm5rnME X-Received: by 2002:a05:620a:294f:b0:6cf:920e:4f3a with SMTP id n15-20020a05620a294f00b006cf920e4f3amr38453098qkp.66.1667899442563; Tue, 08 Nov 2022 01:24:02 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1667899442; cv=none; d=google.com; s=arc-20160816; b=eDy9p8gE/zTLVcySHp6MP2lbcau4MIWplaQZ0myQPKzKPDJHynHmPYeuD4KL6pXcXK dEuGc3vxyCL5NKeQciEqjAJiT1K2MzJADspkMDFcdKxC2/mhFE+pl0IKicp9rG/lqXyk 1oQ+3ERA1rGwkw5fKMvwnyPmPYewAqMfaDFieWI2IDKg8JGn+PVe5OQAk+9z3z1wnEMh fqFkYqd/eIorakVw6w8Yz62N0HhbUdOYMUxRrxGAO6qvAOqzqGiuGOShkzxr9+NtOefW P9dJuwnOKzabp2jFJvDln591hMkNaXr4Nrqwxl7yGjNsM3IM2jR/LYQdcEVLYCE2qlcj wN8w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; 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=uigyf6qq9Am7cRVFYV/7zGs4tD3SV3rZRcR/12kbTNw=; b=ZxIap3QAhgujf7drkD+mm5Lb/hCsz1fFJmolnqt3LmEXTJUE31NC8F3sCCn4EjD+RE vgiTlZUBYPAtxHQITwN/GxoEO9mSTeH1L/X07aBuk/RrayTkiDPmSDuXquU5DMR6xVK+ XMWB65+OKJySwMFWDfdm3AZvBgRi1+J+8Ztl5bfyqWXrJL/OtJyIgK9MDmwB7ynKFAmp jmzN17TPk1iaMx6rUOfAYKYvNAtqMmTeMO0lGojet0GqOo7OyQZIJrRauAdp5TWdyM5e XrDPkl8StdyqdygZpCe/Fq0iLPdHcFRI6b88ZlZfloCsqBPMxTgAD6dtcwXyRh9Cpxig meHA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=gMVXpVuw; 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 Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id bs43-20020a05620a472b00b006fa2e7dc051si6283000qkb.482.2022.11.08.01.24.02 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Tue, 08 Nov 2022 01:24:02 -0800 (PST) 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=gMVXpVuw; 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 Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1osKpF-0007EO-Nf; Tue, 08 Nov 2022 04:23:21 -0500 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 1osKp9-0007Cx-GL for qemu-devel@nongnu.org; Tue, 08 Nov 2022 04:23:16 -0500 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 1osKp6-0005oA-9l for qemu-devel@nongnu.org; Tue, 08 Nov 2022 04:23:15 -0500 Received: by mail-wr1-x435.google.com with SMTP id l14so19990880wrw.2 for ; Tue, 08 Nov 2022 01:23:11 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; 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=uigyf6qq9Am7cRVFYV/7zGs4tD3SV3rZRcR/12kbTNw=; b=gMVXpVuwCKTklzwH67pRe0XXCetjVfuv/THCWKn8zhrIJXv6T3rrflI0NRVuy+dbFV ainMGfNgtL6qEnDWLiUdEf1eXS78Oc3JQpjd5QYklFsfmJ0sLj6UqtGpP/DhpGnJWUsf wSKjBASh56XrOpRGD/W3/LP/SOzMS5vTPTX9jedtN3RgDFglVY94G5Okx4jkdxkslDqD G5+zijTfy93EFleRpTwt57PY6CRYGy+7F8NNKQb8Ut4ZNLNNts/QU4bWA/Xl679NrG1f ni4aNMNlI1O43bRmRlfLZ5rqC3muUpRu+cjb2BdaU8odP2ZkAV2oOX8attLnOlbp2RwI Kspg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; 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=uigyf6qq9Am7cRVFYV/7zGs4tD3SV3rZRcR/12kbTNw=; b=ueEPZCWjSggFGqYOFg7wRMGkw8dliE8mthxBKn9oBWIWhRLdZzg9F4B9LOzWaZJcJC IlfcvtofpGurHEd84DODLK7pwrwLFauKRVfd3LnZxZZPOKUNsmrs245UScmOyc70xHAn ZjhbNl3+sAVeJ3cQjU7O7wmRhIi0qeXmHBChFdVN/IZR3r18SU29Hh9wY1jpPBsQhnIo qTvLqyoeWSz52dLulhdHlIVchAG5U1e/AaZakQk1TN0CPqsvE9vmVJ/OdzXG0hRYJoep fd/ji2Nu7QVCwF1G7yGmqbeG54QhYEhiTEVsqWbmwacAECFTnYTZ1Q2wwZ0ndb+1yW7H 7+iw== X-Gm-Message-State: ACrzQf1J34mYk/L4RcDdNRTbWFXYjbmxqWKr3VdrsXjx+ZR3B25PZLm2 3yUi2H4t/BG6M0qzly6qLL9+bQ== X-Received: by 2002:adf:e40e:0:b0:236:b896:8f55 with SMTP id g14-20020adfe40e000000b00236b8968f55mr32824547wrm.281.1667899390697; Tue, 08 Nov 2022 01:23:10 -0800 (PST) Received: from zen.linaroharston ([185.81.254.11]) by smtp.gmail.com with ESMTPSA id fc15-20020a05600c524f00b003cf57329221sm15459328wmb.14.2022.11.08.01.23.08 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 08 Nov 2022 01:23:09 -0800 (PST) Received: from zen.lan (localhost [127.0.0.1]) by zen.linaroharston (Postfix) with ESMTP id 5434D1FFBA; Tue, 8 Nov 2022 09:23:08 +0000 (GMT) From: =?utf-8?q?Alex_Benn=C3=A9e?= To: qemu-devel@nongnu.org Cc: fam@euphon.net, berrange@redhat.com, f4bug@amsat.org, aurelien@aurel32.net, pbonzini@redhat.com, stefanha@redhat.com, crosa@redhat.com, =?utf-8?q?Alex_Benn=C3=A9e?= , =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= , Wainer dos Santos Moschetta , Beraldo Leal Subject: [PATCH v1 2/9] tests/avocado: improve behaviour waiting for login prompts Date: Tue, 8 Nov 2022 09:23:01 +0000 Message-Id: <20221108092308.1717426-3-alex.bennee@linaro.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20221108092308.1717426-1-alex.bennee@linaro.org> References: <20221108092308.1717426-1-alex.bennee@linaro.org> MIME-Version: 1.0 Received-SPF: pass client-ip=2a00:1450:4864:20::435; envelope-from=alex.bennee@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 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 This attempts to deal with the problem of login prompts not being guaranteed to be terminated with a newline. The solution to this is to peek at the incoming data looking to see if we see an up-coming match before we fall back to the old readline() logic. The reason to mostly rely on readline is because I am occasionally seeing the peek stalling despite data being there. This seems kinda hacky and gross so I'm open to alternative approaches and cleaner python code. Signed-off-by: Alex Bennée Reviewed-by: Philippe Mathieu-Daudé --- tests/avocado/avocado_qemu/__init__.py | 89 +++++++++++++++++++++++++- 1 file changed, 88 insertions(+), 1 deletion(-) diff --git a/tests/avocado/avocado_qemu/__init__.py b/tests/avocado/avocado_qemu/__init__.py index 910f3ba1ea..d6ff68e171 100644 --- a/tests/avocado/avocado_qemu/__init__.py +++ b/tests/avocado/avocado_qemu/__init__.py @@ -131,6 +131,58 @@ def pick_default_qemu_bin(bin_prefix='qemu-system-', arch=None): return path return None +def _peek_ahead(console, min_match, success_message): + """ + peek ahead in the console stream keeping an eye out for the + success message. + + Returns some message to process or None, indicating the normal + readline should occur. + """ + console_logger = logging.getLogger('console') + peek_len = 0 + retries = 0 + + while True: + try: + old_peek_len = peek_len + peek_ahead = console.peek(min_match).decode() + peek_len = len(peek_ahead) + + # if we get stuck too long lets just fallback to readline + if peek_len <= old_peek_len: + retries += 1 + if retries > 10: + return None + + # if we see a newline in the peek we can let safely bail + # and let the normal readline() deal with it + if peek_ahead.endswith(('\n', '\r', '\r\n')): + return None + + # if we haven't seen enough for the whole message but the + # first part matches lets just loop again + if len(peek_ahead) < min_match and \ + success_message[:peek_len] in peek_ahead: + continue + + # if we see the whole success_message we are done, consume + # it and pass back so we can exit to the user + if success_message in peek_ahead: + return console.read(peek_len).decode() + + # of course if we've seen enough then this line probably + # doesn't contain what we are looking for, fallback + if peek_len > min_match: + return None + + except UnicodeDecodeError: + console_logger.log("error in decode of peek") + return None + + # we should never get here + return None + def _console_interaction(test, success_message, failure_message, send_string, keep_sending=False, vm=None): @@ -139,17 +191,52 @@ def _console_interaction(test, success_message, failure_message, vm = test.vm console = vm.console_socket.makefile(mode='rb', encoding='utf-8') console_logger = logging.getLogger('console') + + # Establish the minimum number of bytes we would need to + # potentially match against success_message + if success_message is not None: + min_match = len(success_message) + else: + min_match = 0 + + console_logger.debug("looking for %d:(%s), sending %s (always=%s)", + min_match, success_message, send_string, keep_sending) + while True: + msg = None + + # First send our string, optionally repeating the send next + # cycle. if send_string: vm.console_socket.sendall(send_string.encode()) if not keep_sending: send_string = None # send only once + + # If the console has no data to read we briefly + # sleep before continuing. + if not console.readable(): + time.sleep(0.1) + continue + try: - msg = console.readline().decode().strip() + + # First we shall peek ahead for a potential match to cover waiting + # for lines without any newlines. + if min_match > 0: + msg = _peek_ahead(console, min_match, success_message) + + # otherwise we block here for a full line + if not msg: + msg = console.readline().decode().strip() + except UnicodeDecodeError: + console_logger.debug("skipped unicode error") msg = None + + # if nothing came out we continue and try again if not msg: continue + console_logger.debug(msg) if success_message is None or success_message in msg: break From patchwork Tue Nov 8 09:23:02 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Alex_Benn=C3=A9e?= X-Patchwork-Id: 622642 Delivered-To: patch@linaro.org Received: by 2002:a17:522:c983:b0:460:3032:e3c4 with SMTP id kr3csp2806137pvb; Tue, 8 Nov 2022 01:24:42 -0800 (PST) X-Google-Smtp-Source: AMsMyM4Y8KB1uZMw6VbW2owIU0hdZWNgEoY+eMIT2eK94S+zB4Z72uMxI6o9ViKqNxkK9/SiJZ+e X-Received: by 2002:a05:6214:2a8f:b0:4bb:6c0d:f2e4 with SMTP id jr15-20020a0562142a8f00b004bb6c0df2e4mr47705767qvb.18.1667899482464; Tue, 08 Nov 2022 01:24:42 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1667899482; cv=none; d=google.com; s=arc-20160816; b=n4C7Sssu46iva/1iAu6sWlBE6mJqDzxbyx40eujBKZQrbt4S+gkmTOgwzxjuVWxcl+ R/GDKEWWGCaEnIb6T8tEIy/2B1WhtTs4BYgLqtsp9pWdqKE09vcPEz6mMpS9Cl4O23bZ 1xLAylzdosh6E5N7J/uBux6UMiaCulJDv/mwENo92wIlnqH+EVJCOoCXVbcsRiPUy9B8 lUs5RKork03OZ3e20nhuhOJVkTzR72k4uglmWR1DLrCI/Urp5cEYIo1YI0QHPqejXQzf lobcrjjMn5SevecqyvSwDDPPiMEK9+2CRVOFHbUCgQOyQLOJuQYW37KYB3Hs3adOK1wg dRMQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; 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=qBC/zqhe+w8c2MXv32aYnBZCDh2hCBt3ueKfoisA7Og=; b=pFcnz1ht21WI2yxAEYiNPP/VxZPfPUScBzK7K+tdbkNJHJt/L1bTp2dccLvLkaFfDR 55CqvTt7iZGqRyZb7gzt9S/pO7Ac32jsMn4nsBQrkepdsTMdvVQm4257Bj5OMTP2p15e yoO5kg5lGDV9yvNRqHU64C72M+rtfnNIsrabzmnKOu4HSy9EgDvuWqPiO4Uyk6Jh1c2p 0KZYQORjtw58ic5Jrk/MZAw2WD463n5MSL7fktaaacD30tBsVd9tdMjEkjSijwjGZ0PT v79TyWoWkA/27FhA6XrH1ZrLW8d2SFSRsUXjpIeuQ9gjACdGYmEFsnbds1hBzRbprwmO eQvg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=mo3TP1aG; 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 Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id u7-20020a05622a17c700b0038c17e556ccsi779262qtk.394.2022.11.08.01.24.42 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Tue, 08 Nov 2022 01:24:42 -0800 (PST) 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=mo3TP1aG; 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 Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1osKpW-0007No-Gz; Tue, 08 Nov 2022 04:23:38 -0500 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 1osKpJ-0007Fk-Po for qemu-devel@nongnu.org; Tue, 08 Nov 2022 04:23:25 -0500 Received: from mail-wm1-x333.google.com ([2a00:1450:4864:20::333]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1osKp9-0005oi-7L for qemu-devel@nongnu.org; Tue, 08 Nov 2022 04:23:25 -0500 Received: by mail-wm1-x333.google.com with SMTP id v7so8422859wmn.0 for ; Tue, 08 Nov 2022 01:23:14 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; 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=qBC/zqhe+w8c2MXv32aYnBZCDh2hCBt3ueKfoisA7Og=; b=mo3TP1aGA+/6qyCnthZ1d9uqmrNVikJG5cHs6qEgJsLnOMyQ+eaoUJG0vuZLzHbwmo 0EFJE36UQX4Za3z9lnfIbmeaWENpuiolyD5nq0khPqpyD28Ce4cE9HVdqm62gFoXGMn/ glapaDflqstR+ImKMvJdVDpLNP4T2TwB3gTTDEpeFOZVHwseCnYdTt4Ax/4dm3EA+rK8 3ujZb2yos3DikkwyU2p/nn3XK0L3js7Iou8E8yjRSjGJFg5XQo4tTuG0x1h4Kjc4/mil e77SMFFwAf5Ob0yVUq8yRta9hiV4SsYafKKRt60jHKUZr9RLuh/PpLOTCA0Nq1HI+pIj kKrg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; 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=qBC/zqhe+w8c2MXv32aYnBZCDh2hCBt3ueKfoisA7Og=; b=KBFqfLxyTU0EFmRWnlD4NlXOd5/ez7KqlM9FrU6TVWDBx/Xf6sYQKgKcn0+Odtrb9X 2R3QnVlAACEmZPxNchZzEtSxxMq7FDa95mkJdskwa0EOzo0P8tfkMDHStxumAbUIOuSj 1YA45Js1yCwhLJLpXr6SheeI6GOxDkuwARDszdNUftDFVAB71z/UyO1XJgfdIoHnfoxl M/V+89m59PxrwkKKI9enb5QAMx9C1L69ZrZ4HqTn3IgBbhYwD3MoDg9MMlglSIzj2gCy SOG8Cem7nUYoMzRF92TcBujSuPouA9yMzwp82i4Zeoq+FOX00250StIZPCej3Wx+Jrwh T9DA== X-Gm-Message-State: ACrzQf2oN2JvEEflOcEBkWhMht9AAngKRHB8WiR5BiAm+F0TIL6sZllz Dbw+ClF3xuqbtKdtp71NVNvB/w== X-Received: by 2002:a05:600c:2116:b0:3cf:9db5:1e1a with SMTP id u22-20020a05600c211600b003cf9db51e1amr12561695wml.193.1667899392534; Tue, 08 Nov 2022 01:23:12 -0800 (PST) Received: from zen.linaroharston ([185.81.254.11]) by smtp.gmail.com with ESMTPSA id j15-20020a5d604f000000b00236705daefesm9550600wrt.39.2022.11.08.01.23.08 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 08 Nov 2022 01:23:09 -0800 (PST) Received: from zen.lan (localhost [127.0.0.1]) by zen.linaroharston (Postfix) with ESMTP id 6B98A1FFBB; Tue, 8 Nov 2022 09:23:08 +0000 (GMT) From: =?utf-8?q?Alex_Benn=C3=A9e?= To: qemu-devel@nongnu.org Cc: fam@euphon.net, berrange@redhat.com, f4bug@amsat.org, aurelien@aurel32.net, pbonzini@redhat.com, stefanha@redhat.com, crosa@redhat.com, =?utf-8?q?C=C3=A9dric_Le_Goater?= , =?utf-8?q?Alex_Benn=C3=A9e?= , =?utf-8?q?Philippe_M?= =?utf-8?q?athieu-Daud=C3=A9?= , Wainer dos Santos Moschetta , Beraldo Leal Subject: [PATCH v1 3/9] tests/avocado/machine_aspeed.py: Reduce noise on the console for SDK tests Date: Tue, 8 Nov 2022 09:23:02 +0000 Message-Id: <20221108092308.1717426-4-alex.bennee@linaro.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20221108092308.1717426-1-alex.bennee@linaro.org> References: <20221108092308.1717426-1-alex.bennee@linaro.org> MIME-Version: 1.0 Received-SPF: pass client-ip=2a00:1450:4864:20::333; envelope-from=alex.bennee@linaro.org; helo=mail-wm1-x333.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, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 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 From: Cédric Le Goater The Aspeed SDK images are based on OpenBMC which starts a lot of services. The output noise on the console can break from time to time the test waiting for the logging prompt. Change the U-Boot bootargs variable to add "quiet" to the kernel command line and reduce the output volume. This also drops the test on the CPU id which was nice to have but not essential. Signed-off-by: Cédric Le Goater Message-Id: <20221104075347.370503-1-clg@kaod.org> Signed-off-by: Alex Bennée --- tests/avocado/machine_aspeed.py | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/tests/avocado/machine_aspeed.py b/tests/avocado/machine_aspeed.py index fba6527026..1fc385e1c8 100644 --- a/tests/avocado/machine_aspeed.py +++ b/tests/avocado/machine_aspeed.py @@ -12,6 +12,7 @@ from avocado_qemu import wait_for_console_pattern from avocado_qemu import exec_command from avocado_qemu import exec_command_and_wait_for_pattern +from avocado_qemu import interrupt_interactive_console_until_pattern from avocado.utils import archive from avocado import skipIf @@ -182,6 +183,8 @@ def test_arm_ast2600_evb_buildroot(self): class AST2x00MachineSDK(QemuSystemTest): + EXTRA_BOOTARGS = ' quiet' + # FIXME: Although these tests boot a whole distro they are still # slower than comparable machine models. There may be some # optimisations which bring down the runtime. In the meantime they @@ -194,7 +197,7 @@ def wait_for_console_pattern(self, success_message, vm=None): failure_message='Kernel panic - not syncing', vm=vm) - def do_test_arm_aspeed_sdk_start(self, image, cpu_id): + def do_test_arm_aspeed_sdk_start(self, image): self.require_netdev('user') self.vm.set_console() self.vm.add_args('-drive', 'file=' + image + ',if=mtd,format=raw', @@ -202,9 +205,13 @@ def do_test_arm_aspeed_sdk_start(self, image, cpu_id): self.vm.launch() self.wait_for_console_pattern('U-Boot 2019.04') - self.wait_for_console_pattern('## Loading kernel from FIT Image') + interrupt_interactive_console_until_pattern( + self, 'Hit any key to stop autoboot:', 'ast#') + exec_command_and_wait_for_pattern( + self, 'setenv bootargs ${bootargs}' + self.EXTRA_BOOTARGS, 'ast#') + exec_command_and_wait_for_pattern( + self, 'boot', '## Loading kernel from FIT Image') self.wait_for_console_pattern('Starting kernel ...') - self.wait_for_console_pattern('Booting Linux on physical CPU ' + cpu_id) @skipIf(os.getenv('GITLAB_CI'), 'Running on GitLab') def test_arm_ast2500_evb_sdk(self): @@ -221,7 +228,7 @@ def test_arm_ast2500_evb_sdk(self): archive.extract(image_path, self.workdir) self.do_test_arm_aspeed_sdk_start( - self.workdir + '/ast2500-default/image-bmc', '0x0') + self.workdir + '/ast2500-default/image-bmc') self.wait_for_console_pattern('ast2500-default login:') @skipIf(os.getenv('GITLAB_CI'), 'Running on GitLab') @@ -243,7 +250,7 @@ def test_arm_ast2600_evb_sdk(self): self.vm.add_args('-device', 'ds1338,bus=aspeed.i2c.bus.5,address=0x32'); self.do_test_arm_aspeed_sdk_start( - self.workdir + '/ast2600-default/image-bmc', '0xf00') + self.workdir + '/ast2600-default/image-bmc') self.wait_for_console_pattern('ast2600-default login:') exec_command_and_wait_for_pattern(self, 'root', 'Password:') exec_command_and_wait_for_pattern(self, '0penBmc', 'root@ast2600-default:~#') From patchwork Tue Nov 8 09:23:03 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Alex_Benn=C3=A9e?= X-Patchwork-Id: 622639 Delivered-To: patch@linaro.org Received: by 2002:a17:522:c983:b0:460:3032:e3c4 with SMTP id kr3csp2805714pvb; Tue, 8 Nov 2022 01:23:45 -0800 (PST) X-Google-Smtp-Source: AMsMyM6b/RxLwLDYmm0wvPe7nKbEYtQ+7WeSAMsh5QfOtUvrlZaUCXpHDgLBiOd66i25VnXu8pSs X-Received: by 2002:a05:620a:13fa:b0:6fa:15e3:a277 with SMTP id h26-20020a05620a13fa00b006fa15e3a277mr36734394qkl.479.1667899425237; Tue, 08 Nov 2022 01:23:45 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1667899425; cv=none; d=google.com; s=arc-20160816; b=DXfZuSWoq9AOxpSZrnMAa1XzB3jD4po1bqudG0DvDBeDJ0eQayuzEPqUrGnvSH5tiq lWdO3bsWeUWX+ehxteWK9c1ZENSsgHR3a5wi7wuTRYGB6tZM7Bt1htDWvTwmYnugA/kX gXS0y2EgAwuNTmOfG0w8r8WBsgg8p1zPIexmGbqXYmkI4dLYRMHlLKyD5rHVsvfsj/x8 5gmmTTRQCvVxznKggiS8whWFN3UMhKu9hUM5WohIoWvjYuCim8jNz/OMta38u4lTuYhO 74Sk4BPXDEC0Wa49LqCPMx/qoye8FiOzePkNmWkX57nVIcevi3z7sIH89Fb8ckolZS7d S2OA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; 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=7uXwU/1og6Y5YFCCkzlXH6mHIAZKAJashuR6BqLq8xE=; b=YCJzVSK1xA9VyG2pnse6iiMmYkyo4EeQP31rlHtYxolKN5hDFo52iSGi5wsPQKHwP2 iWKry6lvRKbwYC8uevuEMBr/4fexGVC+Y9qO/+EZz78E0cfLLVVhnGFDa5Cr7dx2C26T Cr8zYfgIH+FmD8HhDOqL6o0/N4QO+0DOljEYyuZRcisgeotQrlnTPNgQINXX6lskmGE1 qA/NIVD98BWiLB8p8uHBG++FlVy34wIQMEHnPcvB+CWZkEuA6j42b9TDLI+AZ9AExU4v zj3G84kLpj1INnXxo8ZrTNaK+LFTuuRSU4p7+ONYbp6XksKtvuXzJixFAcPb4Yk5R/9C 0ilQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=yH0ZvAcV; 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 Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id x17-20020a05620a449100b006fa6df83b05si7015032qkp.258.2022.11.08.01.23.45 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Tue, 08 Nov 2022 01:23:45 -0800 (PST) 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=yH0ZvAcV; 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 Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1osKpJ-0007Fp-VM; Tue, 08 Nov 2022 04:23:25 -0500 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 1osKpB-0007DV-Up for qemu-devel@nongnu.org; Tue, 08 Nov 2022 04:23:17 -0500 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 1osKp8-0005oT-Dl for qemu-devel@nongnu.org; Tue, 08 Nov 2022 04:23:17 -0500 Received: by mail-wr1-x435.google.com with SMTP id a14so19964286wru.5 for ; Tue, 08 Nov 2022 01:23:13 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; 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=7uXwU/1og6Y5YFCCkzlXH6mHIAZKAJashuR6BqLq8xE=; b=yH0ZvAcVvt1bGy2FGEbgsuF7dOcFsPLuKCi0FSL6ftwU7PAMVZA1ZJEW+uJ8I2wWEc sDYUz21gIlMHSR78txluz2Lowl7GCrNdu+KsK62nfXVWXtRZ48By27XYFB84t3rXaUyz G6Nylma9BCS/JJz6OY6+XQ1U6WpHJyjNuVgKE2teAXth8brWX7gGc4WbmkXrYk4jxEwz lC+kuvPctRx84mR63rN5IRDvOyjyczG9aHnNE/zrYJ74imLS2/aDZpU0PPZLuHrs3dfz 63YA7I/zwysuC0NqmAu23F33wyCK/o9FCbAXLi+cIcvc7p/tcOF1Huh69YIvmDyRnEs1 qI2w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; 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=7uXwU/1og6Y5YFCCkzlXH6mHIAZKAJashuR6BqLq8xE=; b=pnpWHrfe0cukQwDAZrw4YZXzJqVf5w2peUBUzPAd+9yf7qhH2CZwtJUJwMiYaIMbKo H1xlaBWkpQmRnP7CqAroeICQqUZojyiYOBZTEb3sq5qM/xMtlbzc7fsA57Y/FoKyaej9 O2NxGKDdnW31jyCBCyDuj6U/Cccs/qLJRx99bxcwuto/jYL0RK+bCWg7+boQiElPBRMA FNdYVZr4HMOAtDIJgMb3yytizq3B2M9PRSf0N1ECl3cbIxeOGLSGaU+aII6v+BYOk2zu aVYXdnGj+a9f012PEjaxIe8juVIqKQnNEZt6GHw25M1HBuofnW0GMQX1Ab7rCI1oXA3M neew== X-Gm-Message-State: ACrzQf2OvCNXVPEq8e/a7TEwYxPMpTP7afxl2XdZf8RLOOflNb1yo34b +DDTWrSP3Q8ebIaRgAUMNW3Hxg== X-Received: by 2002:a05:6000:1052:b0:236:6e8e:8403 with SMTP id c18-20020a056000105200b002366e8e8403mr34588988wrx.178.1667899391778; Tue, 08 Nov 2022 01:23:11 -0800 (PST) Received: from zen.linaroharston ([185.81.254.11]) by smtp.gmail.com with ESMTPSA id ay15-20020a05600c1e0f00b003c6deb5c1edsm11032025wmb.45.2022.11.08.01.23.08 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 08 Nov 2022 01:23:09 -0800 (PST) Received: from zen.lan (localhost [127.0.0.1]) by zen.linaroharston (Postfix) with ESMTP id 85ED81FFBC; Tue, 8 Nov 2022 09:23:08 +0000 (GMT) From: =?utf-8?q?Alex_Benn=C3=A9e?= To: qemu-devel@nongnu.org Cc: fam@euphon.net, berrange@redhat.com, f4bug@amsat.org, aurelien@aurel32.net, pbonzini@redhat.com, stefanha@redhat.com, crosa@redhat.com, =?utf-8?q?Alex_Benn=C3=A9e?= , =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= , Thomas Huth , Wainer dos Santos Moschetta , Beraldo Leal Subject: [PATCH v1 4/9] tests/docker: allow user to override check target Date: Tue, 8 Nov 2022 09:23:03 +0000 Message-Id: <20221108092308.1717426-5-alex.bennee@linaro.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20221108092308.1717426-1-alex.bennee@linaro.org> References: <20221108092308.1717426-1-alex.bennee@linaro.org> MIME-Version: 1.0 Received-SPF: pass client-ip=2a00:1450:4864:20::435; envelope-from=alex.bennee@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 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 This is useful when trying to bisect a particular failing test behind a docker run. For example: make docker-test-clang@fedora \ TARGET_LIST=arm-softmmu \ TEST_COMMAND="meson test qtest-arm/qos-test" \ J=9 V=1 Signed-off-by: Alex Bennée Reviewed-by: Philippe Mathieu-Daudé --- v1 - fix s/target /target./ - CHECK_TARGET -> TEST_COMMAND --- tests/docker/Makefile.include | 2 ++ tests/docker/common.rc | 6 +++--- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include index c87f14477a..fc7a3b7e71 100644 --- a/tests/docker/Makefile.include +++ b/tests/docker/Makefile.include @@ -184,6 +184,7 @@ docker: @echo ' TARGET_LIST=a,b,c Override target list in builds.' @echo ' EXTRA_CONFIGURE_OPTS="..."' @echo ' Extra configure options.' + @echo ' TEST_COMMAND="..." Override the default `make check` target.' @echo ' IMAGES="a b c ..": Restrict available images to subset.' @echo ' TESTS="x y z .." Restrict available tests to subset.' @echo ' J=[0..9]* Overrides the -jN parameter for make commands' @@ -230,6 +231,7 @@ docker-run: docker-qemu-src $(if $(NETWORK),$(if $(subst $(NETWORK),,1),--net=$(NETWORK)),--net=none) \ -e TARGET_LIST=$(subst $(SPACE),$(COMMA),$(TARGET_LIST)) \ -e EXTRA_CONFIGURE_OPTS="$(EXTRA_CONFIGURE_OPTS)" \ + -e TEST_COMMAND="$(TEST_COMMAND)" \ -e V=$V -e J=$J -e DEBUG=$(DEBUG) \ -e SHOW_ENV=$(SHOW_ENV) \ $(if $(NOUSER),, \ diff --git a/tests/docker/common.rc b/tests/docker/common.rc index e6f8cee0d6..9a33df2832 100755 --- a/tests/docker/common.rc +++ b/tests/docker/common.rc @@ -63,12 +63,12 @@ check_qemu() { # default to make check unless the caller specifies if [ $# = 0 ]; then - INVOCATION="check" + INVOCATION="${TEST_COMMAND:-make $MAKEFLAGS check}" else - INVOCATION="$@" + INVOCATION="make $MAKEFLAGS $@" fi - make $MAKEFLAGS $INVOCATION + $INVOCATION } test_fail() From patchwork Tue Nov 8 09:23:04 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Alex_Benn=C3=A9e?= X-Patchwork-Id: 622643 Delivered-To: patch@linaro.org Received: by 2002:a17:522:c983:b0:460:3032:e3c4 with SMTP id kr3csp2806313pvb; Tue, 8 Nov 2022 01:25:02 -0800 (PST) X-Google-Smtp-Source: AMsMyM4VPKmKiwhM8bBlFa4cFkaWUCruG6HNQEk7l73Rj1Hfi5p05Qz3TZ6R6XmNntASOU7FXUiK X-Received: by 2002:a0c:ee6b:0:b0:4bc:16a2:d4ff with SMTP id n11-20020a0cee6b000000b004bc16a2d4ffmr805013qvs.37.1667899501999; Tue, 08 Nov 2022 01:25:01 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1667899501; cv=none; d=google.com; s=arc-20160816; b=AK0GXeMoKG0H6t949ZeqnfQuoev1EAG8rzHPiHZnF4MTMZAt4QxPFRt/b6MbpXtKsK E9l9OM3BVsCerU33BW8w+f0qOXhpY8BzTJl5/SJyR7N0JznAVodTxCMqjG6JueA775X4 /F78DWH8lnpqUDnEkkVrV4NmuTionk1J41GoBeY0Pcz85fVjPP8KHE0ORXRUbVvewc4u AakA8DNsX6Oa/CI+PC3Fb1qD8qfo9Wlgqiv9C0H8hpxRK43f0XXnQmwWbPOLN1knlDtI t2Bu919PofvH4TGyjlwu6WI1tyMFfBdio3J+7O1YxlNF6xg3WjnnKzWG4a2OfLg7Y9fd 6CCA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; 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=+Mtx8odvHwCV0bf17EfBgIaZ7U3IJZv+eeZfeBwc4UQ=; b=0ro0e0RRbxi10w5vjyZTMsx7fxxH4u8IAEeK0EuXa7eaMYDIl731LF7tG1t0xzESvK Dwh6r8v6S08LQChjJ+q5CpNZ+bVj0u8E9ZTGJ7L9Zf+fQHb7oC4T5MXZjEg2bXAgT+Gy bEXJKEMcPW3Rm1+wIAkZdJ/r5qTulaY1qmuswj4C8PCRLPaRi1LQ8urSDcwMkVsBMy1R ZNtThPx/Ip08TI/pSYcQrkEw6ku3PBbsnbeyxoTRocBqZxaglyIC8+PnRTJ0DTtgyVW6 EOy/0YrEHm3LBsp7YaOTnF4SqVyPiHOI5aD+xsArQKuVaPhIDhkif5SWS+gRnvI7z2fa YZ6A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=pylU059h; 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 Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id x7-20020a05620a448700b006ecdfa15141si6677053qkp.140.2022.11.08.01.25.01 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Tue, 08 Nov 2022 01:25:01 -0800 (PST) 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=pylU059h; 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 Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1osKpL-0007GX-Rv; Tue, 08 Nov 2022 04:23:28 -0500 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 1osKpC-0007Dg-Ig for qemu-devel@nongnu.org; Tue, 08 Nov 2022 04:23:18 -0500 Received: from mail-wr1-x430.google.com ([2a00:1450:4864:20::430]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1osKpA-0005p4-7o for qemu-devel@nongnu.org; Tue, 08 Nov 2022 04:23:18 -0500 Received: by mail-wr1-x430.google.com with SMTP id g12so19904049wrs.10 for ; Tue, 08 Nov 2022 01:23:14 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; 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=+Mtx8odvHwCV0bf17EfBgIaZ7U3IJZv+eeZfeBwc4UQ=; b=pylU059h9atBQP2HUJ1P2PS5gPhC528RXgxry6KwVPVJPUphbcC7U9294RnSW+1vxy 7G52xLzTQMlkaXzX8tE1ojTe6NDZN91xdDS+96gOQgKvL9qkfRNq/acxVJsmmONzuiwT 0RoBbKFiG2X4WS/ZU2y8NolvEpt/JT4l2Sp2Ny+9LrTDIJsWPk0jUj0hnoCZgUYX/AS+ /6OG5xIZXCclbNAfUrIsRvXAzJhVHgJDTKu9LV0dh8F7RNuHbNeVAKvUjDrsnn9OyINP WmKfY4roK69ctfmrPmJipUJocf67Kb9y0GX5kTY3VSM8s/hXHg+EKy1GjV2qJbG1Q3yQ VBLA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; 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=+Mtx8odvHwCV0bf17EfBgIaZ7U3IJZv+eeZfeBwc4UQ=; b=LNjAjjK9FeeZAlT8/IcV+cdMZub81NoOyKfMTbYNEp2Xm5lCETEcARtKIvZmjfSjaR pgG7ZRS7HDWhfnUm1YDNUl4Q/09IrsYzQ2jem+OLArz/9SP+8ANTEDi8upVFoC3nbLLX Sg7y0duZ2KyOmRTwMHVj3tihPAF3W/eUeCtiajtYnotvZbzBUSWzSZOFEdDnrN3g0Q0B UfHLw4DhmNTFC7fxRQYgCYeeFsvzcLgCleFBiweprbFRoh8jlOg4Ps307t0X06nEW5a2 xR7D129y4MI2F7GjRrso2vpZZTGVNGlAI5xNh/0zntSwyPR/VI6WSJMMercGAL7W8HpL b6+w== X-Gm-Message-State: ACrzQf2boyPhLr72jmcLjrGjtgC76aFnEOXwb4E+CZdOd0e/KKKBj8bk v0+leratdarmSdbeAiSdHY+mOA== X-Received: by 2002:a5d:4b51:0:b0:236:88a2:267f with SMTP id w17-20020a5d4b51000000b0023688a2267fmr33450222wrs.461.1667899393939; Tue, 08 Nov 2022 01:23:13 -0800 (PST) Received: from zen.linaroharston ([185.81.254.11]) by smtp.gmail.com with ESMTPSA id b9-20020a05600010c900b002368424f89esm9713795wrx.67.2022.11.08.01.23.09 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 08 Nov 2022 01:23:10 -0800 (PST) Received: from zen.lan (localhost [127.0.0.1]) by zen.linaroharston (Postfix) with ESMTP id B16A91FFBD; Tue, 8 Nov 2022 09:23:08 +0000 (GMT) From: =?utf-8?q?Alex_Benn=C3=A9e?= To: qemu-devel@nongnu.org Cc: fam@euphon.net, berrange@redhat.com, f4bug@amsat.org, aurelien@aurel32.net, pbonzini@redhat.com, stefanha@redhat.com, crosa@redhat.com, =?utf-8?q?Alex_Benn=C3=A9e?= , "Michael S. Tsirkin" , Raphael Norwitz , Kevin Wolf , Hanna Reitz , "Dr. David Alan Gilbert" , Viresh Kumar , Mathieu Poirier , qemu-block@nongnu.org (open list:Block layer core), virtio-fs@redhat.com (open list:virtiofs) Subject: [PATCH v1 5/9] hw/virtio: introduce virtio_device_should_start Date: Tue, 8 Nov 2022 09:23:04 +0000 Message-Id: <20221108092308.1717426-6-alex.bennee@linaro.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20221108092308.1717426-1-alex.bennee@linaro.org> References: <20221108092308.1717426-1-alex.bennee@linaro.org> MIME-Version: 1.0 Received-SPF: pass client-ip=2a00:1450:4864:20::430; envelope-from=alex.bennee@linaro.org; helo=mail-wr1-x430.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 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 The previous fix to virtio_device_started revealed a problem in its use by both the core and the device code. The core code should be able to handle the device "starting" while the VM isn't running to handle the restoration of migration state. To solve this dual use introduce a new helper for use by the vhost-user backends who all use it to feed a should_start variable. We can also pick up a change vhost_user_blk_set_status while we are at it which follows the same pattern. Fixes: 9f6bcfd99f (hw/virtio: move vm_running check to virtio_device_started) Fixes: 27ba7b027f (hw/virtio: add boilerplate for vhost-user-gpio device) Signed-off-by: Alex Bennée Cc: "Michael S. Tsirkin" --- include/hw/virtio/virtio.h | 18 ++++++++++++++++++ hw/block/vhost-user-blk.c | 6 +----- hw/virtio/vhost-user-fs.c | 2 +- hw/virtio/vhost-user-gpio.c | 2 +- hw/virtio/vhost-user-i2c.c | 2 +- hw/virtio/vhost-user-rng.c | 2 +- hw/virtio/vhost-user-vsock.c | 2 +- hw/virtio/vhost-vsock.c | 2 +- 8 files changed, 25 insertions(+), 11 deletions(-) diff --git a/include/hw/virtio/virtio.h b/include/hw/virtio/virtio.h index f41b4a7e64..3191c618f3 100644 --- a/include/hw/virtio/virtio.h +++ b/include/hw/virtio/virtio.h @@ -389,6 +389,24 @@ static inline bool virtio_device_started(VirtIODevice *vdev, uint8_t status) return vdev->started; } + return status & VIRTIO_CONFIG_S_DRIVER_OK; +} + +/** + * virtio_device_should_start() - check if device startable + * @vdev - the VirtIO device + * @status - the devices status bits + * + * This is similar to virtio_device_started() but also encapsulates a + * check on the VM status which would prevent a device starting + * anyway. + */ +static inline bool virtio_device_should_start(VirtIODevice *vdev, uint8_t status) +{ + if (vdev->use_started) { + return vdev->started; + } + if (!vdev->vm_running) { return false; } diff --git a/hw/block/vhost-user-blk.c b/hw/block/vhost-user-blk.c index 13bf5cc47a..8feaf12e4e 100644 --- a/hw/block/vhost-user-blk.c +++ b/hw/block/vhost-user-blk.c @@ -222,14 +222,10 @@ static void vhost_user_blk_stop(VirtIODevice *vdev) static void vhost_user_blk_set_status(VirtIODevice *vdev, uint8_t status) { VHostUserBlk *s = VHOST_USER_BLK(vdev); - bool should_start = virtio_device_started(vdev, status); + bool should_start = virtio_device_should_start(vdev, status); Error *local_err = NULL; int ret; - if (!vdev->vm_running) { - should_start = false; - } - if (!s->connected) { return; } diff --git a/hw/virtio/vhost-user-fs.c b/hw/virtio/vhost-user-fs.c index ad0f91c607..1c40f42045 100644 --- a/hw/virtio/vhost-user-fs.c +++ b/hw/virtio/vhost-user-fs.c @@ -123,7 +123,7 @@ static void vuf_stop(VirtIODevice *vdev) static void vuf_set_status(VirtIODevice *vdev, uint8_t status) { VHostUserFS *fs = VHOST_USER_FS(vdev); - bool should_start = virtio_device_started(vdev, status); + bool should_start = virtio_device_should_start(vdev, status); if (vhost_dev_is_started(&fs->vhost_dev) == should_start) { return; diff --git a/hw/virtio/vhost-user-gpio.c b/hw/virtio/vhost-user-gpio.c index 8b40fe450c..677d1c7730 100644 --- a/hw/virtio/vhost-user-gpio.c +++ b/hw/virtio/vhost-user-gpio.c @@ -152,7 +152,7 @@ static void vu_gpio_stop(VirtIODevice *vdev) static void vu_gpio_set_status(VirtIODevice *vdev, uint8_t status) { VHostUserGPIO *gpio = VHOST_USER_GPIO(vdev); - bool should_start = virtio_device_started(vdev, status); + bool should_start = virtio_device_should_start(vdev, status); trace_virtio_gpio_set_status(status); diff --git a/hw/virtio/vhost-user-i2c.c b/hw/virtio/vhost-user-i2c.c index bc58b6c0d1..864eba695e 100644 --- a/hw/virtio/vhost-user-i2c.c +++ b/hw/virtio/vhost-user-i2c.c @@ -93,7 +93,7 @@ static void vu_i2c_stop(VirtIODevice *vdev) static void vu_i2c_set_status(VirtIODevice *vdev, uint8_t status) { VHostUserI2C *i2c = VHOST_USER_I2C(vdev); - bool should_start = virtio_device_started(vdev, status); + bool should_start = virtio_device_should_start(vdev, status); if (vhost_dev_is_started(&i2c->vhost_dev) == should_start) { return; diff --git a/hw/virtio/vhost-user-rng.c b/hw/virtio/vhost-user-rng.c index bc1f36c5ac..8b47287875 100644 --- a/hw/virtio/vhost-user-rng.c +++ b/hw/virtio/vhost-user-rng.c @@ -90,7 +90,7 @@ static void vu_rng_stop(VirtIODevice *vdev) static void vu_rng_set_status(VirtIODevice *vdev, uint8_t status) { VHostUserRNG *rng = VHOST_USER_RNG(vdev); - bool should_start = virtio_device_started(vdev, status); + bool should_start = virtio_device_should_start(vdev, status); if (vhost_dev_is_started(&rng->vhost_dev) == should_start) { return; diff --git a/hw/virtio/vhost-user-vsock.c b/hw/virtio/vhost-user-vsock.c index 7b67e29d83..9431b9792c 100644 --- a/hw/virtio/vhost-user-vsock.c +++ b/hw/virtio/vhost-user-vsock.c @@ -55,7 +55,7 @@ const VhostDevConfigOps vsock_ops = { static void vuv_set_status(VirtIODevice *vdev, uint8_t status) { VHostVSockCommon *vvc = VHOST_VSOCK_COMMON(vdev); - bool should_start = virtio_device_started(vdev, status); + bool should_start = virtio_device_should_start(vdev, status); if (vhost_dev_is_started(&vvc->vhost_dev) == should_start) { return; diff --git a/hw/virtio/vhost-vsock.c b/hw/virtio/vhost-vsock.c index 7dc3c73931..aa16d584ee 100644 --- a/hw/virtio/vhost-vsock.c +++ b/hw/virtio/vhost-vsock.c @@ -70,7 +70,7 @@ static int vhost_vsock_set_running(VirtIODevice *vdev, int start) static void vhost_vsock_set_status(VirtIODevice *vdev, uint8_t status) { VHostVSockCommon *vvc = VHOST_VSOCK_COMMON(vdev); - bool should_start = virtio_device_started(vdev, status); + bool should_start = virtio_device_should_start(vdev, status); int ret; if (vhost_dev_is_started(&vvc->vhost_dev) == should_start) { From patchwork Tue Nov 8 09:23:05 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Alex_Benn=C3=A9e?= X-Patchwork-Id: 622638 Delivered-To: patch@linaro.org Received: by 2002:a17:522:c983:b0:460:3032:e3c4 with SMTP id kr3csp2805584pvb; Tue, 8 Nov 2022 01:23:33 -0800 (PST) X-Google-Smtp-Source: AMsMyM6R58CO4hRJQh00SG5X7Ne4/+BwZzQvypIxhjxPh+aHP/7GBciyUkjIbTovrRmp7Cq12TXJ X-Received: by 2002:ac8:74cb:0:b0:3a5:1c92:4aa4 with SMTP id j11-20020ac874cb000000b003a51c924aa4mr40852159qtr.231.1667899412922; Tue, 08 Nov 2022 01:23:32 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1667899412; cv=none; d=google.com; s=arc-20160816; b=zOQm3sYY1goI3+NqyJsBGL6av0QN1hcbK12vzFS5sZPzbU+d/mNr1cVMOyLx0ApNhc pHxGIx2dK6La1ksryjfEp/fWLiUG9OCECYpKxQIabuc5b/qS/M/5vmiYhBOdnGcnOpGg 9P3xxJOmA7JFGVLmPavlXBsuYuVd0VmTDoQVkHVFAmbdY9D4RYJS/NC/ZUaHPY2jq4zY ENFTgz0lDPxUJpTAKBIdzoVmenNcQQrx346m0sN8RnuI97OHvAQdp5f86M091FqtP7Im y6n2yKEp6m6jaq5JxvDupDrbOfSOljk+E0MCzybfG5Ow0ez4FS5bn5rXCs/vgcam1NpT WuRw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; 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=PETkHCmmDlOk/l4eNJN9a+MTbBUaCZh7Q5yajW/EJQg=; b=pxunmZPcVjzpGxaeD4XsVNdJJ7z12BM7kJIqYiKA0x02pIKSoVM6Ce+ljsNM3z8CD+ O0g3w6J04zUDe3AMbC19ii+14ajbk4gOxmDRFDXJFDj3cjTZHetIFPs2PR6oXR5VYFme 0uAOmk/Sm/B1SATU96R8VLOBNYgWrljZG6ChJC1sSRQ+GWrkntHSuUMrsDGw1Poa2aEz QOoHJyv9Gtyos1LOsP6xy2WJtJFBEf9SSy64XmLgaowIP9x2bzFUDxndyjqntTM+oijw K7YDlJ10qtQS4+NnT6lelbn+X4ArTaO2ebUoF8559m3F2fh4z9GjVZqDjtdOJ95p0Rjf CcZw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b="hPbtvtK/"; 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 Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id l16-20020a37f910000000b006fa696cce73si5960723qkj.497.2022.11.08.01.23.32 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Tue, 08 Nov 2022 01:23:32 -0800 (PST) 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="hPbtvtK/"; 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 Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1osKpG-0007F0-Ov; Tue, 08 Nov 2022 04:23:22 -0500 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 1osKpA-0007Cz-Vs for qemu-devel@nongnu.org; Tue, 08 Nov 2022 04:23:17 -0500 Received: from mail-wm1-x32b.google.com ([2a00:1450:4864:20::32b]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1osKp8-0005os-Cz for qemu-devel@nongnu.org; Tue, 08 Nov 2022 04:23:16 -0500 Received: by mail-wm1-x32b.google.com with SMTP id i5-20020a1c3b05000000b003cfa97c05cdso637969wma.4 for ; Tue, 08 Nov 2022 01:23:13 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; 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=PETkHCmmDlOk/l4eNJN9a+MTbBUaCZh7Q5yajW/EJQg=; b=hPbtvtK/7aEGeHJZpkm4fTHtiVHpzEFss7EumU/eRkqyVZIyvC0m35J6KiCV5OuXJs pumgm04atdVvG+YNmxxP6vePer5bfaKBWfGbLA+Pddf16j+1urASApV3kc+m6AnOrUCa DJnn5jOVCOQ54Y3P54yIwG3f27TBYA0PYUwDRWKJevMzJSG0FYJzP5FtIWUIjwUgAtnV 9das+x+WtVlkjR8G2nzfvtBEPBRKS4DJyja50hpKwBKrPFcTx9f2b5gmFRQxsDbIP9kj CElS34VgLZSbKfKH/NGYrSau+q7REkPUmvjnRfwzv8zG8ZvcLOK+3eK1VP0CkVnhKsa9 B4cg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; 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=PETkHCmmDlOk/l4eNJN9a+MTbBUaCZh7Q5yajW/EJQg=; b=byGZI2phifqaPXblQ9V2PaDNgSMBx3qLVBzNzRB1TLQcDwGuydS2RFsFAzU5kjZc3O iJz7Nc1YHQkZe60rT3zDpjvNTaqX/RG44oT5RuYNIkyDGnD3vPg99LGWdcRzxsX+/+Lu inZ/O/ZzS1lKstvtMsdnkX70ckhyewci5uYIW8spPbdHDTX7n0kzFzdBIWpfliDbtzEI gTaiS8psba93F7bPyvd6ahSae2m4Z0GO40qNOxKQYA7A4acw2wlUHwupJIYBJ728nw7W rn9m9yI0AwxQ4iR4XiCZnfGPFc76oJwe/THZLhgZdOqXZbaCftmkKx9A7z9EBSF1cDYb ltXQ== X-Gm-Message-State: ACrzQf2aEZ5lJf/iwdYvVNCi/xPrObzIahdy4GMiaJRigJB+l8Isi0Xf S6iSTVBBanwFtujvPWKqIOCOYg== X-Received: by 2002:a05:600c:3c82:b0:3b5:60a6:c80f with SMTP id bg2-20020a05600c3c8200b003b560a6c80fmr36754347wmb.199.1667899392914; Tue, 08 Nov 2022 01:23:12 -0800 (PST) Received: from zen.linaroharston ([185.81.254.11]) by smtp.gmail.com with ESMTPSA id z17-20020adfec91000000b002366e8eee11sm9508857wrn.101.2022.11.08.01.23.09 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 08 Nov 2022 01:23:10 -0800 (PST) Received: from zen.lan (localhost [127.0.0.1]) by zen.linaroharston (Postfix) with ESMTP id CF83E1FFBE; Tue, 8 Nov 2022 09:23:08 +0000 (GMT) From: =?utf-8?q?Alex_Benn=C3=A9e?= To: qemu-devel@nongnu.org Cc: fam@euphon.net, berrange@redhat.com, f4bug@amsat.org, aurelien@aurel32.net, pbonzini@redhat.com, stefanha@redhat.com, crosa@redhat.com, =?utf-8?q?Alex_Benn=C3=A9e?= Subject: [PATCH v1 6/9] docs/devel: add a maintainers section to development process Date: Tue, 8 Nov 2022 09:23:05 +0000 Message-Id: <20221108092308.1717426-7-alex.bennee@linaro.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20221108092308.1717426-1-alex.bennee@linaro.org> References: <20221108092308.1717426-1-alex.bennee@linaro.org> MIME-Version: 1.0 Received-SPF: pass client-ip=2a00:1450:4864:20::32b; envelope-from=alex.bennee@linaro.org; helo=mail-wm1-x32b.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 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 We don't currently have a clear place in the documentation to describe the roles and responsibilities of a maintainer. Lets create one so we can. I've moved a few small bits out of other files to try and keep everything in one place. Signed-off-by: Alex Bennée Reviewed-by: Stefan Hajnoczi Reviewed-by: Paolo Bonzini Message-Id: <20221012121152.1179051-2-alex.bennee@linaro.org> --- v2 - s/roll/role - s/projects/project's - mention looking after the long term health of area - add a section on becoming a reviewer - expand becoming section - add footnote about key signing --- docs/devel/code-of-conduct.rst | 2 + docs/devel/index-process.rst | 1 + docs/devel/maintainers.rst | 106 +++++++++++++++++++++++ docs/devel/submitting-a-pull-request.rst | 12 +-- 4 files changed, 113 insertions(+), 8 deletions(-) create mode 100644 docs/devel/maintainers.rst diff --git a/docs/devel/code-of-conduct.rst b/docs/devel/code-of-conduct.rst index 195444d1b4..f734ed0317 100644 --- a/docs/devel/code-of-conduct.rst +++ b/docs/devel/code-of-conduct.rst @@ -1,3 +1,5 @@ +.. _code_of_conduct: + Code of Conduct =============== diff --git a/docs/devel/index-process.rst b/docs/devel/index-process.rst index d0d7a200fd..d50dd74c3e 100644 --- a/docs/devel/index-process.rst +++ b/docs/devel/index-process.rst @@ -8,6 +8,7 @@ Notes about how to interact with the community and how and where to submit patch code-of-conduct conflict-resolution + maintainers style submitting-a-patch trivial-patches diff --git a/docs/devel/maintainers.rst b/docs/devel/maintainers.rst new file mode 100644 index 0000000000..05110909d1 --- /dev/null +++ b/docs/devel/maintainers.rst @@ -0,0 +1,106 @@ +.. _maintainers: + +The Role of Maintainers +======================= + +Maintainers are a critical part of the project's contributor ecosystem. +They come from a wide range of backgrounds from unpaid hobbyists +working in their spare time to employees who work on the project as +part of their job. Maintainer activities include: + + - reviewing patches and suggesting changes + - collecting patches and preparing pull requests + - tending to the long term health of their area + - participating in other project activities + +They are also human and subject to the same pressures as everyone else +including overload and burnout. Like everyone else they are subject +to project's :ref:`code_of_conduct` and should also be exemplars of +excellent community collaborators. + +The MAINTAINERS file +-------------------- + +The `MAINTAINERS +`__ +file contains the canonical list of who is a maintainer. The file +is machine readable so an appropriately configured git (see +:ref:`cc_the_relevant_maintainer`) can automatically Cc them on +patches that touch their area of code. + +The file also describes the status of the area of code to give an idea +of how actively that section is maintained. + +.. list-table:: Meaning of support status in MAINTAINERS + :widths: 25 75 + :header-rows: 1 + + * - Status + - Meaning + * - Supported + - Someone is actually paid to look after this. + * - Maintained + - Someone actually looks after it. + * - Odd Fixes + - It has a maintainer but they don't have time to do + much other than throw the odd patch in. + * - Orphan + - No current maintainer. + * - Obsolete + - Old obsolete code, should use something else. + +Please bear in mind that even if someone is paid to support something +it does not mean they are paid to support you. This is open source and +the code comes with no warranty and the project makes no guarantees +about dealing with bugs or features requests. + + + +Becoming a reviewer +------------------- + +Most maintainers start by becoming subsystem reviewers. While anyone +is welcome to review code on the mailing list getting added to the +MAINTAINERS file with a line like:: + + R: Random Hacker + +will ensure that patches touching a given subsystem will automatically +be CC'd to you. + +Becoming a maintainer +--------------------- + +Maintainers are volunteers who put themselves forward or have been +asked by others to keep an eye on an area of code. They have generally +demonstrated to the community, usually via contributions and code +reviews, that they have a good understanding of the subsystem. They +are also trusted to make a positive contribution to the project and +work well with the other contributors. + +The process is simple - simply send a patch to the list that updates +the ``MAINTAINERS`` file. Sometimes this is done as part of a larger +series when a new sub-system is being added to the code base. This can +also be done by a retiring maintainer who nominates their replacement +after discussion with other contributors. + +Once the patch is reviewed and merged the only other step is to make +sure your GPG key is signed. + +.. _maintainer_keys: + +Maintainer GPG Keys +~~~~~~~~~~~~~~~~~~~ + +GPG is used to sign pull requests so they can be identified as really +coming from the maintainer. If your key is not already signed by +members of the QEMU community, you should make arrangements to attend +a `KeySigningParty `__ (for +example at KVM Forum) or make alternative arrangements to have your +key signed by an attendee. Key signing requires meeting another +community member **in person** [#]_ so please make appropriate +arrangements. + +.. [#] In recent pandemic times we have had to exercise some + flexibility here. Maintainers still need to sign their pull + requests though. diff --git a/docs/devel/submitting-a-pull-request.rst b/docs/devel/submitting-a-pull-request.rst index c9d1e8afd9..a4cd7ebbb6 100644 --- a/docs/devel/submitting-a-pull-request.rst +++ b/docs/devel/submitting-a-pull-request.rst @@ -53,14 +53,10 @@ series) and that "make check" passes before sending out the pull request. As a submaintainer you're one of QEMU's lines of defense against bad code, so double check the details. -**All pull requests must be signed**. If your key is not already signed -by members of the QEMU community, you should make arrangements to attend -a `KeySigningParty `__ (for -example at KVM Forum) or make alternative arrangements to have your key -signed by an attendee. Key signing requires meeting another community -member \*in person\* so please make appropriate arrangements. By -"signed" here we mean that the pullreq email should quote a tag which is -a GPG-signed tag (as created with 'gpg tag -s ...'). +**All pull requests must be signed**. By "signed" here we mean that +the pullreq email should quote a tag which is a GPG-signed tag (as +created with 'gpg tag -s ...'). See :ref:`maintainer_keys` for +details. **Pull requests not for master should say "not for master" and have "PULL SUBSYSTEM whatever" in the subject tag**. If your pull request is From patchwork Tue Nov 8 09:23:06 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Alex_Benn=C3=A9e?= X-Patchwork-Id: 622645 Delivered-To: patch@linaro.org Received: by 2002:a17:522:c983:b0:460:3032:e3c4 with SMTP id kr3csp2806724pvb; Tue, 8 Nov 2022 01:25:49 -0800 (PST) X-Google-Smtp-Source: AMsMyM5rtdTQsBvHnFzhRMeDb3kHToGtyXLznkm4zRtGzt1nLQLwc9u9yHfNAG7vRRfv/916pK4D X-Received: by 2002:a0c:9047:0:b0:4bb:f778:b946 with SMTP id o65-20020a0c9047000000b004bbf778b946mr41547543qvo.77.1667899549522; Tue, 08 Nov 2022 01:25:49 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1667899549; cv=none; d=google.com; s=arc-20160816; b=GeDk/r1TQyjb9IKJGnhG4uVuPJDUbm5ErxltNg/kAIWhF0HrKKDK6/mAMluwn6qqrA 61loI7pHsmwUl5sLS4xDRumqlCpDmzHHHNuBhZAY8hUcKhAA1JOLyrjVEtNEkVqjOFav dp3Lg0Shm4xBOLquQcmAliOo+qXz6ExdrvomM7gAxOqnCI5OAGAxRzTDsjLL2D+H5TZu CIuI8Swa7GhUVJ4qNjpbpuaT26YkC1oTQVOBhWo4KruEHMWUizvW0MtEVCCwqZy2BHW4 QyjfIgKA0BAB1BXBXQ1HV+RLK9vIqhzNmQkq0zJD3aXupqSyeuKce82otnVM1w2cKwFK Ohdg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; 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=U0bUaw8dD9ywC6S/a5dl4BRNFGfCaE47JrvKC4nt86Q=; b=OUGnzb0NcUIOwKgtEHka6qUcJmtNKbGjwLn9567WTYO4c1eBAjiybsAlzEl088Sk8e RruidFmNObY5C4nyJw4coGI/LWNyuSsSP8ndRLmZm4WHBx0Q+T9VSMSEJ9XwpqxWjoWt gI/Ym7GVSOwVbpT3N5Wx+LJXqH24N7QEFFTLRtiHX245z35HSJbWBiaNYC+cP8xpcqXu qKjXGIRVYrPNxkV2wNMVCwqF28nhR2zS7jN91hHzwkpN+TlovUDclHmSPCSqDwvwcOJ4 yqTWjP1jpMPE7ty4yjWsMCKQUKyTFiHKlV40BCQVZlntKCw0uqXuToq55oUP0YZZg2UE 7Qpg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=qLQHKRlZ; 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 Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id l12-20020a37f90c000000b006fa239df48asi2190319qkj.318.2022.11.08.01.25.49 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Tue, 08 Nov 2022 01:25:49 -0800 (PST) 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=qLQHKRlZ; 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 Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1osKpO-0007HA-A4; Tue, 08 Nov 2022 04:23:30 -0500 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 1osKpE-0007ER-Up for qemu-devel@nongnu.org; Tue, 08 Nov 2022 04:23:21 -0500 Received: from mail-wr1-x432.google.com ([2a00:1450:4864:20::432]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1osKpB-0005pP-Cu for qemu-devel@nongnu.org; Tue, 08 Nov 2022 04:23:19 -0500 Received: by mail-wr1-x432.google.com with SMTP id j15so19957175wrq.3 for ; Tue, 08 Nov 2022 01:23:16 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; 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=U0bUaw8dD9ywC6S/a5dl4BRNFGfCaE47JrvKC4nt86Q=; b=qLQHKRlZM4PYVqEZYXi0VpdmRGdlZ4kHL1DfUgmDRmPx+hs3RbDnmmDe0x0C7X6UHc yrX+VUC6b6++fAqWTKBJtz9GBNm0zeK20VvolZx5neAaxggotvlSDoKeuMjW3BWYY6o+ Xq1L94ICZbTn70nhwazmNCSDbGMtswMRKHZnbDaBpA6qk3YzJn1AW0ubhANOKc4VkNSk lw6aDFQpezCMWwrfL/pXRTx5UuWxkUO2CiTj2uZXDhPxXk/aRS+f+OI5/hiUXE7FCRrL cuekS2mYSk5YcuSFfZHStgOjVvfd2J7x5xO5fvj5kWsh5CI2AhnqXLSpE+CE/QLef6dG Gkmg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; 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=U0bUaw8dD9ywC6S/a5dl4BRNFGfCaE47JrvKC4nt86Q=; b=7/0VWrDf+Fl/XaBxZJ90SvSqkD5kuQZwWG60qmPTKIi0JwScGv+tFLe1fzjEUrz5aq rOna4q2ItXisbUJoIrBKzNPAmTYcSUSIFPAf8q6IfX3ad3RsiwZ1iAoxMMSCfMJC4MD0 HbTS/znfoiNT0dUKrzSN6iPKUnWwwcESCpyBtcDo5I0PnCYToWcuozNj+/adbSnaF/Zd YjuJGur9sAO9IDzRYPkmHy+OiazTMZGnkxDTYuJay9/MroHwWM0q2hiBFN3jGPc9U/Op QHNiIe40GzIwUlHf0QPf4/iR8qWLWHo+gQE70Slnw/ygYAgzCW6Sf2QcCvgJgiJfHkti /60Q== X-Gm-Message-State: ACrzQf34pEbzD00f2lFaKQ2PU0mr8w0PTvyvRdkhprybJbbZHAM1PEfg v3On8GLKyWHvGZe3vMz2f4vjww== X-Received: by 2002:a5d:4644:0:b0:236:cb94:4c6c with SMTP id j4-20020a5d4644000000b00236cb944c6cmr29928685wrs.544.1667899394963; Tue, 08 Nov 2022 01:23:14 -0800 (PST) Received: from zen.linaroharston ([185.81.254.11]) by smtp.gmail.com with ESMTPSA id u13-20020a5d434d000000b0023647841c5bsm9553376wrr.60.2022.11.08.01.23.09 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 08 Nov 2022 01:23:11 -0800 (PST) Received: from zen.lan (localhost [127.0.0.1]) by zen.linaroharston (Postfix) with ESMTP id E42401FFBF; Tue, 8 Nov 2022 09:23:08 +0000 (GMT) From: =?utf-8?q?Alex_Benn=C3=A9e?= To: qemu-devel@nongnu.org Cc: fam@euphon.net, berrange@redhat.com, f4bug@amsat.org, aurelien@aurel32.net, pbonzini@redhat.com, stefanha@redhat.com, crosa@redhat.com, =?utf-8?q?Alex_Benn=C3=A9e?= Subject: [PATCH v1 7/9] docs/devel: make language a little less code centric Date: Tue, 8 Nov 2022 09:23:06 +0000 Message-Id: <20221108092308.1717426-8-alex.bennee@linaro.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20221108092308.1717426-1-alex.bennee@linaro.org> References: <20221108092308.1717426-1-alex.bennee@linaro.org> MIME-Version: 1.0 Received-SPF: pass client-ip=2a00:1450:4864:20::432; envelope-from=alex.bennee@linaro.org; helo=mail-wr1-x432.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 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 We welcome all sorts of patches. Signed-off-by: Alex Bennée Reviewed-by: Stefan Hajnoczi Reviewed-by: Paolo Bonzini Message-Id: <20221012121152.1179051-3-alex.bennee@linaro.org> --- docs/devel/submitting-a-patch.rst | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/docs/devel/submitting-a-patch.rst b/docs/devel/submitting-a-patch.rst index fec33ce148..9c7c4331f3 100644 --- a/docs/devel/submitting-a-patch.rst +++ b/docs/devel/submitting-a-patch.rst @@ -3,11 +3,11 @@ Submitting a Patch ================== -QEMU welcomes contributions of code (either fixing bugs or adding new -functionality). However, we get a lot of patches, and so we have some -guidelines about submitting patches. If you follow these, you'll help -make our task of code review easier and your patch is likely to be -committed faster. +QEMU welcomes contributions to fix bugs, add functionality or improve +the documentation. However, we get a lot of patches, and so we have +some guidelines about submitting them. If you follow these, you'll +help make our task of code review easier and your patch is likely to +be committed faster. This page seems very long, so if you are only trying to post a quick one-shot fix, the bare minimum we ask is that: From patchwork Tue Nov 8 09:23:07 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Alex_Benn=C3=A9e?= X-Patchwork-Id: 622644 Delivered-To: patch@linaro.org Received: by 2002:a17:522:c983:b0:460:3032:e3c4 with SMTP id kr3csp2806317pvb; Tue, 8 Nov 2022 01:25:02 -0800 (PST) X-Google-Smtp-Source: AMsMyM4vaRSvB/4/bylIaogSGo0D5OnMym5mIb2ml6yKnleHourxEsLOUl1DUNxKbXpfo8DU9M6a X-Received: by 2002:a05:620a:2984:b0:6fa:6636:d4e5 with SMTP id r4-20020a05620a298400b006fa6636d4e5mr22813259qkp.622.1667899502493; Tue, 08 Nov 2022 01:25:02 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1667899502; cv=none; d=google.com; s=arc-20160816; b=MKnPfC/NxOTlET8mzQAGnb/HJ+wjP4W8fvD1oSTmBwVXiENJCsiPZon110/BmprVxz H7SAVbsDy32v72ebteLe6jRAXOeO2W/aLh8VREJXqA9LrcMaP6/t3fDFtBiqLZkOwO7K w1tFF35RRYt6xMFNHX13c/YzBI+KRVoT0rtjWYiZrR4phPrZU6txWFw1raRvL6Myaw1W vYomDD15XrQVNnPxlEkWt6sf0Atq43+hJTLDg+so7aBozCK2W43m4gqrlu5OflrgRv++ W0gOcIaL8QEp/nfaNKj8nV6S5m8SPVcupQekfciiMtOwsARRRghcOoJgQS0PQESm3jOe wEJA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; 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=Tx51MAAfbml8CiT50HNaT+ihFTBqcLj/nxt/MFriuNA=; b=Gn9UGKvQwymvNG8cszgSQ4bqF8CO4ihCDU0hGPr9MwsJZVqd/bZtmd+h6gQEPo5akk 59CISG2mMpb++3tNDDuHHYMUq4DM09lR3wZrohQ5qXauy63nY7WHCvlz0Zl5NmhFtH+G tKoVIrmSeIaD6OZ5kIZ1uCpuILFJoRt1SQfj6VMhdilaLF4aRJnW06d5hz3tquRzFnrp 7/wVuHVCc463l5k0ByIkuy3z9/hBifwIyoX2zO1Uss+kGGm4cvaJ+WJAb1iIZtwfkV8o bEW0XKGCPRHRVnbndd107XFptfB3rtILbuBEN7gx5xUAoI7g70VWVWu+po8zE7dpKCHp Lckw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=sFDh7eMo; 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 Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id b123-20020a37b281000000b006fa299566afsi5473325qkf.568.2022.11.08.01.25.02 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Tue, 08 Nov 2022 01:25:02 -0800 (PST) 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=sFDh7eMo; 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 Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1osKpU-0007Kd-92; Tue, 08 Nov 2022 04:23:36 -0500 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 1osKpE-0007ES-UR for qemu-devel@nongnu.org; Tue, 08 Nov 2022 04:23:21 -0500 Received: from mail-wr1-x436.google.com ([2a00:1450:4864:20::436]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1osKpB-0005pX-Fg for qemu-devel@nongnu.org; Tue, 08 Nov 2022 04:23:19 -0500 Received: by mail-wr1-x436.google.com with SMTP id j15so19957242wrq.3 for ; Tue, 08 Nov 2022 01:23:17 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; 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=Tx51MAAfbml8CiT50HNaT+ihFTBqcLj/nxt/MFriuNA=; b=sFDh7eMoitvWm8ag6HcNN0yyPQq8J//5Ar0LNIN1AFkFgIhz/XZCkHFrNbsWf7x2d0 rO2vSXOsn6GrE3qA8cewrnxQ6so01LnsZj7zMW/iQ2HwPQ517XeEIhrq3NvVaJqidkIw goD3ISbDssJpS5BKPqOOFfwmLU7U4p6Yd+YlTb+TZhdp79yIW7NIeqPAtn1mRE1oD0wG zVyBfinh+8BhUW2EpLRPewJkJOOXXB67xMpiSmjtUDlVgTcZfLXYJH8wgoOkmVr3/Xyi iIOsGs5ENHAArIbOwsc49OUfhoyu/Kmtftz/y4uoDi1AlZFizNBHwxtg3SuX5bocLss5 c3HQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; 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=Tx51MAAfbml8CiT50HNaT+ihFTBqcLj/nxt/MFriuNA=; b=lApGInWOuERg1279BZnBJehIikalVwZGD/wtH8BC7TwPtlOgCDpkPaBXAuiSGlK0vE 2XZpz8/JbIvmNAUPnx73WdVX8PYUTKYeAlWUcyuDwIG28V7ug1c3STPvxrWzc237dfWx uIXu4kBjd5kZL+XKq9CrJaAE3tBsgaRUP9XO38UOLRxQlXc37YFsk55phJ8vie5pWv8v hhC4xr3er3bbr3eaOxOJsocwluSNVwPgzYC4n7J+6e6u8bILCZoNMm80XnFn8Virnca6 dBGTaL7AW5PLmevfR47opFXN3LBAjACIogpf/OMlasy0xTUUElV9jWyEVw99uhQ6BhiJ 9tMA== X-Gm-Message-State: ACrzQf18q1/BtFYSRMX9bk3t5GiTrI1Q6v2YBsaLzehTgZXMUrwjOPRt 9QS950QICnGk7cxYMohl0VKeuA== X-Received: by 2002:a05:6000:10a:b0:236:6a79:f5cf with SMTP id o10-20020a056000010a00b002366a79f5cfmr33346242wrx.470.1667899395937; Tue, 08 Nov 2022 01:23:15 -0800 (PST) Received: from zen.linaroharston ([185.81.254.11]) by smtp.gmail.com with ESMTPSA id k4-20020a05600c168400b003c6f1732f65sm13660697wmn.38.2022.11.08.01.23.09 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 08 Nov 2022 01:23:11 -0800 (PST) Received: from zen.lan (localhost [127.0.0.1]) by zen.linaroharston (Postfix) with ESMTP id 07DFE1FFC0; Tue, 8 Nov 2022 09:23:09 +0000 (GMT) From: =?utf-8?q?Alex_Benn=C3=A9e?= To: qemu-devel@nongnu.org Cc: fam@euphon.net, berrange@redhat.com, f4bug@amsat.org, aurelien@aurel32.net, pbonzini@redhat.com, stefanha@redhat.com, crosa@redhat.com, =?utf-8?q?Alex_Benn=C3=A9e?= Subject: [PATCH v1 8/9] docs/devel: simplify the minimal checklist Date: Tue, 8 Nov 2022 09:23:07 +0000 Message-Id: <20221108092308.1717426-9-alex.bennee@linaro.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20221108092308.1717426-1-alex.bennee@linaro.org> References: <20221108092308.1717426-1-alex.bennee@linaro.org> MIME-Version: 1.0 Received-SPF: pass client-ip=2a00:1450:4864:20::436; envelope-from=alex.bennee@linaro.org; helo=mail-wr1-x436.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 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 The bullet points are quite long and contain process tips. Move those bits of the bullet to the relevant sections and link to them. Use a table for nicer formatting of the checklist. Signed-off-by: Alex Bennée Reviewed-by: Stefan Hajnoczi Reviewed-by: Paolo Bonzini Message-Id: <20221012121152.1179051-4-alex.bennee@linaro.org> --- v2 - emphasise a Real Name should be used - s/therefor/therefore/ --- docs/devel/submitting-a-patch.rst | 75 ++++++++++++++++++++----------- 1 file changed, 49 insertions(+), 26 deletions(-) diff --git a/docs/devel/submitting-a-patch.rst b/docs/devel/submitting-a-patch.rst index 9c7c4331f3..1f2bde0625 100644 --- a/docs/devel/submitting-a-patch.rst +++ b/docs/devel/submitting-a-patch.rst @@ -12,25 +12,18 @@ be committed faster. This page seems very long, so if you are only trying to post a quick one-shot fix, the bare minimum we ask is that: -- You **must** provide a Signed-off-by: line (this is a hard - requirement because it's how you say "I'm legally okay to contribute - this and happy for it to go into QEMU", modeled after the `Linux kernel - `__ - policy.) ``git commit -s`` or ``git format-patch -s`` will add one. -- All contributions to QEMU must be **sent as patches** to the - qemu-devel `mailing list `__. - Patch contributions should not be posted on the bug tracker, posted on - forums, or externally hosted and linked to. (We have other mailing lists too, - but all patches must go to qemu-devel, possibly with a Cc: to another - list.) ``git send-email`` (`step-by-step setup - guide `__ and `hints and - tips `__) - works best for delivering the patch without mangling it, but - attachments can be used as a last resort on a first-time submission. -- You must read replies to your message, and be willing to act on them. - Note, however, that maintainers are often willing to manually fix up - first-time contributions, since there is a learning curve involved in - making an ideal patch submission. +.. list-table:: Minimal Checklist for Patches + :widths: 35 65 + :header-rows: 1 + + * - Check + - Reason + * - Patches contain Signed-off-by: Real Name + - States you are legally able to contribute the code. See :ref:`patch_emails_must_include_a_signed_off_by_line` + * - Sent as patch emails to ``qemu-devel@nongnu.org`` + - The project uses an email list based workflow. See :ref:`submitting_your_patches` + * - Be prepared to respond to review comments + - Code that doesn't pass review will not get merged. See :ref:`participating_in_code_review` You do not have to subscribe to post (list policy is to reply-to-all to preserve CCs and keep non-subscribers in the loop on the threads they @@ -229,6 +222,19 @@ bisection doesn't land on a known-broken state. Submitting your Patches ----------------------- +The QEMU project uses a public email based workflow for reviewing and +merging patches. As a result all contributions to QEMU must be **sent +as patches** to the qemu-devel `mailing list +`__. Patch +contributions should not be posted on the bug tracker, posted on +forums, or externally hosted and linked to. (We have other mailing +lists too, but all patches must go to qemu-devel, possibly with a Cc: +to another list.) ``git send-email`` (`step-by-step setup guide +`__ and `hints and tips +`__) +works best for delivering the patch without mangling it, but +attachments can be used as a last resort on a first-time submission. + .. _if_you_cannot_send_patch_emails: If you cannot send patch emails @@ -314,10 +320,12 @@ git repository to fetch the original commit. Patch emails must include a ``Signed-off-by:`` line ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ -For more information see `SubmittingPatches 1.12 -`__. -This is vital or we will not be able to apply your patch! Please use -your real name to sign a patch (not an alias or acronym). +Your patches **must** include a Signed-off-by: line. This is a hard +requirement because it's how you say "I'm legally okay to contribute +this and happy for it to go into QEMU". The process is modelled after +the `Linux kernel +`__ +policy. If you wrote the patch, make sure your "From:" and "Signed-off-by:" lines use the same spelling. It's okay if you subscribe or contribute to @@ -327,6 +335,11 @@ include a "From:" line in the body of the email (different from your envelope From:) that will give credit to the correct author; but again, that author's Signed-off-by: line is mandatory, with the same spelling. +There are various tooling options for automatically adding these tags +include using ``git commit -s`` or ``git format-patch -s``. For more +information see `SubmittingPatches 1.12 +`__. + .. _include_a_meaningful_cover_letter: Include a meaningful cover letter @@ -397,9 +410,19 @@ Participating in Code Review ---------------------------- All patches submitted to the QEMU project go through a code review -process before they are accepted. Some areas of code that are well -maintained may review patches quickly, lesser-loved areas of code may -have a longer delay. +process before they are accepted. This will often mean a series will +go through a number of iterations before being picked up by +:ref:`maintainers`. You therefore should be prepared to +read replies to your messages and be willing to act on them. + +Maintainers are often willing to manually fix up first-time +contributions, since there is a learning curve involved in making an +ideal patch submission. However for the best results you should +proactively respond to suggestions with changes or justifications for +your current approach. + +Some areas of code that are well maintained may review patches +quickly, lesser-loved areas of code may have a longer delay. .. _stay_around_to_fix_problems_raised_in_code_review: From patchwork Tue Nov 8 09:23:08 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Alex_Benn=C3=A9e?= X-Patchwork-Id: 622646 Delivered-To: patch@linaro.org Received: by 2002:a17:522:c983:b0:460:3032:e3c4 with SMTP id kr3csp2807169pvb; Tue, 8 Nov 2022 01:26:43 -0800 (PST) X-Google-Smtp-Source: AMsMyM7Q8FqBbgRRHmro3dZLuVOdhPHTJ/NdD2xyMvFvmklA6J+DmlDDEJr06S5NDJFSBX5c5HO1 X-Received: by 2002:a05:6214:d84:b0:4bb:ae9f:63de with SMTP id e4-20020a0562140d8400b004bbae9f63demr50528519qve.97.1667899603707; Tue, 08 Nov 2022 01:26:43 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1667899603; cv=none; d=google.com; s=arc-20160816; b=YgdenmGUJSDpIPCqtqgHSAwakRk2Bh8HxaVOuL8hTjPy/+u10O6xRmQ+hMqJHnuoC1 0p8egfFusi1DyKOqp74YCnzqEFoPuabpgd4YqlCeqjBJHVyKxKWEVpPtzTQcjh/xVmXc bnDjeA/1UpvbourwCOFm3zdhkidQ2M653u+i0NZvVWwddh2tw1pIUXm/qowLkOcAlcLu hU/3SSeQ2Aq2PSgwUkrG8XaGNkqx14XV+tXyvG9Wkvtc5FDUhvt7skvGQnLJgYr6XblV 35dUnJIwLKcDC/HmmUACz5qqRqrnxMbxYBa3e0DU8JClD1a8UGSofi69lOaK51FFuwVx blpw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; 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=yLNK2qdPNMQMvzK3A+Wa0kPh913JqNLXh3QyXKK7U+k=; b=Om/FqCQiYCKk09gExo58Q2AwUYUZ6YV3B37Cdq4aQzYYrUDWPOPxKIJWNnFW6NdrE/ 4LEG0DOdv4MrI2Y9Kk3UlMcORoveKo9A7H/pGor8F1nthddxdwfhay/0wISeFi/TgvI4 JEvIFcnKzDm4HUgsQDAPNvejekzW6OpkUQdX/nqStkzFo//im6jmiD+60kpmVVGVQjJD aODfXc2/vXAjffspq0Xaoi8+g20RTUC6bA9WpjIcJ8Ys5VtcyhPT+E/HDncfpkgUt6p1 2fYOetG+PrxyAocepxwcweF2bwiKkVxxDOTDMhZ1ZC8icpu3M4094gwLVBz02W4NnKNx 0xDw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=u2Z+YKLP; 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 Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id ed8-20020ad44ea8000000b004bdf0bb28besi5601576qvb.418.2022.11.08.01.26.43 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Tue, 08 Nov 2022 01:26:43 -0800 (PST) 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=u2Z+YKLP; 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 Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1osKpM-0007H0-Ql; Tue, 08 Nov 2022 04:23:28 -0500 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 1osKpE-0007EQ-U1 for qemu-devel@nongnu.org; Tue, 08 Nov 2022 04:23:21 -0500 Received: from mail-wr1-x436.google.com ([2a00:1450:4864:20::436]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1osKpC-0005pq-CD for qemu-devel@nongnu.org; Tue, 08 Nov 2022 04:23:20 -0500 Received: by mail-wr1-x436.google.com with SMTP id a14so19964651wru.5 for ; Tue, 08 Nov 2022 01:23:17 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; 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=yLNK2qdPNMQMvzK3A+Wa0kPh913JqNLXh3QyXKK7U+k=; b=u2Z+YKLP0aPMdHuzajDgrxdi213GM3D/1hKyz/jCvCw/YwUnZcRGbGdCrA/QaxQV/L lCDYOSdZYdgIvQtq5BcwgZ+yzqmTEFC1YHxYGLR1yVNfDi7QdMpWSTAzNG7yH7mo9vBf +2114x2Hm1HY+aXxapYEezSSy3hOUP3fhmTSNPEpuCXlIEBUBvjkC2UMBdOnpMgbbGG2 9C70FXtVJ5ipSUFa3MG2Fk94retDjxzbYGLYXUZc+rXbOZ4CA/rkfQO5emnq7C6ccSQx FhQwHLNfteubtBZ32iwVR2r7UreoQNGClrzNlVMlrGwT/WxWmgYVZIrVEc9pqWXTmUzK yyaQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; 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=yLNK2qdPNMQMvzK3A+Wa0kPh913JqNLXh3QyXKK7U+k=; b=NOvnYx91TKV6zvHsgiDB7HQufD6Hae+txjtEiY4tiJ5fte9yz8y72Q8OagH1tUMzI+ JQLzsQl0GusE/BLKg673ZKgvaoHYBdYOzi0yKuQ0rR9zSop7lXW9/uo0Ou7CUhFGBVSJ 8Lwg6LW6I0X4AzXqotavkI8yrGLZZRqOGN+Ad85gJKWKxG+VDqCjo1f2H2CVlSwiMn5c i9Zh9jsKtyU0dJPD5+QbtKh91M9J6pSo7ZZkt1PHKJCDRT0h3+/MnZcRU3LxHwiBDppp 0FkQDrMcTA26M8H5ZY+EuYwQqvH7q1qGWwE9hrH4wpTlHMJw6lwM1vHiHKKDJb4x3Kbo V3Cw== X-Gm-Message-State: ACrzQf0RemTPOiSeJiBci3pLyBwA3niN6oiKmEve6tV0t6AxNJbk3rQZ EwoEoV+NY40/SJWbK2NwUJNyqF88xaWclA== X-Received: by 2002:a5d:644b:0:b0:236:862b:9ef1 with SMTP id d11-20020a5d644b000000b00236862b9ef1mr34495500wrw.691.1667899396924; Tue, 08 Nov 2022 01:23:16 -0800 (PST) Received: from zen.linaroharston ([185.81.254.11]) by smtp.gmail.com with ESMTPSA id s7-20020adfeb07000000b0022a3a887ceasm9538905wrn.49.2022.11.08.01.23.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 08 Nov 2022 01:23:13 -0800 (PST) Received: from zen.lan (localhost [127.0.0.1]) by zen.linaroharston (Postfix) with ESMTP id 1E7871FFC1; Tue, 8 Nov 2022 09:23:09 +0000 (GMT) From: =?utf-8?q?Alex_Benn=C3=A9e?= To: qemu-devel@nongnu.org Cc: fam@euphon.net, berrange@redhat.com, f4bug@amsat.org, aurelien@aurel32.net, pbonzini@redhat.com, stefanha@redhat.com, crosa@redhat.com, =?utf-8?q?Alex_Benn=C3=A9e?= , Markus Armbruster Subject: [PATCH v1 9/9] docs/devel: try and improve the language around patch review Date: Tue, 8 Nov 2022 09:23:08 +0000 Message-Id: <20221108092308.1717426-10-alex.bennee@linaro.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20221108092308.1717426-1-alex.bennee@linaro.org> References: <20221108092308.1717426-1-alex.bennee@linaro.org> MIME-Version: 1.0 Received-SPF: pass client-ip=2a00:1450:4864:20::436; envelope-from=alex.bennee@linaro.org; helo=mail-wr1-x436.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 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 It is important that contributors take the review process seriously and we collaborate in a respectful way while avoiding personal attacks. Try and make this clear in the language. Signed-off-by: Alex Bennée Reviewed-by: Markus Armbruster Reviewed-by: Stefan Hajnoczi Reviewed-by: Paolo Bonzini Message-Id: <20221012121152.1179051-5-alex.bennee@linaro.org> --- docs/devel/submitting-a-patch.rst | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/docs/devel/submitting-a-patch.rst b/docs/devel/submitting-a-patch.rst index 1f2bde0625..80e8693bb6 100644 --- a/docs/devel/submitting-a-patch.rst +++ b/docs/devel/submitting-a-patch.rst @@ -434,14 +434,20 @@ developers will identify bugs, or suggest a cleaner approach, or even just point out code style issues or commit message typos. You'll need to respond to these, and then send a second version of your patches with the issues fixed. This takes a little time and effort on your part, but -if you don't do it then your changes will never get into QEMU. It's also -just polite -- it is quite disheartening for a developer to spend time -reviewing your code and suggesting improvements, only to find that -you're not going to do anything further and it was all wasted effort. +if you don't do it then your changes will never get into QEMU. + +Remember that a maintainer is under no obligation to take your +patches. If someone has spent the time reviewing your code and +suggesting improvements and you simply re-post without either +addressing the comment directly or providing additional justification +for the change then it becomes wasted effort. You cannot demand others +merge and then fix up your code after the fact. When replying to comments on your patches **reply to all and not just the sender** -- keeping discussion on the mailing list means everybody -can follow it. +can follow it. Remember the spirit of the :ref:`code_of_conduct` and +keep discussions respectful and collaborative and avoid making +personal comments. .. _pay_attention_to_review_comments: