From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:57172) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TB4Ke-0000hO-U7 for qemu-devel@nongnu.org; Mon, 10 Sep 2012 09:47:24 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TB4KY-00051B-Tt for qemu-devel@nongnu.org; Mon, 10 Sep 2012 09:47:20 -0400 Message-ID: <504DEF5F.80905@suse.de> Date: Mon, 10 Sep 2012 15:47:11 +0200 From: =?ISO-8859-15?Q?Andreas_F=E4rber?= MIME-Version: 1.0 References: <1347259132-31184-1-git-send-email-david@gibson.dropbear.id.au> <1347259132-31184-4-git-send-email-david@gibson.dropbear.id.au> In-Reply-To: <1347259132-31184-4-git-send-email-david@gibson.dropbear.id.au> Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 3/7] pseries: Use new method to correct reset sequence List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: David Gibson Cc: qemu-ppc@nongnu.org, agraf@suse.de, qemu-devel@nongnu.org Am 10.09.2012 08:38, schrieb David Gibson: > A number of things need to occur during reset of the PAPR > paravirtualized platform in a specific order. For example, the hash > table needs to be cleared before the CPUs are reset, so that they > initialize their register state correctly, and the CPUs need to have > their main reset called before we set up the entry point state on the > boot cpu. We also need to have the main qdev reset happen before the > creation and installation of the device tree for the new boot, because > we need the state of the devices settled to correctly construct the > device tree. >=20 > We currently do the pseries once-per-reset initializations done from a > reset handler. However we can't adequately control when this handler > is called during the reset - in particular we can't guarantee it > happens after all the qdev resets (since qdevs might be registered > after the machine init function has executed). >=20 > This patch uses the new QEMUMachine reset method to to fix this > problem, ensuring the various order dependent reset steps happen in > the correct order. >=20 > Signed-off-by: David Gibson Reviewed-by: Andreas F=E4rber Andreas --=20 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=FCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=F6rffer; HRB 16746 AG N=FCrnbe= rg