From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754959AbcEXKZE (ORCPT ); Tue, 24 May 2016 06:25:04 -0400 Received: from e36.co.us.ibm.com ([32.97.110.154]:33872 "EHLO e36.co.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751518AbcEXKZA (ORCPT ); Tue, 24 May 2016 06:25:00 -0400 X-IBM-Helo: d03dlp03.boulder.ibm.com X-IBM-MailFrom: ego@linux.vnet.ibm.com X-IBM-RcptTo: mpe@ellerman.id.au;linuxppc-dev@lists.ozlabs.org;mikey@neuling.org;paulus@ozlabs.org;linux-kernel@vger.kernel.org Date: Tue, 24 May 2016 15:54:49 +0530 From: Gautham R Shenoy To: "Shreyas B. Prabhu" Cc: mpe@ellerman.id.au, linuxppc-dev@lists.ozlabs.org, paulus@ozlabs.org, linux-kernel@vger.kernel.org, mikey@neuling.org, ego@linux.vnet.ibm.com Subject: Re: [PATCH v3 7/9] powerpc/powernv: Add platform support for stop instruction Message-ID: <20160524102449.GE12860@in.ibm.com> Reply-To: ego@linux.vnet.ibm.com References: <1464016722-7488-1-git-send-email-shreyas@linux.vnet.ibm.com> <1464016722-7488-8-git-send-email-shreyas@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1464016722-7488-8-git-send-email-shreyas@linux.vnet.ibm.com> User-Agent: Mutt/1.5.23 (2014-03-12) X-TM-AS-GCONF: 00 X-Content-Scanned: Fidelis XPS MAILER x-cbid: 16052410-0021-0000-0000-00005218A0A7 X-IBM-AV-DETECTION: SAVI=unused REMOTE=unused XFE=unused Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Shreyas, On Mon, May 23, 2016 at 08:48:40PM +0530, Shreyas B. Prabhu wrote: > @@ -412,7 +517,8 @@ subcore_state_restored: > first_thread_in_core: > > /* > - * First thread in the core waking up from fastsleep. It needs to > + * First thread in the core waking up from any state which can cause > + * partial or complete hypervisor state loss. It needs to > * call the fastsleep workaround code if the platform requires it. > * Call it unconditionally here. The below branch instruction will > * be patched out when the idle states are discovered if platform Please update the comment to "The below branch instruction will be patched out if the platform does not have fastsleep or does not require the workaround. Patching will be performed during the discovery of idle-states." > @@ -423,8 +529,10 @@ pnv_fastsleep_workaround_at_exit: > b fastsleep_workaround_at_exit > > timebase_resync: > - /* Do timebase resync if we are waking up from sleep. Use cr3 value > - * set in exceptions-64s.S */ > + /* > + * Use cr3 which indicates that we are waking up with atleast partial > + * hypervisor state loss to determine if TIMEBASE RESYNC is needed. > + */ > ble cr3,clear_lock > /* Time base re-sync */ > li r0,OPAL_RESYNC_TIMEBASE [..snip..] > @@ -264,6 +275,32 @@ static int __init pnv_init_idle_states(void) > goto out_free; > } > > + if (cpu_has_feature(CPU_FTR_ARCH_300)) { > + psscr_val = kcalloc(dt_idle_states, sizeof(*psscr_val), > + GFP_KERNEL); > + if (!psscr_val) > + goto out_free; > + if (of_property_read_u64_array(power_mgt, > + "ibm,cpu-idle-state-psscr", > + psscr_val, dt_idle_states)) { > + pr_warn("cpuidle-powernv: missing ibm,cpu-idle-states-psscr in DT\n"); > + goto out_free_psscr; > + } > + > + /* > + * Set pnv_first_deep_stop_state to the first stop level > + * to cause hypervisor state loss > + */ > + pnv_first_deep_stop_state = 0xF; #define MAX_STOP_STATES 0xF ? > + for (i = 0; i < dt_idle_states; i++) { > + u64 psscr_rl = psscr_val[i] & PSSCR_RL_MASK; > + > + if ((flags[i] & OPAL_PM_LOSE_FULL_CONTEXT) && > + (pnv_first_deep_stop_state > psscr_rl)) > + pnv_first_deep_stop_state = psscr_rl; > + } > + } > + > for (i = 0; i < dt_idle_states; i++) > supported_cpuidle_states |= flags[i]; >