From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753844AbbICGWU (ORCPT ); Thu, 3 Sep 2015 02:22:20 -0400 Received: from e23smtp08.au.ibm.com ([202.81.31.141]:59005 "EHLO e23smtp08.au.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752040AbbICGWS (ORCPT ); Thu, 3 Sep 2015 02:22:18 -0400 X-Helo: d23dlp03.au.ibm.com X-MailFrom: shreyas@linux.vnet.ibm.com X-RcptTo: linux-pm@vger.kernel.org Message-ID: <55E7E6E2.3090900@linux.vnet.ibm.com> Date: Thu, 03 Sep 2015 11:51:22 +0530 From: Shreyas B Prabhu User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.1.0 MIME-Version: 1.0 To: rjw@rjwysocki.net CC: Thomas Renninger , pavsubra@linux.vnet.ibm.com, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org Subject: Re: [PATCH] cpupower tools: Fix error when running cpupower monitor References: <1438582560-13352-1-git-send-email-shreyas@linux.vnet.ibm.com> <9534146.fFc3TD2C5D@skinner> <55D192AF.5000602@linux.vnet.ibm.com> <55DC58BD.4070108@linux.vnet.ibm.com> In-Reply-To: <55DC58BD.4070108@linux.vnet.ibm.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 15090306-0029-0000-0000-0000021DDE37 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 08/25/2015 05:29 PM, Shreyas B Prabhu wrote: > > > On 08/17/2015 01:22 PM, Shreyas B Prabhu wrote: >> >> >> On 08/10/2015 05:58 PM, Thomas Renninger wrote: >>> On Monday, August 03, 2015 11:46:00 AM Shreyas B. Prabhu wrote: >>>> get_cpu_topology() tries to get topology info from all cpus by reading >>>> files in the topology sysfs dir. If a cpu is offlined, since it doesn't >>>> have topology dir, this function fails and returns -1. This causes >>>> functions relying on get_cpu_topology() to fail. For example- >>>> >>>> $ cpupower monitor >>>> Cannot read number of available processors >>>> >>>> Fix this by skipping fetching topology info for offline cpus. >>> >>> Looks fine. >>> >>> Thanks! >>> >>> Acked-by: Thomas Renninger >>> >> >> Thanks Thomas! >> Rafael, can you please pick this patch? >> >> > > > Hi Rafael, > > If this patch looks good can you please pick this up? > > > Thanks, > Shreyas > Hi Rafael, If this patch looks good can you please pick this up? Thanks, Shreyas