[Qemu-devel] [PATCH for 4.2 v3 0/6] target/mips: Misc patches for 4.2

Aleksandar Markovic posted 6 patches 4 years, 8 months ago
Test asan passed
Test docker-clang@ubuntu failed
Test docker-mingw@fedora passed
Test FreeBSD passed
Test s390x failed
Test checkpatch failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1564512798-25527-1-git-send-email-aleksandar.markovic@rt-rk.com
Maintainers: Aurelien Jarno <aurelien@aurel32.net>, Aleksandar Rikalo <arikalo@wavecomp.com>, Aleksandar Markovic <amarkovic@wavecomp.com>
There is a newer version of this series
default-configs/mips-softmmu-common.mak            |   1 +
disas/mips.c                                       |  10 +
hw/mips/cps.c                                      |  28 +-
hw/misc/Makefile.objs                              |   1 +
hw/misc/mips_dspram.c                              | 153 ++++
include/hw/mips/cps.h                              |   2 +
include/hw/misc/mips_dspram.h                      |  46 ++
target/mips/cpu.h                                  |  11 +-
target/mips/helper.c                               |  24 +-
target/mips/helper.h                               |   7 +
target/mips/internal.h                             |   4 +-
target/mips/machine.c                              |   6 +-
target/mips/op_helper.c                            | 181 +++-
target/mips/translate.c                            | 142 +++-
target/mips/translate_init.inc.c                   |   2 +
tests/tcg/mips/include/test_utils_128.h            |  23 +-
.../mips/user/ase/msa/test_msa_compile_32r5eb.sh   | 917 +++++++++++++++++++++
.../mips/user/ase/msa/test_msa_compile_32r5el.sh   | 917 +++++++++++++++++++++
.../mips/user/ase/msa/test_msa_compile_32r6eb.sh   | 643 ---------------
.../mips/user/ase/msa/test_msa_compile_32r6el.sh   | 643 ---------------
tests/tcg/mips/user/ase/msa/test_msa_run_32r5eb.sh | 371 +++++++++
tests/tcg/mips/user/ase/msa/test_msa_run_32r5el.sh | 371 +++++++++
tests/tcg/mips/user/ase/msa/test_msa_run_32r6eb.sh | 371 ---------
tests/tcg/mips/user/ase/msa/test_msa_run_32r6el.sh | 371 ---------
24 files changed, 3182 insertions(+), 2063 deletions(-)
create mode 100644 hw/misc/mips_dspram.c
create mode 100644 include/hw/misc/mips_dspram.h
create mode 100755 tests/tcg/mips/user/ase/msa/test_msa_compile_32r5eb.sh
create mode 100755 tests/tcg/mips/user/ase/msa/test_msa_compile_32r5el.sh
delete mode 100755 tests/tcg/mips/user/ase/msa/test_msa_compile_32r6eb.sh
delete mode 100755 tests/tcg/mips/user/ase/msa/test_msa_compile_32r6el.sh
create mode 100755 tests/tcg/mips/user/ase/msa/test_msa_run_32r5eb.sh
create mode 100755 tests/tcg/mips/user/ase/msa/test_msa_run_32r5el.sh
delete mode 100644 tests/tcg/mips/user/ase/msa/test_msa_run_32r6eb.sh
delete mode 100755 tests/tcg/mips/user/ase/msa/test_msa_run_32r6el.sh
[Qemu-devel] [PATCH for 4.2 v3 0/6] target/mips: Misc patches for 4.2
Posted by Aleksandar Markovic 4 years, 8 months ago
From: Aleksandar Markovic <amarkovic@wavecomp.com>

This series includes misc MIPS patches intended to be integrated after
4.1 release.

v2->v3:

  - corrected the patch on WatchHi to include "mi" field
  - corrected the patch on WatchHi to bump VMStateDescription version

v1->v2:

  - fixed checkpatch warnings
  - added four new patches on various topics

Aleksandar Markovic (2):
  tests/tcg: target/mips: Add optional printing of more detailed failure
    info
  tests/tcg: target/mips: Fix target configurations for MSA tests

Yongbok Kim (4):
  target/mips: Add support for DSPRAM
  target/mips: Amend CP0 WatchHi register implementation
  target/mips: Implement Global Invalidate TLB instruction
  target/mips: Add emulation of CRC32 instructions

 default-configs/mips-softmmu-common.mak            |   1 +
 disas/mips.c                                       |  10 +
 hw/mips/cps.c                                      |  28 +-
 hw/misc/Makefile.objs                              |   1 +
 hw/misc/mips_dspram.c                              | 153 ++++
 include/hw/mips/cps.h                              |   2 +
 include/hw/misc/mips_dspram.h                      |  46 ++
 target/mips/cpu.h                                  |  11 +-
 target/mips/helper.c                               |  24 +-
 target/mips/helper.h                               |   7 +
 target/mips/internal.h                             |   4 +-
 target/mips/machine.c                              |   6 +-
 target/mips/op_helper.c                            | 181 +++-
 target/mips/translate.c                            | 142 +++-
 target/mips/translate_init.inc.c                   |   2 +
 tests/tcg/mips/include/test_utils_128.h            |  23 +-
 .../mips/user/ase/msa/test_msa_compile_32r5eb.sh   | 917 +++++++++++++++++++++
 .../mips/user/ase/msa/test_msa_compile_32r5el.sh   | 917 +++++++++++++++++++++
 .../mips/user/ase/msa/test_msa_compile_32r6eb.sh   | 643 ---------------
 .../mips/user/ase/msa/test_msa_compile_32r6el.sh   | 643 ---------------
 tests/tcg/mips/user/ase/msa/test_msa_run_32r5eb.sh | 371 +++++++++
 tests/tcg/mips/user/ase/msa/test_msa_run_32r5el.sh | 371 +++++++++
 tests/tcg/mips/user/ase/msa/test_msa_run_32r6eb.sh | 371 ---------
 tests/tcg/mips/user/ase/msa/test_msa_run_32r6el.sh | 371 ---------
 24 files changed, 3182 insertions(+), 2063 deletions(-)
 create mode 100644 hw/misc/mips_dspram.c
 create mode 100644 include/hw/misc/mips_dspram.h
 create mode 100755 tests/tcg/mips/user/ase/msa/test_msa_compile_32r5eb.sh
 create mode 100755 tests/tcg/mips/user/ase/msa/test_msa_compile_32r5el.sh
 delete mode 100755 tests/tcg/mips/user/ase/msa/test_msa_compile_32r6eb.sh
 delete mode 100755 tests/tcg/mips/user/ase/msa/test_msa_compile_32r6el.sh
 create mode 100755 tests/tcg/mips/user/ase/msa/test_msa_run_32r5eb.sh
 create mode 100755 tests/tcg/mips/user/ase/msa/test_msa_run_32r5el.sh
 delete mode 100644 tests/tcg/mips/user/ase/msa/test_msa_run_32r6eb.sh
 delete mode 100755 tests/tcg/mips/user/ase/msa/test_msa_run_32r6el.sh

-- 
2.7.4


Re: [Qemu-devel] [PATCH for 4.2 v3 0/6] target/mips: Misc patches for 4.2
Posted by no-reply@patchew.org 4 years, 8 months ago
Patchew URL: https://patchew.org/QEMU/1564512798-25527-1-git-send-email-aleksandar.markovic@rt-rk.com/



Hi,

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

Type: series
Subject: [Qemu-devel] [PATCH for 4.2 v3 0/6] target/mips: Misc patches for 4.2
Message-id: 1564512798-25527-1-git-send-email-aleksandar.markovic@rt-rk.com

=== TEST SCRIPT BEGIN ===
#!/bin/bash
git rev-parse base > /dev/null || exit 0
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
From https://github.com/patchew-project/qemu
 * [new tag]         patchew/1564512798-25527-1-git-send-email-aleksandar.markovic@rt-rk.com -> patchew/1564512798-25527-1-git-send-email-aleksandar.markovic@rt-rk.com
 - [tag update]      patchew/20190730172508.19911-1-mreitz@redhat.com -> patchew/20190730172508.19911-1-mreitz@redhat.com
Submodule 'capstone' (https://git.qemu.org/git/capstone.git) registered for path 'capstone'
Submodule 'dtc' (https://git.qemu.org/git/dtc.git) registered for path 'dtc'
Submodule 'roms/QemuMacDrivers' (https://git.qemu.org/git/QemuMacDrivers.git) registered for path 'roms/QemuMacDrivers'
Submodule 'roms/SLOF' (https://git.qemu.org/git/SLOF.git) registered for path 'roms/SLOF'
Submodule 'roms/edk2' (https://git.qemu.org/git/edk2.git) registered for path 'roms/edk2'
Submodule 'roms/ipxe' (https://git.qemu.org/git/ipxe.git) registered for path 'roms/ipxe'
Submodule 'roms/openbios' (https://git.qemu.org/git/openbios.git) registered for path 'roms/openbios'
Submodule 'roms/openhackware' (https://git.qemu.org/git/openhackware.git) registered for path 'roms/openhackware'
Submodule 'roms/opensbi' (https://git.qemu.org/git/opensbi.git) registered for path 'roms/opensbi'
Submodule 'roms/qemu-palcode' (https://git.qemu.org/git/qemu-palcode.git) registered for path 'roms/qemu-palcode'
Submodule 'roms/seabios' (https://git.qemu.org/git/seabios.git/) registered for path 'roms/seabios'
Submodule 'roms/seabios-hppa' (https://git.qemu.org/git/seabios-hppa.git) registered for path 'roms/seabios-hppa'
Submodule 'roms/sgabios' (https://git.qemu.org/git/sgabios.git) registered for path 'roms/sgabios'
Submodule 'roms/skiboot' (https://git.qemu.org/git/skiboot.git) registered for path 'roms/skiboot'
Submodule 'roms/u-boot' (https://git.qemu.org/git/u-boot.git) registered for path 'roms/u-boot'
Submodule 'roms/u-boot-sam460ex' (https://git.qemu.org/git/u-boot-sam460ex.git) registered for path 'roms/u-boot-sam460ex'
Submodule 'slirp' (https://git.qemu.org/git/libslirp.git) registered for path 'slirp'
Submodule 'tests/fp/berkeley-softfloat-3' (https://git.qemu.org/git/berkeley-softfloat-3.git) registered for path 'tests/fp/berkeley-softfloat-3'
Submodule 'tests/fp/berkeley-testfloat-3' (https://git.qemu.org/git/berkeley-testfloat-3.git) registered for path 'tests/fp/berkeley-testfloat-3'
Submodule 'ui/keycodemapdb' (https://git.qemu.org/git/keycodemapdb.git) registered for path 'ui/keycodemapdb'
Cloning into 'capstone'...
Submodule path 'capstone': checked out '22ead3e0bfdb87516656453336160e0a37b066bf'
Cloning into 'dtc'...
Submodule path 'dtc': checked out '88f18909db731a627456f26d779445f84e449536'
Cloning into 'roms/QemuMacDrivers'...
Submodule path 'roms/QemuMacDrivers': checked out '90c488d5f4a407342247b9ea869df1c2d9c8e266'
Cloning into 'roms/SLOF'...
Submodule path 'roms/SLOF': checked out 'ba1ab360eebe6338bb8d7d83a9220ccf7e213af3'
Cloning into 'roms/edk2'...
Submodule path 'roms/edk2': checked out '20d2e5a125e34fc8501026613a71549b2a1a3e54'
Submodule 'SoftFloat' (https://github.com/ucb-bar/berkeley-softfloat-3.git) registered for path 'ArmPkg/Library/ArmSoftFloatLib/berkeley-softfloat-3'
Submodule 'CryptoPkg/Library/OpensslLib/openssl' (https://github.com/openssl/openssl) registered for path 'CryptoPkg/Library/OpensslLib/openssl'
Cloning into 'ArmPkg/Library/ArmSoftFloatLib/berkeley-softfloat-3'...
Submodule path 'roms/edk2/ArmPkg/Library/ArmSoftFloatLib/berkeley-softfloat-3': checked out 'b64af41c3276f97f0e181920400ee056b9c88037'
Cloning into 'CryptoPkg/Library/OpensslLib/openssl'...
Submodule path 'roms/edk2/CryptoPkg/Library/OpensslLib/openssl': checked out '50eaac9f3337667259de725451f201e784599687'
Submodule 'boringssl' (https://boringssl.googlesource.com/boringssl) registered for path 'boringssl'
Submodule 'krb5' (https://github.com/krb5/krb5) registered for path 'krb5'
Submodule 'pyca.cryptography' (https://github.com/pyca/cryptography.git) registered for path 'pyca-cryptography'
Cloning into 'boringssl'...
Submodule path 'roms/edk2/CryptoPkg/Library/OpensslLib/openssl/boringssl': checked out '2070f8ad9151dc8f3a73bffaa146b5e6937a583f'
Cloning into 'krb5'...
Submodule path 'roms/edk2/CryptoPkg/Library/OpensslLib/openssl/krb5': checked out 'b9ad6c49505c96a088326b62a52568e3484f2168'
Cloning into 'pyca-cryptography'...
Submodule path 'roms/edk2/CryptoPkg/Library/OpensslLib/openssl/pyca-cryptography': checked out '09403100de2f6f1cdd0d484dcb8e620f1c335c8f'
Cloning into 'roms/ipxe'...
Submodule path 'roms/ipxe': checked out 'de4565cbe76ea9f7913a01f331be3ee901bb6e17'
Cloning into 'roms/openbios'...
Submodule path 'roms/openbios': checked out 'c79e0ecb84f4f1ee3f73f521622e264edd1bf174'
Cloning into 'roms/openhackware'...
Submodule path 'roms/openhackware': checked out 'c559da7c8eec5e45ef1f67978827af6f0b9546f5'
Cloning into 'roms/opensbi'...
Submodule path 'roms/opensbi': checked out 'ce228ee0919deb9957192d723eecc8aaae2697c6'
Cloning into 'roms/qemu-palcode'...
Submodule path 'roms/qemu-palcode': checked out 'bf0e13698872450164fa7040da36a95d2d4b326f'
Cloning into 'roms/seabios'...
Submodule path 'roms/seabios': checked out 'a5cab58e9a3fb6e168aba919c5669bea406573b4'
Cloning into 'roms/seabios-hppa'...
Submodule path 'roms/seabios-hppa': checked out '0f4fe84658165e96ce35870fd19fc634e182e77b'
Cloning into 'roms/sgabios'...
Submodule path 'roms/sgabios': checked out 'cbaee52287e5f32373181cff50a00b6c4ac9015a'
Cloning into 'roms/skiboot'...
Submodule path 'roms/skiboot': checked out '261ca8e779e5138869a45f174caa49be6a274501'
Cloning into 'roms/u-boot'...
Submodule path 'roms/u-boot': checked out 'd3689267f92c5956e09cc7d1baa4700141662bff'
Cloning into 'roms/u-boot-sam460ex'...
Submodule path 'roms/u-boot-sam460ex': checked out '60b3916f33e617a815973c5a6df77055b2e3a588'
Cloning into 'slirp'...
Submodule path 'slirp': checked out 'f0da6726207b740f6101028b2992f918477a4b08'
Cloning into 'tests/fp/berkeley-softfloat-3'...
Submodule path 'tests/fp/berkeley-softfloat-3': checked out 'b64af41c3276f97f0e181920400ee056b9c88037'
Cloning into 'tests/fp/berkeley-testfloat-3'...
Submodule path 'tests/fp/berkeley-testfloat-3': checked out '5a59dcec19327396a011a17fd924aed4fec416b3'
Cloning into 'ui/keycodemapdb'...
Submodule path 'ui/keycodemapdb': checked out '6b3d716e2b6472eb7189d3220552280ef3d832ce'
Switched to a new branch 'test'
c9c4648 tests/tcg: target/mips: Fix target configurations for MSA tests
9edc0c6 tests/tcg: target/mips: Add optional printing of more detailed failure info
9e4deee target/mips: Add emulation of CRC32 instructions
5de6433 target/mips: Implement Global Invalidate TLB instruction
435b831 target/mips: Amend CP0 WatchHi register implementation
278fdc5 target/mips: Add support for DSPRAM

=== OUTPUT BEGIN ===
1/6 Checking commit 278fdc584af8 (target/mips: Add support for DSPRAM)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#94: 
new file mode 100644

total: 0 errors, 1 warnings, 371 lines checked

Patch 1/6 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
2/6 Checking commit 435b83135f13 (target/mips: Amend CP0 WatchHi register implementation)
3/6 Checking commit 5de6433c9fa0 (target/mips: Implement Global Invalidate TLB instruction)
ERROR: spaces required around that '<<' (ctx:ExO)
#371: FILE: target/mips/translate.c:2551:
+<<<<<<< HEAD
 ^

ERROR: spaces required around that '<<' (ctx:OxO)
#371: FILE: target/mips/translate.c:2551:
+<<<<<<< HEAD
   ^

ERROR: spaces required around that '<<' (ctx:OxO)
#371: FILE: target/mips/translate.c:2551:
+<<<<<<< HEAD
     ^

ERROR: spaces required around that '<' (ctx:OxW)
#371: FILE: target/mips/translate.c:2551:
+<<<<<<< HEAD
       ^

ERROR: spaces required around that '==' (ctx:ExO)
#372: FILE: target/mips/translate.c:2552:
+=======
 ^

ERROR: spaces required around that '==' (ctx:OxO)
#372: FILE: target/mips/translate.c:2552:
+=======
   ^

ERROR: spaces required around that '==' (ctx:OxO)
#372: FILE: target/mips/translate.c:2552:
+=======
     ^

ERROR: spaces required around that '=' (ctx:OxE)
#372: FILE: target/mips/translate.c:2552:
+=======
       ^

ERROR: spaces required around that '>>' (ctx:ExO)
#374: FILE: target/mips/translate.c:2554:
+>>>>>>> 4c5da27... target/mips: Implement Global Invalidate TLB instruction
 ^

ERROR: spaces required around that '>>' (ctx:OxO)
#374: FILE: target/mips/translate.c:2554:
+>>>>>>> 4c5da27... target/mips: Implement Global Invalidate TLB instruction
   ^

ERROR: spaces required around that '>>' (ctx:OxO)
#374: FILE: target/mips/translate.c:2554:
+>>>>>>> 4c5da27... target/mips: Implement Global Invalidate TLB instruction
     ^

ERROR: spaces required around that '>' (ctx:OxW)
#374: FILE: target/mips/translate.c:2554:
+>>>>>>> 4c5da27... target/mips: Implement Global Invalidate TLB instruction
       ^

ERROR: spaces required around that '/' (ctx:VxV)
#374: FILE: target/mips/translate.c:2554:
+>>>>>>> 4c5da27... target/mips: Implement Global Invalidate TLB instruction
                          ^

ERROR: spaces required around that ':' (ctx:VxW)
#374: FILE: target/mips/translate.c:2554:
+>>>>>>> 4c5da27... target/mips: Implement Global Invalidate TLB instruction
                               ^

total: 14 errors, 0 warnings, 405 lines checked

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

4/6 Checking commit 9e4deee40d98 (target/mips: Add emulation of CRC32 instructions)
5/6 Checking commit 9edc0c6490a4 (tests/tcg: target/mips: Add optional printing of more detailed failure info)
6/6 Checking commit c9c464880bf8 (tests/tcg: target/mips: Fix target configurations for MSA tests)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#18: 
new file mode 100755

total: 0 errors, 1 warnings, 2576 lines checked

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

Test command exited with code: 1


The full log is available at
http://patchew.org/logs/1564512798-25527-1-git-send-email-aleksandar.markovic@rt-rk.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
Re: [Qemu-devel] [PATCH for 4.2 v3 0/6] target/mips: Misc patches for 4.2
Posted by no-reply@patchew.org 4 years, 8 months ago
Patchew URL: https://patchew.org/QEMU/1564512798-25527-1-git-send-email-aleksandar.markovic@rt-rk.com/



Hi,

This series failed build test on s390x host. Please find the details below.

=== TEST SCRIPT BEGIN ===
#!/bin/bash
# Testing script will be invoked under the git checkout with
# HEAD pointing to a commit that has the patches applied on top of "base"
# branch
set -e

echo
echo "=== ENV ==="
env

echo
echo "=== PACKAGES ==="
rpm -qa

echo
echo "=== UNAME ==="
uname -a

CC=$HOME/bin/cc
INSTALL=$PWD/install
BUILD=$PWD/build
mkdir -p $BUILD $INSTALL
SRC=$PWD
cd $BUILD
$SRC/configure --cc=$CC --prefix=$INSTALL
make -j4
# XXX: we need reliable clean up
# make check -j4 V=1
make install
=== TEST SCRIPT END ===




The full log is available at
http://patchew.org/logs/1564512798-25527-1-git-send-email-aleksandar.markovic@rt-rk.com/testing.s390x/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com