Re: [PATCH v3 1/2] Introduce (x86) CPU model deprecation API

2020-09-18 Thread Robert Hoo
On Fri, 2020-09-18 at 16:42 +, Eduardo Habkost wrote: > ... > > > > --- > > > > Changelog > > > > v3: > > > > Make the deprecation implementation CPUClass generic. > > > > > > > > v2: > > > > Move deprecation check from parse_cpu_option() to > > > > machine_run_board_init(), so > > > > that

[PATCH v2] Add a comment in bios-tables-test.c to clarify the reason behind approach

2020-09-18 Thread Ani Sinha
A comment blob is added in bios-tables-test.c that explains the reasoning behind the process of updating the ACPI table blobs when new tests are added or old tests are modified or code is committed that affect tests. The explanation would help future contributors follow the correct process when

Re: [PULL 00/17] QOM queue, 2020-09-18

2020-09-18 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200918204714.27276-1-ehabk...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20200918204714.27276-1-ehabk...@redhat.com Subject: [PULL 00/17] QOM queue, 2020-09-18

Re: [PATCH v1 0/2] Add timeout mechanism to qmp actions

2020-09-18 Thread Zhenyu Ye
On 2020/9/18 22:06, Fam Zheng wrote: > > I can see how blocking in a slow io_submit can cause trouble for main > thread. I think one way to fix it (until it's made truly async in new > kernels) is moving the io_submit call to thread pool, and wrapped in a > coroutine, perhaps. > I'm not sure if

Re: [PULL 00/17] QOM queue, 2020-09-18

2020-09-18 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200918204714.27276-1-ehabk...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20200918204714.27276-1-ehabk...@redhat.com Subject: [PULL 00/17] QOM queue, 2020-09-18

Re: [PULL 00/17] QOM queue, 2020-09-18

2020-09-18 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200918204714.27276-1-ehabk...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20200918204714.27276-1-ehabk...@redhat.com Subject: [PULL 00/17] QOM queue, 2020-09-18

Re: [PULL 00/17] QOM queue, 2020-09-18

2020-09-18 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200918204714.27276-1-ehabk...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20200918204714.27276-1-ehabk...@redhat.com Subject: [PULL 00/17] QOM queue, 2020-09-18

Re: [PATCH V9 6/6] hw/mips: Add Loongson-3 machine support

2020-09-18 Thread Huacai Chen
Hi, Philippe, On Thu, Sep 17, 2020 at 3:53 PM Philippe Mathieu-Daudé wrote: > > On 9/16/20 12:47 PM, Philippe Mathieu-Daudé wrote: > > On 9/16/20 11:49 AM, Huacai Chen wrote: > >> On Wed, Sep 16, 2020 at 3:56 PM Philippe Mathieu-Daudé > >> wrote: > >>> On 9/16/20 4:12 AM, Huacai Chen wrote: >

Re: [PATCH V9 3/6] target/mips: Add loongson-ext lswc2 group of instructions (Part 1)

2020-09-18 Thread Huacai Chen
Hi, Richard, On Wed, Sep 16, 2020 at 11:15 PM Richard Henderson wrote: > > On 9/15/20 7:12 PM, Huacai Chen wrote: > > +case OPC_GSLQ: > > +gen_base_offset_addr(ctx, t0, rs, lsq_offset); > > +tcg_gen_qemu_ld_tl(t0, t0, ctx->mem_idx, MO_TEQ | > > +

Re: [PULL 00/17] QOM queue, 2020-09-18

2020-09-18 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200918204714.27276-1-ehabk...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20200918204714.27276-1-ehabk...@redhat.com Subject: [PULL 00/17] QOM queue, 2020-09-18

Re: [PULL 00/17] QOM queue, 2020-09-18

2020-09-18 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200918204714.27276-1-ehabk...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20200918204714.27276-1-ehabk...@redhat.com Subject: [PULL 00/17] QOM queue, 2020-09-18

Re: [PULL 00/17] QOM queue, 2020-09-18

2020-09-18 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200918204714.27276-1-ehabk...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20200918204714.27276-1-ehabk...@redhat.com Subject: [PULL 00/17] QOM queue, 2020-09-18

[Bug 1896298] Re: memory leak

2020-09-18 Thread Michael Slade
Note, this also occurs with freeDOS 1.2, at least. Note 2, 4.2 stable does not exhibit the bug. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1896298 Title: memory leak Status in QEMU: New

[Bug 1895703] Re: performance degradation in tcg since Meson switch

2020-09-18 Thread Paolo Bonzini
This patch shold fix the regression: diff --git a/configure b/configure index 0004c46525..0786144043 100755 --- a/configure +++ b/configure @@ -7414,6 +7414,7 @@ NINJA=${ninja:-$PWD/ninjatool} $meson setup \ -Dwerror=$(if test "$werror" = yes; then echo true; else echo false; fi) \

Re: [PULL 00/17] QOM queue, 2020-09-18

2020-09-18 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200918204714.27276-1-ehabk...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20200918204714.27276-1-ehabk...@redhat.com Subject: [PULL 00/17] QOM queue, 2020-09-18

Re: [PATCH 0/6] Convert pc-bios Makefiles to meson

2020-09-18 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200918204759.225810-1-marcandre.lur...@redhat.com/ Hi, This series failed the docker-quick@centos7 build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST

Re: [PATCH RFC 22/22] i386: expand Hyper-V features early

2020-09-18 Thread Eduardo Habkost
On Fri, Sep 04, 2020 at 04:54:31PM +0200, Vitaly Kuznetsov wrote: > To make Hyper-V features appear in e.g. QMP query-cpu-model-expansion we > need to expand and set the corresponding CPUID leaves early. Modify > x86_cpu_get_supported_feature_word() to call newly intoduced Hyper-V > specific

[Bug 1896298] [NEW] memory leak

2020-09-18 Thread Michael Slade
Public bug reported: qemu trunk as of today leaks memory FAST when freedos' edit is running. To reproduce, download: https://www.ibiblio.org/pub/micro/pc- stuff/freedos/files/repositories/1.3/cdrom.iso Then run: $ qemu-system-i386 -cdrom cdrom.iso select your language then select "return to

Re: [PATCH RFC 22/22] i386: expand Hyper-V features early

2020-09-18 Thread Eduardo Habkost
On Fri, Sep 04, 2020 at 04:54:31PM +0200, Vitaly Kuznetsov wrote: > To make Hyper-V features appear in e.g. QMP query-cpu-model-expansion we > need to expand and set the corresponding CPUID leaves early. Modify > x86_cpu_get_supported_feature_word() to call newly intoduced Hyper-V > specific

Re: [PULL 00/17] QOM queue, 2020-09-18

2020-09-18 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200918204714.27276-1-ehabk...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20200918204714.27276-1-ehabk...@redhat.com Subject: [PULL 00/17] QOM queue, 2020-09-18

[Bug 1896096] Re: Git version: Build process is broken in block_curl.c.o

2020-09-18 Thread Peter Maydell
Huh. I thought we ignored the environment LDFLAGS/CFLAGS from much earlier than 5.0 and was about to ask if honouring it was a change introduced with the Meson switch! My (not necessarily very considered) opinion is that generally it's better not to honour the environment CFLAGS and LDFLAGS

Re: [PATCH RFC 12/22] i386: always fill Hyper-V CPUID feature leaves from X86CPU data

2020-09-18 Thread Eduardo Habkost
On Fri, Sep 04, 2020 at 04:54:21PM +0200, Vitaly Kuznetsov wrote: > We have all the required data in X86CPU already and as we are about to > split hyperv_handle_properties() into hyperv_expand_features()/ > hyperv_fill_cpuids() we can remove the blind copy. The functional change > is that QEMU

Re: [PATCH RFC 09/22] i386: add reserved FEAT_HYPERV_ECX CPUID leaf

2020-09-18 Thread Eduardo Habkost
On Fri, Sep 04, 2020 at 04:54:18PM +0200, Vitaly Kuznetsov wrote: > As a preparation to expanding Hyper-V CPU features early, add > reserved FEAT_HYPERV_ECX CPUID leaf. > > Signed-off-by: Vitaly Kuznetsov > --- > target/i386/cpu.c | 5 + > target/i386/cpu.h | 1 + > target/i386/kvm.c | 2 ++

Re: [PATCH RFC 07/22] i386: fill in FEAT_HYPERV_EDX from edx instead of eax

2020-09-18 Thread Eduardo Habkost
On Fri, Sep 04, 2020 at 04:54:16PM +0200, Vitaly Kuznetsov wrote: > There was a typo which went unnoticed. > > Fixes: e48ddcc6ce13 ("i386/kvm: implement 'hv-passthrough' mode") > Signed-off-by: Vitaly Kuznetsov Similar fix had been submitted at:

Re: [PATCH RFC 04/22] i386: move hyperv_interface_id initialization to x86_cpu_realizefn()

2020-09-18 Thread Eduardo Habkost
On Fri, Sep 04, 2020 at 04:54:13PM +0200, Vitaly Kuznetsov wrote: > As a preparation to expanding Hyper-V CPU features early, move > hyperv_interface_id initialization to x86_cpu_realizefn(). > > Signed-off-by: Vitaly Kuznetsov > --- > target/i386/cpu.c | 6 ++ > target/i386/cpu.h | 1 + >

RE: [PATCH v3 06/15] hw/block/nvme: Add support for Namespace Types

2020-09-18 Thread Dmitry Fomichev
> -Original Message- > From: Klaus Jensen > Sent: Friday, September 18, 2020 5:30 PM > To: Dmitry Fomichev > Cc: Keith Busch ; Klaus Jensen > ; Kevin Wolf ; Philippe > Mathieu-Daudé ; Maxim Levitsky > ; Fam Zheng ; Niklas Cassel > ; Damien Le Moal ; > qemu-bl...@nongnu.org;

Re: [Qemu-devel] [PATCH] i386/kvm: fix FEATURE_HYPERV_EDX value in hyperv_passthrough case

2020-09-18 Thread Eduardo Habkost
On Tue, Aug 20, 2019 at 06:30:30PM +0800, Zhenyu Wang wrote: > Fix typo to use correct edx value for FEATURE_HYPERV_EDX when > hyperv_passthrough is enabled. > > Signed-off-by: Zhenyu Wang CCing other maintainers (please use ./scripts/get_maintainer.pl to make sure maintainers don't miss

Re: [PATCH RFC 06/22] i386: move hyperv_limits initialization to x86_cpu_realizefn()

2020-09-18 Thread Eduardo Habkost
On Fri, Sep 04, 2020 at 04:54:15PM +0200, Vitaly Kuznetsov wrote: > As a preparation to expanding Hyper-V CPU features early, move > hyperv_limits initialization to x86_cpu_realizefn(). > > Signed-off-by: Vitaly Kuznetsov > --- > target/i386/cpu.c | 5 + > target/i386/cpu.h | 1 + >

Re: [PATCH RFC 05/22] i386: move hyperv_version_id initialization to x86_cpu_realizefn()

2020-09-18 Thread Eduardo Habkost
On Fri, Sep 04, 2020 at 04:54:14PM +0200, Vitaly Kuznetsov wrote: > As a preparation to expanding Hyper-V CPU features early, move > hyperv_version_id initialization to x86_cpu_realizefn(). > > Signed-off-by: Vitaly Kuznetsov > --- > target/i386/cpu.c | 4 > target/i386/cpu.h | 1 + >

Re: [PATCH RFC 03/22] i386: move hyperv_vendor_id initialization to x86_cpu_realizefn()

2020-09-18 Thread Eduardo Habkost
On Fri, Sep 04, 2020 at 04:54:12PM +0200, Vitaly Kuznetsov wrote: > As a preparation to expanding Hyper-V CPU features early, move > hyperv_vendor_id initialization to x86_cpu_realizefn(). > > Signed-off-by: Vitaly Kuznetsov > --- > target/i386/cpu.c | 15 ++- > target/i386/cpu.h |

[Bug 1896096] Re: Git version: Build process is broken in block_curl.c.o

2020-09-18 Thread Toolybird
Chiming in here as the user who initially suggested Arch use this: --extra-ldflags="$LDFLAGS" qemu-5.0.0 introduced a breaking change whereby LDFLAGS from the environment were ignored. For Arch, this resulted in exclusion of `--as- needed' from the link, which naturally caused dependency chaos.

RE: [PATCH v3 10/15] hw/block/nvme: Support Zoned Namespace Command Set

2020-09-18 Thread Dmitry Fomichev
> -Original Message- > From: Klaus Jensen > Sent: Friday, September 18, 2020 5:20 PM > To: Dmitry Fomichev > Cc: Keith Busch ; Klaus Jensen > ; Kevin Wolf ; Philippe > Mathieu-Daudé ; Maxim Levitsky > ; Fam Zheng ; Niklas Cassel > ; Damien Le Moal ; > qemu-bl...@nongnu.org;

Re: [PATCH v3 10/15] hw/block/nvme: Support Zoned Namespace Command Set

2020-09-18 Thread Klaus Jensen
On Sep 14 07:14, Dmitry Fomichev wrote: > The emulation code has been changed to advertise NVM Command Set when > "zoned" device property is not set (default) and Zoned Namespace > Command Set otherwise. > > Handlers for three new NVMe commands introduced in Zoned Namespace > Command Set

Re: [PATCH] smp: drop support for deprecated (invalid topologies)

2020-09-18 Thread Eduardo Habkost
On Mon, Sep 14, 2020 at 09:37:20AM +0200, Igor Mammedov wrote: > On Fri, 11 Sep 2020 11:04:47 -0400 > "Michael S. Tsirkin" wrote: > > > On Fri, Sep 11, 2020 at 09:32:02AM -0400, Igor Mammedov wrote: > > > it's was deprecated since 3.1 > > > > > > Support for invalid topologies is removed, the

[PATCH 6/6] meson: replace pc-bios/Makefile

2020-09-18 Thread marcandre . lureau
From: Marc-André Lureau Add an alias target "update-dtb" to recompile the device-tree "sources" to the "blob" format. Signed-off-by: Marc-André Lureau --- pc-bios/Makefile| 19 --- pc-bios/meson.build | 12 2 files changed, 12 insertions(+), 19 deletions(-)

Re: [PATCH v1 0/3] Remove the limitation of Intel PT CPUID info

2020-09-18 Thread Eduardo Habkost
Hi Luwei Kang, I was looking for info on intel-pt and just saw this series, and it was never reviewed or merged (sorry for missing it!). Is this still the approach we want to follow for intel-pt? I'm CCing Jiri Denemark because this might be relevant for a libvirt issue related to intel-pt we

[PATCH 0/6] Convert pc-bios Makefiles to meson

2020-09-18 Thread marcandre . lureau
From: Marc-André Lureau Hi, Here is a series of patches to convert the remaining Makefile in pc-bios/ to meson. I have done various tests to check that the resulting binaries are working as expected, but I didn't cover it all, and I am not sure the test su= ite covers them all either. Please

Re: [PATCH v6 10/10] i386: Simplify CPUID_8000_001E for AMD

2020-09-18 Thread Babu Moger
On 9/18/20 4:38 PM, Eduardo Habkost wrote: > On Mon, Aug 31, 2020 at 01:43:07PM -0500, Babu Moger wrote: >> apic_id contains all the information required to build >> CPUID_8000_001E. core_id and node_id is already part of >> apic_id generated by x86_topo_ids_from_apicid. >> >> Also remove the

[PULL 15/17] qom: Remove module_obj_name parameter from OBJECT_DECLARE* macros

2020-09-18 Thread Eduardo Habkost
One of the goals of having less boilerplate on QOM declarations is to avoid human error. Requiring an extra argument that is never used is an opportunity for mistakes. Remove the unused argument from OBJECT_DECLARE_TYPE and OBJECT_DECLARE_SIMPLE_TYPE. Coccinelle patch used to convert all users

[PULL 11/17] target/riscv: Set instance_align on RISCVCPU TypeInfo

2020-09-18 Thread Eduardo Habkost
From: Richard Henderson Fix alignment of CPURISCVState.vreg. Signed-off-by: Richard Henderson Reviewed-by: Alistair Francis Message-Id: <20200916004638.2444147-6-richard.hender...@linaro.org> Signed-off-by: Eduardo Habkost --- target/riscv/cpu.c | 1 + 1 file changed, 1 insertion(+) diff

[PULL 08/17] qom: Allow objects to be allocated with increased alignment

2020-09-18 Thread Eduardo Habkost
From: Richard Henderson It turns out that some hosts have a default malloc alignment less than that required for vectors. We assume that, with compiler annotation on CPUArchState, that we can properly align the vector portion of the guest state. Fix the alignment of the allocation by using

[PULL 00/17] QOM queue, 2020-09-18

2020-09-18 Thread Eduardo Habkost
The following changes since commit 053a4177817db307ec854356e95b5b350800a216: Merge remote-tracking branch 'remotes/philmd-gitlab/tags/fw_cfg-20200918' into staging (2020-09-18 16:34:26 +0100) are available in the Git repository at: git://github.com/ehabkost/qemu.git tags/machine-next-pull

[PULL 01/17] sifive_e: Rename memmap enum constants

2020-09-18 Thread Eduardo Habkost
Some of the enum constant names conflict with a QOM type check macro (SIFIVE_E_PRCI). This needs to be addressed to allow us to transform the QOM type check macros into functions generated by OBJECT_DECLARE_TYPE(). Rename all the constants to SIFIVE_E_DEV_*, to avoid conflicts. Signed-off-by:

[PATCH v2 16/17] pci: allocate pci id for nvme

2020-09-18 Thread Klaus Jensen
From: Klaus Jensen The emulated nvme device (hw/block/nvme.c) is currently using an internal Intel device id. Prepare to change that by allocating a device id under the 1b36 (Red Hat, Inc.) vendor id. Signed-off-by: Klaus Jensen Acked-by: Keith Busch Acked-by: Gerd Hoffmann Reviewed-by:

Re: [PATCH v6 10/10] i386: Simplify CPUID_8000_001E for AMD

2020-09-18 Thread Eduardo Habkost
On Mon, Aug 31, 2020 at 01:43:07PM -0500, Babu Moger wrote: > apic_id contains all the information required to build > CPUID_8000_001E. core_id and node_id is already part of > apic_id generated by x86_topo_ids_from_apicid. > > Also remove the restriction on number bits on core_id and > node_id.

tools/virtiofs: Multi threading seems to hurt performance

2020-09-18 Thread Vivek Goyal
Hi All, virtiofsd default thread pool size is 64. To me it feels that in most of the cases thread pool size 1 performs better than thread pool size 64. I ran virtiofs-tests. https://github.com/rhvgoyal/virtiofs-tests And here are the comparision results. To me it seems that by default we

[PATCH v2 15/17] hw/block/nvme: support multiple namespaces

2020-09-18 Thread Klaus Jensen
From: Klaus Jensen This adds support for multiple namespaces by introducing a new 'nvme-ns' device model. The nvme device creates a bus named from the device name ('id'). The nvme-ns devices then connect to this and registers themselves with the nvme device. This changes how an nvme device is

Re: [PATCH v3 06/15] hw/block/nvme: Add support for Namespace Types

2020-09-18 Thread Klaus Jensen
On Sep 14 07:14, Dmitry Fomichev wrote: > From: Niklas Cassel > > Namespace Types introduce a new command set, "I/O Command Sets", > that allows the host to retrieve the command sets associated with > a namespace. Introduce support for the command set and enable > detection for the NVM Command

Re: [PATCH v3 01/15] hw/block/nvme: Define 64 bit cqe.result

2020-09-18 Thread Keith Busch
On Tue, Sep 15, 2020 at 10:48:49PM +0200, Klaus Jensen wrote: > On Sep 15 20:44, Dmitry Fomichev wrote: > > > > It is not necessary to change it in NST patch since result64 field is not > > used > > in that patch. But if you insist, I can move it to NST patch :) > > You are right of course, but

[PATCH v2 14/17] hw/block/nvme: refactor identify active namespace id list

2020-09-18 Thread Klaus Jensen
From: Klaus Jensen Prepare to support inactive namespaces. Signed-off-by: Klaus Jensen Reviewed-by: Maxim Levitsky --- hw/block/nvme.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/hw/block/nvme.c b/hw/block/nvme.c index d2e211c7cbc2..f533eec55c5c 100644 ---

Re: [PATCH v2] qom: simplify object_find_property / object_class_find_property

2020-09-18 Thread Eduardo Habkost
On Mon, Sep 14, 2020 at 02:56:17PM +0100, Daniel P. Berrangé wrote: > When debugging QEMU it is often useful to put a breakpoint on the > error_setg_internal method impl. > > Unfortunately the object_property_add / object_class_property_add > methods call object_property_find /

[PATCH 1/6] meson: convert pc-bios/s390-ccw

2020-09-18 Thread marcandre . lureau
From: Marc-André Lureau Use a fake subproject: this is a simple workaround to avoid the incompatible flags (such as gprof) that are set globally for the rest of QEMU. Signed-off-by: Marc-André Lureau --- configure| 2 - pc-bios/meson.build | 4 +

[PATCH 3/6] meson: convert pc-bios/optionrom

2020-09-18 Thread marcandre . lureau
From: Marc-André Lureau Checking for linker emulation is a bit tricky with meson, since it actually tries to compile a file with a main(). I didn't bother to port the Darwin or SunOS check, as they are quite old, and not very specific to the compiler version/issue, so they might be outdated.

[PATCH 2/6] optionrom: make kvmapic.S compile with clang

2020-09-18 Thread marcandre . lureau
From: Marc-André Lureau Clang doesn't support specifying segment prefixes before the instruction, and requires specifying them on the address. Signed-off-by: Marc-André Lureau --- pc-bios/optionrom/kvmvapic.S | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH v2 12/17] hw/block/nvme: add support for scatter gather lists

2020-09-18 Thread Klaus Jensen
From: Klaus Jensen For now, support the Data Block, Segment and Last Segment descriptor types. See NVM Express 1.3d, Section 4.4 ("Scatter Gather List (SGL)"). Signed-off-by: Klaus Jensen Signed-off-by: Klaus Jensen Acked-by: Keith Busch --- include/block/nvme.h | 6 +- hw/block/nvme.c

[PULL 13/17] scripts/codeconverter: Update to latest version

2020-09-18 Thread Eduardo Habkost
I'm not documenting every single change in the codeconverter script because most of that code will be deleted once we finish the QOM code conversion. This patch updates the script to the latest version that was used to perform changes in the QOM code. Signed-off-by: Eduardo Habkost Message-Id:

[PULL 06/17] qom: Clean up object_property_get_enum()'s error value

2020-09-18 Thread Eduardo Habkost
From: Markus Armbruster object_property_get_enum() is the only object_property_FOO() that is documented to return an undefined value on error. It does no such thing, actually: it returns 0 on some errors, and -1 on others. Needlessly complicated. Always return -1 on error, and adjust the

[PULL 16/17] Use OBJECT_DECLARE_TYPE when possible

2020-09-18 Thread Eduardo Habkost
This converts existing DECLARE_OBJ_CHECKERS usage to OBJECT_DECLARE_TYPE when possible. $ ./scripts/codeconverter/converter.py -i \ --pattern=AddObjectDeclareType $(git grep -l '' -- '*.[ch]') Signed-off-by: Eduardo Habkost Reviewed-by: Cédric Le Goater Acked-by: Paul Durrant Message-Id:

[PATCH 5/6] build-sys: remove recurse-* rules

2020-09-18 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- Makefile | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/Makefile b/Makefile index cb23123606..4728fcf1bd 100644 --- a/Makefile +++ b/Makefile @@ -135,13 +135,12 @@ Makefile: ; configure: ; .PHONY: all

[PULL 02/17] sifive_u: Rename memmap enum constants

2020-09-18 Thread Eduardo Habkost
Some of the enum constant names conflict with the QOM type check macros (SIFIVE_U_OTP, SIFIVE_U_PRCI). This needs to be addressed to allow us to transform the QOM type check macros into functions generated by OBJECT_DECLARE_TYPE(). Rename all the constants to SIFIVE_U_DEV_*, to avoid conflicts.

[PULL 14/17] qom: Remove ParentClassType argument from OBJECT_DECLARE_SIMPLE_TYPE

2020-09-18 Thread Eduardo Habkost
The requirement to specify the parent class type makes the macro harder to use and easy to misuse (silent bugs can be introduced if the wrong struct type is specified). Simplify the macro by just not declaring any class struct, allowing us to remove the class_size field from the TypeInfo

[PATCH 4/6] build-sys: remove no longer needed ROMS variable

2020-09-18 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- Makefile | 9 - configure | 3 --- 2 files changed, 12 deletions(-) diff --git a/Makefile b/Makefile index 7c60b9dcb8..cb23123606 100644 --- a/Makefile +++ b/Makefile @@ -180,16 +180,7 @@ slirp/all: .git-submodule-status

[PATCH v2 08/17] hw/block/nvme: add symbolic command name to trace events

2020-09-18 Thread Klaus Jensen
From: Klaus Jensen Add the symbolic command name to the pci_nvme_{io,admin}_cmd and pci_nvme_rw trace events. Signed-off-by: Klaus Jensen Reviewed-by: Philippe Mathieu-Daudé --- hw/block/nvme.h | 28 hw/block/nvme.c | 8 +---

[PULL 09/17] target/arm: Set instance_align on CPUARM TypeInfo

2020-09-18 Thread Eduardo Habkost
From: Richard Henderson Fix alignment of CPUARMState.vfp.zregs. Signed-off-by: Richard Henderson Message-Id: <20200916004638.2444147-4-richard.hender...@linaro.org> Signed-off-by: Eduardo Habkost --- target/arm/cpu.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/target/arm/cpu.c

[PULL 10/17] target/ppc: Set instance_align on PowerPCCPU TypeInfo

2020-09-18 Thread Eduardo Habkost
From: Richard Henderson Fix alignment of CPUPPCState.vsr. Signed-off-by: Richard Henderson Acked-by: David Gibson Message-Id: <20200916004638.2444147-5-richard.hender...@linaro.org> Signed-off-by: Eduardo Habkost --- target/ppc/translate_init.c.inc | 1 + 1 file changed, 1 insertion(+)

[PATCH v2 17/17] hw/block/nvme: change controller pci id

2020-09-18 Thread Klaus Jensen
From: Klaus Jensen There are two reasons for changing this: 1. The nvme device currently uses an internal Intel device id. 2. Since commits "nvme: fix write zeroes offset and count" and "nvme: support multiple namespaces" the controller device no longer has the quirks that the

[PULL 05/17] qom: Correct object_class_dynamic_cast_assert() documentation

2020-09-18 Thread Eduardo Habkost
object_class_dynamic_cast_assert() is not used by INTERFACE_CHECK, remove misleading mention of that function in the documentation. Message-Id: <20200916193101.511600-2-ehabk...@redhat.com> Signed-off-by: Eduardo Habkost --- include/qom/object.h | 2 +- 1 file changed, 1 insertion(+), 1

[PULL 12/17] target/s390x: Set instance_align on S390CPU TypeInfo

2020-09-18 Thread Eduardo Habkost
From: Richard Henderson Fix alignment of CPUS390XState.vregs. Signed-off-by: Richard Henderson Message-Id: <20200916004638.2444147-7-richard.hender...@linaro.org> Signed-off-by: Eduardo Habkost --- target/s390x/cpu.c | 1 + 1 file changed, 1 insertion(+) diff --git a/target/s390x/cpu.c

[PULL 03/17] sifive: Move QOM typedefs and add missing includes

2020-09-18 Thread Eduardo Habkost
Some typedefs and macros are defined after the type check macros. This makes it difficult to automatically replace their definitions with OBJECT_DECLARE_TYPE. Patch generated using: $ ./scripts/codeconverter/converter.py -i \ --pattern=QOMStructTypedefSplit $(git grep -l '' -- '*.[ch]')

[PATCH v2 03/17] hw/block/nvme: handle dma errors

2020-09-18 Thread Klaus Jensen
From: Klaus Jensen Handling DMA errors gracefully is required for the device to pass the block/011 test ("disable PCI device while doing I/O") in the blktests suite. With this patch the device passes the test by retrying "critical" transfers (posting of completion entries and processing of

[PULL 07/17] qom: Correct error values in two contracts

2020-09-18 Thread Eduardo Habkost
From: Markus Armbruster object_property_get_bool()'s contract claims it returns NULL on error. Pasto; it returns false. object_property_get_int()'s contract claims it returns "negative". It actually returns -1. All the other object_property_get_FOO() contracts specify the exact error value,

[PULL 04/17] sifive: Use DECLARE_*CHECKER* macros

2020-09-18 Thread Eduardo Habkost
$ ./scripts/codeconverter/converter.py -i \ --pattern=TypeCheckMacro $(git grep -l '' -- '*.[ch]') Signed-off-by: Eduardo Habkost Reviewed-by: Daniel P. Berrangé Message-Id: <20200831210740.126168-12-ehabk...@redhat.com> Signed-off-by: Eduardo Habkost --- hw/intc/sifive_plic.h |

[PATCH v2 09/17] hw/block/nvme: refactor aio submission

2020-09-18 Thread Klaus Jensen
From: Klaus Jensen This pulls block layer aio submission/completion to common functions. For completions, additionally map an AIO error to the Unrecovered Read and Write Fault status codes. Signed-off-by: Klaus Jensen --- hw/block/nvme.h | 25 hw/block/nvme.c | 136

[PATCH v2 02/17] pci: pass along the return value of dma_memory_rw

2020-09-18 Thread Klaus Jensen
From: Klaus Jensen Some devices might want to know the return value of dma_memory_rw, so pass it along instead of ignoring it. There are no existing users of the return value, so this patch should be safe. Signed-off-by: Klaus Jensen Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Michael

[PATCH v2 13/17] hw/block/nvme: add support for sgl bit bucket descriptor

2020-09-18 Thread Klaus Jensen
From: Gollu Appalanaidu This adds support for SGL descriptor type 0x1 (bit bucket descriptor). See the NVM Express v1.3d specification, Section 4.4 ("Scatter Gather List (SGL)"). Signed-off-by: Gollu Appalanaidu Signed-off-by: Klaus Jensen --- hw/block/nvme.c | 33

[PATCH v2 07/17] hw/block/nvme: fix endian conversion

2020-09-18 Thread Klaus Jensen
From: Klaus Jensen The raw NLB field is a 16 bit value, so use le16_to_cpu instead of le32_to_cpu and cast to uint32_t before incrementing the value to not wrap around. Signed-off-by: Klaus Jensen --- hw/block/nvme.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH v2 06/17] hw/block/nvme: add a lba to bytes helper

2020-09-18 Thread Klaus Jensen
From: Klaus Jensen Add the nvme_l2b helper and use it for converting NLB and SLBA to byte counts and offsets. Signed-off-by: Klaus Jensen --- hw/block/nvme.h | 6 ++ hw/block/nvme.c | 12 2 files changed, 10 insertions(+), 8 deletions(-) diff --git a/hw/block/nvme.h

[PATCH v2 11/17] hw/block/nvme: harden cmb access

2020-09-18 Thread Klaus Jensen
From: Klaus Jensen Since the controller has only supported PRPs so far it has not been required to check the ending address (addr + len - 1) of the CMB access for validity since it has been guaranteed to be in range of the CMB. This changes when the controller adds support for SGLs (next

[PATCH v2 00/17] hw/block/nvme: multiple namespaces support

2020-09-18 Thread Klaus Jensen
From: Klaus Jensen This is the next round of my patches for the nvme device. This includes a bit of cleanup and two new features: * support for scatter/gather lists * multiple namespaces support through a new nvme-ns device Finally, the series wraps up with changing the PCI vendor and

[PATCH v2 10/17] hw/block/nvme: default request status to success

2020-09-18 Thread Klaus Jensen
From: Klaus Jensen Make the default request status NVME_SUCCESS so only error status codes have to be set. Signed-off-by: Klaus Jensen Reviewed-by: Philippe Mathieu-Daudé --- hw/block/nvme.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/hw/block/nvme.c

[PATCH v2 01/17] hw/block/nvme: fix typo in trace event

2020-09-18 Thread Klaus Jensen
From: Klaus Jensen Fix a typo in the sq doorbell trace event. Signed-off-by: Klaus Jensen --- hw/block/trace-events | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/block/trace-events b/hw/block/trace-events index ec94c56a4165..8ff4cbc4932c 100644 ---

[PATCH v2 05/17] hw/block/nvme: alignment style fixes

2020-09-18 Thread Klaus Jensen
From: Klaus Jensen Style fixes. Signed-off-by: Klaus Jensen Reviewed-by: Philippe Mathieu-Daudé --- hw/block/nvme.c | 25 + 1 file changed, 13 insertions(+), 12 deletions(-) diff --git a/hw/block/nvme.c b/hw/block/nvme.c index 78c3ac80fd4c..10568cbf8761 100644 ---

[PATCH v2 04/17] hw/block/nvme: commonize nvme_rw error handling

2020-09-18 Thread Klaus Jensen
From: Klaus Jensen Move common error handling to a label. Signed-off-by: Klaus Jensen --- hw/block/nvme.c | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/hw/block/nvme.c b/hw/block/nvme.c index 8ffe407ef658..78c3ac80fd4c 100644 --- a/hw/block/nvme.c +++

Re: [PATCH 1/9] scripts/performance: Refactor topN_perf.py

2020-09-18 Thread Aleksandar Markovic
On Friday, August 28, 2020, Ahmed Karaman wrote: > - Apply pylint and flake8 formatting rules to the script. > - Use 'tempfile' instead of '/tmp' for creating temporary files. > > Signed-off-by: Ahmed Karaman > --- Hello, folks. This series seems forgotten. Can some of you perhaps take a

[PULL 2/4] hw/i386/vmport: Drop superfluous parenthesis around function typedef

2020-09-18 Thread Eduardo Habkost
From: Philippe Mathieu-Daudé Drop superfluous parenthesis around VMPortReadFunc typedef (added in d67f679d99, missed to remove when moved in e595112985). Suggested-by: Richard Henderson Signed-off-by: Philippe Mathieu-Daudé Message-Id: <20200505142836.16903-1-phi...@redhat.com> Signed-off-by:

[PULL 3/4] i386: Simplify CPUID_8000_001d for AMD

2020-09-18 Thread Eduardo Habkost
From: Babu Moger Remove all the hardcoded values and replace with generalized fields. Signed-off-by: Babu Moger Reviewed-by: Igor Mammedov Reviewed-by: Pankaj Gupta Acked-by: Michael S. Tsirkin Message-Id: <159897584649.30750.3939159632943292252.st...@naples-babu.amd.com> Signed-off-by:

[PULL 1/4] i386/kvm: correct the meaning of '0xffffffff' value for hv-spinlocks

2020-09-18 Thread Eduardo Habkost
From: Vitaly Kuznetsov Hyper-V TLFS prior to version 6.0 had a mistake in it: special value '0x' for CPUID 0x4004.EBX was called 'never to retry', this looked weird (like why it's not '0' which supposedly have the same effect?) but nobody raised the question. In TLFS version 6.0 the

[PULL 0/4] x86 queue, 2020-09-18

2020-09-18 Thread Eduardo Habkost
The following changes since commit 053a4177817db307ec854356e95b5b350800a216: Merge remote-tracking branch 'remotes/philmd-gitlab/tags/fw_cfg-20200918' into staging (2020-09-18 16:34:26 +0100) are available in the Git repository at: git://github.com/ehabkost/qemu.git tags/x86-next-pull

[PULL 4/4] i386: Simplify CPUID_8000_001E for AMD

2020-09-18 Thread Eduardo Habkost
From: Babu Moger apic_id contains all the information required to build CPUID_8000_001E. core_id and node_id is already part of apic_id generated by x86_topo_ids_from_apicid. Also remove the restriction on number bits on core_id and node_id. Remove all the hardcoded values and replace with

[PULL 1/2] tests/acceptance: Disable tests dependent of unreliable apt.armbian.com

2020-09-18 Thread Philippe Mathieu-Daudé
Armbian servers are not very reliable and confused the GitLab CI users a few times this month (path updated, archives moved, and now the SSL: CERTIFICATE_VERIFY_FAILED "certificate has expired" error). Time to disable these tests. Users can still use the artifacts from the cache (or manually add

Re: [PATCH v2 04/20] block/block-copy: More explicit call_state

2020-09-18 Thread Vladimir Sementsov-Ogievskiy
17.07.2020 16:45, Max Reitz wrote: On 01.06.20 20:11, Vladimir Sementsov-Ogievskiy wrote: Refactor common path to use BlockCopyCallState pointer as parameter, to prepare it for use in asynchronous block-copy (at least, we'll need to run block-copy in a coroutine, passing the whole parameters as

[PULL 0/2] Acceptance patches for 2020-09-18

2020-09-18 Thread Philippe Mathieu-Daudé
The following changes since commit 053a4177817db307ec854356e95b5b350800a216: Merge remote-tracking branch 'remotes/philmd-gitlab/tags/fw_cfg-20200918' i= nto staging (2020-09-18 16:34:26 +0100) are available in the Git repository at: https://gitlab.com/philmd/qemu.git tags/acceptance-next

Re: [PATCH 16/37] qapi: establish mypy type-checking baseline

2020-09-18 Thread John Snow
On 9/18/20 7:55 AM, Markus Armbruster wrote: Ignorant question: why does this come after PATCH 13 "qapi/common.py: add notational type hints", but before all the other patches adding type hints? John Snow writes: Fix two very minor issues, and then establish a mypy type-checking baseline.

Re: [PATCH v3 0/5] Qemu SEV-ES guest support

2020-09-18 Thread Tom Lendacky
On 9/18/20 5:00 AM, Dr. David Alan Gilbert wrote: * Tom Lendacky (thomas.lenda...@amd.com) wrote: On 9/17/20 12:28 PM, Dr. David Alan Gilbert wrote: * Tom Lendacky (thomas.lenda...@amd.com) wrote: From: Tom Lendacky This patch series provides support for launching an SEV-ES guest. Secure

Re: [PATCH v8 00/17] QEMU cpus.c refactoring part2

2020-09-18 Thread Richard Henderson
On 9/16/20 7:19 AM, Claudio Fontana wrote: > Motivation and higher level steps: > > https://lists.gnu.org/archive/html/qemu-devel/2020-05/msg04628.html > > Current state is mostly all reviewed, but I include here a few resolutions > of rebasing conflicts and a new patch to use current_machine

[PATCH v3 80/81] target/arm: Implement SVE2 complex integer dot product

2020-09-18 Thread Richard Henderson
Signed-off-by: Richard Henderson --- target/arm/helper-sve.h| 10 target/arm/sve.decode | 9 target/arm/sve_helper.c| 99 ++ target/arm/translate-sve.c | 17 +++ 4 files changed, 135 insertions(+) diff --git

Re: [Bug 1896096] Re: Git version: Build process is broken in block_curl.c.o

2020-09-18 Thread Paolo Bonzini
You have to unset LDFLAGS, or remove -pie. Specifying --extra-ldflags is useless because Meson already obeys the LDFLAGS environment variable. Il ven 18 set 2020, 19:11 Frederic Bezies <1896...@bugs.launchpad.net> ha scritto: > And qemu log build using make -V=1. > > Hope it helps. > > **

[PATCH v3 79/81] target/arm: Implement SVE2 fp multiply-add long

2020-09-18 Thread Richard Henderson
From: Stephen Long Implements both vectored and indexed FMLALB, FMLALT, FMLSLB, FMLSLT Signed-off-by: Stephen Long Message-Id: <20200504171240.11220-1-stepl...@quicinc.com> [rth: Rearrange to use float16_to_float32_by_bits.] Signed-off-by: Richard Henderson --- target/arm/helper.h|

[PATCH v3 78/81] target/arm: Implement SVE2 bitwise shift immediate

2020-09-18 Thread Richard Henderson
From: Stephen Long Implements SQSHL/UQSHL, SRSHR/URSHR, and SQSHLU Signed-off-by: Stephen Long Message-Id: <20200430194159.24064-1-stepl...@quicinc.com> Signed-off-by: Richard Henderson --- target/arm/helper-sve.h| 33 + target/arm/sve.decode | 5

[PATCH v3 73/81] target/arm: Implement SVE2 FCVTXNT, FCVTX

2020-09-18 Thread Richard Henderson
From: Stephen Long Signed-off-by: Stephen Long Message-Id: <20200428174332.17162-4-stepl...@quicinc.com> [rth: Use do_frint_mode, which avoids a specific runtime helper.] Signed-off-by: Richard Henderson --- target/arm/sve.decode | 2 ++ target/arm/translate-sve.c | 49

  1   2   3   4   5   6   >