From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f41.google.com (mail-wm1-f41.google.com [209.85.128.41]) (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 561952BEC45 for ; Thu, 18 Dec 2025 22:06:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.41 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766095619; cv=none; b=UOKIkxUu23RNHuwhn5tmzQKEd+Xwvnw1SPOT0HyXCq6dRXqpWXBB+3VNOlt/XJhcH0mgsm0qz1OWHnAJQtWPZiq7Z2uACEw1PclcloSkeGa6cwuZWNrf5WYG0kWNDyyh7MYuRNzsEL3MNBfCHk24M0l4oL8feomu+IQ864ug7JM= 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.41 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-f41.google.com with SMTP id 5b1f17b1804b1-477aa218f20so7619355e9.0 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=dqRZ6h/awoJkr9RYPY9YZrLBdX9JoMGwlki+2rk19rnmDv6m+dQxiFrYaaMpQLu1R7 4VIqx4/2OX1aDIqhAeSFtAyV/lEnFi/XnKC1olPGhX2Rd3IS8UgAoT4mUrrn6tkMenz2 Pm+dSFs/NhqMQ4uCHq9a77fCbkdkR1EYy6sIyUidUoW0jJuBroA7fEjNW0Hc/lnX4pAo EjvyG1ySUTmzSTEyiSB3nU4Vg2F/+6XEk7AMsVu871xDddUkyrhugMUE7dvX9xaHp6dW duk/w/qYRTOzyLVHF+SPnWSq77fQz/dXYSx6ljeADF6cQBMNaXj6/sns3Vc57oY7f2TT BjKw== X-Forwarded-Encrypted: i=1; AJvYcCVcR+unqv8unBRNev21DGHpZnBeecD+q93S2+YGNw5uXGpvP0Y1q6myBuxgk6vAhDqiajkxPjp1sBlrWE0=@vger.kernel.org X-Gm-Message-State: AOJu0YzHXsLcf7rL3Ss1mdbgngQVJ6CwcWmHpkpyC4CxhF2Vov8Lok/N MT6PVUBFAbkk5gb9Z8Q6uk5+CnPdXhQzKbjZNBQ5rl3+ry08prvC8CG4 X-Gm-Gg: AY/fxX57De0km3YkY/dfwMHLkJq10LAsq2a6ehNJWRTzgmMV3PCOzpbfp7ciT702gQ+ 3IlKIW3omNFmLTWG91S5EZjRuQf/aGblWB7JSZHeygbTjr4Qkp2+tkvZKHisp2SwHn4QesLM6uy og8WJjm496MzgX8buC2ufN+yLHkHY5DKSTx69CH9XkssWH8KuqXNndJNbdUwC+XbcWWigbDzvGG JdmjdVrBGXHYG5SEMr0RFVZOCBef3MoE0FEkX06Zu3angse7T4j2JFIh8PJFUyMbK9YQCOAthGG IJlQJEp53Tj/32qnXY5HjwDhOhSrn4cSoOmJ56tqaLudkmAdrVAZmCPc5LlFlb+d7dxJ/ag1adC DWRk4L2dDM1lCVXPNqvGAZ1iLnTc7i8URwSHg+FWwgN7/A+TWLBsTM5NgCBgXqrMj4tdzly1R26 yV3ZW/2EJ9zSm9ZKH6UztNdBZFxDFYmrhGhLIvCqo8HdsT2v3HEdJH 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-kernel@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