From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from out-185.mta0.migadu.com (out-185.mta0.migadu.com [91.218.175.185]) (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 B3637478843 for ; Fri, 15 May 2026 11:13:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=91.218.175.185 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778843609; cv=none; b=u9BW1cBgJfnb+gOuaHm7gL74Og+xyAXHXssmks/pJXWBXxisUQNqdDRO5A3lEAOAhbXoAc4qeOyJsBWXXwtYO+oh4xOnR9HLrzomeHFgwvZB1InGKwfSdwUfMUJHV359BaXnmNuhs6G2PG7Vz1CaPuvBXZqDKHSVeMw0qiyDmQc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778843609; c=relaxed/simple; bh=kag9haX792QrajYmaATaIoZvrsAiqPWws8ASbmeKluU=; h=MIME-Version:Date:Content-Type:From:Message-ID:Subject:To:Cc: In-Reply-To:References; b=d2jii6cQMsN6PouvcCFXLiYlAkow5xrd7QygwT+xLFmcazsnMVdGPakmNsE0n0li7upfoAjMNZm4yquZOt8d2FEQTSJ87i8yw5C/sryaw3aveKDcLDenEs7KCouBbIloGR1SB6Xjeq+b+Ienmg5+2awMS7B5rkVOjyfm/tBJGEk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev; spf=pass smtp.mailfrom=linux.dev; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b=OaF/xShM; arc=none smtp.client-ip=91.218.175.185 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b="OaF/xShM" Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1778843595; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=xY+E/upLJxEjmvM3j7C4ks+xfWlSY3E/fLSmuAvnhNI=; b=OaF/xShMio5pJ01/fel5DMXE3YdlVSumEvuZa6aNn7Uw9tp/Z2OdB1rsusp8RHJiyyZ902 JGsr6f2ap+f4Hdsp637l3HEhTcuEjc0TP1tskvJ/pGVqoUisBO2ugLoqT8yY3ovM4ArsZZ +mjHogAd+q2BkujTSSCy1GoKAvIacKw= Date: Fri, 15 May 2026 11:13:05 +0000 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: "Frank Wunderlich" Message-ID: <935a73a6e458b2d4fed2e59192e725483ca62126@linux.dev> TLS-Required: No Subject: Re: [net-next v8 2/3] net: ethernet: mtk_eth_soc: Add RSS support To: "Jakub Kicinski" , "Mason Chang" Cc: "AngeloGioacchino Del Regno" , netdev@vger.kernel.org, "Russell King" , linux-kernel@vger.kernel.org, "Andrew Lunn" , "Eric Dumazet" , linux-mediatek@lists.infradead.org, "Daniel Golle" , "Matthias Brugger" , "Paolo Abeni" , "Lorenzo Bianconi" , "David S. Miller" , linux-arm-kernel@lists.infradead.org, "Felix Fietkau" , "Frank Wunderlich" In-Reply-To: <20260513185625.48e69837@kernel.org> References: <20260509190938.169290-1-linux@fw-web.de> <20260509190938.169290-3-linux@fw-web.de> <20260513185625.48e69837@kernel.org> X-Migadu-Flow: FLOW_OUT Hi Jakub, thanks for your review, i have to discuss the previous parts (AI-review) with MTK on how to make it better. The changes there seem not trivial for me and this will take some time. Am 14. Mai 2026 um 03:56 schrieb "Jakub Kicinski" : >=20 >=20On Sat, 9 May 2026 21:09:31 +0200 Frank Wunderlich wrote: >=20 >=20>=20 >=20> From: Mason Chang > >=20=20 >=20> Add support for Receive Side Scaling. > >=20=20 >=20> We can adjust SMP affinity with the following command: > > echo [CPU bitmap num] > /proc/irq/[virtual IRQ ID]/smp_affinity, > > with interrupts evenly assigned to 4 CPUs, we were able to measure > > an RX throughput of 7.3Gbps using iperf3 on the MT7988. Further > > optimizations will be carried out in the future. > >=20 >=20Would be great to split this up a little more for ease of review. you mean splitting the code into more separate patches or the commit desc= ription? > >=20 >=20> +static int mtk_rss_init(struct mtk_eth *eth) > > +{ > > + const struct mtk_soc_data *soc =3D eth->soc; > > + const struct mtk_reg_map *reg_map =3D eth->soc->reg_map; > > + struct mtk_rss_params *rss_params =3D ð->rss_params; > >=20 >=20reverse xmas tree should be followed, please fix everywhere in this > submission=20 will=20check again when we have the fixed flow (AI-review). > >=20 >=20> +/* struct mtk_rss_params - This is the structure holding paramete= rs > > + * for the RSS ring > > + * @hash_key The element is used to record the > > + * secret key for the RSS ring > > + * indirection_table The element is used to record the > > + * indirection table for the RSS ring > > + */ > >=20 >=20Quite odd looking comment. Having the right side aligned like that > makes it header to correlate where doc for fields start. > And there's @ missing for indirection_table. dito @Mason: have you found some time for looking into the AI suggestions? Maybe it is better discussing changes here if they match upstream requirements as i did only up-port the SDK-patches with some small changes and made tests :) As a note: i have not activated RSS/LRO on other mt798x because it was not clear, if RSS or LRO is better for them and only mt7988 can do both simultanously. AI also complained about it. And this decision causes changes in code too. It would be great if you can use the actual state here as base for changes so that i do not miss anything when upporting the SDK state again= . regards Frank