From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: [PATCH v8 8/8] cpufreq: intel_pstate: Use CPPC to get max performance Date: Fri, 9 Dec 2016 16:02:24 +0100 Message-ID: References: <20161207190608.boi7svavwrkx3epm@breakpoint.cc> <1481152373.3064.77.camel@linux.intel.com> <4248602.ecJ9y5gJMR@aspire.rjw.lan> <20161209144551.jhnbfsxdxsk4ld6s@linutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <20161209144551.jhnbfsxdxsk4ld6s@linutronix.de> Sender: linux-kernel-owner@vger.kernel.org To: Sebastian Andrzej Siewior Cc: "Rafael J. Wysocki" , Tim Chen , Thomas Gleixner , Ingo Molnar , Borislav Petkov , "Rafael J. Wysocki" , the arch/x86 maintainers , Linux PM , Linux Kernel Mailing List , ACPI Devel Maling List , Peter Zijlstra , jolsa@redhat.com, Srinivas Pandruvada List-Id: linux-acpi@vger.kernel.org On Fri, Dec 9, 2016 at 3:45 PM, Sebastian Andrzej Siewior wrote: > On 2016-12-08 00:29:29 [+0100], Rafael J. Wysocki wrote: >> Agreed, but the bug fixed by the first hunk is real too. I'd fix it a bit >> differently, though: >> >> Tentatively-signed-off-by: Rafael J. Wysocki > > Is there anything you want me to do here? The hunk in > acpi_cppc_processor_exit() is unchanged and is the one that led to the > crash. The other hunk I made (the one you changed) was something I > noticed while looking at the code - nothing that hit me directly. OK, thanks. I'll add a changelog to the patch and resend it later today. Thanks, Rafael