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 93BADA0613 for ; Mon, 26 Aug 2019 07:29:44 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 7A4F61BF76; Mon, 26 Aug 2019 07:29:44 +0200 (CEST) Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id 545991BF49 for ; Mon, 26 Aug 2019 07:29:41 +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 orsmga101.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 25 Aug 2019 22:29:39 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,431,1559545200"; d="scan'208";a="196969679" Received: from dpdk-moyufen01.sh.intel.com ([10.67.111.77]) by fmsmga001.fm.intel.com with ESMTP; 25 Aug 2019 22:29:38 -0700 From: yufengmx To: changqingx.wu@intel.com, dts@dpdk.org Cc: yufengmx Date: Mon, 26 Aug 2019 13:30:47 +0800 Message-Id: <1566797447-122407-2-git-send-email-yufengx.mo@intel.com> X-Mailer: git-send-email 1.9.3 In-Reply-To: <1566797447-122407-1-git-send-email-yufengx.mo@intel.com> References: <1566797447-122407-1-git-send-email-yufengx.mo@intel.com> Subject: [dts] [PATCH V1]tests/pmd_stacked_bonded: set slave ports in right status X-BeenThere: dts@dpdk.org X-Mailman-Version: 2.1.15 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 Sender: "dts" port start all action implies the race condition. The right way is to start top level bond port only, and let it propagate the start action to slave bonded ports and its the real nics. Signed-off-by: yufengmx --- tests/TestSuite_pmd_stacked_bonded.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tests/TestSuite_pmd_stacked_bonded.py b/tests/TestSuite_pmd_stacked_bonded.py index e1ce01a..de126b9 100644 --- a/tests/TestSuite_pmd_stacked_bonded.py +++ b/tests/TestSuite_pmd_stacked_bonded.py @@ -138,7 +138,10 @@ class TestBondingStacked(TestCase): cmds = [ ["port stop all", ''], ["set portlist " + ",".join([str(port) for port in portList]), ''], - ["port start all", ' ', 15]] + # start top level bond port only, and let it propagate the start + # action to slave bond ports and its the real nics. + ["port start {}".format(master_bond_port), ' ', 15] + ] self.bond_inst.d_console(cmds) # blank space command is used to skip LSC event to avoid core dumped issue time.sleep(5) -- 1.9.3