Re: [Qemu-devel] [PATCH qemu v6] memory/iommu: QOM'fy IOMMU MemoryRegion

On Fri, May 05, 2017 at 08:19:30PM +1000, Alexey Kardashevskiy wrote: > This defines new QOM object - IOMMUMemoryRegion - with MemoryRegion > as a parent. > > This moves IOMMU-related fields from MR to IOMMU MR. However to avoid > dymanic QOM casting in fast path (address_space_translate, etc), >

Re: [Qemu-devel] [RFC PATCH 3/8] memory: rename iommu_notifier_init()

On Mon, May 01, 2017 at 02:53:35PM +1000, David Gibson wrote: > On Thu, Apr 27, 2017 at 05:34:15PM +0800, Peter Xu wrote: > > It's new name is iommu_mr_notifier_init(). Again, literal changes only. > > > > Signed-off-by: Peter Xu > > Again, I think this could be folded with

Re: [Qemu-devel] [RFC PATCH 6/8] memory: introduce AddressSpaceOps

On Mon, May 01, 2017 at 02:58:22PM +1000, David Gibson wrote: > On Thu, Apr 27, 2017 at 05:34:18PM +0800, Peter Xu wrote: > > This is something similar to MemoryRegionOps, it's just for address > > spaces to store arch-specific hooks. > > > > The first hook I would like to introduce is

Re: [Qemu-devel] [PATCH v2 23/24] numa: add '-numa cpu, ...' option for property based node mapping

On Wed, May 03, 2017 at 02:57:17PM +0200, Igor Mammedov wrote: > legacy cpu to node mapping is using cpu index values to map > VCPU to node with help of '-numa node,nodeid=node,cpus=x[-y]' > option. However cpu index is internal concept and QEMU users > have to guess /reimplement qemu's logic/ to

Re: [Qemu-devel] [PATCH v6 3/5] shutdown: Add source information to SHUTDOWN and RESET

On Fri, May 05, 2017 at 02:38:08PM -0500, Eric Blake wrote: > Time to wire up all the call sites that request a shutdown or > reset to use the enum added in the previous patch. > > It would have been less churn to keep the common case with no > arguments as meaning guest-triggered, and only

Re: [Qemu-devel] [PATCH v7 00/13] basic channel IO passthrough infrastructure based on vfio

* Cornelia Huck [2017-05-05 14:22:34 +0200]: > On Fri, 5 May 2017 04:03:39 +0200 > Dong Jia Shi wrote: > > > The patch series introduce a basic channel I/O passthrough > > infrastructure based on vfio. > > - Focus on supporting

Re: [Qemu-devel] [PATCH v7 13/13] MAINTAINERS: Add vfio-ccw maintainer

* Cornelia Huck [2017-05-05 14:20:14 +0200]: > On Fri, 5 May 2017 04:03:52 +0200 > Dong Jia Shi wrote: > > > Add Cornelia Huck as the vfio-ccw maintainer. > > > > Signed-off-by: Dong Jia Shi > > --- > >

Re: [Qemu-devel] [PATCH v7 05/13] s390x/css: realize css_create_sch

* Cornelia Huck [2017-05-05 14:11:53 +0200]: > On Fri, 5 May 2017 04:03:44 +0200 > Dong Jia Shi wrote: > > > The S390 virtual css support already has a mechanism to create a > > virtual subchannel and provide it to the guest. However, to

[Qemu-devel] [PATCH 23/23] MAINTAINERS: self-appoint me as reviewer in build/test automation

Signed-off-by: Philippe Mathieu-Daudé --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 9251f575d8..c7520b76db 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1827,6 +1827,7 @@ Build and test automation -

[Qemu-devel] [PATCH 18/23] shippable: temporary disable mips64el-softmmu build

Signed-off-by: Philippe Mathieu-Daudé --- .shippable.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.shippable.yml b/.shippable.yml index 6601243060..7e714991f9 100644 --- a/.shippable.yml +++ b/.shippable.yml @@ -15,8 +15,9 @@ env:

[Qemu-devel] How do you -sd squashfs.img without truncating?

As far as I can tell "qemu-system-arm -M vexpress-a9" only implements sd card, not any conventional hard drive, and it uses an sdcard block size of 262144 bytes rounded down. This means when I create a squashfs image and feed it in through the sd card, it truncates it. Wouldn't -sd rounding _up_

[Qemu-devel] [PATCH 20/23] shippable: add powerpc target

Signed-off-by: Philippe Mathieu-Daudé --- .shippable.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.shippable.yml b/.shippable.yml index 7e714991f9..3a80165b6a 100644 --- a/.shippable.yml +++ b/.shippable.yml @@ -18,6 +18,8 @@ env: # mips64el-softmmu disabled

[Qemu-devel] [PATCH 22/23] MAINTAINERS: add Shippable automation platform URL

Signed-off-by: Philippe Mathieu-Daudé --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index a1d2b3a4d3..9251f575d8 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1833,6 +1833,7 @@ F: .travis.yml F: .shippable.yml F: tests/docker/

[Qemu-devel] [PATCH 16/23] shippable: do not initialize submodules automatically

do it in the 'ci' target when needed. Signed-off-by: Philippe Mathieu-Daudé --- .shippable.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.shippable.yml b/.shippable.yml index fa2d6a4e9b..b3cbca458c 100644 --- a/.shippable.yml +++ b/.shippable.yml @@ -1,4 +1,6 @@

[Qemu-devel] [PATCH 21/23] docker: fix powerpc debian/stable dependencies problem

Signed-off-by: Philippe Mathieu-Daudé --- tests/docker/dockerfiles/debian-powerpc-cross.docker | 7 +++ 1 file changed, 7 insertions(+) diff --git a/tests/docker/dockerfiles/debian-powerpc-cross.docker b/tests/docker/dockerfiles/debian-powerpc-cross.docker index

[Qemu-devel] [RFC PATCH 13/23] docker: add --include-file argument to 'build' command

Signed-off-by: Philippe Mathieu-Daudé --- tests/docker/Makefile.include | 3 ++- tests/docker/docker.py| 5 + 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include index ceff2c1654..47978fb56c

[Qemu-devel] [PATCH 19/23] docker: add powerpc build target

Signed-off-by: Philippe Mathieu-Daudé --- tests/docker/Makefile.include | 1 + .../docker/dockerfiles/debian-powerpc-cross.docker | 23 ++ 2 files changed, 24 insertions(+) create mode 100644

[Qemu-devel] [PATCH 15/23] docker: fix mipsel debian/stable dependencies problem

Signed-off-by: Philippe Mathieu-Daudé --- tests/docker/dockerfiles/debian-mipsel-cross.docker | 7 +++ 1 file changed, 7 insertions(+) diff --git a/tests/docker/dockerfiles/debian-mipsel-cross.docker b/tests/docker/dockerfiles/debian-mipsel-cross.docker index

[Qemu-devel] [PATCH 05/23] docker: compact debian armhf

Signed-off-by: Philippe Mathieu-Daudé --- tests/docker/dockerfiles/debian-armhf-cross.docker | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/tests/docker/dockerfiles/debian-armhf-cross.docker b/tests/docker/dockerfiles/debian-armhf-cross.docker

[Qemu-devel] [PATCH 11/23] shippable: add mipsel target

Signed-off-by: Philippe Mathieu-Daudé --- .shippable.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.shippable.yml b/.shippable.yml index b661e667b3..fa2d6a4e9b 100644 --- a/.shippable.yml +++ b/.shippable.yml @@ -13,6 +13,8 @@ env:

[Qemu-devel] [PATCH 17/23] shippable: use dtc submodule if distrib packages are too old

shippable output: LINKmips64el-softmmu/qemu-system-mips64el ../hw/core/loader-fit.o: In function `load_fit': /root/src/github.com/philmd/qemu/hw/core/loader-fit.c:278: undefined reference to `fdt_first_subnode' /root/src/github.com/philmd/qemu/hw/core/loader-fit.c:286:

[Qemu-devel] [RFC PATCH 14/23] docker: add 'apt-fake' script which generate fake debian packages

Signed-off-by: Philippe Mathieu-Daudé --- tests/docker/Makefile.include | 2 ++ tests/docker/dockerfiles/debian-apt-fake.sh | 46 + tests/docker/dockerfiles/debian.docker | 2 ++ 3 files changed, 50 insertions(+) create mode

[Qemu-devel] [PATCH 10/23] docker: add mipsel build target

Signed-off-by: Philippe Mathieu-Daudé --- tests/docker/Makefile.include | 1 + .../docker/dockerfiles/debian-mipsel-cross.docker | 23 ++ 2 files changed, 24 insertions(+) create mode 100644

[Qemu-devel] [PATCH 04/23] docker: compact debian base

- install common/basic tools at once - one-line Emdebian setup - use eatmydata and remove apt cache to save space - add bison and flex and git Signed-off-by: Philippe Mathieu-Daudé --- tests/docker/dockerfiles/debian.docker | 22 -- 1 file changed, 12

[Qemu-devel] [PATCH 12/23] docker: let _copy_with_mkdir() sub_path argument be optional

Signed-off-by: Philippe Mathieu-Daudé --- tests/docker/docker.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/docker/docker.py b/tests/docker/docker.py index 8747f6a440..6ddc6e4c2a 100755 --- a/tests/docker/docker.py +++ b/tests/docker/docker.py @@

[Qemu-devel] [PATCH 09/23] docker: add extra libs to s390x target to extend codebase coverage

Signed-off-by: Philippe Mathieu-Daudé --- tests/docker/dockerfiles/debian-s390x-cross.docker | 6 ++ 1 file changed, 6 insertions(+) diff --git a/tests/docker/dockerfiles/debian-s390x-cross.docker b/tests/docker/dockerfiles/debian-s390x-cross.docker index

[Qemu-devel] [PATCH 00/23] docker/shippable: cross-build mipsel and powerpc targets

This patchset add 2 more architectures to the cross-build farm. - Reorganize Dockerfiles to use less layers, resulting in smaller images. This also reduce time of image transfer, for example while using: `docker save qemu:debian | ssh remote docker load` - Install more debian packages so the

[Qemu-devel] [PATCH 08/23] docker: add extra libs to arm64 target to extend codebase coverage

Signed-off-by: Philippe Mathieu-Daudé --- tests/docker/dockerfiles/debian-arm64-cross.docker | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/tests/docker/dockerfiles/debian-arm64-cross.docker b/tests/docker/dockerfiles/debian-arm64-cross.docker index

[Qemu-devel] [PATCH 07/23] docker: add extra libs to armhf target to extend codebase coverage

Signed-off-by: Philippe Mathieu-Daudé --- tests/docker/dockerfiles/debian-armhf-cross.docker | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/tests/docker/dockerfiles/debian-armhf-cross.docker b/tests/docker/dockerfiles/debian-armhf-cross.docker index

[Qemu-devel] [RFC PATCH 03/23] shippable: ignore SSL verification

shippable logs: -- git_sync - ssh-agent bash -c 'ssh-add /tmp/ssh/01_deploy; git clone https://github.com/philmd/qemu.git /root/src/github.com/philmd/qemu' Identity added: /tmp/ssh/01_deploy (rsa w/o comment) Cloning into '/root/src/github.com/philmd/qemu'... fatal: unable to access

[Qemu-devel] [PATCH 06/23] docker: compact debian arm64

Signed-off-by: Philippe Mathieu-Daudé --- tests/docker/dockerfiles/debian-arm64-cross.docker | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/tests/docker/dockerfiles/debian-arm64-cross.docker b/tests/docker/dockerfiles/debian-arm64-cross.docker

[Qemu-devel] [PATCH 02/23] shippable: build using all available cpus

Signed-off-by: Philippe Mathieu-Daudé --- .shippable.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.shippable.yml b/.shippable.yml index 231c29b620..5170486ff9 100644 --- a/.shippable.yml +++ b/.shippable.yml @@ -20,4 +20,4 @@ build: ci: - unset

[Qemu-devel] [PATCH 01/23] shippable: use C locale to simplify console output

remove this noise: perl: warning: Setting locale failed. perl: warning: Please check that your locale settings: LANGUAGE = (unset), LC_ALL = "en_US.UTF-8", LC_CTYPE = "en_US.UTF-8", LANG = "en_US.UTF-8" are supported and installed on your system. Signed-off-by: Philippe

[Qemu-devel] [Bug 1274170] Re: qemu window hides in the background on osx

[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1274170 Title: qemu

[Qemu-devel] format=raw,readonly errors

juser@gator:~/temp$ qemu-system-x86_64 -m 256 -display curses -drive file=disk.cow -drive file=boot.img WARNING: Image format was not specified for 'boot.img' and probing guessed raw. Automatically detecting the format is dangerous for raw images, write operations on block 0 will be

Re: [Qemu-devel] boot order=d bug?

neat. missed that. except it doesn't work. juser@gator:~/temp$ qemu-system-x86_64 -drive file=boot.img,format=raw,bootindex=1 qemu-system-x86_64: -drive file=boot.img,format=raw,bootindex=1: Block format 'raw' does not support the option 'bootindex' juser@gator:~/temp$ qemu-system-x86_64

Re: [Qemu-devel] [RFC PATCH 0/8] Shared Virtual Memory virtualization for VT-d

Hi Liu Yi, I haven't started to read the code yet, however, could you detail more please? It emulates a SVM capable iommu device in a VM? Or It speeds up device's DMA access in a VM? Or it is a new facility introduced for a VM? Could you please add a bit more for its usage? Thanks! On

Re: [Qemu-devel] [PATCH V3 05/10] net/net.c: Add vnet header length to SocketReadState

On 05/03/2017 06:42 PM, Jason Wang wrote: On 2017年05月03日 11:43, Zhang Chen wrote: On 05/02/2017 12:53 PM, Jason Wang wrote: On 2017年04月28日 17:47, Zhang Chen wrote: Address Jason Wang's comments add vnet header length to SocketReadState. Instead of saying this, you can add

Re: [Qemu-devel] [PATCH v7 04/13] s390x/css: realize css_sch_build_schib

* Cornelia Huck [2017-05-05 14:04:20 +0200]: > On Fri, 5 May 2017 04:03:43 +0200 > Dong Jia Shi wrote: > > > From: Xiao Feng Ren > > > > The S390 virtual css support already has a mechanism to build virtual

Re: [Qemu-devel] boot order=d bug?

On 07.05.2017 07:42, Carl Karsten wrote: > In the real world I will have a machine with a sata hd and boot the > installer from a usb stick. I want to test this with 2 disk image files. > > The usb stick will be built like so: >

Re: [Qemu-devel] [virtio-dev] Vhost-pci RFC2.0

On 05/05/2017 05:18 PM, Jason Wang wrote: On 2017年05月05日 14:18, Wei Wang wrote: On 05/05/2017 12:05 PM, Jason Wang wrote: On 2017年04月19日 14:38, Wang, Wei W wrote: Hi, We made some design changes to the original vhost-pci design, and want to open a discussion about the latest design

[Qemu-devel] 撤回: Why we need redirect the access to bar0 through the PCI config space access function

wuzongyong (A) 将撤回邮件“Why we need redirect the access to bar0 through the PCI config space access function”。

Re: [Qemu-devel] [virtio-dev] virtio-net: configurable TX queue size

On 05/07/2017 08:02 PM, Yan Vugenfirer wrote: On 5 May 2017, at 12:20, Jason Wang wrote: On 2017年05月05日 13:53, Wei Wang wrote: On 05/05/2017 10:27 AM, Jason Wang wrote: On 2017年05月04日 18:58, Wang, Wei W wrote: Hi, I want to re-open the discussion left long time ago:

Re: [Qemu-devel] dns server not working in QEMU using usermode networking (SLIRP)

On 06.05.2017 00:38, FONNEMANN Mark wrote: >> Could you run tcpdump inside the guest so we are sure what the nslookup call >> emits? > > Is there another way to determine this info? My guest OS is an embedded > system with BusyBox 1.25 and not much else. FWIW, QEMU also has a built-in

[Qemu-devel] [Bug 1581936] Re: Frozen Windows 7 VMs with VGA CVE-2016-3712 fix (2.6.0 and 2.5.1.1)

** Changed in: qemu (Ubuntu) Importance: Undecided => High -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1581936 Title: Frozen Windows 7 VMs with VGA CVE-2016-3712 fix (2.6.0 and 2.5.1.1)

[Qemu-devel] Floating point unit bugs

I made a diagnostic program for the floating point unit. It will test various PowerPC floating point instructions for compatibility with the PowerPC G3 processor. It was tested on a PowerPC G3 and G5 system. The results of the program in qemu-system-ppc were pretty bad. About every

Re: [Qemu-devel] [PATCH] gdbstub: implement remote debugging protocol escapes for command receive

Updated patch with comments addressed: >From 6bce4e5c87c255f10b22d2bf6fc951dde2bbf457 Mon Sep 17 00:00:00 2001 From: Doug Gale Date: Mon, 1 May 2017 12:22:10 -0400 Subject: [PATCH] gdbstub: implement remote debugging protocol escapes for command receive - decode escape

[Qemu-devel] [PATCH v2 1/3] hw/arm/exynos: Move DRAM initialization next boards

Before QOM-ifying the Exynos4 SoC model, move the DRAM initialization from exynos4210.c to exynos4_boards.c because DRAM is board specific, not SoC. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Philippe Mathieu-Daudé --- hw/arm/exynos4210.c | 20

[Qemu-devel] [PATCH v2 2/3] hw/arm/exynos: Declare local variables in some order

Bring some more readability by declaring local function variables: first initialized ones and then the rest (with reversed-christmas-tree order). Signed-off-by: Krzysztof Kozlowski Reviewed-by: Philippe Mathieu-Daudé --- hw/arm/exynos4210.c | 4 ++-- 1 file

[Qemu-devel] [PATCH v2 3/3] hw/arm/exynos: QOM-ify the SoC

Convert the Exynos4210 SoC code into a QOM model which is a preferred approach instead of directly initializing SoC-related devices from the board file. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Philippe Mathieu-Daudé --- hw/arm/exynos4210.c | 18

[Qemu-devel] [PATCH v2 0/3] hw/arm/exynos: QOM-ify the SoC

Hi, Changes since v1: = 1. s/RAM/DRAM/ in commit msg of first patch (as suggested by Philippe). 2. Add Philippe's reviewed-by. Convert the Exynos4210 SoC driver into QOM model. No external dependencies, rebased on v2.9.0-363-g0de9191deb14. Best regards, Krzysztof Krzysztof

Re: [Qemu-devel] [Qemu-arm] [PATCH 2/3] hw/intc/exynos: Constify array of combiner interrupts

On 05/07/2017 08:35 AM, Krzysztof Kozlowski wrote: The static array of interrupt combiner mappings is not modified so it can be made const for code safeness. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Philippe Mathieu-Daudé ---

Re: [Qemu-devel] [Qemu-arm] [PATCH 1/3] hw/arm/exynos: Use type define instead of hard-coded a9mpcore_priv string

On 05/07/2017 08:35 AM, Krzysztof Kozlowski wrote: Use a define for a9mpcore_priv device type name instead of hard-coded string. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Philippe Mathieu-Daudé --- hw/arm/exynos4210.c | 3 ++- 1 file changed, 2

Re: [Qemu-devel] [PATCH] gdbstub: implement remote debugging protocol escapes for command receive

[Oops, forgot to reply all, resending...] Yes, on second thought, <= 2 is off by one. [0] would be the '$', [1] would be the repeated character, and [2] would be the '*'. And yes, there is a missing s->state = RS_IDLE there. Good catch. I'll post updated patch shortly... On Fri, May 5, 2017 at

[Qemu-devel] boot order=d bug?

In the real world I will have a machine with a sata hd and boot the installer from a usb stick. I want to test this with 2 disk image files. The usb stick will be built like so: https://github.com/CarlFK/video-stack-deploy/blob/usbstick/scripts/mk_usb_installer.sh but this will demo the problem

[Qemu-devel] 'blank' address space for unknown arm peripherals?

Greetings. Its my understanding that for creating machine definitions for qemu one uses what amount to 'drivers' or 'hardware simulations'. I was wondering if there was what amounted to a blank or raw memory area one could use as a placeholder for a peripheral which is currently undocumented and

Re: [Qemu-devel] [virtio-dev] virtio-net: configurable TX queue size

> On 5 May 2017, at 12:20, Jason Wang wrote: > > > > On 2017年05月05日 13:53, Wei Wang wrote: >> On 05/05/2017 10:27 AM, Jason Wang wrote: >>> >>> >>> On 2017年05月04日 18:58, Wang, Wei W wrote: Hi, I want to re-open the discussion left long time ago:

[Qemu-devel] [PATCH 0/3] hw/arm/exynos: Fix poweroff

Hi, Beside two minor cleanups, the patchset implements proper Exynos power off. Still the GIC [1] fix is needed because the kernel will not reach power off handlers without it. [1] http://patchwork.ozlabs.org/patch/738320/ Best regards, Krzysztof Krzysztof Kozlowski (3): hw/arm/exynos: Use

[Qemu-devel] [PATCH 1/3] hw/arm/exynos: Use type define instead of hard-coded a9mpcore_priv string

Use a define for a9mpcore_priv device type name instead of hard-coded string. Signed-off-by: Krzysztof Kozlowski --- hw/arm/exynos4210.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/hw/arm/exynos4210.c b/hw/arm/exynos4210.c index

[Qemu-devel] [PATCH 2/3] hw/intc/exynos: Constify array of combiner interrupts

The static array of interrupt combiner mappings is not modified so it can be made const for code safeness. Signed-off-by: Krzysztof Kozlowski --- hw/intc/exynos4210_gic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/intc/exynos4210_gic.c

[Qemu-devel] [PATCH 3/3] hw/arm/exynos: Add support for system poweroff

On all Exynos-based boards, the system powers down itself by driving PS_HOLD signal low. Handle writing to respective PMU register to fix power off failure: reboot: Power down Unable to poweroff system shutdown: 31 output lines suppressed due to ratelimiting Kernel panic - not

Re: [Qemu-devel] [Qemu-arm] [PATCH v2 1/5] hw/intc/exynos4210_gic: Fix GIC memory mappings for secondary CPU

On Mon, Mar 13, 2017 at 04:32:40PM -0300, Philippe Mathieu-Daudé wrote: > On 03/13/2017 03:04 PM, Krzysztof Kozlowski wrote: > > Recent Linux kernel (tested next-20170224) was complaining about missing > > GIC mask and was unable to bring up secondary CPU: > > > > [0.00] NR_IRQS:16

Re: [Qemu-devel] [PULL 00/23] Trivial patches for 2017-05-07

Hi, This series seems to have some coding style problems. See output below for more information: Message-id: cover.1494140527.git@msgid.tls.msk.ru Subject: [Qemu-devel] [PULL 00/23] Trivial patches for 2017-05-07 Type: series === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n=1 total=$(git

[Qemu-devel] [PULL 16/23] virtfs: allow a device id to be specified in the -virtfs option

From: Chris Webb When using a virtfs root filesystem, the mount_tag needs to be set to /dev/root. This can be done long-hand as -fsdev local,id=root,path=/path/to/rootfs,... -device virtio-9p-pci,fsdev=root,mount_tag=/dev/root but the -virtfs shortcut cannot be used as

[Qemu-devel] [PULL 03/23] doc: fix function spelling

From: Marc-André Lureau Signed-off-by: Marc-André Lureau Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Stefan Hajnoczi Signed-off-by: Michael Tokarev --- include/io/channel.h

[Qemu-devel] [PULL 05/23] tests: Ignore more test executables

From: Eric Blake Ignore test executables when building in-tree: test-arm-mptimer introduced in commit 882fac3 test-crypto-hmac introduced in commit 4fd460b test-aio-multithread introduced in commit 0c330a7 Signed-off-by: Eric Blake Reviewed-by: Laurent

[Qemu-devel] [PULL 20/23] jazz_led: fix bad snprintf

From: Paolo Bonzini Detected by GCC 7's -Wformat-truncation. snprintf writes at most 2 bytes here including the terminating NUL, so the result is truncated. In addition, the newline at the end is pointless. Fix the buffer size and the format string. Signed-off-by: Paolo

[Qemu-devel] [PULL 17/23] scripts/qemu-binfmt-conf.sh: Fix shell portability issue

From: Kamil Rytarowski Appease pkgsrc and use portable shell variable comparison. This switches "==" to "=". It should not be a functional change. Signed-off-by: Kamil Rytarowski Reviewed-by: Peter Maydell Reviewed-by: Laurent Vivier

[Qemu-devel] [PULL 21/23] MAINTAINERS: Update paths for main loop

From: Paolo Bonzini Moved by c2b38b2 ("block: move AioContext, QEMUTimer, main-loop to libqemuutil"), let's update MAINTAINERS too. Reported-by: Laurent Vivier Signed-off-by: Paolo Bonzini Reviewed-by: Alex Bennée

[Qemu-devel] [PULL 18/23] scripts: Switch to more portable Perl shebang

From: Kamil Rytarowski The default NetBSD package manager is pkgsrc and it installs Perl along other third party programs under custom and configurable prefix. The default prefix for binary prebuilt packages is /usr/pkg, and the Perl executable lands in /usr/pkg/bin/perl. This

[Qemu-devel] [PULL 07/23] device_tree: fix compiler warnings (clang 5)

From: Philippe Mathieu-Daudé static code analyzer complain: device_tree.c:155:18: warning: Null pointer passed as an argument to a 'nonnull' parameter while ((de = readdir(d)) != NULL) { ^~ Reported-by: Clang Static Analyzer Signed-off-by:

[Qemu-devel] [PULL 02/23] ppc_booke: drop useless assignment

From: KONRAD Frederic The tb_env variable is set two lines above. So just drop the double assignment. Signed-off-by: KONRAD Frederic Reviewed-by: Laurent Vivier Signed-off-by: Michael Tokarev ---

[Qemu-devel] [PULL 10/23] block: Make 'replication_state' an enum

From: Fam Zheng BDRVReplicationState.replication_state is a name with a bit of duplication, plus it could be an enum like BDRVReplicationState.mode, which is more readable and also more straightforward in a debugger. Rename it, and improve the type while at it. Signed-off-by:

[Qemu-devel] [PULL 04/23] Add 'none' as type for drive's if option

From: Craig Jellick Signed-off-by: Craig Jellick Reviewed-by: Stefan Hajnoczi Signed-off-by: Michael Tokarev --- qemu-options.hx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qemu-options.hx

[Qemu-devel] [PULL 22/23] MAINTAINERS: Update paths for AioContext implementation

From: Paolo Bonzini Moved by c2b38b2 ("block: move AioContext, QEMUTimer, main-loop to libqemuutil") Signed-off-by: Paolo Bonzini Reviewed-by: Laurent Vivier Signed-off-by: Michael Tokarev --- MAINTAINERS | 4

[Qemu-devel] [PULL 06/23] usb-ccid: make ccid_write_data_block() cope with null buffers

From: Philippe Mathieu-Daudé static code analyzer complain: hw/usb/dev-smartcard-reader.c:816:5: warning: Null pointer passed as an argument to a 'nonnull' parameter memcpy(p->abData, data, len); ^~~~ Reported-by: Clang Static Analyzer

[Qemu-devel] [PULL 13/23] qemu-doc: Fix broken URLs of amnhltm.zip and dosidle210.zip

From: Thomas Huth There are some broken URLs in the qemu-doc which reference tools that are not available at their original location anymore. Fortunately, they have been mirrored to archive.org, so point to that location instead. Signed-off-by: Thomas Huth

[Qemu-devel] [PULL 23/23] tests: Remove redundant assignment

From: Fam Zheng Signed-off-by: Fam Zheng Reviewed-by: Eric Blake Signed-off-by: Michael Tokarev --- tests/postcopy-test.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/postcopy-test.c

[Qemu-devel] [PULL 19/23] tests: Ignore another built executable (test-hmp)

From: Eric Blake Commit 78f86a2b7 added a new test, but forgot to exclude the built binary from version control. Signed-off-by: Eric Blake Reviewed-by: Thomas Huth Signed-off-by: Michael Tokarev --- tests/.gitignore |

[Qemu-devel] [PULL 15/23] hw/core/generic-loader: Fix crash when running without CPU

From: Thomas Huth When running QEMU with "-M none -device loader,file=kernel.elf", it currently crashes with a segmentation fault, because the "none"-machine does not have any CPU by default and the generic loader code tries to dereference s->cpu. Fix it by adding an

[Qemu-devel] [PULL 09/23] util: Use g_malloc/g_free in envlist.c

From: Saurav Sachidanand Change malloc/strdup/free to g_malloc/g_strdup/g_free in util/envlist.c. Remove NULL checks for pointers returned from g_malloc and g_strdup as they exit in case of failure. Also, update calls to envlist_create to reflect this. Free array

[Qemu-devel] [PULL 00/23] Trivial patches for 2017-05-07

The following changes since commit 12a95f320a36ef66f724a49bb05e4fb553ac5dbe: Merge remote-tracking branch 'kwolf/tags/for-upstream' into staging (2017-05-04 13:44:32 +0100) are available in the git repository at: git://git.corpit.ru/qemu.git tags/trivial-patches-fetch for you to fetch

[Qemu-devel] [PULL 14/23] virtio-blk: Remove useless condition around g_free()

From: Fam Zheng Laszlo spotted and studied this wasteful "if". He pointed out: The original virtio_blk_free_request needed an "if" as it accesses one field, since 671ec3f05655 ("virtio-blk: Convert VirtIOBlockReq.elem to pointer", 2014-06-11); later on in f897bf751fbd

[Qemu-devel] [PULL 01/23] Remove reduntant qemu: from error functions

From: Ishani Chugh This patch removes redundant "qemu:" from error functions. The link to the bitesized task is: http://wiki.qemu-project.org/Contribute/BiteSizedTasks#Error_checking Signed-off-by: Ishani Chugh Reviewed-by:

[Qemu-devel] [PULL 11/23] channel-file: fix wrong parameter comments

From: "sochin.jiang fix wrong parameter comments in channel-file.h" Signed-off-by: sochin.jiang Reviewed-by: Marc-André Lureau Signed-off-by: Michael Tokarev --- include/io/channel-file.h | 2 +- 1 file

[Qemu-devel] [PULL 12/23] use _Static_assert in QEMU_BUILD_BUG_ON

From: Andreas Grapentin QEMU_BUILD_BUG_ON should use C11's _Static_assert, if the compiler supports it, to provide more readable messages on failure. We check for _Static_assert in configure, and set CONFIG_STATIC_ASSERT accordingly. QEMU_BUILD_BUG_ON invokes

[Qemu-devel] [PULL 08/23] qga: fix compiler warnings (clang 5)

From: Philippe Mathieu-Daudé static code analyzer complain: qga/commands-posix.c:2127:9: warning: Null pointer passed as an argument to a 'nonnull' parameter closedir(dp); ^~~~ Reported-by: Clang Static Analyzer Signed-off-by: Philippe Mathieu-Daudé