Patchew URL: https://patchew.org/QEMU/20190916142246.31474-1-mre...@redhat.com/
Hi,
This series failed the docker-quick@centos7 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 ===
#
On Wed, Sep 04, 2019 at 12:50:21PM +0100, Peter Maydell wrote:
> Hi; I've been seeing intermittently on the BSDs this assertion
> running test-aio-multithread as part of 'make check':
>
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> tests/test-aio-multithread -m=quick -k --ta
The M48T59 is a Real Time Clock, not a timer.
Move it under the hw/rtc/ subdirectory.
Signed-off-by: Philippe Mathieu-Daudé
---
MAINTAINERS | 4 +-
hw/ppc/ppc405_boards.c | 2 +-
hw/ppc/prep.c | 2 +-
hw/rtc/Kconfig
On Mon, Sep 16, 2019 at 11:28:23AM +0200, Philippe Mathieu-Daudé wrote:
> Hi David,
>
> On 9/16/19 2:42 AM, David Gibson wrote:
> > On Sun, Sep 15, 2019 at 11:19:34PM +0200, Philippe Mathieu-Daudé wrote:
> >> Quick tests worth to avoid regressions with the 40p machine.
> >> idea from the "Maintain
Move RTC devices under the hw/rtc/ subdirectory.
Signed-off-by: Philippe Mathieu-Daudé
---
MAINTAINERS | 4 ++--
hw/rtc/Kconfig| 3 +++
hw/rtc/Makefile.objs | 1 +
hw/{timer => rtc}/sun4v-rtc.c | 2 +-
hw/rtc/trace-events | 4
hw/spa
The MC146818 is a Real Time Clock, not a timer.
Move it under the hw/rtc/ subdirectory.
Signed-off-by: Philippe Mathieu-Daudé
---
MAINTAINERS | 4 +--
hw/alpha/dp264.c | 2 +-
hw/hppa/machine.c| 2 +-
hw/
If not handled, QEMU will execve itself instead of the emulated
process. The function do_openat already solves the /proc/self/exe
problem, so we can use it to get the executable file descriptor. We
then make a safe call to execveat.
Note that safe_execve has been replaced by safe_execveat, since
The M41T80 is a Real Time Clock, not a timer.
Move it under the hw/rtc/ subdirectory.
Signed-off-by: Philippe Mathieu-Daudé
---
MAINTAINERS| 2 +-
hw/rtc/Kconfig | 4
hw/rtc/Makefile.objs | 1 +
hw/{timer => rtc}/m41t80.c | 0
hw/timer/Kconfig | 4
The TWL92230 is an "energy management device" companion with
a RTC. Since we mostly model the RTC, move it under the hw/rtc/
subdirectory.
Signed-off-by: Philippe Mathieu-Daudé
---
MAINTAINERS | 2 +-
hw/rtc/Kconfig | 4
hw/rtc/Makefile.objs | 1 +
hw/
The DS1338 is a Real Time Clock, not a timer.
Move it under the hw/rtc/ subdirectory.
Signed-off-by: Philippe Mathieu-Daudé
---
hw/rtc/Kconfig | 4
hw/rtc/Makefile.objs | 1 +
hw/{timer => rtc}/ds1338.c | 0
hw/timer/Kconfig | 4
hw/timer/Makefile.objs |
On 9/16/19 12:40 PM, Philippe Mathieu-Daudé wrote:
> On 9/16/19 6:36 PM, Eric Blake wrote:
>> On 9/16/19 4:51 AM, Philippe Mathieu-Daudé wrote:
>>> While the tracing frawework does not forbid trailing newline in
>>
>> framework
>>
>>> events format string, using them lead to confuse output.
>>>
Only 2 source files require the "mc146818rtc_regs.h" header.
Instead of having it processed 12 times, by all objects
using "mc146818rtc.h", include it directly where used.
Signed-off-by: Philippe Mathieu-Daudé
---
hw/rtc/mc146818rtc.c | 1 +
hw/timer/hpet.c | 1 +
include/hw
On 9/16/19 6:36 PM, Eric Blake wrote:
> On 9/16/19 4:51 AM, Philippe Mathieu-Daudé wrote:
>> While the tracing frawework does not forbid trailing newline in
>
> framework
>
>> events format string, using them lead to confuse output.
>> It is the responsibility of the backend to properly end an ev
The "hw/ptimer.h" header is not used, remove it.
Signed-off-by: Philippe Mathieu-Daudé
---
hw/rtc/xlnx-zynqmp-rtc.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/hw/rtc/xlnx-zynqmp-rtc.c b/hw/rtc/xlnx-zynqmp-rtc.c
index f9f09b7296..2bcd14d779 100644
--- a/hw/rtc/xlnx-zynqmp-rtc.c
+++ b/hw/r
* Changes from v1
- Introduce the patch as a bug fix, rather than a security fix
- Use do_openat and safe_execveat instead of copying exec_path
- Extensive test case example
* Test case
I will present a short program that demonstrated the bug, i.e. what
is the expected behavior and w
On 9/16/19 6:08 PM, Cleber Rosa wrote:
> On Sun, Sep 15, 2019 at 11:19:35PM +0200, Philippe Mathieu-Daudé wrote:
>> As of this commit, NetBSD 4.0 is very old. However it is enough to
>> test the PRep/40p machine.
>>
>> User case from:
>> http://mail-index.netbsd.org/port-prep/2017/04/11/msg000112.h
On 9/16/19 10:56 AM, Vladimir Sementsov-Ogievskiy wrote:
> 12.09.2019 3:16, John Snow wrote:
>> Like script_main, but doesn't require a single point of entry.
>> Replace all existing initialization sections with this drop-in replacement.
>>
>> This brings debug support to all existing script-sty
On 9/16/19 4:51 AM, Philippe Mathieu-Daudé wrote:
> While the tracing frawework does not forbid trailing newline in
framework
> events format string, using them lead to confuse output.
> It is the responsibility of the backend to properly end an event
> line.
Why just trailing newline? Should we
Patchew URL:
https://patchew.org/QEMU/20190916155545.29928-1-olivier.d...@polymtl.ca/
Hi,
This series failed build test on FreeBSD host. Please find the details below.
=== TEST SCRIPT BEGIN ===
#!/bin/bash
# Testing script will be invoked under the git checkout with
# HEAD pointing to a commi
On 9/16/19 4:00 AM, Kevin Wolf wrote:
> Am 13.09.2019 um 20:49 hat John Snow geschrieben:
>> On 9/12/19 4:20 AM, Vladimir Sementsov-Ogievskiy wrote:
>>> Also, I'm not sure about "are" suggested by Max. "are" is for plural, but
>>> here I meant
>>> one object: sum of @start and @count.
>>>
>>
>>
16.09.2019 19:32, John Snow wrote:
>
>
> On 9/16/19 10:56 AM, Vladimir Sementsov-Ogievskiy wrote:
>> 12.09.2019 3:16, John Snow wrote:
>>> Like script_main, but doesn't require a single point of entry.
>>> Replace all existing initialization sections with this drop-in replacement.
>>>
>>> This br
The same utility method is already present in two different test
files, so let's consolidate it into a single utility function.
Signed-off-by: Cleber Rosa
---
tests/acceptance/avocado_qemu/__init__.py | 26 +
tests/acceptance/boot_linux_console.py| 47 +++
tes
From: "Paul A. Clarke"
ISA 3.0B added a set of Floating-Point Status and Control Register (FPSCR)
instructions: mffsce, mffscdrn, mffscdrni, mffscrn, mffscrni, mffsl.
This patch adds support for 'mffsce' instruction.
'mffsce' is identical to 'mffs', except that it also clears the exception
enabl
From: "Paul A. Clarke"
ISA 3.0B added a set of Floating-Point Status and Control Register (FPSCR)
instructions: mffsce, mffscdrn, mffscdrni, mffscrn, mffscrni, mffsl.
This patch adds support for 'mffscrn' and 'mffscrni' instructions.
'mffscrn' and 'mffscrni' are similar to 'mffsl', except they d
On Sun, Sep 15, 2019 at 6:07 AM Bin Meng wrote:
>
> Hi Palmer,
>
> On Sun, Sep 15, 2019 at 3:00 AM Palmer Dabbelt wrote:
> >
> > On Fri, 13 Sep 2019 08:25:21 PDT (-0700), bmeng...@gmail.com wrote:
> > > Hi Palmer,
> > >
> > > On Fri, Sep 13, 2019 at 10:33 PM Palmer Dabbelt wrote:
> > >>
> > >> O
On Fri, 13 Sep 2019 11:14:43 +0200
Thomas Huth wrote:
Modified the subject, as suggested by David.
> Since QEMU v2.10, the KVM acceleration does not work on older kernels
> anymore since the code accidentally requires the KVM_CAP_DEVICE_CTRL
> capability now - it should have been optional instea
On 9/16/19 12:39 PM, Vladimir Sementsov-Ogievskiy wrote:
> 16.09.2019 19:32, John Snow wrote:
>>
>>
>> On 9/16/19 10:56 AM, Vladimir Sementsov-Ogievskiy wrote:
>>> 12.09.2019 3:16, John Snow wrote:
Like script_main, but doesn't require a single point of entry.
Replace all existing init
On 16/09/2019 17:48, Philippe Mathieu-Daudé wrote:
> Move RTC devices under the hw/rtc/ subdirectory.
>
> Signed-off-by: Philippe Mathieu-Daudé
I suppose the removal of the header files in "aspeed_rtc.h" is OK.
Reviewed-by: Cédric Le Goater
Thanks,
C.
> ---
> hw/rtc/Makefile.objs
On Mon, Sep 16, 2019 at 9:02 AM Philippe Mathieu-Daudé
wrote:
>
> All these devices do not contain any target-specific. While most
> of them are arch-specific, they are shared between different
> targets of the same arch family (ARM and AArch64, MIPS32/MIPS64,
> endianess, ...).
> Put them into co
Common interface for aio task loops. To be used for improving
performance of synchronous io loops in qcow2, block-stream,
copy-on-read, and may be other places.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: Max Reitz
---
include/block/aio_task.h | 54 +
block/aio_tas
Upcoming asynchronous handling of sub-parts of qcow2 requests will
change number of leaked clusters and even make it racy. As a
preparation, ignore leaks on failure parts in 026.
It's not trivial to just grep or substitute qemu-img output for such
thing. Instead do better: 3 is a error code of qem
On 9/16/19 10:19 AM, Vladimir Sementsov-Ogievskiy wrote:
> Drop meta bitmaps, as they are unused.
>
> Signed-off-by: Vladimir Sementsov-Ogievskiy
believe it or not, I had a local patch that does the same :)
Reviewed-by: John Snow
Similarly to previous commit, prepare for parallelizing write-loop
iterations.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: Max Reitz
---
block/qcow2.c | 154 +-
1 file changed, 90 insertions(+), 64 deletions(-)
diff --git a/block/qco
Further patch will run partial requests of iterations of
qcow2_co_preadv in parallel for performance reasons. To prepare for
this, separate part which may be parallelized into separate function
(qcow2_co_preadv_task).
While being here, also separate encrypted clusters reading to own
function, like
On Sun, Sep 15, 2019 at 11:19:36PM +0200, Philippe Mathieu-Daudé wrote:
> User case from:
> https://tyom.blogspot.com/2019/04/aixprep-under-qemu-how-to.html
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> tests/acceptance/ppc_prep_40p.py | 21 +
> 1 file changed, 21 inserti
It improves performance for fragmented qcow2 images.
Signed-off-by: Vladimir Sementsov-Ogievskiy
---
block/qcow2.h | 3 ++
block/qcow2.c | 125 -
block/trace-events | 1 +
3 files changed, 117 insertions(+), 12 deletions(-)
diff --git a/
Hi all!
Here is an asynchronous scheme for handling fragmented qcow2
reads and writes. Both qcow2 read and write functions loops through
sequential portions of data. The series aim it to parallelize these
loops iterations.
It improves performance for fragmented qcow2 images, I've tested it
as desc
On Mon, Sep 16, 2019 at 9:06 AM Philippe Mathieu-Daudé
wrote:
>
> The PL031 is a Real Time Clock, not a timer.
> Move it under the hw/rtc/ subdirectory.
>
> Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
Alistair
> ---
> MAINTAINERS | 4 ++--
> Make
* Balamuruhan S (bal...@linux.ibm.com) wrote:
> assert `query-migrate` in target doesn't give migration
> status and test errors even if migration succeeds.
>
> In target:
> {'execute': 'query-migrate'}
> {"return": {}}
On which version of qemu?
On the current version I see:
{"QMP": {"version":
On Mon, Sep 16, 2019 at 01:59:22PM -0400, Cleber Rosa wrote:
> On Sun, Sep 15, 2019 at 11:19:36PM +0200, Philippe Mathieu-Daudé wrote:
> > User case from:
> > https://tyom.blogspot.com/2019/04/aixprep-under-qemu-how-to.html
> >
> > Signed-off-by: Philippe Mathieu-Daudé
> > ---
> > tests/acceptan
Hi
On Mon, Sep 16, 2019 at 5:15 PM Dr. David Alan Gilbert
wrote:
>
> * Marc-André Lureau (marcandre.lur...@gmail.com) wrote:
> > Hi
> >
> > On Mon, Sep 16, 2019 at 2:02 PM Dr. David Alan Gilbert
> > wrote:
> > >
> > > (Copying in Stefan since he was looking at DBus for virtiofs)
> > >
> > > * Ma
On Sun, Sep 15, 2019 at 11:19:37PM +0200, Philippe Mathieu-Daudé wrote:
> User case from:
> https://mail.coreboot.org/pipermail/openbios/2018-May/010360.html
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> tests/acceptance/ppc_prep_40p.py | 32
> 1 file changed
On 9/12/19 1:37 AM, Sergio Lopez wrote:
>> I tried to test this patch, but even with it applied, I still got an
>> aio-context crasher by attempting an nbd-server-start, nbd-server-add,
>> nbd-server-stop (intentionally skipping the nbd-server-remove step) on a
>> domain using iothreads, with a ba
Hi,
I'm trying to use qemu-user-static to chroot into a foreign amd64
environment from my ppc64le host.
The host has a 64k page size, while x86_64 uses 4k.
I get those errors while loading shared libraries: "ELF load command
alignment not page-aligned"
Is there any way to fix this? I cannot sim
On Mon, Sep 16, 2019 at 9:31 AM Philippe Mathieu-Daudé
wrote:
>
> The MC146818 is a Real Time Clock, not a timer.
> Move it under the hw/rtc/ subdirectory.
>
> Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
Alistair
> ---
> MAINTAINERS |
On Mon, Sep 16, 2019 at 9:26 AM Philippe Mathieu-Daudé
wrote:
>
> The M48T59 is a Real Time Clock, not a timer.
> Move it under the hw/rtc/ subdirectory.
>
> Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
Alistair
> ---
> MAINTAINERS | 4 +-
> hw
On Mon, Sep 16, 2019 at 9:35 AM Philippe Mathieu-Daudé
wrote:
>
> The M41T80 is a Real Time Clock, not a timer.
> Move it under the hw/rtc/ subdirectory.
>
> Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
Alistair
> ---
> MAINTAINERS| 2 +-
> hw/rtc/Kconf
On Mon, Sep 16, 2019 at 9:30 AM Philippe Mathieu-Daudé
wrote:
>
> Move RTC devices under the hw/rtc/ subdirectory.
>
> Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
Alistair
> ---
> MAINTAINERS | 4 ++--
> hw/rtc/Kconfig| 3 +++
> hw/
On Tue, Sep 10, 2019 at 1:16 AM KONRAD Frederic
wrote:
>
> While debugging an application with GDB the following might happen:
>
> (gdb) return
> Make xxx return now? (y or n) y
> Could not fetch register "fflags"; remote failure reply 'E14'
>
> This is because riscv_gdb_get_fpu calls riscv_csrrw_
On Mon, Sep 16, 2019 at 9:37 AM Philippe Mathieu-Daudé
wrote:
>
> The TWL92230 is an "energy management device" companion with
> a RTC. Since we mostly model the RTC, move it under the hw/rtc/
> subdirectory.
>
> Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
Alistair
> -
On Mon, Sep 16, 2019 at 9:39 AM Philippe Mathieu-Daudé
wrote:
>
> The DS1338 is a Real Time Clock, not a timer.
> Move it under the hw/rtc/ subdirectory.
>
> Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
Alistair
> ---
> hw/rtc/Kconfig | 4
> hw/rtc/Mak
On Mon, Sep 16, 2019 at 8:56 AM Philippe Mathieu-Daudé
wrote:
>
> Move RTC devices under the hw/rtc/ subdirectory.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> hw/rtc/Makefile.objs| 1 +
> hw/rtc/trace-events | 3 +++
> hw/{timer => rtc}/xlnx-
On Mon, Sep 16, 2019 at 9:00 AM Philippe Mathieu-Daudé
wrote:
>
> Move RTC devices under the hw/rtc/ subdirectory.
>
> Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
Alistair
> ---
> hw/rtc/Makefile.objs | 1 +
> hw/{timer => rtc}/exynos4210_rtc.c | 0
> hw
On Mon, Sep 16, 2019 at 9:42 AM Philippe Mathieu-Daudé
wrote:
>
> Only 2 source files require the "mc146818rtc_regs.h" header.
> Instead of having it processed 12 times, by all objects
> using "mc146818rtc.h", include it directly where used.
>
> Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by:
On Mon, Sep 16, 2019 at 9:17 AM Philippe Mathieu-Daudé
wrote:
>
> Move RTC devices under the hw/rtc/ subdirectory.
>
> Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
Alistair
> ---
> hw/rtc/Makefile.objs | 1 +
> hw/{timer => rtc}/aspeed_rtc.c |
On Mon, Sep 16, 2019 at 9:46 AM Philippe Mathieu-Daudé
wrote:
>
> The "hw/ptimer.h" header is not used, remove it.
>
> Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
Alistair
> ---
> hw/rtc/xlnx-zynqmp-rtc.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/hw/rtc/
On 9/12/19 6:31 AM, Kevin Wolf wrote:
>>
>> Yes, I think locking the context during the "if (exp->blk) {" block at
>> nbd/server.c:1646 should do the trick.
That line number has moved over time; which function are you referring to?
>
> We need to be careful to avoid locking things twice, so may
On Mon, Sep 16, 2019 at 11:56:06AM +0200, Philippe Mathieu-Daudé wrote:
> On 9/16/19 11:52 AM, Alex Bennée wrote:
> >
> > Philippe Mathieu-Daudé writes:
> >
> >> Hi David,
> >>
> >> On 9/16/19 2:42 AM, David Gibson wrote:
> >>> On Sun, Sep 15, 2019 at 11:19:34PM +0200, Philippe Mathieu-Daudé wro
On Mon, Sep 16, 2019 at 08:22:42AM +0100, Alex Bennée wrote:
>
> David Gibson writes:
>
> > On Wed, Sep 11, 2019 at 10:33:45AM -0400, Richard Henderson wrote:
> >> On 9/11/19 5:39 AM, Alex Bennée wrote:
> >> > We were incorrectly setting NIP resulting in a segfault. This fixes
> >> > linux-test
When iothreads are in use, the failure to grab the aio context results
in an assertion failure when trying to unlock things during blk_unref,
when trying to unlock a mutex that was not locked. In short, all
calls to nbd_export_put need to done while within the correct aio
context. But since nbd_e
On Mon, Sep 16, 2019 at 05:48:37PM +0200, Philippe Mathieu-Daudé wrote:
> The MC146818 is a Real Time Clock, not a timer.
> Move it under the hw/rtc/ subdirectory.
>
> Signed-off-by: Philippe Mathieu-Daudé
ppc parts
Acked-by: David Gibson
> ---
> MAINTAINERS
On Mon, Sep 16, 2019 at 05:48:38PM +0200, Philippe Mathieu-Daudé wrote:
> The M48T59 is a Real Time Clock, not a timer.
> Move it under the hw/rtc/ subdirectory.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> MAINTAINERS | 4 +-
> hw/ppc/ppc405_boards.c |
On 16/09/2019 17.48, Philippe Mathieu-Daudé wrote:
> All these devices do not contain any target-specific. While most
> of them are arch-specific, they are shared between different
> targets of the same arch family (ARM and AArch64, MIPS32/MIPS64,
> endianess, ...).
> Put them into common-obj-y to
On 16/09/2019 17.48, Philippe Mathieu-Daudé wrote:
> The MC146818 is a Real Time Clock, not a timer.
> Move it under the hw/rtc/ subdirectory.
>
> Signed-off-by: Philippe Mathieu-Daudé
[...]
> diff --git a/include/hw/rtc/mc146818rtc.h b/include/hw/rtc/mc146818rtc.h
> new file mode 100644
> index
201 - 264 of 264 matches
Mail list logo