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 X-Spam-Level: X-Spam-Status: No, score=-3.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id E4570C43331 for ; Fri, 6 Sep 2019 00:04:13 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 12890206BB for ; Fri, 6 Sep 2019 00:04:14 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2388922AbfIFAEN (ORCPT ); Thu, 5 Sep 2019 20:04:13 -0400 Received: from correo.us.es ([193.147.175.20]:59522 "EHLO mail.us.es" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1732493AbfIFAEN (ORCPT ); Thu, 5 Sep 2019 20:04:13 -0400 Received: from antivirus1-rhel7.int (unknown [192.168.2.11]) by mail.us.es (Postfix) with ESMTP id 6B8CADA3D0 for ; Fri, 6 Sep 2019 02:04:09 +0200 (CEST) Received: from antivirus1-rhel7.int (localhost [127.0.0.1]) by antivirus1-rhel7.int (Postfix) with ESMTP id 5F93BB7FFE for ; Fri, 6 Sep 2019 02:04:09 +0200 (CEST) Received: by antivirus1-rhel7.int (Postfix, from userid 99) id 51878CA0F3; Fri, 6 Sep 2019 02:04:09 +0200 (CEST) Received: from antivirus1-rhel7.int (localhost [127.0.0.1]) by antivirus1-rhel7.int (Postfix) with ESMTP id 36FE0DA8E8; Fri, 6 Sep 2019 02:04:07 +0200 (CEST) Received: from 192.168.1.97 (192.168.1.97) by antivirus1-rhel7.int (F-Secure/fsigk_smtp/550/antivirus1-rhel7.int); Fri, 06 Sep 2019 02:04:07 +0200 (CEST) X-Virus-Status: clean(F-Secure/fsigk_smtp/550/antivirus1-rhel7.int) Received: from salvia.here (sys.soleta.eu [212.170.55.40]) (Authenticated sender: pneira@us.es) by entrada.int (Postfix) with ESMTPA id CDB164265A5A; Fri, 6 Sep 2019 02:04:06 +0200 (CEST) X-SMTPAUTHUS: auth mail.us.es From: Pablo Neira Ayuso To: netfilter-devel@vger.kernel.org Cc: davem@davemloft.net, netdev@vger.kernel.org, jakub.kicinski@netronome.com, jiri@resnulli.us, saeedm@mellanox.com, vishal@chelsio.com, vladbu@mellanox.com Subject: [PATCH net-next,v3 0/4] flow_offload: update mangle action representation Date: Fri, 6 Sep 2019 02:03:59 +0200 Message-Id: <20190906000403.3701-1-pablo@netfilter.org> X-Mailer: git-send-email 2.11.0 X-Virus-Scanned: ClamAV using ClamSMTP Sender: netfilter-devel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netfilter-devel@vger.kernel.org This patch updates the mangle action representation: Patch 1) Undo bitwise NOT operation on the mangle mask (coming from tc pedit userspace). Patch 2) mangle value &= mask from the front-end side. Patch 3) adjust offset, length and coalesce consecutive pedit keys into one single action. Patch 4) add support for payload mangling for netfilter. After this patchset: * Offset to payload does not need to be on the 32-bits boundaries anymore. This patchset adds front-end code to adjust the offset and length coming from the tc pedit representation, so drivers get an exact header field offset and length. * This new front-end code coalesces consecutive pedit actions into one single action, so drivers can mangle IPv6 and ethernet address fields in one go, instead once for each 32-bit word. On the driver side, diffstat -t shows that drivers code to deal with payload mangling gets simplified: INSERTED,DELETED,MODIFIED,FILENAME 46,116,0,drivers/net/ethernet/chelsio/cxgb4/cxgb4_tc_flower.c (-70 LOC) 12,28,0,drivers/net/ethernet/chelsio/cxgb4/cxgb4_tc_flower.h (-16 LOC) 26,54,0,drivers/net/ethernet/mellanox/mlx5/core/en_tc.c (-27 LOC) 89,111,0,drivers/net/ethernet/netronome/nfp/flower/action.c (-17 LOC) While, on the front-end side the balance is the following: 123,22,0,net/sched/cls_api.c (+101 LOC) Changes since v2: * Fix is_action_keys_supported() breakage in mlx5 reported by Vlad Buslov. Pablo Neira Ayuso (4): net: flow_offload: flip mangle action mask net: flow_offload: bitwise AND on mangle action value field net: flow_offload: mangle action at byte level netfilter: nft_payload: packet mangling offload support .../net/ethernet/chelsio/cxgb4/cxgb4_tc_flower.c | 163 +++++----------- .../net/ethernet/chelsio/cxgb4/cxgb4_tc_flower.h | 40 ++-- drivers/net/ethernet/mellanox/mlx5/core/en_tc.c | 80 +++----- drivers/net/ethernet/netronome/nfp/flower/action.c | 204 ++++++++++----------- include/net/flow_offload.h | 7 +- net/netfilter/nft_payload.c | 73 ++++++++ net/sched/cls_api.c | 144 ++++++++++++--- 7 files changed, 378 insertions(+), 333 deletions(-) -- 2.11.0