From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id AAAD12BAE; Fri, 13 Jul 2018 18:41:33 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga104.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 13 Jul 2018 09:41:31 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.51,348,1526367600"; d="scan'208";a="72127659" Received: from aburakov-mobl.ger.corp.intel.com (HELO [10.252.0.191]) ([10.252.0.191]) by fmsmga001.fm.intel.com with ESMTP; 13 Jul 2018 09:41:09 -0700 To: Reshma Pattan , thomas@monjalon.net, dev@dpdk.org Cc: jananeex.m.parthasarathy@intel.com, stable@dpdk.org References: <1531498808-21940-9-git-send-email-reshma.pattan@intel.com> From: "Burakov, Anatoly" Message-ID: <0bdc3a80-4c8c-ea03-c8b3-04ff15a285be@intel.com> Date: Fri, 13 Jul 2018 17:41:08 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: <1531498808-21940-9-git-send-email-reshma.pattan@intel.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH v2 08/10] autotest: add new test cases to autotest 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: , X-List-Received-Date: Fri, 13 Jul 2018 16:41:34 -0000 On 13-Jul-18 5:20 PM, Reshma Pattan wrote: > Autotest is enhanced with additional test cases > being added to autotest_data.py You're also removing PCI autotest - commit message needs to call this out as well. > > Cc: stable@dpdk.org > > Signed-off-by: Reshma Pattan > Signed-off-by: Jananee Parthasarathy > --- > test/test/autotest_data.py | 370 ++++++++++++++++++++++++++++++++++++++++++++- > 1 file changed, 362 insertions(+), 8 deletions(-) > > diff --git a/test/test/autotest_data.py b/test/test/autotest_data.py > + { > + "Name": "Dump_ring", > + "Command": "dump_ring", > + "Func": default_autotest, > + "Report": None, > + }, > + { > + "Name": "Quit", > + "Command": "quit", > + "Func": default_autotest, > + "Report": None, > + }, Why is "quit" an autotest? If you want to test "quit" functionality, i would suggest putting it at the end of non-parallel tests, to avoid situation where test prematurely stops (for example, this will happen on FreeBSD, where there's only one thread executing these tests). But really, having it in this list is IMO dubious. > + { > + "Name": "Set rxtx mode", > + "Command": "set_rxtx_mode", > + "Func": default_autotest, > + "Report": None, > + }, > + { > + "Name": "Set rxtx anchor", > + "Command": "set_rxtx_anchor", > + "Func": default_autotest, > + "Report": None, > + }, > + { > + "Name": "Set rxtx sc", > + "Command": "set_rxtx_sc", > + "Func": default_autotest, > + "Report": None, > + }, > + { > + "Name": "Event eth rx adapter autotest", > + "Command": "event_eth_rx_adapter_autotest", > + "Func": default_autotest, > + "Report": None, > + }, > + { > + "Name": "Rawdev autotest", > + "Command": "rawdev_autotest", > + "Func": default_autotest, > + "Report": None, > + }, > + { > + "Name": "Kvargs autotest", > + "Command": "kvargs_autotest", > + "Func": default_autotest, > + "Report": None, > + }, > + { > + "Name": "Dump physmem", > + "Command": "dump_physmem", > + "Func": default_autotest, > + "Report": None, > + }, > + { > + "Name": "Dump memzone", > + "Command": "dump_memzone", > + "Func": default_autotest, > + "Report": None, > + }, Here and in other "dump" autotests - do these dump tests work as autotests? Default autotest function expects a "Test OK" at the end of test, and i don't think dump autotests return that, so this will (should?) cause a very long timeout and a test failure. These should have "dump_autotest" as "Func" value - that function correctly parses output of dump autotests (or, to be more accurate, it doesn't - as long as test doesn't crash, it's considered to be successful :) ). Also, for readability, we should move these dump tests together and have all dump tests one after the other. -- Thanks, Anatoly