From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ozlabs.org (ozlabs.org [IPv6:2401:3900:2:1::2]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3t4Pzg0GmnzDt3Q for ; Thu, 27 Oct 2016 22:38:55 +1100 (AEDT) In-Reply-To: <20161013021714.6624-1-npiggin@gmail.com> To: Nicholas Piggin , linuxppc-dev@lists.ozlabs.org From: Michael Ellerman Cc: "Shreyas B . Prabhu" , Nicholas Piggin , "Gautham R . Shenoy" Subject: Re: powerpc/64s: relocation, register save fixes for system reset interrupt Message-Id: <3t4Pzf6KdDz9rxw@ozlabs.org> Date: Thu, 27 Oct 2016 22:38:54 +1100 (AEDT) List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thu, 2016-13-10 at 02:17:14 UTC, Nicholas Piggin wrote: > This patch does a couple of things. First of all, powernv immediately > explodes when running a relocated kernel, because the system reset > exception for handling sleeps does not do correct relocated branches. > > Secondly, the sleep handling code trashes the condition and cfar > registers, which we would like to preserve for debugging purposes (for > non-sleep case exception). > > This patch changes the exception to use the standard format that saves > registers before any tests or branches are made. It adds the test for > idle-wakeup as an "extra" to break out of the normal exception path. > Then it branches to a relocated idle handler that calls the various > idle handling functions. > > After this patch, POWER8 CPU simulator now boots powernv kernel that is > running at non-zero. > > Cc: Balbir Singh > Cc: Shreyas B. Prabhu > Cc: Gautham R. Shenoy > Signed-off-by: Nicholas Piggin > Acked-by: Gautham R. Shenoy > Acked-by: Balbir Singh Applied to powerpc fixes, thanks. https://git.kernel.org/powerpc/c/fb479e44a9e240a23c2d208c2ace23 cheers