[Qemu-devel] [PATCH 2/4] tpm: wrap stX_be_p in tpm_cmd_set_XYZ functions

2018-02-02 Thread Stefan Berger
Wrap the calls to stl_be_p and stw_be_p in tpm_cmd_set_XYZ functions that are similar to existing getters. Signed-off-by: Stefan Berger Reviewed-by: Marc-André Lureau --- hw/tpm/tpm_util.c | 6 +++--- hw/tpm/tpm_util.h | 15

Re: [Qemu-devel] [PATCH 3/3] target/ppc: generalize check on radix when in HV mode

2018-02-02 Thread Cédric Le Goater
On 02/02/2018 03:43 AM, Suraj Jitindar Singh wrote: > On Wed, 2018-01-31 at 09:27 +0100, Cédric Le Goater wrote: >> On a POWER9 processor, the first doubleword of the PTCR indicates >> whether the partition uses HPT or Radix Trees translation. Use that >> bit to check for radix mode on powernv

[Qemu-devel] [PATCH v3 06/12] vl: drop no_quit variable

2018-02-02 Thread Gerd Hoffmann
Not used any more, delete it. Signed-off-by: Gerd Hoffmann --- vl.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/vl.c b/vl.c index 25e784be63..c17dedfa4e 100644 --- a/vl.c +++ b/vl.c @@ -152,7 +152,6 @@ int vga_interface_type = VGA_NONE; static

[Qemu-devel] [PATCH v3 07/12] egl-headless: use DisplayOptions

2018-02-02 Thread Gerd Hoffmann
Switch egl-headless ui to use qapi DisplayOptions for configuration. Signed-off-by: Gerd Hoffmann --- include/ui/console.h | 2 +- ui/egl-headless.c| 2 +- vl.c | 3 ++- qapi/ui.json | 5 +++-- 4 files changed, 7 insertions(+), 5 deletions(-) diff

Re: [Qemu-devel] [PATCH v3 24/50] qapi: add some struct member tests

2018-02-02 Thread Markus Armbruster
Marc-André Lureau writes: > Hi > > On Sat, Dec 9, 2017 at 10:07 AM, Markus Armbruster wrote: >> Marc-André Lureau writes: [...] >>> diff --git a/tests/qapi-schema/struct-member-type.json >>>

Re: [Qemu-devel] [PATCH v3 17/50] qapi: do not define enumeration value explicitely

2018-02-02 Thread Markus Armbruster
Marc-André Lureau writes: > Hi > > On Fri, Dec 8, 2017 at 8:50 AM, Markus Armbruster wrote: >> Marc-André Lureau writes: >> >>> On Thu, Dec 7, 2017 at 5:23 PM, Markus Armbruster wrote:

[Qemu-devel] [PATCH v3] linux-user: Fix register used for 6th and 7th syscall argument on aarch64

2018-02-02 Thread Guido Günther
This unbreaks the testcase from http://lists.nongnu.org/archive/html/qemu-arm/2018-01/msg00514.html Thanks to Laurent Vivier for spotting the 7th one. Signed-off-by: Guido Günther Tested-by: Philippe Mathieu-Daudé Suggested-by: Laurent Vivier

[Qemu-devel] [PATCH v3 10/12] vl: drop full_screen variable

2018-02-02 Thread Gerd Hoffmann
Not used any more, delete it. Signed-off-by: Gerd Hoffmann --- vl.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/vl.c b/vl.c index eb8aca9479..899fcad75e 100644 --- a/vl.c +++ b/vl.c @@ -149,7 +149,6 @@ static int rtc_utc = 1; static int rtc_date_offset = -1; /* -1

Re: [Qemu-devel] [RFC] kvm: x86: export vCPU halted state to sysfs

2018-02-02 Thread Eduardo Habkost
(CCing qemu-devel) On Fri, Feb 02, 2018 at 09:21:59AM -0500, Luiz Capitulino wrote: > On Fri, 2 Feb 2018 14:19:38 + > Daniel P. Berrangé wrote: > > On Fri, Feb 02, 2018 at 12:15:54PM -0200, Eduardo Habkost wrote: [...] > > > It would be also interesting to update QEMU

Re: [Qemu-devel] [PATCH 1/2] qcow2: add overlap check for bitmap directory

2018-02-02 Thread Vladimir Sementsov-Ogievskiy
02.02.2018 16:00, Max Reitz wrote: On 2018-02-02 13:07, Vladimir Sementsov-Ogievskiy wrote: 29.01.2018 18:34, Max Reitz wrote: On 2017-11-30 17:47, Vladimir Sementsov-Ogievskiy wrote: Signed-off-by: Vladimir Sementsov-Ogievskiy ---   block/qcow2.h  |  7

Re: [Qemu-devel] [PATCH 1/3] target/ppc: add basic support for PTCR on POWER9

2018-02-02 Thread Cédric Le Goater
On 02/02/2018 03:41 AM, Suraj Jitindar Singh wrote: >>> +/* >>> + * Partition table definitions >>> + */ >>> +#define PTCR_PTAB 0x0000ULL /* Partition >>> Table Base */ >>> +#define PTCR_PTAS 0x001FULL /* Partition >>> Table Size */ >>> + >>

Re: [Qemu-devel] [PATCH] pcie-root-port: let it has higher migrate priority

2018-02-02 Thread Peter Xu
On Thu, Feb 01, 2018 at 07:51:31PM +, Dr. David Alan Gilbert wrote: > * Peter Xu (pet...@redhat.com) wrote: > > In the past, we prioritized IOMMU migration so that we have such a > > priority order: > > > > IOMMU > PCI Devices > > > > When migrating a guest with both vIOMMU and

[Qemu-devel] [PATCH v2 0/2] Add git-publish config file

2018-02-02 Thread Fam Zheng
v2: Add README paragraph [Marc-André, Stefan] Fix 'trivial' profile [Marc-André] Rename profiles [Stefan] Fam Zheng (2): Add a git-publish configuration file README: Document 'git-publish' workflow .gitpublish | 58 ++ README

[Qemu-devel] [PULL 0/2] Audio 20180202 patches

2018-02-02 Thread Gerd Hoffmann
The following changes since commit b05631954d6dfe93340d516660397e2c1a2a5dd6: Merge remote-tracking branch 'remotes/rth/tags/pull-hppa-20180131' into staging (2018-01-31 15:50:29 +) are available in the git repository at: git://git.kraxel.org/qemu tags/audio-20180202-pull-request

Re: [Qemu-devel] [PATCH v6 13/23] hmp: display memory encryption support in 'info kvm'

2018-02-02 Thread Brijesh Singh
On 2/2/18 7:08 AM, Daniel P. Berrangé wrote: > On Thu, Feb 01, 2018 at 08:04:43PM +, Dr. David Alan Gilbert wrote: >> * Brijesh Singh (brijesh.si...@amd.com) wrote: >>> >>> On 2/1/18 11:58 AM, Dr. David Alan Gilbert wrote: * Brijesh Singh (brijesh.si...@amd.com) wrote: > update

Re: [Qemu-devel] [Qemu-ppc] [PATCH] spapr: add missing break in h_get_cpu_characteristics()

2018-02-02 Thread Daniel Henrique Barboza
On 02/01/2018 05:47 PM, Greg Kurz wrote: Detected by Coverity (CID 1385702). This fixes the recently added hypercall to let guests properly apply Spectre and Meltdown workarounds. Paolo Bonzini reported this error in a reply to the pull request that added the patch: "Re: [Qemu-ppc]

[Qemu-devel] [PATCH v3 01/12] vl: deprecate -no-frame

2018-02-02 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann --- vl.c | 4 qemu-doc.texi | 7 +++ 2 files changed, 11 insertions(+) diff --git a/vl.c b/vl.c index e517a8d995..ac0efca708 100644 --- a/vl.c +++ b/vl.c @@ -2104,6 +2104,8 @@ static DisplayType select_display(const char *p)

[Qemu-devel] [PATCH v3 03/12] vl: rename DisplayType to LegacyDisplayType

2018-02-02 Thread Gerd Hoffmann
qapi DisplayType will replace the current enum. For the transition both will coexist though, so rename it so we don't have a name clash. Signed-off-by: Gerd Hoffmann --- vl.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/vl.c b/vl.c index

[Qemu-devel] [PATCH v3 05/12] sdl: use DisplayOptions

2018-02-02 Thread Gerd Hoffmann
Switch sdl ui to use qapi DisplayOptions for configuration. Signed-off-by: Gerd Hoffmann --- include/ui/console.h | 8 ui/sdl.c | 19 +-- ui/sdl2.c| 33 +++-- vl.c | 13 +++--

Re: [Qemu-devel] [PATCH 1/1] nbd: implement bdrv_get_info callback

2018-02-02 Thread Edgar Kaziakhmedov
On 01/26/2018 05:28 PM, Eric Blake wrote: On 01/26/2018 06:39 AM, Edgar Kaziakhmedov wrote: PIng So, let me know if I need to make any changes in patch On 1/18/18 1:09 PM, Paolo Bonzini wrote: On 18/01/2018 12:51, Edgar Kaziakhmedov wrote: +static int nbd_get_info(BlockDriverState *bs,

[Qemu-devel] [PATCH] s390x/sclp: fix event mask handling

2018-02-02 Thread Christian Borntraeger
commit 67915de9f038 ("s390x/event-facility: variable-length event masks") switches the sclp receive/send mask. This broke the sclp lm console. Signed-off-by: Christian Borntraeger Fixes: commit 67915de9f038 ("s390x/event-facility: variable-length event masks") Cc:

Re: [Qemu-devel] [PATCH v2 0/6] qmp dirty bitmap API

2018-02-02 Thread Vladimir Sementsov-Ogievskiy
22.01.2018 20:23, John Snow wrote: On 01/22/2018 07:22 AM, Vladimir Sementsov-Ogievskiy wrote: 22.01.2018 12:20, Vladimir Sementsov-Ogievskiy wrote: 20.01.2018 02:30, John Snow wrote: On 01/16/2018 07:54 AM, Vladimir Sementsov-Ogievskiy wrote: Hi all. There are three qmp commands, needed

[Qemu-devel] [PATCH v3 09/12] cocoa: use DisplayOptions

2018-02-02 Thread Gerd Hoffmann
Switch cocoa ui to use qapi DisplayOptions for configuration. Signed-off-by: Gerd Hoffmann --- include/ui/console.h | 4 ++-- vl.c | 3 ++- qapi/ui.json | 6 -- ui/cocoa.m | 4 ++-- 4 files changed, 10 insertions(+), 7 deletions(-) diff

Re: [Qemu-devel] [PATCH] s390x/sclp: fix event mask handling

2018-02-02 Thread Christian Borntraeger
On 02/02/2018 10:42 AM, Christian Borntraeger wrote: > commit 67915de9f038 ("s390x/event-facility: variable-length event > masks") switches the sclp receive/send mask. This broke the sclp > lm console. > > Signed-off-by: Christian Borntraeger > Fixes: commit

[Qemu-devel] [PATCH v2 2/2] README: Document 'git-publish' workflow

2018-02-02 Thread Fam Zheng
Signed-off-by: Fam Zheng --- .gitpublish | 1 + README | 30 +- 2 files changed, 30 insertions(+), 1 deletion(-) diff --git a/.gitpublish b/.gitpublish index 170bd2ed48..7542e878fc 100644 --- a/.gitpublish +++ b/.gitpublish @@ -26,6 +26,7 @@

Re: [Qemu-devel] [PATCH V9 3/4] pvrdma: initial implementation

2018-02-02 Thread Marcel Apfelbaum
On 02/02/2018 14:08, Dotan Barak wrote: > Reviewed-by: Dotan Barak > The Mellanox review for the RDMA code is very much appreciated! Thanks Dotan, we know you put much effort into it and the V9 re-spin quality just went up :) Marcel >

Re: [Qemu-devel] [PATCH v3 1/1] s390x/cpu: expose the guest crash information

2018-02-02 Thread Eric Blake
On 02/02/2018 08:37 AM, Christian Borntraeger wrote: > This patch is the s390 implementation of guest crash information, > similar to commit d187e08dc4 ("i386/cpu: add crash-information QOM > property") and the related commits. We will detect several crash > reasons, with the "disabled wait" being

[Qemu-devel] [PATCH v3 08/12] curses: use DisplayOptions

2018-02-02 Thread Gerd Hoffmann
Switch curses ui to use qapi DisplayOptions for configuration. Signed-off-by: Gerd Hoffmann --- include/ui/console.h | 4 ++-- ui/curses.c | 2 +- vl.c | 4 +++- qapi/ui.json | 5 +++-- 4 files changed, 9 insertions(+), 6 deletions(-) diff

Re: [Qemu-devel] [PATCH 1/1] nbd: implement bdrv_get_info callback

2018-02-02 Thread Edgar Kaziakhmedov
On 02/02/2018 05:15 PM, Eric Blake wrote: On 02/02/2018 08:06 AM, Edgar Kaziakhmedov wrote: However, it would be nice to remove can_write_zeroes_with_unmap from BlockDriverInfo, and make bdrv_can_write_zeroes_with_unmap just return !!(bs->supported_zero_flags & BDRV_REQ_MAY_UNMAP).  Kevin,

[Qemu-devel] [PATCH v3 02/12] vl: deprecate -alt-grab and -ctrl-grab

2018-02-02 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann --- vl.c | 8 qemu-doc.texi | 6 ++ 2 files changed, 14 insertions(+) diff --git a/vl.c b/vl.c index ac0efca708..fa19a61500 100644 --- a/vl.c +++ b/vl.c @@ -2115,6 +2115,8 @@ static DisplayType select_display(const char *p)

[Qemu-devel] [PATCH 3/4] MAINTAINERS: add pointer to tpm-next repository

2018-02-02 Thread Stefan Berger
Signed-off-by: Stefan Berger --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index f8deaf6..d352d16 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1593,6 +1593,7 @@ F: include/hw/acpi/tpm.h F: include/sysemu/tpm* F:

Re: [Qemu-devel] [PATCH 1/2] qcow2: add overlap check for bitmap directory

2018-02-02 Thread Vladimir Sementsov-Ogievskiy
02.02.2018 16:53, Max Reitz wrote: On 2018-02-02 14:48, Vladimir Sementsov-Ogievskiy wrote: 02.02.2018 16:00, Max Reitz wrote: On 2018-02-02 13:07, Vladimir Sementsov-Ogievskiy wrote: 29.01.2018 18:34, Max Reitz wrote: On 2017-11-30 17:47, Vladimir Sementsov-Ogievskiy wrote: Signed-off-by:

Re: [Qemu-devel] [PATCH 1/3] target/ppc: add basic support for PTCR on POWER9

2018-02-02 Thread Cédric Le Goater
On 02/02/2018 03:34 AM, Suraj Jitindar Singh wrote: > On Wed, 2018-01-31 at 09:27 +0100, Cédric Le Goater wrote: >> The Partition Table Control Register (PTCR) is a hypervisor >> privileged >> SPR. It contains the host real address of the Partition Table and its >> size. >> >> Signed-off-by:

[Qemu-devel] [PATCH 4/4] tpm: tis: move one-line function into caller

2018-02-02 Thread Stefan Berger
Signed-off-by: Stefan Berger --- hw/tpm/tpm_tis.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/hw/tpm/tpm_tis.c b/hw/tpm/tpm_tis.c index 08f41d2..f81168a 100644 --- a/hw/tpm/tpm_tis.c +++ b/hw/tpm/tpm_tis.c @@ -946,11 +946,6 @@ static

Re: [Qemu-devel] [PATCH] pcie-root-port: let it has higher migrate priority

2018-02-02 Thread Marcel Apfelbaum
On 02/02/2018 12:04, Peter Xu wrote: > On Thu, Feb 01, 2018 at 10:01:31PM +0200, Marcel Apfelbaum wrote: > > [...] > >> Root ports can't be nested, anyway, I suppose the migration should >> follow the bus numbering order. > > Could I ask whether this is a must? And if yes, why? > Not sure.

[Qemu-devel] [PATCH 1/4] tpm: Split off tpm_crb_reset function

2018-02-02 Thread Stefan Berger
Split off the tpm_crb_reset function part from tpm_crb_realize that we need to run every time the machine resets. Also register our reset function with the system since TYPE_DEVICE seems to not get a reset otherwise. Signed-off-by: Stefan Berger --- v1->v2:

Re: [Qemu-devel] [RFC PATCH qemu] qmp: Add qom-list-properties to list QOM object properties

2018-02-02 Thread Markus Armbruster
Alexey Kardashevskiy writes: > On 01/02/18 04:22, Markus Armbruster wrote: >> Alexey Kardashevskiy writes: >> >>> There is already 'device-list-properties' which does most of the job, >>> however it does not handle everything returned by qom-list-types such >>>

[Qemu-devel] [PATCH v3 11/12] vl: drop request_opengl variable

2018-02-02 Thread Gerd Hoffmann
Switch over the one leftover user to qapi DisplayType. The delete the unused request_opengl variable. Signed-off-by: Gerd Hoffmann --- vl.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/vl.c b/vl.c index 899fcad75e..4ef774e783 100644 --- a/vl.c

[Qemu-devel] [PATCH v3 00/12] rework display initialization, part one

2018-02-02 Thread Gerd Hoffmann
Hi, This series is the first part of my the qemu display initialization update. Changes: * Create a QAPI DisplayOptions type for display configuration. * Switch all display initialization calls to accept DisplayOptions instead of a bunch of bools. v2: add 'default' in the patch shich

Re: [Qemu-devel] [Qemu-ppc] [PATCH] spapr: add missing break in h_get_cpu_characteristics()

2018-02-02 Thread Greg Kurz
On Fri, 2 Feb 2018 07:11:08 -0200 Daniel Henrique Barboza wrote: > On 02/01/2018 05:47 PM, Greg Kurz wrote: > > Detected by Coverity (CID 1385702). This fixes the recently added hypercall > > to let guests properly apply Spectre and Meltdown workarounds. > > Paolo

[Qemu-devel] [PATCH v3 04/12] gtk: add and use DisplayOptions + DisplayGTK

2018-02-02 Thread Gerd Hoffmann
Add QAPI DisplayType enum, DisplayOptions union and DisplayGTK struct. Switch gtk configuration to use the qapi type. Some bookkeeping (fullscreen for example) is done twice now, this is temporary until more/all UIs are switched over to qapi configuration. Signed-off-by: Gerd Hoffmann

[Qemu-devel] [PATCH v3] iotests: Fix CID for VMDK afl image

2018-02-02 Thread Fam Zheng
This reverts commit 76bf133c4 which updated the reference output, and fixed the reference image, because the code path we want to exercise is actually the invalid image size. The descriptor block in the image, which includes the CID to verify, has been invalid since the reference image was added.

[Qemu-devel] [PATCH v2 1/2] Add a git-publish configuration file

2018-02-02 Thread Fam Zheng
git-publish [1] is a convenient tool to send patches and has been popular among QEMU developers. Recently it has been made available in Fedora official repo thanks to Stefan's work. One nice feature of the tool is a per-project configuration with profiles, especially in which the cccmd option is

Re: [Qemu-devel] [PATCH v3] iotests: Fix CID for VMDK afl image

2018-02-02 Thread Max Reitz
On 2018-02-02 06:23, Fam Zheng wrote: > This reverts commit 76bf133c4 which updated the reference output, and > fixed the reference image, because the code path we want to exercise is > actually the invalid image size. > > The descriptor block in the image, which includes the CID to verify, has

Re: [Qemu-devel] [PATCH 1/2] qcow2: add overlap check for bitmap directory

2018-02-02 Thread Vladimir Sementsov-Ogievskiy
29.01.2018 18:34, Max Reitz wrote: On 2017-11-30 17:47, Vladimir Sementsov-Ogievskiy wrote: Signed-off-by: Vladimir Sementsov-Ogievskiy --- block/qcow2.h | 7 +-- block/qcow2-refcount.c | 12 block/qcow2.c | 6 ++ 3 files

[Qemu-devel] [PATCH 0/4] tpm: A fix and cleanups

2018-02-02 Thread Stefan Berger
The following patches fix the resetting of the CRB interface and wrap calls to st{w,l}_be_p in tpm_cmd_set_XYZ functions. We also clean up a one-liner in the TIS. Stefan Stefan Berger (4): tpm: Split off tpm_crb_reset function tpm: wrap stX_be_p in tpm_cmd_set_XYZ functions MAINTAINERS:

[Qemu-devel] [PATCH v3 12/12] vl: drop display_type variable

2018-02-02 Thread Gerd Hoffmann
Switch over all leftover users to qapi DisplayType. Then delete the unused display_type variable. Add 'default' DisplayType, which isn't an actual display type but a placeholder for "user didn't specify a display". It will be replaced by the DisplayType actually used, which in turn depends on

[Qemu-devel] [PULL 1/2] hw/audio/wm8750: move WM8750 declarations from i2c/i2c.h to audio/wm8750.h

2018-02-02 Thread Gerd Hoffmann
From: Philippe Mathieu-Daudé while here use TYPE_WM8750 and declare a data_req_cb() typedef. Signed-off-by: Philippe Mathieu-Daudé Message-id: 20170919123053.32675-1-f4...@amsat.org Signed-off-by: Gerd Hoffmann ---

Re: [Qemu-devel] [PATCH V9 3/4] pvrdma: initial implementation

2018-02-02 Thread Dotan Barak via Qemu-devel
Reviewed-by: Dotan Barak From: Marcel Apfelbaum To: qemu-devel@nongnu.org Cc: ehabk...@redhat.com; yuval.sh...@oracle.com; mar...@redhat.com; pbonz...@redhat.com; m...@redhat.com; coh...@redhat.com; dotan...@yahoo.com Sent: Thursday, February 1,

Re: [Qemu-devel] [PATCH v6 18/23] sev: emit the SEV_MEASUREMENT event

2018-02-02 Thread Daniel P . Berrangé
On Fri, Feb 02, 2018 at 09:11:41AM -0600, Brijesh Singh wrote: > > > On 02/01/2018 11:27 AM, Dr. David Alan Gilbert wrote: > > * Brijesh Singh (brijesh.si...@amd.com) wrote: > > > > > > > > > On 1/30/18 2:08 PM, Dr. David Alan Gilbert wrote: > > > > * Brijesh Singh (brijesh.si...@amd.com)

Re: [Qemu-devel] [Qemu-ppc] [PATCH] spapr: add missing break in h_get_cpu_characteristics()

2018-02-02 Thread Daniel Henrique Barboza
On 02/02/2018 08:00 AM, Greg Kurz wrote: On Fri, 2 Feb 2018 07:11:08 -0200 Daniel Henrique Barboza wrote: On 02/01/2018 05:47 PM, Greg Kurz wrote: Detected by Coverity (CID 1385702). This fixes the recently added hypercall to let guests properly apply Spectre

Re: [Qemu-devel] [PATCH v2] block: maintain persistent disabled bitmaps

2018-02-02 Thread Vladimir Sementsov-Ogievskiy
31.01.2018 22:04, Max Reitz wrote: On 2018-01-29 19:43, Max Reitz wrote: On 2018-01-22 11:41, Vladimir Sementsov-Ogievskiy wrote: To maintain load/store disabled bitmap there is new approach: - deprecate @autoload flag of block-dirty-bitmap-add, make it ignored - store enabled bitmaps as

Re: [Qemu-devel] [PATCH] s390x/sclp: fix event mask handling

2018-02-02 Thread Cornelia Huck
On Fri, 2 Feb 2018 10:43:18 +0100 Christian Borntraeger wrote: > On 02/02/2018 10:42 AM, Christian Borntraeger wrote: > > commit 67915de9f038 ("s390x/event-facility: variable-length event > > masks") switches the sclp receive/send mask. This broke the sclp > > lm console.

[Qemu-devel] [PATCH RFC 00/21] Modularize generated QAPI code

2018-02-02 Thread Markus Armbruster
Our qapi-schema.json is composed of modules connected by include directives, but the generated code is monolithic all the same: one qapi-types.h with all the types, one qapi-visit.h with all the visitors, and so forth. These monolithic headers get included all over the place. In my "build

[Qemu-devel] [PULL 1/3] ui: fix mixup between qnum and qcode in SDL1 key handling

2018-02-02 Thread Gerd Hoffmann
From: Daniel P. Berrangé The previous commit: commit 2ec78706d188df7d3dab43d07b19b05ef7800a44 Author: Daniel P. Berrange Date: Wed Jan 17 16:47:15 2018 + ui: convert GTK and SDL1 frontends to keycodemapdb changed the x_keymap.c keymap

Re: [Qemu-devel] [RFC] kvm: x86: export vCPU halted state to sysfs

2018-02-02 Thread Eduardo Habkost
On Fri, Feb 02, 2018 at 03:07:18PM +, Daniel P. Berrangé wrote: > On Fri, Feb 02, 2018 at 12:50:14PM -0200, Eduardo Habkost wrote: > > (CCing qemu-devel) > > > > On Fri, Feb 02, 2018 at 09:21:59AM -0500, Luiz Capitulino wrote: > > > On Fri, 2 Feb 2018 14:19:38 + > > > Daniel P. Berrangé

[Qemu-devel] [PATCH RFC 20/21] Include less of qapi-types.h

2018-02-02 Thread Markus Armbruster
In my "build everything" tree, a change to the types in qapi-schema.json triggers a recompile of about 4500 out of 4800 objects. The previous commit split up the generated qapi-types.h. Replace includes of qapi-types.h (i.e. all types) by includes of parts where possible. To illustrate the

[Qemu-devel] [PULL 3/3] ui: correctly advance output buffer when writing SASL data

2018-02-02 Thread Gerd Hoffmann
From: Daniel P. Berrangé In this previous commit: commit 8f61f1c5a6bc06438a1172efa80bc7606594fa07 Author: Daniel P. Berrange Date: Mon Dec 18 19:12:20 2017 + ui: track how much decoded data we consumed when doing SASL encoding I

Re: [Qemu-devel] Qemu Trace

2018-02-02 Thread Peter Maydell
On 2 February 2018 at 10:08, Stefan Hajnoczi wrote: > On Thu, Feb 01, 2018 at 04:30:10PM +0100, Nesrine Zouari wrote: >> I am a computer engineering student and I am actually working on my >> graduation project at Lauterbach company. The project is about Qemu Trace >> and as

[Qemu-devel] [PATCH RFC 05/21] qapi: Turn generators into modules

2018-02-02 Thread Markus Armbruster
The next commit will introduce a common driver program for all generators. The generators need to be modules for that. qapi2texi.py already is. Make the other generators follow suit. The changes are actually trivial. Obvious in the diffs once you view them with whitespace changes ignored.

[Qemu-devel] [PULL v2 04/10] tests: virtio-9p: wait for completion in the test code

2018-02-02 Thread Greg Kurz
In order to test request cancellation, we will need to send multiple requests and wait for the associated replies. Since we poll the ISR to know if a request completed, we may have several replies to parse when we detect ISR was set to 1. This patch moves the waiting out of the reply parsing

[Qemu-devel] [PULL v2 00/10] 9p patches for 2.12 20180202

2018-02-02 Thread Greg Kurz
The following changes since commit b05631954d6dfe93340d516660397e2c1a2a5dd6: Merge remote-tracking branch 'remotes/rth/tags/pull-hppa-20180131' into staging (2018-01-31 15:50:29 +) are available in the git repository at: https://github.com/gkurz/qemu.git tags/for-upstream for you to

[Qemu-devel] [PULL v2 03/10] tests: virtio-9p: move request tag to the test functions

2018-02-02 Thread Greg Kurz
It doesn't really makes sense to hide the request tag from the test functions. It prevents to test the 9p server behavior when passed a wrong tag (ie, still in use or different from P9_NOTAG for a version request). Also the spec says that a tag is reusable as soon as the corresponding request was

Re: [Qemu-devel] [PATCH] pcie-root-port: let it has higher migrate priority

2018-02-02 Thread Peter Xu
On Thu, Feb 01, 2018 at 10:01:31PM +0200, Marcel Apfelbaum wrote: [...] > Root ports can't be nested, anyway, I suppose the migration should > follow the bus numbering order. Could I ask whether this is a must? And if yes, why? > > The question now is what happens if the migration is

Re: [Qemu-devel] [PATCH 3/4] MAINTAINERS: add pointer to tpm-next repository

2018-02-02 Thread Eric Blake
On 02/02/2018 08:44 AM, Stefan Berger wrote: > Signed-off-by: Stefan Berger > --- > MAINTAINERS | 1 + > 1 file changed, 1 insertion(+) Reviewed-by: Eric Blake > > diff --git a/MAINTAINERS b/MAINTAINERS > index f8deaf6..d352d16 100644 > ---

Re: [Qemu-devel] [RFC] kvm: x86: export vCPU halted state to sysfs

2018-02-02 Thread Eric Blake
On 02/02/2018 08:50 AM, Eduardo Habkost wrote: > (CCing qemu-devel) > > On Fri, Feb 02, 2018 at 09:21:59AM -0500, Luiz Capitulino wrote: >> On Fri, 2 Feb 2018 14:19:38 + >> Daniel P. Berrangé wrote: >>> On Fri, Feb 02, 2018 at 12:15:54PM -0200, Eduardo Habkost wrote: >

[Qemu-devel] [PATCH RFC 06/21] qapi-gen: New common driver for code and doc generators

2018-02-02 Thread Markus Armbruster
Whenever qapi-schema.json changes, we run six programs eleven times to update eleven files. This is silly. Replace the six programs by a single program that spits out all eleven files. Signed-off-by: Markus Armbruster --- Makefile |

[Qemu-devel] [PATCH RFC 02/21] qapi: Generate up-to-date copyright notice

2018-02-02 Thread Markus Armbruster
Each generator carries a copyright notice for the generator itself, and another one for the files it generates. Only the former have been updated along the way, the latter have not, and are all out of date. Fix by copying the generator's copyright notice to the generated files instead.

[Qemu-devel] [PATCH RFC 07/21] qapi: Move parse_command_line() next to its only use

2018-02-02 Thread Markus Armbruster
Signed-off-by: Markus Armbruster --- scripts/qapi-gen.py| 52 +++- scripts/qapi/common.py | 54 -- 2 files changed, 51 insertions(+), 55 deletions(-) diff --git

Re: [Qemu-devel] [PATCH RFC 03/21] qapi: New classes QAPIGenC, QAPIGenH, QAPIGenDoc

2018-02-02 Thread Eric Blake
On 02/02/2018 07:03 AM, Markus Armbruster wrote: > These classes encapsulate accumulating and writing output. > > Convert C code generation to QAPIGenC and QAPIGenH. The conversion is > rather shallow: most of the output accumulation is not converted. > Left for later. > > The indentation

Re: [Qemu-devel] [PATCH RFC 02/21] qapi: Generate up-to-date copyright notice

2018-02-02 Thread Eric Blake
On 02/02/2018 07:03 AM, Markus Armbruster wrote: > Each generator carries a copyright notice for the generator itself, > and another one for the files it generates. Only the former have been > updated along the way, the latter have not, and are all out of date. > > Fix by copying the generator's

Re: [Qemu-devel] [PATCH RFC 04/21] qapi: Reduce use of global variables in generators some

2018-02-02 Thread Eric Blake
On 02/02/2018 07:03 AM, Markus Armbruster wrote: > In preparation of the next commit, which will turn the generators into > modules. These global variables will become local to main() then. > > Signed-off-by: Markus Armbruster > --- Reviewed-by: Eric Blake

[Qemu-devel] [PULL v2 01/10] 9pfs: drop v9fs_register_transport()

2018-02-02 Thread Greg Kurz
No good reasons to do this outside of v9fs_device_realize_common(). Signed-off-by: Greg Kurz Reviewed-by: Stefano Stabellini --- hw/9pfs/9p.c | 6 +- hw/9pfs/9p.h | 10 ++ hw/9pfs/virtio-9p-device.c | 8 ++--

Re: [Qemu-devel] [PATCH v5 1/6] machine: Convert the valid cpu types to use cpu_model

2018-02-02 Thread Eduardo Habkost
On Thu, Feb 01, 2018 at 04:42:05PM -0800, Alistair Francis wrote: > As cpu_type is not a user visible string let's convert the > valid_cpu_types to compare against cpu_model instead. This way we have a > user friendly string to report back. > > Once we have a cpu_type to cpu_model conversion this

[Qemu-devel] [PULL v2 05/10] tests: virtio-9p: use the synth backend

2018-02-02 Thread Greg Kurz
The purpose of virtio-9p-test is to test the virtio-9p device, especially the 9p server state machine. We don't really care what fsdev backend we're using. Moreover, if we want to be able to test the flush request or a device reset with in-flights I/O, it is close to impossible to achieve with a

Re: [Qemu-devel] [PATCH v4 0/5] coroutine-lock: polymorphic CoQueue

2018-02-02 Thread Fam Zheng
On Fri, Feb 2, 2018 at 6:01 AM, wrote: > === OUTPUT BEGIN === > Checking PATCH 1/5: test-coroutine: add simple CoMutex test... > ERROR: do not initialise statics to 0 or NULL > #30: FILE: tests/test-coroutine.c:198: > +static bool locked = false; > > total: 1 errors, 0

Re: [Qemu-devel] [PATCH v3 17/39] qcow2: Update l2_allocate() to support L2 slices

2018-02-02 Thread Alberto Garcia
On Thu 01 Feb 2018 07:22:16 PM CET, Max Reitz wrote: > On 2018-02-01 16:43, Alberto Garcia wrote: >> On Thu 01 Feb 2018 04:23:09 PM CET, Anton Nefedov wrote: > However, I'm wondering whether this is the best approach. The old > L2 table is probably not going to be used after this

[Qemu-devel] [PATCH RFC 04/21] qapi: Reduce use of global variables in generators some

2018-02-02 Thread Markus Armbruster
In preparation of the next commit, which will turn the generators into modules. These global variables will become local to main() then. Signed-off-by: Markus Armbruster --- scripts/qapi-commands.py | 9 + scripts/qapi-event.py | 15 +++

Re: [Qemu-devel] [PATCH] s390x/sclp: fix event mask handling

2018-02-02 Thread Claudio Imbrenda
On Fri, 2 Feb 2018 09:42:41 + Christian Borntraeger wrote: > commit 67915de9f038 ("s390x/event-facility: variable-length event > masks") switches the sclp receive/send mask. This broke the sclp > lm console. Reviewed-by: Claudio Imbrenda

[Qemu-devel] [PATCH RFC 19/21] qapi/types: Generate separate .h, .c for each module

2018-02-02 Thread Markus Armbruster
Our qapi-schema.json is composed of modules connected by include directives, but the generated code is monolithic all the same: one qapi-types.h with all the types, one qapi-visit.h with all the visitors, and so forth. These monolithic headers get included all over the place. In my "build

Re: [Qemu-devel] [PATCH v3 05/12] sdl: use DisplayOptions

2018-02-02 Thread Eric Blake
On 02/02/2018 05:10 AM, Gerd Hoffmann wrote: > Switch sdl ui to use qapi DisplayOptions for configuration. > > Signed-off-by: Gerd Hoffmann > --- > include/ui/console.h | 8 > ui/sdl.c | 19 +-- > ui/sdl2.c| 33

Re: [Qemu-devel] [PATCH v3 07/12] egl-headless: use DisplayOptions

2018-02-02 Thread Eric Blake
On 02/02/2018 05:10 AM, Gerd Hoffmann wrote: > Switch egl-headless ui to use qapi DisplayOptions for configuration. > > Signed-off-by: Gerd Hoffmann > --- > include/ui/console.h | 2 +- > ui/egl-headless.c| 2 +- > vl.c | 3 ++- > qapi/ui.json | 5

Re: [Qemu-devel] [PATCH] qemu-img: Fixed grammatical error in dump_human_image_check

2018-02-02 Thread Max Reitz
On 2018-02-02 14:37, Max Reitz wrote: > On 2017-12-02 23:37, Shravan Rajinikanth wrote: >> Signed-off-by: Shravan Rajinikanth >> --- >> qemu-img.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/qemu-img.c b/qemu-img.c >> index 68b375f..bea9268

[Qemu-devel] [PATCH RFC 17/21] qapi/types qapi/visit: Generate built-in stuff into separate files

2018-02-02 Thread Markus Armbruster
Linking code from multiple separate QAPI schemata into the same program is possible, but involves some weirdness around built-in types: * We generate code for built-in types into .c only with option --builtins. The user is responsible to generate code for exactly one QAPI schema per program

[Qemu-devel] [PULL 0/1] Vga 20180202 patches

2018-02-02 Thread Gerd Hoffmann
The following changes since commit b05631954d6dfe93340d516660397e2c1a2a5dd6: Merge remote-tracking branch 'remotes/rth/tags/pull-hppa-20180131' into staging (2018-01-31 15:50:29 +) are available in the git repository at: git://git.kraxel.org/qemu tags/vga-20180202-pull-request for you

Re: [Qemu-devel] Qemu Trace

2018-02-02 Thread Nesrine Zouari
Hello Mr. Stefan, Thank you for your response. To answer your question : the trace solution should be architecture independent.This is the best for us. We aim to test it at least for ARM/ARM64 , x86 and x64 architectures. But even if there will be some differences , we can accept it. Regards,

[Qemu-devel] [PULL v2 06/10] tests: virtio-9p: add LOPEN operation test

2018-02-02 Thread Greg Kurz
Trivial test of a successful open. Signed-off-by: Greg Kurz Reviewed-by: Stefan Hajnoczi --- hw/9pfs/9p-synth.c | 5 + hw/9pfs/9p-synth.h | 1 + tests/virtio-9p-test.c | 47 +++ 3 files changed, 53

Re: [Qemu-devel] [PATCH v3 08/12] curses: use DisplayOptions

2018-02-02 Thread Eric Blake
On 02/02/2018 05:10 AM, Gerd Hoffmann wrote: > Switch curses ui to use qapi DisplayOptions for configuration. > > Signed-off-by: Gerd Hoffmann > --- > include/ui/console.h | 4 ++-- > ui/curses.c | 2 +- > vl.c | 4 +++- > qapi/ui.json | 5

Re: [Qemu-devel] [PATCH v3 12/12] vl: drop display_type variable

2018-02-02 Thread Eric Blake
On 02/02/2018 05:10 AM, Gerd Hoffmann wrote: > Switch over all leftover users to qapi DisplayType. > Then delete the unused display_type variable. > > Add 'default' DisplayType, which isn't an actual display type but > a placeholder for "user didn't specify a display". It will be replaced > by

Re: [Qemu-devel] [PULL 17/51] readline: add a free function

2018-02-02 Thread Greg Kurz
On Thu, 1 Feb 2018 19:10:06 -0500 Paolo Bonzini wrote: > On 01/02/2018 19:00, Alex Williamson wrote: > > On Tue, 16 Jan 2018 15:16:59 +0100 > > Paolo Bonzini wrote: > > > >> From: Marc-André Lureau > >> > >> Fixes leaks

Re: [Qemu-devel] [PATCH 1/2] qcow2: add overlap check for bitmap directory

2018-02-02 Thread Max Reitz
On 2018-02-02 14:48, Vladimir Sementsov-Ogievskiy wrote: > 02.02.2018 16:00, Max Reitz wrote: >> On 2018-02-02 13:07, Vladimir Sementsov-Ogievskiy wrote: >>> 29.01.2018 18:34, Max Reitz wrote: On 2017-11-30 17:47, Vladimir Sementsov-Ogievskiy wrote: > Signed-off-by: Vladimir

[Qemu-devel] [PULL v2 10/10] tests/virtio-9p: explicitly handle potential integer overflows

2018-02-02 Thread Greg Kurz
Signed-off-by: Greg Kurz Reviewed-by: Eric Blake Reviewed-by: Stefan Hajnoczi --- tests/virtio-9p-test.c | 31 +-- 1 file changed, 21 insertions(+), 10 deletions(-) diff --git a/tests/virtio-9p-test.c

[Qemu-devel] [PATCH v3 1/1] s390x/cpu: expose the guest crash information

2018-02-02 Thread Christian Borntraeger
This patch is the s390 implementation of guest crash information, similar to commit d187e08dc4 ("i386/cpu: add crash-information QOM property") and the related commits. We will detect several crash reasons, with the "disabled wait" being the most important one, since this is used by all s390

Re: [Qemu-devel] [PATCH v7 for-2.12 21/25] block: Purify .bdrv_refresh_filename()

2018-02-02 Thread Max Reitz
On 2017-12-04 19:25, Max Reitz wrote: > On 2017-12-04 17:37, Alberto Garcia wrote: >> On Mon 20 Nov 2017 09:10:00 PM CET, Max Reitz wrote: >>> -static void blkdebug_refresh_filename(BlockDriverState *bs, QDict *options) >>> +static void blkdebug_refresh_filename(BlockDriverState *bs) >>> { >>>

Re: [Qemu-devel] [PATCH 4/6] target/arm: Add "-cpu max" support

2018-02-02 Thread Peter Maydell
On 26 January 2018 at 15:44, Philippe Mathieu-Daudé wrote: > On 01/26/2018 11:33 AM, Peter Maydell wrote: >> On 26 January 2018 at 14:29, Philippe Mathieu-Daudé wrote: >>> Why not use arm_any_initfn() here? >> >> That function (and the 'any' cpu) are

Re: [Qemu-devel] [PATCH] qemu-img: Fixed grammatical error in dump_human_image_check

2018-02-02 Thread Max Reitz
On 2017-12-02 23:37, Shravan Rajinikanth wrote: > Signed-off-by: Shravan Rajinikanth > --- > qemu-img.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/qemu-img.c b/qemu-img.c > index 68b375f..bea9268 100644 > --- a/qemu-img.c > +++ b/qemu-img.c >

[Qemu-devel] [PULL 2/2] hw/audio/sb16.c: change dolog() to qemu_log_mask()

2018-02-02 Thread Gerd Hoffmann
From: John Arbuckle Changes all the occurrances of dolog() to qemu_log_mask(). Signed-off-by: John Arbuckle Message-id: 20180201172744.7504-1-programmingk...@gmail.com Signed-off-by: Gerd Hoffmann --- hw/audio/sb16.c |

Re: [Qemu-devel] [RFC 0/2] virtio-vhost-user: add virtio-vhost-user device

2018-02-02 Thread Wei Wang
On 02/02/2018 01:08 AM, Michael S. Tsirkin wrote: On Tue, Jan 30, 2018 at 08:09:19PM +0800, Wei Wang wrote: Issues: Suppose we have both the vhost and virtio-net set up, and vhost pmd <-> virtio-net pmd communication works well. Now, vhost pmd exits (virtio-net pmd is still there). Some time

Re: [Qemu-devel] [PATCH] s390x/sclp: fix event mask handling

2018-02-02 Thread Cornelia Huck
On Fri, 2 Feb 2018 11:33:01 +0100 Cornelia Huck wrote: > On Fri, 2 Feb 2018 10:43:18 +0100 > Christian Borntraeger wrote: > > > On 02/02/2018 10:42 AM, Christian Borntraeger wrote: > > > commit 67915de9f038 ("s390x/event-facility: variable-length

[Qemu-devel] [PATCH RFC 14/21] qapi: Generate in source order

2018-02-02 Thread Markus Armbruster
The generators' conversion to visitors (merge commit 9e72681d16) changed the processing order of entities from source order to alphabetical order. The next commit needs source order, so change it back. Signed-off-by: Markus Armbruster --- scripts/qapi/common.py

  1   2   3   >