From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 36FD6A0573; Wed, 4 Mar 2020 18:31:24 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 4B4FB2C02; Wed, 4 Mar 2020 18:31:23 +0100 (CET) Received: from us-smtp-delivery-1.mimecast.com (us-smtp-1.mimecast.com [207.211.31.81]) by dpdk.org (Postfix) with ESMTP id 0382723D for ; Wed, 4 Mar 2020 18:31:21 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1583343081; 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: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=SkkVwQznF3CHtI7a8H7Rx6QZudk90CnjIkG3zRjkB24=; b=JzbZMaTG3utWe+bQJofNtTsoRyradatgPa80RpTOhfBAsRZJHz5UksJ7s4s4FETJD1NsOT cyjbvix+CJzg5ZrTnEct7d/YYBvC2GbJK/GaMeKfShvCmK4PHGThgYfYKPY5IPWj2yij2N fPrBAnsf5JCd4WV81C7Gqim1ItA11Xg= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-281-g1tcI2gmPqiidbbDRs93UQ-1; Wed, 04 Mar 2020 12:31:17 -0500 X-MC-Unique: g1tcI2gmPqiidbbDRs93UQ-1 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 41A4B8017CC; Wed, 4 Mar 2020 17:31:15 +0000 (UTC) Received: from dhcp-25.97.bos.redhat.com (unknown [10.18.25.177]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 247505E7A0; Wed, 4 Mar 2020 17:31:09 +0000 (UTC) From: Aaron Conole To: Ruifeng Wang Cc: maicolgabriel@hotmail.com, bruce.richardson@intel.com, dev@dpdk.org, david.marchand@redhat.com, gavin.hu@arm.com, honnappa.nagarahalli@arm.com, juraj.linkes@pantheon.tech, nd@arm.com References: <20200225073236.135581-1-ruifeng.wang@arm.com> <20200228041904.195597-1-ruifeng.wang@arm.com> <20200228041904.195597-3-ruifeng.wang@arm.com> Date: Wed, 04 Mar 2020 12:31:08 -0500 In-Reply-To: <20200228041904.195597-3-ruifeng.wang@arm.com> (Ruifeng Wang's message of "Fri, 28 Feb 2020 12:19:04 +0800") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux) MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain Content-Transfer-Encoding: quoted-printable Subject: Re: [dpdk-dev] [PATCH v2 2/2] ci: add test suite run without hugepage X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Ruifeng Wang writes: > This test suite is derived from fast-tests suite. Cases in this > suite are run with '--no-huge' flag. > > The suite aims to cover as many as possible test cases out of the > fast-tests suites in the environments without huge pages support, > like containers. > > Signed-off-by: Ruifeng Wang > Reviewed-by: Gavin Hu > --- I like this much more. Few comments. > .travis.yml | 10 +++++-- > app/test/meson.build | 71 > ++++++++++++++++++++++++++++++++++++++++++++ You should update doc/guides/prog_guide/meson_ut.rst to include some detail about the new tests suite. > 2 files changed, 79 insertions(+), 2 deletions(-) > > diff --git a/.travis.yml b/.travis.yml > index b64a81bd0..eed1d96db 100644 > --- a/.travis.yml > +++ b/.travis.yml > @@ -40,7 +40,7 @@ jobs: > - env: DEF_LIB=3D"static" > arch: amd64 > compiler: gcc > - - env: DEF_LIB=3D"shared" RUN_TESTS=3D1 > + - env: DEF_LIB=3D"shared" RUN_TESTS=3D1 TEST_SUITES=3D"fast-tests nohu= ge-tests" > arch: amd64 > compiler: gcc > - env: DEF_LIB=3D"shared" BUILD_DOCS=3D1 > @@ -63,7 +63,7 @@ jobs: > - env: DEF_LIB=3D"static" > arch: amd64 > compiler: clang > - - env: DEF_LIB=3D"shared" RUN_TESTS=3D1 > + - env: DEF_LIB=3D"shared" RUN_TESTS=3D1 TEST_SUITES=3D"fast-tests nohu= ge-tests" > arch: amd64 > compiler: clang > - env: DEF_LIB=3D"shared" BUILD_DOCS=3D1 > @@ -101,6 +101,9 @@ jobs: > - env: DEF_LIB=3D"static" > arch: arm64 > compiler: gcc > + - env: DEF_LIB=3D"shared" RUN_TESTS=3D1 TEST_SUITES=3D"nohuge-tests" > + arch: arm64 > + compiler: gcc > - env: DEF_LIB=3D"shared" BUILD_DOCS=3D1 > arch: arm64 > compiler: gcc > @@ -124,3 +127,6 @@ jobs: > - env: DEF_LIB=3D"shared" > arch: arm64 > compiler: clang > + - env: DEF_LIB=3D"shared" RUN_TESTS=3D1 TEST_SUITES=3D"nohuge-tests" > + arch: arm64 > + compiler: clang > diff --git a/app/test/meson.build b/app/test/meson.build > index 0a2ce710f..162a1a76f 100644 > --- a/app/test/meson.build > +++ b/app/test/meson.build > @@ -237,6 +237,60 @@ fast_test_names =3D [ > 'thash_autotest', > ] Shouldn't we also trim the list of fast-tests? Otherwise, these tests will run twice. ex: https://travis-ci.com/ovsrobot/dpdk/jobs/292037684 > +nohuge_test_names =3D [ > + 'byteorder_autotest', > + 'cmdline_autotest', > + 'common_autotest', > + 'cpuflags_autotest', > + 'cycles_autotest', > + 'debug_autotest', > + 'eal_flags_n_opt_autotest', > + 'eal_flags_no_huge_autotest', > + 'eal_flags_vdev_opt_autotest', > + 'eal_fs_autotest', > + 'errno_autotest', > + 'event_ring_autotest', > + 'fib_autotest', > + 'fib6_autotest', > + 'interrupt_autotest', > + 'logs_autotest', > + 'lpm_autotest', > + 'lpm6_autotest', > + 'memcpy_autotest', > + 'meter_autotest', > + 'per_lcore_autotest', > + 'prefetch_autotest', > + 'rcu_qsbr_autotest', > + 'red_autotest', > + 'rib_autotest', > + 'rib6_autotest', > + 'ring_autotest', > + 'rwlock_rda_autotest', > + 'rwlock_rds_wrm_autotest', > + 'rwlock_rde_wro_autotest', > + 'sched_autotest', > + 'spinlock_autotest', > + 'string_autotest', > + 'tailq_autotest', > + 'user_delay_us', > + 'version_autotest', > + 'crc_autotest', > + 'delay_us_sleep_autotest', > + 'eventdev_common_autotest', > + 'fbarray_autotest', > + 'ipsec_autotest', > + 'kni_autotest', > + 'kvargs_autotest', > + 'member_autotest', > + 'metrics_autotest', > + 'power_cpufreq_autotest', > + 'power_autotest', > + 'power_kvm_vm_autotest', > + 'reorder_autotest', > + 'service_autotest', > + 'thash_autotest', > +] > + > perf_test_names =3D [ > 'ring_perf_autotest', > 'mempool_perf_autotest', > @@ -341,6 +395,10 @@ if dpdk_conf.has('RTE_LIBRTE_RING_PMD') > =09fast_test_names +=3D 'latencystats_autotest' > =09driver_test_names +=3D 'link_bonding_mode4_autotest' > =09fast_test_names +=3D 'pdump_autotest' > +=09nohuge_test_names +=3D 'ring_pmd_autotest' > +=09nohuge_test_names +=3D 'bitratestats_autotest' > +=09nohuge_test_names +=3D 'latencystats_autotest' > +=09nohuge_test_names +=3D 'pdump_autotest' > endif > =20 > if dpdk_conf.has('RTE_LIBRTE_POWER') > @@ -430,6 +488,19 @@ foreach arg : fast_test_names > =09endif > endforeach > =20 > +foreach arg : nohuge_test_names > +=09if host_machine.system() =3D=3D 'linux' > +=09=09test(arg, dpdk_test, > +=09=09=09 env : ['DPDK_TEST=3D' + arg], > +=09=09=09 args : test_args + > +=09=09=09=09 ['--no-huge'] + ['-m 1024'] + > +=09=09=09=09 ['--file-prefix=3D@0@'.format(arg)], > +=09=09timeout : timeout_seconds_fast, > +=09=09is_parallel : false, > +=09=09suite : 'nohuge-tests') > +=09endif > +endforeach > + > foreach arg : perf_test_names > =09test(arg, dpdk_test, > =09env : ['DPDK_TEST=3D' + arg],