Received: by 2002:a05:7412:85a1:b0:e2:908c:2ebd with SMTP id n33csp99245rdh; Mon, 30 Oct 2023 15:21:14 -0700 (PDT) X-Google-Smtp-Source: AGHT+IHY213GaGFi7Xp8XMVU6DyEjKkmv7qG0Ang2obM6wb5WnuX+b4GuFUfqdPWmkKIJwIBx4Yj X-Received: by 2002:a05:6358:5924:b0:168:e7ab:6877 with SMTP id g36-20020a056358592400b00168e7ab6877mr10776833rwf.6.1698704474635; Mon, 30 Oct 2023 15:21:14 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1698704474; cv=none; d=google.com; s=arc-20160816; b=J0gE55N2AnqxN5TJP957ge+2+rNwObIdD5Nq7Oxs91YH/ID1uwlm/rlUBXciGc0jqL qJAC7V8Uu0RqobomUFn+Rfmja+l8C25+Z2bqUNYy2E6E/0FO/ZVKDVQeACAJPINFUTKT w1pMtwfwasxSArO3xxnNugIvOG1/xNsWNvf8Aq1AngYxdGYMulb2G7kdpTl/PmR6s9lf oKO2Nf90XTF0FinvimHAlmZMK3E2RHwtSYFnpqXkoL4Xco5eNRM1bGhd6dbKK4Kv9WfQ Xn0O1xwgxxdIZA/xNJT6FcYnxpHXUCZsuOJh2o4/ZDUgONPY0yGZeyQO4THEchOHjjeN QRhA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=hwrn6ID3BUu3z/5xyPAOjQ/FmaKW4prpre5IoiNlsn4=; fh=Xg4JAZHYCaYSj69qysTaAArbPbAkobOBeX0gq7jvuCw=; b=MmogHKmnCLzMTfE2CICD4NP/hREbC77Xb8ZfvnS/wK0+hUgfELjmtj5aH8g1ZhQAxu +xw40BOhX/amUqO/VAMi06lpR8eCWILHQQvDQaxNGd/qqzYY+nV+zneMhY1XEGmSA/8D kE+3dQvsFqj28zoWP3U4JOlxOZvBUfWdSo1hTmgHogA1jv7pDQPV9QCV6ePM/3WBqBOz DJNQqCxkx+Hf3BZhJr25izKsMTSg3N0LBp2cZpsYQmjE3TFkD/O8IYRIvtFeh8wT8wL2 +WL02tS8iL110NyVK0a3/cB167bx2RGREeIS1gdzc+DrxfUTx/MGxmte3LK2S2bjdbTo K6wg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b="d/1FaI2D"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.35 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from groat.vger.email (groat.vger.email. [23.128.96.35]) by mx.google.com with ESMTPS id u31-20020a63455f000000b005824bad8f83si60401pgk.846.2023.10.30.15.21.13 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 30 Oct 2023 15:21:14 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.35 as permitted sender) client-ip=23.128.96.35; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b="d/1FaI2D"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.35 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by groat.vger.email (Postfix) with ESMTP id 44C228080EF9; Mon, 30 Oct 2023 15:20:59 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at groat.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229817AbjJ3WUo (ORCPT + 99 others); Mon, 30 Oct 2023 18:20:44 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47506 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229672AbjJ3WUm (ORCPT ); Mon, 30 Oct 2023 18:20:42 -0400 Received: from mail-wm1-x335.google.com (mail-wm1-x335.google.com [IPv6:2a00:1450:4864:20::335]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 52C9D110; Mon, 30 Oct 2023 15:20:39 -0700 (PDT) Received: by mail-wm1-x335.google.com with SMTP id 5b1f17b1804b1-40850b244beso38746635e9.2; Mon, 30 Oct 2023 15:20:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1698704437; x=1699309237; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=hwrn6ID3BUu3z/5xyPAOjQ/FmaKW4prpre5IoiNlsn4=; b=d/1FaI2DGYYdGx2SB+bq+8yeLnL7F1tS90Zom40hblMQrCNqymbHYsQ5UEvxZNwzmW 0eMQB+Ce0w2Plf6wVOUFk/9Oyo3FjNr4qTaXmCXYTnH3fZGr3z5BWcnUAUvPoNhISDSE hC4FdzPQV/TqujUSvmkliEBbvNYAuuPmvdTQf2WsHEUOht7L+V/yOxrmfPCJje9zmz2y zOlK6+wFbHzyPd4RVlGkM7vZRjgSG1VbeB4ccwYZ0LnYAlcyd7kM9LMebzC8rCgJcr5p S6+0NMZM6LCs1bN0wVtB2LEtzCfwqdZJcAaGCAwyXiahiywPQ8Ndc+AgrvoxA4DChVm6 Jkkw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1698704437; x=1699309237; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=hwrn6ID3BUu3z/5xyPAOjQ/FmaKW4prpre5IoiNlsn4=; b=EdGAM+eYWDXhgHfNV6coMQTqtd0OhHuuBw5vMZwrfWgFWiaKSF8V94NUDV7RQMfI4L H2lws7tnhzsRmlVEzVBqUdGhmL440zNnoS4Llz/7cPXTqBPKbREDqeWUTVlpRG4nhUFo h+x2Xpsi+2px/aKr2JKefkflcrEHYgDqYfdgDTwstnFF5jH97NkbjgTlBrTHaQUrwciH +C44X7F4o1vdojnztd8ao1rhGSxQEvShSLsMs889l92L7+LXZtcb9M5UqO5AvWgTFXR+ a3IWUwPjmWCC1+fg7+w6y046PGspFrV6rt0X50Ef2AfdYJvxessI7gxrAyo3vyIuDvC6 DubA== X-Gm-Message-State: AOJu0YzvwGLY1a1x2DJ+1P14Nm0K9l6Uoy1T7y67VhRf+volGKbpeaWZ OloyDxTAz3p51iENzXtOkJc= X-Received: by 2002:a05:600c:468e:b0:409:78e:1c42 with SMTP id p14-20020a05600c468e00b00409078e1c42mr9592992wmo.22.1698704437359; Mon, 30 Oct 2023 15:20:37 -0700 (PDT) Received: from skbuf ([188.26.57.160]) by smtp.gmail.com with ESMTPSA id f7-20020a1c6a07000000b00401b242e2e6sm391576wmc.47.2023.10.30.15.20.36 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 30 Oct 2023 15:20:37 -0700 (PDT) Date: Tue, 31 Oct 2023 00:20:35 +0200 From: Vladimir Oltean To: Linus Walleij Cc: Andrew Lunn , Florian Fainelli , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH net v2] net: dsa: tag_rtl4_a: Bump min packet size Message-ID: <20231030222035.oqos7v7sdq5u6mti@skbuf> References: <20231030-fix-rtl8366rb-v2-1-e66e1ef7dbd2@linaro.org> <20231030141623.ufzhb4ttvxi3ukbj@skbuf> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Spam-Status: No, score=-0.6 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on groat.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (groat.vger.email [0.0.0.0]); Mon, 30 Oct 2023 15:20:59 -0700 (PDT) On Mon, Oct 30, 2023 at 10:50:31PM +0100, Linus Walleij wrote: > > you said that what increments is Dot1dTpPortInDiscards > No this was a coincidence, we can rule this out. I see commit 86dd9868b878 ("net: dsa: tag_rtl4_a: Support also egress tags") also mentions: "Qingfang came up with the solution: we need to pad the ethernet frame to 60 bytes using eth_skb_pad(), then the switch will happily accept frames with custom tags.". So the __skb_put_padto() was something very empirical in the first place. Since it's all problematic, would you mind removing the __skb_put_padto() altogether from rtl4a_tag_xmit(), and let me know what is the output for the following sweep through packet sizes? I truly wonder if it's just for small and large packets that we see packet drops, or if it's something repetitive throughout the range as well. for size in $(seq 0 1476); do if ping 10.0.0.56 -s $size -W 1 -c 1 -q >/dev/null; then echo "$((size + 42)): OK"; else echo "$((size + 42)): NOK"; fi; done