From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from [140.186.70.92] (port=55396 helo=eggs.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1PpOf8-0006yd-Ck
	for qemu-devel@nongnu.org; Tue, 15 Feb 2011 12:26:08 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <adam@os.inf.tu-dresden.de>) id 1PpOf5-000525-1O
	for qemu-devel@nongnu.org; Tue, 15 Feb 2011 12:26:04 -0500
Received: from os.inf.tu-dresden.de ([141.76.48.99]:48567)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <adam@os.inf.tu-dresden.de>) id 1PpOf4-00051R-Q1
	for qemu-devel@nongnu.org; Tue, 15 Feb 2011 12:26:02 -0500
Date: Tue, 15 Feb 2011 18:25:58 +0100
From: Adam Lackorzynski <adam@os.inf.tu-dresden.de>
Subject: Re: [Qemu-devel] [PATCH 1/3] target-arm: Setup smpboot code in all
	setups
Message-ID: <20110215172558.GJ19666@os.inf.tu-dresden.de>
References: <20110215104852.GB19666@os.inf.tu-dresden.de>
	<AANLkTik570wef0SHL3RfhEb-6O8m1T6io_NcRZEvOBuA@mail.gmail.com>
	<20110215131245.GE19666@os.inf.tu-dresden.de>
	<AANLkTin0Bk5G7vDEMn25wDNrve8b-TMc1CxiXopU6LFF@mail.gmail.com>
	<20110215143017.GF19666@os.inf.tu-dresden.de>
	<AANLkTikd+dbzhnV30YkrocDo-1htmFzCqsp0Xpdo8D1N@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
In-Reply-To: <AANLkTikd+dbzhnV30YkrocDo-1htmFzCqsp0Xpdo8D1N@mail.gmail.com>
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Vincent Palatin <vincent.palatin_qemu@polytechnique.org>
Cc: Peter Maydell <peter.maydell@linaro.org>, qemu-devel@nongnu.org

Hi,

On Tue Feb 15, 2011 at 10:02:05 -0500, Vincent Palatin wrote:
> >> Moving in the right direction, but it would be cleaner if the secondary
> >> CPU reset was handled inside arm_boot.c, I think (there is a TODO
> >> in that file to that effect). Then we could get rid of the cpu reset
> >> hook from realview.c.
> >
> > Like the following?
> 
> This assumes that all the ARM SMP platforms are booting their
> secondary CPU the same way as the emulated Realview.
> For example, I'm currently writing a Tegra2 (dual A9) SoC emulation
> and the second CPU is halted when the platform starts and I cannot
> re-use the current smpboot firmware chunk. My current workaround is to
> use "info->nb_cpus = 1" and do the init in the board code. Forcing the
> reset function will probably not help.

The smpboot code also halts the CPUs, i.e. they are waiting in wfi.
What would you like to have instead? Maybe it's not so different...


Adam
-- 
Adam                 adam@os.inf.tu-dresden.de
  Lackorzynski         http://os.inf.tu-dresden.de/~adam/