Re: [RFC/RFT v1] net: ethernet: mtk_eth_soc: drop generic vlan rx offload, only use DSA untagging

[Frank Wunderlich]

Am 16. April 2023 11:52:31 MESZ schrieb "Arınç ÜNAL" <arinc.unal@xxxxxxxxxx>:
>On 16.04.2023 12:10, Frank Wunderlich wrote:
>> From: Felix Fietkau <nbd@xxxxxxxx>
>> 
>> Through testing I found out that hardware vlan rx offload support seems to
>> have some hardware issues. At least when using multiple MACs and when receiving
>> tagged packets on the secondary MAC, the hardware can sometimes start to emit
>> wrong tags on the first MAC as well.
>> 
>> In order to avoid such issues, drop the feature configuration and use the
>> offload feature only for DSA hardware untagging on MT7621/MT7622 devices which
>> only use one MAC.
>
>MT7621 devices most certainly use both MACs.
>
>> 
>> Tested-by: Frank Wunderlich <frank-w@xxxxxxxxxxxxxxx>
>> Signed-off-by: Felix Fietkau <nbd@xxxxxxxx>
>> Signed-off-by: Frank Wunderlich <frank-w@xxxxxxxxxxxxxxx>
>> ---
>> used felix Patch as base and ported up to 6.3-rc6 which seems to get lost
>> and the original bug is not handled again.
>> 
>> it reverts changes from vladimirs patch
>> 
>> 1a3245fe0cf8 net: ethernet: mtk_eth_soc: fix DSA TX tag hwaccel for switch port 0
>
>Do I understand correctly that this is considered being reverted because the feature it fixes is being removed?

As far as i understood, vladimirs patch fixes one
cornercase of hw rx offload where felix original
patch was fixing more..sent it as rft to you to test
if your bug (which vladimir fixed) is not coming in
again. If it does we can try to merge both
attempts. But current state has broken vlan on
bpi-r3 non-dsa gmac1 (sfp-wan).

>Arınç


regards Frank