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 118C6156FA for ; Thu, 26 Oct 2023 17:17:26 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="dHvoQu1U" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 233D2C433C8; Thu, 26 Oct 2023 17:17:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1698340646; bh=IYv7ckSXIHomFlKV00BZVQ1yGlkJ2beREQcNt3zDRVc=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=dHvoQu1UlmF46AFzWTo+akFbYWGVbF1hdsl/BX11sJiebolkHxK2ffePGHFzNu4od ASuMlX1Z5seL2K1lUTDkzAd25sfSDnASH9lKNolMBOAu7QykEgQQqEs9vGBoUgaa7g MHjzVjaj5g06QdtRRe6o9zol7hMjMVtST6J6DtxWpXtTCGHJ+rQrdMIO3mSmJm11Km XYaJ2Kk7JibIARfHE5VWs/yubzbNzw41+WA0GClnmokxxrfP35A1e/mSgSPgTyB9Q/ 1WZug+PZVkv1RlkMpbkmmvsobR7oEptljS7MR13svBQ72edRTfMpO7zUTCv0nU3+Gn relCIysZ3PuAg== Message-ID: Date: Thu, 26 Oct 2023 11:17:25 -0600 Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH net] inet: shrink struct flowi_common Content-Language: en-US To: Eric Dumazet , "David S . Miller" , Jakub Kicinski , Paolo Abeni Cc: netdev@vger.kernel.org, eric.dumazet@gmail.com, wenxu References: <20231025141037.3448203-1-edumazet@google.com> From: David Ahern In-Reply-To: <20231025141037.3448203-1-edumazet@google.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 10/25/23 8:10 AM, Eric Dumazet wrote: > I am looking at syzbot reports triggering kernel stack overflows > involving a cascade of ipvlan devices. > > We can save 8 bytes in struct flowi_common. > > This patch alone will not fix the issue, but is a start. > > Fixes: 24ba14406c5c ("route: Add multipath_hash in flowi_common to make user-define hash") > Signed-off-by: Eric Dumazet > Cc: wenxu > --- > include/net/flow.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Reviewed-by: David Ahern