[Qemu-devel] [PATCH] spapr: fallback to raw mode if best compat mode cannot be set during CAS

2017-08-16 Thread Greg Kurz
KVM PR doesn't allow to set a compat mode. This causes ppc_set_compat_all() to fail and we return H_HARDWARE to the guest right away. This is excessive: even if we favor compat mode since commit 152ef803ceb19, we should at least fallback to raw mode if the guest supports it. This patch modifies

Re: [Qemu-devel] [Qemu-block] [PATCH v2 1/2] block: use internal filter node in backup

2017-08-16 Thread Stefan Hajnoczi
On Tue, Aug 15, 2017 at 11:18:53AM +0300, Manos Pitsidianakis wrote: > block/backup.c currently uses before write notifiers on the targeted > node. We can create a filter node instead to intercept write requests > for the backup job on the BDS level, instead of the BlockBackend level. > > This is

Re: [Qemu-devel] [PATCH for-2.9?] configure: Remove unused code (found by shellcheck)

2017-08-16 Thread Stefan Weil
It looks like this patch got lost somehow. Stefan See also https://patchwork.codeaurora.org/patch/210129/ Am 28.03.2017 um 20:49 schrieb Stefan Weil: > smartcard_cflags is no longer needed since commit > 0b22ef0f57a8910d849602bef0940edcd0553d2c. > > Signed-off-by: Stefan Weil

Re: [Qemu-devel] [PATCH 06/26] qapi2texi: minor python code simplification

2017-08-16 Thread Markus Armbruster
Marc-André Lureau writes: > Signed-off-by: Marc-André Lureau > --- > scripts/qapi2texi.py | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/scripts/qapi2texi.py b/scripts/qapi2texi.py > index

Re: [Qemu-devel] [PATCH 05/26] visitor: pass size of strings array to enum visitor

2017-08-16 Thread Markus Armbruster
Marc-André Lureau writes: > The size is known at compile time, this avoids having to compute to > check array boundaries. > > Additionally, the following conditional enum entry change will create > "hole" in the generated _lookup tables, that should be skipped. > >

Re: [Qemu-devel] [PATCH] hw/s390x/ipl: The s390-ipl device is not hot-pluggable

2017-08-16 Thread Cornelia Huck
On Wed, 16 Aug 2017 07:30:58 +0200 Thomas Huth wrote: > The s390-ipl device can not be created by the user, since it is meant only > to be instantiated once internally to load the ROMs and kernel. If the user > tries to do a "device_add s390-ipl" via the monitor later, QEMU

Re: [Qemu-devel] [PATCH RFC] block: add block-insert-node QMP command

2017-08-16 Thread Eric Blake
On 08/16/2017 07:11 AM, Manos Pitsidianakis wrote: >> >> Given a qcow2 image backing chain: >> >> base <- active >> >> there are four BDS (2 format, 2 protocol). Ideally, I could add >> filtering to any one of those four nodes (a filter on the base protocol >> level restricts how much guest data

Re: [Qemu-devel] [PATCH RFC] block: add block-insert-node QMP command

2017-08-16 Thread Manos Pitsidianakis
On Wed, Aug 16, 2017 at 06:59:25AM -0500, Eric Blake wrote: On 08/16/2017 04:41 AM, Manos Pitsidianakis wrote: +## +# @block-insert-node: +# +# Insert a filter node between a specific edge in the block driver state graph. +# @parent: the name of the parent node or device +# @node:the name

Re: [Qemu-devel] [PATCH v6 2/3] qmp: introduce query-memory-size-summary command

2017-08-16 Thread Eric Blake
On 08/16/2017 06:06 AM, Vadim Galitsyn wrote: > Command above provides the following memory information in bytes: My general preference for reading a commit message is to treat the subject line as a one-line summary (the what), and then the commit message body as something that can be read

Re: [Qemu-devel] [PATCH v6 1/3] Extend "info numa" with hotplugged memory information

2017-08-16 Thread Eric Blake
On 08/16/2017 06:06 AM, Vadim Galitsyn wrote: > Report amount of hotplugged memory in addition to total > amount per NUMA node. > meta-comment: when sending a multi-patch series, it's best to include a 0/3 cover letter. This, and more tips, are found at

Re: [Qemu-devel] [PULL 7/7] nbd-client: Fix regression when server sends garbage

2017-08-16 Thread Vladimir Sementsov-Ogievskiy
15.08.2017 19:51, Eric Blake wrote: On 08/15/2017 10:50 AM, Vladimir Sementsov-Ogievskiy wrote: 15.08.2017 18:09, Eric Blake wrote: When we switched NBD to use coroutines for qemu 2.9 (in particular, commit a12a712a), we introduced a regression: if a server sends us garbage (such as a

Re: [Qemu-devel] [PATCH RFC] block: add block-insert-node QMP command

2017-08-16 Thread Eric Blake
On 08/16/2017 04:41 AM, Manos Pitsidianakis wrote: >>> +## >>> +# @block-insert-node: >>> +# >>> +# Insert a filter node between a specific edge in the block driver >>> state graph. >>> +# @parent: the name of the parent node or device >>> +# @node:the name of the node to insert under parent

Re: [Qemu-devel] [PATCH RFC 4/9] tests: Add vm test lib

2017-08-16 Thread Eric Blake
On 08/16/2017 03:55 AM, Stefan Hajnoczi wrote: > I'm not aware of a QMP equivalent for "info usernet". It may be > necessary to implement a query-usernet command if you don't want to use > HMP. It's possible to run any HMP command from within QMP, using human-monitor-command. Parsing the

Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator

2017-08-16 Thread Cornelia Huck
On Wed, 16 Aug 2017 12:18:07 +0100 Peter Maydell wrote: > On 16 August 2017 at 11:51, Paolo Bonzini wrote: > > On 16/08/2017 10:26, Cornelia Huck wrote: > >> Prefer to use the tcg accelarator if it is available: This is our only > >> real smoke

[Qemu-devel] [FIX PATCH v0] spapr: Allow configure-connector to be called multiple times for LMBs

2017-08-16 Thread Bharata B Rao
In case of in-kernel memory hot unplug, when the guest is not able to remove all the LMBs that are requested for removal, it will add back any LMBs that have been successfully removed. The DR Connectors of these LMBs wouldn't have been unconfigured and hence the addition of these LMBs will result

Re: [Qemu-devel] [PATCH v2] qemu-iothread: IOThread supports the GMainContext event loop

2017-08-16 Thread Paolo Bonzini
On 16/08/2017 11:52, Stefan Hajnoczi wrote: > > while (!atomic_read(>stopping)) { > if (atomic_read(>use_glib_event_loop)) { > iothread_glib_event_loop(); > break; > } > > aio_poll(iothread->ctx, true); > } Alternatively: while

Re: [Qemu-devel] [PATCH] x86: Skip check apic_id_limit for Xen

2017-08-16 Thread Paolo Bonzini
On 16/08/2017 02:22, Lan Tianyu wrote: > Xen vIOMMU device model will be in Xen hypervisor. Skip vIOMMU > check for Xen here when vcpu number is more than 255. I think you still need to do a check for vIOMMU being enabled. Paolo > Signed-off-by: Lan Tianyu > --- >

Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator

2017-08-16 Thread Peter Maydell
On 16 August 2017 at 11:51, Paolo Bonzini wrote: > On 16/08/2017 10:26, Cornelia Huck wrote: >> Prefer to use the tcg accelarator if it is available: This is our only >> real smoke test for tcg, and fast enough to use it for that. > > I'm not sure this is required for 2.10.

Re: [Qemu-devel] [PATCH v5 2/3] qmp: introduce query-memory-size-summary command

2017-08-16 Thread Vadim Galitsyn
Hi Guys, Thanks again! Just sent patch v6 which takes into account comments you left. Best regards, Vadim On Wed, Aug 16, 2017 at 8:01 AM, Markus Armbruster wrote: > Vadim Galitsyn writes: > > > Hi Guys, > > > > Thank you for the input! >

[Qemu-devel] [PATCH v6 2/3] qmp: introduce query-memory-size-summary command

2017-08-16 Thread Vadim Galitsyn
Command above provides the following memory information in bytes: * base-memory - size of "base" memory specified with command line option -m. * plugged-memory - amount of memory that was hot-plugged. If target does not have CONFIG_MEM_HOTPLUG enabled, no value is reported.

[Qemu-devel] [PATCH v6 1/3] Extend "info numa" with hotplugged memory information

2017-08-16 Thread Vadim Galitsyn
Report amount of hotplugged memory in addition to total amount per NUMA node. Signed-off-by: Vadim Galitsyn Cc: Eduardo Habkost Cc: David Hildenbrand Cc: qemu-devel@nongnu.org --- include/qemu/typedefs.h | 1 +

[Qemu-devel] [PATCH v6 3/3] hmp: introduce 'info memory_size_summary' command

2017-08-16 Thread Vadim Galitsyn
This command is an equivalent of QMP command query-memory-size-summary. It provides the following memory information in bytes: * base-memory - size of "base" memory specified with command line option -m. * plugged-memory - amount of memory that was hot-plugged. If target does not have

Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator

2017-08-16 Thread Paolo Bonzini
On 16/08/2017 10:26, Cornelia Huck wrote: > Prefer to use the tcg accelarator if it is available: This is our only > real smoke test for tcg, and fast enough to use it for that. I'm not sure this is required for 2.10. Yes, it means the coverage from "make check" is worse, but that's it. Paolo

Re: [Qemu-devel] [PATCH v6 0/4] Generic PCIE-PCI Bridge

2017-08-16 Thread Marcel Apfelbaum
On 13/08/2017 18:49, Aleksandr Bezzubikov wrote: This series introduces a new device - Generic PCI Express to PCI bridge, and also makes all necessary changes to enable hotplug of the bridge itself and any device into the bridge. Hi Aleksandr, Thanks for all the effort you put in this series.

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

2017-08-16 Thread Marcel Apfelbaum
On 13/08/2017 19:03, Aleksandr Bezzubikov wrote: Now PCI bridges get a bus range number on a system init, basing on currently plugged devices. That's why when one wants to hotplug another bridge, it needs his child bus, which the parent is unable to provide (speaking about virtual device). The

Re: [Qemu-devel] [PATCH v6 3/3] pci: enable RedHat PCI bridges to reserve additional resource on PCI init

2017-08-16 Thread Marcel Apfelbaum
On 13/08/2017 19:03, Aleksandr Bezzubikov wrote: In case of Red Hat Generic PCIE Root Port reserve additional buses and/or IO/MEM/PREF space, which values are provided in a vendor-specific capability. Signed-off-by: Aleksandr Bezzubikov --- src/fw/pciinit.c | 99

Re: [Qemu-devel] [PATCH] PCI: PCIe access should always be little endian

2017-08-16 Thread Marcel Apfelbaum
On 15/08/2017 17:44, Matt Redfearn wrote: PCIe busses are always little endian, so set the endianness of the memory region to little endian rather than native such that operations work as expected on big endian targets. Signed-off-by: Matt Redfearn Hi Matt, Thanks

Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator

2017-08-16 Thread Thomas Huth
On 16.08.2017 10:26, Cornelia Huck wrote: > Prefer to use the tcg accelarator if it is available: This is our only > real smoke test for tcg, and fast enough to use it for that. > > Fixes: 480bc11e6 ("boot-serial-test: fallback to kvm accelerator") > Reported-by: Richard Henderson

Re: [Qemu-devel] [PATCH 04/26] qapi: generate a literal qobject for introspection

2017-08-16 Thread Markus Armbruster
Marc-André Lureau writes: > Replace the generated json string with a literal qobject. The later is > easier to deal with, at run time, as well as compile time: at run time as well as compile time >#if

Re: [Qemu-devel] [PATCH RFC 0/9] tests: Add VM based build tests (for non-x86_64 and/or non-Linux)

2017-08-16 Thread Kamil Rytarowski
On 16.08.2017 12:13, Fam Zheng wrote: > On Wed, 08/16 11:24, Kamil Rytarowski wrote: >> On 16.08.2017 09:20, Fam Zheng wrote: >>> Build tests in one 32 bit Linux guest and three BSD images are defined in >>> this >>> series. This is a more managable way than the manually maintained virtual >>>

Re: [Qemu-devel] [PATCHv3 00/02] Replace the COLO comparing thread with IOThread

2017-08-16 Thread Fam Zheng
On Wed, 08/16 17:14, Wang yong wrote: > From: Wang Yong > > Note: This series is based on the patch of "qemu-iothread: > IOThread supports theGMainContext event loop". The syntax for patchew to pick up the dependency automatically: Based-on:

Re: [Qemu-devel] [PATCH RFC 0/9] tests: Add VM based build tests (for non-x86_64 and/or non-Linux)

2017-08-16 Thread Fam Zheng
On Wed, 08/16 11:24, Kamil Rytarowski wrote: > On 16.08.2017 09:20, Fam Zheng wrote: > > Build tests in one 32 bit Linux guest and three BSD images are defined in > > this > > series. This is a more managable way than the manually maintained virtual > > machines in patchew. Also, one big

Re: [Qemu-devel] [PATCH RFC 7/9] tests: Add NetBSD image

2017-08-16 Thread Fam Zheng
On Wed, 08/16 11:31, Kamil Rytarowski wrote: > On 16.08.2017 09:21, Fam Zheng wrote: > > The image is prepared following instructions as in: > > > > https://wiki.qemu.org/Hosts/BSD > > > > Signed-off-by: Fam Zheng > > --- > > tests/vm/netbsd | 44

Re: [Qemu-devel] [PATCH v6 2/3] pci: add QEMU-specific PCI capability structure

2017-08-16 Thread Marcel Apfelbaum
On 13/08/2017 19:03, Aleksandr Bezzubikov wrote: On PCI init PCI bridge devices may need some extra info about bus number to reserve, IO, memory and prefetchable memory limits. QEMU can provide this with special vendor-specific PCI capability. This capability is intended to be used only for Red

Re: [Qemu-devel] [PATCH v2] qemu-iothread: IOThread supports the GMainContext event loop

2017-08-16 Thread Stefan Hajnoczi
On Tue, Aug 15, 2017 at 03:28:15PM +0800, Wang yong wrote: > From: Wang Yong > > IOThread uses AioContext event loop and does not run a GMainContext. > Therefore,chardev cannot work in IOThread,such as the chardev is > used for colo-compare packets reception. > > This

Re: [Qemu-devel] [RFC 5/6] migration: store listen task tag

2017-08-16 Thread Peter Xu
On Tue, Aug 15, 2017 at 05:47:08PM +0800, Peter Xu wrote: > On Tue, Aug 15, 2017 at 10:27:07AM +0100, Daniel P. Berrange wrote: > > On Tue, Aug 15, 2017 at 04:50:06PM +0800, Peter Xu wrote: > > > On Tue, Aug 15, 2017 at 09:37:14AM +0100, Daniel P. Berrange wrote: > > > > On Tue, Aug 15, 2017 at

Re: [Qemu-devel] [PATCHv3 00/02] Replace the COLO comparing thread with IOThread

2017-08-16 Thread no-reply
Hi, This 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. Message-id: 1502874842-4191-1-git-send-email-wang.yong...@zte.com.cn Subject: [Qemu-devel] [PATCHv3 00/02] Replace the COLO

Re: [Qemu-devel] [PATCH RFC] block: add block-insert-node QMP command

2017-08-16 Thread Manos Pitsidianakis
On Tue, Aug 15, 2017 at 05:12:42PM -0500, Eric Blake wrote: On 08/15/2017 02:45 AM, Manos Pitsidianakis wrote: block-insert-node and its pair command block-remove-node provide runtime insertion and removal of filter nodes. block-insert-node takes a (parent, child) and (node, child) pair of

Re: [Qemu-devel] [PATCH v2] hw/acpi: Select an node with memory for mapping memory hole to

2017-08-16 Thread Dou Liyang
Hi Igor, At 08/16/2017 05:18 PM, Igor Mammedov wrote: On Wed, 16 Aug 2017 09:26:51 +0800 Dou Liyang wrote: Currently, Using the fisrt node without memory on the machine makes QEMU unhappy. With this example command line: ... \ -m 1024M,slots=4,maxmem=32G \

Re: [Qemu-devel] [PATCH v3 0/5] target-ppc/spapr: Add FWNMI support in QEMU for PowerKVM guests

2017-08-16 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Message-id: 150287457293.9760.17827532208744487789.stgit@aravinda Subject: [Qemu-devel] [PATCH v3 0/5] target-ppc/spapr: Add FWNMI support in QEMU for PowerKVM guests Type: series === TEST SCRIPT

Re: [Qemu-devel] [PATCH RFC 7/9] tests: Add NetBSD image

2017-08-16 Thread Kamil Rytarowski
On 16.08.2017 09:21, Fam Zheng wrote: > The image is prepared following instructions as in: > > https://wiki.qemu.org/Hosts/BSD > > Signed-off-by: Fam Zheng > --- > tests/vm/netbsd | 44 > 1 file changed, 44 insertions(+) > create

Re: [Qemu-devel] [PATCHv3 00/02] Replace the COLO comparing thread with IOThread

2017-08-16 Thread no-reply
Hi, This series failed build test on FreeBSD host. Please find the details below. Type: series Message-id: 1502874842-4191-1-git-send-email-wang.yong...@zte.com.cn Subject: [Qemu-devel] [PATCHv3 00/02] Replace the COLO comparing thread with IOThread === TEST SCRIPT BEGIN === #!/bin/sh #

Re: [Qemu-devel] [PATCHv3 00/02] Replace the COLO comparing thread with IOThread

2017-08-16 Thread no-reply
Hi, This series failed build test on s390x host. Please find the details below. Type: series Subject: [Qemu-devel] [PATCHv3 00/02] Replace the COLO comparing thread with IOThread Message-id: 1502874842-4191-1-git-send-email-wang.yong...@zte.com.cn === TEST SCRIPT BEGIN === #!/bin/bash #

Re: [Qemu-devel] [PATCH RFC 0/9] tests: Add VM based build tests (for non-x86_64 and/or non-Linux)

2017-08-16 Thread Kamil Rytarowski
On 16.08.2017 09:20, Fam Zheng wrote: > Build tests in one 32 bit Linux guest and three BSD images are defined in this > series. This is a more managable way than the manually maintained virtual > machines in patchew. Also, one big advantage of ephemeral VMs over long > running > guests is the

Re: [Qemu-devel] [PATCH RFC 2/9] qemu.py: Add variable vga type

2017-08-16 Thread Kamil Rytarowski
On 16.08.2017 09:20, Fam Zheng wrote: > Some guests behave differently when no VGA is detected. Add a variable > to allow override the "none" default. > > Signed-off-by: Fam Zheng > --- > scripts/qemu.py | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git

Re: [Qemu-devel] Crash when deleting the diag288 watchdog

2017-08-16 Thread Cornelia Huck
On Wed, 16 Aug 2017 07:05:37 +0200 Thomas Huth wrote: > Hi, > > I recently noticed that QEMU abort()s if you try to remove the diag288 > watchdog. For example: > > $ qemu-system-s390x -nographic -nodefaults -S -monitor stdio > QEMU 2.9.92 monitor - type 'help' for more

[Qemu-devel] [PATCHv3 01/02] colo-compare: Use IOThread to Check old packet regularly and Process pactkets of the primary

2017-08-16 Thread Wang yong
From: Wang Yong Remove the task which check old packet in the comparing thread, then use IOthread context timer to handle it. Process pactkets in the IOThread which arrived over the socket. we use iothread_get_g_main_context to create a new g_main_loop in the

[Qemu-devel] [PATCHv3 02/02] colo-compare: Update the COLO document to add the IOThread configuration

2017-08-16 Thread Wang yong
From: Wang Yong Update colo-proxy.txt,add IOThread configuration. Later we have to configure IOThread,if not COLO can not work. Signed-off-by: Wang Yong Signed-off-by: Wang Guang --- docs/colo-proxy.txt | 3 ++- 1 file

Re: [Qemu-devel] [PATCH v2] hw/acpi: Select an node with memory for mapping memory hole to

2017-08-16 Thread Igor Mammedov
On Wed, 16 Aug 2017 09:26:51 +0800 Dou Liyang wrote: > Currently, Using the fisrt node without memory on the machine makes > QEMU unhappy. With this example command line: > ... \ > -m 1024M,slots=4,maxmem=32G \ > -numa node,nodeid=0 \ > -numa

[Qemu-devel] [PATCH v3 5/5] ppc: spapr: Enable FWNMI capability

2017-08-16 Thread Aravinda Prasad
Enable the KVM capability KVM_CAP_PPC_FWNMI so that the KVM causes guest exit with NMI as exit reason when it encounters a machine check exception on the address belonging to a guest. Without this capability enabled, KVM redirects machine check exceptions to guest's 0x200 vector. Signed-off-by:

[Qemu-devel] [PATCHv3 00/02] Replace the COLO comparing thread with IOThread

2017-08-16 Thread Wang yong
From: Wang Yong Note: This series is based on the patch of "qemu-iothread: IOThread supports theGMainContext event loop". It's a good idea to use IOThread instead of COLO comparing thread. comparing thread can be completely replaced by IOThread, so this idea came.

[Qemu-devel] [PATCH v3 2/5] ppc: spapr: Handle "ibm, nmi-register" and "ibm, nmi-interlock" RTAS calls

2017-08-16 Thread Aravinda Prasad
This patch adds support in QEMU to handle "ibm,nmi-register" and "ibm,nmi-interlock" RTAS calls. The machine check notification address is saved when the OS issues "ibm,nmi-register" RTAS call. This patch also handles the case when multiple processors experience machine check at or about the

[Qemu-devel] [PATCH v3 4/5] target/ppc: Handle NMI guest exit

2017-08-16 Thread Aravinda Prasad
Memory error such as bit flips that cannot be corrected by hardware are passed on to the kernel for handling. If the memory address in error belongs to guest then guest kernel is responsible for taking suitable action. Patch [1] enhances KVM to exit guest with exit reason set to KVM_EXIT_NMI in

[Qemu-devel] [PATCH v3 1/5] ppc: spapr: Register and handle HCALL to receive updated RTAS region

2017-08-16 Thread Aravinda Prasad
Receive updates from SLOF about the updated rtas-base. A separate patch for SLOF [1] adds functionality to invoke a private HCALL whenever OS issues instantiate-rtas with a new rtas-base. This is required as QEMU needs to know the updated rtas-base as it allocates error reporting structure in

[Qemu-devel] [PATCH v3 3/5] Wrapper function to wait on condition for the main loop mutex

2017-08-16 Thread Aravinda Prasad
Introduce a wrapper function to wait on condition for the main loop mutex. This function atomically releases the main loop mutex and causes the calling thread to block on the condition. Signed-off-by: Aravinda Prasad --- cpus.c |5 +

[Qemu-devel] [PATCH v3 0/5] target-ppc/spapr: Add FWNMI support in QEMU for PowerKVM guests

2017-08-16 Thread Aravinda Prasad
This series of patches adds support for FWNMI in PowerKVM guests. Memory error such as bit flips that cannot be corrected by hardware is passed on to the kernel for handling by raising machine check exception (an NMI). Upon such machine check exception, if the address in error belongs to guest

Re: [Qemu-devel] [PATCH for-2.10 v2 1/2] hw/acpi: Call acpi_set_pci_info when no ACPI tables needed

2017-08-16 Thread Igor Mammedov
On Tue, 15 Aug 2017 22:24:08 +0300 "Michael S. Tsirkin" wrote: > On Tue, Aug 15, 2017 at 02:07:51PM +0200, Igor Mammedov wrote: > > On Tue, 15 Aug 2017 12:15:48 +0100 > > Anthony PERARD wrote: > > > > > To do PCI passthrough with Xen, the property

Re: [Qemu-devel] [PATCH v2] specs: Describe the TPM support in QEMU

2017-08-16 Thread Laszlo Ersek
On 08/16/17 10:33, Javier Martinez Canillas wrote: > On 07/11/2017 09:36 PM, Laszlo Ersek wrote: >> On 07/11/17 16:31, Stefan Berger wrote: >>> This patch adds a description of the current TPM support in QEMU >>> to the specs. >>> >>> Several public specs are referenced via their landing page on

Re: [Qemu-devel] [PATCH 02/26] qobject: replace dump_qobject() by qobject_to_string()

2017-08-16 Thread Markus Armbruster
Marc-André Lureau writes: > The dump functions is generally useful for any qobject user, for > testing, debugging etc. > > The callback-based output is replaced by string allocation. Trading > efficiency for ease-of-use is okay here. > > Signed-off-by: Marc-André

Re: [Qemu-devel] ask a question

2017-08-16 Thread Stefan Hajnoczi
On Wed, Aug 16, 2017 at 02:25:19PM +0800, 皮智 wrote: > Hi,I meet a problem with vm, my environment is qemu-kvm-2.3, > virtio-win-0.1.126, host(centos7.2), guest(windows10). > > libvirt xml has four virtio-serial channel as follows: CCing the libvirt mailing list. This may be a libvirt issue. >

Re: [Qemu-devel] [PATCH 03/26] qboject: add literal qobject type

2017-08-16 Thread Markus Armbruster
In the subject: s/qboject: add/qobject: Add/ Marc-André Lureau writes: > Promote LiteralQObject from tests/check-qjson.c to qobject/qlit.c, > allowing to statically declare complex qobjects. > > Add a helper qobject_from_qlit() to instantiate a literal qobject to a

Re: [Qemu-devel] [PATCH RFC 0/9] tests: Add VM based build tests (for non-x86_64 and/or non-Linux)

2017-08-16 Thread Stefan Hajnoczi
On Wed, Aug 16, 2017 at 03:20:55PM +0800, Fam Zheng wrote: > Build tests in one 32 bit Linux guest and three BSD images are defined in this > series. This is a more managable way than the manually maintained virtual > machines in patchew. Also, one big advantage of ephemeral VMs over long >

Re: [Qemu-devel] [PATCH RFC 4/9] tests: Add vm test lib

2017-08-16 Thread Stefan Hajnoczi
On Wed, Aug 16, 2017 at 03:20:59PM +0800, Fam Zheng wrote: > +class BaseVM(object): > +GUEST_USER = "qemu" > +GUEST_PASS = "qemupass" > +ROOT_PASS = "qemupass" > + > +# The script to run in the guest that builds QEMU > +BUILD_SCRIPT = "" > +# The guest name, to be

Re: [Qemu-devel] [PATCH v2] specs: Describe the TPM support in QEMU

2017-08-16 Thread Javier Martinez Canillas
On 07/11/2017 09:36 PM, Laszlo Ersek wrote: > On 07/11/17 16:31, Stefan Berger wrote: >> This patch adds a description of the current TPM support in QEMU >> to the specs. >> >> Several public specs are referenced via their landing page on the >> trustedcomputinggroup.org website. >> >>

Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator

2017-08-16 Thread Laurent Vivier
On 16/08/2017 10:26, Cornelia Huck wrote: > Prefer to use the tcg accelarator if it is available: This is our only > real smoke test for tcg, and fast enough to use it for that. > > Fixes: 480bc11e6 ("boot-serial-test: fallback to kvm accelerator") > Reported-by: Richard Henderson

Re: [Qemu-devel] [PATCH RFC 3/9] qemu.py: Add "wait()" method

2017-08-16 Thread Stefan Hajnoczi
On Wed, Aug 16, 2017 at 03:20:58PM +0800, Fam Zheng wrote: > Signed-off-by: Fam Zheng > --- > scripts/qemu.py | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/scripts/qemu.py b/scripts/qemu.py > index e5f314efdb..3e7eb44035 100644 > --- a/scripts/qemu.py > +++

[Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator

2017-08-16 Thread Cornelia Huck
Prefer to use the tcg accelarator if it is available: This is our only real smoke test for tcg, and fast enough to use it for that. Fixes: 480bc11e6 ("boot-serial-test: fallback to kvm accelerator") Reported-by: Richard Henderson Signed-off-by: Cornelia Huck

Re: [Qemu-devel] [PATCH] tests: switch tests to accel=kvm:tcg

2017-08-16 Thread Cornelia Huck
On Tue, 15 Aug 2017 21:49:16 +0200 Thomas Huth wrote: > On 15.08.2017 17:47, Richard Henderson wrote: > > On 08/14/2017 08:33 AM, Cornelia Huck wrote: > >> On Mon, 14 Aug 2017 17:34:15 +0300 > >> "Michael S. Tsirkin" wrote: > >> > >>> Speed up tests on

Re: [Qemu-devel] [PATCH] tests: switch tests to accel=kvm:tcg

2017-08-16 Thread Cornelia Huck
On Tue, 15 Aug 2017 17:04:35 -0300 Philippe Mathieu-Daudé wrote: > On 08/15/2017 12:47 PM, Richard Henderson wrote: > > On 08/14/2017 08:33 AM, Cornelia Huck wrote: > >> On Mon, 14 Aug 2017 17:34:15 +0300 > >> "Michael S. Tsirkin" wrote: > >> > >>> Speed up

Re: [Qemu-devel] [PATCH RFC 0/9] tests: Add VM based build tests (for non-x86_64 and/or non-Linux)

2017-08-16 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Message-id: 20170816072104.24420-1-f...@redhat.com Subject: [Qemu-devel] [PATCH RFC 0/9] tests: Add VM based build tests (for non-x86_64 and/or non-Linux) Type: series === TEST SCRIPT BEGIN ===

[Qemu-devel] [PATCH RFC 0/9] tests: Add VM based build tests (for non-x86_64 and/or non-Linux)

2017-08-16 Thread Fam Zheng
Build tests in one 32 bit Linux guest and three BSD images are defined in this series. This is a more managable way than the manually maintained virtual machines in patchew. Also, one big advantage of ephemeral VMs over long running guests is the reduced RAM usage of host, which makes it possible

[Qemu-devel] [PATCH RFC 1/9] gitignore: Ignore vm test images

2017-08-16 Thread Fam Zheng
Signed-off-by: Fam Zheng --- .gitignore | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.gitignore b/.gitignore index cf65316863..693e2f3009 100644 --- a/.gitignore +++ b/.gitignore @@ -52,6 +52,8 @@ /vscclient /vhost-user-scsi /fsdev/virtfs-proxy-helper

[Qemu-devel] [PATCH RFC 4/9] tests: Add vm test lib

2017-08-16 Thread Fam Zheng
This is the common code to implement a "VM test" to 1) Download and initialize a pre-defined VM that has necessary dependencies to build QEMU and SSH access. 2) Archive $SRC_PATH to a .tar file. 3) Boot the VM, and pass the source tar file to the guest. 4) SSH into the VM, untar the

[Qemu-devel] [PATCH RFC 3/9] qemu.py: Add "wait()" method

2017-08-16 Thread Fam Zheng
Signed-off-by: Fam Zheng --- scripts/qemu.py | 5 + 1 file changed, 5 insertions(+) diff --git a/scripts/qemu.py b/scripts/qemu.py index e5f314efdb..3e7eb44035 100644 --- a/scripts/qemu.py +++ b/scripts/qemu.py @@ -144,6 +144,11 @@ class QEMUMachine(object):

[Qemu-devel] [PATCH RFC 5/9] tests: Add ubuntu.i386 image

2017-08-16 Thread Fam Zheng
This adds a 32bit guest. The official LTS cloud image is downloaded and initialized with cloud-init. Signed-off-by: Fam Zheng --- tests/vm/ubuntu.i386 | 88 1 file changed, 88 insertions(+) create mode 100755

[Qemu-devel] [PATCH RFC 2/9] qemu.py: Add variable vga type

2017-08-16 Thread Fam Zheng
Some guests behave differently when no VGA is detected. Add a variable to allow override the "none" default. Signed-off-by: Fam Zheng --- scripts/qemu.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/scripts/qemu.py b/scripts/qemu.py index

[Qemu-devel] [PATCH RFC 8/9] tests: Add OpenBSD image

2017-08-16 Thread Fam Zheng
The image is prepared following instructions as in: https://wiki.qemu.org/Hosts/BSD Signed-off-by: Fam Zheng --- tests/vm/openbsd | 45 + 1 file changed, 45 insertions(+) create mode 100755 tests/vm/openbsd diff --git

[Qemu-devel] [PATCH RFC 6/9] tests: Add FreeBSD image

2017-08-16 Thread Fam Zheng
The image is prepared following instructions as in: https://wiki.qemu.org/Hosts/BSD Signed-off-by: Fam Zheng --- tests/vm/freebsd | 44 1 file changed, 44 insertions(+) create mode 100755 tests/vm/freebsd diff --git

[Qemu-devel] [PATCH RFC 9/9] Makefile: Add rules to run vm tests

2017-08-16 Thread Fam Zheng
Signed-off-by: Fam Zheng --- Makefile | 2 ++ configure | 2 +- tests/vm/Makefile.include | 41 + 3 files changed, 44 insertions(+), 1 deletion(-) create mode 100644 tests/vm/Makefile.include diff --git

[Qemu-devel] [PATCH RFC 7/9] tests: Add NetBSD image

2017-08-16 Thread Fam Zheng
The image is prepared following instructions as in: https://wiki.qemu.org/Hosts/BSD Signed-off-by: Fam Zheng --- tests/vm/netbsd | 44 1 file changed, 44 insertions(+) create mode 100755 tests/vm/netbsd diff --git

[Qemu-devel] [PATCH] x86: Skip check apic_id_limit for Xen

2017-08-16 Thread Lan Tianyu
Xen vIOMMU device model will be in Xen hypervisor. Skip vIOMMU check for Xen here when vcpu number is more than 255. Signed-off-by: Lan Tianyu --- hw/i386/pc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/i386/pc.c b/hw/i386/pc.c index

Re: [Qemu-devel] [PATCH for-2.11 v2 4/5] qmp-shell: Accept QMP command as argument

2017-08-16 Thread Markus Armbruster
Eduardo Habkost writes: > On Tue, Aug 15, 2017 at 12:03:53PM +0200, Markus Armbruster wrote: >> Eduardo Habkost writes: >> >> Suggest to insert here: >> >> If additional arguments QMP-COMMAND ARG=VAL... are given, run just >> that QMP command

[Qemu-devel] ask a question

2017-08-16 Thread 皮智
Hi,I meet a problem with vm, my environment is qemu-kvm-2.3, virtio-win-0.1.126, host(centos7.2), guest(windows10). libvirt xml has four virtio-serial channel as follows:

Re: [Qemu-devel] [PATCH v6 3/7] qemu.py: use python logging system

2017-08-16 Thread Markus Armbruster
Eduardo Habkost writes: > On Tue, Aug 15, 2017 at 10:10:12AM +0200, Markus Armbruster wrote: >> Please spell "Python" with a capital "P" (it's a proper name). >> >> Amador Pahim writes: >> >> > Let's provide extra control and flexibility by using python

[Qemu-devel] [PATCH v3 2/2] x86: Increase max vcpu number to 8192

2017-08-16 Thread Lan Tianyu
For HPC usage case, it will create a huge VM with vcpus number as same as host cpus and this requires more vcpus support in a single VM. This patch is to increase max vcpu number from 288 to 8192 which is current default maximum cpu number for Linux kernel. Signed-off-by: Lan Tianyu

[Qemu-devel] [PATCH v3 1/2] pc: add 2.11 machine type

2017-08-16 Thread Lan Tianyu
Signed-off-by: Lan Tianyu --- hw/i386/pc_piix.c| 14 +++--- hw/i386/pc_q35.c | 14 +++--- include/hw/compat.h | 3 +++ include/hw/i386/pc.h | 3 +++ 4 files changed, 28 insertions(+), 6 deletions(-) diff --git a/hw/i386/pc_piix.c

Re: [Qemu-devel] [PATCH v5 2/3] qmp: introduce query-memory-size-summary command

2017-08-16 Thread Markus Armbruster
Vadim Galitsyn writes: > Hi Guys, > > Thank you for the input! > >> "hotunpluggable" is ugly. What about just "pluggable"? > > Markus, I think "pluggable" is a bit misleading here. Some people can take > it like a maximum amount of memory that can be hot-added

<    1   2