mbox series

[v3,00/11] current fpu/next queue (tests & build fix)

Message ID 20190122215016.18697-1-alex.bennee@linaro.org
Headers show
Series current fpu/next queue (tests & build fix) | expand

Message

Alex Bennée Jan. 22, 2019, 9:50 p.m. UTC 
Hi,

The last softfloat PR failed on a number of big endian machines. This
has been fixed with a patch to the fp-test platform.h. In the meantime
there have been a few fixes to the check-softfloat rules to improve
re-building and reporting of failing tests.

The following patches need review
  patch 0009/tests Makefile add floating point tests.patch

I'll send out a proper pull request once all the automated testing has
completed. Anyone with access to big endian hardware it would be cool
if you could give the series a spin as well.

Alex Bennée (3):
  tests/Makefile: add floating point tests
  scripts/archive-source: include softfloat tests
  tests/Makfile: add check-softfloat rule

Emilio G. Cota (7):
  fp-bench: fix update_random_ops
  fp-bench: remove wrong exponent raise in fill_random
  berkeley-testfloat-3: pull changes
  tests/fp/Makefile: do not use gcc-only -W flags
  fp-test: fix signature of slow_clear_flags and qemu_clear_flags
  tests/fp/platform.h: include config-host.h
  softfloat: enforce softfloat if the host's FMA is broken

Thomas Huth (1):
  include/fpu/softfloat: Fix compilation with Clang on s390x

 fpu/softfloat.c                |  33 ++++++++
 include/fpu/softfloat-macros.h |   2 +-
 scripts/archive-source.sh      |   2 +-
 tests/Makefile.include         | 137 ++++++++++++++++++++++++++++++++-
 tests/fp/Makefile              |   3 +-
 tests/fp/berkeley-testfloat-3  |   2 +-
 tests/fp/fp-bench.c            |  15 ++--
 tests/fp/fp-test.c             |   4 +-
 tests/fp/platform.h            |   1 +
 9 files changed, 183 insertions(+), 16 deletions(-)

Comments

no-reply@patchew.org Jan. 31, 2019, 5:56 p.m. UTC | #1 
Patchew URL: https://patchew.org/QEMU/20190122215016.18697-1-alex.bennee@linaro.org/



Hi,

This series seems to have some coding style problems. See output below for
more information:

Message-id: 20190122215016.18697-1-alex.bennee@linaro.org
Subject: [Qemu-devel] [PATCH v3 00/11] current fpu/next queue (tests & build fix)
Type: series

=== TEST SCRIPT BEGIN ===
#!/bin/bash
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===

Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
Switched to a new branch 'test'
1c3d170 tests/Makfile: add check-softfloat rule
ee2215b scripts/archive-source: include softfloat tests
d3b2f6b tests/Makefile: add floating point tests
4e35542 include/fpu/softfloat: Fix compilation with Clang on s390x
a31f3e2 softfloat: enforce softfloat if the host's FMA is broken
6ec5d0f tests/fp/platform.h: include config-host.h
2128df3 fp-test: fix signature of slow_clear_flags and qemu_clear_flags
62c6bb9 tests/fp/Makefile: do not use gcc-only -W flags
04ffa7c berkeley-testfloat-3: pull changes
c8d89ed fp-bench: remove wrong exponent raise in fill_random
3d24285 fp-bench: fix update_random_ops

=== OUTPUT BEGIN ===
1/11 Checking commit 3d242858ec62 (fp-bench: fix update_random_ops)
2/11 Checking commit c8d89edb4800 (fp-bench: remove wrong exponent raise in fill_random)
3/11 Checking commit 04ffa7c5628e (berkeley-testfloat-3: pull changes)
4/11 Checking commit 62c6bb96922f (tests/fp/Makefile: do not use gcc-only -W flags)
5/11 Checking commit 2128df3a9377 (fp-test: fix signature of slow_clear_flags and qemu_clear_flags)
6/11 Checking commit 6ec5d0fb8a93 (tests/fp/platform.h: include config-host.h)
7/11 Checking commit a31f3e28d914 (softfloat: enforce softfloat if the host's FMA is broken)
8/11 Checking commit 4e35542f0d62 (include/fpu/softfloat: Fix compilation with Clang on s390x)
9/11 Checking commit d3b2f6b91a6d (tests/Makefile: add floating point tests)
10/11 Checking commit ee2215b0ae09 (scripts/archive-source: include softfloat tests)
ERROR: line over 90 characters
#23: FILE: scripts/archive-source.sh:29:
+submodules="dtc ui/keycodemapdb tests/fp/berkeley-softfloat-3 tests/fp/berkeley-testfloat-3"

total: 1 errors, 0 warnings, 8 lines checked

Patch 10/11 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

11/11 Checking commit 1c3d170d7ae3 (tests/Makfile: add check-softfloat rule)
=== OUTPUT END ===

Test command exited with code: 1


The full log is available at
http://patchew.org/logs/20190122215016.18697-1-alex.bennee@linaro.org/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to patchew-devel@redhat.com