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 E7E0D3BB4A for ; Thu, 26 Feb 2026 00:27:26 +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=1772065647; cv=none; b=Ei5sXQnbICvo7qquB/ma+lBSoBenIdGy/lm7OqzOQZ96+Nnvg5320Vh1e0zhC55bXkL7oQz0vcpxEaY/qFrIxraZJrQJadus+5Rkwf+4n6H3Dg6D6IsZbRRncpawJuA3bQotflNiQmkWlFWeHcZQH8FZOrktuCoWHrnRgXmBwpc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772065647; c=relaxed/simple; bh=sVg/+8oGP0q8Fd7sGobugvZnXJY3Faobh5SDoOTT5pc=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=GW8lYoPR/Ju2ddeMmEf/RMhcSZRDwepxve+qhbR7PF5Dn8tLCuXkQ2iUKABZU/QukaUQDSBDy0nqefUlzz4Lr1+/b/JImIhBXC/0dLlp4scJP8yetynBkPNEDYvJ0AoOz6KB1eVSPt1PCvMLy9kAafHdtesnLZ+YH7nYf/qVhoU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=qkLrxnBw; 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="qkLrxnBw" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 1A0BBC116D0; Thu, 26 Feb 2026 00:27:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1772065646; bh=sVg/+8oGP0q8Fd7sGobugvZnXJY3Faobh5SDoOTT5pc=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=qkLrxnBwhGmX69GWS6y21avfaLIRwuPd8EMIfy2gXabRVN5uak2OsiH4tnY341aI0 4yPPsh+d6mLPfLD2+TF9Eq1dexgmyW5tBMmCjciEnRmlroH0fwmABAeCUkZ8vKjX/E R7J3iPhu16dR0d+wLGWgeLzLsMmxcuolgjADgoE9NuMgdwC5txd9YrPzG2r1EejRRN ob8TkHgGJL8hzLWczems5gbnygrMFHL7eMjhd6yChp+U3t0oZmhNyBRiJRfN03LKGR iNpW4EQBoRQ3vlVLIrc+LmTH6uEcKlwlPyShQj3M+lpBoKcq/hq8D4n5SdZsssk/hf JR73d4mw4w7Pg== Date: Wed, 25 Feb 2026 16:27:25 -0800 From: Jakub Kicinski To: "Russell King (Oracle)" Cc: Andrew Lunn , Alexandre Torgue , Andrew Lunn , "David S. Miller" , Eric Dumazet , linux-arm-kernel@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, netdev@vger.kernel.org, Paolo Abeni Subject: Re: [PATCH net-next] net: stmmac: fix .ndo_fix_features() Message-ID: <20260225162725.551bc03c@kernel.org> In-Reply-To: References: <20260224173037.7871e5ac@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-Transfer-Encoding: 7bit On Wed, 25 Feb 2026 10:27:09 +0000 Russell King (Oracle) wrote: > The options as I see it are: > 1. scan the transmit queue configuration, if any have TBS enabled, > disable TSO support for the entire interface. > > or > > 2. rip out TSO support, making the code simpler, and thereby removing > the need to try and fix the problems here, and making this patch > unnecessary. normally corner cases like this TSO + TBS thing are handled in .ndo_features_check. The driver can selectively clear the TSO feature for a single packet it sees heading down the stack towards its TBS queue. The stack will then run GSO and feed it segments one by one. FWIW Andrew's suggestion to do the GSO in the driver is very much legit, but I agree that its orthogonal if you're trying to simply fix brokenness.