From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jakub Kicinski Subject: Re: [PATCH net-next 0/8] nfp: offload LAG for tc flower egress Date: Fri, 25 May 2018 19:47:28 -0700 Message-ID: <20180525194728.7aa4a116@cakuba> References: <20180524022255.18548-1-jakub.kicinski@netronome.com> <20180525064809.GG2295@nanopsycho> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: davem@davemloft.net, netdev@vger.kernel.org, oss-drivers@netronome.com, Jay Vosburgh , Veaceslav Falico , Andy Gospodarek To: Jiri Pirko , John Hurley Return-path: Received: from mail-qt0-f193.google.com ([209.85.216.193]:34535 "EHLO mail-qt0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030648AbeEZCrd (ORCPT ); Fri, 25 May 2018 22:47:33 -0400 Received: by mail-qt0-f193.google.com with SMTP id m5-v6so8958144qti.1 for ; Fri, 25 May 2018 19:47:32 -0700 (PDT) In-Reply-To: <20180525064809.GG2295@nanopsycho> Sender: netdev-owner@vger.kernel.org List-ID: On Fri, 25 May 2018 08:48:09 +0200, Jiri Pirko wrote: > Thu, May 24, 2018 at 04:22:47AM CEST, jakub.kicinski@netronome.com wrote: > >Hi! > > > >This series from John adds bond offload to the nfp driver. Patch 5 > >exposes the hash type for NETDEV_LAG_TX_TYPE_HASH to make sure nfp > >hashing matches that of the software LAG. This may be unnecessarily > >conservative, let's see what LAG maintainers think :) > > So you need to restrict offload to only certain hash algo? In mlxsw, we > just ignore the lag setting and do some hw default hashing. Would not be > enough? Note that there's a good reason for it, as you see, in team, the > hashing is done in a BPF function and could be totally arbitrary. > Your patchset effectively disables team offload for nfp. My understanding is that the project requirements only called for L3/L4 hash algorithm offload, hence the temptation to err on the side of caution and not offload all the bond configurations. John can provide more details. Not being able to offload team is unfortunate indeed.