[Qemu-devel] [PATCH] configure: fix qemu-ga missing '.exe' extension on windows

Reported-by: Sameeh Jubran <sjub...@redhat.com> Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- original report from Sameeh Jubran: http://lists.nongnu.org/archive/html/qemu-devel/2017-07/msg00906.html Makefile | 2 +- configure | 2 +- 2 files changed, 2 insertions(+),

Re: [Qemu-devel] [PATCH for 2.10 07/35] qcow2: fix null pointer dereference

Hi Eric, On 07/24/2017 03:46 PM, Eric Blake wrote: On 07/24/2017 01:27 PM, Philippe Mathieu-Daudé wrote: If find_bitmap_by_name() fails we have bm=NULL and go to the 'fail' label, then call bitmap_free(bm) which does g_free(bm->name) with bm=NULL... Clang's scan-build-5.0 output: block/qc

Re: [Qemu-devel] [RFC PATCH for 2.10 1/3] docker: ensure NOUSER for travis images

do you think? Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org> Tested-by: Philippe Mathieu-Daudé <f4...@amsat.org> # Expand all the pre-requistes for each docker image and test combination $(foreach i,$(DOCKER_IMAGES), \

Re: [Qemu-devel] [PATCH for 2.10 11/35] i2c/exynos4210: correctly check i2c_recv() return value

On 07/24/2017 06:13 PM, Peter Maydell wrote: On 24 July 2017 at 19:27, Philippe Mathieu-Daudé <f4...@amsat.org> wrote: i2c_recv() returns -1 on error, if the I2CCON_ACK_GEN bit was not set this code was setting i2cds = -1. i2c/exynos4210_i2c.c:117:20: warning: Loss of sign in im

Re: [Qemu-devel] [PATCH for 2.10 27/35] syscall: fix dereference of undefined pointer

On 07/24/2017 04:33 PM, Laurent Vivier wrote: Le 24/07/2017 à 20:27, Philippe Mathieu-Daudé a écrit : Clang's scan-build-5.0 reports: linux-user/syscall.c:5581:9: warning: Dereference of undefined pointer value if (*host_rt_dev_ptr != 0) { ^~~~ Reported-by: Clang

Re: [Qemu-devel] [PATCH 00/11] Make memory_region_init_ram() and friends handle migration

On 07/23/2017 04:58 PM, Peter Maydell wrote: On 22 July 2017 at 05:47, Philippe Mathieu-Daudé <f4...@amsat.org> wrote: I'm seeing memleaks using the malta machine, they come from the smbus_eeprom_init() in hw/i2c/smbus_eeprom.c which does: uint8_t *eeprom_buf = g_malloc0(8 * 256);

Re: [Qemu-devel] [PATCH] hw/arm: use defined type name instead of hard-coded string

On 07/27/2017 09:36 AM, Peter Maydell wrote: On 17 July 2017 at 14:30, Philippe Mathieu-Daudé <f4...@amsat.org> wrote: Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- Hi Peter, this patch is waiting the IDE queue to enters since it depen

Re: [Qemu-devel] [PATCH for 2.10 0/4] check dtc submodule is outdated

On 07/26/2017 06:40 PM, Philippe Mathieu-Daudé wrote:> patch 4: if no system libdtc and submodule present, compile the dtc submodule and verify it is at least v1.4.2. Prefixed RFC because I'm not sure about these 3 lines: +make -C dtc 1>/dev/null which can leads to: /bin/sh: 1:

[Qemu-devel] [PATCH for 2.10 0/8] docs: fix broken paths

docs/ \ | sed -ne "s/.* \(docs[^ :)}\"\']*\).*/\1/p" \ | sed -e 's/\(.*\)\.$/\1/p' | sort -u | while read p;do ls -ld $p 1>/dev/null done Regards, Phil. Cleber Rosa (1): docs: fix broken paths to docs/interop dir Philippe Mathieu-Daudé (7): docs: fix br

[Qemu-devel] [PATCH for 2.10 4/8] docs: fix broken paths to docs/devel/atomics.txt

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- docs/devel/lockcnt.txt | 2 +- include/qemu/atomic.h | 4 ++-- tcg/README | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/docs/devel/lockcnt.txt b/docs/devel/lockcnt.txt index 2a79b3205b..7c09

[Qemu-devel] [PATCH for 2.10 6/8] docs: fix broken paths to docs/config/ich9-ehci-uhci.cfg

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- docs/usb2.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/usb2.txt b/docs/usb2.txt index b9e7548073..09df45b5b1 100644 --- a/docs/usb2.txt +++ b/docs/usb2.txt @@ -50,7 +50,7 @@ companion controllers wi

[Qemu-devel] [PATCH for 2.10 2/8] docs: fix broken paths to docs/interop/qcow2.txt

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- docs/qcow2-cache.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/qcow2-cache.txt b/docs/qcow2-cache.txt index 1fdd6f9ce7..b0571de4b8 100644 --- a/docs/qcow2-cache.txt +++ b/docs/qcow2-cache.txt @@ -15,7

[Qemu-devel] [PATCH for 2.10 1/8] docs: fix broken paths to docs/interop dir

From: Cleber Rosa <cr...@redhat.com> With the move of some docs to docs/interop on d59157e, a couple of references where not updated. Signed-off-by: Cleber Rosa <cr...@redhat.com> [PMD: fixed another reference in docs/interop/qmp-spec.txt] Signed-off-by: Philippe Mathieu-Daudé <

[Qemu-devel] [PATCH for 2.10 5/8] docs: fix broken paths to docs/devel/tracing.txt

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- audio/trace-events | 2 +- block/trace-events | 2 +- chardev/trace-events| 2 +- crypto/trace-events | 2 +- hw/9pfs/trace-events| 2 +- hw/acpi/trace-events

[Qemu-devel] [PATCH for 2.10 7/8] docs: fix broken paths to docs/specs/ivshmem-spec.txt

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- docs/specs/pci-ids.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/specs/pci-ids.txt b/docs/specs/pci-ids.txt index 95adee07d6..bb99a0257e 100644 --- a/docs/specs/pci-ids.txt +++ b/docs/specs/pci-i

[Qemu-devel] [PATCH for 2.10 3/8] docs: fix broken paths to docs/devel/qapi-code-gen.txt

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- docs/devel/writing-qmp-commands.txt | 2 +- include/qapi/visitor.h | 2 +- qapi/introspect.json| 2 +- qapi/qapi-util.c| 2 +- scripts/qapi2texi.py| 2 +- 5 files c

[Qemu-devel] [PATCH for 2.10 8/8] docs: fix broken paths to docs/spin/

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- docs/spin/aio_notify.promela| 6 +++--- docs/spin/aio_notify_accept.promela | 4 ++-- docs/spin/aio_notify_bug.promela| 4 ++-- docs/spin/tcg-exclusive.promela | 2 +- include/block/aio.h | 2 +- 5

Re: [Qemu-devel] [Qemu-trivial] [PATCH for 2.10 v2 18/20] 9pfs: avoid sign conversion error simplifying the code

On 07/27/2017 08:40 AM, Greg Kurz wrote: On Wed, 26 Jul 2017 23:42:22 -0300 Philippe Mathieu-Daudé <f4...@amsat.org> wrote: Reviewed-by: Greg Kurz <gr...@kaod.org> Now, I'm not sure this can be merged during hard freeze since it is more code cleanup than actual bug fixing... Hm

[Qemu-devel] [PATCH 08/47] MAINTAINERS: add missing entry for virtio

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 69987b5e5b..6da3264d4e 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1046,6 +1046,7 @@ F: hw/virtio/Makefile.objs F: hw/virtio/trace-

[Qemu-devel] [PATCH 22/47] MAINTAINERS: add missing M68k entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 14109f4513..06b092bf85 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -156,6 +156,7 @@ M: Laurent Vivier <laur...@vivier.eu> S: M

[Qemu-devel] [PATCH 16/47] MAINTAINERS: add missing entry for gdb

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 9670a3901a..653c2a1db1 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1320,6 +1320,7 @@ L: qemu-devel@nongnu.org S: Odd Fixes F: g

[Qemu-devel] [PATCH 13/47] MAINTAINERS: add missing entry for virtio/rng

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index a9f950248a..842c66c325 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1103,6 +1103,7 @@ F: include/hw/virtio/virtio-rng.h F: include/

[Qemu-devel] [PATCH 26/47] MAINTAINERS: add missing NiosII entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 7354afb9f4..a4df19d304 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -199,6 +199,7 @@ S: Maintained F: target/nios2/ F: hw/nios2/ F:

[Qemu-devel] [PATCH 37/47] MAINTAINERS: update docs/devel/ entries

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 7649d57e0f..23de632424 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1438,7 +1438,7 @@ F: tests/test-qapi-

[Qemu-devel] [PATCH 35/47] MAINTAINERS: add missing entries for throttling infra

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index a2b008e305..b9a132d964 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1584,8 +1584,10 @@ M: Alberto

[Qemu-devel] [PATCH 29/47] MAINTAINERS: add missing SH-4 entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 9a0da13bcd..bb1cb1fc1c 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -235,6 +235,7 @@ F: target/sh4/ F: hw/sh4/ F: disas/sh4.c F: i

[Qemu-devel] [RFC PATCH 41/47] MAINTAINERS: add missing I2C entries

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index fb6a82c1ad..46807decf5 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1630,6 +1630,12 @@ F: include/sysemu/replay.h F: docs/repl

[Qemu-devel] [RFC PATCH 44/47] MAINTAINERS: add missing entry for tilegx

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 4 1 file changed, 4 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 453f5fb3d7..dc1a69051c 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -283,6 +283,10 @@ F: hw/tricore/ F: include/hw/tricore/ F: d

[Qemu-devel] [RFC PATCH 45/47] MAINTAINERS: add missing entries for loader

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- RFC because I'm not sure 'loader' is the same goal than 'generic-loader' MAINTAINERS | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index dc1a69051c..224890643d 100644 --- a/MAINTAINERS

[Qemu-devel] [RFC PATCH 43/47] MAINTAINERS: add missing TPM entries

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 7 +++ 1 file changed, 7 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 10c34f2d6d..453f5fb3d7 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1642,6 +1642,13 @@ F: bt-*.c F: hw/bt/ F: include/h

Re: [Qemu-devel] Status and RFC of patchew testings on QEMU

On Mon, Jul 17, 2017 at 7:20 AM, Daniel P. Berrange wrote: > I'm a little concerned about the fact that we've now got three different > sets of tests that are being run on pull requests. There are the tests hmm 4? there is also Shippable cross-builds:

[Qemu-devel] [PATCH 09/47] MAINTAINERS: add missing entry for virtio/9p

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 6da3264d4e..4f9ce6e686 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1056,6 +1056,7 @@ S: Supported F: hw/9pfs/ F: fsdev/ F: tests/

[Qemu-devel] [PATCH 04/47] MAINTAINERS: add missing USB entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index f10252c292..3b472d7a09 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1008,6 +1008,7 @@ F: docs/usb2.txt F: docs/usb-storage.txt F: i

[Qemu-devel] [PATCH 17/47] MAINTAINERS: add missing Guest Agent entries

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 4 1 file changed, 4 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 653c2a1db1..8cb94af6c5 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1448,6 +1448,10 @@ QEMU Guest Agent M: Michael Rot

[Qemu-devel] [PATCH 10/47] MAINTAINERS: add missing entry for virtio/blk

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 4f9ce6e686..2958dd2479 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1067,6 +1067,7 @@ S: Supported F: hw/block/virtio-blk.c F: hw

[Qemu-devel] [PATCH 12/47] MAINTAINERS: add missing entry for virtio/input

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index bfdb2f8928..a9f950248a 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1084,6 +1084,7 @@ M: Gerd Hoffmann <kra...@redhat.com> S

[Qemu-devel] [PATCH 21/47] MAINTAINERS: add missing CRIS entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 286ff7e509..14109f4513 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -131,6 +131,7 @@ F: hw/cris/ F: include/hw/cris/ F: tests/tcg/cr

[Qemu-devel] [PATCH 24/47] MAINTAINERS: add missing MicroBlaze entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 1424788a9d..fa6b0d189a 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -165,6 +165,7 @@ S: Maintained F: target/microblaze/ F: hw/micr

[Qemu-devel] [PATCH 39/47] MAINTAINERS: add missing SSI entries

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- RFC because I'm not sure m25p80 fits. MAINTAINERS | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 54e35fdab9..83597fca4e 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1010,7 +1010,9 @@ M:

[Qemu-devel] [PATCH 33/47] MAINTAINERS: add missing Xtensa entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 6d805b081d..b2c9945299 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -271,6 +271,7 @@ S: Maintained F: target/xtensa/ F: hw/xten

[Qemu-devel] [PATCH 38/47] MAINTAINERS: update docs/interop/ entries

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 23de632424..54e35fdab9 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1283,7 +1283,7 @@ F: block/dirty-bitma

[Qemu-devel] [RFC PATCH 46/47] MAINTAINERS: add missing entries for Coccinelle scripts

and voluntary myself as reviewer Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- I'm not a Coccinelle specialist but willing to learn and help! MAINTAINERS | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 2248

[Qemu-devel] [RFC PATCH 42/47] MAINTAINERS: add missing Bluetooth entries

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 46807decf5..10c34f2d6d 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1636,6 +1636,12 @@ F: hw/i2c/core.c F: include/hw/i2c/i2

Re: [Qemu-devel] [PATCH] build-sys: add --disable-vhost-user

On 07/26/2017 02:52 PM, Michael S. Tsirkin wrote: On Tue, Jul 18, 2017 at 06:34:37PM +0200, Marc-André Lureau wrote: ... In the future, we may want to hide vhost-user from QAPI/introspection with conditional compilation, although the design of this hasn't been fully fleshed out yet and

[Qemu-devel] [PATCH 00/47] add missing entries in MAINTAINERS

in each patch. The criteria I used is "If I change this file, who should I notify? Who would be interested?" Regards, Phil. Philippe Mathieu-Daudé (47): MAINTAINERS: add missing entry for documentation MAINTAINERS: add missing ARM entries MAINTAINERS: add missing STM32 entry M

[Qemu-devel] [PATCH 07/47] MAINTAINERS: add missing entry for vfio/ccw

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index ddf6f3f6d8..69987b5e5b 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1029,6 +1029,7 @@ S: Supported F: hw/vfio/ccw.c F: hw/s390

[Qemu-devel] [PATCH 03/47] MAINTAINERS: add missing STM32 entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 795f89f709..f10252c292 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -556,6 +556,7 @@ F: hw/char/stm32f2xx_usart.c F: hw

[Qemu-devel] [RFC PATCH 15/47] MAINTAINERS: add missing VMWare entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- RFC because I'm not sure about tests/ MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index d8aef164ca..9670a3901a 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1140,6 +1140,7 @@ M:

[Qemu-devel] [PATCH 19/47] MAINTAINERS: add missing qcow2 entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 6b83dac812..2dcdda4cd5 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1856,6 +1856,7 @@ M: Max Reitz <mre...@redhat.com>

[Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 8cb94af6c5..6b83dac812 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1633,6 +1633,7 @@ Common code M: Richard Henderson <r...@tw

[Qemu-devel] [PATCH 25/47] MAINTAINERS: add missing MIPS entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index fa6b0d189a..7354afb9f4 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -182,6 +182,7 @@ F: include/hw/intc/mips_gic.h F: include/hw

[Qemu-devel] [PATCH 32/47] MAINTAINERS: add missing UniCore32 entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 7997c90be8..6d805b081d 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -253,6 +253,7 @@ S: Maintained F: target/unicore32/ F: hw/uni

[Qemu-devel] [PATCH 34/47] MAINTAINERS: add missing X86 entries

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index b2c9945299..a2b008e305 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -263,6 +263,8 @@ S: Maintained F: target/i386/ F: hw/i386/ F:

[Qemu-devel] [PATCH 27/47] MAINTAINERS: add missing OpenRISC entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index a4df19d304..35a7253e22 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -207,6 +207,7 @@ S: Odd Fixes F: target/openrisc/ F: hw/openri

[Qemu-devel] [PATCH 40/47] MAINTAINERS: add missing entry for AIO

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 83597fca4e..fb6a82c1ad 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1253,6 +1253,7 @@ F: util/aio-*.c F: block/io.c F: migration

[Qemu-devel] [PATCH 36/47] MAINTAINERS: add missing entry test for megasas

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index b9a132d964..7649d57e0f 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1144,6 +1144,7 @@ L: qemu-bl...@nongnu.org S: Supported F: h

[Qemu-devel] [RFC PATCH 47/47] script to find outdated entry in MAINTAINERS

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- scripts/check_maintainer.sh | 21 + 1 file changed, 21 insertions(+) create mode 100755 scripts/check_maintainer.sh diff --git a/scripts/check_maintainer.sh b/scripts/check_maintainer.sh new file mode 100755

[Qemu-devel] [PATCH 05/47] MAINTAINERS: add missing KVM entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 3b472d7a09..ece02522be 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -278,6 +278,7 @@ S: Supported F: */kvm.* F: accel/kvm/ F: i

[Qemu-devel] [PATCH 02/47] MAINTAINERS: add missing ARM entries

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 972118e70b..795f89f709 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -120,6 +120,8 @@ F: include/hw/cpu/a*mpcore.h F: disas/ar

[Qemu-devel] [RFC PATCH 01/47] MAINTAINERS: add missing entry for documentation

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- because having only a "Build system architecture" entry in Documentation seems odd to me, so RFC. MAINTAINERS | 3 +++ 1 file changed, 3 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 5ea273f899..9

[Qemu-devel] [PATCH 11/47] MAINTAINERS: add missing entry for virtio/ccw

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 2958dd2479..bfdb2f8928 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1075,6 +1075,7 @@ M: Cornelia Huck <coh...@redhat.com>

[Qemu-devel] [PATCH 20/47] MAINTAINERS: add missing PCI entries

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 3 +++ 1 file changed, 3 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 2dcdda4cd5..286ff7e509 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -933,6 +933,9 @@ F: include/hw/pci/* F: hw/misc/pci-testdev.c

[Qemu-devel] [PATCH 14/47] MAINTAINERS: add missing entry for virtio/crypto

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 842c66c325..d8aef164ca 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -,6 +,7 @@ S: Supported F: hw/virtio/virtio-crypto.c

[Qemu-devel] [PATCH 06/47] MAINTAINERS: add missing entry for vhost

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index ece02522be..ddf6f3f6d8 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1035,6 +1035,7 @@ vhost M: Michael S. Tsirkin <m...@redha

[Qemu-devel] [PATCH 30/47] MAINTAINERS: add missing SPARC entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index bb1cb1fc1c..e363a18631 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -245,6 +245,7 @@ F: target/sparc/ F: hw/sparc/ F: hw/sparc

[Qemu-devel] [PATCH 23/47] MAINTAINERS: add missing LM32 entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 06b092bf85..1424788a9d 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -150,6 +150,7 @@ F: hw/*/milkymist-* F: include/hw/char/lm32_juar

[Qemu-devel] [PATCH 28/47] MAINTAINERS: add missing PowerPC entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 35a7253e22..9a0da13bcd 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -218,6 +218,7 @@ F: target/ppc/ F: hw/ppc/ F: include/hw/p

[Qemu-devel] [PATCH 31/47] MAINTAINERS: add missing TriCore entry

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index e363a18631..7997c90be8 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -277,6 +277,7 @@ S: Maintained F: target/tricore/ F: hw/trico

Re: [Qemu-devel] Status and RFC of patchew testings on QEMU

Hi Fam, On 07/17/2017 03:35 AM, Fam Zheng wrote: So far we have these tests running by patchew on each patch series: * Docker tests Basically it is make docker-test-quick@centos6 \ docker-test-build@min-glib \ docker-test-mingw@fedora" *

Re: [Qemu-devel] Status and RFC of patchew testings on QEMU

On Mon, Jul 17, 2017 at 10:40 AM, Max Reitz wrote: > On 2017-07-17 13:31, Kevin Wolf wrote: >> I think we had it in 'make check' for a while, but I seem to remember >> that people complained about things like the disk space that some of the >> tests needed for temporary files,

Re: [Qemu-devel] [PATCH qemu] Revert "elf-loader: warn about invalid endianness"

On 07/26/2017 12:47 AM, Alexey Kardashevskiy wrote: On 21/07/17 18:05, Philippe Mathieu-Daudé wrote: On 07/21/2017 04:30 AM, Alexey Kardashevskiy wrote: On 21/07/17 16:48, Philippe Mathieu-Daudé wrote: I submitted this patch because I spent some time debugging while QEMU was failing silently

Re: [Qemu-devel] [PATCH] fix qemu-system-unicore32 crashing when calling without -kernel

On 07/26/2017 12:04 PM, Eduardo Otubo wrote: Starting qemu-system-unicore32 without the -kernel parameter results in an assert() returns false and aborts qemu. This patch replaces it with a proper error message followed by exit(1). Signed-off-by: Eduardo Otubo ---

Re: [Qemu-devel] [PATCH] fix qemu-system-unicore32 crashing when calling without -kernel

On 07/26/2017 01:05 PM, Thomas Huth wrote: On 26.07.2017 18:00, Philippe Mathieu-Daudé wrote: On 07/26/2017 12:04 PM, Eduardo Otubo wrote: Starting qemu-system-unicore32 without the -kernel parameter results in an assert() returns false and aborts qemu. This patch replaces it with a proper

Re: [Qemu-devel] hw/core/loader-fit.c:105:41: error: expected expression

On 07/26/2017 01:46 PM, Programmingkid wrote: On Jul 26, 2017, at 6:24 AM, Peter Maydell <peter.mayd...@linaro.org> wrote: On 26 July 2017 at 06:15, Programmingkid <programmingk...@gmail.com> wrote: On Jul 26, 2017, at 12:13 AM, Philippe Mathieu-Daudé <f4...@amsat.org> wrote

Re: [Qemu-devel] [PATCH 1/4] configure: remember the user to run 'git submodule' command in source dir

On 07/26/2017 06:51 PM, Eric Blake wrote: On 07/26/2017 04:40 PM, Philippe Mathieu-Daudé wrote: In the subject: s/remember/remind/ Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- configure | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/confi

[Qemu-devel] [RFC PATCH for 2.10 4/4] fdt: compile dtc submodule to check it is up-to-date

Reported-by: John Arbuckle <programmingk...@gmail.com> Message-Id: <65ed9743-b53c-4e6a-866f-c88365091...@gmail.com> Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- configure | 24 +++- 1 file changed, 19 insertions(+), 5 deletions(-) diff -

Re: [Qemu-devel] [PATCH for 2.10 0/4] check dtc submodule is outdated

On 07/26/2017 06:56 PM, Eric Blake wrote: On 07/26/2017 04:47 PM, no-re...@patchew.org wrote: This series failed build test on s390x host. Please find the details below. [...] === PACKAGES === >> dtc-1.4.2-1.fc25.s390x === TEST BEGIN === Using CC: /home/fam/bin/cc ERROR: fdt disabled

Re: [Qemu-devel] hw/core/loader-fit.c:105:41: error: expected expression

On 07/26/2017 01:46 PM, Programmingkid wrote: On Jul 26, 2017, at 6:24 AM, Peter Maydell <peter.mayd...@linaro.org> wrote: On 26 July 2017 at 06:15, Programmingkid <programmingk...@gmail.com> wrote: On Jul 26, 2017, at 12:13 AM, Philippe Mathieu-Daudé <f4...@amsat.org> wrote

[Qemu-devel] [PATCH for 2.10 0/4] check dtc submodule is outdated

t=mips64el-softmmu --extra-cflags=-fmax-errors=1 ERROR: Your DTC submodule is outdated. Your options: (1) Preferred: Install the DTC (libfdt) devel package (2) Update the DTC submodule, using: (in /home/phil/source/qemu) git submodule update --init dtc Regards, Phil.

[Qemu-devel] [PATCH for 2.10 2/4] fdt: check fdt_required condition can be satisfied _after_ testing libfdt

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- configure | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/configure b/configure index 063de32773..0d5bdb3ae9 100755 --- a/configure +++ b/configure @@ -3558,15 +3558,6 @@ for target in $targe

[Qemu-devel] [PATCH 1/4] configure: remember the user to run 'git submodule' command in source dir

Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- configure | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/configure b/configure index f8b1d014d7..063de32773 100755 --- a/configure +++ b/configure @@ -502,6 +502,7 @@ if test -f "

[Qemu-devel] [PATCH for 2.10 3/4] fdt: probe for v1.4.2 using fdt_setprop_inplace_namelen_partial()

instead of fdt_first_subnode() which is v1.4.0 Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- configure | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 0d5bdb3ae9..2d803d6a77 100755 --- a/configure +++ b/configure @@ -

Re: [Qemu-devel] [PATCH for 2.10 20/35] arm/boot: fix undefined instruction on secondary smp cpu bootloader

On 07/24/2017 06:06 PM, Peter Maydell wrote: On 24 July 2017 at 19:27, Philippe Mathieu-Daudé <f4...@amsat.org> wrote: In a ARM multicore system, write_secondary_boot() only initializes fixups for FIXUP_GIC_CPU_IF and FIXUP_BOOTREG, while smpboot[] also uses FIXUP_DSB. This r

Re: [Qemu-devel] [PATCH for 2.10 16/35] usb/dev-mtp: fix use of uninitialized values

On 07/25/2017 09:34 AM, Gerd Hoffmann wrote: case CMD_GET_OBJECT_INFO: -o = usb_mtp_object_lookup(s, c->argv[0]); +if (c->argc > 0) { +o = usb_mtp_object_lookup(s, c->argv[0]); +} How about zero-initializing c->argv instead? I checked the MTP specs

Re: [Qemu-devel] [PATCH for 2.10 03/35] thunk: check nb_fields is valid before continuing

On 07/24/2017 03:37 PM, Eric Blake wrote: On 07/24/2017 01:27 PM, Philippe Mathieu-Daudé wrote: thunk.c:91:32: warning: Call to 'malloc' has an allocation size of 0 bytes se->field_offsets[i] = malloc(nb_fields * sizeof(

Re: [Qemu-devel] [PATCH for 2.10 17/35] usb/dev-mtp: fix use of uninitialized values

On 07/24/2017 03:27 PM, Philippe Mathieu-Daudé wrote: Reported-by: Clang Static Analyzer Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- hw/usb/dev-mtp.c | 36 +--- 1 file changed, 21 insertions(+), 15 deletions(-) diff --git a/hw/usb/dev-m

Re: [Qemu-devel] [PATCH] configure: fix qemu-ga missing '.exe' extension on windows

On 07/25/2017 11:56 PM, Michael Roth wrote: Quoting Michael Roth (2017-07-25 21:53:41) Quoting Philippe Mathieu-Daudé (2017-07-25 20:45:31) [...] That change was done explicitly via fafcaf1d so that `make qemu-ga` and `make` without --disable-tools specified to configure will generate the MSI

Re: [Qemu-devel] [PATCH] configure: fix qemu-ga missing '.exe' extension on windows

As noticed by Michael Roth the ./configure entry for qemu-ga is missing the $(EXESUF) on purpose (see fafcaf1d). Patch dropped for 2.10 On 07/24/2017 10:15 PM, Philippe Mathieu-Daudé wrote: Reported-by: Sameeh Jubran <sjub...@redhat.com> Signed-off-by: Philippe Mathieu-Daudé <f4...@

Re: [Qemu-devel] hw/core/loader-fit.c:105:41: error: expected expression

Hi John, On 07/25/2017 07:55 PM, Programmingkid wrote: While compiling the mips64el-softmmu target I encountered these errors: CC hw/display/g364fb.o hw/core/loader-fit.c:105:41: error: expected expression *addr = fdt32_to_cpu(*(fdt32_t *)prop);

Re: [Qemu-devel] [PATCH for 2.10 v2 19/20] spapr_vio: fix overflow of qdevs in spapr_dt_vdevice()

On 07/27/2017 12:43 AM, David Gibson wrote: On Wed, Jul 26, 2017 at 11:42:23PM -0300, Philippe Mathieu-Daudé wrote: sizeof(ptr) was used instead of sizeof(struct)... also use g_malloc_n() which take care of possible type overflow. hw/ppc/spapr_vio.c:641:22: warning: The code calls sizeof

Re: [Qemu-devel] [PATCH for 2.10 2/2] Revert "elf-loader: warn about invalid endianness"

pseries machine already checks for ELF_LOAD_WRONG_ENDIAN and tries with big_endian=0. Signed-off-by: Alexey Kardashevskiy <a...@ozlabs.ru> Signed-off-by: Aurelien Jarno <aurel...@aurel32.net> Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- hw/core/loader.c | 1 - 1

Re: [Qemu-devel] [PATCH for 2.10 1/2] hw/mips: load_elf_strerror to report kernel loading failure

or instead of fprintf. Signed-off-by: Aurelien Jarno <aurel...@aurel32.net> Thank you Aurelien :) Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org> Tested-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- hw/mips/mips_fulong2e.c | 15 +-- hw/mips/mips_malta.c|

[Qemu-devel] [PATCH for 2.10 v2 08/20] vfio/platform: fix use of freed memory

free the data _after_ using it. hw/vfio/platform.c:126:29: warning: Use of memory after it is freed qemu_set_fd_handler(*pfd, NULL, NULL, NULL); ^~~~ Reported-by: Clang Static Analyzer Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> Reviewed-by:

[Qemu-devel] [PATCH for 2.10 v2 15/20] thunk: assert nb_fields is valid

thunk.c:91:32: warning: Call to 'malloc' has an allocation size of 0 bytes se->field_offsets[i] = malloc(nb_fields * sizeof(int)); ^~~ Reported-by: Clang Static Analyzer Signed-off-by: Philippe Mathieu-Daudé <f4...@ams

[Qemu-devel] [PATCH for 2.10 v2 00/20] fix bugs reported by Clang Static Analyzer

reAPI.gets \ -enable-checker security.insecureAPI.strcpy \ -disable-checker unix.Vfork \ make -k -j4 Philippe Mathieu-Daudé (20): tests: add missing dependency to build QTEST_QEMU_BINARY loader: check get_image_size() return value ivshmem: fix incorrect error handling in ivshmem_recv_m

[Qemu-devel] [PATCH for 2.10 v2 04/20] nbd: fix memory leak in nbd_opt_go()

nbd/client.c:385:12: warning: Potential leak of memory pointed to by 'buf' Reported-by: Clang Static Analyzer Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> Reviewed-by: Eric Blake <ebl...@redhat.com> --- nbd/client.c | 8 +--- 1 file changed, 5 insertions(+), 3 deleti

[Qemu-devel] [PATCH for 2.10 v2 07/20] net/eth: fix incorrect check of iov_to_buf() return value

ddr); ^ ~~ net/eth.c:475:34: warning: The code calls sizeof() on a pointer type. This can produce an unexpected result return bytes_read == sizeof(src_addr); ^ ~~ Reported-by: Clang Static Analyzer Signed-off-by: Philippe Mathieu-Daudé

[Qemu-devel] [PATCH for 2.10 v2 10/20] m68k/translate: fix incorrect copy/paste

|| op == CC_OP_ADDL) { ^ Reported-by: Clang Static Analyzer Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> Reviewed-by: Laurent Vivier <laur...@vivier.eu> --- target/m68k/translate.c | 2 +- 1 file changed,

[Qemu-devel] [PATCH for 2.10 v2 20/20] i2c/exynos4210: fix write to I2CADD register, bit 0 is not mapped

>From the Exynos4210 User Manual [1]: 14.4.1.3 I2CADDn (MULTI-MASTER I2C-Bus Address Register) [7-1] slave address, latched from the I2C-bus. bit [0] is not mapped. [1]: Exynos_4_Dual_45nm_User_Manaul_Public_REV1.00-0.pdf Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --

[Qemu-devel] [PATCH for 2.10 v2 03/20] ivshmem: fix incorrect error handling in ivshmem_recv_msg()

. hw/misc/ivshmem.c:650:14: warning: Loss of sign in implicit conversion } while (n < sizeof(msg)); ^ Reported-by: Clang Static Analyzer Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> Reviewed-by: Markus Armbruster <arm...@redhat.com> --- hw/misc/ivshmem

<    2   3   4   5   6   7   8   9   10   11   >