From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiri Pirko Subject: Re: [patch net-next v2 01/10] cls_bpf: move prog offload->netdev check into drivers Date: Mon, 13 Nov 2017 07:25:38 +0100 Message-ID: <20171113062538.GA1986@nanopsycho> References: <20171112155604.24061-1-jiri@resnulli.us> <20171112155604.24061-2-jiri@resnulli.us> <20171112181418.3ca6ff54@cakuba> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: netdev@vger.kernel.org, davem@davemloft.net, jhs@mojatatu.com, xiyou.wangcong@gmail.com, mlxsw@mellanox.com, andrew@lunn.ch, vivien.didelot@savoirfairelinux.com, f.fainelli@gmail.com, ast@kernel.org, daniel@iogearbox.net, simon.horman@netronome.com, pieter.jansenvanvuuren@netronome.com, john.hurley@netronome.com To: Jakub Kicinski Return-path: Received: from mail-wr0-f193.google.com ([209.85.128.193]:52611 "EHLO mail-wr0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751236AbdKMGZk (ORCPT ); Mon, 13 Nov 2017 01:25:40 -0500 Received: by mail-wr0-f193.google.com with SMTP id j23so13411586wra.9 for ; Sun, 12 Nov 2017 22:25:40 -0800 (PST) Content-Disposition: inline In-Reply-To: <20171112181418.3ca6ff54@cakuba> Sender: netdev-owner@vger.kernel.org List-ID: Mon, Nov 13, 2017 at 03:14:18AM CET, jakub.kicinski@netronome.com wrote: >On Sun, 12 Nov 2017 16:55:55 +0100, Jiri Pirko wrote: >> From: Jiri Pirko >> >> In order to remove tp->q usage in cls_bpf, the offload->netdev check >> needs to be moved to individual drivers as only they will have access >> to appropriate struct net_device. >> >> Signed-off-by: Jiri Pirko > >This seems not entirely correct and it adds unnecessary code. I think What is not correct? >the XDP and cls_bpf handling could be unified, making way for binding >the same program to multiple ports of the same device. Would you mind >waiting a day for me to send corrections to BPF offload? Well I'm trying to get this in before net-next closes...