[Qemu-devel] [PATCH] iotests: Print full path of bad output if mismatch

So it is easier to copy paste the path. Signed-off-by: Fam Zheng --- tests/qemu-iotests/check | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/qemu-iotests/check b/tests/qemu-iotests/check index d504b6e455..4583a0c269 100755 ---

[Qemu-devel] [PATCH 3/3] iotests: Add "quit during block migration" case 195

Signed-off-by: Fam Zheng --- tests/qemu-iotests/195 | 97 ++ tests/qemu-iotests/195.out | 19 + tests/qemu-iotests/group | 1 + 3 files changed, 117 insertions(+) create mode 100755 tests/qemu-iotests/195 create mode

[Qemu-devel] [PATCH 2/3] migration: Cancel migration at exit

bdrv_close_all() would abort() due to op blockers added by BMDS, clean up migration states when main loop quits to avoid that. Signed-off-by: Fam Zheng --- include/migration/misc.h | 1 + migration/migration.c| 7 ++- vl.c | 3 +++ 3 files changed,

[Qemu-devel] [PATCH 0/3] migration: Fix crash by cleaning up before quit

Quit command causes asssertion failure in block layer due to op blockers added by BMDS, if there is an active block migration. Fixing this by calling migration cleaning up functions at the end of main() before bdrv_close_all() is called. Fam Zheng (3): migration: Allow ram_save_cleanup to be

[Qemu-devel] [PATCH 1/3] migration: Allow ram_save_cleanup to be called with empty state

So that we can do cleanup unconditionally at the end of main(). Signed-off-by: Fam Zheng --- migration/ram.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/migration/ram.c b/migration/ram.c index e18b3e2d4f..37e6a71241 100644 --- a/migration/ram.c +++ b/migration/ram.c

Re: [Qemu-devel] [PATCH 2/3] kvm: check KVM_CAP_NR_VCPUS with kvm_vm_check_extension()

On 14.09.2017 21:25, Greg Kurz wrote: > On a modern server-class ppc host with the following CPU topology: > > Architecture: ppc64le > Byte Order:Little Endian > CPU(s):32 > On-line CPU(s) list: 0,8,16,24 > Off-line CPU(s) list: 1-7,9-15,17-23,25-31 >

Re: [Qemu-devel] [PATCH 1/3] kvm: check KVM_CAP_SYNC_MMU with kvm_vm_check_extension()

On 14.09.2017 21:25, Greg Kurz wrote: > On a server-class ppc host, this capability depends on the KVM type, > ie, HV or PR. If both KVM are present in the kernel, we will always > get the HV specific value, even if we explicitely requested PR on > the command line. > > This can have an impact if

Re: [Qemu-devel] [PATCH 3/3] ppc/kvm: check some capabilities with kvm_vm_check_extension()

On 14.09.2017 21:25, Greg Kurz wrote: > The following capabilities are VM specific: > - KVM_CAP_PPC_SMT_POSSIBLE > - KVM_CAP_PPC_HTAB_FD BTW, looks like kvmppc_has_cap_htab_fd() is dead code ... should we either remove it or check it somewhere? > - KVM_CAP_PPC_ALLOC_HTAB > > If both KVM HV and

Re: [Qemu-devel] [PULL 00/18] ppc-for-2.11 queue 20170915

On Thu, Sep 14, 2017 at 09:26:49PM -0700, no-re...@patchew.org wrote: > Hi, > > This series seems to have some coding style problems. See output below for > more information: > > Subject: [Qemu-devel] [PULL 00/18] ppc-for-2.11 queue 20170915 > Message-id:

Re: [Qemu-devel] [PATCH 00/10] Support the Capstone disassembler

On 09/14/2017 03:35 PM, Richard Henderson wrote: As occasionally discussed on this list, due to licensing conflicts, we are restricted to a version of libopcodes that pre-dates its upstream re-licensing to gplv3. That makes our copy rather old and dated. I've already seen this as problematic

Re: [Qemu-devel] [PATCH 04/10] disas: Support the Capstone disassembler library

Hi Richard, see inlined comments. On 09/14/2017 03:35 PM, Richard Henderson wrote: If configured, prefer this over our rather dated copy of the GPLv2-only binutils. This will be especially apparent with the proposed vector extensions to TCG, as disas/i386.c does not handle AVX.

Re: [Qemu-devel] [RFC 00/15] QMP: out-of-band (OOB) execution support

On Thu, Sep 14, 2017 at 07:53:15PM +0100, Dr. David Alan Gilbert wrote: > * Marc-André Lureau (marcandre.lur...@gmail.com) wrote: > > Hi > > > > On Thu, Sep 14, 2017 at 9:50 AM, Peter Xu wrote: > > > This series was born from this one: > > > > > >

Re: [Qemu-devel] [PULL 00/18] ppc-for-2.11 queue 20170915

Hi, This series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PULL 00/18] ppc-for-2.11 queue 20170915 Message-id: 20170915035130.8354-1-da...@gibson.dropbear.id.au Type: series === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n=1

[Qemu-devel] [PULL 12/18] xics: fix several error leaks

From: Greg Kurz If object_property_get_link() fails then it allocates an error, which must be freed before returning. The error_get_pretty() function is merely an accessor to the error message and doesn't free anything. The error.h header indicates how to do it right: * Pass

[Qemu-devel] [PULL 13/18] spapr_cpu_core: fail gracefully with non-pseries machine types

From: Greg Kurz Since commit 7cca3e466eb0 ("ppc: spapr: Move VCPU ID calculation into sPAPR"), QEMU aborts when started with a *-spapr-cpu-core device and a non-pseries machine. Let's rely on the already existing call to object_dynamic_cast() instead of using the SPAPR_MACHINE()

[Qemu-devel] [PULL 17/18] spapr_events: use QTAILQ_FOREACH_SAFE() in spapr_clear_pending_events()

From: Greg Kurz QTAILQ_FOREACH_SAFE() must be used when removing the current element inside the loop block. This fixes a user-after-free error introduced by commit 56258174238eb and reported by Coverity (CID 1381017). Signed-off-by: Greg Kurz Signed-off-by:

[Qemu-devel] [PULL 10/18] spapr_pci: handle FDT creation errors with _FDT()

From: Greg Kurz libfdt failures when creating the FDT should cause QEMU to terminate. Let's use the _FDT() macro which does just that instead of propagating the error to the caller. spapr_populate_pci_child_dt() no longer needs to return a value in this case. Note that, on the

Re: [Qemu-devel] [RFC 00/15] QMP: out-of-band (OOB) execution support

On Thu, Sep 14, 2017 at 07:56:04PM +0100, Dr. David Alan Gilbert wrote: > * Peter Xu (pet...@redhat.com) wrote: > > This series was born from this one: > > > > https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg04310.html > > Are patches 1..6 separable and mergable without the rest ?

[Qemu-devel] [PULL 15/18] spapr_pci: don't create 64-bit MMIO window if we don't need to

From: Greg Kurz When running a pseries-2.2 or older machine type, we get the following lines in info mtree: address-space: memory ... - (prio 0, i/o): alias pci@8002000.mmio64-alias @p...@8002000.mmio

[Qemu-devel] [PULL 09/18] spapr_pci: use the common _FDT() helper

From: Greg Kurz All other users in hw/ppc already consider an error when building the FDT to be fatal, even on hotplug paths. There's no valid reason for spapr_pci to behave differently. So let's used the common _FDT() helper which terminates QEMU when libfdt fails.

[Qemu-devel] [PULL 16/18] spapr_cpu_core: cleaning up qdev_get_machine() calls

From: Greg Kurz This patch removes the qdev_get_machine() calls that are made in spapr_cpu_core.c in situations where we can get an existing pointer for the MachineState by either passing it as an argument to the function or by using other already available pointers. Credits to

[Qemu-devel] [PULL 07/18] ppc/xive: fix OV5_XIVE_EXPLOIT bits

From: Cédric Le Goater On POWER9, the Client Architecture Support (CAS) negotiation process determines whether the guest operates in XIVE Legacy compatibility or in XIVE exploitation mode. Now that we have initial guest support for the XIVE interrupt controller, let's fix the bits

[Qemu-devel] [PULL 01/18] net: Add SunGEM device emulation as found on Apple UniNorth

From: Benjamin Herrenschmidt This adds a simplistic emulation of the Sun GEM ethernet controller found in Apple ASICs. Currently we only support the Apple UniNorth 1.x variant, but the other Apple or Sun variants should mostly be a matter of adding PCI IDs options. We

[Qemu-devel] [PULL 18/18] ppc/kvm: use kvm_vm_check_extension() in kvmppc_is_pr()

From: Greg Kurz If the host has both KVM PR and KVM HV loaded and we pass: -machine pseries,accel=kvm,kvm-type=PR the kvmppc_is_pr() returns false instead of true. Since the helper is mostly used as fallback, it doesn't have any real impact with recent kernels. A

[Qemu-devel] [PULL 14/18] spapr_pci: convert sprintf() to g_strdup_printf()

From: Greg Kurz In order to follow a QEMU common practice. Signed-off-by: Greg Kurz Signed-off-by: David Gibson --- hw/ppc/spapr_pci.c | 21 - 1 file changed, 12 insertions(+), 9 deletions(-) diff --git

[Qemu-devel] [PULL 02/18] hw/ppc/spapr.c: cleaning up qdev_get_machine() calls

From: Daniel Henrique Barboza This patch removes the qdev_get_machine() calls that are made in spapr.c in situations where we can get an existing pointer for the MachineState by either passing it as an argument to the function or by using other already available

[Qemu-devel] [PULL 05/18] spapr_pci: use g_strdup_printf()

From: Greg Kurz Building strings with g_strdup_printf() instead of snprintf() is a QEMU common practice. Signed-off-by: Greg Kurz Signed-off-by: David Gibson --- hw/ppc/spapr_pci.c | 19 ++- 1 file changed, 10

[Qemu-devel] [PULL 06/18] spapr: only update SDR1 once per-cpu during CAS

From: Greg Kurz Commit b55d295e3ec9 added the possibility to support HPT resizing with KVM. In the case of PR, we need to pass the userspace address of the HPT to KVM using the SDR1 slot. This is handled by kvmppc_update_sdr1() which uses CPU_FOREACH() to update all CPUs. It is

[Qemu-devel] [PULL 08/18] spapr: fix CAS-generated reset

From: Cédric Le Goater The OV5_MMU_RADIX_300 requires special handling in the CAS negotiation process. It is cleared from the option vector of the guest before evaluating the changes and re-added later. But, when testing for a possible CAS reset : spapr->cas_reboot =

[Qemu-devel] [PULL 00/18] ppc-for-2.11 queue 20170915

The following changes since commit 3dabde1128b671f36ac6cb36b97b273139964420: Merge remote-tracking branch 'remotes/dgilbert/tags/pull-hmp-20170914' into staging (2017-09-14 16:33:02 +0100) are available in the git repository at: git://github.com/dgibson/qemu.git tags/ppc-for-2.11-20170915

[Qemu-devel] [PULL 04/18] spapr_pci: drop useless check in spapr_populate_pci_child_dt()

From: Greg Kurz spapr_phb_get_loc_code() either returns a non-null pointer, or aborts if g_strdup_printf() failed to allocate memory. Signed-off-by: Greg Kurz [dwg: Grammatical fix to commit message] Reviewed-by: Philippe Mathieu-Daudé

[Qemu-devel] [PULL 03/18] spapr_pci: drop useless check in spapr_phb_vfio_get_loc_code()

From: Greg Kurz g_strdup_printf() either returns a non-null pointer, or aborts if it failed to allocate memory. Signed-off-by: Greg Kurz [dwg: Grammatical fix to commit message] Acked-by: Philippe Mathieu-Daudé Signed-off-by: David Gibson

[Qemu-devel] [PULL 11/18] vfio, spapr: Fix levels calculation

From: Alexey Kardashevskiy The existing tries to round up the number of pages but @pages is always calculated as the rounded up value minus one which makes ctz64() always return 0 and have create.levels always set 1. This removes wrong "-1" and allows having more than 1 levels.

Re: [Qemu-devel] [RFC 00/15] QMP: out-of-band (OOB) execution support

On Thu, Sep 14, 2017 at 04:19:11PM +0100, Stefan Hajnoczi wrote: > On Thu, Sep 14, 2017 at 01:15:09PM +0200, Marc-André Lureau wrote: > > There should be a limit in the number of requests the thread can > > queue. Before the patch, the limit was enforced by system socket > > buffering I think.

Re: [Qemu-devel] [PATCH 1/3] kvm: check KVM_CAP_SYNC_MMU with kvm_vm_check_extension()

On Thu, Sep 14, 2017 at 09:25:22PM +0200, Greg Kurz wrote: > On a server-class ppc host, this capability depends on the KVM type, > ie, HV or PR. If both KVM are present in the kernel, we will always > get the HV specific value, even if we explicitely requested PR on > the command line. > > This

Re: [Qemu-devel] [RFC 14/15] qmp: support out-of-band (oob) execution

On Thu, Sep 14, 2017 at 04:33:34PM +0100, Stefan Hajnoczi wrote: > On Thu, Sep 14, 2017 at 03:50:35PM +0800, Peter Xu wrote: > > diff --git a/docs/devel/qapi-code-gen.txt b/docs/devel/qapi-code-gen.txt > > index 61fa167..47d16bb 100644 > > --- a/docs/devel/qapi-code-gen.txt > > +++

Re: [Qemu-devel] [PATCH 2/3] kvm: check KVM_CAP_NR_VCPUS with kvm_vm_check_extension()

On Thu, Sep 14, 2017 at 09:25:32PM +0200, Greg Kurz wrote: 1;4803;0c> On a modern server-class ppc host with the following CPU topology: > > Architecture: ppc64le > Byte Order:Little Endian > CPU(s):32 > On-line CPU(s) list: 0,8,16,24 > Off-line CPU(s) list:

Re: [Qemu-devel] [PATCH 10/10] target/mips: Support Capstone in disas_set_info

On 09/14/2017 03:35 PM, Richard Henderson wrote: Cc: Aurelien Jarno Cc: Yongbok Kim Signed-off-by: Richard Henderson --- target/mips/cpu.h| 2 ++ target/mips/cpu.c| 8

Re: [Qemu-devel] [PATCH] amd_iommu: Return error on machines with no PCI

On Thu, Sep 14, 2017 at 05:31:38PM -0300, Eduardo Habkost wrote: > On Thu, Sep 14, 2017 at 10:24:23PM +0200, Thomas Huth wrote: > > On 14.09.2017 22:18, Mohammed Gamal wrote: > > > Starting the following command line causes a segfault > > > qemu-system-x86_64 -S -machine isapc,accel=kvm -device

Re: [Qemu-devel] [RFC] Buffers/caches in VirtIO Balloon driver stats

On 09/12/2017 10:29 PM, Tomáš Golembiovský wrote: On Tue, 29 Aug 2017 20:01:53 +0800 Wei Wang wrote: On 08/29/2017 05:57 PM, Stefan Hajnoczi wrote: On Sun, Aug 27, 2017 at 11:30:33PM +0200, Tomáš Golembiovský wrote: Hi, I have CCed the relevant mailing lists and

Re: [Qemu-devel] [PATCH] scsi-bus: correct responses for INQUIRY and REQUEST SENSE

Quoting Paolo Bonzini (2017-08-22 08:20:23) > On 19/08/2017 00:55, Laszlo Ersek wrote: > > Reported-by: Laszlo Ersek > > Fixes: ded6ddc5a7b95217557fa360913d1213e12d4a6d > > And also: > > Cc: qemu-sta...@nongnu.org FYI patch freeze for v2.10.1 is Sep 25th. Hannes, were you

Re: [Qemu-devel] [Qemu-stable] [PATCH v2 0/2] vhost-user-bridge reconnect regression

Quoting Marc-André Lureau (2017-08-29 10:57:01) > Hi > > On Tue, Aug 29, 2017 at 5:34 PM, Michael S. Tsirkin wrote: > > On Tue, Aug 29, 2017 at 05:27:49PM +0200, Marc-André Lureau wrote: > >> Hi, > >> > >> libvhost-user doesn't support resuming with the same trick vubr had > >>

Re: [Qemu-devel] [Qemu-block] [PATCH] rbd: Detect rbd image resizes and propagate them

On Wed, Sep 13, 2017 at 6:47 PM, John Snow wrote: > On 09/13/2017 05:36 PM, Adam Wolfe Gordon via Qemu-devel wrote: >> On Wed, Sep 13, 2017 at 2:53 PM, John Snow wrote: >> We have a storage orchestration service that manages our ceph block >> storage clusters

Re: [Qemu-devel] [PATCH v2 5/5] arm: drop intermediate cpu_model -> cpu type parsing and use cpu type directly

On Thu, Sep 14, 2017 at 12:50 AM, Igor Mammedov wrote: > On Thu, 14 Sep 2017 00:47:20 -0300 > Philippe Mathieu-Daudé wrote: > >> Hi Igor, >> >> awesome clean refactor! > Thanks, > > there is more patches on this topic for other targets to post > but it's

[Qemu-devel] [Bug 1716767] Re: file(1) fails with "Invalid argument" on qemu-sh4-user

Please post patches to the qemu-devel mailing list for discussion instead of attaching them to the bugtracker. Thanks! -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1716767 Title: file(1) fails

Re: [Qemu-devel] [PATCH v2 3/6] aarch64-softmmu.mak: Use an ARM specific config

On Thu, Sep 14, 2017 at 5:50 AM, Peter Maydell wrote: > On 1 September 2017 at 22:00, Alistair Francis > wrote: >> In preperation for having an ARM and MicroBlaze ZynqMP machine let's >> split out the current ARM specific config options. >>

Re: [Qemu-devel] [PATCH] amd_iommu: Return error on machines with no PCI

On Thu, Sep 14, 2017 at 10:24:23PM +0200, Thomas Huth wrote: > On 14.09.2017 22:18, Mohammed Gamal wrote: > > Starting the following command line causes a segfault > > qemu-system-x86_64 -S -machine isapc,accel=kvm -device amd-iommu > > > > This is due to the fact that the machine type 'isapc'

Re: [Qemu-devel] [PATCH] intel_iommu: Return error on machines with no PCI

On 14.09.2017 22:17, Mohammed Gamal wrote: > Starting the following command line causes a segfault > qemu-system-x86_64 -S -machine isapc,accel=kvm -device intel-iommu > > This is due to the fact that the machine type 'isapc' doesn't have > a PCI bus, while intel_iommu doesn't check if the

Re: [Qemu-devel] [PATCH] amd_iommu: Return error on machines with no PCI

On 14.09.2017 22:18, Mohammed Gamal wrote: > Starting the following command line causes a segfault > qemu-system-x86_64 -S -machine isapc,accel=kvm -device amd-iommu > > This is due to the fact that the machine type 'isapc' doesn't have > a PCI bus, while amd_iommu doesn't check if the machine

Re: [Qemu-devel] [PATCH v8 06/13] qemu.py: make sure we only remove files we create

On Thu, Sep 14, 2017 at 10:18 PM, Eduardo Habkost wrote: > On Thu, Sep 14, 2017 at 10:05:50PM +0200, Amador Pahim wrote: >> On Thu, Sep 14, 2017 at 9:46 PM, Eduardo Habkost wrote: >> > On Thu, Sep 14, 2017 at 09:38:13PM +0200, Amador Pahim wrote: >> >>

[Qemu-devel] [PATCH] amd_iommu: Return error on machines with no PCI

Starting the following command line causes a segfault qemu-system-x86_64 -S -machine isapc,accel=kvm -device amd-iommu This is due to the fact that the machine type 'isapc' doesn't have a PCI bus, while amd_iommu doesn't check if the machine has PCI support and subsequently does a null-pointer

Re: [Qemu-devel] [PATCH v8 06/13] qemu.py: make sure we only remove files we create

On Thu, Sep 14, 2017 at 10:05:50PM +0200, Amador Pahim wrote: > On Thu, Sep 14, 2017 at 9:46 PM, Eduardo Habkost wrote: > > On Thu, Sep 14, 2017 at 09:38:13PM +0200, Amador Pahim wrote: > >> On Tue, Sep 5, 2017 at 5:18 AM, Fam Zheng wrote: > >> > On Fri,

[Qemu-devel] [PATCH] amd_iommu: Return error on machines with no PCI

Starting the following command line causes a segfault qemu-system-x86_64 -S -machine isapc,accel=kvm -device amd-iommu This is due to the fact that the machine type 'isapc' doesn't have a PCI bus, while amd_iommu doesn't check if the machine has PCI support and subsequently does a null-pointer

[Qemu-devel] [PATCH] intel_iommu: Return error on machines with no PCI

Starting the following command line causes a segfault qemu-system-x86_64 -S -machine isapc,accel=kvm -device intel-iommu This is due to the fact that the machine type 'isapc' doesn't have a PCI bus, while intel_iommu doesn't check if the machine has PCI support and subsequently does a

Re: [Qemu-devel] [PATCH v8 06/13] qemu.py: make sure we only remove files we create

On Thu, Sep 14, 2017 at 9:46 PM, Eduardo Habkost wrote: > On Thu, Sep 14, 2017 at 09:38:13PM +0200, Amador Pahim wrote: >> On Tue, Sep 5, 2017 at 5:18 AM, Fam Zheng wrote: >> > On Fri, 09/01 13:28, Amador Pahim wrote: >> >> To launch a VM, we need to create

Re: [Qemu-devel] [PATCH RESEND v7 0/3] Red Hat PCI bridge resource reserve capability

On Thu, Sep 14, 2017 at 11:15:43AM +0300, Aleksandr Bezzubikov wrote: > 2017-09-10 22:40 GMT+03:00 Marcel Apfelbaum : > > On 10/09/2017 21:34, Aleksandr Bezzubikov wrote: > >> And what about this series? The matching QEMU series has been applied, > >> that's why there should be

Re: [Qemu-devel] [PATCH v2 3/7] mips: split cpu_mips_realize_env() out of cpu_mips_init()

On Wed, Aug 30, 2017 at 07:52:21PM -0300, Philippe Mathieu-Daudé wrote: > so it can be used in mips_cpu_realizefn() in the next commit > > Signed-off-by: Philippe Mathieu-Daudé > Tested-by: Igor Mammedov > Tested-by: James Hogan

Re: [Qemu-devel] [PATCH v8 06/13] qemu.py: make sure we only remove files we create

On Thu, Sep 14, 2017 at 09:38:13PM +0200, Amador Pahim wrote: > On Tue, Sep 5, 2017 at 5:18 AM, Fam Zheng wrote: > > On Fri, 09/01 13:28, Amador Pahim wrote: > >> To launch a VM, we need to create basically two files: the monitor > >> socket (if it's a UNIX socket) and the qemu

Re: [Qemu-devel] [PATCH v8 06/13] qemu.py: make sure we only remove files we create

On Tue, Sep 5, 2017 at 5:18 AM, Fam Zheng wrote: > On Fri, 09/01 13:28, Amador Pahim wrote: >> To launch a VM, we need to create basically two files: the monitor >> socket (if it's a UNIX socket) and the qemu log file. >> >> For the qemu log file, we currently just open the path,

Re: [Qemu-devel] [PATCH v2 0/3] hostmem-file: Add "discard-data" option

Series queued on machine-next. On Thu, Aug 24, 2017 at 04:23:12PM -0300, Eduardo Habkost wrote: > This series adds a new "discard-data" option to > memory-backend-file. The new option will be useful if somebody > is sharing RAM contents on a pre-existing file using share=on, > but don't need

Re: [Qemu-devel] [PATCH v8 00/13] scripts/qemu.py fixes and cleanups

Patches 01-05 were queued on my python-next branch: https://github.com/ehabkost/qemu/commits/python-next Please use python-next as base for v9, as I plan to submit a pull request with the contents of python-next soon. On Fri, Sep 01, 2017 at 01:28:16PM +0200, Amador Pahim wrote: > Changes

[Qemu-devel] [PATCH 3/3] ppc/kvm: check some capabilities with kvm_vm_check_extension()

The following capabilities are VM specific: - KVM_CAP_PPC_SMT_POSSIBLE - KVM_CAP_PPC_HTAB_FD - KVM_CAP_PPC_ALLOC_HTAB If both KVM HV and KVM PR are present, checking them always return the HV value, even if we explicitely requested to use PR. This has no visible effect for

[Qemu-devel] [PATCH 1/3] kvm: check KVM_CAP_SYNC_MMU with kvm_vm_check_extension()

On a server-class ppc host, this capability depends on the KVM type, ie, HV or PR. If both KVM are present in the kernel, we will always get the HV specific value, even if we explicitely requested PR on the command line. This can have an impact if we're using hugepages or a balloon device. Since

[Qemu-devel] [PATCH 0/3] kvm: use kvm_vm_check_extension() with VM capabilities

Some VM capabilities are currently checked with kvm_check_extension(). This doesn't have any impact for most host architectures because they don't depend on the KVM type. However, this is a problem for server-class ppc hosts that can support the PR and HV KVM types. Both implementations can

[Qemu-devel] [PATCH 2/3] kvm: check KVM_CAP_NR_VCPUS with kvm_vm_check_extension()

On a modern server-class ppc host with the following CPU topology: Architecture: ppc64le Byte Order:Little Endian CPU(s):32 On-line CPU(s) list: 0,8,16,24 Off-line CPU(s) list: 1-7,9-15,17-23,25-31 Thread(s) per core:1 If both KVM PR and KVM HV loaded

Re: [Qemu-devel] [PATCH v2] osdep: Fix ROUND_UP(64-bit, 32-bit)

On 09/14/2017 06:49 AM, Eric Blake wrote: > When using bit-wise operations that exploit the power-of-two > nature of the second argument of ROUND_UP(), we still need to > ensure that the mask is as wide as the first argument (done > by using a ternary to force proper arithmetic promotion). >

Re: [Qemu-devel] [Bug 1716767] Re: file(1) fails with "Invalid argument" on qemu-sh4-user

On 09/13/2017 08:37 PM, James Clarke wrote: > With the attached patch, qemu-sh4-static now works for file: I can also confirm that the patch fixes the problem for me. Would be great if it could get merged in one form or another. -- .''`. John Paul Adrian Glaubitz : :' : Debian Developer -

[Qemu-devel] [Bug 932490] Re: Qemu fails on -fda /dev/fd0 when no medium is present

** Changed in: qemu Status: Incomplete => Won't Fix -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/932490 Title: Qemu fails on -fda /dev/fd0 when no medium is present Status in QEMU:

[Qemu-devel] [PATCH 3/4] tcg/sparc: Fully convert tcg_target_op_def

Signed-off-by: Richard Henderson --- tcg/sparc/tcg-target.inc.c | 239 ++--- 1 file changed, 137 insertions(+), 102 deletions(-) diff --git a/tcg/sparc/tcg-target.inc.c b/tcg/sparc/tcg-target.inc.c index

[Qemu-devel] [PATCH 4/4] tcg/mips: Fully convert tcg_target_op_def

Signed-off-by: Richard Henderson --- tcg/mips/tcg-target.inc.c | 324 -- 1 file changed, 170 insertions(+), 154 deletions(-) diff --git a/tcg/mips/tcg-target.inc.c b/tcg/mips/tcg-target.inc.c index 1c09ec7d5b..ce4030602f

[Qemu-devel] 087 failing without aio

Hi Kevin, 087 fails if you've not built with aio, do we have any easy tests that let us skip it? (I was building on a cleanly installed borrowed farm box for a different architecture and hadn't installed the full set). === aio=native without O_DIRECT === Testing: QMP_VERSION {"return": {}}

[Qemu-devel] [PATCH 0/4] tcg: Fully convert tcg_target_op_def

The tcg/aarch64 version has already been posted as a part of my vector operations patch set. This is simply source code quality improvement, therefore I skip tcg/tci. But this takes care of all of the rest. r~ Richard Henderson (4): tcg/arm: Fully convert tcg_target_op_def tcg/ppc: Fully

[Qemu-devel] [PATCH 1/4] tcg/arm: Fully convert tcg_target_op_def

Signed-off-by: Richard Henderson --- tcg/arm/tcg-target.inc.c | 186 +++ 1 file changed, 107 insertions(+), 79 deletions(-) diff --git a/tcg/arm/tcg-target.inc.c b/tcg/arm/tcg-target.inc.c index 14599a8685..98a12535a5

[Qemu-devel] [PATCH 2/4] tcg/ppc: Fully convert tcg_target_op_def

Signed-off-by: Richard Henderson --- tcg/ppc/tcg-target.inc.c | 321 +-- 1 file changed, 168 insertions(+), 153 deletions(-) diff --git a/tcg/ppc/tcg-target.inc.c b/tcg/ppc/tcg-target.inc.c index 8ffc7a7205..879885b68b

Re: [Qemu-devel] [RFC 00/15] QMP: out-of-band (OOB) execution support

* Peter Xu (pet...@redhat.com) wrote: > This series was born from this one: > > https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg04310.html Are patches 1..6 separable and mergable without the rest ? Dave > The design comes from Markus, and also the whole-bunch-of discussions > in

Re: [Qemu-devel] [RFC 00/15] QMP: out-of-band (OOB) execution support

* Marc-André Lureau (marcandre.lur...@gmail.com) wrote: > Hi > > On Thu, Sep 14, 2017 at 9:50 AM, Peter Xu wrote: > > This series was born from this one: > > > > https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg04310.html > > > > The design comes from Markus, and

[Qemu-devel] [PATCH 06/10] target/arm: Support Capstone in disas_set_info

Cc: qemu-...@nongnu.org Signed-off-by: Richard Henderson --- target/arm/cpu.c | 21 ++--- 1 file changed, 18 insertions(+), 3 deletions(-) diff --git a/target/arm/cpu.c b/target/arm/cpu.c index a1acce3c7a..92159ca0b1 100644 --- a/target/arm/cpu.c

[Qemu-devel] [PATCH 05/10] target/i386: Support Capstone in disas_set_info

Signed-off-by: Richard Henderson --- target/i386/cpu.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/target/i386/cpu.c b/target/i386/cpu.c index b869a69c53..c3980b3864 100644 --- a/target/i386/cpu.c +++ b/target/i386/cpu.c @@ -51,6 +51,8 @@ #include

[Qemu-devel] [PATCH 03/10] disas: Remove unused flags arguments

Now that every target is using the disas_set_info hook, the flags argument is unused. Remove it. Signed-off-by: Richard Henderson --- include/disas/disas.h | 4 ++-- include/exec/log.h| 4 ++-- disas.c | 15

[Qemu-devel] [PATCH 10/10] target/mips: Support Capstone in disas_set_info

Cc: Aurelien Jarno Cc: Yongbok Kim Signed-off-by: Richard Henderson --- target/mips/cpu.h| 2 ++ target/mips/cpu.c| 8 target/mips/translate_init.c | 36

[Qemu-devel] [PATCH 01/10] target/i386: Convert to disas_set_info hook

Signed-off-by: Richard Henderson --- disas.c | 22 ++ monitor.c | 21 - target/i386/cpu.c | 12 target/i386/translate.c | 8 +--- 4 files changed, 15 insertions(+), 48

[Qemu-devel] [PATCH 09/10] target/sparc: Support Capstone in disas_set_info

Cc: Mark Cave-Ayland Signed-off-by: Richard Henderson --- target/sparc/cpu.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/target/sparc/cpu.c b/target/sparc/cpu.c index 0806d699e6..7eabf410de 100644 --- a/target/sparc/cpu.c

[Qemu-devel] [PATCH 04/10] disas: Support the Capstone disassembler library

If configured, prefer this over our rather dated copy of the GPLv2-only binutils. This will be especially apparent with the proposed vector extensions to TCG, as disas/i386.c does not handle AVX. Signed-off-by: Richard Henderson --- include/disas/bfd.h | 4

[Qemu-devel] [PATCH 08/10] target/s390x: Support Capstone in disas_set_info

Signed-off-by: Richard Henderson --- target/s390x/cpu.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/target/s390x/cpu.c b/target/s390x/cpu.c index 74b3e4fd0d..a4f7ff3da8 100644 --- a/target/s390x/cpu.c +++ b/target/s390x/cpu.c @@ -42,6 +42,7 @@ #include

[Qemu-devel] [PATCH 02/10] target/ppc: Convert to disas_set_info hook

Cc: qemu-...@nongnu.org Signed-off-by: Richard Henderson --- disas.c | 33 - monitor.c | 5 - target/ppc/translate.c | 5 + target/ppc/translate_init.c | 21 + 4

[Qemu-devel] [PATCH 07/10] target/ppc: Support Capstone in disas_set_info

Cc: qemu-...@nongnu.org Signed-off-by: Richard Henderson --- target/ppc/translate_init.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/target/ppc/translate_init.c b/target/ppc/translate_init.c index c7b4a7c02a..b976784d21 100644 ---

[Qemu-devel] [PATCH 00/10] Support the Capstone disassembler

As occasionally discussed on this list, due to licensing conflicts, we are restricted to a version of libopcodes that pre-dates its upstream re-licensing to gplv3. That makes our copy rather old and dated. I've already seen this as problematic for s390x guest. I'm sure the same problem exists

Re: [Qemu-devel] [PATCH v2 3/5] vl.c: convert cpu_model to cpu type and set of global properties before machine_init()

On 09/13/2017 01:04 PM, Igor Mammedov wrote: All machines that support user specified cpu_model either call cpu_generic_init() or cpu_class_by_name()/CPUClass::parse_features to parse feature string and to get CPU type to create. Which leads to code duplication and hard-codding default CPU

Re: [Qemu-devel] [PATCH v2 11/19] s390x: allow only 1 CPU with TCG

On 06.09.2017 23:20, Richard Henderson wrote: > On 09/06/2017 11:16 AM, Matthew Rosato wrote: >> On 09/04/2017 11:43 AM, David Hildenbrand wrote: >>> Specifying more than 1 CPU (e.g. -smp 5) leads to SIGP errors (the >>> guest tries to bring these CPUs up but fails), because we don't support >>>

Re: [Qemu-devel] [Qemu devel v8 PATCH 2/5] msf2: Microsemi Smartfusion2 System Register block

+static uint64_t msf2_sysreg_read(void *opaque, hwaddr offset, +unsigned size) +{ +MSF2SysregState *s = opaque; +uint32_t ret = 0; + +offset >>= 2; +if (offset < ARRAY_SIZE(s->regs)) { This comment is controversial, I'll let Peter nod. The SYSREG behaves differently

[Qemu-devel] [PULL 02/18] target/arm: Clear exclusive monitor on v7M reset, exception entry/exit

For M profile we must clear the exclusive monitor on reset, exception entry and exception exit. We weren't doing any of these things; fix this bug. Signed-off-by: Peter Maydell Reviewed-by: Alistair Francis Reviewed-by: Richard Henderson

[Qemu-devel] [PULL 03/18] target/arm: Get PRECISERR and IBUSERR the right way round

For a bus fault, the M profile BFSR bit PRECISERR means a bus fault on a data access, and IBUSERR means a bus fault on an instruction access. We had these the wrong way around; fix this. Signed-off-by: Peter Maydell Reviewed-by: Alistair Francis

[Qemu-devel] [PULL 08/18] xlnx-ep108: Rename to ZCU102

From: Alistair Francis The EP108 is a early access development board. Now that silicon is in production people have access to the ZCU102. Let's rename the internal QEMU files and variables to use the ZCU102. There is no functional change here as the EP108 is still a

Re: [Qemu-devel] [PATCH v2 5/5] arm: drop intermediate cpu_model -> cpu type parsing and use cpu type directly

On Wed, Sep 13, 2017 at 9:04 AM, Igor Mammedov wrote: > there are 2 use cases to deal with: > 1: fixed CPU models per board/soc > 2: boards with user configurable cpu_model and fallback to > default cpu_model if user hasn't specified one explicitly > > For the 1st >

[Qemu-devel] [PULL 09/18] xlnx-zcu102: Manually create the machines

From: Alistair Francis In preperation for future work let's manually create the Xilnx machines. This will allow us to set properties for the machines in the future. Signed-off-by: Alistair Francis Reviewed-by: Edgar E. Iglesias

[Qemu-devel] [PULL 07/18] target/arm: Rename 'type' to 'excret' in do_v7m_exception_exit()

In the v7M and v8M ARM ARM, the magic exception return values are referred to as EXC_RETURN values, and in QEMU we use V7M_EXCRET_* constants to define bits within them. Rename the 'type' variable which holds the exception return value in do_v7m_exception_exit() to excret, making it clearer that

[Qemu-devel] [PULL 10/18] xlnx-zcu102: Add a machine level secure property

From: Alistair Francis Add a machine level secure property. This defaults to false and can be set to true using this machine command line argument: -machine xlnx-zcu102,secure=on This follows what the ARM virt machine does. This property only applies to the

[Qemu-devel] [PULL 06/18] target/arm: Add and use defines for EXCRET constants

The exception-return magic values get some new bits in v8M, which makes some bit definitions for them worthwhile. We don't use the bit definitions for the switch on the low bits which checks the return type for v7M, because this is defined in the v7M ARM ARM as a set of valid values rather than

[Qemu-devel] [PULL 05/18] target/arm: Remove unnecessary '| 0xf0000000' from do_v7m_exception_exit()

In do_v7m_exception_exit(), there's no need to force the high 4 bits of 'type' to 1 when calling v7m_exception_taken(), because we know that they're always 1 or we could not have got to this "handle return to magic exception return address" code. Remove the unnecessary ORs. Signed-off-by: Peter

[Qemu-devel] [PULL 16/18] hw/arm/virt: Set INTx/gsi mapping

From: Pranavkumar Sawargaonkar Let's provide the GPEX host bridge with the INTx/gsi mapping. This is needed for INTx/gsi routing. Signed-off-by: Pranavkumar Sawargaonkar Signed-off-by: Tushar Jagad Signed-off-by: Eric

  1   2   3   4   >