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 C5987C433F5 for ; Wed, 8 Dec 2021 07:12:21 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S244461AbhLHHPv (ORCPT ); Wed, 8 Dec 2021 02:15:51 -0500 Received: from smtp-relay-internal-1.canonical.com ([185.125.188.123]:42380 "EHLO smtp-relay-internal-1.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S240004AbhLHHPv (ORCPT ); Wed, 8 Dec 2021 02:15:51 -0500 Received: from mail-pg1-f197.google.com (mail-pg1-f197.google.com [209.85.215.197]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by smtp-relay-internal-1.canonical.com (Postfix) with ESMTPS id 080553F1BA for ; Wed, 8 Dec 2021 07:12:18 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=canonical.com; s=20210705; t=1638947538; bh=HnHitcAxy5IK/OBfpwXFwbcANQpnXZnDkjq+md2LzKU=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version; b=f4P02E83eUUCRKAdM4e689RYuwEhZr6ixCcUv5HbrpqgZKx6n27Eta9GSJEkfdtuR 53/lso+bOkynV20+u7pC7WcmbSFIxl/w6jRlpVJaAXAFuWM+2AJ8K8cHCEBUUE7aS5 b4pONqyoJpJT1Crjzq/7IaSwJ8jx5FhSjUnCtlShNeipnBYK3t8W4nAZmX20H4FJao kia1Ev9dNZWqfnAyg3k7GR9ZnUN0JxmnkvMhXAsYbg9/domooFVDqY1xb8T9jzU1h6 cualjG2hLnmBBLgohMsqIS/5/HrcD3dx2ZjNvlhfELcGFhn8CGR05zfzX5zdty069+ tctAjlMYGMXlw== Received: by mail-pg1-f197.google.com with SMTP id i12-20020a63584c000000b00330ec6e2c37so815280pgm.7 for ; Tue, 07 Dec 2021 23:12:17 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=HnHitcAxy5IK/OBfpwXFwbcANQpnXZnDkjq+md2LzKU=; b=cOZBOetdYy9bAwrYLj2HPH9xkUSNK/BcjmIYnTpQsKCiJX7/LudM8nVkmhVpq5M0JN +po7XLm0p18awntHL/iPZrPBDRUsh20vyQEMXZ+x7rIqTQ8vgoH2uPEswqsCazttsFYi 1juqlJfDANpPlrJJEYoOVxHEk57D1sZTop12DlT1OSUO9PzHlrcecJg2huyIZtwbQowo AMURCfW5fhHvI708VTyN3KvRN3i7EKFl3Od1uniC9d27xfNfcpwRE8tAZJ+Dr5wXDzIS 1dhZp8keio+XVSlDqex3KcPNq0cu7AF+ImszdKHQrYMK3r3SYSi2QaRsAz2Z8I+6lXjI RHmg== X-Gm-Message-State: AOAM533DO9lLe5MW6mjkH4XXpuo21bzE6uXkQp9Q9VKSIp5fHC8srD/s P4QUwudRkAuHEHVtNlHbZgOs5gNeFw0hzZ5oGJfJ9/NhKb0jMQv5cJ3sDNyrppe8K8lFjlGr78V 0xPjcg+vY9pWjBrwy4kSiTctBMnCLMQSwlU62Aagw X-Received: by 2002:a17:90b:1d0b:: with SMTP id on11mr4926237pjb.163.1638947536664; Tue, 07 Dec 2021 23:12:16 -0800 (PST) X-Google-Smtp-Source: ABdhPJwl/Wa5GJglcK7GX5ZcWjy7TK24G/ysS+nwdSmTg8a2IuruvhOXnZZNFhsAR+v08zrhmczRfg== X-Received: by 2002:a17:90b:1d0b:: with SMTP id on11mr4926217pjb.163.1638947536421; Tue, 07 Dec 2021 23:12:16 -0800 (PST) Received: from localhost.localdomain (223-140-212-111.emome-ip.hinet.net. [223.140.212.111]) by smtp.gmail.com with ESMTPSA id y190sm1946116pfb.203.2021.12.07.23.12.13 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 07 Dec 2021 23:12:15 -0800 (PST) From: Po-Hsu Lin To: po-hsu.lin@canonical.com, netdev@vger.kernel.org, linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org Cc: davem@davemloft.net, kuba@kernel.org, skhan@linuxfoundation.org Subject: [PATCH] selftests: icmp_redirect: pass xfail=0 to log_test() for non-xfail cases Date: Wed, 8 Dec 2021 15:11:51 +0800 Message-Id: <20211208071151.63971-1-po-hsu.lin@canonical.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org If any sub-test in this icmp_redirect.sh is failing but not expected to fail. The script will complain: ./icmp_redirect.sh: line 72: [: 1: unary operator expected This is because when the sub-test is not expected to fail, we won't pass any value for the xfail local variable in log_test() and thus it's empty. Fix this by passing 0 as the 4th variable to log_test() for non-xfail cases. Signed-off-by: Po-Hsu Lin --- tools/testing/selftests/net/icmp_redirect.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/testing/selftests/net/icmp_redirect.sh b/tools/testing/selftests/net/icmp_redirect.sh index ecbf57f..7b9d6e3 100755 --- a/tools/testing/selftests/net/icmp_redirect.sh +++ b/tools/testing/selftests/net/icmp_redirect.sh @@ -311,7 +311,7 @@ check_exception() ip -netns h1 ro get ${H1_VRF_ARG} ${H2_N2_IP} | \ grep -E -v 'mtu|redirected' | grep -q "cache" fi - log_test $? 0 "IPv4: ${desc}" + log_test $? 0 "IPv4: ${desc}" 0 # No PMTU info for test "redirect" and "mtu exception plus redirect" if [ "$with_redirect" = "yes" ] && [ "$desc" != "redirect exception plus mtu" ]; then -- 2.7.4