From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from na01-bl2-obe.outbound.protection.outlook.com (mail-bl2on0136.outbound.protection.outlook.com [65.55.169.136]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3BC501A1D13 for ; Fri, 7 Aug 2015 13:19:43 +1000 (AEST) Date: Fri, 7 Aug 2015 11:19:32 +0800 From: Chenhui Zhao Subject: Re: [PATCH 1/3] Powerpc: mpc85xx: refactor the PM operations To: Scott Wood CC: , , , Tang Yuantian , "linuxppc-dev@lists.ozlabs.org" Message-ID: <1438917572.2431.0@remotesmtp.freescale.net> In-Reply-To: <1438884172.2097.155.camel@freescale.com> References: <1438334444-31919-1-git-send-email-b29983@freescale.com> <1438387178.19345.77.camel@freescale.com> <1438601578.7515.2@remotesmtp.freescale.net> <1438633568.2097.35.camel@freescale.com> <1438769477.21522.0@remotesmtp.freescale.net> <1438829848.2097.129.camel@freescale.com> <1438834837.23384.0@remotesmtp.freescale.net> <1438839985.2097.151.camel@freescale.com> <1438840466.23384.2@remotesmtp.freescale.net> <1438884172.2097.155.camel@freescale.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8"; format=flowed List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Fri, Aug 7, 2015 at 2:02 AM, Scott Wood wrote: > On Thu, 2015-08-06 at 13:54 +0800, Chenhui Zhao wrote: >> On Thu, Aug 6, 2015 at 1:46 PM, Scott Wood >> wrote: >> > On Thu, 2015-08-06 at 12:20 +0800, Chenhui Zhao wrote: >> > > On Thu, Aug 6, 2015 at 10:57 AM, Scott Wood >> > > >> > > wrote: >> > > > On Wed, 2015-08-05 at 18:11 +0800, Chenhui Zhao wrote: >> > > > > On Tue, Aug 4, 2015 at 4:26 AM, Scott Wood >> > > >> > > > > wrote: >> > > > > > On Mon, 2015-08-03 at 19:32 +0800, Chenhui Zhao wrote: >> > > > > > > > >> > > > > > >> > > > > > > On Sat, Aug 1, 2015 at 7:59 AM, Scott Wood >> > > > > >> > > > > > > wrote: >> > > > > > >> > > > > > > > >> > > > > > > > Could you explain irq_mask()? Why would there >> still be >> > > IRQs >> > > > > > > destined >> > > > > > > > for >> > > > > > > > this CPU at this point? >> > > > > > > >> > > > > > > This function just masks irq by setting the >> registers in >> > > RCPM >> > > > > (for >> > > > > > > example, RCPM_CPMIMR, RCPM_CPMCIMR). Actually, all >> irqs to >> > > > > this CPU >> > > > > > > have been migrated to other CPUs. >> > > > > > >> > > > > > So why do we need to set those bits in RCPM? Is it just >> > > caution? >> > > > > >> > > > > Setting these bits can mask interrupts signalled to RCPM >> from >> > > MPIC >> > > > > as a >> > > > > means of >> > > > > waking up from a lower power state. So, cores will not be >> > > waked up >> > > > > unexpectedly. >> > > > >> > > > Why would the MPIC be signalling those interrupts if they've >> been >> > > > masked at >> > > > the MPIC? >> > > > >> > > > -Scott >> > > > >> > > >> > > The interrupts to RCPM from MPIC are IRQ, Machine Check, NMI >> and >> > > Critical interrupts. Some of them didn't be masked in MPIC. >> > >> > What interrupt could actually happen to a sleeping cpu that this >> > protects >> > against? >> > >> > -Scott >> >> Not sure. Maybe spurious interrupts or hardware exceptions. > > Spurious interrupts happen due to race conditions. They don't happen > because > the MPIC is bored and decides to ring a CPU's doorbell and hide in > the bushes. > > If by "hardware exceptions" you mean machine checks, how would such a > machine > check be generated by a core that is off? > >> However, setting them make sure dead cpus can not be waked up >> unexpectedly. > > I'm not seeing enough value here to warrant resurrecting the old > sleep node > stuff. > > -Scott My guess maybe not accurate. My point is that electronic parts don't always work as expected. Taking preventative measures can make the system more robust. In addition, this step is required in deep sleep procedure. -Chenhui