From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ingo Molnar Subject: Re: [PATCH] x86: intel_epb: Take CONFIG_PM into account Date: Fri, 10 May 2019 08:01:56 +0200 Message-ID: <20190510060156.GA20852@gmail.com> References: <1637073.gl2OfxWTjI@aspire.rjw.lan> <1627338.1fd8ofggM8@kreacher> <20190509174338.GA24432@splinter> <3431308.1mSSVdqTRr@kreacher> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <3431308.1mSSVdqTRr@kreacher> Sender: linux-kernel-owner@vger.kernel.org To: "Rafael J. Wysocki" Cc: Linux PM , Ido Schimmel , x86 , LKML , Len Brown , Srinivas Pandruvada , Laura Abbott , Thomas Gleixner , Peter Zijlstra , Simon Schricker , Borislav Petkov , Hannes Reinecke List-Id: linux-pm@vger.kernel.org * Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > Commit b9c273babce7 (PM / arch: x86: MSR_IA32_ENERGY_PERF_BIAS sysfs > interface) caused kernels built with CONFIG_PM unset to crash on > systems supporting the Performance and Energy Bias Hint (EPB), > because it attempts to add files to sysfs directories that don't > exist on those systems. > > Prevent that from happening by taking CONFIG_PM into account so > that the code depending on it is not compiled at all when it is > not set. > > Fixes: b9c273babce7 (PM / arch: x86: MSR_IA32_ENERGY_PERF_BIAS sysfs interface) > Reported-by: Ido Schimmel > Tested-by: Ido Schimmel > Signed-off-by: Rafael J. Wysocki > --- > arch/x86/kernel/cpu/intel_epb.c | 22 +++++++++++++++++++++- > 1 file changed, 21 insertions(+), 1 deletion(-) Acked-by: Ingo Molnar Thanks, Ingo