Re: [PULL 0/7] s390x update

2021-05-11 Thread Cornelia Huck
On Tue, 11 May 2021 13:02:08 +0100
Peter Maydell  wrote:

> On Thu, 6 May 2021 at 16:44, Cornelia Huck  wrote:
> >
> > The following changes since commit d45a5270d075ea589f0b0ddcf963a5fea1f500ac:
> >
> >   Merge remote-tracking branch 
> > 'remotes/vivier2/tags/trivial-branch-for-6.1-pull-request' into staging 
> > (2021-05-05 13:52:00 +0100)
> >
> > are available in the Git repository at:
> >
> >   https://gitlab.com/cohuck/qemu.git tags/s390x-20210506
> >
> > for you to fetch changes up to 2337ca0dd1747851cf596e90a2360578e3f9a473:
> >
> >   vfio-ccw: Attempt to clean up all IRQs on error (2021-05-06 13:49:07 
> > +0200)
> >
> > 
> > s390x fixes and cleanups; also related fixes in xtensa and arm
> >
> > 
> >
> > Eric Farman (2):
> >   vfio-ccw: Permit missing IRQs
> >   vfio-ccw: Attempt to clean up all IRQs on error
> >
> > Ilya Leoshkevich (4):
> >   target/s390x: Fix translation exception on illegal instruction
> >   target/arm: Make sure that commpage's tb->size != 0
> >   target/xtensa: Make sure that tb->size != 0
> >   accel/tcg: Assert that tb->size != 0 after translation
> >
> > Philippe Mathieu-Daudé (1):
> >   hw/s390x/ccw: Register qbus type in abstract TYPE_CCW_DEVICE parent  
> 
> This asserts running x86-64 linux-user:
> 
> /home/petmay01/linaro/qemu-for-merges/build/all-linux-static/x86_64-linux-user/qemu-x86_64
> -L ./gnemul/qemu-x86_64 x86_64/ls -l dummyfile
> qemu-x86_64: ../../accel/tcg/translate-all.c:1915: tb_gen_code:
> Assertion `tb->size != 0' failed.
> qemu-x86_64: ../../accel/tcg/cpu-exec.c:766: cpu_exec: Assertion
> `!have_mmap_lock()' failed.
> Makefile:6: recipe for target 'test' failed
> 
> This is for the TARGET_VSYSCALL_PAGE case in i386_tr_translate_insn(),
> which generates a zero-size TB.

Ilya, can you please take a look?




Re: [PULL 0/7] s390x update

2021-05-11 Thread Peter Maydell
On Thu, 6 May 2021 at 16:44, Cornelia Huck  wrote:
>
> The following changes since commit d45a5270d075ea589f0b0ddcf963a5fea1f500ac:
>
>   Merge remote-tracking branch 
> 'remotes/vivier2/tags/trivial-branch-for-6.1-pull-request' into staging 
> (2021-05-05 13:52:00 +0100)
>
> are available in the Git repository at:
>
>   https://gitlab.com/cohuck/qemu.git tags/s390x-20210506
>
> for you to fetch changes up to 2337ca0dd1747851cf596e90a2360578e3f9a473:
>
>   vfio-ccw: Attempt to clean up all IRQs on error (2021-05-06 13:49:07 +0200)
>
> 
> s390x fixes and cleanups; also related fixes in xtensa and arm
>
> 
>
> Eric Farman (2):
>   vfio-ccw: Permit missing IRQs
>   vfio-ccw: Attempt to clean up all IRQs on error
>
> Ilya Leoshkevich (4):
>   target/s390x: Fix translation exception on illegal instruction
>   target/arm: Make sure that commpage's tb->size != 0
>   target/xtensa: Make sure that tb->size != 0
>   accel/tcg: Assert that tb->size != 0 after translation
>
> Philippe Mathieu-Daudé (1):
>   hw/s390x/ccw: Register qbus type in abstract TYPE_CCW_DEVICE parent

This asserts running x86-64 linux-user:

/home/petmay01/linaro/qemu-for-merges/build/all-linux-static/x86_64-linux-user/qemu-x86_64
-L ./gnemul/qemu-x86_64 x86_64/ls -l dummyfile
qemu-x86_64: ../../accel/tcg/translate-all.c:1915: tb_gen_code:
Assertion `tb->size != 0' failed.
qemu-x86_64: ../../accel/tcg/cpu-exec.c:766: cpu_exec: Assertion
`!have_mmap_lock()' failed.
Makefile:6: recipe for target 'test' failed

This is for the TARGET_VSYSCALL_PAGE case in i386_tr_translate_insn(),
which generates a zero-size TB.

thanks
-- PMM



[PULL 0/7] s390x update

2021-05-06 Thread Cornelia Huck
The following changes since commit d45a5270d075ea589f0b0ddcf963a5fea1f500ac:

  Merge remote-tracking branch 
'remotes/vivier2/tags/trivial-branch-for-6.1-pull-request' into staging 
(2021-05-05 13:52:00 +0100)

are available in the Git repository at:

  https://gitlab.com/cohuck/qemu.git tags/s390x-20210506

for you to fetch changes up to 2337ca0dd1747851cf596e90a2360578e3f9a473:

  vfio-ccw: Attempt to clean up all IRQs on error (2021-05-06 13:49:07 +0200)


s390x fixes and cleanups; also related fixes in xtensa and arm



Eric Farman (2):
  vfio-ccw: Permit missing IRQs
  vfio-ccw: Attempt to clean up all IRQs on error

Ilya Leoshkevich (4):
  target/s390x: Fix translation exception on illegal instruction
  target/arm: Make sure that commpage's tb->size != 0
  target/xtensa: Make sure that tb->size != 0
  accel/tcg: Assert that tb->size != 0 after translation

Philippe Mathieu-Daudé (1):
  hw/s390x/ccw: Register qbus type in abstract TYPE_CCW_DEVICE parent

 accel/tcg/translate-all.c |  1 +
 hw/s390x/3270-ccw.c   |  1 -
 hw/s390x/ccw-device.c |  1 +
 hw/s390x/ccw-device.h |  1 +
 hw/s390x/s390-ccw.c   |  2 --
 hw/s390x/virtio-ccw.c |  1 -
 hw/vfio/ccw.c | 18 +++---
 target/arm/translate.c|  2 ++
 target/s390x/translate.c  | 16 +++-
 target/xtensa/translate.c |  3 +++
 10 files changed, 30 insertions(+), 16 deletions(-)

-- 
2.30.2




Re: [PULL 0/7] s390x update

2020-06-19 Thread Peter Maydell
On Thu, 18 Jun 2020 at 16:39, Cornelia Huck  wrote:
>
> The following changes since commit 26bf4a29213b432eb390726c698a1915550a9cf9:
>
>   Merge remote-tracking branch 
> 'remotes/kraxel/tags/usb-20200617-pull-request' into staging (2020-06-17 
> 16:24:24 +0100)
>
> are available in the Git repository at:
>
>   https://github.com/cohuck/qemu tags/s390x-20200618
>
> for you to fetch changes up to 458e056257e67254546e58158f3f74ce040c7ca1:
>
>   docs/s390x: fix vfio-ap device_del description (2020-06-18 12:14:06 +0200)
>
> 
> s390x update:
> - update Linux headers to 5.8-rc1 (for vfio-ccw path handling)
> - vfio-ccw: add support for path handling
> - documentation fix
>


Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/5.1
for any user-visible changes.

-- PMM



Re: [PULL 0/7] s390x update

2020-06-18 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200618153854.271723-1-coh...@redhat.com/



Hi,

This series failed the asan build test. Please find the testing commands and
their output below. If you have Docker installed, you can probably reproduce it
locally.

=== TEST SCRIPT BEGIN ===
#!/bin/bash
export ARCH=x86_64
make docker-image-fedora V=1 NETWORK=1
time make docker-test-debug@fedora TARGET_LIST=x86_64-softmmu J=14 NETWORK=1
=== TEST SCRIPT END ===

  GEN docs/interop/qemu-qmp-ref.7
  CC  qga/commands.o
  CC  qga/guest-agent-command-state.o
/usr/bin/ld: 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o):
 warning: common of `__interception::real_vfork' overridden by definition from 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o)
  CC  qga/main.o
  CC  qga/commands-posix.o
  CC  qga/channel-posix.o
---
  GEN docs/interop/qemu-ga-ref.html
  GEN docs/interop/qemu-ga-ref.txt
  GEN docs/interop/qemu-ga-ref.7
/usr/bin/ld: 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o):
 warning: common of `__interception::real_vfork' overridden by definition from 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o)
  LINKqemu-ga
  LINKqemu-keymap
/usr/bin/ld: 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o):
 warning: common of `__interception::real_vfork' overridden by definition from 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o)
  LINKivshmem-client
  LINKivshmem-server
/usr/bin/ld: 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o):
 warning: common of `__interception::real_vfork' overridden by definition from 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o)
/usr/bin/ld: 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o):
 warning: common of `__interception::real_vfork' overridden by definition from 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o)
/usr/bin/ld: 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o):
 warning: common of `__interception::real_vfork' overridden by definition from 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o)
  LINKqemu-nbd
  LINKqemu-storage-daemon
/usr/bin/ld: 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o):
 warning: common of `__interception::real_vfork' overridden by definition from 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o)
  LINKqemu-img
/usr/bin/ld: 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o):
 warning: common of `__interception::real_vfork' overridden by definition from 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o)
/usr/bin/ld: 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o):
 warning: common of `__interception::real_vfork' overridden by definition from 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o)
  AS  pc-bios/optionrom/multiboot.o
  AS  pc-bios/optionrom/linuxboot.o
  CC  pc-bios/optionrom/linuxboot_dma.o
---
  AS  pc-bios/optionrom/pvh.o
  LINKqemu-edid
  CC  pc-bios/optionrom/pvh_main.o
/usr/bin/ld: 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o):
 warning: common of `__interception::real_vfork' overridden by definition from 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o)
  BUILD   pc-bios/optionrom/multiboot.img
  BUILD   pc-bios/optionrom/linuxboot.img
  BUILD   pc-bios/optionrom/linuxboot_dma.img
  BUILD   pc-bios/optionrom/kvmvapic.img
  BUILD   pc-bios/optionrom/multiboot.raw
  BUILD   pc-bios/optionrom/linuxboot.raw
/usr/bin/ld: 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o):
 warning: common of `__interception::real_vfork' overridden by definition from 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o)
  BUILD   pc-bios/optionrom/linuxboot_dma.raw
  BUILD   pc-bios/optionrom/kvmvapic.raw
  SIGNpc-bios/optionrom/multiboot.bin
---
  BUILD   pc-bios/optionrom/pvh.raw
  SIGNpc-bios/optionrom/pvh.bin
  LINKscsi/qemu-pr-helper
/usr/bin/ld: 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o):
 warning: common of `__interception::real_vfork' overridden by definition from 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o)
/usr/bin/ld: 
/usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o):
 warning: common of 

[PULL 0/7] s390x update

2020-06-18 Thread Cornelia Huck
The following changes since commit 26bf4a29213b432eb390726c698a1915550a9cf9:

  Merge remote-tracking branch 'remotes/kraxel/tags/usb-20200617-pull-request' 
into staging (2020-06-17 16:24:24 +0100)

are available in the Git repository at:

  https://github.com/cohuck/qemu tags/s390x-20200618

for you to fetch changes up to 458e056257e67254546e58158f3f74ce040c7ca1:

  docs/s390x: fix vfio-ap device_del description (2020-06-18 12:14:06 +0200)


s390x update:
- update Linux headers to 5.8-rc1 (for vfio-ccw path handling)
- vfio-ccw: add support for path handling
- documentation fix



Christian Borntraeger (1):
  docs/s390x: fix vfio-ap device_del description

Cornelia Huck (1):
  Linux headers: update

Eric Farman (3):
  vfio-ccw: Refactor cleanup of regions
  vfio-ccw: Refactor ccw irq handler
  s390x/css: Refactor the css_queue_crw() routine

Farhan Ali (2):
  vfio-ccw: Add support for the schib region
  vfio-ccw: Add support for the CRW region and IRQ

 docs/system/s390x/vfio-ap.rst|   8 +-
 hw/s390x/css.c   |  57 +++-
 hw/s390x/s390-ccw.c  |  21 ++
 hw/vfio/ccw.c| 208 ++--
 include/hw/s390x/css.h   |   4 +-
 include/hw/s390x/s390-ccw.h  |   1 +
 include/standard-headers/asm-x86/kvm_para.h  |  17 +-
 include/standard-headers/drm/drm_fourcc.h| 140 +++-
 include/standard-headers/linux/ethtool.h |  16 +-
 include/standard-headers/linux/virtio_ids.h  |   1 +
 include/standard-headers/linux/virtio_mem.h  | 211 
 include/standard-headers/linux/virtio_ring.h |  48 ++-
 linux-headers/asm-arm64/mman.h   |   8 +
 linux-headers/asm-generic/unistd.h   |   4 +-
 linux-headers/asm-mips/unistd_n32.h  |   1 +
 linux-headers/asm-mips/unistd_n64.h  |   1 +
 linux-headers/asm-mips/unistd_o32.h  |   1 +
 linux-headers/asm-powerpc/unistd_32.h|   1 +
 linux-headers/asm-powerpc/unistd_64.h|   1 +
 linux-headers/asm-s390/unistd_32.h   |   1 +
 linux-headers/asm-s390/unistd_64.h   |   1 +
 linux-headers/asm-x86/kvm.h  |  20 +-
 linux-headers/asm-x86/unistd.h   |  11 +-
 linux-headers/asm-x86/unistd_32.h|   1 +
 linux-headers/asm-x86/unistd_64.h|   1 +
 linux-headers/asm-x86/unistd_x32.h   |   1 +
 linux-headers/linux/kvm.h|  18 +-
 linux-headers/linux/psp-sev.h|   2 +
 linux-headers/linux/vfio.h   | 322 +++
 linux-headers/linux/vfio_ccw.h   |  19 ++
 linux-headers/linux/vhost.h  |   4 +
 target/s390x/ioinst.c|   3 +-
 32 files changed, 1075 insertions(+), 78 deletions(-)
 create mode 100644 include/standard-headers/linux/virtio_mem.h

-- 
2.25.4