Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     netdev@vger.kernel.org
Cc:     "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        Xiaoliang Yang <xiaoliang.yang_1@nxp.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        UNGLinuxDriver@microchip.com, Andrew Lunn <andrew@lunn.ch>,
        Vivien Didelot <vivien.didelot@gmail.com>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Michael Walle <michael@walle.cc>,
        Vinicius Costa Gomes <vinicius.gomes@intel.com>,
        Maxim Kochetkov <fido_max@inbox.ru>,
        Colin Foster <colin.foster@in-advantage.com>,
        Richie Pearn <richard.pearn@nxp.com>,
        linux-kernel@vger.kernel.org
Subject: [PATCH net 0/3] Fixes for Felix DSA driver calculation of tc-taprio guard bands
Date:   Sat,  3 Sep 2022 00:56:59 +0300
Message-Id: <20220902215702.3895073-1-vladimir.oltean@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?qHofrv4RToULxLBby8x1rEEmq/x/q6Epqn/Gx88wfO/ShAHF4ueJhnRH/62v?=
 =?us-ascii?Q?bcHonCuAt9KBBfc3mobVbzOBg+01i79wVgDKXwkGDBmVrql04yecDB51+7OS?=
 =?us-ascii?Q?+HlM7SKkwlXrkKUasicXyXD5l5N7BXtpOIs2QKj1BtpaMG93mJ2w8GbfkHbR?=
 =?us-ascii?Q?rkFRXxta9J8lBxwnE5RpwCILIWVBwculP3hsTbzWXIEueaGDuYoLy97AFXf9?=
 =?us-ascii?Q?bElCrUcKrEczHtxwV0GKgCi1lzxe6aiWBQDi91ovodykpBeW3BtsreEWsA7B?=
 =?us-ascii?Q?vIY/6VpBPjAuhjJ4h0BDVQi+9dUiEX0rdMc+K7QKWtgxHJdhThyti2fdYvQ6?=
 =?us-ascii?Q?bCEkYf0soQWEQOmJv8xLiQiTG3aLP6dygKGA3Jdy+8EN/AdfB0vtI94joo7L?=
 =?us-ascii?Q?39izC+GgPnY3X2BERCaB1JFbxCfmH0zLEGGywQVYmbPFoyJMmR+7vk1mXCya?=
 =?us-ascii?Q?PoU0xaFmpwp4jW62zYAglw5l058kXsj2u4d9bEEPEApGsCA1Jl8k/dKVxlDV?=
 =?us-ascii?Q?D2rxOiCafMqm43pfdffv7YOPNPq0jKiuR5NABUirU8dt0Qqqn6G6zbEG5uxq?=
 =?us-ascii?Q?ZlpiUJbzoXVi4gz+zn1aXDYKoKNS6iXKEqIV4K5K6trO296fRtTR/bHnLrNy?=
 =?us-ascii?Q?iGnF3QM2Tzn3nKAV0aQy6N66EXF5P93R94acxho9QIAwnpOGlkLL2XUDGUmL?=
 =?us-ascii?Q?8XsY+TlwtSCf/f3DRK7wMyQ1AlZSUENlavn16a5fdIfEi7XG/OcGcCM7fOJc?=
 =?us-ascii?Q?/nJjJU5Nf8THoq7XQ6IYsdGTIUDgio11arTtPmZSx9vuorUKSNUOM2my9NOI?=
 =?us-ascii?Q?EFQyA9ITZSsxxlgrcB4GSC+TO3/CsDtf16T7kZjQ0gqT4l0GhwUMpKFv1nK/?=
 =?us-ascii?Q?cJ7RzhlGs4Xu6D5sZbtDNKpqFfvujfUmoE8//7cF9p+CFSPq6V1t6QBYEfVg?=
 =?us-ascii?Q?ce66McT5p10Me42iQz5+WAQg0wxvy0wlna63sd6wrY/fvssetRzhBZnGU4KZ?=
 =?us-ascii?Q?428FY1cIJC4Ho+1bFTfbqAQaII9eSmAeGR/rTDHSFiwvxp8a3/VvC3hdnfyb?=
 =?us-ascii?Q?hXeiZXv3zb2hMrfpb4Tk776TqnyjseL3qpRajoOSe8Moj9N7GwIp7YrUJeTl?=
 =?us-ascii?Q?JaTuCYSG3Qmb64mo9NWDLj0tJ0xcwrkmLJQTtWg3baonCPgOts0dSQTbztPE?=
 =?us-ascii?Q?NgbUUOYXeTWLSmtJLpL3T+whNIF3ks3ZDK0wLTYKZ21GcUcIGxp2d9kkYNhm?=
 =?us-ascii?Q?yO/388z2js3fiJXZ+uDC1kbYuPII3AzT9fgmSsWVqwJLxWiytQ1RCzzj1kxJ?=
 =?us-ascii?Q?be/g8tl79OEUCmrAbY5H2B0J3lKADQ/Qa8elMwBnHCgPJ9pktXQpGdsVDA7E?=
 =?us-ascii?Q?pr5BGtmYiFr81kbMjIOSvTRWmj0/54KX1S4tZr1SlADkvsVBAArG1QX87tvD?=
 =?us-ascii?Q?Hz1ii+HhGv1XqoyA4cBOVhEviBj+ZHQCCawk9ZKqfp67lQtKtlwtkV6hzgXa?=
 =?us-ascii?Q?zg4GL+yOQQRyqeNkn3Awu1U5T/uDY/8dp1BPl6pEUdWuPBpi5YRACEPX2LLm?=
 =?us-ascii?Q?btD3aHyS2SR8DpXfOK7OuH3ApTqKcXFtCbeS4+jzA40+vS+WUiEy+B3MUPrr?=
 =?us-ascii?Q?Mw=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 40fe3f4f-d216-49ab-8b34-08da8d2e1861
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Sep 2022 21:57:14.5904
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 0NsIYyMez8GBnS1xNAOYzW0M5sd9ytc4EUM+quaCtaJhDhKEBcoYvJoihD9OWDcaCv+THP9Iax5ktIVt5iTrYg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB4576
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,
        SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no
        version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

This series fixes some bugs which are not quite new, but date from v5.13
when static guard bands were enabled by Michael Walle to prevent
tc-taprio overruns.

The investigation started when Xiaoliang asked privately what is the
expected max SDU for a traffic class when its minimum gate interval is
10 us. The answer, as it turns out, is not an L1 size of 1250 octets,
but half of that, since otherwise, the switch will not consider frames
for egress scheduling, because the static guard band is larger than the
time interval.

The fix for that (patch 1/3) is relatively small, but during testing, it
became apparent that cut-through forwarding prevents oversized frame
dropping from working properly. This is solved through the larger patch
2/3. Finally, patch 3/3 fixes one more tc-taprio locking problem found
through code inspection.

Vladimir Oltean (3):
  net: dsa: felix: allow small tc-taprio windows to send at least some
    packets
  net: dsa: felix: disable cut-through forwarding for frames oversized
    for tc-taprio
  net: dsa: felix: access QSYS_TAG_CONFIG under tas_lock in
    vsc9959_sched_speed_set

 drivers/net/dsa/ocelot/felix_vsc9959.c | 141 ++++++++++++++++---------
 1 file changed, 93 insertions(+), 48 deletions(-)

-- 
2.34.1