From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:48280)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <ehabkost@redhat.com>) id 1anAJZ-0003A9-Ba
	for qemu-devel@nongnu.org; Mon, 04 Apr 2016 15:37:34 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <ehabkost@redhat.com>) id 1anAJY-0006ME-J8
	for qemu-devel@nongnu.org; Mon, 04 Apr 2016 15:37:33 -0400
Date: Mon, 4 Apr 2016 16:37:23 -0300
From: Eduardo Habkost <ehabkost@redhat.com>
Message-ID: <20160404193723.GD22665@thinpad.lan.raisama.net>
References: <1458910214-12239-1-git-send-email-aleksandar.markovic@rt-rk.com>
	<1458910214-12239-2-git-send-email-aleksandar.markovic@rt-rk.com>
	<56F9A3D1.2050402@twiddle.net>
	<EF5FA6C3467F85449672C3E735957B859168FB0A@BADAG02.ba.imgtec.org>
	<CAFEAcA_Rp+8hGC6sos3H3t2A8UDgLUf3KvrM3-C3zZAFCc_WEA@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAFEAcA_Rp+8hGC6sos3H3t2A8UDgLUf3KvrM3-C3zZAFCc_WEA@mail.gmail.com>
Subject: Re: [Qemu-devel] [PATCH 1/2] softfloat: Enable
 run-time-configurable meaning of signaling NaN bit
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/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: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: "proljc@gmail.com" <proljc@gmail.com>, "kbastian@mail.uni-paderborn.de" <kbastian@mail.uni-paderborn.de>, "mark.cave-ayland@ilande.co.uk" <mark.cave-ayland@ilande.co.uk>, "qemu-devel@nongnu.org" <qemu-devel@nongnu.org>, "agraf@suse.de" <agraf@suse.de>, "blauwirbel@gmail.com" <blauwirbel@gmail.com>, "jcmvbkbc@gmail.com" <jcmvbkbc@gmail.com>, Aleksandar Markovic <Aleksandar.Markovic@imgtec.com>, "qemu-arm@nongnu.org" <qemu-arm@nongnu.org>, "qemu-ppc@nongnu.org" <qemu-ppc@nongnu.org>, Petar Jovanovic <Petar.Jovanovic@imgtec.com>, "pbonzini@redhat.com" <pbonzini@redhat.com>, Miodrag Dinic <Miodrag.Dinic@imgtec.com>, "edgar.iglesias@gmail.com" <edgar.iglesias@gmail.com>, "gxt@mprc.pku.edu.cn" <gxt@mprc.pku.edu.cn>, Leon Alrae <Leon.Alrae@imgtec.com>, "afaerber@suse.de" <afaerber@suse.de>, "aurelien@aurel32.net" <aurelien@aurel32.net>, Richard Henderson <rth@twiddle.net>

On Mon, Apr 04, 2016 at 02:31:47PM +0100, Peter Maydell wrote:
> On 4 April 2016 at 14:21, Aleksandar Markovic
> <Aleksandar.Markovic@imgtec.com> wrote:
> > B. arm - explicitely sets other fields of float_status,
> >       explicit invocation of set_snan_bit_is_one(0) added
> 
> We zero the float_status structs on reset, because they are earlier
> in the CPUARMState structure than the 'features' field (and so the
> memset() in arm_cpu_reset() will clear them). So you don't
> need to explicitly zero a field like this. I expect the other
> architectures are the same.

Even if it is not zeroed on reset, it is zeroed on object_new().
Isn't that enough?

-- 
Eduardo