From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1hONdi-00067q-MT for mharc-qemu-trivial@gnu.org; Wed, 08 May 2019 10:33:46 -0400 Received: from eggs.gnu.org ([209.51.188.92]:54243) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1hONdg-00066C-IP for qemu-trivial@nongnu.org; Wed, 08 May 2019 10:33:45 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hONdf-0004Tr-DQ for qemu-trivial@nongnu.org; Wed, 08 May 2019 10:33:44 -0400 Received: from mx1.redhat.com ([209.132.183.28]:63006) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hONdc-0004Sj-OL; Wed, 08 May 2019 10:33:40 -0400 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 26C2B11DBA8; Wed, 8 May 2019 14:33:39 +0000 (UTC) Received: from blackfin.pond.sub.org (ovpn-116-28.ams2.redhat.com [10.36.116.28]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 5684E1001DFA; Wed, 8 May 2019 14:33:35 +0000 (UTC) Received: by blackfin.pond.sub.org (Postfix, from userid 1000) id C0A641132B35; Wed, 8 May 2019 16:33:33 +0200 (CEST) From: Markus Armbruster To: Alex =?utf-8?Q?Benn=C3=A9e?= Cc: Peter Maydell , Laurent Vivier , Thomas Huth , Eduardo Habkost , Viktor Prutyanov , QEMU Trivial , Michael Tokarev , Jason Wang , Mark Cave-Ayland , Laurent Vivier , Fabien Chouteau , QEMU Developers , Gerd Hoffmann , Aurelien Jarno , Paolo Bonzini , Michael Roth , Andreas =?utf-8?Q?F=C3=A4rber?= , Artyom Tarasenko , Richard Henderson References: <20190502185835.15185-1-laurent@vivier.eu> <87sgtv4wjo.fsf@zen.linaroharston> Date: Wed, 08 May 2019 16:33:33 +0200 In-Reply-To: <87sgtv4wjo.fsf@zen.linaroharston> ("Alex =?utf-8?Q?Benn?= =?utf-8?Q?=C3=A9e=22's?= message of "Fri, 03 May 2019 12:17:15 +0100") Message-ID: <87imulaude.fsf@dusky.pond.sub.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.38]); Wed, 08 May 2019 14:33:40 +0000 (UTC) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: Re: [Qemu-trivial] [Qemu-devel] [PULL 00/13] Trivial branch patches X-BeenThere: qemu-trivial@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 08 May 2019 14:33:45 -0000 Alex Benn=C3=A9e writes: > Peter Maydell writes: > >> On Thu, 2 May 2019 at 19:58, Laurent Vivier wrote: >>> >>> The following changes since commit 8482ff2eb3bb95020eb2f370a9b3ea26511e= 41df: >>> >>> Merge remote-tracking branch 'remotes/jnsnow/tags/bitmaps-pull-reques= t' into staging (2019-05-02 12:04:51 +0100) >>> >>> are available in the Git repository at: >>> >>> git://github.com/vivier/qemu.git tags/trivial-branch-pull-request >>> >>> for you to fetch changes up to f7c03a68b814254df414977ff299468fbf0fa1c0: >>> >>> sockets: avoid string truncation warnings when copying UNIX path (201= 9-05-02 20:50:47 +0200) >>> >>> ---------------------------------------------------------------- >>> Pull request trivial branch 2019-05-02 >>> >>> ---------------------------------------------------------------- >> >> Markus's "Clean up includes" patch breaks 'make check' on >> big-endian hosts: >> cd /home/pm215/qemu/build/all/tests/fp && ./fp-test -s -l 1 -r all i32_= to_f16 i >> 64_to_f16 i32_to_f32 i64_to_f32 i32_to_f64 i64_to_f64 i32_to_f128 i64_to= _f128 > >> int-to-float.out 2>&1 || (cat int-to-float.out && exit 1;) >>>> Testing i32_to_f16, rounding near_even >> ^M372 tests total. >> ^M372 tests performed. >> In 372 tests, no errors found in i32_to_f16, rounding near_even. >>>> Testing i32_to_f16, rounding minMag >> [...] >> ^MErrors found in i32_to_f128: >> 1006FFFF =3D> +0000.000000000000401B006FFFF00000 ..... >> expected +401B.006FFFF000000000000000000000 ..... >> ^M0001DDEB =3D> +0000.000000000000400FDDEB00000000 ..... >> expected +400F.DDEB000000000000000000000000 ..... >> ^MFFF6FFFC =3D> +0000.000000000000C012200080000000 ..... >> expected -4012.2000800000000000000000000000 ..... >> ^MFFFFFFF8 =3D> +0000.000000000000C002000000000000 ..... >> expected -4002.0000000000000000000000000000 ..... >> [etc] > > Considering the header clean-ups moved bswap related stuff and the > patterns look incorrectly swapped something has gone fishy. > > If it's just stuff touching f128 then we have some magic in > softfloat-types: > > typedef struct { > #ifdef HOST_WORDS_BIGENDIAN > uint64_t high, low; > #else > uint64_t low, high; > #endif > } float128; > > but I would have though HOST_WORDS_BIGENDIAN is in the config. I shall > have a poke once I get onto the s390 machine. Any luck? In my own poking, I stumbled over #ifndef HOST_WORDS_BIGENDIAN #define LITTLEENDIAN 1 /* otherwise do not define it */ #endif in platform.h. Is LITTLEENDIAN unused, or am I confused? [...] From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-10.9 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PULL_REQUEST,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6B1E6C04A6B for ; Wed, 8 May 2019 14:35:08 +0000 (UTC) Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 3DFE021655 for ; Wed, 8 May 2019 14:35:08 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 3DFE021655 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=redhat.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Received: from localhost ([127.0.0.1]:38237 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1hONf1-0006hR-IC for qemu-devel@archiver.kernel.org; Wed, 08 May 2019 10:35:07 -0400 Received: from eggs.gnu.org ([209.51.188.92]:54231) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1hONde-00064u-6F for qemu-devel@nongnu.org; Wed, 08 May 2019 10:33:43 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hONdc-0004T9-Vn for qemu-devel@nongnu.org; Wed, 08 May 2019 10:33:42 -0400 Received: from mx1.redhat.com ([209.132.183.28]:63006) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hONdc-0004Sj-OL; Wed, 08 May 2019 10:33:40 -0400 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 26C2B11DBA8; Wed, 8 May 2019 14:33:39 +0000 (UTC) Received: from blackfin.pond.sub.org (ovpn-116-28.ams2.redhat.com [10.36.116.28]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 5684E1001DFA; Wed, 8 May 2019 14:33:35 +0000 (UTC) Received: by blackfin.pond.sub.org (Postfix, from userid 1000) id C0A641132B35; Wed, 8 May 2019 16:33:33 +0200 (CEST) From: Markus Armbruster To: Alex =?utf-8?Q?Benn=C3=A9e?= References: <20190502185835.15185-1-laurent@vivier.eu> <87sgtv4wjo.fsf@zen.linaroharston> Date: Wed, 08 May 2019 16:33:33 +0200 In-Reply-To: <87sgtv4wjo.fsf@zen.linaroharston> ("Alex =?utf-8?Q?Benn?= =?utf-8?Q?=C3=A9e=22's?= message of "Fri, 03 May 2019 12:17:15 +0100") Message-ID: <87imulaude.fsf@dusky.pond.sub.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.38]); Wed, 08 May 2019 14:33:40 +0000 (UTC) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: Re: [Qemu-devel] [PULL 00/13] Trivial branch patches X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Laurent Vivier , Peter Maydell , Thomas Huth , Eduardo Habkost , Viktor Prutyanov , QEMU Trivial , Mark Cave-Ayland , Jason Wang , Michael Tokarev , Laurent Vivier , Fabien Chouteau , QEMU Developers , Gerd Hoffmann , Andreas =?utf-8?Q?F=C3=A4rber?= , Paolo Bonzini , Richard Henderson , Michael Roth , Aurelien Jarno , Artyom Tarasenko Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: "Qemu-devel" Alex Benn=C3=A9e writes: > Peter Maydell writes: > >> On Thu, 2 May 2019 at 19:58, Laurent Vivier wrote: >>> >>> The following changes since commit 8482ff2eb3bb95020eb2f370a9b3ea26511e= 41df: >>> >>> Merge remote-tracking branch 'remotes/jnsnow/tags/bitmaps-pull-reques= t' into staging (2019-05-02 12:04:51 +0100) >>> >>> are available in the Git repository at: >>> >>> git://github.com/vivier/qemu.git tags/trivial-branch-pull-request >>> >>> for you to fetch changes up to f7c03a68b814254df414977ff299468fbf0fa1c0: >>> >>> sockets: avoid string truncation warnings when copying UNIX path (201= 9-05-02 20:50:47 +0200) >>> >>> ---------------------------------------------------------------- >>> Pull request trivial branch 2019-05-02 >>> >>> ---------------------------------------------------------------- >> >> Markus's "Clean up includes" patch breaks 'make check' on >> big-endian hosts: >> cd /home/pm215/qemu/build/all/tests/fp && ./fp-test -s -l 1 -r all i32_= to_f16 i >> 64_to_f16 i32_to_f32 i64_to_f32 i32_to_f64 i64_to_f64 i32_to_f128 i64_to= _f128 > >> int-to-float.out 2>&1 || (cat int-to-float.out && exit 1;) >>>> Testing i32_to_f16, rounding near_even >> ^M372 tests total. >> ^M372 tests performed. >> In 372 tests, no errors found in i32_to_f16, rounding near_even. >>>> Testing i32_to_f16, rounding minMag >> [...] >> ^MErrors found in i32_to_f128: >> 1006FFFF =3D> +0000.000000000000401B006FFFF00000 ..... >> expected +401B.006FFFF000000000000000000000 ..... >> ^M0001DDEB =3D> +0000.000000000000400FDDEB00000000 ..... >> expected +400F.DDEB000000000000000000000000 ..... >> ^MFFF6FFFC =3D> +0000.000000000000C012200080000000 ..... >> expected -4012.2000800000000000000000000000 ..... >> ^MFFFFFFF8 =3D> +0000.000000000000C002000000000000 ..... >> expected -4002.0000000000000000000000000000 ..... >> [etc] > > Considering the header clean-ups moved bswap related stuff and the > patterns look incorrectly swapped something has gone fishy. > > If it's just stuff touching f128 then we have some magic in > softfloat-types: > > typedef struct { > #ifdef HOST_WORDS_BIGENDIAN > uint64_t high, low; > #else > uint64_t low, high; > #endif > } float128; > > but I would have though HOST_WORDS_BIGENDIAN is in the config. I shall > have a poke once I get onto the s390 machine. Any luck? In my own poking, I stumbled over #ifndef HOST_WORDS_BIGENDIAN #define LITTLEENDIAN 1 /* otherwise do not define it */ #endif in platform.h. Is LITTLEENDIAN unused, or am I confused? [...]