Static Analyzer
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Reviewed-by: Eric Blake <ebl...@redhat.com>
Reviewed-by: Alistair Francis <alistair.fran...@xilinx.com>
---
hw/core/loader.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/core/loa
This allow a one liner from fresh repository clone, i.e.:
./configure && make -j check-qtest-aarch64
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Reviewed-by: Alex Bennée <alex.ben...@linaro.org>
Reviewed-by: Stefan Hajnoczi <stefa...@redhat.com>
not hit since 2009! :)
linux-user/elfload.c:1102:20: warning: Out of bound memory access (access
exceeds upper limit of memory block)
(*regs[i]) = tswap32(env->gregs[i]);
~~~^~~~
Reported-by: Clang Static Analyzer
Signed-off-by: Philippe Mathieu-Da
Extract the (correct) cleaning code as a new function vnc_free_addresses() then
use it to remove the memory leaks.
Reported-by: Clang Static Analyzer
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Reviewed-by: Daniel P. Berrange <berra...@redhat.com>
---
u
hw/vfio/pci.c:308:29: warning: Use of memory after it is freed
qemu_set_fd_handler(*pfd, NULL, NULL, vdev);
^~~~
Reported-by: Clang Static Analyzer
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Reviewed-by: Paolo Bonzini <pbonz...@redhat.com
Static Analyzer
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Reviewed-by: Eric Blake <ebl...@redhat.com>
---
block/qcow2-refcount.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/block/qcow2-refcount.c b/block/qcow2-refcount.c
index c9b0dcb4f3..168fc
linux-user/syscall.c:5581:9: warning: Dereference of undefined pointer value
if (*host_rt_dev_ptr != 0) {
^~~~
Reported-by: Clang Static Analyzer
Suggested-by: Laurent Vivier <lviv...@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
value
if (addr->sa_family == AF_NETLINK && len >= sizeof(struct sockaddr_nl)) {
~~~ ^
Reported-by: Clang Static Analyzer
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Reviewed-by: Laurent Vivier <laur...@vivier.eu>
---
linux-user/sy
linux-user/syscall.c:555:25: warning: Out of bound memory access (accessed
memory precedes memory block)
target_fd_trans[fd] = trans;
^~~
Reported-by: Clang Static Analyzer
Suggested-by: Laurent Vivier <lviv...@redhat.com>
Signed-off-by: Philippe Mathieu-Dau
hw/bt/sdp.c:753:5: warning: Potential leak of memory pointed to by 'data'
qsort(record->attribute_list, record->attributes,
^
Reported-by: Clang Static Analyzer
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
hw/bt/*:
get_maintainer.pl: No maintainers found
);
^ ~~
hw/ppc/spapr_vio.c:648:23: warning: The code calls sizeof() on a pointer type.
This can produce an unexpected result
qsort(qdevs, num, sizeof(qdev), compare_reg);
^ ~~
Reported-by: Clang Static Analyzer
Signed-off-by: Philippe Mathieu-Daudé <
(note this is how other functions also handle the errors).
hw/9pfs/9p.c:948:18: warning: Loss of sign in implicit conversion
offset = err;
^~~
Reported-by: Clang Static Analyzer
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
hw/9pfs/9p.c | 6 ++
Hi Lukáš,
On 07/24/2017 09:36 AM, Lukáš Doktor wrote:
Dne 22.7.2017 v 03:30 Philippe Mathieu-Daudé napsal(a):
Hi Lukáš,
Since comment/indent fixes and code changes are not related I'd rather see this
split in at least 2 patches.
Hello Philippe, thank you for the review, I'm wondering what
Useful to build a whole set at once.
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
Makefile | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/Makefile b/Makefile
index 5f18243d05..da899522e4 100644
--- a/Makefile
+++ b/Makefile
@@ -814,6 +814,11 @@
On Tue, Jul 25, 2017 at 3:13 AM, Lukáš Doktor <ldok...@redhat.com> wrote:
> Dne 25.7.2017 v 08:04 Philippe Mathieu-Daudé napsal(a):
>> Hi Lukáš,
>>
>> On 07/24/2017 09:36 AM, Lukáš Doktor wrote:
>>> Dne 22.7.2017 v 03:30 Philippe Mathieu-Daudé napsal(a):
&g
On 07/19/2017 04:45 PM, Peter Maydell wrote:
The sparc-linux-user test fails:
/home/petmay01/linaro/qemu-for-merges/build/all-linux-static/sparc-linux-user/qemu-sparc
-L ./gnemul/qemu-sparc sparc/ls -l dummyfile
Inconsistency detected by ld.so: rtld.c: 858: dl_main: Assertion
Hi Pranith,
On 12/01/2016 02:28 AM, Pranith Kumar wrote:
Hello,
The following patch series adds litmus tests to test consistency for
MTTCG enabled qemu. These patches apply on top of the clean up
tests/tcg folder made by my previous patch series.
The tests were generated using the litmus
Hi Daniel,
On 07/12/2017 01:25 PM, Daniel P. Berrange wrote:
When trying to debug problems with tests it is natural to set
DEBUG=1 when starting the docker environment. Unfortunately
this has a side-effect of enabling an eth0 network interface
in the container, which changes the operating
On 07/12/2017 06:46 PM, Philippe Mathieu-Daudé wrote:
now trying old debian release:
$ docker run --rm -it debian:wheezy sh -c "cat /etc/debian_version";
echo $?
7.11
0
$ docker run --rm -it debian:wheezy bash -c "cat /etc/debian_version";
echo $?
139
Indeed using
)
Merge sockets 2017/07/11 v2
$ git diff pull-sockets-2017-07-11-1..pull-sockets-2017-07-11-2
As I understand, diff since v1 is:
- restricted sockets-proto tests to x86_64
- run tests using INADDR_LOOPBACK instead of INADDR_ANY
signed PR, else:
Tested-by: Philippe Mathieu-Daudé
Hi Alex, Fam,
I wanted to try this patch but got:
$ make docker-test-quick@centos6 NETWORK=1
BUILD centos6
The command '/bin/sh -c yum install -y epel-release' returned a non-zero
code: 139
Traceback (most recent call last):
File "./tests/docker/docker.py", line 382, in
Hi Jaroslaw,
Thank for reporting this, however this fix has been reviewed:
http://patchwork.ozlabs.org/patch/781982/
On 07/12/2017 10:52 AM, Jaroslaw Pelczar wrote:
Running Ubuntu with systemd on AArch64 Cortex-A53 and qemu is compiled
with
./configure --target-list=aarch64-softmmu
On 07/18/2017 02:51 PM, Eric Blake wrote:
On 07/18/2017 01:09 AM, Philippe Mathieu-Daudé wrote:
Applied using the Coccinelle semantic patch scripts/coccinelle/use_osdep.cocci
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
hw/net/rocker/rocker.c | 2 +-
1 file chan
On 07/18/2017 11:37 AM, Thomas Huth wrote:
On 18.07.2017 13:42, Michael Walle wrote:
Am 2017-07-18 08:09, schrieb Philippe Mathieu-Daudé:
Applied using the Coccinelle semantic patch
scripts/coccinelle/use_osdep.cocci
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
QEMU_IS_A
00
s56= s57= d28=
s58= s59= d29=
s60= s61= d30=
s62= s63= d31=0000
FPSCR: 0000
:)
Philippe Mathieu-Daudé (8):
MAINTAINERS: update tcg entries
MAINTAINERS: u
moved in a9ded601..244f1441 to accel/
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
MAINTAINERS | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 839f7ca063..06006fc7df 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@
moved in 92229a57 to accel/
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
MAINTAINERS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 06006fc7df..86a08c5aac 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -273,8 +273,8 @@ Over
fault (core dumped)
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
configure | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/configure b/configure
index c571ad14e5..510f443e06 100755
--- a/configure
+++ b/configure
@@ -6225,7 +6225,11 @@ fi
i
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
MAINTAINERS |7 -
Makefile.target |2 -
configure| 32 +-
disas.c |5 +-
disas/Makefile.objs |4 -
disas/tci.c | 61 ---
include/disas/bf
moved in 56e2cd24..28b99f47 to accel/
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
MAINTAINERS | 1 -
1 file changed, 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 86a08c5aac..530293044b 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -323,7 +323,6 @@ M: S
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
.travis.yml | 3 ---
1 file changed, 3 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index 27a2d9cfb3..d10ee5ed79 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -74,9 +74,6 @@ matrix:
- env: CONFIG="--enable-gpr
moved in 244f1441 to tcg/
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
MAINTAINERS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 530293044b..9bad523060 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1654,7 +1654,7 @@ TCI
ace 0x7fa1904b0aa0 [0: ffc00036cd24]
TODO qemu/tci.c:1049: tcg_qemu_tb_exec()
qemu/tci.c:1049: tcg fatal error
Aborted
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
tcg/tci.c | 1 -
1 file changed, 1 deletion(-)
d
On Thu, Jun 29, 2017 at 4:04 PM, Laurent Vivier <laur...@vivier.eu> wrote:
> Signed-off-by: Laurent Vivier <laur...@vivier.eu>
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>
> ---
> fpu/softfloat-specialize.h | 10 ++
> f
On 06/29/2017 02:16 PM, Alistair Francis wrote:
Signed-off-by: Alistair Francis <alistair.fran...@xilinx.com>
Acked-by: Edgar E. Iglesias <edgar.igles...@xilinx.com>
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
Changes since RFC:
- Include more bitmasks f
I got this "Disassembler disagrees with translator over instruction
decoding" message asking to get reported here.
What happens here is coreboot incorrectly emits a Pentium2 instruction
while I'm running qemu with a Pentium cpu. I didn't know what to expect
but got this error message, then
moved in 92229a57 to accel/
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
MAINTAINERS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 06006fc7df..86a08c5aac 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -273,8 +273,8 @@ Over
moved in 56e2cd24..28b99f47 to hw/xen/ and hw/i386/xen/
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Acked-by: Anthony PERARD <anthony.per...@citrix.com>
---
MAINTAINERS | 1 -
1 file changed, 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 86a08c5aac..53029
moved in a9ded601..244f1441 to accel/
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
MAINTAINERS | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 839f7ca063..06006fc7df 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@
device and voluntary myself as
reviewer.
Regards,
Phil.
v2:
- fix inaccurate Xen commit message (review feedback from Anthony PERARD)
- add "Unimplemented" device
v1:
- update TCG/KVM/Xen/TCI entries
Philippe Mathieu-Daudé (5):
MAINTAINERS: update TCG entries
MAINTAINERS: update KVM
e char devices to chardev/
Leaving the empty file around, causes tracetool to generate
an empty .dtrace file which makes the dtrace compiler throw
a syntax error.
Signed-off-by: Daniel P. Berrange <berra...@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>
-
inlined create_unimplemented_device() calls sysbus_mmio_map_overlap().
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Reviewed-by: Eric Blake <ebl...@redhat.com>
---
This is a patch from my previous series "various easy cleanups".
v3:
- Add Eric Blake R-b
v2:
- A
On 06/29/2017 05:46 AM, Thomas Huth wrote:>> I figured out MAINTAINERS
was unsync, so added patches 1-4, they are not really
tci-related.
Since they are not related to TCI at all, please submit these as
separate series.
Ok.
Patches 5,6 are trivial fixes to let the codebase quite sane if
moved in 244f1441 to tcg/
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
MAINTAINERS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 530293044b..9bad523060 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1654,7 +1654,7 @@ TCI
Also voluntary myself as reviewer
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
MAINTAINERS | 7 +++
1 file changed, 7 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 9bad523060..890c25f0a1 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1155,6 +1155,13 @@ F
On 06/29/2017 12:47 PM, Paolo Bonzini wrote:
On 29/06/2017 17:46, Thomas Huth wrote:
TCI is enabled only with a specific configure argument if your machine
is not supported by TCG. This would break _build_ configurations, not
user configurations. It's a remote possibility that users are
On 06/29/2017 01:29 PM, Eric Blake wrote:
On 06/28/2017 08:02 PM, Philippe Mathieu-Daudé wrote:
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Double-sob seems odd.
eh you never know... I need to improve my ma
On 06/29/2017 01:52 PM, Stefan Weil wrote:
Am 29.06.2017 um 18:29 schrieb Eric Blake:
On 06/28/2017 08:02 PM, Philippe Mathieu-Daudé wrote:
remove some copy/paste leftover, code seems sane.
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Signed-off-by: Philippe Mathieu-Dau
Hi Peter, Juan,
On 06/28/2017 08:30 AM, Juan Quintela wrote:
From: Peter Xu
Let the old man "MigrationState" join the object family. Direct benefit
is that we can start to use all the property features derived from
current QDev, like: HW_COMPAT_* bits, command line setup
On Mon, Jul 3, 2017 at 12:13 PM, Peter Maydell <peter.mayd...@linaro.org> wrote:
> Add documentation comments describing the public API of the
> ptimer countdown timer.
>
> Signed-off-by: Peter Maydell <peter.mayd...@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <
rlier.
>
> Signed-off-by: Fam Zheng <f...@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>
> ---
> include/qemu/thread-posix.h | 4
> include/qemu/thread-win32.h | 5 +
> util/qemu-thread-posix.c| 27
On Thu, Jun 29, 2017 at 7:39 AM, Anthony PERARD
<anthony.per...@citrix.com> wrote:
> On Wed, Jun 28, 2017 at 10:02:55PM -0300, Philippe Mathieu-Daudé wrote:
>> moved in 56e2cd24..28b99f47 to accel/
>
> That is not accurate, files have been moved to hw/i386/xen/ as writ
On Thu, Jun 29, 2017 at 10:27 AM, Alex Bennée <alex.ben...@linaro.org> wrote:
> Philippe Mathieu-Daudé <f4...@amsat.org> writes:
>> On 06/21/2017 12:42 PM, Alex Bennée wrote:> By default we use the QEMU
>> projects qemu:debian-FOO-cross images as
>>> RISU h
Hi Frederic,
On 06/29/2017 06:28 AM, KONRAD Frederic wrote:
This helps the board developer by asserting that system_clock_rate is not
null. Using systick with a zero rate will lead to a deadlock so better showing
the error.
Signed-off-by: KONRAD Frederic
---
On 06/29/2017 09:43 AM, Peter Maydell wrote:
On 29 June 2017 at 13:35, Philippe Mathieu-Daudé <f4...@amsat.org> wrote:
This is true it is better to abort here than risking a deadlock.
However it seems to me they are 3 issues here:
- the deadlock pattern is caused by using a global va
Hi Alex,
On 06/21/2017 12:42 PM, Alex Bennée wrote:> By default we use the QEMU
projects qemu:debian-FOO-cross images as
RISU hackers are likely to be QEMU developers too. However any docker
tag can be passed on the command line.
Any thought about starting to push images?
Hi Peter, Paolo,
On 07/04/2017 02:02 PM, Peter Maydell wrote:
Add a documentation comment for memory_region_allocate_system_memory().
In particular, the reason for this function's existence and the
requirement on board code to call it exactly once are non-obvious.
Signed-off-by: Peter Maydell
R_V7M_CONTROL_SPSEL_MASK) != 0);
+}
+env->v7m.control &= ~R_V7M_CONTROL_NPRIV_MASK;
+env->v7m.control |= val & R_V7M_CONTROL_NPRIV_MASK;
break;
default:
qemu_log_mask(LOG_GUEST_ERROR, "Attempt to write unknown special"
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Regards,
Phil.
On 07/05/2017 09:23 PM, Richard Henderson wrote:
We were treating FREG as an index and REG as a TCGv.
Making FREG return a TCGv is both less confusing and
a step toward cleaner banking of cpu_fregs.
Signed-off-by: Richard Henderson <r...@twiddle.net>
Reviewed-by: Philippe Mathieu-Dau
-by: Alistair Francis <alistair.fran...@xilinx.com>
> Acked-by: Edgar E. Iglesias <edgar.igles...@xilinx.com>
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>
> ---
>
> util/oslib-win32.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git
ets of the floating-point coprocessor. These undefined values yield the
value 0.0 [ floatx80_zero ] in the M68040FPSP."
^ with a such comment around:
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>
+[0x00] = floatx80_pi, /* Pi */
+[0x0
On 06/27/2017 02:58 PM, Laurent Vivier wrote:
Le 27/06/2017 à 17:45, Philippe Mathieu-Daudé a écrit :
+static const floatx80 fpu_rom[128] = {
+[0x00] = floatx80_pi, /* Pi */
+[0x0b] = make_floatx80(0x3ffd, 0x9a209a84fbcff798ULL), /*
Log10(2
I find this patch aesthetically very nice :)
On Tue, 27 Jun 2017 00:03:25 +0200
Laurent Vivier wrote:
> fmovecr moves a floating point constant from the
> FPU ROM to a floating point register.
>
> Signed-off-by: Laurent Vivier
> Reviewed-by: Richard
Hi Juan,
On 04/25/2017 07:37 PM, Juan Quintela wrote:
uint8_t has existed since . all this century?
Signed-off-by: Juan Quintela
---
hw/audio/fmopl.c | 8
hw/audio/fmopl.h | 39 ---
2 files changed, 24 insertions(+), 23
Hi Juan, is there a benefit in not squashing the previous stdint commits?
On 04/25/2017 07:37 PM, Juan Quintela wrote:
Signed-off-by: Juan Quintela
---
hw/audio/fmopl.c | 42 +-
hw/audio/fmopl.h | 54
<stef...@aporeto.com>
Reviewed-by: Greg Kurz <gr...@kaod.org>
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>
CC: anthony.per...@citrix.com
CC: jgr...@suse.com
---
hw/block/xen_blkif.h | 2 +-
hw/usb/xen-usb.c | 2 +-
include/hw/x
Hi Juan,
Same here, why not squashing as "Use stdint instead of dead GUSEMU32"?
On 04/25/2017 07:37 PM, Juan Quintela wrote:
Signed-off-by: Juan Quintela
---
hw/audio/gus.c | 2 +-
hw/audio/gusemu.h | 12 +---
hw/audio/gusemu_hal.c | 2 +-
On 04/25/2017 07:37 PM, Juan Quintela wrote:
Signed-off-by: Juan Quintela <quint...@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
hw/audio/fmopl.c | 2 +-
hw/audio/fmopl.h | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/hw/audio/
On 04/25/2017 07:37 PM, Juan Quintela wrote:
Signed-off-by: Juan Quintela <quint...@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
hw/audio/fmopl.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/hw/audio/fmopl.c b/hw/audio/f
t;use warnings;" approach. There is no functional change to the
default behavior.
Signed-off-by: Kamil Rytarowski <n...@gmx.com>
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
scripts/checkpatch.pl | 3 ++-
scripts/clean-header-guards.pl | 3 ++-
scripts/cleanup-trace-
grammatically correct.
Suggested-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Suggested-by: Max Reitz <mre...@redhat.com>
Signed-off-by: Eric Blake <ebl...@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
v11: retitle [was "qemu-io: Don't open-co
On 08/04/2017 02:20 PM, Peter Maydell wrote:
Define a new MachineClass field ignore_memory_transaction_failures.
If this is flag is true then the CPU will ignore memory transaction
failures which should cause the CPU to take an exception due to an
access to an unassigned physical address; the
Hi Olaf,
On 07/28/2017 10:11 AM, Olaf Hering wrote:
g_malloc0_n is available since glib-2.24. To allow build with older glib
versions use the generic g_new0, which is already used in many other
places in the code.
Can you provide information about which
distrib/release/version/[packages?]
On 07/28/2017 03:50 PM, Richard Henderson wrote:
Found by Coverity.
"Found by Coverity (CID 1378273)."
Reported-by: Paolo Bonzini <pbonz...@redhat.com>
Signed-off-by: Richard Henderson <r...@twiddle.net>
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.
On 07/28/2017 05:51 AM, Peter Maydell wrote:
On 28 July 2017 at 00:59, Philippe Mathieu-Daudé <f4...@amsat.org> wrote:
Hi Peter,
On 07/27/2017 07:59 AM, Peter Maydell wrote:
For an M profile v7PMSA, the system space (0xe000 - 0x) can
never be executable, even if the guest
On 07/28/2017 05:42 AM, Peter Maydell wrote:
On 27 July 2017 at 23:58, Philippe Mathieu-Daudé <f4...@amsat.org> wrote:
On 07/27/2017 07:43 PM, Philippe Mathieu-Daudé wrote:
On 07/27/2017 07:59 AM, Peter Maydell wrote:
[...]
-u32p += env->cp15.c6_rgnr;
+u32p += env-&g
On Mon, Jun 26, 2017 at 12:20 PM, Patrick Steinhardt wrote:
The function `v9fs_xattrcreate` makes use of the define `XATTR_SIZE_MAX`
to reject attempts of creating xattrs with an invalid size, which is
defined in . On glibc-based systems, this header is
indirectly included via , ,
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
sorry, I missed them in my review :(
hw/misc/mps2-scc.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/misc/mps2-scc.c b/hw/misc/mps2-scc.c
index cc58d26f29..32be2a9df1 100644
--- a/hw/misc/mps2-scc.c
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
tests/docker/dockerfiles/centos6.docker | 10 ++
1 file changed, 10 insertions(+)
diff --git a/tests/docker/dockerfiles/centos6.docker
b/tests/docker/dockerfiles/centos6.docker
index 8588a12eab..f6aae13f29 100644
--- a
is available on
DockerHub and the problem reported on [1] is using SUSE SLE11.
Regards,
Phil.
[1] http://lists.nongnu.org/archive/html/qemu-devel/2017-07/msg08993.html
Philippe Mathieu-Daudé (4):
docker: use one package per line in CentOS config
docker: add Xen libs to centos6 image
docker
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
tests/docker/dockerfiles/centos7.docker | 31 +++
1 file changed, 31 insertions(+)
create mode 100644 tests/docker/dockerfiles/centos7.docker
diff --git a/tests/docker/dockerfiles/centos7.docker
b
Hi Alistair,
On 07/28/2017 07:16 PM, Alistair Francis wrote:
Convert any remaining uses of fprintf(stderr, "warning:"...
to use warn_report() instead. This helps standardise on a single
method of printing warnings to the user.
All of the warnings were changed using this command:
find ./*
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
tests/docker/dockerfiles/centos6.docker | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tests/docker/dockerfiles/centos6.docker
b/tests/docker/dockerfiles/centos6.docker
index 9b91e832c2..8588a12eab
I'm a bit lost with this error:
(master)$ make check-help V=1
cc -nostdlib -o check-help.mo
cc: fatal error: no input files
compilation terminated.
rules.mak:115: recipe for target 'check-help.mo' failed
make: *** [check-help.mo] Error 1
Phil.
On 07/28/2017 08:50 AM, Paolo Bonzini wrote:
On 28/07/2017 08:24, Fam Zheng wrote:
On Fri, 07/28 02:36, Philippe Mathieu-Daudé wrote:
diff --git a/MAINTAINERS b/MAINTAINERS
index 224890643d..7854792765 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1537,6 +1537,12 @@ Checkpatch
S: Odd Fixes
With the move of some docs to docs/interop on ac06724a71, a couple of references
were not updated.
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 +-
With the move of some docs/ to docs/devel/ on ac06724a71, no references were
updated.
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Reviewed-by: Stefan Hajnoczi <stefa...@redhat.com>
---
audio/trace-events | 2 +-
block/trace-events | 2 +-
c
When this file was rewritten/renamed in fdee2025dd, a reference path was not
updated.
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
sed -e 's/\(.*\)\.$/\1/p' \
| sort -u \
| xargs ls -d >/dev/null
Regards,
Phil.
Cleber Rosa (1):
docs: fix broken paths to docs/interop dir
Philippe Mathieu-Daudé (7):
docs: fix broken paths to docs/interop/qcow2.txt
docs: fix broken paths to docs/devel/qapi-code-gen.txt
docs: fi
With the move of some docs/ to docs/devel/ on ac06724a71, a couple of
references were not updated.
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
With the move of some docs/ to docs/devel/ on ac06724a71, a reference path was
not updated.
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..09df
With the move of some docs to docs/interop on d59157ea05, a reference path
was not updated.
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Reviewed-by: Kevin Wolf <kw...@redhat.com>
---
docs/qcow2-cache.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
With the move of some docs/ to docs/devel/ on ac06724a71, some references were
not updated.
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 ++--
From: Cleber Rosa <cr...@redhat.com>
With the move of some docs to docs/interop on d59157e, a couple of
references were not updated.
Signed-off-by: Cleber Rosa <cr...@redhat.com>
[PMD: fixed a typo and another reference of docs/interop/qmp-spec.txt]
Signed-off-by: Philippe Mathi
This ease rebase/cherry-pick, also it is faster to visually find if a package
is here.
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
tests/docker/dockerfiles/centos6.docker | 18 ++
1 file changed, 14 insertions(+), 4 deletions(-)
diff --git a/tests/
On 07/28/2017 08:56 PM, Alistair Francis wrote:
On Thu, Jul 27, 2017 at 10:36 PM, Philippe Mathieu-Daudé
<f4...@amsat.org> wrote:
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
RFC because I'm not sure m25p80 fits.
MAINTAINERS | 2 ++
1 file changed, 2 insertions(+)
On 08/08/2017 05:00 PM, Eric Blake wrote:
On 08/08/2017 01:32 PM, John Snow wrote:
Out with the old, in with the new.
Signed-off-by: John Snow
---
hw/ide/piix.c | 11
hw/ide/trace-events | 33
hw/ide/via.c
Hi Stefan,
On 08/09/2017 07:16 AM, Stefan Hajnoczi wrote:
[...]> Python scripts needed to build QEMU are the highest priority. They
are invoked by ./configure or make. I've identified the following:
scripts/signrom.py
scripts/qapi*.py
scripts/modules/module_block.py
scripts/tracetool*
Hi Andrew,
On 07/31/2017 10:04 PM, Andrew Jeffery wrote:
The reset width register controls how the pulse on the SoC's WDTRST{1,2}
pins behaves. A pulse is emitted if the external reset bit is set in
WDT_CTRL. WDT_RESET_WIDTH requires magic bit patterns to configure both
push-pull/open-drain and
Hi Laurent,
On Wed, Aug 2, 2017 at 7:32 AM, Laurent Vivier wrote:
> With pseries machine type a negative core-id is not managed properly:
> -1 gives an inaccurate error message ("core -1 already populated"),
> -2 crashes QEMU (core dump)
>
> As it seems a negative value is
701 - 800 of 40268 matches
Mail list logo