From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752815AbcGHEUQ (ORCPT ); Fri, 8 Jul 2016 00:20:16 -0400 Received: from mx0b-001b2d01.pphosted.com ([148.163.158.5]:59062 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751428AbcGHEUN (ORCPT ); Fri, 8 Jul 2016 00:20:13 -0400 X-IBM-Helo: d28dlp01.in.ibm.com X-IBM-MailFrom: shreyas@linux.vnet.ibm.com X-IBM-RcptTo: linux-kernel@vger.kernel.org Date: Fri, 08 Jul 2016 09:49:59 +0530 From: Shreyas B Prabhu User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.1.0 MIME-Version: 1.0 To: Michael Neuling , mpe@ellerman.id.au CC: benh@au1.ibm.com, paulus@ozlabs.org, ego@linux.vnet.ibm.com, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org, maddy@linux.vnet.ibm.com Subject: Re: [PATCH v7 07/11] powerpc/powernv: Add platform support for stop instruction References: <1467924432-29003-1-git-send-email-shreyas@linux.vnet.ibm.com> <1467924432-29003-8-git-send-email-shreyas@linux.vnet.ibm.com> <1467944442.27479.155.camel@neuling.org> In-Reply-To: <1467944442.27479.155.camel@neuling.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 16070804-0008-0000-0000-000002DA4E04 X-IBM-AV-DETECTION: SAVI=unused REMOTE=unused XFE=unused x-cbparentid: 16070804-0009-0000-0000-00000E01C1C5 Message-Id: <577F29EF.9090405@linux.vnet.ibm.com> X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2016-07-08_02:,, signatures=0 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 suspectscore=0 malwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1604210000 definitions=main-1607080040 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 07/08/2016 07:50 AM, Michael Neuling wrote: > >> diff --git a/arch/powerpc/include/asm/cpuidle.h b/arch/powerpc/include/asm/cpuidle.h >> index d2f99ca..3d7fc06 100644 >> --- a/arch/powerpc/include/asm/cpuidle.h >> +++ b/arch/powerpc/include/asm/cpuidle.h >> @@ -13,6 +13,8 @@ >> #ifndef __ASSEMBLY__ >> extern u32 pnv_fastsleep_workaround_at_entry[]; >> extern u32 pnv_fastsleep_workaround_at_exit[]; >> + >> +extern u64 pnv_first_deep_stop_state; > > mpe asked a question about this which you neither answered or addressed. > "Should this have some safe initial value?" > > I'm thinking we could do this which is what you have in the init call. > u64 pnv_first_deep_stop_state = MAX_STOP_STATE; > I missed the comment. I'll make the change. > >> @@ -439,7 +540,18 @@ timebase_resync: >> */ >> bne cr4,clear_lock >> >> - /* Restore per core state */ >> + /* >> + * First thread in the core to wake up and its waking up with >> + * complete hypervisor state loss. Restore per core hypervisor >> + * state. >> + */ >> +BEGIN_FTR_SECTION >> + ld r4,_PTCR(r1) >> + mtspr SPRN_PTCR,r4 >> + ld r4,_RPR(r1) >> + mtspr SPRN_RPR,r4 > > RPR looks wrong here. This should be on POWER8 too. > > This has changed since v6 and not noted in the v7 comments. Why are you > changing this now? > RPR is a per-core resource in P9. So with this patch, RPR will continue to be restored per-subcore in P8 and will restored once per core in P9. >> +END_FTR_SECTION_IFSET(CPU_FTR_ARCH_300) >> + >> ld r4,_TSCR(r1) >> mtspr SPRN_TSCR,r4 >> ld r4,_WORC(r1) >> @@ -461,9 +573,7 @@ common_exit: >> >> /* Waking up from winkle */ >> >> - /* Restore per thread state */ >> - bl __restore_cpu_power8 >> - >> +BEGIN_MMU_FTR_SECTION >> /* Restore SLB from PACA */ >> ld r8,PACA_SLBSHADOWPTR(r13) >> >> @@ -477,6 +587,9 @@ common_exit: >> slbmte r6,r5 >> 1: addi r8,r8,16 >> .endr >> +END_MMU_FTR_SECTION_IFCLR(MMU_FTR_RADIX) >> + >> + /* Restore per thread state */ > > This FTR section is too big It ends up at 25 instructions with the loop. > Probably better like this: > > BEGIN_MMU_FTR_SECTION > b no_segments > END_MMU_FTR_SECTION_IFSET(MMU_FTR_RADIX) > /* Restore SLB from PACA */ > ld r8,PACA_SLBSHADOWPTR(r13) > > .rept SLB_NUM_BOLTED > li r3, SLBSHADOW_SAVEAREA > LDX_BE r5, r8, r3 > addi r3, r3, 8 > LDX_BE r6, r8, r3 > andis. r7,r5,SLB_ESID_V@h > beq 1f > slbmte r6,r5 > 1: addi r8,r8,16 > .endr > > no_segments: > Cool. Will make the change. Thanks, Shreyas