From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Lunn Subject: Re: [Query]: DSA Understanding Date: Mon, 13 Aug 2018 15:38:17 +0200 Message-ID: <20180813133817.GA18645@lunn.ch> References: <20180809120159.GA16359@lunn.ch> <20180809125637.GC1813@lunn.ch> <20180809153543.GD20006@lunn.ch> <20180809172334.GA25449@lunn.ch> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Florian Fainelli , netdev To: "Lad, Prabhakar" Return-path: Received: from vps0.lunn.ch ([185.16.172.187]:34110 "EHLO vps0.lunn.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728661AbeHMQUj (ORCPT ); Mon, 13 Aug 2018 12:20:39 -0400 Content-Disposition: inline In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: > > I agree, this should be padding packets correctly, can you still > > instrument cpsw to make sure that what comes to its ndo_start_xmit() is > > ETH_ZLEN + tag_len or more? > > > Yes I can confirm the skb->len is always >= 62 (ETH_ZLEN + 2) Which switch are you using? Marvell switches use either 4 or 8 bytes of tag. Broadcom has 4, KSZ has 1 for packets going to the switch, lan9303 has 4, mtd uses 4, qca has 2. Andrew