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 kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 05DCCC02185 for ; Sat, 18 Jan 2025 17:10:05 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 3E5756B0082; Sat, 18 Jan 2025 12:10:05 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 3962D6B0083; Sat, 18 Jan 2025 12:10:05 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 235D86B0085; Sat, 18 Jan 2025 12:10:05 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0010.hostedemail.com [216.40.44.10]) by kanga.kvack.org (Postfix) with ESMTP id 03F7E6B0082 for ; Sat, 18 Jan 2025 12:10:04 -0500 (EST) Received: from smtpin25.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay06.hostedemail.com (Postfix) with ESMTP id 9A54BB1901 for ; Sat, 18 Jan 2025 17:10:04 +0000 (UTC) X-FDA: 83021210328.25.4DF9655 Received: from mail-wr1-f48.google.com (mail-wr1-f48.google.com [209.85.221.48]) by imf03.hostedemail.com (Postfix) with ESMTP id A23D120005 for ; Sat, 18 Jan 2025 17:10:02 +0000 (UTC) Authentication-Results: imf03.hostedemail.com; dkim=pass header.d=gmail.com header.s=20230601 header.b=XBJnOm3L; spf=pass (imf03.hostedemail.com: domain of david.laight.linux@gmail.com designates 209.85.221.48 as permitted sender) smtp.mailfrom=david.laight.linux@gmail.com; dmarc=pass (policy=none) header.from=gmail.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1737220202; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=3KlgTW8qoEnUZT7EjjZ/u9PK9WlkNroef5My8RmY27k=; b=UQGJAt/X42vi8g+i5nEIT4Fue4BDPeAT72rtyBFCklMP200D3mRUgWePwF2z8Pb4ZbnFR6 8I8A1QHlFSvtftmavn7W9cqJv7nYptPLeF1Uhoa1Ud2SLf0SL/v0SrZ/AzTATN4e+/MeQA sjXzWJm+F0j3cSXc5FZWl9WMXEj1Z/c= ARC-Authentication-Results: i=1; imf03.hostedemail.com; dkim=pass header.d=gmail.com header.s=20230601 header.b=XBJnOm3L; spf=pass (imf03.hostedemail.com: domain of david.laight.linux@gmail.com designates 209.85.221.48 as permitted sender) smtp.mailfrom=david.laight.linux@gmail.com; dmarc=pass (policy=none) header.from=gmail.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1737220202; a=rsa-sha256; cv=none; b=bo/9gYOfYEK8JIOv8TqmD6UbbxkCBcYBAQPXwH7n8y1vYzGJGBnO6TeYlw16Z6JtfDMGnX 9WSbuwfcow1vek3iveT88mSvWhurAGyTniQXi6oIyT5yb37MGLSYkeJ2/dRFDLa/cSSRb/ yFo92LqxBmRVVTqzIdAfnQFKEurEesk= Received: by mail-wr1-f48.google.com with SMTP id ffacd0b85a97d-385e1fcb0e1so1618435f8f.2 for ; Sat, 18 Jan 2025 09:10:02 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1737220201; x=1737825001; darn=kvack.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:from:to:cc:subject:date :message-id:reply-to; bh=3KlgTW8qoEnUZT7EjjZ/u9PK9WlkNroef5My8RmY27k=; b=XBJnOm3LloFBqiyBIpNKYTm1WLdCtccXZADItGnfdshFaXJkvhTQp+pNROvWxvH1SO pow8OgI74NYEUVuXJVCsfoit8z79w60gTSpsj8L83X68QGW9qgeGylOun2KNH1uyE3JB RefATQpHI+bT+bULirbF0lo4Jsw2y/daajGkz0JB4/hR43CoEetgq23T/El+Hj/jA0Ii SZ3KN/jgTSXlD3hCdyG28GxFNFSZ72TYci5NiXDtaS9iO2pgATDncjObPrQbKJYVie6Q SHfXoi6pNZpiGXMHmlDK5Tye2KX+CHJJQf/LL7/MSd2MgczIiQXyuCf6VXjc+JYePasb CKww== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1737220201; x=1737825001; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=3KlgTW8qoEnUZT7EjjZ/u9PK9WlkNroef5My8RmY27k=; b=PY0ma9JliOg/qSVc8jUz984NfD6rhF/YoPstA2B19c4lNBLLHOWAGPJlF4vz923Uji Zcg0HmTmd91XA6Y1RrexKICWI7TcZie06X8HiI6yMYkTbej8dSOyTykREzKl98X+NHhG 2cNPFZyT+4VKT5hqpGpcyy4oIb+SHdjmid9erhTURJ8S75RFCkqPEZ/ZBccG9P0Aq+qY IwPAKRXZ8sGOFTH8Kbey6MHcyob5TCTOTdDhgtA+/l6/JpErzk9Z8dwg2W6UQU9HVDjF ZkiRY8sx7EcOuOaeUp1nUTRapQ0Oc2YZzs9OzwHnDCkL6fLN4FPgL85zWna1/UwV60PH 6uww== X-Forwarded-Encrypted: i=1; AJvYcCVZfjL7+RvdxLppwlK5VDO/7aJx74mAH1sF3IGgR3o5v3WA6MMWRi4vQBOAVCPmg63IQIzNSnp2Zg==@kvack.org X-Gm-Message-State: AOJu0Yy9uDjkhcRuSZoXTrz/WlRwAZdM6nlmXoqIZ42ooy+wyuXWZDse 8WXfHPD6wfPw/7RM0ZABYIjo4rhyATbd+FLXr7YArED0wSF7VU1j X-Gm-Gg: ASbGncsEy20mE5xzU6HuseBGuFii91oRRdUsCbtXM4v4jopPFCaFpyOy1nvTdfmN8gR OdxrGMWZkUGDf4ZZ74lyNe1VBpGiQW+ewq3vxGpnUbfWJVGpYqZ2yhs+bEcw0YJkBpX5WoafmBD m6ubdFF0alP5iF229grTNk7vGlDte0+5KAHVUmCtnnN+gO3RwnKhazPDE0BGsswh9Y+HcgwzoC9 Trbs9KY14jftgaFts19wQ4W50GQwCIvFTPcNzscStOHJnFu7ol+CVILYimCkqZt6U2iiWl+KRiB FCshIBpvieg6Czrtne+UR67iUmixu5dT X-Google-Smtp-Source: AGHT+IHMW6aZx7u19ZQpXVb/KS9UCuBHquSRSELoo8MnVynv6LS9GzSLQvXkC5ETEtQhY9v53d8rGw== X-Received: by 2002:a5d:6d86:0:b0:385:df6b:7ef6 with SMTP id ffacd0b85a97d-38bf57c94a2mr8056076f8f.51.1737220200795; Sat, 18 Jan 2025 09:10:00 -0800 (PST) Received: from pumpkin (82-69-66-36.dsl.in-addr.zen.co.uk. [82.69.66.36]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-438904087cbsm75613495e9.3.2025.01.18.09.09.59 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 18 Jan 2025 09:10:00 -0800 (PST) Date: Sat, 18 Jan 2025 17:09:59 +0000 From: David Laight To: Guenter Roeck Cc: David Laight , Linus Torvalds , 'Arnd Bergmann' , "'linux-kernel@vger.kernel.org'" , 'Jens Axboe' , 'Matthew Wilcox' , 'Christoph Hellwig' , 'Andrew Morton' , 'Andy Shevchenko' , 'Dan Carpenter' , "'Jason A . Donenfeld'" , "'pedro.falcato@gmail.com'" , 'Mateusz Guzik' , "'linux-mm@kvack.org'" , 'Lorenzo Stoakes' , intel-xe@lists.freedesktop.org, intel-gfx@lists.freedesktop.org, David Airlie , Simona Vetter , Jani Nikula , Rodrigo Vivi Subject: Re: Buiild error in i915/xe (was: [PATCH next 4/7] minmax.h: Use BUILD_BUG_ON_MSG() for the lo < hi test in clamp()) Message-ID: <20250118170959.3aa56f4d@pumpkin> In-Reply-To: References: <34d53778977747f19cce2abb287bb3e6@AcuMS.aculab.com> X-Mailer: Claws Mail 4.1.1 (GTK 3.24.38; arm-unknown-linux-gnueabihf) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Rspamd-Server: rspam05 X-Rspamd-Queue-Id: A23D120005 X-Stat-Signature: sxgbpwwu8pkc85rf9rfe7n1dkgmwd8xs X-Rspam-User: X-HE-Tag: 1737220202-557238 X-HE-Meta: U2FsdGVkX1/fgxeDdulKx+b2o6jW4svS/MYV4JK6YX2lVxwt7AIJiUwJneL6ixoD9gvk94tlpG1nGeXKvruK+Y2LuYNvmbAZ+p/+BBYF1TXiuqXweWeYGrjY3f9qrHZ1VjDzfz9d+YRlhmp3XvHdFQTz2VshNVlue72MQ+aWtqtq+B//arc1PIWYiVhG+J+FY18vYVLRLeMCj4sySVGY1S7TbRbUt+vCpMm+1SzOtMPgIY4CV6sbq4X6g8Vld6igKiqr+u9YFANFmRwbeMfLV826uQiOEin9knAYLBL8V7l1QDx+Frr72uzWEqCekfwa9AFv3uIuWzARWKMphW8mawQyszq6RnIJd7PsyCQQ0XmNB7qxDJemSBi9TKRPp0AqyRu4eMpoJDaC8p0NewsFisz/nDdf5QQPZeuGqHI3oyUpNWN7oIaL0Q+NPVSQ7R5Kr/8CRHmoTld3eVovImCYJZC9aBoVYRn/ogemlJV5yKN8r5Xw5e1RfhQfRLl3U48CPwvAlI0Y24CCMJIFOPjrfJiaME6ZZH8z4dmOc++0B8zmXAF1A3YbAcqBv6gvVofXvmgQwCgRNcOicm7/Mm7Obr5UyfhNj+PjKGpR06mQVSHBQVkO6b4e7VBz69pZkETZlfvqbvBUz2KBdBYaSDwpqslowsUEYLD3tGn0mr56neuSQA4X9hZ45GcwYwJPpZp+KseV7cCEifKG9HWTogZ+SSXW/W+roqnLEB7cAjJMkbNUyPg99r2I1RIBTpXTiWEwBxvUlV2Mllxtk/pu+d1+tt7gbmkIbaZ3i7HgVSf5sYrkQFZj9SveW5x6HvPQJYJWvNLKuZcml8enVxULtNeaN6/dMMThLvdlFBkSDqn0ItNALqXdlJ1qwEULXcPjPkQv3P2YrVargKmF3vuS64coNcDzqIlYz1FhjjGj70m5DaC2ak7mPEX67A53Wj5Qzidq2EQ8wRPZbp6z0ztXJ8d 86K9AcVV 0iY82V7ZZgQF0yoZ5BOqKIB6zm8VDYnun+Tr7TNfCzB0BtIQW1HP7LKWiwi6TNeF7HSJh7mB2ihcI4ct/MQqDRiJ6g+bn88FhxpWJKvhmvAhKCPRhyClMGgBLAGKxyh8Y2lqu+0lj7SqmjC3BLy3jraNc6LHtjzo65LW1YF2JCWEWNlPi/4zpv3Mwwucw76rsvm135kmxbSeblP8VBzRM2T3Om+XfjhzczS+n1vfFsIAu8MrQN2QOD1sDYb45/qcaCQ+FgF8KlnroQnWzLebfHEkovULRrU7gqHzZR0fBBg1GSMAOhna9o95XJU7lXmriL2jHkP5Lx+4Q+dtdv9YgIDZmQgWC39p0pdLAYE5FLFd1z08ZEA6pXq5RRV3C2Z0eMmcVBXxVdvXHoYILoSutrEvvaOzt5vXWh/zGLSoN9Stvarx+Ufc8EeCu6h/OhYIzPe+lZaEpeEIU0G69YxpuRPe4Wy9Uk7nJ1oW5R0wRbP7qBpmyi/G0KRhcd/M66NY6k0vkms070QHwv8JvA6kOIIbVniWC2jPB05jx0QgW1L62//w7a/K6gVYal7zKRMYPYJpWydCwr9gwt+x2SKDHS1xgw+WBAu5TU4W57fZ9S2Ku/P1jlMjM83H8RURraRnOVhcmUr2aX3hMn3sBSeQQr0L7+oXSqvd5nVIkCM6rDOKxuQw= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On Sat, 18 Jan 2025 08:13:06 -0800 Guenter Roeck wrote: > Hi, > > On Mon, Nov 18, 2024 at 07:13:31PM +0000, David Laight wrote: > > Use BUILD_BUG_ON_MSG(statically_true(ulo > uhi), ...) for the sanity > > check of the bounds in clamp(). > > Gives better error coverage and one less expansion of the arguments. > > > > Signed-off-by: David Laight > > This patch triggers a build error when trying to build parisc:allmodconfig. > See error message and bisect log below. > > I don't think there is anything wrong with the patch. The underlying > problem seems to be that parisc:allmodconfig enables CONFIG_DRM_XE which > tries to build the affected file even though CONFIG_DRM_I915 is not > enabled/supported on parisc. This has appeared before. Any idea which inlined copy of scale() is causing the problem. On the face of it they all look ok. If you can reproduce it maybe try commenting out some of the calls. David > > Copying XE maintainers for feedback/advice. > > Thanks, > Guenter > > --- > Building parisc:allmodconfig ... failed > -------------- > Error log: > In file included from : > drivers/gpu/drm/i915/display/intel_backlight.c: In function 'scale': > include/linux/compiler_types.h:542:45: error: call to '__compiletime_assert_415' declared with attribute error: clamp() low limit source_min greater than high limit source_max > 542 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__) > | ^ > include/linux/compiler_types.h:523:25: note: in definition of macro '__compiletime_assert' > 523 | prefix ## suffix(); \ > | ^~~~~~ > include/linux/compiler_types.h:542:9: note: in expansion of macro '_compiletime_assert' > 542 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__) > | ^~~~~~~~~~~~~~~~~~~ > include/linux/build_bug.h:39:37: note: in expansion of macro 'compiletime_assert' > 39 | #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg) > | ^~~~~~~~~~~~~~~~~~ > include/linux/minmax.h:188:9: note: in expansion of macro 'BUILD_BUG_ON_MSG' > 188 | BUILD_BUG_ON_MSG(statically_true(ulo > uhi), \ > | ^~~~~~~~~~~~~~~~ > include/linux/minmax.h:195:9: note: in expansion of macro '__clamp_once' > 195 | __clamp_once(type, val, lo, hi, __UNIQUE_ID(v_), __UNIQUE_ID(l_), __UNIQUE_ID(h_)) > | ^~~~~~~~~~~~ > include/linux/minmax.h:206:28: note: in expansion of macro '__careful_clamp' > 206 | #define clamp(val, lo, hi) __careful_clamp(__auto_type, val, lo, hi) > | ^~~~~~~~~~~~~~~ > drivers/gpu/drm/i915/display/intel_backlight.c:48:22: note: in expansion of macro 'clamp' > 48 | source_val = clamp(source_val, source_min, source_max); > | ^~~~~ > > --- > # bad: [0907e7fb35756464aa34c35d6abb02998418164b] Add linux-next specific files for 20250117 > # good: [5bc55a333a2f7316b58edc7573e8e893f7acb532] Linux 6.13-rc7 > git bisect start 'HEAD' 'v6.13-rc7' > # bad: [195cedf4deacf84167c32b866ceac1cf4a16df15] Merge branch 'main' of git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next.git > git bisect bad 195cedf4deacf84167c32b866ceac1cf4a16df15 > # bad: [e8c0711b153b0db806410d8b31ed23b590f4eab4] Merge branch 'xtensa-for-next' of git://github.com/jcmvbkbc/linux-xtensa.git > git bisect bad e8c0711b153b0db806410d8b31ed23b590f4eab4 > # bad: [81d45722d699e594c66c150c8f7a0ec2e2bc9be6] Merge branch 'for-next/perf' of git://git.kernel.org/pub/scm/linux/kernel/git/will/linux.git > git bisect bad 81d45722d699e594c66c150c8f7a0ec2e2bc9be6 > # bad: [7acb844a672defb15cf202a501815ec22c68c800] foo > git bisect bad 7acb844a672defb15cf202a501815ec22c68c800 > # good: [fb2368075b135f174264071b851330649d55f9d0] mm/damon/core: add damos_filter->allow field > git bisect good fb2368075b135f174264071b851330649d55f9d0 > # bad: [fc83c501e385753c90db7316faf9fd4158caaa96] minmax.h: remove some #defines that are only expanded once > git bisect bad fc83c501e385753c90db7316faf9fd4158caaa96 > # good: [b04d305df1171448df5e87802c4d1f1022cc5784] ocfs2: use a folio in ocfs2_map_and_dirty_page() > git bisect good b04d305df1171448df5e87802c4d1f1022cc5784 > # good: [7e01619507058f90ab603acec482951f3c452aaa] kthread: correct comments before kthread_queue_work() > git bisect good 7e01619507058f90ab603acec482951f3c452aaa > # good: [21b510a64c223707caa6db6176128779f0806a73] nilfs2: correct return value kernel-doc descriptions for ioctl functions > git bisect good 21b510a64c223707caa6db6176128779f0806a73 > # good: [6afb87f23458f2d4e4334ee5a4efb8b0d07af68b] nilfs2: handle errors that nilfs_prepare_chunk() may return > git bisect good 6afb87f23458f2d4e4334ee5a4efb8b0d07af68b > # good: [8f6d46fed0bad163e5146fea1fdff150039235b2] minmax.h: reduce the #define expansion of min(), max() and clamp() > git bisect good 8f6d46fed0bad163e5146fea1fdff150039235b2 > # bad: [7a70c678548d71e609b95dbddf2d411a02d13b54] minmax.h: move all the clamp() definitions after the min/max() ones > git bisect bad 7a70c678548d71e609b95dbddf2d411a02d13b54 > # bad: [37f375aab0c585388b90d1af6968454fc2769cb9] minmax.h: use BUILD_BUG_ON_MSG() for the lo < hi test in clamp() > git bisect bad 37f375aab0c585388b90d1af6968454fc2769cb9 > # first bad commit: [37f375aab0c585388b90d1af6968454fc2769cb9] minmax.h: use BUILD_BUG_ON_MSG() for the lo < hi test in clamp() > >