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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 422E6C43334 for ; Mon, 25 Jul 2022 16:57:54 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236583AbiGYQ5x (ORCPT ); Mon, 25 Jul 2022 12:57:53 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51470 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235549AbiGYQ5w (ORCPT ); Mon, 25 Jul 2022 12:57:52 -0400 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 6764CDED3 for ; Mon, 25 Jul 2022 09:57:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1658768270; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=zUTQcsRLeLjivibUTZmQzNSPdRIp0K6U7bQi13E1Qj8=; b=Fuv1MDgui55xaEk1+IKmHxvIVrs2WZw3fbxlI140G3FxByiFqbPcuy7Y+kxkgOb2L1nkHp 6jh6X5FP8cIHE74jcMHxfRQwxubEZNyUB/zyimbVkVdSbtikuK4kw5SqSfnCAUazSzKHq3 xTZcHzE8X5mei2ycCwpn67lYRvkuHOU= Received: from mail-wm1-f70.google.com (mail-wm1-f70.google.com [209.85.128.70]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-375-gR9Z1SODP6qGTHYe97YYpA-1; Mon, 25 Jul 2022 12:57:49 -0400 X-MC-Unique: gR9Z1SODP6qGTHYe97YYpA-1 Received: by mail-wm1-f70.google.com with SMTP id 189-20020a1c02c6000000b003a2d01897e4so6417760wmc.9 for ; Mon, 25 Jul 2022 09:57:48 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=zUTQcsRLeLjivibUTZmQzNSPdRIp0K6U7bQi13E1Qj8=; b=fns8G9zSyXjB5kyXMwumWTnJFQ1Yb05d6NE0pD5mHnznIEhH3aPfmkzFDvgf4JkC+i Nw3C4Vnc2+S60AuirkMKHoebw/CA/8TTZNyP7Tuzk6dPqtQV4HbktXVImMyVmC/4XnOT H1rKkeUkvdJzVkJDwXPZRN2UZk+jT4VzTH7tuWrSRWuS3N3nZuezxNbwQv36DwCfi+Pg F3knTxxyZ3FVoTHpxs+OSYfBNNz5c2kUrhBkVyJPINqUh52nTQsRhs4+ktGr3bC7FXbX x9gLceO8UaE/IAU8oGpXkCji8dW0tv4s8cCSYreF/oxfvvxxTi3CxAV2Cmt1Zc/1fwtP ABxA== X-Gm-Message-State: AJIora+k2CwfEgQg/rv5Cyni/BoMnmyJTBhG08ljeG/bEepHFNMCpj31 hNUlq0Y/AgNHUakYJaw6btnMEuVJqTZVlljZNFQniiy2SUg7IKx4r8Q3wLDn3cPsdsb5DQIWikg x4S7+8vHToqkeuJML X-Received: by 2002:a5d:52c4:0:b0:21e:428a:912b with SMTP id r4-20020a5d52c4000000b0021e428a912bmr8284279wrv.395.1658768268029; Mon, 25 Jul 2022 09:57:48 -0700 (PDT) X-Google-Smtp-Source: AGRyM1vRXdoRgIcwEQFMM1Bbckaj4HHI9azZqFwkj6SGqynNPbemdAV9RYTJlHMOrSfp468++8vnzA== X-Received: by 2002:a5d:52c4:0:b0:21e:428a:912b with SMTP id r4-20020a5d52c4000000b0021e428a912bmr8284265wrv.395.1658768267774; Mon, 25 Jul 2022 09:57:47 -0700 (PDT) Received: from pc-4.home (2a01cb058918ce00dd1a5a4f9908f2d5.ipv6.abo.wanadoo.fr. [2a01:cb05:8918:ce00:dd1a:5a4f:9908:f2d5]) by smtp.gmail.com with ESMTPSA id p12-20020a05600c358c00b0039c96b97359sm16080728wmq.37.2022.07.25.09.57.46 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 25 Jul 2022 09:57:47 -0700 (PDT) Date: Mon, 25 Jul 2022 18:57:45 +0200 From: Guillaume Nault To: Matthias May Cc: netdev@vger.kernel.org, davem@davemloft.net, yoshfuji@linux-ipv6.org, dsahern@kernel.org, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, nicolas.dichtel@6wind.com, eyal.birger@gmail.com, linux-kernel@vger.kernel.org Subject: Re: [PATCH 0/2 net-next] geneve: fix TOS inheriting Message-ID: <20220725165745.GC18808@pc-4.home> References: <20220724003741.57816-1-matthias.may@westermo.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220724003741.57816-1-matthias.may@westermo.com> Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On Sun, Jul 24, 2022 at 02:37:39AM +0200, Matthias May wrote: > Is there any reason why the flowlabel should be restricted to these > 3 bits? It's a bug, likely a copy/paste mistake. > The patch simply removes the usage of this macro, but i don't > know if there was a specific intention behind that. RT_TOS() is an old macro used to interprete IPv4 TOS as described in the obsolete RFC 1349. It's conceptually wrong to use it even in IPv4 code, although, given the current state of the code, most of the existing calls have no consequence. But using RT_TOS() in IPv6 code is always a bug: IPv6 never had a "TOS" field to be interpreted the RFC 1349 way. There's no historical compatibility to worry about.