From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 13D56C433EF for ; Tue, 16 Nov 2021 12:21:20 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id D48D761882 for ; Tue, 16 Nov 2021 12:21:19 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org D48D761882 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linutronix.de Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: MIME-Version:Message-ID:Date:References:In-Reply-To:Subject:Cc:To:From: Reply-To:Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=EIdNpn6sKAhPh33t4PPRwOYu5N14c7paDjb4aPuPKZU=; b=KzR/jmjS3vKlzPSKi+vsniPA0f At9fH3eKF206HyStxfbTsU12uL72lMzt9vo2I7ANrs7v+gkDnBr7gINSIz6FlqPQcrhVD2KlvQWsY rokN070TKzC3cKVHQZ9Fom6PDrJNNqVqNFZx0iK5eKtSJH3I0eDj3ufsIiDgDCUwqpkb9gvlGsHfb ha4BI2lWIMS6Y+253r8GDEn5hF7c1h++jod1bCtV1AW+iEMwx2PHUPctXMu67C1uhF8UiXc3MTMEK eBjU4+nwFoHDMrdQX1Wx3WEokqZFhbn54a+qI0VfgOAEiaaeIVk7uVuXbemAA6Cign3b5Q9cPJn+H YJ1bYNLQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mmxQT-001i2z-Rp; Tue, 16 Nov 2021 12:19:02 +0000 Received: from galois.linutronix.de ([193.142.43.55]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mmxQQ-001i2Z-NM for linux-arm-kernel@lists.infradead.org; Tue, 16 Nov 2021 12:19:00 +0000 From: Kurt Kanzenbach DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020; t=1637065134; 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: in-reply-to:in-reply-to:references:references; bh=UxstMfuhLeN5/jMedYHgGOo7lkIy2tu51aj5ea1+e58=; b=uYIo1Vt2tdTSXYliTzbzT6S6b0CrV5Gr4nA28gFpToXF5O1N1997tpEdijFmcGr5/r5rL4 xQiUKC3xXFEah3n3usNFY2C7CBGNXHqVaQw5MHliEHhtRzwQeuRbG81nlA/0ivud4wmSos +zNO658/K2HU0nOQX2NdXim4vTEjkmwpZJ/hwyZZYN5LGpMqv3ENg/yKU3cO1pivb8g7Si 4zNhT0YNQwKWaQM27+jk13Fy32qLvCnkSEHCRg5m12EGfyi6dD9U8ZnRJsHK/qadvmljqg VpeVYeXpg5pCmI1+z53GAKcTL1VtU121h3n6zoCuNUZ4DOfXjklRM9Yv1Eg8fA== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020e; t=1637065134; 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: in-reply-to:in-reply-to:references:references; bh=UxstMfuhLeN5/jMedYHgGOo7lkIy2tu51aj5ea1+e58=; b=ZmZWJTEapiDDCqzr36mlSXGjduoTOQVb+V2nbXl+6eLeRFytjAZFC/T1Kz7KiZGBc2J97j 3beddcUAXXZMzHBQ== To: Yannick Vignon , Vladimir Oltean Cc: "netdev@vger.kernel.org" , Giuseppe Cavallaro , Alexandre Torgue , Jose Abreu , "David S. Miller" , Jakub Kicinski , Maxime Coquelin , "linux-stm32@st-md-mailman.stormreply.com" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , Xiaoliang Yang Subject: Re: [PATCH net] net: stmmac: allow a tc-taprio base-time of zero In-Reply-To: <144f229b-fc8b-92fd-1031-f24fcc740064@oss.nxp.com> References: <20211108202854.1740995-1-vladimir.oltean@nxp.com> <87bl2t3fkq.fsf@kurt> <20211109103504.ahl2djymnevsbhoj@skbuf> <87h7cl1j41.fsf@kurt> <144f229b-fc8b-92fd-1031-f24fcc740064@oss.nxp.com> Date: Tue, 16 Nov 2021 13:18:53 +0100 Message-ID: <87v90s9tua.fsf@kurt> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211116_041858_957047_A7110E69 X-CRM114-Status: GOOD ( 20.15 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: multipart/mixed; boundary="===============5347040554739960984==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============5347040554739960984== Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha512; protocol="application/pgp-signature" --=-=-= Content-Type: text/plain Content-Transfer-Encoding: quoted-printable Hi Yannick, On Tue Nov 09 2021, Yannick Vignon wrote: > Hi Kurt, > > On 11/9/2021 3:47 PM, Kurt Kanzenbach wrote: >> On Tue Nov 09 2021, Vladimir Oltean wrote: >>> On Tue, Nov 09, 2021 at 09:20:53AM +0100, Kurt Kanzenbach wrote: >>>> Hi Vladimir, >>>> >>>> On Mon Nov 08 2021, Vladimir Oltean wrote: >>>>> Commit fe28c53ed71d ("net: stmmac: fix taprio configuration when >>>>> base_time is in the past") allowed some base time values in the past, >>>>> but apparently not all, the base-time value of 0 (Jan 1st 1970) is st= ill >>>>> explicitly denied by the driver. >>>>> >>>>> Remove the bogus check. >>>>> >>>>> Fixes: b60189e0392f ("net: stmmac: Integrate EST with TAPRIO schedule= r API") >>>>> Signed-off-by: Vladimir Oltean >>>> >>>> I've experienced the same problem and wanted to send a patch for >>>> it. Thanks! >>>> >>>> Reviewed-by: Kurt Kanzenbach >>> >>> Cool. So you had that patch queued up? What other stmmac patches do you >>> have queued up? :). >>=20 >> I'm experiencing some problems with XDP using this driver. We're >> currently investigating. > > Could you elaborate a bit? It was a combination of oddities within the PCP based VLAN steering and bugs in my application. No driver issues. The last issue I have is packet loss from time to time. Still debugging. > I've been using XDP a lot with the stmmac driver recently, and while I=20 > did see issues initially, most of them got fixed by using a recent=20 > enough kernel, thanks to the following commits: > . a6451192da2691dcf39507bd ("net: stmmac: fix kernel panic due to NULL=20 > pointer dereference of xsk_pool") > . 2b9fff64f03219d78044d1ab ("net: stmmac: fix kernel panic due to NULL=20 > pointer dereference of buf->xdp") > . 81d0885d68ec427e62044cf4 ("net: stmmac: Fix overall budget calculation= =20 > for rxtx_napi") > > There was one remaining issue for which I need to push a fix: if you=20 > remove an XDP program from an interface while transmitting traffic, you=20 > are likely to trigger a kernel panic. I'll try to push a patch for that=20 > soon. OK, great. Thanks, Kurt --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQJSBAEBCgA8FiEEooWgvezyxHPhdEojeSpbgcuY8KYFAmGToa0eHGt1cnQua2Fu emVuYmFjaEBsaW51dHJvbml4LmRlAAoJEHkqW4HLmPCmNfkP/21fsU6JI1QnFA6+ d5SZvciv0J2GEQ4aSXLeixh3ciUdx66t5+Sug8hen2yCJcEsbl6nCpIg/I0JcpZG OYvSvnj2FC1CfHPqaqeDaslMcUtKtUjsvC2T7RnO5avSgn9otk9p7khEVd1VOWwL Q7Cq+ee5ykcQ5UE180RtkGrL/9qRmRhKYEifvGdyJoQE7VagXXD/wlnk5Wbjvhfw JrXpnBMtiQbrRzgRtYtnjqDMW+M4HGVSDVe0LOIhcR7MD0vm8kyJ15Q38Oepn+xI c6W7mIlhBtw0lDwaX9lsj7Mk5mfF72Op0qzA+FtmYxw1tzH6yZyHpyQK3Jkl4KRu qkF4wmLpac/SWJCWj23lTanxohDpw1Hf3jsKJBD2221/2cFBjbDmEDKCpJQz4cMX 3kZvPse5CyaQ2C/Bf87yIJrkxh+sxfixj0TIECMDgNsTjY8yS0uZ+GffLkfkDzjx /AVs58RoSJBVWcA7zFdd/jTo9kcYe9vx/w8dYumA7XIBy4gRon2M8m11bjMnnSul bwDoWI6VYqgTLTogZWvdPodr3bo+cEDfgBowbfbSuGMqCk0KP0zrIV6TJxDSkQK6 iC0n8vd3+CDv3/ZwbVCK+Zg561D/gvLYyrO9l7xeYBUyAjME6/XDIoBAgHln8eST O4VKyIuoB+PNyWz6O9T1BCkssk1h =M39l -----END PGP SIGNATURE----- --=-=-=-- --===============5347040554739960984== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============5347040554739960984==--