From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f49.google.com (mail-wm1-f49.google.com [209.85.128.49]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4D60629D27D for ; Thu, 18 Dec 2025 22:06:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.49 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766095619; cv=none; b=mPROt4QeSKUoD5TIl+gz7qu8nwHNRfjMdrzwpYpf/M7COrqSYIUXsYXa7wWpvooiP4qDb1UCfnjU9F88T4UkOLvbm5wpXg5O28i/QX4Idbtv16euOg69jVMGm8gxUh8u1h3OobtE5PGzemKl770R2z9GjPsHiTECsd1vdpL3DRk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766095619; c=relaxed/simple; bh=J4Nn2NlGfDoMoLD2jX96Gn59VTZ3FbVS9Sq+YlKTdug=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=k4G3sXMV0t3/caHQc27RIfR6mmqTlFUXwFIn0aS4ATqc9NSDgKqAydjfDcbY9QEFVRHwwYf/FL0GMec/A/VUepbST5b0L3pQChSWIVOnZfxgy/3hww4/ze9HbwyhUnDxnkQwq5MX0yUcve8tNdpxxZdDjaw96qRoKqF6rNQ5biQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=liKa0KNf; arc=none smtp.client-ip=209.85.128.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="liKa0KNf" Received: by mail-wm1-f49.google.com with SMTP id 5b1f17b1804b1-4775e891b5eso4853055e9.2 for ; Thu, 18 Dec 2025 14:06:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1766095617; x=1766700417; darn=vger.kernel.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=FCWNyqqNvxqk0sIuAi6HOTqP3nuQgibcAJ59og/NAJg=; b=liKa0KNfSTlpJ0iuaL+p+4S0MpSmTpX9aE6o7KTOJ4tWOjF/2WPsQTVt+WcI6RtCxk aQTODRmXd5geqIg+VVirxkzakC0b6jlg2gGV7YjMeD3uGS5G9wezTa5PzxHIL0A6Rbl0 RgxtZDfCJsXR1MgSk+ahQCQA9+zMjQxyyEfwTVT/8gPfb//3SKWbw+RnVvmLfn8LSvax +Pk0ck1dkIIPqKfkHmaR3HFoJ+28UAb8em+W1IeQs/xEZi5x/v6sUtFYra83UrWCOQdq WwtOcbCZD5NoybFueDbdIW8mk6uEadc2PhTVvR2QhK09ab0IOiTLrkro1jx067TP95l5 bbNA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1766095617; x=1766700417; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=FCWNyqqNvxqk0sIuAi6HOTqP3nuQgibcAJ59og/NAJg=; b=KxMtYBG/mLoxLuZd6CyqvBDKDWaXs4hFDqqvMHFZUuJde22LpYJpOuGGEbip5YlRo+ hbE7+ckQ2s+8uaF1I8e3Dm+xmCseM6NdNrN8kXVm4eR0r7b9GUanVJck63b7hRaZeq7m LVmQq3/YH5biTxl20fWv9MqfUMdUK/wkk+nZmLOGP5DKBnYkLGOYTiua2A0GIQaFWAXk BdApz/yxqyAn0gW8jza3WBc5d3nVG759eaSPpV2XJHLV2RMdfWDAgPyCT3kOIe8DbfIo 8ymh03skxrsUoIkzWOrEHxt/K88a+00RB+9PBNf+3mXBNVTSJxLtZ++pg4IRIl+I9s1K cj3Q== X-Forwarded-Encrypted: i=1; AJvYcCUNCXCrtcKZluPhZS/CgUGCgBmnD5PP8zMXmIuQ3+YXcUiPP3Osf+HUf48Ysv8hVZPWKIRuVc23fc0hmK8=@vger.kernel.org X-Gm-Message-State: AOJu0YzNRecxv+o8dSflGrKAYvOPqftraNaSnJ2Hr07J0KpYiIVaVhbz 3B73NL/AG7dhWTo7NU//lKVX7SQDiZzj4m7FmDv4xrY2SfxjnQ5yIdeB X-Gm-Gg: AY/fxX6iBFB9IRqT6SFj+nqaHwGzf9nbnmdMkG5pWcoS7JKSXze2TMzWhSglnfxkhoa 8EEDWNY8WvfY+nO85N2OUPnks+vKT9/OeOMWSOXWWm+91oKt1lnCkewvmmcN1N2cY4zXOBDOwDs /kE1Iqm+FHUNomT/YnR29SjxrDesQpq2riFzkf6TKPNiJQzJahKfTcvSVOOldmsPinAwwFBqXbW pcbTPLaFDpqSngMGDHSLgbZcfDREViEENF3oMjorNtUjPEmOIvnbbZ4f5TZAFvARZpXngxDampI Q7lA8CgBWGfQ/xlRiCaGw5jb/z4hr1fVA73Q7QiimIZsdNZBd1kivuU8YXDq8RWxomblXSJVx9o YHZJNsC3c5TQ422oYHNpZYnTrOSDztquWmx4oP2N56a66f0YOefmOir59V/iVeVjJP+KYFke5Ml 2xlbkS+hCkPzToNlcy4tATe6yMaNRMEPFIMbBSQH1OXxmIZ4WAAN+s X-Google-Smtp-Source: AGHT+IGTunC8kRSxjImDPeRC9HvKyQNV1Ibv0KtdSMjVnMShq1E8PEmRrIAevTE0VIl4mZ+C4NDiJQ== X-Received: by 2002:a05:600c:3b8d:b0:477:7725:c16a with SMTP id 5b1f17b1804b1-47d1953da58mr7304975e9.10.1766095616415; Thu, 18 Dec 2025 14:06:56 -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-47be2723d19sm71561025e9.2.2025.12.18.14.06.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 18 Dec 2025 14:06:56 -0800 (PST) Date: Thu, 18 Dec 2025 22:06:51 +0000 From: David Laight To: Linus Torvalds Cc: Vincent Mailhol , Nathan Chancellor , Nicolas Schier , Nick Desaulniers , Bill Wendling , Justin Stitt , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Chris Mason , David Sterba , linux-kbuild@vger.kernel.org, linux-sparse@vger.kernel.org, linux-kernel@vger.kernel.org, llvm@lists.linux.dev, dri-devel@lists.freedesktop.org, linux-btrfs@vger.kernel.org Subject: Re: [PATCH 0/2] kbuild: remove gcc's -Wtype-limits Message-ID: <20251218220651.5cdde06f@pumpkin> In-Reply-To: References: <20251218-remove_wtype-limits-v1-0-735417536787@kernel.org> <20251218202644.0bd24aa8@pumpkin> X-Mailer: Claws Mail 4.1.1 (GTK 3.24.38; arm-unknown-linux-gnueabihf) Precedence: bulk X-Mailing-List: linux-kbuild@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Fri, 19 Dec 2025 08:34:05 +1200 Linus Torvalds wrote: > On Fri, 19 Dec 2025 at 08:26, David Laight wrote: > > > > One possibility is to conditionally add _Pragma() > > No. That compiler warning is pure and utter garbage. I have pointed it > out fopr *years*, and compiler people don't get it. > > So that warning just needs to die. It's shit. It's wrong. True - especially for code like: if (x < 0 || x > limit) return ... where the code is correct even with 'accidental' conversion of a negative signed value to a large unsigned one. clang seems to have a dozen similar warnings, all of which are a PITA for kernel code - like rejecting !(4 << 16). _Pragma() might be usable for -Wshadow, which is generally useful for local variables (but not global functions like log() and j0()). (I usually enable it and fix up the consequences.) Things like the masked userspace access define which carefully creates a readonly variable that shadows a user local would need to disable that one. David