From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pf0-f193.google.com ([209.85.192.193]:35604 "EHLO mail-pf0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932090AbeB1QjW (ORCPT ); Wed, 28 Feb 2018 11:39:22 -0500 Received: by mail-pf0-f193.google.com with SMTP id y186so1214524pfb.2 for ; Wed, 28 Feb 2018 08:39:22 -0800 (PST) Message-ID: <1519835960.30253.3.camel@gmail.com> Subject: [PATCH net] test_bpf: reduce MAX_TESTRUNS From: Eric Dumazet To: Daniel Borkmann , Alexei Starovoitov Cc: netdev Date: Wed, 28 Feb 2018 08:39:20 -0800 In-Reply-To: <1519690325.3258.12.camel@gmail.com> References: <1519671166.3258.10.camel@gmail.com> <1519690325.3258.12.camel@gmail.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: netdev-owner@vger.kernel.org List-ID: From: Eric Dumazet For tests that are using the maximal number of BPF instruction, each run takes 20 usec. Looping 10,000 times on them totals 200 ms, which is bad when the loop is not preemptible. test_bpf: #264 BPF_MAXINSNS: Call heavy transformations jited:1 19248 18548 PASS test_bpf: #269 BPF_MAXINSNS: ld_abs+get_processor_id jited:1 20896 PASS Lets divide by ten the number of iterations, so that max latency is 20ms. We could use need_resched() to break the loop earlier if we believe 20 ms is too much. Signed-off-by: Eric Dumazet ---  lib/test_bpf.c |    2 +-  1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/test_bpf.c b/lib/test_bpf.c index b4e22345963f339ffe05c974bc111ae7da9dc58f..d8dc618b223bb5168b0c7e9d9f4316eef3fbfa34 100644 --- a/lib/test_bpf.c +++ b/lib/test_bpf.c @@ -27,7 +27,7 @@ /* General test specific settings */ #define MAX_SUBTESTS 3 -#define MAX_TESTRUNS 10000 +#define MAX_TESTRUNS 1000 #define MAX_DATA 128 #define MAX_INSNS 512 #define MAX_K 0xffffFFFF