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 347AD407590; Thu, 22 Jan 2026 08:14:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769069675; cv=none; b=uPyVCzURRL1bkdgz7WEWvC/BuAowoFF1+S7cArGc3oAJMf7Na3LBK3BBUeJuFtTCeL9veRqzE1LQIhTU/h9gq4PNI9rV8LFpSwwnSNVG1RHsch6XFCf0vqa/A1wxInY56Ko43PDd8xfMxtzQIdGLCggz/LCxkNDNkowfNi9PdUg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769069675; c=relaxed/simple; bh=+5qCEXADRX+3Rn7uzuXBcjl85v9QdvSiUb4V5YDgzjM=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=EZRKMkSUq2cSKRXKfis0rf4GHRGlf78tFhIIxrHo0P3YOfkWPeyPi2GV4xzbCtUer4GUiIil9So+eKx80xIpbi5rh1kLPVN2SXNurk/WihDElgPTk5NV/ExhSOBV+IsuIKAx4gY0PeHBrXWfvqo1DEGdsnDPZcBKH7t9/kuhQOo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=f/vnElbE; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="f/vnElbE" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 1399BC116C6; Thu, 22 Jan 2026 08:14:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1769069674; bh=+5qCEXADRX+3Rn7uzuXBcjl85v9QdvSiUb4V5YDgzjM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=f/vnElbEX7MM6IS2jPrcOMtvZrWZkYldPYc6X8wBQRkJmAkkRbXnxgCcKQU0C+L3u qbeYyRDIxM05D2RUSFGVF50vTnZK8TFn4r7CwvBSQwSFX5R4bBvVYMo8UNrv2iEsPi 8p3zCzKxDXtDyK1OBbDWbQMZ3/51Bd2h67a0H2U31tkrBU/RuVPfEYzGkZ4jFzqWZv TeFwh6z+F4fNqVzhkLeWDi4LQZH/hSDmvbVehaxil7WDCtkPooTIjfVX3ud2DUjc1d RELqtjeNEiarMv0Gq+GLW2hK/+oJTj39FBcgWNX2yNU3Ub2wff/Yqn4Mfii9VrToD0 LkdHNwL70Q7gA== Date: Thu, 22 Jan 2026 08:14:30 +0000 From: Simon Horman To: Jakub Kicinski Cc: Geert Uytterhoeven , "David S. Miller" , David Ahern , Eric Dumazet , Paolo Abeni , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH net-next v2] net/tcp_sigpool: Enable compile-testing Message-ID: References: <20260117163304.20caae7c@kernel.org> <20260121172828.7207108c@kernel.org> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20260121172828.7207108c@kernel.org> On Wed, Jan 21, 2026 at 05:28:28PM -0800, Jakub Kicinski wrote: > On Wed, 21 Jan 2026 19:55:38 +0000 Simon Horman wrote: > > On Mon, Jan 19, 2026 at 05:26:37PM +0100, Geert Uytterhoeven wrote: > > > > this code cannot be used today. If this code regresses and someone > > > > wants to start using it on 32b they'll have to fix it. > > > > > > > > Please LMK if I'm misunderstanding or there's another argument (not > > > > mentioned in the commit message). > > > > > > In general, we want to be able to test-compile as much code as possible > > > on all platforms, but not bother everyone who configures and builds a > > > kernel for his system. Until commit 37a183d3b7cdb873, that included > > > the tcp_sigpool code, and any build regressions would be caught soon, > > > and fixed (presumably). Of course that still doesn't guarantee the > > > code would actually work on 32-bit, but successful compilation is a > > > first step... > > > > > > As the maintainer, the decision is yours, though. > > > > Thanks Geert, > > > > In my understanding COMPILE_TEST is about expanding compile-test coverage, > > and that is what this patch does. > > Mine is that the goal of COMPILE_TEST is to make hard to reach but > *reachable* configurations. Canonical example a driver which can > only be useful on a single SOC may be hiding under that SOCs configs > but we don't want allconfig to miss it. Or randconfig to struggle > to each it. > > If we change a core API we want to be able to build all the drivers > that it may break. > > Here the situation is different, this code is already reachable by > allmodconfig. The patch makes it also build for what I referred to > as very unusual and Geert as "on't be dead for at least a decade" > platforms.. But crucially the code currently _never_ builds for > those platforms. > > Please don't repost this. Let us agree to disagree on the intention of COMPILE_TEST. And let this thread rest.