From patchwork Tue Sep 20 17:04:01 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Jiale, SongX" X-Patchwork-Id: 116475 Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id F3C67A00C3; Tue, 20 Sep 2022 10:51:13 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D1B504069B; Tue, 20 Sep 2022 10:51:13 +0200 (CEST) Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by mails.dpdk.org (Postfix) with ESMTP id 393954069B for ; Tue, 20 Sep 2022 10:51:12 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1663663872; x=1695199872; h=from:to:cc:subject:date:message-id; bh=+Gf6m2nkq8ldW33udsGCdUFTcYkGNGJRIX9KAvLXj6M=; b=Hkj6AW24gcivM6wUvNmgKsNO5zFJZIhArCIc6lyNqbf+Ce8UyRRsqIUJ 2hgQj3snW0mbtp2PrkZXefL6hQNwQ0MBY3D2qZe8enOob6ashDO2yJdDm KKsSgsnFefnsneRdk+bSleGkwmP3CEfikfmE0ehNymHoVuvrH6X7y+Pof lgawItTWV1ER7K2uLGZ2fkoBMfjoc277yaUcpjeUKtDIxTdw3hgUnRubJ TpegMiHSk7nWOcV9g2DxyEphGQ0caDHY/SvSaaJhbQ/m1uje/u9LMZaBO 9dtl8K9EHw90xibJYP3VQXvkUmoTCkiNVvFXiK4+X3RCVQuNWFhfPhGLv g==; X-IronPort-AV: E=McAfee;i="6500,9779,10475"; a="286687654" X-IronPort-AV: E=Sophos;i="5.93,330,1654585200"; d="scan'208";a="286687654" Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Sep 2022 01:51:11 -0700 X-IronPort-AV: E=Sophos;i="5.93,330,1654585200"; d="scan'208";a="618834758" Received: from unknown (HELO cvl_tetser_105.icx.intel.com) ([10.239.252.94]) by orsmga002-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Sep 2022 01:51:09 -0700 From: Song Jiale To: dts@dpdk.org Cc: Song Jiale Subject: [dts] [PATCH V2 1/2] tests/ice_dcf_switch_filter_gtpu: optimization script Date: Tue, 20 Sep 2022 17:04:01 +0000 Message-Id: <20220920170402.14146-1-songx.jiale@intel.com> X-Mailer: git-send-email 2.17.1 X-BeenThere: dts@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: test suite reviews and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dts-bounces@dpdk.org 1.add timeout 2.the "testpmd_status" is the wrong status when launch app timeout, it will affect other cases test.therefore, add "kill_all" to ensure that all dpdk processes are closed. Signed-off-by: Song Jiale --- tests/TestSuite_ice_dcf_switch_filter_gtpu.py | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) v2: -redesign optimization scheme. diff --git a/tests/TestSuite_ice_dcf_switch_filter_gtpu.py b/tests/TestSuite_ice_dcf_switch_filter_gtpu.py index 0f48d78d..8c63a976 100644 --- a/tests/TestSuite_ice_dcf_switch_filter_gtpu.py +++ b/tests/TestSuite_ice_dcf_switch_filter_gtpu.py @@ -3592,8 +3592,7 @@ class ICEDCFSwitchFilterGTPUTest(TestCase): """ dcf switch need reload driver to ensure create rule sucessful """ - self.dut.send_expect("rmmod ice", "# ", 15) - self.dut.send_expect("modprobe ice", "# ", 15) + self.dut.send_expect("rmmod ice && modprobe ice", "# ", 60) def create_testpmd_command(self): """ @@ -3617,7 +3616,7 @@ class ICEDCFSwitchFilterGTPUTest(TestCase): launch testpmd with the command """ command = self.create_testpmd_command() - self.dut.send_expect(command, "testpmd> ", 15) + self.dut.send_expect(command, "testpmd> ", 30) self.testpmd_status = "running" self.dut.send_expect("set portlist 1", "testpmd> ", 15) self.dut.send_expect("set fwd rxonly", "testpmd> ", 15) @@ -3822,12 +3821,10 @@ class ICEDCFSwitchFilterGTPUTest(TestCase): self.dut.send_expect("flow flush 0", "testpmd> ", 15) self.dut.send_expect("clear port stats all", "testpmd> ", 15) self.dut.send_expect("quit", "#", 15) - # kill all DPDK application - self.dut.kill_all() # destroy vfs for port_id in self.dut_ports: self.dut.destroy_sriov_vfs_by_port(port_id) - self.testpmd_status = "close" + self.testpmd_status = "close" if getattr(self, "session_secondary", None): self.dut.close_session(self.session_secondary) @@ -3971,6 +3968,8 @@ class ICEDCFSwitchFilterGTPUTest(TestCase): Run after each test case. """ self.destroy_testpmd_and_vf() + # kill all DPDK application + self.dut.kill_all() def tear_down_all(self): """