On 17.09.19 16:25, Andrey Shinkevich wrote:
>
>
> On 17/09/2019 12:20, Max Reitz wrote:
>> null-aio may not be whitelisted. Skip all test cases that require it.
>>
>> (And skip the whole test if null-co is not whitelisted.)
> This solution has been persisting since v1. Nevertheless, I am still n
On 2019/9/18 上午9:44, Tian, Kevin wrote:
From: Jason Wang [mailto:jasow...@redhat.com]
Sent: Tuesday, September 17, 2019 6:36 PM
On 2019/9/17 下午4:48, Tian, Kevin wrote:
From: Jason Wang [mailto:jasow...@redhat.com]
Sent: Monday, September 16, 2019 4:33 PM
On 2019/9/16 上午9:51, Tian, Kevin wro
On 2019/9/18 上午9:31, Tian, Kevin wrote:
From: Alex Williamson [mailto:alex.william...@redhat.com]
Sent: Tuesday, September 17, 2019 10:54 PM
On Tue, 17 Sep 2019 08:48:36 +
"Tian, Kevin" wrote:
From: Jason Wang [mailto:jasow...@redhat.com]
Sent: Monday, September 16, 2019 4:33 PM
On 20
Hi Stefan,
>
> Hi,
> LWN.net is a popular open source news site that covers Linux and other
> open source communities (Python, GNOME, Debian, etc). It has published
> a few KVM articles in the past too.
>
> Let's raise awareness of QEMU, KVM, and libvirt by submitting articles
> covering
> KV
On Tue, Sep 17, 2019 at 01:54:24PM +0200, Cédric Le Goater wrote:
> On 31/07/2019 16:12, Cédric Le Goater wrote:
> > Hello,
> >
> > The QEMU PowerNV machine emulates a baremetal OpenPOWER system and
> > acts as an hypervisor (L0). Supporting emulation of KVM to run guests
> > (L1) requires a few m
The memory_region_tb_read tracepoint is unreachable, since notdirty
is supposed to apply only to reads. The memory_region_tb_write
tracepoint is mis-named, because notdirty is not only used for TB
invalidation. It is also used for e.g. VGA RAM updates.
Replace memory_region_tb_write with memory_
RFC because this doesn't work, and I don't quite understand why.
The only failing test is {i386,x86_64} pxe-test -- the other
migration tests that use notdirty all pass.
Note that if you try to reproduce this on x86, you'll likely
have to --disable-kvm, as otherwise the pxe-test will skip tcg.
An
This macro no longer has a non-empty definition.
Signed-off-by: Richard Henderson
---
accel/tcg/atomic_template.h | 12
accel/tcg/cputlb.c | 1 -
accel/tcg/user-exec.c | 1 -
3 files changed, 14 deletions(-)
diff --git a/accel/tcg/atomic_template.h b/accel/tcg/atom
Pages that we want to track for NOTDIRTY are RAM. We do not
really need to go through the I/O path to handle them.
Create cpu_notdirty_write() from the corpses of
memory_notdirty_write_prepare and memory_notdirty_write_complete.
Use this new function to implement all of the notdirty handling.
Th
Stefan Hajnoczi writes:
> Hi,
> LWN.net is a popular open source news site that covers Linux and other
> open source communities (Python, GNOME, Debian, etc). It has published
> a few KVM articles in the past too.
>
> Let's raise awareness of QEMU, KVM, and libvirt by submitting articles
> cov
Alex Bennée writes:
> You will need to update gdb-xml/$ARCH.xml and the appropriate
arch_read/write register functions in gdbstub.
For some mysterious reason my previous emails didn't go to the list
successfully. Let me try sending this one without including any code or link.
The MIPS targe
On Tue, Sep 17, 2019 at 04:49:56PM -0500, Paul Clarke wrote:
> On 9/17/19 3:46 PM, Richard Henderson wrote:
> > On 9/16/19 1:02 PM, Paul A. Clarke wrote:
> >> From: "Paul A. Clarke"
> >>
> >> ISA 3.0B added a set of Floating-Point Status and Control Register (FPSCR)
> >> instructions: mffsce, mffs
On Tue, Sep 17, 2019 at 10:44:25AM +0100, Dr. David Alan Gilbert wrote:
> * Balamuruhan S (bal...@linux.ibm.com) wrote:
> > On Mon, Sep 16, 2019 at 03:50:06PM +0100, Dr. David Alan Gilbert wrote:
> > > * Balamuruhan S (bal...@linux.ibm.com) wrote:
> > > > add migration test to query machine types s
On Thu, 09/12 19:07, John Snow wrote:
>
>
> On 9/11/19 9:52 PM, no-re...@patchew.org wrote:
> > Patchew URL:
> > https://patchew.org/QEMU/20190912014442.5757-1-js...@redhat.com/
> >
> >
> >
> > Hi,
> >
> > This series seems to have some coding style problems. See output below for
> > more in
> From: Tian, Kevin
> Sent: Wednesday, September 18, 2019 9:32 AM
>
> > From: Alex Williamson [mailto:alex.william...@redhat.com]
> > Sent: Tuesday, September 17, 2019 10:54 PM
> >
> > On Tue, 17 Sep 2019 08:48:36 +
> > "Tian, Kevin" wrote:
> >
> > > > From: Jason Wang [mailto:jasow...@redhat
On 9/17/19 4:22 PM, Alistair Francis wrote:
> Palmer Dabbelt (2):
> RISC-V: Handle bus errors in the page table walker
> RISC-V: Implement cpu_do_transaction_failed
Reviewed-by: Richard Henderson
r~
I went through the uses of mie in the entire hypervisor patch series and it
seems like it would be much simpler to just have two non-pointer fields in
the CPU struct: mie and vsie. To if an interrupt is pending, you are either
running with V=0 in which case the contents of vsie can be ignored, or y
> From: Jason Wang [mailto:jasow...@redhat.com]
> Sent: Tuesday, September 17, 2019 6:36 PM
>
> On 2019/9/17 下午4:48, Tian, Kevin wrote:
> >> From: Jason Wang [mailto:jasow...@redhat.com]
> >> Sent: Monday, September 16, 2019 4:33 PM
> >>
> >>
> >> On 2019/9/16 上午9:51, Tian, Kevin wrote:
> >>> Hi,
On 2019/9/18 0:00, Dr. David Alan Gilbert (git) wrote:
> From: "Dr. David Alan Gilbert"
>
> The virtio-fs virtio device provides shared file system access using
> the FUSE protocol carried over virtio.
> The actual file server is implemented in an external vhost-user-fs device
> backend proces
> From: Alex Williamson [mailto:alex.william...@redhat.com]
> Sent: Tuesday, September 17, 2019 10:54 PM
>
> On Tue, 17 Sep 2019 08:48:36 +
> "Tian, Kevin" wrote:
>
> > > From: Jason Wang [mailto:jasow...@redhat.com]
> > > Sent: Monday, September 16, 2019 4:33 PM
> > >
> > >
> > > On 2019/9/
On 2019/9/18 0:00, Dr. David Alan Gilbert (git) wrote:
> From: "Dr. David Alan Gilbert"
>
> Add the PCI version of vhost-user-fs.
>
> Launch QEMU like this:
>
> qemu -chardev socket,path=/tmp/vhost-fs.sock,id=chr0
>-device x-vhost-user-fs-pci,tag=myfs,chardev=chr0
>
> Signed-off-b
This allow remote control of the baudrate and other comms
parameters.
Signed-off-by: Sylvain Munaut
---
chardev/char-socket.c | 232 ++
chardev/char.c| 6 ++
qapi/char.json| 3 +
3 files changed, 201 insertions(+), 40 deletions(-)
diff
On Tue, Sep 17, 2019 at 03:42:12PM +0200, Igor Mammedov wrote:
> On Tue, 17 Sep 2019 16:44:42 +0800
> Peter Xu wrote:
>
> > On Mon, Sep 16, 2019 at 09:23:47AM -0400, Igor Mammedov wrote:
> > > PS:
> > > I don't have access to a suitable system to test it.
> >
> > Hmm I feel like it would be go
On Tue, Sep 17, 2019 at 12:20:56PM +0200, Greg Kurz wrote:
> Ensure that hints are added even if errp is &error_fatal or &error_abort.
>
> Signed-off-by: Greg Kurz
Acked-by: David Gibson
> ---
> hw/ppc/mac_newworld.c |7 +--
> hw/ppc/spapr.c|7 +--
> hw/ppc/spapr_pci.c
On Tue, Sep 17, 2019 at 12:19:52PM +1000, David Gibson wrote:
> 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 wro
We can turn logging on/off globally instead of per-function.
Remove use_log from run_job, and use python logging to turn on
diffable output when we run through a script entry point.
iotest 245 changes output order due to buffering reasons.
Signed-off-by: John Snow
---
tests/qemu-iotests/030
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-style iotests.
Any specification for supported_oses=['linux'] was dropped as explained
in the previous commit
verify_platform will check an explicit whitelist and blacklist instead.
The default will now be assumed to be allowed to run anywhere.
For tests that do not specify their platforms explicitly, this has the effect of
enabling these tests on non-linux platforms. For tests that always specified
linux
Since this one is nicely factored to use a single entry point,
use script_main to run the tests.
Signed-off-by: John Snow
Reviewed-by: Vladimir Sementsov-Ogievskiy
---
tests/qemu-iotests/258 | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/tests/qemu-iotests/258 b/tes
Any one of the iotests.main, iotests.script_main, or
iotests.script_initialize functions can specify verify_protocols.
Remove the last users of calling the function individually.
Signed-off-by: John Snow
Reviewed-by: Vladimir Sementsov-Ogievskiy
---
tests/qemu-iotests/207 | 4 ++--
tests/qemu-
This series uses python logging to enable output conditionally on
iotests.log(). We unify an initialization call (which also enables
debugging output for those tests with -d) and then make the switch
inside of iotests.
It will help alleviate the need to create logged/unlogged versions
of all the v
On Tue, Sep 17, 2019 at 07:10:13PM -0400, John Snow wrote:
> On 9/17/19 5:48 PM, Eduardo Habkost wrote:
> > On Tue, Sep 17, 2019 at 03:57:26PM +0200, Kevin Wolf wrote:
> >> Am 11.06.2019 um 19:12 hat Eduardo Habkost geschrieben:
> >>> On Tue, Jun 11, 2019 at 05:07:55PM +0100, Peter Maydell wrote:
>
On Wed, Sep 11, 2019 at 7:55 AM Jonathan Behrens wrote:
>
> Version 0.4 of the hypervisor spec no longer talks about swapping registers.
> Instead when running in VS-mode some of the supervisor registers are
> "aliased" and actually refer to alternate versions. Implementations are of
> course s
On Tue, Sep 17, 2019 at 3:28 AM Philippe Mathieu-Daudé
wrote:
>
> Hi Alistair,
>
> On 9/16/19 11:44 PM, Alistair Francis wrote:
> > 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 Mathie
On Mon, Jul 01, 2019 at 07:25:27PM -0300, Eduardo Habkost wrote:
> On Mon, Jun 10, 2019 at 01:58:50PM +0100, Peter Maydell wrote:
> > On Fri, 7 Jun 2019 at 22:16, Eduardo Habkost wrote:
> > >
> > > The following changes since commit
> > > 185b7ccc11354cbd69b6d53bf8d831dd964f6c88:
> > >
> > > Me
From: Palmer Dabbelt
We directly access physical memory while walking the page tables on
RISC-V, but while doing so we were using cpu_ld*() which does not report
bus errors. This patch converts the page table walker over to use
address_space_ld*(), which allows bus errors to be detected.
Signed
The do_unassigned_access hook has been deprecated and RISC-V is the last
user of it. Let's instead update the RISC-V implementation to use
do_transaction_failed instead.
After this series I used the 'git grep' regexes in
docs/devel/loads-stores.rst and these are the memory accesses inside
target/r
From: Palmer Dabbelt
This converts our port over from cpu_do_unassigned_access to
cpu_do_transaction_failed, as cpu_do_unassigned_access has been
deprecated.
Signed-off-by: Palmer Dabbelt
Signed-off-by: Alistair Francis
---
target/riscv/cpu.c| 2 +-
target/riscv/cpu.h| 7 +++
On 9/17/19 5:48 PM, Eduardo Habkost wrote:
> On Tue, Sep 17, 2019 at 03:57:26PM +0200, Kevin Wolf wrote:
>> Am 11.06.2019 um 19:12 hat Eduardo Habkost geschrieben:
>>> On Tue, Jun 11, 2019 at 05:07:55PM +0100, Peter Maydell wrote:
On Tue, 11 Jun 2019 at 17:03, Eduardo Habkost 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 brings debug support to all existing script-sty
On 9/17/19 3:46 PM, Richard Henderson wrote:
> On 9/16/19 1:02 PM, Paul A. Clarke wrote:
>> 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
On 9/17/19 3:45 PM, Richard Henderson wrote:
> On 9/16/19 1:02 PM, Paul A. Clarke wrote:
>> +#define FP_DRN2 (1ull << FPSCR_DRN2)
>> +#define FP_DRN1 (1ull << FPSCR_DRN1)
>> +#define FP_DRN0 (1ull << FPSCR_DRN0)
>> +#define FP_DRN (FP_DRN2 | FP_DRN1 | FP_DRN0)
>
>
On Tue, Sep 17, 2019 at 03:57:26PM +0200, Kevin Wolf wrote:
> Am 11.06.2019 um 19:12 hat Eduardo Habkost geschrieben:
> > On Tue, Jun 11, 2019 at 05:07:55PM +0100, Peter Maydell wrote:
> > > On Tue, 11 Jun 2019 at 17:03, Eduardo Habkost wrote:
> > > >
> > > > On Tue, Jun 11, 2019 at 04:50:34PM +01
On Tue, Sep 17, 2019 at 09:19:01PM +0200, Adrian Moreno wrote:
> That way the state can be correctly restored when the device is opened
> again. This might happen if the backend is restarted.
>
> Buglink: https://bugzilla.redhat.com/show_bug.cgi?id=1738768
> Reported-by: Pei Zhang
> Fixes: 6ab79a
On 9/17/19 4:57 PM, Alex Bennée wrote:
>
> Richard Henderson writes:
>
>> On 9/17/19 2:41 PM, Alex Bennée wrote:
>>> +for (i = 0; i < ARRAY_SIZE(round_flags); ++i) {
>>> +fesetround(round_flags[i].flag);
>>
>> If we're going to make this a generic test, perhaps
>> continue if fesetro
On 9/17/19 5:00 PM, Alex Bennée wrote:
>>> +add_f32_const(0x8p-152);
>>> +add_f32_const(0x8p-152);
>>> +add_f32_const(0x8p-152);
>>
>> Why are you adding 3 of the same?
>
> To replicate the 1841491 test case where the same number is used for
> a/b/c
Then we really ought to be more exp
Richard Henderson writes:
> On 9/17/19 2:41 PM, Alex Bennée wrote:
>> +/* From https://bugs.launchpad.net/qemu/+bug/1841491 */
>> +add_f32_const(0x1.cp-1022);
>> +add_f32_const(0x1.1p-1);
>> +add_f32_const(0x0.1p-1022);
>
> These three constan
Richard Henderson writes:
> On 9/17/19 2:41 PM, Alex Bennée wrote:
>> +for (i = 0; i < ARRAY_SIZE(round_flags); ++i) {
>> +fesetround(round_flags[i].flag);
>
> If we're going to make this a generic test, perhaps
> continue if fesetround fails?
I ifdef'ed the missing symbols? Can th
On 9/16/19 1:02 PM, Paul A. Clarke wrote:
> 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 t
On 9/14/19 10:35 AM, Markus Armbruster wrote:
> Entity checking goes back to commit ac88219a6c "qapi: New QAPISchema
> intermediate representation", v2.5.0. It's designed to work as
> follows: QAPISchema.check() calls .check() for all the schema's
> entities. An entity's .check() recurses into an
On 9/14/19 10:35 AM, Markus Armbruster wrote:
> All callers pass a dict argument to @keys, except check_keys() passes
> a dict's .keys(). Drop .keys() there, and rename parameter @keys to
> @value.
>
> Signed-off-by: Markus Armbruster
> ---
> scripts/qapi/common.py | 8
> 1 file change
On 9/14/19 10:35 AM, Markus Armbruster wrote:
> QAPISchemaObjectType.check() does nothing for types that have been
> checked already. Except the "has been checked" predicate is broken
> for empty types: self.members is [] then, which isn't true. The bug
> is harmless, but fix it anyway: use self.
On 9/14/19 10:35 AM, Markus Armbruster wrote:
> Easy since the previous commit provides .checked.
>
> Signed-off-by: Markus Armbruster
> ---
> scripts/qapi/common.py | 11 ++-
> 1 file changed, 10 insertions(+), 1 deletion(-)
>
Reviewed-by: Eric Blake
--
Eric Blake, Principal Softwa
On 9/16/19 1:02 PM, Paul A. Clarke wrote:
> +#define FP_DRN2 (1ull << FPSCR_DRN2)
> +#define FP_DRN1 (1ull << FPSCR_DRN1)
> +#define FP_DRN0 (1ull << FPSCR_DRN0)
> +#define FP_DRN (FP_DRN2 | FP_DRN1 | FP_DRN0)
Why not just 7ull << FPSCR_DRN?
Are the individual DRN
On 9/14/19 10:35 AM, Markus Armbruster wrote:
> check_keys() parameter expr_elem expects a dictionary with keys 'expr'
> and 'info'. Passing the two values separately is simpler, so do that.
>
> Signed-off-by: Markus Armbruster
> ---
> scripts/qapi/common.py | 19 ---
> 1 file c
On 9/14/19 10:35 AM, Markus Armbruster wrote:
> Commit bceae7697f "qapi script: support enum type as discriminator in
> union" made check_exprs() add the implicit enum types of simple unions
> to global @enum_types. I'm not sure it was needed even then. It's
> certainly not needed now. Delete it
On 9/16/19 11:48 AM, Philippe Mathieu-Daudé wrote:
> include/hw/rtc/mc146818rtc.h | 38
Same rebase failure as for patch 4?
r~
On 9/16/19 9:58 AM, David Hildenbrand wrote:
> Let's add the simple test based on the example from the PoP.
>
> Signed-off-by: David Hildenbrand
> ---
> tests/tcg/s390x/Makefile.target | 1 +
> tests/tcg/s390x/mvo.c | 25 +
> 2 files changed, 26 insertions(+)
>
On 9/16/19 9:57 AM, David Hildenbrand wrote:
> The last remaining bit for MVC is handling destructive overlaps in a
> fault-safe way.
>
> Signed-off-by: David Hildenbrand
> ---
> target/s390x/mem_helper.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
Reviewed-by: Richard Henderso
On 9/16/19 9:57 AM, David Hildenbrand wrote:
> As we are moving between address spaces, we can use access_memmove_idx()
> without checking for destructive overlaps (especially of real storage
> locations):
> "Each storage operand is processed left to right. The
> storage-operand-consistency
On 9/16/19 9:57 AM, David Hildenbrand wrote:
> Replace fast_memmove() variants by access_memmove() variants, that
> first try to probe access to all affected pages (maximum is two pages).
>
> Introduce access_get_byte()/access_set_byte(). We might be able to speed
> up memmove in special cases eve
On 9/16/19 9:57 AM, David Hildenbrand wrote:
> Replace fast_memset() by access_memset(), that first tries to probe
> access to all affected pages (maximum is two). We'll use the same
> mechanism for other types of accesses soon.
>
> Only in very rare cases (especially TLB_NOTDIRTY), we'll have to
Hello,
I submitted September 12. a bug report about wrong handling of
infinity values for m68k emulator.
https://bugs.launchpad.net/qemu/+bug/1843651
The analysis I made in the bug report is wrong, because
I did not know that m68k FPU really uses 80-bit representations internally.
It th
On 9/16/19 9:57 AM, David Hildenbrand wrote:
> ... and don't perform any move in case the length is zero.
>
> Signed-off-by: David Hildenbrand
> ---
> target/s390x/mem_helper.c | 6 ++
> 1 file changed, 6 insertions(+)
Reviewed-by: Richard Henderson
r~
On 9/16/19 9:57 AM, David Hildenbrand wrote:
> Triggered by a review comment from Richard, also MVCOS has a 32-bit
> length in 24/31-bit addressing mode. Add a new helper.
>
> Rename wrap_length() to wrap_length31().
>
> Signed-off-by: David Hildenbrand
> ---
> target/s390x/mem_helper.c | 14 ++
On 9/16/19 9:57 AM, David Hildenbrand wrote:
> Let's stay within single pages.
>
> ... and indicate cc=3 in case there is work remaining. Keep unicode
> padding simple.
>
> While reworking, properly wrap the addresses.
>
> Signed-off-by: David Hildenbrand
> ---
> target/s390x/mem_helper.c | 54
On 9/16/19 9:57 AM, David Hildenbrand wrote:
> Process max 4k bytes at a time, writing back registers between the
> accesses. The instruction is interruptible.
> "For operands longer than 2K bytes, access exceptions are not
> recognized for locations more than 2K bytes beyond the current lo
On 9/17/19 3:30 PM, Alex Bennée wrote:
>
> John Snow writes:
>
>> On 9/17/19 2:58 PM, Alex Bennée wrote:
>>>
>>> John Snow writes:
>>>
It's not used; so remove the distraction.
>>>
>>> It's a developer feature so you can run specific builds against the VM
>>> images. I certainly have us
On 9/17/19 2:41 PM, Alex Bennée wrote:
> +for (i = 0; i < ARRAY_SIZE(round_flags); ++i) {
> +fesetround(round_flags[i].flag);
If we're going to make this a generic test, perhaps
continue if fesetround fails?
r~
On 2019-09-11 16:58, Stefan Hajnoczi wrote:
The "latency" parameter wasn't covered by the documentation.
Signed-off-by: Stefan Hajnoczi
Reviewed-by: Zoltán Kővágó
---
How is this parameter related to buffer-length?
Pulseaudio being a client-server architecture is a bit different than
the
On 9/17/19 7:35 AM, Vladimir Sementsov-Ogievskiy wrote:
> 12.09.2019 3:16, John Snow wrote:
>> We can turn logging on/off globally instead of per-function.
>>
>> Remove use_log from run_job, and use python logging to turn on
>> diffable output when we run through a script entry point.
>>
>> iote
On 9/17/19 2:41 PM, Alex Bennée wrote:
> +/* From https://bugs.launchpad.net/qemu/+bug/1841491 */
> +add_f32_const(0x1.cp-1022);
> +add_f32_const(0x1.1p-1);
> +add_f32_const(0x0.1p-1022);
These three constants do not fit in float32.
> +add_f
On 9/17/19 8:41 PM, Alex Bennée wrote:
> Finger trouble in a previous clean-up inadvertently set
> DEBIAN_PARTIAL_IMAGES instead of DOCKER_PARTIAL_IMAGES.
Arf :)
> Fixes: 44d5a8bf5d2
> Signed-off-by: Alex Bennée
> ---
> tests/docker/Makefile.include | 2 +-
> 1 file changed, 1 insertion(+), 1 d
On 9/17/19 8:55 PM, John Snow wrote:
> We spelled it debian-9-mxe, but the image is debian9-mxe.
Oops I missed that, good catch.
Fixes: 44d5a8bf5d2
Reviewed-by: Philippe Mathieu-Daudé
> Signed-off-by: John Snow
> ---
> tests/docker/Makefile.include | 2 +-
> 1 file changed, 1 insertion(+), 1
John Snow writes:
> On 9/17/19 2:58 PM, Alex Bennée wrote:
>>
>> John Snow writes:
>>
>>> It's not used; so remove the distraction.
>>
>> It's a developer feature so you can run specific builds against the VM
>> images. I certainly have used it, unless you are saying it's broken now?
>>
>
> Wh
On 2019-09-11 16:58, Stefan Hajnoczi wrote:
Fixes: f0b3d811529 ("audio: -audiodev command line option: documentation")
Signed-off-by: Stefan Hajnoczi
---
qemu-options.hx | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/qemu-options.hx b/qemu-options.hx
index bbfd936d29.
On 9/17/19 12:14 PM, Peter Maydell wrote:
>> +static Property arm_cpu_gt_cntfrq_property =
>> +DEFINE_PROP_UINT64("cntfrq", ARMCPU, gt_cntfrq,
>> + (1000 * 1000 * 1000) / GTIMER_SCALE);
> I think it would be helpful to have a comment saynig what units
> thi
That way the state can be correctly restored when the device is opened
again. This might happen if the backend is restarted.
Buglink: https://bugzilla.redhat.com/show_bug.cgi?id=1738768
Reported-by: Pei Zhang
Fixes: 6ab79a20af3a (do not call vhost_net_cleanup() on running net from char
user even
On 9/17/19 7:40 AM, Peter Maydell wrote:
> On Tue, 17 Sep 2019 at 11:49, Philippe Mathieu-Daudé
> wrote:
>>
>> Cc'ing PPC folks and Laurent.
>>
>> On 9/16/19 11:06 PM, Niccolò Belli wrote:
>>> Hi,
>>> I'm trying to use qemu-user-static to chroot into a foreign amd64
>>> environment from my ppc64l
On 9/17/19 10:46 AM, Kevin Wolf wrote:
> Am 17.09.2019 um 16:39 hat Eric Blake geschrieben:
>> On 9/17/19 5:20 AM, Greg Kurz wrote:
>>> Ensure that hints are added even if errp is &error_fatal or &error_abort.
>>>
>>> Signed-off-by: Greg Kurz
>>> ---
>>> block/backup.c |7 +--
>>>
Patchew URL:
https://patchew.org/QEMU/20190917122644.15736-1-johan...@sipsolutions.net/
Hi,
This series failed the docker-mingw@fedora build test. Please find the testing
commands and
their output below. If you have Docker installed, you can probably reproduce it
locally.
=== TEST SCRIPT BEG
Pavel Dovgalyuk writes:
> The set of patches include the block-related updates
> of the record/replay icount feature:
> - application of 'snapshot' option on the file layer instead of
>the top one: command line and documentation fix
> - implementation of bdrv_snapshot_goto for blkreplay d
Patchew URL:
https://patchew.org/QEMU/20190917122644.15736-1-johan...@sipsolutions.net/
Hi,
This series failed the asan build test. Please find the testing commands and
their output below. If you have Docker installed, you can probably reproduce it
locally.
=== TEST SCRIPT BEGIN ===
#!/bin/ba
On 9/17/19 2:58 PM, Alex Bennée wrote:
>
> John Snow writes:
>
>> It's not used; so remove the distraction.
>
> It's a developer feature so you can run specific builds against the VM
> images. I certainly have used it, unless you are saying it's broken now?
>
What consumes it? I can't find
It's not used; so remove the distraction.
Signed-off-by: John Snow
---
tests/vm/Makefile.include | 2 --
1 file changed, 2 deletions(-)
diff --git a/tests/vm/Makefile.include b/tests/vm/Makefile.include
index fea348e845..61758101fa 100644
--- a/tests/vm/Makefile.include
+++ b/tests/vm/Makefile.
We spelled it debian-9-mxe, but the image is debian9-mxe.
Signed-off-by: John Snow
---
tests/docker/Makefile.include | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
index 50a400b573..7eac1516f6 100644
--- a/tests/d
John Snow writes:
> It's not used; so remove the distraction.
It's a developer feature so you can run specific builds against the VM
images. I certainly have used it, unless you are saying it's broken now?
>
> Signed-off-by: John Snow
> ---
> tests/vm/Makefile.include | 2 --
> 1 file chang
On 9/17/19 2:41 PM, Alex Bennée wrote:
> From: John Snow
>
> Fedora23 is but a distant twinkle.
> The sanitizer works again, and even if not,
> we have --enable-sanitizers now.
>
> Signed-off-by: John Snow
> Message-Id: <20190912014442.5757-1-js...@redhat.com>
> Signed-off-by: Alex Bennée
H
Patchew URL:
https://patchew.org/QEMU/20190917122644.15736-1-johan...@sipsolutions.net/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Subject: [Qemu-devel] [RFC] contrib: add vhost-user-sim
Message-id: 20190917122644.15736-1-johan...@sipsolut
From: John Snow
Fedora23 is but a distant twinkle.
The sanitizer works again, and even if not,
we have --enable-sanitizers now.
Signed-off-by: John Snow
Message-Id: <20190912014442.5757-1-js...@redhat.com>
Signed-off-by: Alex Bennée
---
tests/docker/test-clang | 6 +-
1 file changed, 1 in
This is broadly similar to the existing fcvt test for ARM but using
the generic float testing framework. We should be able to pare down
the ARM fcvt test case to purely half-precision with or without the
Alt HP provision.
Signed-off-by: Alex Bennée
---
tests/tcg/aarch64/float_convs.ref | 748 +
Now we have fixed the signal delivary bug we can remove this horrible
hack from the system.
Cc: Richard Henderson
Signed-off-by: Alex Bennée
---
tests/tcg/configure.sh | 1 +
tests/tcg/multiarch/Makefile.target | 11 +++
2 files changed, 4 insertions(+), 8 deletions(-)
di
From: John Snow
Oops; there's no argv here.
Signed-off-by: John Snow
Message-Id: <20190913193821.17756-1-js...@redhat.com>
---
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 29613afd489..bc7a470ca
We were incorrectly using the 64-bit AIX ABI instead of the 32-bit
SYSV ABI for setting NIP for the signal handler.
Signed-off-by: Alex Bennée
Reviewed-by: Richard Henderson
---
v2
- change to wording
---
linux-user/ppc/signal.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff
These were missed in the recent de-tangling so have been updated to be
more actuate. I've also built up ARM_TESTS in a manner similar to
AARCH64_TESTS for better consistency.
Signed-off-by: Alex Bennée
Reviewed-by: Peter Maydell
---
tests/tcg/Makefile.target | 7 +--
tests/tcg/aarc
Hi,
This is the current status of the testing/next queue. It contains a
number of fixes for docker and podman as well as some additional tests
for floating point under TCG I've been experimenting with.
Although we have reasonable confidence in large chunks of the core
softfloat code thanks to che
Finger trouble in a previous clean-up inadvertently set
DEBIAN_PARTIAL_IMAGES instead of DOCKER_PARTIAL_IMAGES.
Fixes: 44d5a8bf5d2
Signed-off-by: Alex Bennée
---
tests/docker/Makefile.include | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/docker/Makefile.include b/test
On 9/14/19 10:35 AM, Markus Armbruster wrote:
> We normalize shorthand to longhand forms in check_expr(): enumeration
> values with normalize_enum(), feature values with
> normalize_features(), struct members, union branches and alternate
> branches with normalize_members(). If conditions are an e
On 9/14/19 10:34 AM, Markus Armbruster wrote:
> Commit 87adbbffd4..3e270dcacc "qapi: Add 'if' to (implicit
> struct|union|alternate) members" (v4.0.0) neglected test coverage, and
> promptly failed to check the conditions. Review fail.
>
> Recent commit "tests/qapi-schema: Demonstrate insufficien
Patchew URL:
https://patchew.org/QEMU/156872146565.1757.3033215873677512474.stgit@pasha-Precision-3630-Tower/
Hi,
This series failed the docker-mingw@fedora build test. Please find the testing
commands and
their output below. If you have Docker installed, you can probably reproduce it
locally
1 - 100 of 336 matches
Mail list logo