From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1Lbt6t-0004nt-J9
	for qemu-devel@nongnu.org; Tue, 24 Feb 2009 03:57:51 -0500
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1Lbt6r-0004mn-Pt
	for qemu-devel@nongnu.org; Tue, 24 Feb 2009 03:57:50 -0500
Received: from [199.232.76.173] (port=43714 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1Lbt6r-0004mk-Kh
	for qemu-devel@nongnu.org; Tue, 24 Feb 2009 03:57:49 -0500
Received: from mail-fx0-f161.google.com ([209.85.220.161]:61510)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <laurent.desnogues@gmail.com>) id 1Lbt6r-0002sa-8n
	for qemu-devel@nongnu.org; Tue, 24 Feb 2009 03:57:49 -0500
Received: by fxm5 with SMTP id 5so2512220fxm.10
	for <qemu-devel@nongnu.org>; Tue, 24 Feb 2009 00:57:48 -0800 (PST)
MIME-Version: 1.0
In-Reply-To: <49A3B366.5070603@nomovok.com>
References: <49A3A9EA.4040300@nomovok.com>
	<761ea48b0902240027k45a6b687o3d1367957a849d00@mail.gmail.com>
	<49A3B366.5070603@nomovok.com>
Date: Tue, 24 Feb 2009 09:57:48 +0100
Message-ID: <761ea48b0902240057u6b82773er55780766ec3a23ba@mail.gmail.com>
Subject: Re: [Qemu-devel] [PATCH] Fix qemu_ld64 on arm
From: Laurent Desnogues <laurent.desnogues@gmail.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
Reply-To: qemu-devel@nongnu.org
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.gnu.org/pipermail/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: qemu-devel@nongnu.org

On Tue, Feb 24, 2009 at 9:44 AM, Pablo Virolainen
<Pablo.Virolainen@nomovok.com> wrote:
> Laurent Desnogues kirjoitti:
>>
>> On Tue, Feb 24, 2009 at 9:03 AM, Pablo Virolainen
>> <Pablo.Virolainen@nomovok.com> wrote:
>>>
>>> Emulating fldl on arm doesn't seem to work too well. It's the way
>>> qemu_ld64
>>> is translated to arm instructions.
>>>
>>> =A0 =A0 =A0 tcg_out_ld32_12(s, COND_AL, data_reg, addr_reg, 0);
>>> =A0 =A0 =A0 tcg_out_ld32_12(s, COND_AL, data_reg2, addr_reg, 4);
>>
>> It seems you are using an old version of qemu. =A0Taking a quick
>> look at current svn, I think your patch is not needed anymore.
>
> So I should get the source from other source than
> svn co svn://svn.savannah.nongnu.org/qemu/trunk ?

OK, I am gonna get some more coffee...  I mistakenly thought
you were talking about ARM front-end.  Sorry for the
confusion.

Your patch looks correct (but bear in mind, that I did not take
more coffee and that I never played with ARM back-end).


Laurent