From patchwork Sat Dec 21 03:20:01 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Martin Varghese X-Patchwork-Id: 1214414 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming-netdev@ozlabs.org Delivered-To: patchwork-incoming-netdev@ozlabs.org Authentication-Results: ozlabs.org; spf=none (no SPF record) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=netdev-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b="pZlgGOgc"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 47frVR0j9kz9sPV for ; Sat, 21 Dec 2019 14:21:02 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726705AbfLUDVA (ORCPT ); Fri, 20 Dec 2019 22:21:00 -0500 Received: from mail-pj1-f68.google.com ([209.85.216.68]:35753 "EHLO mail-pj1-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726571AbfLUDVA (ORCPT ); Fri, 20 Dec 2019 22:21:00 -0500 Received: by mail-pj1-f68.google.com with SMTP id s7so4977030pjc.0 for ; Fri, 20 Dec 2019 19:21:00 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:subject:date:message-id:in-reply-to:references; bh=bzIfo5QToARCZWLeqG3x/wMqUmg7NXaLyDxKE0gqU2E=; b=pZlgGOgcByJE+H6g8DXuk7ZqU0Au8b68v4dDmLt9WZLM0/v8W3d+HMKboiaNSVSJgC uqB5HcqzP5qH2brEXwvr+okmTbXIvrUVSdIzikcJ9u22BiPTlCMW8JFPYRpwYjloAdz+ Vf6GgaasBcSp3Z5iCX4ccK/Hs/1AAmw6UQyCBPGiyyocbe4M/m9ERFMAUnCI1IXLnsQs BHuVktV5NCcGtz93C38elAoJbbNs88q/V+ypmzwVI6MhxUnCqIrCZU0I4ZuFP7RMVCPl Lc4EOsWZguMd0AVFFVShyuGvYBMrhxMk/5R+FYxaRSE8fuyHxNpmFPODxZZFapB9yIZK HDCQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id:in-reply-to :references; bh=bzIfo5QToARCZWLeqG3x/wMqUmg7NXaLyDxKE0gqU2E=; b=IUpicgHjMDlpk8+lbHFxoAkE4JxLYM54mtBuIcKooEN5kinHLqx2LBMcA7Ob2bdYPD AXbumSeptKbEmBdHA10SGH0PJtHYc5cUg5jXK2s8tg9A4ZQ/FD2Y6rCBsRjM++B2irDP Ah4Dsn00862eNmjvowz0q+pMMA2Tu4H2DnrHHRigpAbIsN4eshkMBn8oSUDGwDKDLkNB BlXi4tGCmcjW1B+I4gVM6xoBTRv5jbhH3N+2izmKblkIOjWsM6uaXXovDtJsTtSRWgZz i1CGf3g7w53s7eHsDkKaolhkDrkXTtf4DXWu/T9ogIcT0lXleHYZxRdDJleRZnwjQEW6 R59w== X-Gm-Message-State: APjAAAXP4AFU64VyqyNFNEtTu/gLUmtUlwGs8HXbkWF9M4Z1HDPFmPrN /thHteR/TqOrM5Tb9kQq+ecz5LtN X-Google-Smtp-Source: APXvYqwE8Bb0xW7Cjq3rZpYo1eNCKHuvCW22VtrTXX+dJBp3AoqiXADo2EzKRJHrI5ZpJbPzV72+AQ== X-Received: by 2002:a17:90a:cb8c:: with SMTP id a12mr20392640pju.71.1576898459589; Fri, 20 Dec 2019 19:20:59 -0800 (PST) Received: from localhost.localdomain ([42.109.147.248]) by smtp.gmail.com with ESMTPSA id w6sm15639951pfq.99.2019.12.20.19.20.57 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Fri, 20 Dec 2019 19:20:59 -0800 (PST) From: Martin Varghese To: netdev@vger.kernel.org, davem@davemloft.net, pshelar@ovn.org, scott.drennan@nokia.com, jbenc@redhat.com, martin.varghese@nokia.com Subject: [PATCH net-next v5 1/3] net: skb_mpls_push() modified to allow MPLS header push at start of packet. Date: Sat, 21 Dec 2019 08:50:01 +0530 Message-Id: <5dbc2dbc222ff778861ef08b4e0a68a49a7afeb1.1576896417.git.martin.varghese@nokia.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: References: Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org From: Martin Varghese The existing skb_mpls_push() implementation always inserts mpls header after the mac header. L2 VPN use cases requires MPLS header to be inserted before the ethernet header as the ethernet packet gets tunnelled inside MPLS header in those cases. Signed-off-by: Martin Varghese Acked-by: Pravin B Shelar --- Changes in v2: - Fixed comments section of skb_mpls_push(). - Added skb_reset_mac_len() in skb_mpls_push(). The mac len changes when MPLS header in inserted at the start of the packet. net/core/skbuff.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/net/core/skbuff.c b/net/core/skbuff.c index 973a71f..d90c827 100644 --- a/net/core/skbuff.c +++ b/net/core/skbuff.c @@ -5472,12 +5472,15 @@ static void skb_mod_eth_type(struct sk_buff *skb, struct ethhdr *hdr, } /** - * skb_mpls_push() - push a new MPLS header after the mac header + * skb_mpls_push() - push a new MPLS header after mac_len bytes from start of + * the packet * * @skb: buffer * @mpls_lse: MPLS label stack entry to push * @mpls_proto: ethertype of the new MPLS header (expects 0x8847 or 0x8848) * @mac_len: length of the MAC header + * @ethernet: flag to indicate if the resulting packet after skb_mpls_push is + * ethernet * * Expects skb->data at mac header. * @@ -5501,7 +5504,7 @@ int skb_mpls_push(struct sk_buff *skb, __be32 mpls_lse, __be16 mpls_proto, return err; if (!skb->inner_protocol) { - skb_set_inner_network_header(skb, mac_len); + skb_set_inner_network_header(skb, skb_network_offset(skb)); skb_set_inner_protocol(skb, skb->protocol); } @@ -5510,6 +5513,7 @@ int skb_mpls_push(struct sk_buff *skb, __be32 mpls_lse, __be16 mpls_proto, mac_len); skb_reset_mac_header(skb); skb_set_network_header(skb, mac_len); + skb_reset_mac_len(skb); lse = mpls_hdr(skb); lse->label_stack_entry = mpls_lse; From patchwork Sat Dec 21 03:20:23 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Martin Varghese X-Patchwork-Id: 1214415 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming-netdev@ozlabs.org Delivered-To: patchwork-incoming-netdev@ozlabs.org Authentication-Results: ozlabs.org; spf=none (no SPF record) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=netdev-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b="f5oa8u80"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 47frVs2xv5z9sPV for ; Sat, 21 Dec 2019 14:21:25 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726717AbfLUDVX (ORCPT ); Fri, 20 Dec 2019 22:21:23 -0500 Received: from mail-pj1-f68.google.com ([209.85.216.68]:34335 "EHLO mail-pj1-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726537AbfLUDVX (ORCPT ); Fri, 20 Dec 2019 22:21:23 -0500 Received: by mail-pj1-f68.google.com with SMTP id s94so4476174pjc.1 for ; Fri, 20 Dec 2019 19:21:22 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:subject:date:message-id:in-reply-to:references; bh=b3hiJ3A3rMjBROFjEn7dlrVf4haGfG2YX4vYzJvUu0g=; b=f5oa8u80m09fA+W5dUO9DYzR/TIh/7dMBBTH98+gLslroANsa8qxCAHCqbgEiCI+S6 0yPF5vnjAHjo+p+ncQNHVmGIOtPkSqJITmrtj7A8FnQ9nuhR6DPbob479+hqZ5AW1J94 WvnYcqQBkOyRqNfQMjoWpin07CDRHAYH3+88stP+K+RG1G7ttK8caBQ0KuNoSQZmrcpq B1gtjezC7mT/xEz0iLug7mBaAqN3C6ivqJluwqwI2M6CU8SwmhA4CtCJHCIaaIlUGauf 8/EmoVCISDaS2i0ZOeo2FALkPrXJDOaI648PSRhvlo22vr5bO6iH0Zi7iiXqn8VWWZuv 5D5g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id:in-reply-to :references; bh=b3hiJ3A3rMjBROFjEn7dlrVf4haGfG2YX4vYzJvUu0g=; b=XfycsWTdzUsdJaP7XI/+nLXp2QdFOpIbRb7h/fvbDdBMbJFt6bdsiOp/SrobDMUt7m mYY7+YifiCtbfM74T2XdROezljAGLrfQe/Kf9yu9Z95jDQIlfdoD3PgXi9BlrBC2t0bw 81nOZK05eb2oQ9xCOjVPKbPvwGBNRcJ4prsvTC6sp5Jewjt8bEKXOlKATn8Ad7xC5toZ fnB6tOC2LLXDKOCfJ/hfpLTqjWK0YV95bA5CDFSlIN1/cjErYdhnkpTErVPADiPF9b1L 9rBqUPJk3F8iTWxU3/bAjoDWCJfie0luWkm1IDk+SBcXa6bd+mniinmMUEWZA+6qn51B AmfA== X-Gm-Message-State: APjAAAW3tz9b33ny15VdwZy+9Fscs+R5Wwh+Sub9rjeN3Lt22ERFTHbC snLYPSdYHh9VaRzs7uxb20+/+mDQ X-Google-Smtp-Source: APXvYqwxtnN8wHU9ZNATg0HmJdEROyGG/T9PC7q6+sVMAYxCi0NJ4YtPhfPxMuZ2AXKedrd9Ba7XwA== X-Received: by 2002:a17:90a:1a0d:: with SMTP id 13mr817793pjk.129.1576898482471; Fri, 20 Dec 2019 19:21:22 -0800 (PST) Received: from localhost.localdomain ([42.109.147.248]) by smtp.gmail.com with ESMTPSA id g9sm15080287pfm.150.2019.12.20.19.21.20 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Fri, 20 Dec 2019 19:21:22 -0800 (PST) From: Martin Varghese To: netdev@vger.kernel.org, davem@davemloft.net, pshelar@ovn.org, scott.drennan@nokia.com, jbenc@redhat.com, martin.varghese@nokia.com Subject: [PATCH net-next v5 2/3] net: Rephrased comments section of skb_mpls_pop() Date: Sat, 21 Dec 2019 08:50:23 +0530 Message-Id: X-Mailer: git-send-email 1.9.1 In-Reply-To: References: Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org From: Martin Varghese Rephrased comments section of skb_mpls_pop() to align it with comments section of skb_mpls_push(). Signed-off-by: Martin Varghese Acked-by: Pravin B Shelar --- net/core/skbuff.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/core/skbuff.c b/net/core/skbuff.c index d90c827..44b0894 100644 --- a/net/core/skbuff.c +++ b/net/core/skbuff.c @@ -5533,7 +5533,7 @@ int skb_mpls_push(struct sk_buff *skb, __be32 mpls_lse, __be16 mpls_proto, * @skb: buffer * @next_proto: ethertype of header after popped MPLS header * @mac_len: length of the MAC header - * @ethernet: flag to indicate if ethernet header is present in packet + * @ethernet: flag to indicate if the packet is ethernet * * Expects skb->data at mac header. * From patchwork Sat Dec 21 03:20:46 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Martin Varghese X-Patchwork-Id: 1214416 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming-netdev@ozlabs.org Delivered-To: patchwork-incoming-netdev@ozlabs.org Authentication-Results: ozlabs.org; spf=none (no SPF record) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=netdev-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b="ivO35WQ0"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 47frWJ0ZPrz9sPV for ; Sat, 21 Dec 2019 14:21:47 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726733AbfLUDVq (ORCPT ); Fri, 20 Dec 2019 22:21:46 -0500 Received: from mail-pf1-f195.google.com ([209.85.210.195]:44127 "EHLO mail-pf1-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726537AbfLUDVp (ORCPT ); Fri, 20 Dec 2019 22:21:45 -0500 Received: by mail-pf1-f195.google.com with SMTP id 195so5402153pfw.11 for ; Fri, 20 Dec 2019 19:21:45 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:subject:date:message-id:in-reply-to:references; bh=EZStfT8Y3+w7g5IMR3LS7oaTKGYlLTmJ96boV7VDtjw=; b=ivO35WQ0tjPT9z3TRQ79vwM1fCy9zhYYG9GjCHK6Zg1C5jsS407xuY8LF91O99N4Cr U7m/jBJgpau2BNuf1dgniVOPJL1FBB/JAymHm4vsyW7HxewCOLjV1z1g1FiNJomq++DM J7mnNL1eYgKtQ/8X6rfKtuuBypTZ9fAT5n4i1SK95uSL1jl+b8iv40ThEQJK0NkpJ/nY UA8ZvQT2mvk26vB0AIWsax/BapQEGF1a2BbqlRwcd12ww+yjvLl7kc/m7N73l7fPd+CN BIotpFRPcly3rkYJx1V308VceMhUUTIlGx3SyEEYgCHcJ2rGfVVkJBscEe7MvP+1NQGX EmqQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id:in-reply-to :references; bh=EZStfT8Y3+w7g5IMR3LS7oaTKGYlLTmJ96boV7VDtjw=; b=P1p1GxKGFpsE9bwKtqvfmQ+yBf+HaZHda//6Vl1TslyOGh9DTQ37JJuLiUtdUlnSFu aO+dD1uMXmjmr0TUpzJBUH1EIO9YkYyTI44QGVZ70zfm/JadFU2Y/XhskJ+40wAQReNc 3yDt3u6SyTYEPwAMJn2XumZDWekJWkW8KlyrvLKtyyMEkd78nF1dpL6bsKDFQF4TisBX qP/G++UoFHtkTMRJoF074LVYj0tZeRBrWQA0FRKG/gB/74mpSqdgA6fbd9cuDXTKkhPt Kmjer8n6YpdZ13f4YM7htApFmDF+lROdwkm15MmlOh0rBY/KLDmvm+KgJeW7OwTDx0VV Oytg== X-Gm-Message-State: APjAAAXKl6YdjS9Lbp3htbaZOBVA9SUHBZpc4p6olwhTCz9F2y63BxJg e8mt4mWB6n7+kD7GNtGyWSUTrZ73 X-Google-Smtp-Source: APXvYqxMvlvq0d5fByk45clktHLkQ9gFKV952vXLOvihDuUfcfTnRlhwQryB3+/56QDnjXAsoyAdIg== X-Received: by 2002:a63:1807:: with SMTP id y7mr17587842pgl.94.1576898504876; Fri, 20 Dec 2019 19:21:44 -0800 (PST) Received: from localhost.localdomain ([42.109.147.248]) by smtp.gmail.com with ESMTPSA id 200sm14806170pfz.121.2019.12.20.19.21.42 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Fri, 20 Dec 2019 19:21:44 -0800 (PST) From: Martin Varghese To: netdev@vger.kernel.org, davem@davemloft.net, pshelar@ovn.org, scott.drennan@nokia.com, jbenc@redhat.com, martin.varghese@nokia.com Subject: [PATCH net-next v5 3/3] openvswitch: New MPLS actions for layer 2 tunnelling Date: Sat, 21 Dec 2019 08:50:46 +0530 Message-Id: <4cb29736c3fad6d660df246ef75623db0bd4a864.1576896417.git.martin.varghese@nokia.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: References: Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org From: Martin Varghese The existing PUSH MPLS action inserts MPLS header between ethernet header and the IP header. Though this behaviour is fine for L3 VPN where an IP packet is encapsulated inside a MPLS tunnel, it does not suffice the L2 VPN (l2 tunnelling) requirements. In L2 VPN the MPLS header should encapsulate the ethernet packet. The new mpls action ADD_MPLS inserts MPLS header at the start of the packet or at the start of the l3 header depending on the value of l3 tunnel flag in the ADD_MPLS arguments. POP_MPLS action is extended to support ethertype 0x6558. Signed-off-by: Martin Varghese Acked-by: Pravin B Shelar --- Changes in v2: - PTAP_POP_MPLS action removed. - Special handling for ethertype 0 added in PUSH_MPLS. - Refactored push_mpls function to cater existing push_mpls and ptap_push_mpls actions. - mac len to specify the MPLS header location added in PTAP_PUSH_MPLS arguments. Changes in v3: - Special handling for ethertype 0 removed. - Added support for ether type 0x6558. - Removed mac len from PTAP_PUSH_MPLS argument list - used l2_tun flag to distinguish l2 and l3 tunnelling. - Extended PTAP_PUSH_MPLS handling to cater PUSH_MPLS action also. Changes in v4: - Removed extra blank lines. - Replaced bool l2_tun with u16 tun flags in struct ovs_action_ptap_push_mpls. Changes in v5: - Renamed PTAP_PUSH_MPLS action to ADD_MPLS. - Replaced l2 tunnel flag with l3 tunnel flag. - In ADD_MPLS configuration, the code to check for l2 header is changed from (mac_proto != MAC_PROTO_NONE) to (mac_proto == MAC_PROTO_ETHERNET). include/uapi/linux/openvswitch.h | 31 +++++++++++++++++++++++++++++++ net/openvswitch/actions.c | 30 ++++++++++++++++++++++++------ net/openvswitch/flow_netlink.c | 34 ++++++++++++++++++++++++++++++++++ 3 files changed, 89 insertions(+), 6 deletions(-) diff --git a/include/uapi/linux/openvswitch.h b/include/uapi/linux/openvswitch.h index a87b44c..ae2bff1 100644 --- a/include/uapi/linux/openvswitch.h +++ b/include/uapi/linux/openvswitch.h @@ -673,6 +673,32 @@ struct ovs_action_push_mpls { }; /** + * struct ovs_action_add_mpls - %OVS_ACTION_ATTR_ADD_MPLS action + * argument. + * @mpls_lse: MPLS label stack entry to push. + * @mpls_ethertype: Ethertype to set in the encapsulating ethernet frame. + * @tun_flags: MPLS tunnel attributes. + * + * The only values @mpls_ethertype should ever be given are %ETH_P_MPLS_UC and + * %ETH_P_MPLS_MC, indicating MPLS unicast or multicast. Other are rejected. + */ +struct ovs_action_add_mpls { + __be32 mpls_lse; + __be16 mpls_ethertype; /* Either %ETH_P_MPLS_UC or %ETH_P_MPLS_MC */ + __u16 tun_flags; +}; + +#define OVS_MPLS_L3_TUNNEL_FLAG_MASK (1 << 0) /* Flag to specify the place of + * insertion of MPLS header. + * When false, the MPLS header + * will be inserted at the start + * of the packet. + * When true, the MPLS header + * will be inserted at the start + * of the l3 header. + */ + +/** * struct ovs_action_push_vlan - %OVS_ACTION_ATTR_PUSH_VLAN action argument. * @vlan_tpid: Tag protocol identifier (TPID) to push. * @vlan_tci: Tag control identifier (TCI) to push. The CFI bit must be set @@ -892,6 +918,10 @@ struct check_pkt_len_arg { * @OVS_ACTION_ATTR_CHECK_PKT_LEN: Check the packet length and execute a set * of actions if greater than the specified packet length, else execute * another set of actions. + * @OVS_ACTION_ATTR_ADD_MPLS: Push a new MPLS label stack entry at the + * start of the packet or at the start of the l3 header depending on the value + * of l3 tunnel flag in the tun_flags field of OVS_ACTION_ATTR_ADD_MPLS + * argument. * * Only a single header can be set with a single %OVS_ACTION_ATTR_SET. Not all * fields within a header are modifiable, e.g. the IPv4 protocol and fragment @@ -927,6 +957,7 @@ enum ovs_action_attr { OVS_ACTION_ATTR_METER, /* u32 meter ID. */ OVS_ACTION_ATTR_CLONE, /* Nested OVS_CLONE_ATTR_*. */ OVS_ACTION_ATTR_CHECK_PKT_LEN, /* Nested OVS_CHECK_PKT_LEN_ATTR_*. */ + OVS_ACTION_ATTR_ADD_MPLS, /* struct ovs_action_add_mpls. */ __OVS_ACTION_ATTR_MAX, /* Nothing past this will be accepted * from userspace. */ diff --git a/net/openvswitch/actions.c b/net/openvswitch/actions.c index 4c83954..7fbfe2a 100644 --- a/net/openvswitch/actions.c +++ b/net/openvswitch/actions.c @@ -161,16 +161,17 @@ static int do_execute_actions(struct datapath *dp, struct sk_buff *skb, const struct nlattr *attr, int len); static int push_mpls(struct sk_buff *skb, struct sw_flow_key *key, - const struct ovs_action_push_mpls *mpls) + __be32 mpls_lse, __be16 mpls_ethertype, __u16 mac_len) { int err; - err = skb_mpls_push(skb, mpls->mpls_lse, mpls->mpls_ethertype, - skb->mac_len, - ovs_key_mac_proto(key) == MAC_PROTO_ETHERNET); + err = skb_mpls_push(skb, mpls_lse, mpls_ethertype, mac_len, !!mac_len); if (err) return err; + if (!mac_len) + key->mac_proto = MAC_PROTO_NONE; + invalidate_flow_key(key); return 0; } @@ -185,6 +186,9 @@ static int pop_mpls(struct sk_buff *skb, struct sw_flow_key *key, if (err) return err; + if (ethertype == htons(ETH_P_TEB)) + key->mac_proto = MAC_PROTO_ETHERNET; + invalidate_flow_key(key); return 0; } @@ -1229,10 +1233,24 @@ static int do_execute_actions(struct datapath *dp, struct sk_buff *skb, execute_hash(skb, key, a); break; - case OVS_ACTION_ATTR_PUSH_MPLS: - err = push_mpls(skb, key, nla_data(a)); + case OVS_ACTION_ATTR_PUSH_MPLS: { + struct ovs_action_push_mpls *mpls = nla_data(a); + + err = push_mpls(skb, key, mpls->mpls_lse, + mpls->mpls_ethertype, skb->mac_len); break; + } + case OVS_ACTION_ATTR_ADD_MPLS: { + struct ovs_action_add_mpls *mpls = nla_data(a); + __u16 mac_len = 0; + + if (mpls->tun_flags & OVS_MPLS_L3_TUNNEL_FLAG_MASK) + mac_len = skb->mac_len; + err = push_mpls(skb, key, mpls->mpls_lse, + mpls->mpls_ethertype, mac_len); + break; + } case OVS_ACTION_ATTR_POP_MPLS: err = pop_mpls(skb, key, nla_get_be16(a)); break; diff --git a/net/openvswitch/flow_netlink.c b/net/openvswitch/flow_netlink.c index 65c2e34..7da4230 100644 --- a/net/openvswitch/flow_netlink.c +++ b/net/openvswitch/flow_netlink.c @@ -79,6 +79,7 @@ static bool actions_may_change_flow(const struct nlattr *actions) case OVS_ACTION_ATTR_SET_MASKED: case OVS_ACTION_ATTR_METER: case OVS_ACTION_ATTR_CHECK_PKT_LEN: + case OVS_ACTION_ATTR_ADD_MPLS: default: return true; } @@ -3005,6 +3006,7 @@ static int __ovs_nla_copy_actions(struct net *net, const struct nlattr *attr, [OVS_ACTION_ATTR_METER] = sizeof(u32), [OVS_ACTION_ATTR_CLONE] = (u32)-1, [OVS_ACTION_ATTR_CHECK_PKT_LEN] = (u32)-1, + [OVS_ACTION_ATTR_ADD_MPLS] = sizeof(struct ovs_action_add_mpls), }; const struct ovs_action_push_vlan *vlan; int type = nla_type(a); @@ -3072,6 +3074,33 @@ static int __ovs_nla_copy_actions(struct net *net, const struct nlattr *attr, case OVS_ACTION_ATTR_RECIRC: break; + case OVS_ACTION_ATTR_ADD_MPLS: { + const struct ovs_action_add_mpls *mpls = nla_data(a); + + if (!eth_p_mpls(mpls->mpls_ethertype)) + return -EINVAL; + + if (mpls->tun_flags & OVS_MPLS_L3_TUNNEL_FLAG_MASK) { + if (vlan_tci & htons(VLAN_CFI_MASK) || + (eth_type != htons(ETH_P_IP) && + eth_type != htons(ETH_P_IPV6) && + eth_type != htons(ETH_P_ARP) && + eth_type != htons(ETH_P_RARP) && + !eth_p_mpls(eth_type))) + return -EINVAL; + mpls_label_count++; + } else { + if (mac_proto == MAC_PROTO_ETHERNET) { + mpls_label_count = 1; + mac_proto = MAC_PROTO_NONE; + } else { + mpls_label_count++; + } + } + eth_type = mpls->mpls_ethertype; + break; + } + case OVS_ACTION_ATTR_PUSH_MPLS: { const struct ovs_action_push_mpls *mpls = nla_data(a); @@ -3109,6 +3138,11 @@ static int __ovs_nla_copy_actions(struct net *net, const struct nlattr *attr, * recirculation. */ proto = nla_get_be16(a); + + if (proto == htons(ETH_P_TEB) && + mac_proto != MAC_PROTO_NONE) + return -EINVAL; + mpls_label_count--; if (!eth_p_mpls(proto) || !mpls_label_count)