From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from e23smtp06.au.ibm.com (e23smtp06.au.ibm.com [202.81.31.148]) by dpdk.org (Postfix) with ESMTP id A0638374E for ; Tue, 8 Mar 2016 11:07:58 +0100 (CET) Received: from localhost by e23smtp06.au.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Tue, 8 Mar 2016 20:07:56 +1000 Received: from d23dlp01.au.ibm.com (202.81.31.203) by e23smtp06.au.ibm.com (202.81.31.212) with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted; Tue, 8 Mar 2016 20:07:55 +1000 X-IBM-Helo: d23dlp01.au.ibm.com X-IBM-MailFrom: gowrishankar.m@linux.vnet.ibm.com X-IBM-RcptTo: dts@dpdk.org Received: from d23relay09.au.ibm.com (d23relay09.au.ibm.com [9.185.63.181]) by d23dlp01.au.ibm.com (Postfix) with ESMTP id ACC392CE8054 for ; Tue, 8 Mar 2016 21:07:54 +1100 (EST) Received: from d23av02.au.ibm.com (d23av02.au.ibm.com [9.190.235.138]) by d23relay09.au.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id u28A7kXf61472920 for ; Tue, 8 Mar 2016 21:07:54 +1100 Received: from d23av02.au.ibm.com (localhost [127.0.0.1]) by d23av02.au.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id u28A7MmC025709 for ; Tue, 8 Mar 2016 21:07:22 +1100 Received: from chozha.in.ibm.com (chozha.in.ibm.com [9.124.124.138] (may be forged)) by d23av02.au.ibm.com (8.14.4/8.14.4/NCO v10.0 AVin) with ESMTP id u28A7Dh5024744; Tue, 8 Mar 2016 21:07:21 +1100 From: Gowrishankar To: dts Date: Tue, 8 Mar 2016 15:36:44 +0530 Message-Id: <1457431607-31596-7-git-send-email-gowrishankar.m@linux.vnet.ibm.com> X-Mailer: git-send-email 1.7.10.4 In-Reply-To: <1457431607-31596-1-git-send-email-gowrishankar.m@linux.vnet.ibm.com> References: <1457431607-31596-1-git-send-email-gowrishankar.m@linux.vnet.ibm.com> X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 16030810-0021-0000-0000-000002ED6F2A Subject: [dts] [PATCH v2 6/9] framework: fix get_core_list to return all lcores 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: , X-List-Received-Date: Tue, 08 Mar 2016 10:07:59 -0000 From: Gowri Shankar Irrespective of whether threading is enabled or not, "all" option to get_core_ list should return all available lcores. Current "th" option breaks the test on loading PMD on an unavailable lcore because cpu core ranges is returned. Signed-off-by: Gowrishankar --- framework/crb.py | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/framework/crb.py b/framework/crb.py index 453796f..33ceb8c 100644 --- a/framework/crb.py +++ b/framework/crb.py @@ -574,17 +574,13 @@ class Crb(object): # return thread list return map(str, thread_list) - def get_core_list(self, config, th=False, socket=-1): + def get_core_list(self, config, socket=-1): """ Get lcore array according to the core config like "all", "1S/1C/1T". We can specify the physical CPU socket by paramter "socket". """ if config == 'all': - - if th: - return [n['thread'] for n in self.cores] - else: - return [n for n in range(0, self.number_of_cores - 1)] + return [n['thread'] for n in self.cores] m = re.match("([1234])S/([1-9]+)C/([12])T", config) -- 1.7.10.4