qemu-commits
[Top][All Lists]
Advanced
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] 276ba1: tests/tcg: add a simple s390x test

From: GitHub
Subject: [Qemu-commits] [qemu/qemu] 276ba1: tests/tcg: add a simple s390x test
Date: Mon, 24 Sep 2018 04:26:40 -0700 
  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 276ba120edf4b3a0422ea51cc7afe1dd1093da65
      
https://github.com/qemu/qemu/commit/276ba120edf4b3a0422ea51cc7afe1dd1093da65
  Author: Pavel Zbitskiy <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M MAINTAINERS
    A tests/tcg/s390x/Makefile.target
    A tests/tcg/s390x/hello-s390x.c

  Log Message:
  -----------
  tests/tcg: add a simple s390x test

Copied from alpha.

Signed-off-by: Pavel Zbitskiy <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Thomas Huth <address@hidden>
Reviewed-by: David Hildenbrand <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: e1db291b9b3b53e9d570a9f93a1caf014a07ab64
      
https://github.com/qemu/qemu/commit/e1db291b9b3b53e9d570a9f93a1caf014a07ab64
  Author: Pavel Zbitskiy <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M target/s390x/insn-data.def
    M target/s390x/translate.c

  Log Message:
  -----------
  target/s390x: add BAL and BALR instructions

These instructions are provided for compatibility purposes and are
used only by old software, in the new code BAS and BASR are preferred.
The difference between the old and new instruction exists only in the
24-bit mode.

In addition, fix BAS polluting high 32 bits of the first operand in
24- and 31-bit addressing modes.

Signed-off-by: Pavel Zbitskiy <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: David Hildenbrand <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: dc95b31dac65adb92256e67a5f0fc88ab37404c2
      
https://github.com/qemu/qemu/commit/dc95b31dac65adb92256e67a5f0fc88ab37404c2
  Author: Pavel Zbitskiy <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M target/s390x/mem_helper.c
    M target/s390x/translate.c
    M tests/tcg/s390x/Makefile.target
    A tests/tcg/s390x/csst.c

  Log Message:
  -----------
  target/s390x: fix CSST decoding and runtime alignment check

CSST is defined as:

    C(0xc802, CSST,    SSF,   CASS, la1, a2, 0, 0, csst, 0)

It means that the first parameter is handled by in1_la1().
in1_la1() fills addr1 field, and not in1.

Furthermore, when extract32() is used for the alignment check, the
third parameter should specify the number of trailing bits that must
be 0. For FC these numbers are:

    FC=0 (word, 4 bytes):        2
    FC=1 (double word, 8 bytes): 3
    FC=2 (quad word, 16 bytes):  4

For SC these numbers correspond to the size:

    SC=0: 0
    SC=1: 1
    SC=2: 2
    SC=3: 3
    SC=4: 4

Signed-off-by: Pavel Zbitskiy <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: David Hildenbrand <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: 478d360cd937afe01a1234044ab04a26b73020be
      
https://github.com/qemu/qemu/commit/478d360cd937afe01a1234044ab04a26b73020be
  Author: Pavel Zbitskiy <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M target/s390x/translate.c
    M tests/tcg/s390x/Makefile.target
    A tests/tcg/s390x/ipm.c

  Log Message:
  -----------
  target/s390x: fix IPM polluting irrelevant bits

Suppose psw.mask=0x0000000080000000, cc=2, r1=0 and we do "ipm 1".
This command must touch only bits 32-39, so the expected output
is r1=0x20000000. However, currently qemu yields r1=0x20008000,
because irrelevant parts of PSW leak into r1 during program mask
transfer.

Signed-off-by: Pavel Zbitskiy <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: David Hildenbrand <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: ad8c851d2e772397e0c35148a16a8fbb559b2a2e
      
https://github.com/qemu/qemu/commit/ad8c851d2e772397e0c35148a16a8fbb559b2a2e
  Author: Pavel Zbitskiy <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M target/s390x/mem_helper.c
    M tests/tcg/s390x/Makefile.target
    A tests/tcg/s390x/exrl-trt.c
    A tests/tcg/s390x/exrl-trtr.c

  Log Message:
  -----------
  target/s390x: add EX support for TRT and TRTR

Improves "b213c9f5: target/s390x: Implement TRTR" by introducing the
intermediate functions, which are compatible with dx_helper type.

Signed-off-by: Pavel Zbitskiy <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: David Hildenbrand <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: 3cea09271b3b3a0c4d0ffa3b56ea671bf75d82c3
      
https://github.com/qemu/qemu/commit/3cea09271b3b3a0c4d0ffa3b56ea671bf75d82c3
  Author: Pavel Zbitskiy <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M target/s390x/mem_helper.c
    M tests/tcg/s390x/Makefile.target
    A tests/tcg/s390x/pack.c

  Log Message:
  -----------
  target/s390x: fix PACK reading 1 byte less and writing 1 byte more

PACK fails on the test from the Principles of Operation: F1F2F3F4
becomes 0000234C instead of 0001234C due to an off-by-one error.
Furthermore, it overwrites one extra byte to the left of F1.

If len_dest is 0, then we only want to flip the 1st byte and never loop
over the rest. Therefore, the loop condition should be > and not >=.

If len_src is 1, then we should flip the 1st byte and pack the 2nd.
Since len_src is already decremented before the loop, the first
condition should be >=, and not >.

Likewise for len_src == 2 and the second condition.

Signed-off-by: Pavel Zbitskiy <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: David Hildenbrand <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: 2d6ff33a03332d2a7dc0ece31a8eb8d49c0194ee
      
https://github.com/qemu/qemu/commit/2d6ff33a03332d2a7dc0ece31a8eb8d49c0194ee
  Author: Thomas Huth <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M hw/s390x/virtio-ccw.c

  Log Message:
  -----------
  hw/s390x/virtio-ccw: Consolidate calls to virtio_ccw_unrealize()

Currently, every virtio-ccw device explicitely sets its unrealize
function to virtio_ccw_unrealize() in its class_init function.
We can simplify this by using a common unrealize function, just like
it is already done for the realize functions.

Signed-off-by: Thomas Huth <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: 901f5f162a32461226fa1bb4be81e71ae7a939a7
      
https://github.com/qemu/qemu/commit/901f5f162a32461226fa1bb4be81e71ae7a939a7
  Author: Thomas Huth <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M MAINTAINERS
    M hw/s390x/Makefile.objs
    A hw/s390x/virtio-ccw-serial.c
    M hw/s390x/virtio-ccw.c

  Log Message:
  -----------
  hw/s390x: Move virtio-ccw-serial code to a separate file

The code should only be enabled if CONFIG_VIRTIO_SERIAL has been set.
This can be done best if the code resides in a separate file.

Signed-off-by: Thomas Huth <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: dcfd82860eb9e63c9e161a9639e04fb10afc1d74
      
https://github.com/qemu/qemu/commit/dcfd82860eb9e63c9e161a9639e04fb10afc1d74
  Author: Thomas Huth <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M hw/s390x/Makefile.objs
    A hw/s390x/virtio-ccw-balloon.c
    M hw/s390x/virtio-ccw.c

  Log Message:
  -----------
  hw/s390x: Move virtio-ccw-balloon code to a separate file

The code should only be enabled if CONFIG_VIRTIO_BALLOON has been
set. This can be done best if the code resides in a separate file.

Signed-off-by: Thomas Huth <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: e594bfedd1a82ca2ec1b0a4c19950ecffa40dde6
      
https://github.com/qemu/qemu/commit/e594bfedd1a82ca2ec1b0a4c19950ecffa40dde6
  Author: Thomas Huth <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M hw/s390x/Makefile.objs
    A hw/s390x/virtio-ccw-scsi.c
    M hw/s390x/virtio-ccw.c

  Log Message:
  -----------
  hw/s390x: Move virtio-ccw-scsi code to a separate file

The code should only be enabled if CONFIG_VIRTIO_SCSI has been set.
This can be done best if the code resides in a separate file.

Signed-off-by: Thomas Huth <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: dcdc7ad3ac891a38643fd08e4e4bc808c50421c7
      
https://github.com/qemu/qemu/commit/dcdc7ad3ac891a38643fd08e4e4bc808c50421c7
  Author: Thomas Huth <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M hw/s390x/Makefile.objs
    A hw/s390x/virtio-ccw-rng.c
    M hw/s390x/virtio-ccw.c

  Log Message:
  -----------
  hw/s390x: Move virtio-ccw-rng code to a separate file

The code should only be enabled if CONFIG_VIRTIO_RNG has been set.
This can be done best if the code resides in a separate file.

Signed-off-by: Thomas Huth <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: 56541db44217df62cbdcfaccb3ca013ef0f3586c
      
https://github.com/qemu/qemu/commit/56541db44217df62cbdcfaccb3ca013ef0f3586c
  Author: Thomas Huth <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M hw/s390x/Makefile.objs
    A hw/s390x/virtio-ccw-9p.c
    M hw/s390x/virtio-ccw.c

  Log Message:
  -----------
  hw/s390x: Move virtio-ccw-9p code to a separate file

The code should only be enabled if CONFIG_VIRTIO_9P and CONFIG_VIRTFS
have been set. This can be done best if the code resides in a separate
file.

Signed-off-by: Thomas Huth <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: a8e1b5ffc7ce390b11232e311e6cfd8c1ed11055
      
https://github.com/qemu/qemu/commit/a8e1b5ffc7ce390b11232e311e6cfd8c1ed11055
  Author: Thomas Huth <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M hw/s390x/Makefile.objs
    A hw/s390x/virtio-ccw-crypto.c
    M hw/s390x/virtio-ccw.c

  Log Message:
  -----------
  hw/s390x: Move virtio-ccw-crypto code to a separate file

The code should only be enabled if CONFIG_VIRTIO_CRYPTO has been set.
This can be done best if the code resides in a separate file.

Signed-off-by: Thomas Huth <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: d8d4d62c38a48cc204b94833a7733c171371d959
      
https://github.com/qemu/qemu/commit/d8d4d62c38a48cc204b94833a7733c171371d959
  Author: Thomas Huth <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M MAINTAINERS
    M hw/s390x/Makefile.objs
    A hw/s390x/vhost-vsock-ccw.c
    M hw/s390x/virtio-ccw.c

  Log Message:
  -----------
  hw/s390x: Move vhost-vsock-ccw code to a separate file

The code should only be enabled if CONFIG_VHOST_VSOCK has been set.
This can be done best if the code resides in a separate file.

Signed-off-by: Thomas Huth <address@hidden>
Message-Id: <address@hidden>
[CH: updated MAINTAINERS]
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: 5c76546878324d7b8aad6c0a8221e2bdde2228a7
      
https://github.com/qemu/qemu/commit/5c76546878324d7b8aad6c0a8221e2bdde2228a7
  Author: Thomas Huth <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M hw/s390x/Makefile.objs
    A hw/s390x/virtio-ccw-gpu.c
    M hw/s390x/virtio-ccw.c

  Log Message:
  -----------
  hw/s390x: Move virtio-ccw-gpu code to a separate file

The code should only be enabled if CONFIG_VIRTIO_GPU has been set. This
can be done best if the code resides in a separate file.

Signed-off-by: Thomas Huth <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: 90b90a0f8fb9f0b6fea08894998700f8e82b8a7a
      
https://github.com/qemu/qemu/commit/90b90a0f8fb9f0b6fea08894998700f8e82b8a7a
  Author: Thomas Huth <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M hw/s390x/Makefile.objs
    A hw/s390x/virtio-ccw-input.c
    M hw/s390x/virtio-ccw.c

  Log Message:
  -----------
  hw/s390x: Move virtio-ccw-input code to a separate file

The code should only be enabled if CONFIG_VIRTIO_INPUT has been set.
This can be done best if the code resides in a separate file.

Signed-off-by: Thomas Huth <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: efbc1783bd82b8062c5958193b35052998de3258
      
https://github.com/qemu/qemu/commit/efbc1783bd82b8062c5958193b35052998de3258
  Author: Thomas Huth <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M hw/s390x/Makefile.objs
    A hw/s390x/virtio-ccw-net.c
    M hw/s390x/virtio-ccw.c

  Log Message:
  -----------
  hw/s390x: Move virtio-ccw-net code to a separate file

The code should only be enabled if CONFIG_VIRTIO_NET has been set.
This can be done best if the code resides in a separate file.

Signed-off-by: Thomas Huth <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: 331cf66ea4b7b657f3709a14b08de1d11624462f
      
https://github.com/qemu/qemu/commit/331cf66ea4b7b657f3709a14b08de1d11624462f
  Author: Thomas Huth <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M hw/s390x/Makefile.objs
    A hw/s390x/virtio-ccw-blk.c
    M hw/s390x/virtio-ccw.c

  Log Message:
  -----------
  hw/s390x: Move virtio-ccw-blk code to a separate file

The code should only be enabled if CONFIG_VIRTIO_BLK has been set.
This can be done best if the code resides in a separate file.

Signed-off-by: Thomas Huth <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: ef17064acc752d47377b1b263ef328e1281ab48e
      
https://github.com/qemu/qemu/commit/ef17064acc752d47377b1b263ef328e1281ab48e
  Author: Pavel Zbitskiy <address@hidden>
  Date:   2018-08-28 (Tue, 28 Aug 2018)

  Changed paths:
    M target/s390x/translate.c

  Log Message:
  -----------
  target/s390x: use regular spaces in translate.c

In a few places translate.c contains non-breaking spaces (0xc2 0xa0)
instead of regular ones (0x20):

  7c 7c c2 a0 63 63
  7c 7c 20    63 63
  |  |        c  c

This confuses some text editors.

Signed-off-by: Pavel Zbitskiy <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: David Hildenbrand <address@hidden>
Reviewed-by: Thomas Huth <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: 2fde22f8ae1a69253dff5c7a73c39d3a72fa94a1
      
https://github.com/qemu/qemu/commit/2fde22f8ae1a69253dff5c7a73c39d3a72fa94a1
  Author: Peter Maydell <address@hidden>
  Date:   2018-09-24 (Mon, 24 Sep 2018)

  Changed paths:
    M MAINTAINERS
    M hw/s390x/Makefile.objs
    A hw/s390x/vhost-vsock-ccw.c
    A hw/s390x/virtio-ccw-9p.c
    A hw/s390x/virtio-ccw-balloon.c
    A hw/s390x/virtio-ccw-blk.c
    A hw/s390x/virtio-ccw-crypto.c
    A hw/s390x/virtio-ccw-gpu.c
    A hw/s390x/virtio-ccw-input.c
    A hw/s390x/virtio-ccw-net.c
    A hw/s390x/virtio-ccw-rng.c
    A hw/s390x/virtio-ccw-scsi.c
    A hw/s390x/virtio-ccw-serial.c
    M hw/s390x/virtio-ccw.c
    M target/s390x/insn-data.def
    M target/s390x/mem_helper.c
    M target/s390x/translate.c
    A tests/tcg/s390x/Makefile.target
    A tests/tcg/s390x/csst.c
    A tests/tcg/s390x/exrl-trt.c
    A tests/tcg/s390x/exrl-trtr.c
    A tests/tcg/s390x/hello-s390x.c
    A tests/tcg/s390x/ipm.c
    A tests/tcg/s390x/pack.c

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20180829' into staging

- various fixes and improvements in the tcg code
- split off the individual virtio-ccw devices into separate files

# gpg: Signature made Wed 29 Aug 2018 10:38:03 BST
# gpg:                using RSA key DECF6B93C6F02FAF
# gpg: Good signature from "Cornelia Huck <address@hidden>"
# gpg:                 aka "Cornelia Huck <address@hidden>"
# gpg:                 aka "Cornelia Huck <address@hidden>"
# gpg:                 aka "Cornelia Huck <address@hidden>"
# gpg:                 aka "Cornelia Huck <address@hidden>"
# Primary key fingerprint: C3D0 D66D C362 4FF6 A8C0  18CE DECF 6B93 C6F0 2FAF

* remotes/cohuck/tags/s390x-20180829:
  target/s390x: use regular spaces in translate.c
  hw/s390x: Move virtio-ccw-blk code to a separate file
  hw/s390x: Move virtio-ccw-net code to a separate file
  hw/s390x: Move virtio-ccw-input code to a separate file
  hw/s390x: Move virtio-ccw-gpu code to a separate file
  hw/s390x: Move vhost-vsock-ccw code to a separate file
  hw/s390x: Move virtio-ccw-crypto code to a separate file
  hw/s390x: Move virtio-ccw-9p code to a separate file
  hw/s390x: Move virtio-ccw-rng code to a separate file
  hw/s390x: Move virtio-ccw-scsi code to a separate file
  hw/s390x: Move virtio-ccw-balloon code to a separate file
  hw/s390x: Move virtio-ccw-serial code to a separate file
  hw/s390x/virtio-ccw: Consolidate calls to virtio_ccw_unrealize()
  target/s390x: fix PACK reading 1 byte less and writing 1 byte more
  target/s390x: add EX support for TRT and TRTR
  target/s390x: fix IPM polluting irrelevant bits
  target/s390x: fix CSST decoding and runtime alignment check
  target/s390x: add BAL and BALR instructions
  tests/tcg: add a simple s390x test

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/850a8242a530...2fde22f8ae1a
      **NOTE:** This service has been marked for deprecation: 
https://developer.github.com/changes/2018-04-25-github-services-deprecation/

      Functionality will be removed from GitHub.com on January 31st, 2019.
reply via email to
[Prev in Thread] Current Thread [Next in Thread]