Received: by 2002:a05:6358:7058:b0:131:369:b2a3 with SMTP id 24csp2779723rwp; Fri, 14 Jul 2023 11:10:03 -0700 (PDT) X-Google-Smtp-Source: APBJJlGcgxuj0PQA4FxEWr4Lhbh2Tg9zmVhsDfujk691Hs8QKUClq83fRIEbnlKL9bVdnosMK9iA X-Received: by 2002:a17:902:e890:b0:1b7:f64b:378a with SMTP id w16-20020a170902e89000b001b7f64b378amr3836775plg.16.1689358203654; Fri, 14 Jul 2023 11:10:03 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1689358203; cv=none; d=google.com; s=arc-20160816; b=iyuolTeoN2vo7MF6RGe9RGoTU1e6f1t4lHlo+Ahcxso1YLanD0us7kxFPDtP/5064X lxgZZAHriyBBu1lhbf8wSm4C8gMXX8RcyamijcQ5NS8y82jsNMkd8mVBjgwTLnHHkeCi RKy5RJThnGMJF9JZOb61c2GX6k7IvSyYJe1nYSPTBkyk9pH4Ul4cEoG02iOBCu4hiEmB UVXIyJ0fB49B0Co33E5qbEX2svS8ltXgM4Uedl3LYKUNaro3hpXVn8OX2nggQSMjiHOn ntf/eCRV8A3qnJZOXSGJKn3f1Cvro5on77Z/bDQxu6TKZJE6wDW2LtADMiNzw0WMivNJ IK9A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :date:message-id:dkim-signature; bh=I7DDN8GbZMG3KlpuHTN+v0LnOTBPGrIWfYxu7LMvL/8=; fh=8+e0U3hBgeRtowNkiGLQQPEytK8VQyHQkyGLazmetpc=; b=HUGgPntOSDLlu/fUHyN6sHmLA8mMoT68vLy/IXf4bEP8eO3QnCnj/GBRp108sHhsSc +f/DO1ZRLgv7HYCHvxHqQTgjp7mnsC8V5mRDbE5032VWtzoKBWTEomx6knkQkPIXrkBJ Ip7jGDTFRq8F1Vmjuhr7R6nn//IVes/1jQKwVfVSHPUXO5pW0HyLM1iSuo0ysvx0UghV Q3Jc8ENjscX5H/daWBSU9so36Ick9RxhahFXaqvUM+XfuHqTHmtn/HfpBGCormdmkrs5 /pmttQ6EbrJV9X7Ib95Ix8LGxNOekC5uhxaX4xM0DmVDrZfLCfqRvz74F8qgT48dGbIw sM8A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@mojatatu-com.20221208.gappssmtp.com header.s=20221208 header.b=EsObdH5G; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id lb16-20020a170902fa5000b001b896086ca1si7263530plb.136.2023.07.14.11.09.51; Fri, 14 Jul 2023 11:10:03 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@mojatatu-com.20221208.gappssmtp.com header.s=20221208 header.b=EsObdH5G; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235997AbjGNRvC (ORCPT + 99 others); Fri, 14 Jul 2023 13:51:02 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35668 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235954AbjGNRuv (ORCPT ); Fri, 14 Jul 2023 13:50:51 -0400 Received: from mail-oo1-xc32.google.com (mail-oo1-xc32.google.com [IPv6:2607:f8b0:4864:20::c32]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D48333ABE for ; Fri, 14 Jul 2023 10:50:21 -0700 (PDT) Received: by mail-oo1-xc32.google.com with SMTP id 006d021491bc7-565f3881cbeso1524030eaf.2 for ; Fri, 14 Jul 2023 10:50:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=mojatatu-com.20221208.gappssmtp.com; s=20221208; t=1689356948; x=1691948948; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=I7DDN8GbZMG3KlpuHTN+v0LnOTBPGrIWfYxu7LMvL/8=; b=EsObdH5GmfJY05wMbzoDxMZl7qGlDG1hB3mWeD4J9KH+vhPUO8vzmrTibNYCg3HFsX N/yHTsKtySUpQBqoznTOulPdXkVf5BdhhRbu9EJ5Plf9zeBRSvrgCTYrz9g9r9YKgv7P /THveqSATuTYP4pJoUUX65Uan3QgCrP+vldmXjs13ym6CziZ2rBvZP2Xn0T2UKRIVHRM AochMDIvMr3iWvDt4ot+U2i+6lwRdJJyOJtOSxXxo0HTi0zV9wlssvgbjSZEyuoRkFiD d7e0tq9zTPgg6hRtHTm4SCr/NXDD25LFVWKsc5TWQsXVNx76aIbvTrm0/36SWRYONc5H 2iZA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1689356948; x=1691948948; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=I7DDN8GbZMG3KlpuHTN+v0LnOTBPGrIWfYxu7LMvL/8=; b=PB4QiH4tkOAqSiisKso2y3n1ZsHoRtXoFwgv+bURakGR10tfd4v+UcjsSTM+lS3Byf z2qYTxRdv1iCAEEK1fBBni/AgttBRY1XFR00ZFIT+iH9FKf5gvtT297i+0mPx6c3T9Ao 1m04w1GpBP2OJEIuhcsh9L6BMF4xbEV61Pu0mUZZ9Ad5f16amy1F8HCE0E0WAJ0G8zvH FLbjC4eEindNb+wNqasaIA/F1Q4vwsqKU7YnEGIm0tfjxULU5OJznV3ioBdzNkqEFTtg mYiVf2NjWb1gtvVNZBAh9v622h9qbKofI2VdQXVy9N99vvQV8nMtFGc4oUU60KKQAA4e rgxQ== X-Gm-Message-State: ABy/qLaOVfZa03qkp56j2gCFqFSDre2KmRMI9Q194BKzPFVnRxB3I6t6 IoHkGp28dRo5uf9jywHN3/08Yw== X-Received: by 2002:a4a:d298:0:b0:566:6928:2e3c with SMTP id h24-20020a4ad298000000b0056669282e3cmr5186911oos.9.1689356947824; Fri, 14 Jul 2023 10:49:07 -0700 (PDT) Received: from ?IPV6:2804:14d:5c5e:44fb:860d:7d54:c6cc:ceb3? ([2804:14d:5c5e:44fb:860d:7d54:c6cc:ceb3]) by smtp.gmail.com with ESMTPSA id 184-20020a4a1dc1000000b0055e3dd89c12sm3065807oog.1.2023.07.14.10.49.03 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 14 Jul 2023 10:49:07 -0700 (PDT) Message-ID: Date: Fri, 14 Jul 2023 14:49:01 -0300 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.11.0 Subject: Re: [PATCH net 1/3] selftests: tc: set timeout to 15 minutes Content-Language: en-US To: shaozhengchao , Matthieu Baerts , Jamal Hadi Salim , Cong Wang , Jiri Pirko , Shuah Khan , Kees Cook , "David S. Miller" , Paul Blakey , Marcelo Ricardo Leitner , mptcp@lists.linux.dev Cc: Shuah Khan , linux-kernel@vger.kernel.org, netdev@vger.kernel.org, linux-kselftest@vger.kernel.org, stable@vger.kernel.org References: <20230713-tc-selftests-lkft-v1-0-1eb4fd3a96e7@tessares.net> <20230713-tc-selftests-lkft-v1-1-1eb4fd3a96e7@tessares.net> From: Pedro Tammela In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,NICE_REPLY_A,RCVD_IN_DNSWL_BLOCKED,SPF_HELO_NONE,SPF_NONE, T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 13/07/2023 23:25, shaozhengchao wrote: > > > On 2023/7/14 5:16, Matthieu Baerts wrote: >> When looking for something else in LKFT reports [1], I noticed that the >> TC selftest ended with a timeout error: >> >>    not ok 1 selftests: tc-testing: tdc.sh # TIMEOUT 45 seconds >> >> The timeout had been introduced 3 years ago, see the Fixes commit below. >> >> This timeout is only in place when executing the selftests via the >> kselftests runner scripts. I guess this is not what most TC devs are >> using and nobody noticed the issue before. >> >> The new timeout is set to 15 minutes as suggested by Pedro [2]. It looks >> like it is plenty more time than what it takes in "normal" conditions. >> >> Fixes: 852c8cbf34d3 ("selftests/kselftest/runner.sh: Add 45 second >> timeout per test") >> Cc: stable@vger.kernel.org >> Link: >> https://qa-reports.linaro.org/lkft/linux-next-master/build/next-20230711/testrun/18267241/suite/kselftest-tc-testing/test/tc-testing_tdc_sh/log [1] >> Link: >> https://lore.kernel.org/netdev/0e061d4a-9a23-9f58-3b35-d8919de332d7@tessares.net/T/ [2] >> Suggested-by: Pedro Tammela >> Signed-off-by: Matthieu Baerts >> --- >>   tools/testing/selftests/tc-testing/settings | 1 + >>   1 file changed, 1 insertion(+) >> >> diff --git a/tools/testing/selftests/tc-testing/settings >> b/tools/testing/selftests/tc-testing/settings >> new file mode 100644 >> index 000000000000..e2206265f67c >> --- /dev/null >> +++ b/tools/testing/selftests/tc-testing/settings >> @@ -0,0 +1 @@ >> +timeout=900 >> > I remember last year when I tested all the tdc cases(qdisc + filter + > action + infra) in my vm machine, it took me nearly 20 minutes. > So I think it should be more than 1200 seconds if all cases need to be > tested. > > Maybe we should really optimize the parallel execution process of tdc. Let's try to spend some cycles improving the tdc code performance first. TDC boils down essentially to: - Setup namespace (if needed) - Setup network interfaces - Spawn a few processes - Match a regex - Bring down namespace Nothing above screams expensive, so I'm sure there are some low hanging fruits to improve the overall wall time even in debug kernels.