From mboxrd@z Thu Jan 1 00:00:00 1970 From: Srinivas Pandruvada Subject: Re: [PATCH v1 0/2] crash on reading cpufreq sysfs Date: Thu, 08 Oct 2015 08:17:57 -0700 Message-ID: <1444317477.2971.1.camel@linux.intel.com> References: <1444251044-11231-1-git-send-email-srinivas.pandruvada@linux.intel.com> <1543959.r5bbusBA6I@vostro.rjw.lan> <1444261800.7437.7.camel@spandruv-desk3.jf.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: Received: from mga11.intel.com ([192.55.52.93]:19682 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933524AbbJHPSV (ORCPT ); Thu, 8 Oct 2015 11:18:21 -0400 In-Reply-To: Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: "Rafael J. Wysocki" Cc: "Rafael J. Wysocki" , Rafael Wysocki , Viresh Kumar , "linux-pm@vger.kernel.org" On Thu, 2015-10-08 at 02:17 +0200, Rafael J. Wysocki wrote: > On Thu, Oct 8, 2015 at 1:50 AM, Srinivas Pandruvada > wrote: > > On Thu, 2015-10-08 at 00:50 +0200, Rafael J. Wysocki wrote: > > > On Wednesday, October 07, 2015 01:50:42 PM Srinivas Pandruvada > > > wrote: > > > > Fixes for crash on reading cpufreq sysfs for offline cpus > > > > > > > > v1 > > > > Fix commit discription on Patch 1/2. Included Acks from Viresh. > > > > It has both patches, even though only one patch has change. > > > > > > > > v0 > > > > two patches: one for acp-cpufreq and one for cpufreq core > > > > > > > > Srinivas Pandruvada (2): > > > > cpufreq: acpi_cpufreq: prevent crash on reading > > > > freqdomain_cpus > > > > cpufreq: prevent lockup on reading > > > > scaling_available_frequencies > > > > > > Both applied, but they look like -stable material to me. > > > > > I tested in 4.1 kernels, since on offline, cpufreq folder is not > > available there is no issue. > > For 4.2 kernels the I didn't see crash fixed by cpufreq core patch > > (cat > > scaling_available_frequencies), but the "cat freqdomain_cpus" still > > causes crash. So we need to apply cpufreq:acpi-cpufreq patch only. > > So > > one patch is applicable for stable tree. > > So [1/2] should go into 4.2.y and [2/2] fixes a regression introduced > in this cycle, right? Yes. Thanks, Srinivas > > Thanks, > Rafael