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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 02463C433EF for ; Thu, 7 Apr 2022 17:29:58 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1346485AbiDGRb4 (ORCPT ); Thu, 7 Apr 2022 13:31:56 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36060 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1346401AbiDGRbq (ORCPT ); Thu, 7 Apr 2022 13:31:46 -0400 Received: from vps0.lunn.ch (vps0.lunn.ch [185.16.172.187]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8F7E666CB3 for ; Thu, 7 Apr 2022 10:28:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lunn.ch; s=20171124; h=In-Reply-To:Content-Disposition:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:From:Sender:Reply-To:Subject: Date:Message-ID:To:Cc:MIME-Version:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description:Content-Disposition:In-Reply-To:References; bh=+CcXuAo3p6A3oY3eBmd5VnRnmY9mVb6FELo/+Al/yL8=; b=Wm6nmdHrggU7RYx9EbUjwSxr0Z g1LZtbl1NG7YHZh8crIPPm1vTxuLCkLRRAjNVHV/aszw/VieTFPXYUBDmcYDyeYN/85b+JT/X93YX cupDZFwHLxGu79J2o1djWEyaC4mZESgI22qNtgRY6wZSitF2E3YkilNN0WeXwC24Z+eU=; Received: from andrew by vps0.lunn.ch with local (Exim 4.94.2) (envelope-from ) id 1ncVvp-00EgQL-JM; Thu, 07 Apr 2022 19:28:29 +0200 Date: Thu, 7 Apr 2022 19:28:29 +0200 From: Andrew Lunn To: Felix Fietkau Cc: David Miller , Jakub Kicinski , Paolo Abeni , netdev@vger.kernel.org, matthias.bgg@gmail.com, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org Subject: Re: [PATCH v2 00/14] MediaTek SoC flow offload improvements + wireless support Message-ID: References: <20220405195755.10817-1-nbd@nbd.name> <164925181755.19554.1627872315624407424.git-patchwork-notify@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On Thu, Apr 07, 2022 at 07:00:36PM +0200, Felix Fietkau wrote: > On 07.04.22 17:57, Andrew Lunn wrote: > > On Wed, Apr 06, 2022 at 01:30:17PM +0000, patchwork-bot+netdevbpf@kernel.org wrote: > > > Hello: > > > > > > This series was applied to netdev/net-next.git (master) > > > by David S. Miller : > > > > > > On Tue, 5 Apr 2022 21:57:41 +0200 you wrote: > > > > This series contains the following improvements to mediatek ethernet flow > > > > offload support: > > > > > - support dma-coherent on ethernet to improve performance > > > > - add ipv6 offload support > > > > - rework hardware flow table entry handling to improve dealing with hash > > > > collisions and competing flows > > > > - support creating offload entries from user space > > > > - support creating offload entries with just source/destination mac address, > > > > vlan and output device information > > > > - add driver changes for supporting the Wireless Ethernet Dispatch core, > > > > which can be used to offload flows from ethernet to MT7915 PCIe WLAN > > > > devices > > > > Hi David > > > > It seems very early to merge this. The discussion of if the files are > > even in the right places has not even finished. And Arnd seems to not > > want parts of this in his subsystem. And there are some major > > architecture issues which need discussing... > > > > I think you should revert this. > How about I simply send follow-up patches that move the relevant pieces to > net? There has just been comments from Rob about the binding. I've not yet looked at the code, but if i remember correctly, v1 had some interaction with the DSA tagger, so i do want to look at it. I'm also wondering if there is anything common here with IPA. It is an accelerator which sits between the WiFi and the mobile phone baseband device. I really would prefer that a proper review of this code was made, by netdev people, and the bigger architecture questions looked at. So far, all the reviewers have been from outside netdev. Andrew