From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 435DA30F83; Thu, 16 Nov 2023 13:59:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="zWuN6oCT" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 7BF8AC433C8; Thu, 16 Nov 2023 13:59:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1700143157; bh=oreDXTUFD1p7EOCadmsrNnBKYaZXlHAjodag6YCr5sw=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=zWuN6oCTXkMwNE1HalrHk0dL6JJ8Ftpu1Xvijzw1uQn9ypVpax/WXZjD1QFBmkLsE 7EEYSUUDX3B6/9BC9tYRVezGFj/V60Xt3IvQSHsbl9/N1ZBVfWPhyqCgJPwPbFkMMy sMUma/KePpFUutEufuw34VLxQDJI4skUef9Rk+/0= Date: Thu, 16 Nov 2023 08:59:13 -0500 From: Greg Kroah-Hartman To: Bagas Sanjaya Cc: Nikolai Kondrashov , workflows@vger.kernel.org, Joe Perches , Andy Whitcroft , Theodore Ts'o , David Gow , Steven Rostedt , Mark Brown , Shuah Khan , "Darrick J . Wong" , Charles Han , Linux Kernel Unit Tests , Linux Kernel Self Tests , Veronika Kabatova , CKI , kernelci@lists.linux.dev Subject: Re: [PATCH 1/3] MAINTAINERS: Introduce V: field for required tests Message-ID: <2023111601-unsecured-decode-3fc1@gregkh> References: <20231115175146.9848-1-Nikolai.Kondrashov@redhat.com> <20231115175146.9848-2-Nikolai.Kondrashov@redhat.com> Precedence: bulk X-Mailing-List: kernelci@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Thu, Nov 16, 2023 at 08:20:18PM +0700, Bagas Sanjaya wrote: > On Wed, Nov 15, 2023 at 07:43:49PM +0200, Nikolai Kondrashov wrote: > > Make scripts/checkpatch.pl ensure any added V: fields reference > > documented test suites only, and output a warning if a change to a > > subsystem doesn't certify the required test suites were executed, > > if any. > > > > If the test suite description includes a "Command", then checkpatch.pl > > will output it as the one executing the suite. The command should run > > with only the kernel tree and the regular developer environment set up. > > But, at the same time, could simply output instructions for installing > > any extra dependencies (or pull some automatically). The idea is to > > get the developer into feedback loop quicker and easier, so they have > > something to run and iterate on, even if it involves installing some > > more stuff first. Therefore it's a good idea to add such wrappers to the > > kernel tree proper and refer to them from the tests.rst. > > Does it also apply to trivial patches (e.g. spelling or checkpatch fixes > as seen on drivers/staging/)? You are assuming that drivers/staging/ has actual tests :)