[Qemu-devel] [PATCH V3 4/6] migration: add postcopy downtime into MigrationIncommingState

This patch add request to kernel space for UFFD_FEATURE_THREAD_ID, in case when this feature is provided by kernel. DowntimeContext is incapsulated inside migration.c. Signed-off-by: Alexey Perevalov --- include/migration/migration.h | 12

[Qemu-devel] [PATCH V3 2/6] migration: pass ptr to MigrationIncomingState into migration ufd_version_check & postcopy_ram_supported_by_host

That tiny refactoring is necessary to be able to set UFFD_FEATURE_THREAD_ID while requesting features, and then to create downtime context in case when kernel supports it. Signed-off-by: Alexey Perevalov --- include/migration/postcopy-ram.h | 2 +-

[Qemu-devel] [PATCH V3 5/6] migration: calculate downtime on dst side

This patch provides downtime calculation per vCPU, as a summary and as a overlapped value for all vCPUs. This approach was suggested by Peter Xu, as an improvements of previous approch where QEMU kept tree with faulted page address and cpus bitmask in it. Now QEMU is keeping array with faulted

[Qemu-devel] [PATCH V3 6/6] migration: trace postcopy total downtime

It's not possible to transmit it back to source host, due to RP protocol is not expandable. Signed-off-by: Alexey Perevalov --- migration/postcopy-ram.c | 2 ++ migration/trace-events | 1 + 2 files changed, 3 insertions(+) diff --git a/migration/postcopy-ram.c

[Qemu-devel] [PATCH V3 3/6] migration: split ufd_version_check onto receive/request features part

This modification is necessary for userfault fd features which are required to be requested from userspace. UFFD_FEATURE_THREAD_ID is a one of such "on demand" feature, which will be introduced in the next patch. QEMU need to use separate userfault file descriptor, due to userfault context has

[Qemu-devel] [PATCH V3 0/6] calculate downtime for postcopy live migration

This is third version of patch set. First version was tagged as RFC, second was without version tag. Difference since previous version (V3 -> V2) - Downtime calculation approach was changed, thanks to Peter Xu - Due to previous point no more need to keep GTree as well as bitmap of cpus.

[Qemu-devel] [PATCH V3 1/6] userfault: add pid into uffd_msg & update UFFD_FEATURE_*

This commit duplicates header of "userfaultfd: provide pid in userfault msg" into linux kernel. Signed-off-by: Alexey Perevalov --- linux-headers/linux/userfaultfd.h | 5 + 1 file changed, 5 insertions(+) diff --git a/linux-headers/linux/userfaultfd.h

Re: [Qemu-devel] Subject: [PATCH] target-ppc: Add global timer group A to open-pic.

On Fri, Apr 21, 2017 at 02:58:23PM -0700, Aaron Larson wrote: > Add global timer group A to open-pic. This patch is still somewhat > dubious because I'm not sure how to determine what QEMU wants for the > timer frequency. Suggestions solicited. > > Signed-off-by: Aaron Larson

[Qemu-devel] [PATCH] monitor: Check whether TCG is enabled before running the "info jit" code

The "info jit" command currently aborts on Mac OS X with the message "qemu_mutex_lock: Invalid argument" when running with "-M accel=qtest". We should only call into the TCG code here if TCG has really been enabled and initialized. Signed-off-by: Thomas Huth --- monitor.c | 5

[Qemu-devel] [PATCH v15 20/21] file-posix: Add image locking to perm operations

This extends the permission bits of op blocker API to external using Linux OFD locks. Each permission in @perm and @shared_perm is represented by a locked byte in the image file. Requesting a permission in @perm is translated to a shared lock of the corresponding byte; rejecting to share the

[Qemu-devel] [PATCH v15 14/21] iotests: 172: Use separate images for multiple devices

To avoid image lock failures. Signed-off-by: Fam Zheng --- tests/qemu-iotests/172 | 55 +- tests/qemu-iotests/172.out | 50 + 2 files changed, 56 insertions(+), 49 deletions(-) diff --git

[Qemu-devel] [PATCH v15 21/21] qemu-iotests: Add test case 153 for image locking

Signed-off-by: Fam Zheng --- tests/qemu-iotests/153 | 220 + tests/qemu-iotests/153.out | 390 + tests/qemu-iotests/group | 1 + 3 files changed, 611 insertions(+) create mode 100755

[Qemu-devel] [PATCH v15 11/21] iotests: 085: Avoid image locking conflict

In the case where we test the expected error when a blockdev-snapshot target already has a backing image, the backing chain is opened multiple times. This will be a problem when we use image locking, so use a different backing file that is not already open. Signed-off-by: Fam Zheng

[Qemu-devel] [PATCH v15 16/21] file-posix: Add 'locking' option

Making this option available even before implementing it will let converting tests easier: in coming patches they can specify the option already when necessary, before we actually write code to lock the images. Signed-off-by: Fam Zheng --- block/file-posix.c | 5 + 1 file

[Qemu-devel] [PATCH v15 09/21] iotests: 046: Prepare for image locking

The qemu-img info command is executed while VM is running, add -U option to avoid the image locking error. Signed-off-by: Fam Zheng --- tests/qemu-iotests/046 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/qemu-iotests/046 b/tests/qemu-iotests/046

[Qemu-devel] [PATCH v15 19/21] osdep: Add qemu_lock_fd and qemu_unlock_fd

They are wrappers of POSIX fcntl "file private locking", with a convenient "try lock" wrapper implemented with F_OFD_GETLK. Signed-off-by: Fam Zheng Reviewed-by: Max Reitz --- include/qemu/osdep.h | 3 +++ util/osdep.c | 48

[Qemu-devel] [PATCH v15 18/21] block: Reuse bs as backing hd for drive-backup sync=none

Signed-off-by: Fam Zheng --- blockdev.c | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/blockdev.c b/blockdev.c index 4927914..4e04dec 100644 --- a/blockdev.c +++ b/blockdev.c @@ -3174,6 +3174,7 @@ static BlockJob *do_drive_backup(DriveBackup

[Qemu-devel] [PATCH v15 13/21] iotests: 091: Quit QEMU before checking image

Signed-off-by: Fam Zheng Reviewed-by: Max Reitz --- tests/qemu-iotests/091 | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/qemu-iotests/091 b/tests/qemu-iotests/091 index 32bbd56..10ac4a8 100755 --- a/tests/qemu-iotests/091 +++

[Qemu-devel] [PATCH v15 10/21] iotests: 055: Don't attach the target image already for drive-backup

Double attach is not a valid usage of the target image, drive-backup will open the blockdev itself so skip the add_drive call in this case. Signed-off-by: Fam Zheng Reviewed-by: Max Reitz --- tests/qemu-iotests/055 | 32 ++-- 1

[Qemu-devel] [PATCH v15 17/21] tests: Disable image lock in test-replication

The COLO block replication architecture requires one disk to be shared between primary and secondary, in the test both processes use posix file protocol (instead of over NBD) so it is affected by image locking. Disable the lock. Signed-off-by: Fam Zheng ---

[Qemu-devel] [PATCH v15 06/21] qemu-img: Update documentation for -U

Signed-off-by: Fam Zheng --- qemu-img-cmds.hx | 36 ++-- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/qemu-img-cmds.hx b/qemu-img-cmds.hx index 8ac7822..ae309c0 100644 --- a/qemu-img-cmds.hx +++ b/qemu-img-cmds.hx @@ -10,15

[Qemu-devel] [PATCH v15 12/21] iotests: 087: Don't attach test image twice

The test scenario doesn't require the same image, instead it focuses on the duplicated node-name, so use null-co to avoid locking conflict. Reviewed-by: Max Reitz Signed-off-by: Fam Zheng --- tests/qemu-iotests/087 | 6 ++ 1 file changed, 2

[Qemu-devel] [PATCH v15 08/21] iotests: 030: Prepare for image locking

qemu-img and qemu-io commands when guest is running need "-U" option, add it. Signed-off-by: Fam Zheng --- tests/qemu-iotests/030 | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/tests/qemu-iotests/030 b/tests/qemu-iotests/030 index

[Qemu-devel] [PATCH v15 15/21] tests: Use null-co:// instead of /dev/null as the dummy image

Signed-off-by: Fam Zheng Reviewed-by: Max Reitz --- tests/drive_del-test.c| 2 +- tests/nvme-test.c | 2 +- tests/usb-hcd-uhci-test.c | 2 +- tests/usb-hcd-xhci-test.c | 2 +- tests/virtio-blk-test.c | 2 +- tests/virtio-scsi-test.c | 5 +++--

[Qemu-devel] [PATCH v15 07/21] qemu-io: Add --force-share option

Add --force-share/-U to program options and -U to open subcommand. Signed-off-by: Fam Zheng --- qemu-io.c | 42 ++ 1 file changed, 34 insertions(+), 8 deletions(-) diff --git a/qemu-io.c b/qemu-io.c index 427cbae..cf4b876 100644 ---

[Qemu-devel] [PATCH v15 03/21] block: Add, parse and store "force-share" option

Signed-off-by: Fam Zheng --- block.c | 17 + include/block/block.h | 1 + include/block/block_int.h | 1 + qapi/block-core.json | 3 +++ 4 files changed, 22 insertions(+) diff --git a/block.c b/block.c index fce77bf..9db39b6 100644

[Qemu-devel] [PATCH v15 05/21] qemu-img: Add --force-share option to subcommands

This will force the opened images to allow sharing all permissions with other programs. Signed-off-by: Fam Zheng --- qemu-img.c | 154 ++--- 1 file changed, 118 insertions(+), 36 deletions(-) diff --git a/qemu-img.c

[Qemu-devel] [PATCH v15 04/21] block: Respect "force-share" in perm propagating

Signed-off-by: Fam Zheng --- block.c | 32 1 file changed, 24 insertions(+), 8 deletions(-) diff --git a/block.c b/block.c index 9db39b6..e9f4750 100644 --- a/block.c +++ b/block.c @@ -1430,6 +1430,22 @@ static int

[Qemu-devel] [PATCH v15 01/21] block: Make bdrv_perm_names public

It can be used outside of block.c for making user friendly messages. Signed-off-by: Fam Zheng --- block.c | 2 +- include/block/block.h | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/block.c b/block.c index 1e668fb..fce77bf 100644 ---

[Qemu-devel] [PATCH v15 02/21] block: Define BLK_PERM_MAX

This is the order of the largest possible permission. Signed-off-by: Fam Zheng --- include/block/block.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/block/block.h b/include/block/block.h index eb0565d..a798f10 100644 --- a/include/block/block.h +++

[Qemu-devel] [PATCH v15 00/21] block: Image locking series

v15: Rework a number of things, especially around what and how lockings are done. [Kevin] - Map each permission to a locked byte. - Make the new option --force-share-perms, and require read-only=on. - Update test case 153 accordingly. - Only add -U where necessary in

Re: [Qemu-devel] [PULL 00/47] ppc-for-2.10 queue 20170424

On Mon, Apr 24, 2017 at 02:12:19PM +0100, Peter Maydell wrote: > On 24 April 2017 at 02:58, David Gibson wrote: > > The following changes since commit 32c7e0ab755745e961f1772e95cac381cc68769d: > > > > Merge remote-tracking branch > >

Re: [Qemu-devel] Subject: [PATCH] target-ppc: Add global timer group A to open-pic.

On Sun, Apr 23, 2017 at 07:37:55PM -0500, alar...@ddci.com wrote: > David Gibson wrote on 04/23/2017 06:17:22 > PM: > > > From: David Gibson > > To: Aaron Larson > > Cc: ag...@suse.de, qemu-devel@nongnu.org,

Re: [Qemu-devel] [PATCH 2/2] ipmi: don't try to open a NULL filename

On Tue, Apr 25, 2017 at 04:06:17PM +0200, Cédric Le Goater wrote: > On 04/25/2017 03:31 PM, Peter Maydell wrote: > > On 25 April 2017 at 07:51, Cédric Le Goater wrote: > >> Currenlty, the code relies on the fact that open() handles NULL > >> filenames but that can cause an error

Re: [Qemu-devel] [PATCH 2/2] ipmi: don't try to open a NULL filename

On Tue, Apr 25, 2017 at 08:51:41AM +0200, Cédric Le Goater wrote: > Currenlty, the code relies on the fact that open() handles NULL > filenames but that can cause an error with new clang: > > hw/core/loader.c:67:15: runtime error: null pointer passed as argument 1, > which is declared to

[Qemu-devel] [PATCH] block: Remove NULL check in bdrv_co_flush

Reported by Coverity. We already use bs in bdrv_inc_in_flight before checking for NULL. It is unnecessary as all callers pass non-NULL bs, so drop it. Signed-off-by: Fam Zheng --- block/io.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/block/io.c

Re: [Qemu-devel] [PATCH 10/21] xen: import ring.h from xen

On 04/25/2017 03:35 PM, Stefano Stabellini wrote: Do not use the ring.h header installed on the system. Instead, import the header into the QEMU codebase. This avoids problems when QEMU is built against a Xen version too old to provide all the ring macros. Signed-off-by: Stefano Stabellini

Re: [Qemu-devel] [PATCH 25/26] audio: un-export OPLResetChip

On 04/25/2017 07:37 PM, Juan Quintela wrote: Signed-off-by: Juan Quintela Reviewed-by: Philippe Mathieu-Daudé --- hw/audio/fmopl.c | 2 +- hw/audio/fmopl.h | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/hw/audio/fmopl.c

Re: [Qemu-devel] [PATCH 26/26] audio: Use ARRAY_SIZE from qemu/osdep.h

On 04/25/2017 07:37 PM, Juan Quintela wrote: Signed-off-by: Juan Quintela Reviewed-by: Philippe Mathieu-Daudé --- hw/audio/fmopl.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/hw/audio/fmopl.c b/hw/audio/fmopl.c index

Re: [Qemu-devel] [PATCH 19/26] audio: GUSsample is int16_t

Hi Juan, Same here, why not squashing as "Use stdint instead of dead GUSEMU32"? On 04/25/2017 07:37 PM, Juan Quintela wrote: Signed-off-by: Juan Quintela --- hw/audio/gus.c | 2 +- hw/audio/gusemu.h | 12 +--- hw/audio/gusemu_hal.c | 2 +-

Re: [Qemu-devel] [PATCH 10/26] audio: Remove INT32

Hi Juan, is there a benefit in not squashing the previous stdint commits? On 04/25/2017 07:37 PM, Juan Quintela wrote: Signed-off-by: Juan Quintela --- hw/audio/fmopl.c | 42 +- hw/audio/fmopl.h | 54

Re: [Qemu-devel] [PATCH 05/26] audio: Remove UINT8

Hi Juan, On 04/25/2017 07:37 PM, Juan Quintela wrote: uint8_t has existed since . all this century? Signed-off-by: Juan Quintela --- hw/audio/fmopl.c | 8 hw/audio/fmopl.h | 39 --- 2 files changed, 24 insertions(+), 23

Re: [Qemu-devel] [PATCH v4] qga: Add support network interfacestatistics in

＞ + */＞ +colon = strchr(line, ':')＞ +if (!colon) {＞ + continue＞ +}＞ +*colon = '\0'＞ +if (colon - path_len ＞= line && strcmp(colon - path_len, path) == 0) {colon - path_len ＞ line would imply the current line isn't a matcheither, so i think

Re: [Qemu-devel] [PATCH for 2.9 v3 10/10] block: Fix bdrv_co_flush early return

On Tue, 04/25 17:16, Kevin Wolf wrote: > Am 10.04.2017 um 17:05 hat Fam Zheng geschrieben: > > bdrv_inc_in_flight and bdrv_dec_in_flight are mandatory for > > BDRV_POLL_WHILE to work, even for the shortcut case where flush is > > unnecessary. Move the if block to below bdrv_dec_in_flight, and BTW

Re: [Qemu-devel] [PULL 10/11] block: Fix bdrv_co_flush early return

On Tue, 04/25 16:00, Peter Maydell wrote: > > Coverity points out that there's a problem here -- we call > bdrv_inc_in_flight(bs), which assumes bs is not NULL, before > we do the test for whether bs is NULL. > > Presumably the NULL check needs to be pulled up earlier in > the function? > Yes,

Re: [Qemu-devel] [PATCH v3 3/3] vfio-pci: process non fatal error of AER

On Tue, 25 Apr 2017 23:32:52 +0300 "Michael S. Tsirkin" wrote: > On Tue, Mar 28, 2017 at 08:55:04PM -0600, Alex Williamson wrote: > > On Wed, 29 Mar 2017 02:59:34 +0300 > > "Michael S. Tsirkin" wrote: > > > > > On Tue, Mar 28, 2017 at 10:12:25AM -0600, Alex

Re: [Qemu-devel] [PATCH 00/26] Audio Cleanup

Hi, This series seems to have some coding style problems. See output below for more information: Message-id: 20170425223739.6703-1-quint...@redhat.com Subject: [Qemu-devel] [PATCH 00/26] Audio Cleanup Type: series === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n=1 total=$(git log --oneline

[Qemu-devel] [PULL 7/8] qga: Add 'guest-get-users' command

From: Vinzenz Feenstra A command that will list all currently logged in users, and the time since when they are logged in. Examples: virsh # qemu-agent-command F25 '{ "execute": "guest-get-users" }' {"return":[{"login-time":1490622289.903835,"user":"root"}]} virsh #

[Qemu-devel] [PULL 5/8] qga: Add 'guest-get-host-name' command

From: Vinzenz Feenstra Retrieving the guest host name is a very useful feature for virtual management systems. This information can help to have more user friendly VM access details, instead of an IP there would be the host name. Also the host name reported can be used to

[Qemu-devel] [PULL 3/8] qga-win: Fix a bug where qemu-ga service is stuck during stop operation

From: Sameeh Jubran After triggering a freeze command without any following thaw command, qemu-ga will not respond to stop operation. This behaviour is wanted on Linux as there is no time limit for a freeze command and we want to prevent quitting in the middle of freeze, on

[Qemu-devel] [PULL 0/8] qemu-ga patch queue

The following changes since commit fe491fa85c4634453b340b18046aae2eaf8147db: Merge remote-tracking branch 'remotes/agraf/tags/signed-s390-for-upstream' into staging (2017-04-25 14:48:55 +0100) are available in the git repository at: git://github.com/mdroth/qemu.git

[Qemu-devel] [PULL 6/8] qga: improve fsfreeze documentations

From: Marc-André Lureau Some users find the fsfreeze behaviour confusing. Add some notes about invalid mount points and Windows usage. Related to: https://bugzilla.redhat.com/show_bug.cgi?id=1436976 Signed-off-by: Marc-André Lureau

[Qemu-devel] [PULL 1/8] qemu-ga: Make QGA VSS provider service run only when needed

From: Sameeh Jubran Currently the service runs in background on boot even though it is not needed and once it is running it never stops. The service needs to be running only during freeze operation and it should be stopped after executing thaw. Signed-off-by: Sameeh Jubran

[Qemu-devel] [PULL 4/8] qga-win: Fix Event Viewer errors caused by qemu-ga

From: Sameeh Jubran When the command "guest-fsfreeze-freeze" is executed it causes the VSS service to log the error below in the Event Viewer. This error is caused by an issue in the function "CommitSnapshots" in provider.cpp: * When VSS_TIMEOUT_MSEC expires the funtion

[Qemu-devel] [PULL 2/8] qga-win: Enable 'can-offline' field in 'guest-get-vcpus' reply

From: Sameeh Jubran The QGA schema states: @can-offline: Whether offlining the VCPU is possible. This member is always filled in by the guest agent when the structure is returned, and always ignored on input (hence it can be

[Qemu-devel] [PULL 8/8] qga: Add `guest-get-timezone` command

From: Vinzenz Feenstra Adds a new command `guest-get-timezone` reporting the currently configured timezone on the system. The information on what timezone is currently is configured is useful in case of Windows VMs where the offset of the hardware clock is required to have

Re: [Qemu-devel] [PATCH 3/4] migration: spapr: migrate ccs_list in spapr state

Quoting Daniel Henrique Barboza (2017-04-24 17:08:27) > From: Jianjun Duan > > ccs_list in spapr state maintains the device tree related > information on the rtas side for hotplugged devices. In racing > situations between hotplug events and migration operation, a rtas

[Qemu-devel] [PATCH 25/26] audio: un-export OPLResetChip

Signed-off-by: Juan Quintela --- hw/audio/fmopl.c | 2 +- hw/audio/fmopl.h | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/hw/audio/fmopl.c b/hw/audio/fmopl.c index 99d09c5..dc9043c 100644 --- a/hw/audio/fmopl.c +++ b/hw/audio/fmopl.c @@ -1036,7 +1036,7

Re: [Qemu-devel] [PATCH] RFC: vmcoreinfo device

On Tue, Apr 25, 2017 at 11:35:23PM +0300, Michael S. Tsirkin wrote: > On Tue, Apr 25, 2017 at 05:29:20PM -0300, Eduardo Habkost wrote: > > On Mon, Apr 24, 2017 at 05:03:55PM +0400, Marc-André Lureau wrote: > > [...] > > > diff --git a/include/hw/compat.h b/include/hw/compat.h > > > index

[Qemu-devel] [PATCH 24/26] audio: Remove unused typedefs

Signed-off-by: Juan Quintela --- hw/audio/fmopl.h | 4 1 file changed, 4 deletions(-) diff --git a/hw/audio/fmopl.h b/hw/audio/fmopl.h index 8410275..f89af08 100644 --- a/hw/audio/fmopl.h +++ b/hw/audio/fmopl.h @@ -4,10 +4,6 @@ #include typedef void

Re: [Qemu-devel] [PATCH 4/4] migration: spapr: migrate pending_events of spapr state

Quoting Daniel Henrique Barboza (2017-04-24 17:08:28) > From: Jianjun Duan > > In racing situations between hotplug events and migration operation, > a rtas hotplug event could have not yet be delivered to the source > guest when migration is started. In this case the

[Qemu-devel] [PATCH 21/26] audio: OPLSetUpdateHandler is not used anywhere

Signed-off-by: Juan Quintela --- hw/audio/fmopl.c | 6 +- hw/audio/fmopl.h | 1 - 2 files changed, 1 insertion(+), 6 deletions(-) diff --git a/hw/audio/fmopl.c b/hw/audio/fmopl.c index f91e700..694a77b 100644 --- a/hw/audio/fmopl.c +++ b/hw/audio/fmopl.c @@ -1141,11

[Qemu-devel] [PATCH 15/26] audio: GUSbyte is uint8_t

Signed-off-by: Juan Quintela --- hw/audio/gusemu.h | 6 ++--- hw/audio/gusemu_hal.c | 58 - hw/audio/gusemu_mixer.c | 4 ++-- 3 files changed, 33 insertions(+), 35 deletions(-) diff --git a/hw/audio/gusemu.h

[Qemu-devel] [PATCH 16/26] audio: remove GUSchar

Signed-off-by: Juan Quintela --- hw/audio/gusemu.h | 2 -- hw/audio/gusemu_mixer.c | 8 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/hw/audio/gusemu.h b/hw/audio/gusemu.h index 3a69222..1c1a63c 100644 --- a/hw/audio/gusemu.h +++

[Qemu-devel] [PATCH 14/26] audio: Remove unused fields

These were used for the remove stuff. Signed-off-by: Juan Quintela --- hw/audio/fmopl.h | 6 -- 1 file changed, 6 deletions(-) diff --git a/hw/audio/fmopl.h b/hw/audio/fmopl.h index df790a0..8730ead 100644 --- a/hw/audio/fmopl.h +++ b/hw/audio/fmopl.h @@ -76,12 +76,6

Re: [Qemu-devel] [PATCH 2/4] hw/ppc: migrating the DRC state of hotplugged devices

Quoting Daniel Henrique Barboza (2017-04-24 17:08:26) > In pseries, a firmware abstraction called Dynamic Reconfiguration > Connector (DRC) is used to assign a particular dynamic resource > to the guest and provide an interface to manage configuration/removal > of the resource associated with it.

[Qemu-devel] [PATCH 26/26] audio: Use ARRAY_SIZE from qemu/osdep.h

Signed-off-by: Juan Quintela --- hw/audio/fmopl.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/hw/audio/fmopl.c b/hw/audio/fmopl.c index dc9043c..202f752 100644 --- a/hw/audio/fmopl.c +++ b/hw/audio/fmopl.c @@ -34,15 +34,11 @@ #include

[Qemu-devel] [PATCH 12/26] audio: Remove Unused OPL_TYPE_*

Since we removed the previous unused devices, they are not used anymore. Signed-off-by: Juan Quintela --- hw/audio/fmopl.c | 16 hw/audio/fmopl.h | 3 --- 2 files changed, 19 deletions(-) diff --git a/hw/audio/fmopl.c b/hw/audio/fmopl.c index

[Qemu-devel] [PATCH 23/26] audio: UpdateHandler is not used anymore

Signed-off-by: Juan Quintela --- hw/audio/fmopl.c | 2 -- hw/audio/fmopl.h | 2 -- 2 files changed, 4 deletions(-) diff --git a/hw/audio/fmopl.c b/hw/audio/fmopl.c index 5b8a884..99d09c5 100644 --- a/hw/audio/fmopl.c +++ b/hw/audio/fmopl.c @@ -1147,7 +1147,6 @@ int

[Qemu-devel] [PATCH 22/26] audio: IRQHandler is not used anymore

Signed-off-by: Juan Quintela --- hw/audio/fmopl.c | 4 hw/audio/fmopl.h | 2 -- 2 files changed, 6 deletions(-) diff --git a/hw/audio/fmopl.c b/hw/audio/fmopl.c index 694a77b..5b8a884 100644 --- a/hw/audio/fmopl.c +++ b/hw/audio/fmopl.c @@ -260,8 +260,6 @@ static

[Qemu-devel] [PATCH 11/26] audio: Unfold OPLSAMPLE

It was used only once, and now it was always int16_t. Signed-off-by: Juan Quintela --- hw/audio/fmopl.c | 2 +- hw/audio/fmopl.h | 11 --- 2 files changed, 1 insertion(+), 12 deletions(-) diff --git a/hw/audio/fmopl.c b/hw/audio/fmopl.c index 8f935f6..b1cb4b4

[Qemu-devel] [PATCH 18/26] audio: GUSword is uint16_t

Signed-off-by: Juan Quintela --- hw/audio/gusemu_hal.c | 14 +++--- hw/audio/gusemu_mixer.c | 8 2 files changed, 11 insertions(+), 11 deletions(-) diff --git a/hw/audio/gusemu_hal.c b/hw/audio/gusemu_hal.c index 444a2bb..3dd7239 100644 ---

[Qemu-devel] [PATCH 10/26] audio: Remove INT32

Signed-off-by: Juan Quintela --- hw/audio/fmopl.c | 42 +- hw/audio/fmopl.h | 54 -- 2 files changed, 45 insertions(+), 51 deletions(-) diff --git a/hw/audio/fmopl.c

[Qemu-devel] [PATCH 20/26] audio: OPLSetIRQHandler is not used anywhere

Signed-off-by: Juan Quintela --- hw/audio/fmopl.c | 5 - hw/audio/fmopl.h | 1 - 2 files changed, 6 deletions(-) diff --git a/hw/audio/fmopl.c b/hw/audio/fmopl.c index 48db828..f91e700 100644 --- a/hw/audio/fmopl.c +++ b/hw/audio/fmopl.c @@ -1141,11 +1141,6 @@ void

[Qemu-devel] [PATCH 09/26] audio: remove INT16

Signed-off-by: Juan Quintela --- hw/audio/fmopl.c | 2 +- hw/audio/fmopl.h | 5 ++--- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/hw/audio/fmopl.c b/hw/audio/fmopl.c index 47754e8..ebd3dbb 100644 --- a/hw/audio/fmopl.c +++ b/hw/audio/fmopl.c @@ -984,7

[Qemu-devel] [PATCH 06/26] audio: remove UINT16

More modernitation. Signed-off-by: Juan Quintela --- hw/audio/fmopl.h | 1 - 1 file changed, 1 deletion(-) diff --git a/hw/audio/fmopl.h b/hw/audio/fmopl.h index 3df8942..1891a22 100644 --- a/hw/audio/fmopl.h +++ b/hw/audio/fmopl.h @@ -10,7 +10,6 @@ /* compiler dependence

[Qemu-devel] [PATCH 19/26] audio: GUSsample is int16_t

Signed-off-by: Juan Quintela --- hw/audio/gus.c | 2 +- hw/audio/gusemu.h | 12 +--- hw/audio/gusemu_hal.c | 2 +- hw/audio/gusemu_mixer.c | 8 4 files changed, 7 insertions(+), 17 deletions(-) diff --git a/hw/audio/gus.c

[Qemu-devel] [PATCH 08/26] audio: Remove INT8

Signed-off-by: Juan Quintela --- hw/audio/fmopl.h | 1 - 1 file changed, 1 deletion(-) diff --git a/hw/audio/fmopl.h b/hw/audio/fmopl.h index b52f039..bede671 100644 --- a/hw/audio/fmopl.h +++ b/hw/audio/fmopl.h @@ -10,7 +10,6 @@ /* compiler dependence */ #ifndef

[Qemu-devel] [PATCH 17/26] audio: GUSword is uint16_t

Signed-off-by: Juan Quintela --- hw/audio/gusemu.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/hw/audio/gusemu.h b/hw/audio/gusemu.h index 1c1a63c..69dadef 100644 --- a/hw/audio/gusemu.h +++ b/hw/audio/gusemu.h @@ -28,11 +28,9 @@ /* data types (need to be adjusted

[Qemu-devel] [PATCH 05/26] audio: Remove UINT8

uint8_t has existed since . all this century? Signed-off-by: Juan Quintela --- hw/audio/fmopl.c | 8 hw/audio/fmopl.h | 39 --- 2 files changed, 24 insertions(+), 23 deletions(-) diff --git a/hw/audio/fmopl.c

[Qemu-devel] [PATCH 04/26] audio: YM3812 was always defined

So, remove the ifdefs. Signed-off-by: Juan Quintela --- hw/audio/fmopl.c | 4 hw/audio/fmopl.h | 4 2 files changed, 8 deletions(-) diff --git a/hw/audio/fmopl.c b/hw/audio/fmopl.c index 1e05efc..282662a 100644 --- a/hw/audio/fmopl.c +++ b/hw/audio/fmopl.c @@

[Qemu-devel] [PATCH 07/26] audio: remove UINT32

Signed-off-by: Juan Quintela --- hw/audio/fmopl.c | 14 +++--- hw/audio/fmopl.h | 17 - 2 files changed, 15 insertions(+), 16 deletions(-) diff --git a/hw/audio/fmopl.c b/hw/audio/fmopl.c index 3d14b45..47754e8 100644 --- a/hw/audio/fmopl.c +++

[Qemu-devel] [PATCH 13/26] audio: Remove type field

It was not used anymore as now there is only one type of devices. Signed-off-by: Juan Quintela --- hw/audio/adlib.c | 2 +- hw/audio/fmopl.c | 20 hw/audio/fmopl.h | 7 +-- 3 files changed, 10 insertions(+), 19 deletions(-) diff --git

[Qemu-devel] [PATCH 01/26] adlib: Remove support for YMF262

Notice that the code was supposed to be in the file ymf262.h, that has never been on qemu source tree. Signed-off-by: Juan Quintela --- hw/audio/adlib.c | 45 + 1 file changed, 1 insertion(+), 44 deletions(-) diff --git

[Qemu-devel] [PATCH 02/26] audio: remove Y8950 configuration

Include file has never been on qemu and it has been undefined from the very beginning. Signed-off-by: Juan Quintela --- hw/audio/Makefile.objs | 2 - hw/audio/fmopl.c | 146 - hw/audio/fmopl.h | 20 --- 3

[Qemu-devel] [PATCH 00/26] Audio Cleanup

Hi This are an old series that were hidden on my harddisk. To give you one idea, I had a patch to remove this: commit 8307c294a355bbf3c5352e00877365b0cda66d52 Author: Nutan Shinde Date: Wed Oct 7 22:02:54 2015 +0530 Remove macros IO_READ_PROTO and

[Qemu-devel] [PATCH 03/26] audio: Remove YM3526 support

It was never compiled in. Signed-off-by: Juan Quintela --- hw/audio/fmopl.c | 4 ++-- hw/audio/fmopl.h | 2 -- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/hw/audio/fmopl.c b/hw/audio/fmopl.c index 29f6d3c..1e05efc 100644 --- a/hw/audio/fmopl.c +++

[Qemu-devel] [PATCH v5] crypto: qcrypto_random_bytes() now works on windows w/o any other crypto libs

If no crypto library is included in the build, QEMU uses qcrypto_random_bytes() to generate random data. That function tried to open /dev/urandom or /dev/random and if opening both files failed it errored out. Those files obviously do not exist on windows, so there the code uses CryptGenRandom().

Re: [Qemu-devel] [PATCH 1/4] migration: alternative way to set instance_id in SaveStateEntry

Quoting Daniel Henrique Barboza (2017-04-24 17:08:25) > From: Jianjun Duan > > In QOM (QEMU Object Model) migrated objects are identified with instance_id > which is calculated automatically using their path in the QOM composition > tree. For some objects, this path

[Qemu-devel] [PATCH 37/41] migration: Move last funtions to misc.h

After this, nothing outside of migration uses migration.h Signed-off-by: Juan Quintela --- include/migration/migration.h | 7 --- include/migration/misc.h | 8 +++- ui/spice-core.c | 2 +- 3 files changed, 8 insertions(+), 9 deletions(-) diff

Re: [Qemu-devel] [PATCH v4] crypto: qcrypto_random_bytes() now works on windows w/o any other crypto libs

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: 1493157212-22070-1-git-send-email-gm.ijew...@web.de Subject: [Qemu-devel] [PATCH v4] crypto:

[Qemu-devel] [PATCH 36/41] migration: Move more exported functions to migration/misc.h

Signed-off-by: Juan Quintela --- hw/i386/pc_piix.c | 2 +- hw/ppc/spapr.c| 2 +- include/migration/migration.h | 6 -- include/migration/misc.h | 6 ++ migration/migration.c | 1 + qdev-monitor.c| 2 +- vl.c

[Qemu-devel] [PATCH 41/41] migration: Remove unneeded includes

Signed-off-by: Juan Quintela --- include/migration/misc.h | 2 ++ migration/block.c | 6 -- migration/colo-failover.c | 2 ++ migration/colo.c | 2 -- migration/colo.h | 3 --- migration/exec.c | 2 -- migration/fd.c| 2 --

[Qemu-devel] [PATCH 40/41] migration: Make savevm.c target independent

It only needed TARGET_PAGE_SIZE/BITS/BITS_MIN values, so just export them from exec.h Signed-off-by: Juan Quintela --- Makefile.target| 2 +- exec.c | 9 + include/exec/target_page.h | 2 ++ migration/Makefile.objs| 2 +-

[Qemu-devel] [PATCH 35/41] migration: create global_state.c

It don't belong anywhere else, just the global state where everybody can stick other things. Signed-off-by: Juan Quintela --- hw/i386/pc_piix.c| 1 + hw/ppc/spapr.c | 1 + include/migration/global_state.h | 26 +++

[Qemu-devel] [PATCH 38/41] migration: Move migration.h to migration/

Nothing uses it outside of migration.h Signed-off-by: Juan Quintela --- migration/block.c| 2 +- migration/channel.c | 2 +- migration/colo-comm.c| 2 +- migration/colo.c

[Qemu-devel] [PATCH 26/41] migration: Move include/migration/block.h into migration/

All functions were internal, except blk_mig_init() that is exported in misc.h now. Signed-off-by: Juan Quintela --- include/migration/misc.h | 4 migration/block.c| 3 ++- {include/migration => migration}/block.h | 1 -

[Qemu-devel] [PATCH 28/41] migration: Split registration functions from vmstate.h

They are indpendent, and nowadays almost every device register things with qdev->vmsd. Signed-off-by: Juan Quintela --- hw/net/vmxnet3.c | 1 + hw/ppc/spapr.c | 1 + hw/s390x/s390-skeys.c| 1 + hw/s390x/s390-virtio-ccw.c | 1 +

[Qemu-devel] [PATCH 33/41] migration: Commands are only used inside migration.c

So, move them there. Notice that we export functions that send commands, not the command themselves. Signed-off-by: Juan Quintela --- include/migration/migration.h | 15 -- migration/migration.c | 46 +++ 2 files

[Qemu-devel] [PATCH 39/41] exec: Create include for target_page_size()

That is the only function that we need from exec.c, and having to include the whole sysemu.h for this. Signed-off-by: Juan Quintela --- exec.c | 1 + include/exec/target_page.h | 20 include/sysemu/sysemu.h| 1 -

  1   2   3   4   5   >