From patchwork Wed Feb 2 15:24:21 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Guillaume Nault X-Patchwork-Id: 539970 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 A634DC433FE for ; Wed, 2 Feb 2022 15:24:28 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S241662AbiBBPY1 (ORCPT ); Wed, 2 Feb 2022 10:24:27 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]:34431 "EHLO us-smtp-delivery-124.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230484AbiBBPY1 (ORCPT ); Wed, 2 Feb 2022 10:24:27 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1643815466; 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; bh=wbfCqbIp07biNHmVcfksEUeSGRpB450MoXKgsGlsSvU=; b=HMemB6W1MJl6f/EhBa1diVX0153MfAtp76qCqgrONHmB7YxuDWviql5atT5wf7tQUbNAug hL/PwPRm59oTyVT9nYyrauHIOA23qzamnckV8SheO1F/2CziX7PO3O9VBfEGp/f2dDpQlF rvh3xt0uOb9uKpiNi4kfHekyXZ9R5lc= Received: from mail-wr1-f72.google.com (mail-wr1-f72.google.com [209.85.221.72]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-353-LAm3A_J_O_mHOOUvRXhBPg-1; Wed, 02 Feb 2022 10:24:26 -0500 X-MC-Unique: LAm3A_J_O_mHOOUvRXhBPg-1 Received: by mail-wr1-f72.google.com with SMTP id x4-20020adfbb44000000b001d83e815683so6908704wrg.8 for ; Wed, 02 Feb 2022 07:24:25 -0800 (PST) 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:mime-version :content-disposition; bh=wbfCqbIp07biNHmVcfksEUeSGRpB450MoXKgsGlsSvU=; b=o8jUmZTpiEJccXzy3caphtBke1SomP8WaxQTQwBbwCm8cqtsbHX2DLarltOGAVorVw Dhz1Ixg39H8wcPa9IOwL5xxmX4sOPJCKSDh8eppCURdgt0O7+rFjDh757RRbil5YP4e7 VwEL+4FeyBhrBjE9Q6oz/WxJljo6BPT+mCpSD6NH9HJgTtQbix46Re6HtCOJh5KAvn1N hpyZaryERA+hneCJYzvEUWtVKNcV3dGt4297eRAgJHfX1+qWRatxhU83qXS+t29QSSd8 m0H88x7mmNpIYxTAzF/RLigi51ufdT4/+Q9vRz5na3yUy83mrv8jLXd02/YFErBty0IT rSbw== X-Gm-Message-State: AOAM5311/oEkAihzuSt4zlbKpP/OqE/FlCzNEHo0Ym1v3S8so0Scb5hQ R2U69aLC599bCe17Ci4/i7xkYDhMPc04YPFwdSs6YfZav9UJV8aQTFiY4pQZYQXTyVojMz9acel SJl/SPK/L71hLNg/OC7Wf/g2jNIT+ X-Received: by 2002:adf:d1e9:: with SMTP id g9mr25495217wrd.166.1643815464416; Wed, 02 Feb 2022 07:24:24 -0800 (PST) X-Google-Smtp-Source: ABdhPJxlOhXtDJ+HPBsWYCYPkfTLAvav6APznjRzlOdHckDMQxh4xTH4iZD4qtIRH5SWVifZEpvEqg== X-Received: by 2002:adf:d1e9:: with SMTP id g9mr25495200wrd.166.1643815464205; Wed, 02 Feb 2022 07:24:24 -0800 (PST) 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 ay3sm4988031wmb.44.2022.02.02.07.24.23 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 02 Feb 2022 07:24:23 -0800 (PST) Date: Wed, 2 Feb 2022 16:24:21 +0100 From: Guillaume Nault To: David Miller , Jakub Kicinski Cc: netdev@vger.kernel.org, Shuah Khan , linux-kselftest@vger.kernel.org, Florian Westphal Subject: [PATCH net-next] selftests: rtnetlink: Use more sensible tos values Message-ID: MIME-Version: 1.0 Content-Disposition: inline Precedence: bulk List-ID: X-Mailing-List: linux-kselftest@vger.kernel.org Using tos 0x1 with 'ip route get ...' doesn't test much of the tos option handling: 0x1 just sets an ECN bit, which is cleared by inet_rtm_getroute() before doing the fib lookup. Let's use 0x10 instead, which is actually taken into account in the route lookup (and is less surprising for the reader). For consistency, use 0x10 for the IPv6 route lookup too (IPv6 currently doesn't clear ECN bits, but might do so in the future). Signed-off-by: Guillaume Nault --- No Fixes tag, since this is for net-next and the original test wasn't actually broken in the first place. tools/testing/selftests/net/rtnetlink.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/testing/selftests/net/rtnetlink.sh b/tools/testing/selftests/net/rtnetlink.sh index c9ce3dfa42ee..0900c5438fbb 100755 --- a/tools/testing/selftests/net/rtnetlink.sh +++ b/tools/testing/selftests/net/rtnetlink.sh @@ -216,9 +216,9 @@ kci_test_route_get() check_err $? ip route get fe80::1 dev "$devdummy" > /dev/null check_err $? - ip route get 127.0.0.1 from 127.0.0.1 oif lo tos 0x1 mark 0x1 > /dev/null + ip route get 127.0.0.1 from 127.0.0.1 oif lo tos 0x10 mark 0x1 > /dev/null check_err $? - ip route get ::1 from ::1 iif lo oif lo tos 0x1 mark 0x1 > /dev/null + ip route get ::1 from ::1 iif lo oif lo tos 0x10 mark 0x1 > /dev/null check_err $? ip addr add dev "$devdummy" 10.23.7.11/24 check_err $?