Received: by 2002:a05:6a11:4021:0:0:0:0 with SMTP id ky33csp4967222pxb; Tue, 28 Sep 2021 07:54:21 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzmtFSotPvqm5YgsBjzbeubMT148/rsJDB4uYHVD+Rp3jm83iffxrA4bfH1DdoHNQmfhnE9 X-Received: by 2002:a05:6a00:807:b0:44b:586b:8edb with SMTP id m7-20020a056a00080700b0044b586b8edbmr5928080pfk.9.1632840860768; Tue, 28 Sep 2021 07:54:20 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1632840860; cv=none; d=google.com; s=arc-20160816; b=Ipe+MIXTWWyHY17Vaalw33pm+1oMw69/Z2RxBYZFVx0toUHl8UwYHuu/tfP3mw6RIZ LZV13m8vcny09jIBEoOgjLI0N3kn+97i4AAK4jKsj1y92lqRnTRba2njZSZyf6HKtsir lMMKmFaTeY3dgvoVkh7+A9a7IjAJYHNHHYNK3THID8IOpcJXCqYVC7bxWuyJXQs+URXz sunbzccK6wukLP5Fue0iqDuJuMzgCDoCif6K1FjPfZtRURDEIQuvpzxZMZlGHFEt4dsN WbfQ5cBnjpYd33VU6rxnWLxnzAfevudXTmSlPOuj5kmZ0fNTVmTOUY9KfVk17qfXRKoI E8CQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:organization :from:references:cc:to:subject:reply-to:dkim-signature; bh=8k4vzjTEvvF0RcP+mluO0fPYoFQpr61i483WOpfKr1k=; b=aq7Fab5JebD/XN1JgqXc77SqftPtgz0S4Wy2VXstEZT1C7eXTKQFg+s+30HhDjF2Le F0biMsXB5ZsQA8i7g7UOrgAzr0l3TgrweSCdDyzgslfOanNtPzZ3WP40/mlBIRjTw1mZ uCPPPu0nPRr8KYl/QpQns5WoBJh+rb5zTcSiLC4ADPWMxWPmS7amfgroESdEQgZ/4uyM FCwkfyaZBkZWLBs2gjF1VnWELs82K8U5v8AAiymwuDbxnCGDSNlav/vB5NfewYDwrV1d 4g6nI3FM8gB9ok2UUo7Z2ssrlhVcO+ihUFVpM7OdKGtHhRMs3j1eUF4oP4Bu8DJOlPJr vEgg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@6wind.com header.s=google header.b=TpeYbEuJ; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=6wind.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id z21si29190196pgv.564.2021.09.28.07.54.03; Tue, 28 Sep 2021 07:54:20 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@6wind.com header.s=google header.b=TpeYbEuJ; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=6wind.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S241407AbhI1Ox2 (ORCPT + 99 others); Tue, 28 Sep 2021 10:53:28 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57730 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S241152AbhI1OxZ (ORCPT ); Tue, 28 Sep 2021 10:53:25 -0400 Received: from mail-wm1-x329.google.com (mail-wm1-x329.google.com [IPv6:2a00:1450:4864:20::329]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id ABA79C06161C for ; Tue, 28 Sep 2021 07:51:43 -0700 (PDT) Received: by mail-wm1-x329.google.com with SMTP id b204-20020a1c80d5000000b0030cd967c674so2120337wmd.0 for ; Tue, 28 Sep 2021 07:51:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind.com; s=google; h=reply-to:subject:to:cc:references:from:organization:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=8k4vzjTEvvF0RcP+mluO0fPYoFQpr61i483WOpfKr1k=; b=TpeYbEuJJ4BseQvOvsvFWNckRy2BTd89M8cj9jh37927UcZldckqMJlKOP/SF4KkEy CQPdqeyZbTIxb2pF0lgUpKUsXhXVEptfWKthyBYpNh+pPjkN5Dg+olzMgwSoRNV7QloU LCtYS+3x50jkmVUMP6mR8Ci5d16ZUIN7/WE5kvjc1EAuMKc17IrCPEYtlpOLNAq9PI5E s/yeti1LAQlHBI8WhQzlOwD7Efhby2VXhPz+7tfA38omrJDQxjzjBu0IL8unvJ0RObNJ jOYXqz0QVbZW7uv2HbaVJS8BYkjwQzj6noZiofXvlQiZ3V2YZtY1yASn77l6YQki/mEQ nPpQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:reply-to:subject:to:cc:references:from :organization:message-id:date:user-agent:mime-version:in-reply-to :content-language:content-transfer-encoding; bh=8k4vzjTEvvF0RcP+mluO0fPYoFQpr61i483WOpfKr1k=; b=K7xJdT/KUJ70xf2DPgm8ffssYBOUANU6A6dUFVM33TN4bOGYTxscCQEA+L4guQnumb mx2rws0uPBUhmidHRAnJoZyCPE2A/NY/XtGu6X5pXKaFsuIPylSFXyDzoWDHB05WxHpf 9LnmGYkvEq6GVsVxG9zY9Unl+obRJ0+gUw+ec9lrDTyWUdBLKbBuRV+fb8QzjOnrDfqN /nrKeFohikMzgKnjQaWN/049hSWQQ+BRZhcn6Zbu+zV4s1vwmw5QQA7SsJ+3qsrhO/tj jDXKr0I+o8bhTLtZ+4TIWgGwIobfnyC3Z3AG222PuvH0/euukKgrR4m+rs1yLVLLrNOV hqYQ== X-Gm-Message-State: AOAM5339gGrwLAox/kvF5rjuZvoHPHCcdGRlBC6za+SueY225w5YiT8M QI/ST+bPaWhdc3SmjG14pFN2gv4izq1lrA== X-Received: by 2002:a05:600c:22d6:: with SMTP id 22mr5266861wmg.17.1632840702114; Tue, 28 Sep 2021 07:51:42 -0700 (PDT) Received: from ?IPv6:2a01:e0a:410:bb00:e0fd:c68f:ea32:2084? ([2a01:e0a:410:bb00:e0fd:c68f:ea32:2084]) by smtp.gmail.com with ESMTPSA id j19sm19877425wra.92.2021.09.28.07.51.41 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 28 Sep 2021 07:51:41 -0700 (PDT) Reply-To: nicolas.dichtel@6wind.com Subject: Re: [PATCH net-next v5] net: openvswitch: IPv6: Add IPv6 extension header support To: Cpp Code Cc: netdev@vger.kernel.org, pshelar@ovn.org, "David S. Miller" , Jakub Kicinski , ovs dev , linux-kernel@vger.kernel.org References: <20210920182038.1510501-1-cpp.code.lv@gmail.com> <0d70b112-dc7a-7083-db8d-183782b8ef8f@6wind.com> From: Nicolas Dichtel Organization: 6WIND Message-ID: Date: Tue, 28 Sep 2021 16:51:40 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Le 27/09/2021 à 21:12, Cpp Code a écrit : > To use this code there is a part of code in the userspace. We want to > keep compatibility when we only update userspace part code or only > kernel part code. This means we should have same values for constants > and we can only add new ones at the end of list. All attributes after OVS_KEY_ATTR_CT_STATE (ie 7 attributes) were added before OVS_KEY_ATTR_TUNNEL_INFO. Why is it not possible anymore? Regards, Nicolas > > Best, > Tom > > On Wed, Sep 22, 2021 at 11:02 PM Nicolas Dichtel > wrote: >> >> Le 20/09/2021 à 20:20, Toms Atteka a écrit : >>> This change adds a new OpenFlow field OFPXMT_OFB_IPV6_EXTHDR and >>> packets can be filtered using ipv6_ext flag. >>> >>> Signed-off-by: Toms Atteka >>> --- >>> include/uapi/linux/openvswitch.h | 12 +++ >>> net/openvswitch/flow.c | 140 +++++++++++++++++++++++++++++++ >>> net/openvswitch/flow.h | 14 ++++ >>> net/openvswitch/flow_netlink.c | 24 +++++- >>> 4 files changed, 189 insertions(+), 1 deletion(-) >>> >>> diff --git a/include/uapi/linux/openvswitch.h b/include/uapi/linux/openvswitch.h >>> index a87b44cd5590..dc6eb5f6399f 100644 >>> --- a/include/uapi/linux/openvswitch.h >>> +++ b/include/uapi/linux/openvswitch.h >>> @@ -346,6 +346,13 @@ enum ovs_key_attr { >>> #ifdef __KERNEL__ >>> OVS_KEY_ATTR_TUNNEL_INFO, /* struct ip_tunnel_info */ >>> #endif >>> + >>> +#ifndef __KERNEL__ >>> + PADDING, /* Padding so kernel and non kernel field count would match */ >>> +#endif >>> + >>> + OVS_KEY_ATTR_IPV6_EXTHDRS, /* struct ovs_key_ipv6_exthdr */ >> Naive question, why not moving OVS_KEY_ATTR_IPV6_EXTHDRS above >> OVS_KEY_ATTR_TUNNEL_INFO? >> >> >> >> Regards, >> Nicolas