From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Gleixner Subject: Re: [PATCH] powercap: intel_rapl: PSys support Date: Fri, 18 Mar 2016 10:52:40 +0100 (CET) Message-ID: References: <1458253409-13318-1-git-send-email-srinivas.pandruvada@linux.intel.com> <20160317225100.GA3805@pd.tnic> <1458255363.3861.40.camel@linux.intel.com> <20160317185532.7ab0adc5@yairi> Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Return-path: Received: from www.linutronix.de ([62.245.132.108]:35963 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751280AbcCRJyS (ORCPT ); Fri, 18 Mar 2016 05:54:18 -0400 In-Reply-To: <20160317185532.7ab0adc5@yairi> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: Jacob Pan Cc: Srinivas Pandruvada , Borislav Petkov , rjw@rjwysocki.net, linux-pm@vger.kernel.org, mingo@redhat.com, x86@kernel.org, "H. Peter Anvin" , LKML , Peter Zijlstra On Thu, 17 Mar 2016, Jacob Pan wrote: > On Thu, 17 Mar 2016 15:56:03 -0700 > Srinivas Pandruvada wrote: > > > > > +#define MSR_PLATFORM_ENERGY_STATUS 0x0000064d > > > > +#define MSR_PLATFORM_POWER_LIMIT 0x0000065c > > > Please add those defines to intel_rapl.c as they're being used only > > > there, apparently. > > OK. I will move them. > > I think it could be used by perf rapl pmu also, but later. Right. And they should be added to perf _BEFORE_ that powercap stuff utilizes them. Thanks, tglx