From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Viktorin Subject: [PATCH v5 08/10] app/test: convert current pci_test into a single test case Date: Mon, 13 Jun 2016 17:07:43 +0200 Message-ID: <1465830465-30058-9-git-send-email-viktorin@rehivetech.com> References: <1465830465-30058-1-git-send-email-viktorin@rehivetech.com> Cc: Jan Viktorin , Thomas Monjalon , David Marchand , Bruce Richardson To: dev@dpdk.org Return-path: Received: from wes1-so1.wedos.net (wes1-so1.wedos.net [46.28.106.15]) by dpdk.org (Postfix) with ESMTP id 6AC54558D for ; Mon, 13 Jun 2016 17:12:56 +0200 (CEST) In-Reply-To: <1465830465-30058-1-git-send-email-viktorin@rehivetech.com> In-Reply-To: <1465805550-30640-1-git-send-email-viktorin@rehivetech.com> References: <1465805550-30640-1-git-send-email-viktorin@rehivetech.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" The current test_pci is just a single test case that tests the blacklisting of devices. Rename it to test_pci_blacklist and call it from the test_pci. The setup and cleanup are moved out of the test_pci_blacklist entirely to cover all other tests. Signed-off-by: Jan Viktorin --- v4 * the definitions of setup and cleanup functions are NOT moved (T. Monjalon) --- app/test/test_pci.c | 19 ++++++++++++++----- 1 file changed, 14 insertions(+), 5 deletions(-) diff --git a/app/test/test_pci.c b/app/test/test_pci.c index 50078a0..28d710b 100644 --- a/app/test/test_pci.c +++ b/app/test/test_pci.c @@ -178,17 +178,14 @@ test_pci_cleanup(void) return 0; } -int -test_pci(void) +static int +test_pci_blacklist(void) { struct rte_devargs_list save_devargs_list; printf("Dump all devices\n"); rte_eal_pci_dump(stdout); - if (test_pci_setup()) - return -1; - rte_eal_pci_register(&my_driver); rte_eal_pci_register(&my_driver2); @@ -224,6 +221,18 @@ test_pci(void) rte_eal_pci_unregister(&my_driver); rte_eal_pci_unregister(&my_driver2); + return 0; +} + +int +test_pci(void) +{ + if (test_pci_setup()) + return -1; + + if (test_pci_blacklist()) + return -1; + if (test_pci_cleanup()) return -1; -- 2.8.0