Re: [Qemu-devel] [PATCH] migration: Update docs to discourage version bumps

"Dr. David Alan Gilbert (git)" writes: > From: "Dr. David Alan Gilbert" > > Version bumps break backwards migration; update the docs > to explain to people that's bad and how to avoid it. > > Signed-off-by: Dr. David Alan Gilbert > --- > docs/migration.txt | 42

Re: [Qemu-devel] [PATCH 1/2] cirrus: fix patterncopy checks

Seems to work. Took me a while to realize whether the else case was safe, but given the patternfill functions' access patterns and CIRRUS_BLTBUFSIZE being 8k it should be fine. On Thu, Feb 09, 2017 at 02:02:20PM +0100, Gerd Hoffmann wrote: > The blit_region_is_unsafe checks don't work correctly fo

Re: [Qemu-devel] VirtIO issues after 83d768b5640946b7da55ce8335509df297e2c7cd

On 03/02/17 01:14, Brian Rak wrote: We recently upgraded to qemu 2.8.0, and noticed a bunch of issues with various non-linux operating systems. I was able to bisect this down to 83d768b5640946b7da55ce8335509df297e2c7cd being the commit that breaks things (0687c37c5eeef8580b31cc6e1202d874833ae38a

Re: [Qemu-devel] kvm bug in __rmap_clear_dirty during live migration

Hi, Chris Friesen, did you solve the problem? On 2017/2/9 22:37, Herongguang (Stephen) wrote: Hi. I had a problem when I just repeatedly live migrate a vm between two compute nodes. The phenomenon was that the KVM module was crashed and then the host rebooted. However I cannot reliably trigger

Re: [Qemu-devel] [PATCH v5 05/10] ACPI: Add Virtual Machine Generation ID support

On Thu, 9 Feb 2017 12:39:30 -0800 Ben Warren wrote: > > On Feb 9, 2017, at 12:24 PM, Laszlo Ersek wrote: > > > > On 02/09/17 21:02, Ben Warren wrote: > >> > >>> On Feb 9, 2017, at 11:27 AM, Laszlo Ersek >>> > wrote: > >>> > >>> On 02/09/17 18:23, Igor Mammedov w

Re: [Qemu-devel] [PATCH 7/7] machine: unify [pc_|spapr_]query_hotpluggable_cpus() callbacks

On Fri, 10 Feb 2017 10:53:39 +1100 David Gibson wrote: > On Thu, Feb 09, 2017 at 12:08:38PM +0100, Igor Mammedov wrote: > > All callbacks FOO_query_hotpluggable_cpus() are practically > > the same except of setting vcpus_count to different values. > > Convert them to a generic machine_query_hotpl

Re: [Qemu-devel] [PATCH] cputlb: Don't assume do_unassigned_access() never returns

On 9 February 2017 at 23:32, Richard Henderson wrote: > On 02/10/2017 07:58 AM, Peter Maydell wrote: >> I use the M profile unassigned_access hook to implement >> the "high addresses are magic, but only if you're in >> Handler mode" behaviour. So for Thread mode I want the >> default handling. You

Re: [Qemu-devel] [PATCH] osdep.h: pull in sys/sysmacros.h for major/minor/makedev

On 10 February 2017 at 06:12, Mike Frysinger wrote: > Linux C libs are moving away from implicit header pollution with > sys/types.h > > Signed-off-by: Mike Frysinger > --- > include/qemu/osdep.h | 4 > 1 file changed, 4 insertions(+) > > diff --git a/include/qemu/osdep.h b/include/qemu/osd

Re: [Qemu-devel] [RFC 0/4] vITS save/restore

Hi Peter, all, On 07/02/2017 15:36, Peter Maydell wrote: > On 26 January 2017 at 09:19, Eric Auger wrote: >> This series allows ITS save/restore and migration use cases. >> It relies on not upstreamed kernel series ([1] & [2]) and >> QEMU not upstreamed series [3]. >> >> ITS tables are flushed in

Re: [Qemu-devel] kvm bug in __rmap_clear_dirty during live migration

Hi, Chris Friesen, did you solve the problem? On 2017/2/9 22:37, Herongguang (Stephen) wrote: Hi. I had a problem when I just repeatedly live migrate a vm between two compute nodes. The phenomenon was that the KVM module was crashed and then the host rebooted. However I cannot reliably trigger

[Qemu-devel] [PATCH 1/3] mac99: replace debug printf with trace points

Signed-off-by: Laurent Vivier --- hw/ppc/mac_newworld.c | 15 +++ hw/ppc/trace-events | 4 2 files changed, 7 insertions(+), 12 deletions(-) diff --git a/hw/ppc/mac_newworld.c b/hw/ppc/mac_newworld.c index 2bfdb64..3702dd1 100644 --- a/hw/ppc/mac_newworld.c +++ b/hw/ppc/mac_n

[Qemu-devel] [PATCH 0/3] ppc: replace debug printf with trace points

This series replaces the remaining DPRINTF() found under hw/ppc and replace them with trace points. Laurent Vivier (3): mac99: replace debug printf with trace points ppc4xx: replace debug printf with trace points spapr: replace debug printf with trace points hw/ppc/mac_newworld.c | 15 +++-

[Qemu-devel] [PATCH 2/3] ppc4xx: replace debug printf with trace points

Signed-off-by: Laurent Vivier --- hw/ppc/ppc4xx_pci.c | 13 +++-- hw/ppc/trace-events | 4 2 files changed, 7 insertions(+), 10 deletions(-) diff --git a/hw/ppc/ppc4xx_pci.c b/hw/ppc/ppc4xx_pci.c index 683218e..dc19682 100644 --- a/hw/ppc/ppc4xx_pci.c +++ b/hw/ppc/ppc4xx_pci.c @@ -

[Qemu-devel] [PATCH 3/3] spapr: replace debug printf with trace points

Signed-off-by: Laurent Vivier --- hw/ppc/spapr_ovec.c | 17 +++-- hw/ppc/trace-events | 4 2 files changed, 7 insertions(+), 14 deletions(-) diff --git a/hw/ppc/spapr_ovec.c b/hw/ppc/spapr_ovec.c index 3eb1d59..02ed740 100644 --- a/hw/ppc/spapr_ovec.c +++ b/hw/ppc/spapr_ovec.c

Re: [Qemu-devel] [Help] Windows2012 as Guest 64+cores on KVM Halts

Hi, > Behalf Of Yan Vugenfirer > > Hi, > > First off all to use more than 64 CPUs on Windows you should use CPU grouping. > Each group can contain maximum 64 CPUs. Here is a good article explaining > possible configurations: > https://msdn.microsoft.com/en-us/windows/hardware/drivers/devtest/boo

Re: [Qemu-devel] [PATCH 0/3] ppc: replace debug printf with trace points

Hi, Your series failed automatic build test. Please find the testing commands and their output below. If you have docker installed, you can probably reproduce it locally. Subject: [Qemu-devel] [PATCH 0/3] ppc: replace debug printf with trace points Message-id: 20170210092724.6470-1-lviv...@redhat

[Qemu-devel] [PATCH] monitor: add poll-* properties into query-iothreads result

IOthreads were recently extended by new properties that can enable/disable and configure aio polling. This will also allow other tools that uses QEMU to probe for existence of those new properties via query-qmp-schema. Signed-off-by: Pavel Hrdina --- The other possible way would be to introduce

Re: [Qemu-devel] [PATCH] monitor: add poll-* properties into query-iothreads result

Hi, Your series failed automatic build test. Please find the testing commands and their output below. If you have docker installed, you can probably reproduce it locally. Subject: [Qemu-devel] [PATCH] monitor: add poll-* properties into query-iothreads result Message-id: 3163c16d6ab4257f7be9ad4

[Qemu-devel] [PATCH qemu 0/7] KVM: race-free exit from KVM_RUN without POSIX signals

The purpose of the KVM_SET_SIGNAL_MASK API is to let userspace "kick" a VCPU out of KVM_RUN through a POSIX signal. A signal is attached to a dummy signal handler; by blocking the signal outside KVM_RUN and unblocking it inside, this possible race is closed: VCPU thread

[Qemu-devel] [PATCH 2/7] KVM: x86: cleanup SIGBUS handlers

This patch should have no semantic change. Signed-off-by: Paolo Bonzini --- target/i386/kvm.c | 81 --- 1 file changed, 42 insertions(+), 39 deletions(-) diff --git a/target/i386/kvm.c b/target/i386/kvm.c index 27fd050..0c48dfd 100644 --- a/ta

[Qemu-devel] [PATCH 7/7] KVM: use KVM_CAP_IMMEDIATE_EXIT

The purpose of the KVM_SET_SIGNAL_MASK API is to let userspace "kick" a VCPU out of KVM_RUN through a POSIX signal. A signal is attached to a dummy signal handler; by blocking the signal outside KVM_RUN and unblocking it inside, this possible race is closed: VCPU thread

[Qemu-devel] [PATCH 1/7] cpus: remove ugly cast on sigbus_handler

The cast is there because sigbus_handler is invoked via sigfd_handler. But it feels just wrong to use struct qemu_signalfd_siginfo in the prototype of a function that is passed to sigaction. Instead, do a simple-minded conversion of qemu_signalfd_siginfo to siginfo_t. Signed-off-by: Paolo Bonzini

[Qemu-devel] [PATCH 3/7] cpus: reorganize signal handling code

Move the KVM "eat signals" code under CONFIG_LINUX, in preparation for moving it to kvm-all.c; reraise non-MCE SIGBUS immediately, without passing it to KVM. Signed-off-by: Paolo Bonzini --- cpus.c | 63 ++-- include/qemu/osdep.h | 9

[Qemu-devel] [PATCH 4/7] KVM: remove kvm_arch_on_sigbus

Build it on kvm_arch_on_sigbus_vcpu instead. They do the same for "action optional" SIGBUSes, and the main thread should never get "action required" SIGBUSes because it blocks the signal. Signed-off-by: Paolo Bonzini --- include/sysemu/kvm.h | 1 - kvm-all.c| 9 - target/a

[Qemu-devel] [PATCH 5/7] KVM: do not use sigtimedwait to catch SIGBUS

Call kvm_on_sigbus_vcpu asynchronously from the VCPU thread. Information for the SIGBUS can be stored in thread-local variables and processed later in kvm_cpu_exec. Signed-off-by: Paolo Bonzini --- cpus.c | 31 +-- include/sysemu/kvm.h | 5 - kvm-al

[Qemu-devel] [PATCH 6/7] KVM: move SIG_IPI handling to kvm-all.c

This lets us remove a bunch of CONFIG_LINUX defines. Signed-off-by: Paolo Bonzini --- cpus.c | 62 +--- include/sysemu/kvm.h | 5 ++--- kvm-all.c| 60 +- kvm-stub.c

Re: [Qemu-devel] [PATCH qemu 0/7] KVM: race-free exit from KVM_RUN without POSIX signals

Hi, Your series failed automatic build test. Please find the testing commands and their output below. If you have docker installed, you can probably reproduce it locally. Subject: [Qemu-devel] [PATCH qemu 0/7] KVM: race-free exit from KVM_RUN without POSIX signals Message-id: 20170210095012.1603

Re: [Qemu-devel] [PATCH v5 00/10] Add support for VM Generation ID

On 02/05/17 10:11, b...@skyportsystems.com wrote: > From: Ben Warren > > This patch set adds support for passing a GUID to Windows guests. It is a > re-implementation of previous patch sets written by Igor Mammedov et al, but > this time passing the GUID data as a fw_cfg blob. > > This patch se

[Qemu-devel] [PATCH v2 6/7] spapr: reuse machine->possible_cpus instead of cores[]

Replace SPAPR specific cores[] array with generic machine->possible_cpus and store core objects there. It makes cores bookkeeping similar to x86 cpus and will allow to unify similar code. It would allow to replace cpu_index based NUMA node mapping with iproperty based one (for -device created cores

[Qemu-devel] [PATCH v2 8/7] machine: replace query_hotpluggable_cpus() callback with has_hotpluggable_cpus flag

Generic helper machine_query_hotpluggable_cpus() replaced target specific query_hotpluggable_cpus() callbacks so there is no need in it anymore. However inon NULL callback value is used to detect/report hotpluggable cpus support, therefore it can be removed completely. Replace it with MachineClass.

[Qemu-devel] [PATCH v6 0/2] docs: Improve sample configuration files

Changes since [v5] * Address review comments: - use the native serial console rather than the VirtIO based one, and enable monitor multiplexing so that the user can switch between the two; - add GIC version configuration; - improve wording all over the place. * Improve

Re: [Qemu-devel] [PATCH v5 00/10] Add support for VM Generation ID

On Fri, 10 Feb 2017 11:12:13 +0100 Laszlo Ersek wrote: > On 02/05/17 10:11, b...@skyportsystems.com wrote: > > From: Ben Warren > > > > This patch set adds support for passing a GUID to Windows guests. It is a > > re-implementation of previous patch sets written by Igor Mammedov et al, but > >

[Qemu-devel] [PATCH] Add a blog post with the presentations from the FOSDEM 2017

The videos are now available online, so here's a blog post that contains the links to the QEMU-related videos. Signed-off-by: Thomas Huth --- _posts/2017-02-10-fosdem-presentations.md | 30 ++ 1 file changed, 30 insertions(+) create mode 100644 _posts/2017-02-10-fosd

Re: [Qemu-devel] [PATCH 0/7] pc/spapr: unify handling of possible CPUs

On Thu, 9 Feb 2017 12:08:31 +0100 Igor Mammedov wrote: [...] > > Series depends on: > [PATCH 0/3] spapr: fix cpu core hotunplug call flow > https://www.mail-archive.com/qemu-devel@nongnu.org/msg427214.html > which is staged for next pull req in SPAPR tree. David, could you merge it throug

Re: [Qemu-devel] [PATCH] Add a blog post with the presentations from the FOSDEM 2017

Hi, Your series failed automatic build test. Please find the testing commands and their output below. If you have docker installed, you can probably reproduce it locally. Subject: [Qemu-devel] [PATCH] Add a blog post with the presentations from the FOSDEM 2017 Message-id: 1486722639-6586-1-git-s

[Qemu-devel] [PATCH v6 1/2] q35: Improve sample configuration files

Instead of having a single sample configuration file, we now have several: * q35-emulated.cfg documents the default devices QEMU adds to a q35 guest and the additional devices that are pretty much guaranteed to be present in a physical q35-based machine; * q35-virtio-graphical.cfg

[Qemu-devel] [PATCH v6 2/2] mach-virt: Provide sample configuration files

These are very much like the sample configuration files for q35, and can be used both as documentation and as a starting point for creating your own guest. Two sample configuration files are provided: * mach-virt-graphical.cfg can be used to start a fully-featured (USB, graphical console, e

Re: [Qemu-devel] [Qemu-block] [PATCH v3 08/18] qcow: make encrypt_sectors encrypt in place

On Thu 26 Jan 2017 11:18:17 AM CET, Daniel P. Berrange wrote: > Instead of requiring separate input/output buffers for > encrypting data, change encrypt_sectors() to assume > use of a single buffer, encrypting in place. One current > caller uses the same buffer for input/output already > and the ot

Re: [Qemu-devel] [PATCH] Add a blog post with the presentations from the FOSDEM 2017

On 10.02.2017 11:35, no-re...@patchew.org wrote: > Hi, > > Your series failed automatic build test. Please find the testing commands and > their output below. If you have docker installed, you can probably reproduce > it > locally. Sorry, I forgot to put "qemu-web" in the subject :-/ Thomas

[Qemu-devel] [PATCH v2] migration: Update docs to discourage version bumps

From: "Dr. David Alan Gilbert" Version bumps break backwards migration; update the docs to explain to people that's bad and how to avoid it. Signed-off-by: Dr. David Alan Gilbert --- docs/migration.txt | 58 ++ 1 file changed, 58 insertions(+

Re: [Qemu-devel] [PATCH] migration: Update docs to discourage version bumps

* Markus Armbruster (arm...@redhat.com) wrote: > "Dr. David Alan Gilbert (git)" writes: > > > From: "Dr. David Alan Gilbert" > > > > Version bumps break backwards migration; update the docs > > to explain to people that's bad and how to avoid it. > > > > Signed-off-by: Dr. David Alan Gilbert >

Re: [Qemu-devel] [PATCH 04/18] nbd/client: refactor nbd_receive_starttls

09.02.2017 17:41, Eric Blake wrote: On 02/09/2017 12:20 AM, Vladimir Sementsov-Ogievskiy wrote: 07.02.2017 19:32, Eric Blake wrote: On 02/03/2017 09:47 AM, Vladimir Sementsov-Ogievskiy wrote: Split out nbd_receive_simple_option to be reused for structured reply option. +return ""; Can you

Re: [Qemu-devel] [PATCH] Add a blog post with the presentations from the FOSDEM 2017

On 10/02/2017 11:30, Thomas Huth wrote: > +During last weekend, on February 4th and 5th, the > +[FOSDEM 2017](https://fosdem.org/2017/) conference took place in Brussels, > +which featured a [Virtualisation and IaaS > +track](https://fosdem.org/2017/schedule/track/virtualisation_and_iaas/). > + >

Re: [Qemu-devel] [PATCH] Add a blog post with the presentations from the FOSDEM 2017

On 10.02.2017 12:24, Paolo Bonzini wrote: > > > On 10/02/2017 11:30, Thomas Huth wrote: >> +During last weekend, on February 4th and 5th, the >> +[FOSDEM 2017](https://fosdem.org/2017/) conference took place in Brussels, >> +which featured a [Virtualisation and IaaS >> +track](https://fosdem.org/

Re: [Qemu-devel] [PATCH] Add a blog post with the presentations from the FOSDEM 2017

On 10/02/2017 12:28, Thomas Huth wrote: > On 10.02.2017 12:24, Paolo Bonzini wrote: >> >> >> On 10/02/2017 11:30, Thomas Huth wrote: >>> +During last weekend, on February 4th and 5th, the >>> +[FOSDEM 2017](https://fosdem.org/2017/) conference took place in Brussels, >>> +which featured a [Virtua

Re: [Qemu-devel] [PATCH] tcg: handle EXCP_ATOMIC exception properly

On 10/02/2017 02:45, Pranith Kumar wrote: > The current method of executing atomic code in a guest uses > cpu_exec_step_atomic() from the outermost loop. This causes an abort() > when single stepping over atomic code since debug exception longjmp > will point to the the setlongjmp in cpu_exec().

Re: [Qemu-devel] [PATCH 1/2] cirrus: fix patterncopy checks

On 09/02/2017 14:02, Gerd Hoffmann wrote: > The blit_region_is_unsafe checks don't work correctly for the > patterncopy source. It's a fixed-sized region, which doesn't > depend on cirrus_blt_{width,height}. So go do the check in > cirrus_bitblt_common_patterncopy instead, then tell blit_is_unsaf

Re: [Qemu-devel] [PATCH 2/2] Revert "cirrus: allow zero source pitch in pattern fill rops"

On 09/02/2017 14:02, Gerd Hoffmann wrote: > This reverts commit 5858dd1801883309bdd208d72ddb81c4e9fee30c. > > Conflicts: > hw/display/cirrus_vga.c > > Cc: Wolfgang Bumiller > Cc: Dr. David Alan Gilbert > Signed-off-by: Gerd Hoffmann Reviewed-by: Laurent Vivier > --- > hw/display/cirr

Re: [Qemu-devel] [PATCH v6 2/2] mach-virt: Provide sample configuration files

On 02/10/17 11:38, Andrea Bolognani wrote: > These are very much like the sample configuration files > for q35, and can be used both as documentation and as > a starting point for creating your own guest. > > Two sample configuration files are provided: > > * mach-virt-graphical.cfg can be used

Re: [Qemu-devel] [PATCH v2 1/3] migration: add MigrationState arg for ram_save_/compressed_/page()

* Pavel Butsykin (pbutsy...@virtuozzo.com) wrote: > Cosmetic patch. The use of ms variable instead of migrate_get_current() > looks nicer, especially when there reuse. > > Signed-off-by: Pavel Butsykin Reviewed-by: Dr. David Alan Gilbert > --- > migration/ram.c | 15 --- > 1 file

Re: [Qemu-devel] [Help] Windows2012 as Guest 64+cores on KVM Halts

On 10/02/2017 10:31, Gonglei (Arei) wrote: > But We tested the same cases on Xen platform and VMware, and > the guest booted successfully. Were these two also tested with enlightenments enabled? TCG surely isn't. Paolo > Today I also tested on tcg, the guest also boots successfully. > > BTW,

Re: [Qemu-devel] [PATCH] tcg: handle EXCP_ATOMIC exception properly

Pranith Kumar writes: > The current method of executing atomic code in a guest uses > cpu_exec_step_atomic() from the outermost loop. This causes an abort() > when single stepping over atomic code since debug exception longjmp > will point to the the setlongjmp in cpu_exec(). Another issue with

Re: [Qemu-devel] [PATCH 06/22] char: add a /chardevs container

Hi On Fri, Feb 10, 2017 at 4:18 AM Paolo Bonzini wrote: > > > On 07/02/2017 21:03, Marc-André Lureau wrote: > > Hi > > > > - Original Message - > >> > >> > >> On 02/02/2017 15:51, Marc-André Lureau wrote: > >>> +if (QTAILQ_IN_USE(chr, next)) { > >>> +QTAILQ_REMOVE(&chardevs,

Re: [Qemu-devel] [PATCH] tcg: handle EXCP_ATOMIC exception properly

On 10/02/2017 13:13, Alex Bennée wrote: >> +if (atomic_read(&other_cpu->running) && >> !qemu_cpu_is_self(other_cpu)) { > The comment above reads: > > Must only be called from outside cpu_exec. > > So we need to revise this comment. Is this really a limitation or was it > originally th

Re: [Qemu-devel] [PATCH] tcg: handle EXCP_ATOMIC exception properly

Paolo Bonzini writes: > On 10/02/2017 02:45, Pranith Kumar wrote: >> The current method of executing atomic code in a guest uses >> cpu_exec_step_atomic() from the outermost loop. This causes an abort() >> when single stepping over atomic code since debug exception longjmp >> will point to the t

Re: [Qemu-devel] [PATCH v2 2/3] add 'release-ram' migrate capability

* Pavel Butsykin (pbutsy...@virtuozzo.com) wrote: > This feature frees the migrated memory on the source during postcopy-ram > migration. In the second step of postcopy-ram migration when the source vm > is put on pause we can free unnecessary memory. It will allow, in particular, > to start relaxi

Re: [Qemu-devel] [PATCH v2 2/3] add 'release-ram' migrate capability

* Dr. David Alan Gilbert (dgilb...@redhat.com) wrote: > * Pavel Butsykin (pbutsy...@virtuozzo.com) wrote: > > This feature frees the migrated memory on the source during postcopy-ram > > migration. In the second step of postcopy-ram migration when the source vm > > is put on pause we can free unnec

Re: [Qemu-devel] [PATCH 06/22] char: add a /chardevs container

On 10/02/2017 13:14, Marc-André Lureau wrote: > Hi > > On Fri, Feb 10, 2017 at 4:18 AM Paolo Bonzini > wrote: > > > > On 07/02/2017 21:03, Marc-André Lureau wrote: > > Hi > > > > - Original Message - > >> > >> > >> On 02/02/2017

Re: [Qemu-devel] [PATCH] tcg: handle EXCP_ATOMIC exception properly

On 10/02/2017 13:18, Alex Bennée wrote: >> I think you can unlock/lock the iothread here, and also call > > The iothread is already unlocked by this point (see tcg_cpu_exec). Is this patch on top of the MTTCG branch? If not, cpu_handle_exception runs with the iothread lock taken doesn't it? >>

Re: [Qemu-devel] [PATCH v2 3/3] migration: discard non-dirty ram pages after the start of postcopy

* Pavel Butsykin (pbutsy...@virtuozzo.com) wrote: > After the start of postcopy migration there are some non-dirty pages which > have > already been migrated. These pages are no longer needed on the source vm so > that > we can free them and it doen't hurt to complete the migration. > > Signed-o

Re: [Qemu-devel] [PATCH] tcg: handle EXCP_ATOMIC exception properly

Paolo Bonzini writes: > On 10/02/2017 13:13, Alex Bennée wrote: >>> +if (atomic_read(&other_cpu->running) && >>> !qemu_cpu_is_self(other_cpu)) { >> The comment above reads: >> >> Must only be called from outside cpu_exec. >> >> So we need to revise this comment. Is this really a limita

Re: [Qemu-devel] [PATCH] tests/prom-env: Ease time-out problems on slow hosts

On 9 February 2017 at 23:15, David Gibson wrote: > On Thu, Feb 09, 2017 at 02:37:26PM +, Peter Maydell wrote: >> On 9 February 2017 at 14:13, Thomas Huth wrote: >> > Peter Maydell recently ran into time-out problems with the >> > prom-env test on a rather slow ARM board. To tackle this issue,

Re: [Qemu-devel] [PATCH] tests/prom-env: Ease time-out problems on slow hosts

On 10.02.2017 13:48, Peter Maydell wrote: > On 9 February 2017 at 23:15, David Gibson wrote: >> On Thu, Feb 09, 2017 at 02:37:26PM +, Peter Maydell wrote: >>> On 9 February 2017 at 14:13, Thomas Huth wrote: Peter Maydell recently ran into time-out problems with the prom-env test on

Re: [Qemu-devel] [PATCH] tcg: handle EXCP_ATOMIC exception properly

Paolo Bonzini writes: > On 10/02/2017 13:18, Alex Bennée wrote: >>> I think you can unlock/lock the iothread here, and also call >> >> The iothread is already unlocked by this point (see tcg_cpu_exec). > > Is this patch on top of the MTTCG branch? If not, cpu_handle_exception > runs with the io

Re: [Qemu-devel] [PATCH 06/22] char: add a /chardevs container

Hi On Fri, Feb 10, 2017 at 4:26 PM Paolo Bonzini wrote: > > > > > But if you unparent with the last ref, you remove the burden of knowing > > if the object has been parented from the user. I don't see why that > > would conflict with object_unparent(), you could still unparent(), and > > keep th

Re: [Qemu-devel] [PATCH 06/22] char: add a /chardevs container

On 10/02/2017 13:59, Marc-André Lureau wrote: > > But if you unparent with the last ref, you remove the burden of knowing > > if the object has been parented from the user. I don't see why that > > would conflict with object_unparent(), you could still unparent(), and > > keep the

Re: [Qemu-devel] [PATCH] tcg: handle EXCP_ATOMIC exception properly

On 10/02/2017 13:33, Alex Bennée wrote: > That said for TCG system emulation EXCP_ATOMIC is currently broken with > respect to debugging. However for the initial guests/host combination of > ARMv7/8 on x86_64 we don't need the fallback with pretty much 99% of > deployed hosts. How about the follo

Re: [Qemu-devel] [PATCH 1/3] mac99: replace debug printf with trace points

On 02/10/2017 06:27 AM, Laurent Vivier wrote: Signed-off-by: Laurent Vivier Reviewed-by: Philippe Mathieu-Daudé --- hw/ppc/mac_newworld.c | 15 +++ hw/ppc/trace-events | 4 2 files changed, 7 insertions(+), 12 deletions(-) diff --git a/hw/ppc/mac_newworld.c b/hw/ppc/ma

Re: [Qemu-devel] [PATCH 2/3] ppc4xx: replace debug printf with trace points

On 02/10/2017 06:27 AM, Laurent Vivier wrote: Signed-off-by: Laurent Vivier Reviewed-by: Philippe Mathieu-Daudé --- hw/ppc/ppc4xx_pci.c | 13 +++-- hw/ppc/trace-events | 4 2 files changed, 7 insertions(+), 10 deletions(-) diff --git a/hw/ppc/ppc4xx_pci.c b/hw/ppc/ppc4xx_pci

Re: [Qemu-devel] [PATCH 3/3] spapr: replace debug printf with trace points

On 02/10/2017 06:27 AM, Laurent Vivier wrote: Signed-off-by: Laurent Vivier Reviewed-by: Philippe Mathieu-Daudé --- hw/ppc/spapr_ovec.c | 17 +++-- hw/ppc/trace-events | 4 2 files changed, 7 insertions(+), 14 deletions(-) diff --git a/hw/ppc/spapr_ovec.c b/hw/ppc/spapr_

[Qemu-devel] Sparc sun4v machine currently broken?

Hi, is it just me, or could it be that the sun4v machine is currently broken? I get the following error when I try to run QEMU with this machine: qemu-system-sparc64 -nographic -M sun4v qemu: fatal: Trap 0x0010 while trap level (6) >= MAXTL (6), Error state pc: 0200 npc: 000

Re: [Qemu-devel] [PATCH] tcg: handle EXCP_ATOMIC exception properly

On Fri, Feb 10, 2017 at 7:29 AM, Paolo Bonzini wrote: > > > On 10/02/2017 13:18, Alex Bennée wrote: > >> I think you can unlock/lock the iothread here, and also call > > > > The iothread is already unlocked by this point (see tcg_cpu_exec). > > Is this patch on top of the MTTCG branch? If not, c

Re: [Qemu-devel] [Qemu-arm] [PATCH 0/9] Rewrite NVIC to not depend on the GIC

On 2 February 2017 at 20:02, Peter Maydell wrote: > This patchset is the revamp of the NVIC code from Michael > Davidsaver's patchset of a year ago. > > Despite some superficial similarities of register layout, the > M-profile NVIC is really very different from the A-profile GIC. Our > current at

Re: [Qemu-devel] [PATCH 0/3] Risu PPC improvements

On 9 February 2017 at 05:58, Nikunj A Dadhania wrote: > The series contains some fixes/cleanup for ppc64le > > Nikunj A Dadhania (3): > risu_ppc64le: set r23 as the comment says > risu_ppc64le: zero the xer register > risu_ppc64le: remove aarch64 related comments > > risugen_ppc64.pm | 9 ++

Re: [Qemu-devel] [Qemu-arm] [PATCH] target-arm: Declare virtio-mmio as dma-coherent in dt

Hi, FWTS supports multiple architectures: - amd64 - arm64 - armhf - i386 - ppc64el - s390x Package details are available @ https://launchpad.net/~firmware-testing-team/+archive/ubuntu/ppa-fwts-stable/+packages FWTS also supports device tree to some extent, thanks to contributions from Jeremy Ke

Re: [Qemu-devel] Asking about USB for QEMU

Hi, If I understand right: You want to run a zynq qemu on a centos host? On 02/09/2017 09:25 AM, Wojciech Zebrowski wrote: > Hi > > I try connect my host linux centos USB to Qemu for Xilinx Zynq solution. > I do following step : > 1.Check usb vendor id , product id: lsusb > > > > > *Bus 001

Re: [Qemu-devel] [PATCH 1/9] armv7m: Rename nvic_state to NVICState

On 02/02/2017 05:02 PM, Peter Maydell wrote: Rename the nvic_state struct to NVICState, to match our naming conventions. Signed-off-by: Peter Maydell Reviewed-by: Philippe Mathieu-Daudé --- hw/intc/armv7m_nvic.c | 44 ++-- 1 file changed, 22 inserti

Re: [Qemu-devel] [PATCH 2/9] armv7m: Implement reading and writing of PRIGROUP

On 02/02/2017 05:02 PM, Peter Maydell wrote: Add a state field for the v7M PRIGROUP register and implent reading and writing it. The current NVIC doesn't honour the values written, but the new version will. Signed-off-by: Peter Maydell Reviewed-by: Philippe Mathieu-Daudé --- hw/intc/armv7

Re: [Qemu-devel] [PATCH 9/9] armv7m: VECTCLRACTIVE and VECTRESET are UNPREDICTABLE

On 02/02/2017 05:02 PM, Peter Maydell wrote: From: Michael Davidsaver The VECTCLRACTIVE and VECTRESET bits in the AIRCR are both documented as UNPREDICTABLE if you write a 1 to them when the processor is not halted in Debug state (ie stopped and under the control of an external JTAG debugger).

Re: [Qemu-devel] [PATCH] tcg: handle EXCP_ATOMIC exception properly

Paolo Bonzini writes: > On 10/02/2017 13:33, Alex Bennée wrote: >> That said for TCG system emulation EXCP_ATOMIC is currently broken with >> respect to debugging. However for the initial guests/host combination of >> ARMv7/8 on x86_64 we don't need the fallback with pretty much 99% of >> deploy

Re: [Qemu-devel] [PATCH] tcg: handle EXCP_ATOMIC exception properly

On 10/02/2017 15:37, Alex Bennée wrote: >>> - drop Pranith's patch for the current MTTCG series >>> - replace with an error/abort on EXCP_ATOMIC >> Don't even replace it? :) > > I guess - the failure mode is only when we single step? Should we not > try to emit a warning like we do for the m

Re: [Qemu-devel] [PATCH v11 20/24] target-arm/powerctl: defer cpu reset work to CPU context

On 9 February 2017 at 17:09, Alex Bennée wrote: > When switching a new vCPU on we want to complete a bunch of the setup > work before we start scheduling the vCPU thread. To do this cleanly we > defer vCPU setup to async work which will run the vCPUs execution > context as the thread is woken up.

Re: [Qemu-devel] [PATCH] migration: madvise error_report fixup!

* Pavel Butsykin (pbutsy...@virtuozzo.com) wrote: > Signed-off-by: Pavel Butsykin Reviewed-by: Dr. David Alan Gilbert I'll merge it in with the others when I queue it. Dave > --- > migration/qemu-file.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/migration/q

Re: [Qemu-devel] [PATCH v11 23/24] hw/misc/imx6_src: defer clearing of SRC_SCR reset bits

On 9 February 2017 at 17:09, Alex Bennée wrote: > The arm_reset_cpu/set_cpu_on/set_cpu_off() functions do their work > asynchronously in the target vCPUs context. As a result we need to > ensure the SRC_SCR reset bits correctly report the reset status at the > right time. To do this we defer the c

Re: [Qemu-devel] [PATCH] migration: remove myself as maintainer

* Amit Shah (amit.s...@redhat.com) wrote: > I'm switching jobs, and I'm not sure I can continue maintaining migration. > > Signed-off-by: Amit Shah Queued. > --- > MAINTAINERS | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 00b81f1..4382bcc 100644 > -

Re: [Qemu-devel] [PATCH] MAINTAINERS: update my email address

* Amit Shah (amit.s...@redhat.com) wrote: > I'm leaving my job at Red Hat, this email address will stop working next week. > Update it to one that I will have access to later. > > Signed-off-by: Amit Shah Queued (while I'm doing the maintainer removal anyway). Dave > --- > MAINTAINERS | 4 ++-

Re: [Qemu-devel] [PATCH V3 0/4] Add vPMU vPMU support under TCG mode

On 7 February 2017 at 21:49, Wei Huang wrote: > QEMU has implemented cycle count support for guest VM under TCG mode. > But this feature is not complete. In fact using perf inside a > 64-bit Linux guest VM (under TCG) can cause the following kernel panic > because some PMU registers are not implem

Re: [Qemu-devel] [Qemu-arm] [PATCH v2 0/4] target-arm: Add some omitted dma cache coherency flags

On 9 February 2017 at 12:53, Alexander Graf wrote: > ARM is amazing when it comes to cache coherency and VMs. While any sane > architecture allows the host to override the guest's caching attributes, > that's very hard to do on ARM. > > That means that the guest may directly access guest memory by

Re: [Qemu-devel] [PATCH v3] migrate: Introduce a 'dc->vmsd' check to avoid segfault for --only-migratable

* Ashijeet Acharya (ashijeetacha...@gmail.com) wrote: > Commit a3a3d8c7 introduced a segfault bug while checking for > 'dc->vmsd->unmigratable' which caused QEMU to crash when trying to add > devices which do no set their 'dc->vmsd' yet while initialization. > Place a 'dc->vmsd' check prior to it s

[Qemu-devel] [PULL 0/2] Travis updates

The following changes since commit 7d2c6c95511e42dffe2b263275e09957723d0ff4: Merge remote-tracking branch 'remotes/kraxel/tags/pull-usb-20170206-1' into staging (2017-02-06 11:44:08 +) are available in the git repository at: https://github.com/stsquad/qemu.git tags/pull-travis-10022017-

[Qemu-devel] [PULL 1/2] .travis.yml: don't specify CONFIG twice

Signed-off-by: Alex Bennée --- .travis.yml | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/.travis.yml b/.travis.yml index d83e2d493b..3dc1e6439d 100644 --- a/.travis.yml +++ b/.travis.yml @@ -104,8 +104,7 @@ matrix: - wget -O - http://people.linaro.org/~alex.benne

[Qemu-devel] [PULL 2/2] .travis.yml: split VM based builds

The Trusty based builds run a little slower than the main container based ones. This is also true for the latest version of Clang. The builds are getting very close (and occasionally run over) the 50 minute timeout. Rather than partitioning by target I just split them into linux-user and system bui

Re: [Qemu-devel] [PATCH V3 0/4] Add vPMU vPMU support under TCG mode

On 10 February 2017 at 15:00, Peter Maydell wrote: > On 7 February 2017 at 21:49, Wei Huang wrote: >> QEMU has implemented cycle count support for guest VM under TCG mode. >> But this feature is not complete. In fact using perf inside a >> 64-bit Linux guest VM (under TCG) can cause the following

Re: [Qemu-devel] [PATCH v2 0/3] migration capability to discard the migrated ram pages

* Pavel Butsykin (pbutsy...@virtuozzo.com) wrote: > This feature frees the migrated memory on the source during postcopy-ram > migration. In the second step of postcopy-ram migration when the source vm > is put on pause we can free unnecessary memory. It will allow, in particular, > to start relaxi

[Qemu-devel] [PATCH v2] tests/prom-env: Ease time-out problems on slow hosts

Peter Maydell recently ran into time-out problems with the prom-env test on a rather slow ARM board. To tackle this issue, we can speed up the test by running QEMU with "-nodefaults" for the pseries machine, so that SLOF has less devices to scan during boot, and by using the "nvramrc" environment v

Re: [Qemu-devel] [PATCH v6 2/2] mach-virt: Provide sample configuration files

On Fri, 2017-02-10 at 12:43 +0100, Laszlo Ersek wrote: > So, what speaks against adding "-serial mon:stdio" here too? Even with a > graphical guest, the monitor is useful. And, if you care about firmware > logs (who doesn't? ;)), seeing serial output is good. (Same applies to > the guest kernel --

Re: [Qemu-devel] [PATCH 0/4] aspeed: miscellaneous small fixes

On 9 February 2017 at 13:47, Cédric Le Goater wrote: > Hello, > > Here is a short series fixing small issues in the SMC controller model > of the Aspeed SoC. > > Thanks, > > C. > > Cédric Le Goater (4): > aspeed: check for negative values returned by blk_getlength() > aspeed: remove useless co

Re: [Qemu-devel] [PATCH v11 23/24] hw/misc/imx6_src: defer clearing of SRC_SCR reset bits

Peter Maydell writes: > On 9 February 2017 at 17:09, Alex Bennée wrote: >> The arm_reset_cpu/set_cpu_on/set_cpu_off() functions do their work >> asynchronously in the target vCPUs context. As a result we need to >> ensure the SRC_SCR reset bits correctly report the reset status at the >> right

Re: [Qemu-devel] [PATCH] migration: consolidate VMStateField.start

* Halil Pasic (pa...@linux.vnet.ibm.com) wrote: > The member VMStateField.start is used for two things, partial data > migration for VBUFFER data (basically provide migration for a > sub-buffer) and for locating next in QTAILQ. > > The implementation of the VBUFFER feature is broken when VMSTATE_A

Re: [Qemu-devel] [PATCH 0/5] risu: add m68k target support

On 7 February 2017 at 18:33, Laurent Vivier wrote: > This series add the files needed to test the m68k target. > > For the moment, it doesn't support FPU instructions. > > Laurent Vivier (5): > m68k: implement risugen module > m68k: implement server and client side > m68k: add risufile with

  1   2   >