From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id C5CE7F588E2 for ; Mon, 20 Apr 2026 14:47:11 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 2F1FC10E694; Mon, 20 Apr 2026 14:47:11 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=fail reason="signature verification failed" (2048-bit key; secure) header.d=infradead.org header.i=@infradead.org header.b="Y8RUxH+d"; dkim-atps=neutral Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) by gabe.freedesktop.org (Postfix) with ESMTPS id 3A5C810E694 for ; Mon, 20 Apr 2026 14:47:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=desiato.20200630; h=In-Reply-To:Content-Transfer-Encoding: Content-Type:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date: Sender:Reply-To:Content-ID:Content-Description; bh=eCyjhK/Q5fvtzDJtYR2G5sGLVBYO/Uljn2Rnbl90Gu0=; b=Y8RUxH+dN4uV3ck9NPBEDW9CGM fgtqPHdQaI5QFweHYVIYYEEuBsHHRv6yywnyvpCwRBGLzYy/o7bvAcPVG1Ymao+JTWrvwvVZnTwVm IN21XtGPpXtODoG9r4/dBnrjnRWgSij6UTDsxuArM4siJySJi1QR11AhSta/rwWUzR00ipJ8xCNf5 9NXwEoh98VnqRKI7YdyJ0qYcFFjr7ctuQ5/prJtTJ6muED0U+YWgDgXkgXhJbkFc7OVRZmVrYGQYz GRd63Qs3K7/OG8khcL8MmtGYcH9hTyO6FhamVrohS+Miz4IsxuND1CLL6wiXVaLeaNMKl5hLkOZOx WOzwBaBA==; Received: from 2001-1c00-8d85-4b00-266e-96ff-fe07-7dcc.cable.dynamic.v6.ziggo.nl ([2001:1c00:8d85:4b00:266e:96ff:fe07:7dcc] helo=noisy.programming.kicks-ass.net) by desiato.infradead.org with esmtpsa (Exim 4.98.2 #2 (Red Hat Linux)) id 1wEptv-00000008Dyi-18py; Mon, 20 Apr 2026 14:47:03 +0000 Received: by noisy.programming.kicks-ass.net (Postfix, from userid 1000) id C9CC2301BDE; Mon, 20 Apr 2026 16:47:02 +0200 (CEST) Date: Mon, 20 Apr 2026 16:47:02 +0200 From: Peter Zijlstra To: Albert Esteve Cc: Arnd Bergmann , Brendan Higgins , David Gow , Rae Moar , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Jonathan Corbet , Shuah Khan , Andrew Morton , linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org, linux-kselftest@vger.kernel.org, kunit-dev@googlegroups.com, dri-devel@lists.freedesktop.org, workflows@vger.kernel.org, linux-doc@vger.kernel.org, Guenter Roeck , Linux Kernel Functional Testing , Dan Carpenter , =?iso-8859-1?Q?Ma=EDra?= Canal , Alessandro Carminati , Simona Vetter Subject: Re: [PATCH v7 4/5] drm: Suppress intentional warning backtraces in scaling unit tests Message-ID: <20260420144702.GM3102624@noisy.programming.kicks-ass.net> References: <20260420-kunit_add_support-v7-0-e8bc6e0f70de@redhat.com> <20260420-kunit_add_support-v7-4-e8bc6e0f70de@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20260420-kunit_add_support-v7-4-e8bc6e0f70de@redhat.com> X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" On Mon, Apr 20, 2026 at 02:28:06PM +0200, Albert Esteve wrote: > From: Guenter Roeck > > The drm_test_rect_calc_hscale and drm_test_rect_calc_vscale unit tests > intentionally trigger warning backtraces by providing bad parameters to > the tested functions. What is tested is the return value, not the existence > of a warning backtrace. Suppress the backtraces to avoid clogging the > kernel log and distraction from real problems. > > Tested-by: Linux Kernel Functional Testing > Acked-by: Dan Carpenter > Acked-by: Maíra Canal > Cc: Maarten Lankhorst > Cc: David Airlie > Cc: Daniel Vetter > Signed-off-by: Guenter Roeck > Signed-off-by: Alessandro Carminati > Signed-off-by: Albert Esteve > --- > drivers/gpu/drm/tests/drm_rect_test.c | 14 ++++++++++++++ > 1 file changed, 14 insertions(+) > > diff --git a/drivers/gpu/drm/tests/drm_rect_test.c b/drivers/gpu/drm/tests/drm_rect_test.c > index 17e1f34b76101..1dd7d819165e7 100644 > --- a/drivers/gpu/drm/tests/drm_rect_test.c > +++ b/drivers/gpu/drm/tests/drm_rect_test.c > @@ -409,8 +409,15 @@ static void drm_test_rect_calc_hscale(struct kunit *test) > const struct drm_rect_scale_case *params = test->param_value; > int scaling_factor; > > + /* > + * drm_rect_calc_hscale() generates a warning backtrace whenever bad > + * parameters are passed to it. This affects all unit tests with an > + * error code in expected_scaling_factor. > + */ > + KUNIT_START_SUPPRESSED_WARNING(test); > scaling_factor = drm_rect_calc_hscale(¶ms->src, ¶ms->dst, > params->min_range, params->max_range); > + KUNIT_END_SUPPRESSED_WARNING(test); Would not something like: scoped_kunit_suppress() { scaling_factor = drm_rect_calc_hscale(¶ms->src, ¶ms->dst, params->min_range, params->max_range); } be better? Also, how can you stand all this screaming in the code?