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 BA3507765B for ; Tue, 12 Dec 2023 16:25:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="fg/+edAC" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 09358C433C8; Tue, 12 Dec 2023 16:25:51 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1702398351; bh=NOX09/14OJ0ZefdYzqvqVjJ/qqQBXuOz9+e0JR84QzY=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=fg/+edAC5Z9+TcxEAvjAWyWtnq5/aY4UtvczdsBSyp5lTAA8IJeAO6EGXgfRGUD92 WC/iT75x4pX21s86TSFTHGSYPlELH/cZNSg3HzgXXUNswNcFIX1qlxtRZWtDn8rj+c oMjVE/M8AlQQSgIfYy63d9p/3AtLUGuQmx8fw2qKFSTtNVVSZIU68mk8cILQzWN8+V 9RAvMsDeuN/bB//+/uWMfcbvSbD0biAw49AXPoECmHedgvD3+sUiwxzTVa2MUy6nzK t1X87sjhsdmBSJ4cYCDRuD8G3TY8lmhyogse5Ljw5fIQwvcRIrKaRYqr9441lqRT2x +MNFTwbKZ1Mvg== Date: Tue, 12 Dec 2023 08:25:50 -0800 From: Jakub Kicinski To: Ioana Ciornei Cc: davem@davemloft.net, edumazet@google.com, pabeni@redhat.com, netdev@vger.kernel.org Subject: Re: [PATCH 8/8] dpaa2-switch: cleanup the egress flood of an unused FDB Message-ID: <20231212082550.17962ffc@kernel.org> In-Reply-To: <6ipoutbyuvlgj33k3ytmrrrtlikabpu7mc3ysv5wemfjgv6o27@wjrdfhg3vwvg> References: <20231204163528.1797565-1-ioana.ciornei@nxp.com> <20231204163528.1797565-9-ioana.ciornei@nxp.com> <20231205200455.2292acf6@kernel.org> <6ipoutbyuvlgj33k3ytmrrrtlikabpu7mc3ysv5wemfjgv6o27@wjrdfhg3vwvg> 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-Transfer-Encoding: 7bit On Tue, 12 Dec 2023 14:09:43 +0200 Ioana Ciornei wrote: > > Is this not a fix? FWIW the commit message is a bit hard to parse, > > rephrasing would help.. > > This is not actually fixing up an issue which can be seen but maybe it's > better to just have it through the net tree. I will split the patch set > and send some of them through net. > > I'll rephrase the commit message and add a bit more information. Same goes for all the other changes. You don't have to send the patches to net if there's not user visible impact, just please add the explanations to the commit messages.