From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] usertools: fix cpu_layout script for multithreads of more than 2 Date: Sun, 30 Apr 2017 15:29:19 +0200 Message-ID: <2353390.jtLUV08Iqy@xps> References: <1874376.tUCind7prS@xps> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, Andriy Berestovskyy To: Gowrishankar Return-path: Received: from out4-smtp.messagingengine.com (out4-smtp.messagingengine.com [66.111.4.28]) by dpdk.org (Postfix) with ESMTP id 1504A6932 for ; Sun, 30 Apr 2017 15:29:20 +0200 (CEST) In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 28/04/2017 14:25, Andriy Berestovskyy: > Works fine on ThunderX and does not brake Intel either. > > Reviewed-by: Andriy Berestovskyy > Tested-by: Andriy Berestovskyy > > Andriy > > On 28.04.2017 13:58, Thomas Monjalon wrote: > > Andriy, please would you like to review this patch? > > > > 28/04/2017 12:34, Gowrishankar: > >> From: Gowrishankar Muthukrishnan > >> > >> Current usertools/cpu_layout.py is broken to handle multithreads of count more > >> than 2 as in IBM powerpc P8 servers. Below patch addressed this issue. Also, > >> added minor exception catch on failing to open unavailable sys file in case of > >> multithread=off configuration in server. > >> > >> Patch has been verified not to break existing topology configurations > >> and also not changing anything in current output. > >> > >> Signed-off-by: Gowrishankar Muthukrishnan Applied, thanks