Re: [Qemu-devel] drive-backup

2016-02-25 Thread Fam Zheng
On Thu, 02/25 08:49, Stefan Priebe - Profihost AG wrote: > > Am 22.02.2016 um 23:08 schrieb John Snow: > > > > > > On 02/22/2016 03:21 PM, Stefan Priebe wrote: > >> Hello, > >> > >> is there any chance or hack to work with a bigger cluster size for the > >> drive backup job? > >> > >> See: > >>

Re: [Qemu-devel] [PATCH] loader: fix potential memory leak

2016-02-25 Thread Cao jin
sorry, It just came to me that I forgot to cc qemu-trivial On 02/24/2016 05:12 PM, Cao jin wrote: Signed-off-by: Cao jin --- hw/core/loader.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/hw/core/loader.c b/hw/core/loader.c index 3a57415..24fea65 100644 --- a/hw/core/loader.c +++

Re: [Qemu-devel] [PATCH] qemu-ga: Fixed minor version switch issue

2016-02-25 Thread Leonid Bloch
On Feb 25, 2016 4:09 AM, "Michael Roth" wrote: > > Quoting Leonid Bloch (2016-01-11 03:12:41) > > With automatically generated GUID, on minor version changes, an error > > occurred, stating that there is a problem with the installer. > > Now, a notification is shown, warning the user that another

Re: [Qemu-devel] kvm: "warning: host doesn't support requested feature: CPUID.01H:ECX.x2apic [bit 21]"

2016-02-25 Thread Jan Kiszka
On 2016-02-25 09:33, Lan Tianyu wrote: > 2016-02-20 17:00 GMT+08:00 Paolo Bonzini : >> >> >> - Original Message - >>> From: "Jan Kiszka" >>> To: "Eduardo Habkost" , "Paolo Bonzini" >>> >>> Cc: "qemu-devel" , "kvm" >>> Sent: Saturday, February 20, 2016 9:09:32 AM >>> Subject: kvm: "warni

Re: [Qemu-devel] [OpenBIOS] Trying to figure out what 9 means to the escc

2016-02-25 Thread Hervé Poussineau
Le 24/02/2016 21:13, Programmingkid a écrit : With the below patch I can see that Mac OS 9.0 is having escc_mem_write() go to the SERIAL_CTRL case. Then to the W_CMD case. Then to the CMD_HI case. This then causes newreg to be set to the value 9. What I need to know is what is this value for.

Re: [Qemu-devel] KVM call for agenda for 2016-03-01

2016-02-25 Thread Thomas Huth
On 23.02.2016 16:33, Juan Quintela wrote: > > Hi > > Please, send any topic that you are interested in covering. > > At the end of Monday I will send an email with the agenda or the > cancellation of the call, so hurry up. Shall we talk about CPU hotplug again? There are a lot of discussions go

Re: [Qemu-devel] [PULL 05/23] pc-dimm: add pc_dimm_build_list()

2016-02-25 Thread Michael S. Tsirkin
On Thu, Feb 25, 2016 at 10:01:18AM +0300, Vladimir Sementsov-Ogievskiy wrote: > Stop! It builds fine here. Did you check out my tree? > > Hey, stubs/pc_dimm.c disappeared from this patch: > === from [PATCH v7 0/5] don't use NVDIMM for balooning/[PATCH 3/5] > pc-dimm: add pc_dimm_build_li

Re: [Qemu-devel] [PATCH 1/2] virtio-balloon: export all balloon statistics

2016-02-25 Thread Markus Armbruster
"Denis V. Lunev" writes: > On 02/24/2016 06:43 PM, Eric Blake wrote: >> On 02/24/2016 07:31 AM, Michael S. Tsirkin wrote: >>> Roman Kagan writes: On Tue, Feb 23, 2016 at 05:49:21PM +0200, Michael S. Tsirkin wrote: > On Tue, Feb 23, 2016 at 06:29:33PM +0300, Denis V. Lunev wrote: > >

Re: [Qemu-devel] [PATCH 1/2] virtio-balloon: export all balloon statistics

2016-02-25 Thread Michael S. Tsirkin
On Thu, Feb 25, 2016 at 09:44:06AM +0100, Markus Armbruster wrote: > "Denis V. Lunev" writes: > > > On 02/24/2016 06:43 PM, Eric Blake wrote: > >> On 02/24/2016 07:31 AM, Michael S. Tsirkin wrote: > >>> Roman Kagan writes: > On Tue, Feb 23, 2016 at 05:49:21PM +0200, Michael S. Tsirkin wrote

Re: [Qemu-devel] [PULL 05/23] pc-dimm: add pc_dimm_build_list()

2016-02-25 Thread Vladimir Sementsov-Ogievskiy
On 25.02.2016 11:39, Michael S. Tsirkin wrote: On Thu, Feb 25, 2016 at 10:01:18AM +0300, Vladimir Sementsov-Ogievskiy wrote: Stop! It builds fine here. Did you check out my tree? anyway, this file should be here, as it is added into Makefile: --- a/stubs/Makefile.objs +++ b/stubs/Makefile.ob

Re: [Qemu-devel] Making all TB invalidation asynchronous (MTTCG safe_work)?

2016-02-25 Thread Frederic Konrad
Hi Alex, We decided in Seattle to make this flag per tb (eg move it to the tb struct). On 24/02/2016 18:30, Alex Bennée wrote: > Hi, > > So I've been working on reducing MTTCG tb_lock contention and currently > have a tb_lock around the following code (in my cpu_exec): > > /* Note: we do it

Re: [Qemu-devel] [PATCH 3/3] replay: introduce block devices record/replay

2016-02-25 Thread Pavel Dovgalyuk
> From: Kevin Wolf [mailto:kw...@redhat.com] > > > Coroutines aren't randomly assigned to threads, but threads actively > > > enter coroutines. To my knowledge this happens only when starting a > > > request (either vcpu or I/O thread; consistent per device) or by a > > > callback when some event h

Re: [Qemu-devel] [PULL 05/23] pc-dimm: add pc_dimm_build_list()

2016-02-25 Thread Michael S. Tsirkin
On Thu, Feb 25, 2016 at 11:55:56AM +0300, Vladimir Sementsov-Ogievskiy wrote: > On 25.02.2016 11:39, Michael S. Tsirkin wrote: > >On Thu, Feb 25, 2016 at 10:01:18AM +0300, Vladimir Sementsov-Ogievskiy wrote: > >>Stop! > >It builds fine here. Did you check out my tree? > > anyway, this file should

[Qemu-devel] [PULL v2 00/23] vhost, virtio, pci, pc

2016-02-25 Thread Michael S. Tsirkin
Changes from v1: drop a missing dependency from the makefile. The following changes since commit 8eb779e4223a18db9838a49ece1bc72cfdfb7761: Merge remote-tracking branch 'remotes/kevin/tags/for-upstream' into staging (2016-02-22 16:55:41 +) are available in the git repository at: git

[Qemu-devel] [PULL v2 01/23] bios-linker-loader: document+validate input

2016-02-25 Thread Michael S. Tsirkin
While guest/host ABI is documented in hw/acpi/bios-linker-loader.c, the API was left undocumented. This adds documentation for all API functions. Additionally, input is validated to make sure all pointers fall within range of provided files. To allow this validation for checksum commands, bios_l

[Qemu-devel] [PULL v2 05/23] pc-dimm: add pc_dimm_build_list()

2016-02-25 Thread Michael S. Tsirkin
From: Vladimir Sementsov-Ogievskiy Like pc_dimm_build_list_sorted but not sorted - for cases where sorting is not necessary. Add stubbed version too - for targets without CONFIG_MEM_HOTPLUG. Signed-off-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. T

[Qemu-devel] [PULL v2 04/23] pc-dimm: rename pc_dimm_built_list()

2016-02-25 Thread Michael S. Tsirkin
From: Vladimir Sementsov-Ogievskiy s/pc_dimm_built_list/pc_dimm_build_list_sorted - need for add pc_dimm_build_list (not sorted) in the next patch - fix typo (built -> build) Signed-off-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Igor Mammedov Reviewed-by: Michael S. Tsirkin Signed-off-by

[Qemu-devel] [PULL v2 03/23] move get_current_ram_size to virtio-balloon.c

2016-02-25 Thread Michael S. Tsirkin
From: Vladimir Sementsov-Ogievskiy get_current_ram_size() is used only in virtio-balloon.c This patch moves it into virtio-balloon and make it static, to allow some balloon-specific tuning. Signed-off-by: Vladimir Sementsov-Ogievskiy Signed-off-by: Denis V. Lunev Reviewed-by: Eric Blake Revie

[Qemu-devel] [PULL v2 02/23] vhost-user: don't merge regions with different fds

2016-02-25 Thread Michael S. Tsirkin
vhost currently merges regions with contiguious virtual and physical addresses. This breaks for vhost-user since that also needs fds to match. Add a vhost_ops entry to compare the fds for vhost-user only. Cc: qemu-sta...@nongnu.org Cc: Victor Kaplansky Signed-off-by: Michael S. Tsirkin --- in

[Qemu-devel] [PULL v2 12/23] virtio: add AioContext-specific function for host notifiers

2016-02-25 Thread Michael S. Tsirkin
From: Paolo Bonzini This is used to register ioeventfd with a dataplane thread. Signed-off-by: Paolo Bonzini Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin Reviewed-by: Fam Zheng Acked-by: Stefan Hajnoczi --- include/hw/virtio/virtio.h | 2 ++ hw/virtio/virtio.c

[Qemu-devel] [PULL v2 10/23] block-migration: acquire AioContext as necessary

2016-02-25 Thread Michael S. Tsirkin
From: Paolo Bonzini This is needed because dataplane will run during block migration as well. The block device migration code is quite liberal in taking the iothread mutex. For simplicity, keep it the same way, even though one could actually choose between the BQL (for regular BlockDriverStates

[Qemu-devel] [PULL v2 06/23] virtio-balloon: rewrite get_current_ram_size()

2016-02-25 Thread Michael S. Tsirkin
From: Vladimir Sementsov-Ogievskiy Use pc_dimm_built_list() instead of qmp_pc_dimm_device_list() Actually, Qapi is not related to this internal helper. Signed-off-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- hw/virtio/virtio-balloon.

[Qemu-devel] [PULL v2 14/23] virtio-blk: fix "disabled data plane" mode

2016-02-25 Thread Michael S. Tsirkin
From: Paolo Bonzini In disabled mode, virtio-blk dataplane seems to be enabled, but flow actually goes through the normal virtio path. This patch simplifies a bit the handling of disabled mode. In disabled mode, virtio_blk_handle_output might be called even if s->dataplane is not NULL. This is

[Qemu-devel] [PULL v2 08/23] pci core: function pci_host_bus_register() cleanup

2016-02-25 Thread Michael S. Tsirkin
From: Cao jin remove unused param, and rename the other to a meaningful one. Signed-off-by: Cao jin Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin Reviewed-by: Eduardo Habkost --- hw/pci/pci.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/hw/pc

[Qemu-devel] [PULL v2 13/23] virtio: export vring_notify as virtio_should_notify

2016-02-25 Thread Michael S. Tsirkin
From: Paolo Bonzini Virtio dataplane needs to trigger the irq manually through the guest notifier. Export virtio_should_notify so that it can be used around event_notifier_set. Signed-off-by: Paolo Bonzini Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin Acked-by: Cornelia H

[Qemu-devel] [PULL v2 22/23] q35: Remove unused q35-acpi-dsdt.aml file

2016-02-25 Thread Michael S. Tsirkin
From: Eduardo Habkost The file was used only by older machine-types, and it is not needed anymore. Signed-off-by: Eduardo Habkost Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin Reviewed-by: Laszlo Ersek Reviewed-by: Markus Armbruster Reviewed-by: Igor Mammedov --- Makef

[Qemu-devel] [PULL v2 11/23] vring: make vring_enable_notification return void

2016-02-25 Thread Michael S. Tsirkin
From: Paolo Bonzini Make the API more similar to the regular virtqueue API. This will help when modifying the code to not use vring.c anymore. Signed-off-by: Paolo Bonzini Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin Acked-by: Cornelia Huck Reviewed-by: Fam Zheng Acked

[Qemu-devel] [PULL v2 15/23] virtio-blk: do not use vring in dataplane

2016-02-25 Thread Michael S. Tsirkin
From: Paolo Bonzini Signed-off-by: Paolo Bonzini Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin Reviewed-by: Fam Zheng Acked-by: Stefan Hajnoczi --- hw/block/dataplane/virtio-blk.h | 1 + include/hw/virtio/virtio-blk.h | 3 -- hw/block/dataplane/virtio-blk.c | 112 ++

[Qemu-devel] [PULL v2 07/23] balloon: Use only 'pc-dimm' type dimm for ballooning

2016-02-25 Thread Michael S. Tsirkin
From: Vladimir Sementsov-Ogievskiy For now there are only two dimm's: pc-dimm and nvdimm. This patch is actually needed to disable ballooning on nvdimm. But, to avoid future bugs, instead of disallowing nvdimm, we allow only pc-dimm. So, if someone adds new dimm which should be balloon-able, then

[Qemu-devel] [PULL v2 16/23] virtio-scsi: do not use vring in dataplane

2016-02-25 Thread Michael S. Tsirkin
From: Paolo Bonzini Signed-off-by: Paolo Bonzini Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin Acked-by: Stefan Hajnoczi --- include/hw/virtio/virtio-scsi.h | 21 + hw/scsi/virtio-scsi-dataplane.c | 196 +--- hw/scsi/virtio-scsi.c

[Qemu-devel] [PULL v2 09/23] pci core: function pci_bus_init() cleanup

2016-02-25 Thread Michael S. Tsirkin
From: Cao jin remove unused param Signed-off-by: Cao jin Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin Reviewed-by: Eduardo Habkost --- hw/pci/pci.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/hw/pci/pci.c b/hw/pci/pci.c index b071361..e676

[Qemu-devel] [PULL v2 21/23] ich9: Remove enable_tco arguments from init functions

2016-02-25 Thread Michael S. Tsirkin
From: Eduardo Habkost The enable_tco arguments are always true, so they are not needed anymore. Signed-off-by: Eduardo Habkost Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin Reviewed-by: Laszlo Ersek Reviewed-by: Markus Armbruster Reviewed-by: Igor Mammedov --- include/

Re: [Qemu-devel] CPU hotplug, again

2016-02-25 Thread David Gibson
On Wed, Feb 24, 2016 at 02:41:17PM +0100, Igor Mammedov wrote: > On Wed, 24 Feb 2016 22:28:22 +1100 > David Gibson wrote: > > > On Wed, Feb 24, 2016 at 11:48:33AM +0100, Igor Mammedov wrote: > > > On Wed, 24 Feb 2016 13:01:06 +1100 > > > David Gibson wrote: > > > > > > > On Tue, Feb 23, 2016

[Qemu-devel] [PULL v2 17/23] vring: remove

2016-02-25 Thread Michael S. Tsirkin
From: Paolo Bonzini Signed-off-by: Paolo Bonzini Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin Reviewed-by: Fam Zheng Acked-by: Stefan Hajnoczi --- include/hw/virtio/dataplane/vring-accessors.h | 75 include/hw/virtio/dataplane/vring.h | 51 --- hw/virti

[Qemu-devel] [PULL v2 23/23] q35: No need to check gigabyte_align

2016-02-25 Thread Michael S. Tsirkin
From: Eduardo Habkost gigabyte_align is always true on q35, so we don't need the !gigabyte_align compat code anymore. Signed-off-by: Eduardo Habkost Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin Reviewed-by: Laszlo Ersek Reviewed-by: Markus Armbruster Reviewed-by: Igor M

[Qemu-devel] [PULL v2 20/23] machine: Remove no_tco field

2016-02-25 Thread Michael S. Tsirkin
From: Eduardo Habkost The field is always set to zero, so it is not necessary anymore. Signed-off-by: Eduardo Habkost Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin Reviewed-by: Laszlo Ersek Reviewed-by: Markus Armbruster Reviewed-by: Igor Mammedov --- include/hw/boards

[Qemu-devel] [PULL v2 19/23] q35: Remove old machine versions

2016-02-25 Thread Michael S. Tsirkin
From: Eduardo Habkost Migration with q35 was not possible before commit 04329029a8c539eb5f75dcb6d8b016f0c53a031a, because q35 unconditionally creates an ich9-ahci device, that was marked as unmigratable. So all q35 machine classes before pc-q35-2.4 were not migratable, so there's no point in keep

[Qemu-devel] [PULL v2 18/23] tests/vhost-user-bridge: fix build on 32 bit systems

2016-02-25 Thread Michael S. Tsirkin
Mainly casts between void * and uint64_t, and wrong format for size_t. Signed-off-by: Michael S. Tsirkin --- tests/vhost-user-bridge.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/tests/vhost-user-bridge.c b/tests/vhost-user-bridge.c index f2b6c67..0779ba2 1

Re: [Qemu-devel] [PATCH 1/2] virtio-balloon: export all balloon statistics

2016-02-25 Thread Roman Kagan
On Thu, Feb 25, 2016 at 10:54:17AM +0200, Michael S. Tsirkin wrote: > On Thu, Feb 25, 2016 at 09:44:06AM +0100, Markus Armbruster wrote: > > "Denis V. Lunev" writes: > > > > > On 02/24/2016 06:43 PM, Eric Blake wrote: > > >> On 02/24/2016 07:31 AM, Michael S. Tsirkin wrote: > > >>> Roman Kagan w

[Qemu-devel] [PATCH] doc/memory.txt: fix typo

2016-02-25 Thread Cao jin
to match structure MemoryRegionOps definition Signed-off-by: Cao jin --- And there seems are some out of date content in this file, like "cpu_register_io_memory()" mentioned in the last line, which is removed several years ago in commit 97161e177. I am sure how to modify it, So I take the liberty

[Qemu-devel] [PATCH 0/3] Add some more files to the MAINTAINERS file

2016-02-25 Thread Thomas Huth
These three patches add some more USB, Spice and UI related files to the MAINTAINERS file. Gerd, I'm not sure whether you want to take the third patch as is, since this adds the whole include/ui/ folder to one of your sections, so feel free to ignore it if you think it is too generic. Thomas Huth

[Qemu-devel] [PATCH 3/3] MAINTAINERS: Add an entry for the include/ui/ folder

2016-02-25 Thread Thomas Huth
The ui/ folder is listed in the "Graphics" section, so I think the "include/ui/" folder should be listed there, too. Signed-off-by: Thomas Huth --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 18f2c17..72e55fa 100644 --- a/MAINTAINERS +++ b/MAIN

[Qemu-devel] [PATCH 1/3] MAINTAINERS: Add some missing entries for USB related files

2016-02-25 Thread Thomas Huth
USB-related docs and include files should go into the USB section of the MAINTAINERS file. Signed-off-by: Thomas Huth --- MAINTAINERS | 4 1 file changed, 4 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 606d9c0..67d70a9 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -853,6 +853

[Qemu-devel] [PATCH 2/3] MAINTAINERS: Add spice-display.h to the SPICE section

2016-02-25 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 67d70a9..18f2c17 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1087,6 +1087,7 @@ SPICE M: Gerd Hoffmann S: Supported F: include/ui/qemu-spice.h +F: include/ui/spice-

[Qemu-devel] [PATCH] MAINTAINERS: Add docs/migration.txt to the "Migration" section

2016-02-25 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 72e55fa..9148966 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1247,6 +1247,7 @@ F: include/migration/ F: migration/ F: scripts/vmstate-static-checker.py F: tests/vms

Re: [Qemu-devel] [PULL 0/8] tcg queued patches

2016-02-25 Thread Peter Maydell
On 23 February 2016 at 18:33, Richard Henderson wrote: > A revision of the Feb 15 pull, with patch 5 adjusted to work around > the reported Werror bug in gcc 4.8.2, and Peter's renaming patch > set included. > > > r~ > > > The following changes since commit 90ce6e2644db2c47d72f364b4de57342e50bd10a

[Qemu-devel] [Bug 1543057] Re: Warnings are treated as errors

2016-02-25 Thread madship
I have same error "PCIDevice has no member named 'qdev'" with you. Did you find any solutions to this error? -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1543057 Title: Warnings are treated as err

Re: [Qemu-devel] [Qemu-ppc] Migrating decrementer

2016-02-25 Thread Mark Cave-Ayland
On 25/02/16 05:00, Mark Cave-Ayland wrote: > On 25/02/16 04:33, Mark Cave-Ayland wrote: > >> cpu_start/resume(): >> cpu->tb_env->tb_offset = >> qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL) * tb_env->tb_freq + >> cpu->tb_env->tb_offset - >> qemu_clock_get_ns(QEMU_CLOCK_HOS

[Qemu-devel] [Bug 1549654] [NEW] qemu-system-arm emulator

2016-02-25 Thread Mehdi Amiri
Public bug reported: Hi, I don't know if this is a bug or a feature in new QEMU software. I was following an online tutorial using QEMU to develop a simple bare-metal program for qemu-system-arm. I decided to try a more recent software and I got surprised when I found the small code can not run

Re: [Qemu-devel] kvm: "warning: host doesn't support requested feature: CPUID.01H:ECX.x2apic [bit 21]"

2016-02-25 Thread Lan Tianyu
2016-02-20 17:00 GMT+08:00 Paolo Bonzini : > > > - Original Message - >> From: "Jan Kiszka" >> To: "Eduardo Habkost" , "Paolo Bonzini" >> >> Cc: "qemu-devel" , "kvm" >> Sent: Saturday, February 20, 2016 9:09:32 AM >> Subject: kvm: "warning: host doesn't support requested feature: >> CP

Re: [Qemu-devel] [PULL 05/23] pc-dimm: add pc_dimm_build_list()

2016-02-25 Thread Vladimir Sementsov-Ogievskiy
On 25.02.2016 12:11, Michael S. Tsirkin wrote: On Thu, Feb 25, 2016 at 11:55:56AM +0300, Vladimir Sementsov-Ogievskiy wrote: On 25.02.2016 11:39, Michael S. Tsirkin wrote: On Thu, Feb 25, 2016 at 10:01:18AM +0300, Vladimir Sementsov-Ogievskiy wrote: Stop! It builds fine here. Did you check ou

[Qemu-devel] [PATCH] MAINTAINERS: Remove the old s390-virtio machine

2016-02-25 Thread Thomas Huth
The old s390-virtio machine has been removed last year, so we don't need the corresponding section in the MAINTAINERS file anymore. Signed-off-by: Thomas Huth --- MAINTAINERS | 6 -- 1 file changed, 6 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 9148966..341574c 100644 --- a/MA

Re: [Qemu-devel] [PULL v2 05/23] pc-dimm: add pc_dimm_build_list()

2016-02-25 Thread Vladimir Sementsov-Ogievskiy
here stubs are deleted. it will break compilation for targets without memory hotplug, it was discussed on list. stubs from my patch are necessary. On 25.02.2016 12:16, Michael S. Tsirkin wrote: From: Vladimir Sementsov-Ogievskiy Like pc_dimm_build_list_sorted but not sorted - for cases where

Re: [Qemu-devel] [PATCH] doc/memory.txt: fix typo

2016-02-25 Thread Peter Maydell
On 25 February 2016 at 09:32, Cao jin wrote: > to match structure MemoryRegionOps definition > > Signed-off-by: Cao jin > --- > And there seems are some out of date content in this file, like > "cpu_register_io_memory()" mentioned in the last line, which is removed > several > years ago in commi

Re: [Qemu-devel] [PATCH] MAINTAINERS: Remove the old s390-virtio machine

2016-02-25 Thread Thomas Huth
On 25.02.2016 10:57, Thomas Huth wrote: > The old s390-virtio machine has been removed last year, so we don't > need the corresponding section in the MAINTAINERS file anymore. > > Signed-off-by: Thomas Huth > --- > MAINTAINERS | 6 -- > 1 file changed, 6 deletions(-) > > diff --git a/MAINTA

[Qemu-devel] [PATCH] Postcopy: Fix sync count in info migrate

2016-02-25 Thread Dr. David Alan Gilbert (git)
From: "Dr. David Alan Gilbert" I'd missed the sync count off in the postcopy case. Signed-off-by: Dr. David Alan Gilbert --- migration/migration.c | 1 + 1 file changed, 1 insertion(+) diff --git a/migration/migration.c b/migration/migration.c index fc5e50b..34d3e5f 100644 --- a/migration/mig

Re: [Qemu-devel] [PATCH] doc/memory.txt: fix typo

2016-02-25 Thread Cao jin
On 02/25/2016 06:00 PM, Peter Maydell wrote: On 25 February 2016 at 09:32, Cao jin wrote: to match structure MemoryRegionOps definition Signed-off-by: Cao jin --- And there seems are some out of date content in this file, like "cpu_register_io_memory()" mentioned in the last line, which is

Re: [Qemu-devel] [PATCH 1/2] virtio-balloon: export all balloon statistics

2016-02-25 Thread Michael S. Tsirkin
On Thu, Feb 25, 2016 at 12:30:21PM +0300, Roman Kagan wrote: > On Thu, Feb 25, 2016 at 10:54:17AM +0200, Michael S. Tsirkin wrote: > > On Thu, Feb 25, 2016 at 09:44:06AM +0100, Markus Armbruster wrote: > > > "Denis V. Lunev" writes: > > > > > > > On 02/24/2016 06:43 PM, Eric Blake wrote: > > > >>

Re: [Qemu-devel] [PULL 05/23] pc-dimm: add pc_dimm_build_list()

2016-02-25 Thread Paolo Bonzini
On 25/02/2016 10:54, Vladimir Sementsov-Ogievskiy wrote: > On 25.02.2016 12:11, Michael S. Tsirkin wrote: >> On Thu, Feb 25, 2016 at 11:55:56AM +0300, Vladimir Sementsov-Ogievskiy >> wrote: >>> On 25.02.2016 11:39, Michael S. Tsirkin wrote: On Thu, Feb 25, 2016 at 10:01:18AM +0300, Vladimir

Re: [Qemu-devel] [PATCH 1/9] dma-helpers: Expose the sg mapping logic

2016-02-25 Thread Alex Pyrgiotis
Hi Paolo, Thanks a lot for your clarifications. See my comments inline: (tl;dr: I suggest we reconsider Fam Zheng's attempt to remove the global bounce buffer, which would make dma-helpers simpler and unblock this patch) On 02/22/2016 12:43 PM, Paolo Bonzini wrote: > > > On 19/02/2016 12:50, A

[Qemu-devel] [PATCH] MAINTAINERS: Add an entry for virtio header files

2016-02-25 Thread Thomas Huth
Files in the include/hw/virtio/ folder should be included in the "virtio" sections of the MAINTAINERS file. Signed-off-by: Thomas Huth --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 341574c..cda55c4 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @

Re: [Qemu-devel] [PULL 05/23] pc-dimm: add pc_dimm_build_list()

2016-02-25 Thread Michael S. Tsirkin
On Thu, Feb 25, 2016 at 11:09:16AM +0100, Paolo Bonzini wrote: > > > On 25/02/2016 10:54, Vladimir Sementsov-Ogievskiy wrote: > > On 25.02.2016 12:11, Michael S. Tsirkin wrote: > >> On Thu, Feb 25, 2016 at 11:55:56AM +0300, Vladimir Sementsov-Ogievskiy > >> wrote: > >>> On 25.02.2016 11:39, Micha

[Qemu-devel] [PATCH] MAINTAINERS: Add an entry for the include/hw/virtio/dataplane folder

2016-02-25 Thread Thomas Huth
dataplane header files belong to virtio-blk, so there should be a corresponding entry in the MAINTAINERS file. Signed-off-by: Thomas Huth --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index cda55c4..901a1b2 100644 --- a/MAINTAINERS +++ b/MAINTAINER

[Qemu-devel] [Bug 1543057] Re: Warnings are treated as errors

2016-02-25 Thread Peter Maydell
(a) That warnings are treated as errors is a feature, not a bug (it happens for development builds only) (b) the definition of struct PCIDevice in include/hw/pci/pci.h starts with "DeviceState qdev;" so it's not clear to me how that error could be produced in the first place I see the original

Re: [Qemu-devel] [PATCH 1/9] dma-helpers: Expose the sg mapping logic

2016-02-25 Thread Paolo Bonzini
On 25/02/2016 11:10, Alex Pyrgiotis wrote: > All normal regions in a QEMUSGList point to an address range in the > guest's RAM. The MMIO regions of QEMU's virtual devices, however, do not > correspond to such an address range, so QEMU must create a bounce buffer > to represent them. This bounce b

Re: [Qemu-devel] [PULL 05/23] pc-dimm: add pc_dimm_build_list()

2016-02-25 Thread Michael S. Tsirkin
On Thu, Feb 25, 2016 at 11:09:16AM +0100, Paolo Bonzini wrote: > > > On 25/02/2016 10:54, Vladimir Sementsov-Ogievskiy wrote: > > On 25.02.2016 12:11, Michael S. Tsirkin wrote: > >> On Thu, Feb 25, 2016 at 11:55:56AM +0300, Vladimir Sementsov-Ogievskiy > >> wrote: > >>> On 25.02.2016 11:39, Micha

Re: [Qemu-devel] [RFC] QMP: add query-hotpluggable-cpus

2016-02-25 Thread Igor Mammedov
On Thu, 25 Feb 2016 12:03:21 +1100 David Gibson wrote: > On Wed, Feb 24, 2016 at 12:03:41PM +0100, Igor Mammedov wrote: > > On Wed, 24 Feb 2016 21:51:19 +1100 > > David Gibson wrote: > > > > > On Wed, Feb 24, 2016 at 09:42:10AM +0100, Markus Armbruster wrote: > > > > David Gibson writes: >

Re: [Qemu-devel] [PATCH] MAINTAINERS: Add an entry for the include/hw/virtio/dataplane folder

2016-02-25 Thread Paolo Bonzini
On 25/02/2016 11:14, Thomas Huth wrote: > dataplane header files belong to virtio-blk, so there should be a > corresponding entry in the MAINTAINERS file. > > Signed-off-by: Thomas Huth > --- > MAINTAINERS | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > in

Re: [Qemu-devel] [PATCH] MAINTAINERS: Remove the old s390-virtio machine

2016-02-25 Thread Cornelia Huck
On Thu, 25 Feb 2016 11:02:03 +0100 Thomas Huth wrote: > By the way, I think you can now also remove the "X: > hw/s390x/s390-virtio-bus.[ch]" entry from the virtio-ccw machine, since > these two files do not exist anymore. I'd apply such a patch as well :)

Re: [Qemu-devel] [PATCH] MAINTAINERS: Remove the old s390-virtio machine

2016-02-25 Thread Cornelia Huck
On Thu, 25 Feb 2016 10:57:54 +0100 Thomas Huth wrote: > The old s390-virtio machine has been removed last year, so we don't > need the corresponding section in the MAINTAINERS file anymore. > > Signed-off-by: Thomas Huth > --- > MAINTAINERS | 6 -- > 1 file changed, 6 deletions(-) Thanks,

Re: [Qemu-devel] [PATCH] MAINTAINERS: Add an entry for the include/hw/virtio/dataplane folder

2016-02-25 Thread Thomas Huth
On 25.02.2016 11:23, Paolo Bonzini wrote: > > > On 25/02/2016 11:14, Thomas Huth wrote: >> dataplane header files belong to virtio-blk, so there should be a >> corresponding entry in the MAINTAINERS file. >> >> Signed-off-by: Thomas Huth >> --- >> MAINTAINERS | 1 + >> 1 file changed, 1 inserti

Re: [Qemu-devel] [PATCH] MAINTAINERS: Add an entry for the include/hw/virtio/dataplane folder

2016-02-25 Thread Paolo Bonzini
On 25/02/2016 11:27, Thomas Huth wrote: > On 25.02.2016 11:23, Paolo Bonzini wrote: >> >> >> On 25/02/2016 11:14, Thomas Huth wrote: >>> dataplane header files belong to virtio-blk, so there should be a >>> corresponding entry in the MAINTAINERS file. >>> >>> Signed-off-by: Thomas Huth >>> --- >

[Qemu-devel] [PATCH v5 1/8] hw/timer: QOM'ify etraxfs_timer

2016-02-25 Thread xiaoqiang zhao
assign etraxfs_timer_init to etraxfs_timer_info.instance_init and drop the SysBusDeviceClass::init Reviewed-by: Edgar E. Iglesias Signed-off-by: xiaoqiang zhao --- hw/timer/etraxfs_timer.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/hw/timer/etraxfs_timer.

[Qemu-devel] [PATCH v5 4/8] hw/timer: QOM'ify m48txx_sysbus (pass 2)

2016-02-25 Thread xiaoqiang zhao
assign DeviceClass::vmsd instead of using vmstate_register function Signed-off-by: xiaoqiang zhao --- hw/timer/m48t59.c | 24 ++-- 1 file changed, 22 insertions(+), 2 deletions(-) diff --git a/hw/timer/m48t59.c b/hw/timer/m48t59.c index 3c683aa..f5897d8 100644 --- a/hw/timer

[Qemu-devel] [PATCH v5 5/8] hw/timer: QOM'ify milkymist_sysctl

2016-02-25 Thread xiaoqiang zhao
* split the old SysBus init function into an instance_init and a Device realize function * use DeviceClass::realize instead of SysBusDeviceClass::init Reviewed-by: Peter Maydell Signed-off-by: xiaoqiang zhao --- hw/timer/milkymist-sysctl.c | 21 + 1 file changed, 13 insert

[Qemu-devel] [PATCH v5 3/8] hw/timer: QOM'ify m48txx_sysbus (pass 1)

2016-02-25 Thread xiaoqiang zhao
* split the old SysBus init function into an instance_init and a Device realize function * use DeviceClass::realize instead of SysBusDeviceClass::init Signed-off-by: xiaoqiang zhao --- hw/timer/m48t59.c | 35 ++- 1 file changed, 18 insertions(+), 17 deletions(-)

[Qemu-devel] [PATCH v5 8/8] hw/timer: QOM'ify grlib_gptimer

2016-02-25 Thread xiaoqiang zhao
* split the old SysBus init function into an instance_init and a Device realize function * use DeviceClass::realize instead of SysBusDeviceClass::init Signed-off-by: xiaoqiang zhao --- hw/timer/grlib_gptimer.c | 32 +++- 1 file changed, 19 insertions(+), 13 deletion

[Qemu-devel] [PATCH v5 7/8] hw/timer: QOM'ify slavio_timer

2016-02-25 Thread xiaoqiang zhao
rename slavio_timer_init1 to slavio_timer_init and assign it to slavio_timer_info.instance_init, then we drop the SysBusDeviceClass::init Signed-off-by: xiaoqiang zhao --- hw/timer/slavio_timer.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/hw/timer/slavio_tim

[Qemu-devel] [PATCH v5 2/8] hw/timer: QOM'ify lm32_timer

2016-02-25 Thread xiaoqiang zhao
* split the old SysBus init function into an instance_init and a Device realize function * use DeviceClass::realize instead of SysBusDeviceClass::init Reviewed-by: Peter Maydell Signed-off-by: xiaoqiang zhao --- hw/timer/lm32_timer.c | 19 --- 1 file changed, 12 insertions(+),

[Qemu-devel] [PATCH v5 6/8] hw/timer: QOM'ify puv3_ost

2016-02-25 Thread xiaoqiang zhao
assign puv3_ost_init to puv3_ost_info.instance_init and drop the SysBusDeviceClass::init Signed-off-by: xiaoqiang zhao --- hw/timer/puv3_ost.c | 18 +- 1 file changed, 5 insertions(+), 13 deletions(-) diff --git a/hw/timer/puv3_ost.c b/hw/timer/puv3_ost.c index 93650b7..72c87ba

[Qemu-devel] [PATCH v5 0/8] QOM'ify hw/timer/*

2016-02-25 Thread xiaoqiang zhao
This patch series QOM'ify timer code under hw/timer directory. Main idea is to split the initfn's work, some to TypeInfo.instance_init and some is placed in DeviceClass::realize. Drop the use of SysBusDeviceClass::init if possible. Patch 3,4 (m48t59) has been tested in a sparc vm with debian linux

[Qemu-devel] [PATCH] MAINTAINERS: Remove entry for hw/s390x/s390-virtio-bus.[ch]

2016-02-25 Thread Thomas Huth
The files have been deleted recently, no need to keep these entries anymore. Signed-off-by: Thomas Huth --- MAINTAINERS | 1 - 1 file changed, 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index cda55c4..0f15d88 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -660,7 +660,6 @@ M: Alexander

Re: [Qemu-devel] [PULL 00/19] Misc changes for 2016-02-24

2016-02-25 Thread Peter Maydell
On 24 February 2016 at 13:27, Paolo Bonzini wrote: > The following changes since commit dd5e38b19d7cb07d317e1285941d8245c01da540: > > Merge remote-tracking branch > 'remotes/pmaydell/tags/pull-target-arm-20160218-1' into staging (2016-02-18 > 15:20:35 +) > > are available in the git reposi

Re: [Qemu-devel] [PATCH 1/2] virtio-balloon: export all balloon statistics

2016-02-25 Thread Markus Armbruster
"Michael S. Tsirkin" writes: > On Thu, Feb 25, 2016 at 12:30:21PM +0300, Roman Kagan wrote: >> On Thu, Feb 25, 2016 at 10:54:17AM +0200, Michael S. Tsirkin wrote: >> > On Thu, Feb 25, 2016 at 09:44:06AM +0100, Markus Armbruster wrote: >> > > "Denis V. Lunev" writes: >> > > >> > > > On 02/24/201

Re: [Qemu-devel] [PATCH] MAINTAINERS: Remove entry for hw/s390x/s390-virtio-bus.[ch]

2016-02-25 Thread Cornelia Huck
On Thu, 25 Feb 2016 11:45:00 +0100 Thomas Huth wrote: > The files have been deleted recently, no need to keep these entries > anymore. > > Signed-off-by: Thomas Huth > --- > MAINTAINERS | 1 - > 1 file changed, 1 deletion(-) Thanks, applied.

Re: [Qemu-devel] [PATCH v5 4/8] hw/timer: QOM'ify m48txx_sysbus (pass 2)

2016-02-25 Thread Peter Maydell
On 25 February 2016 at 10:30, xiaoqiang zhao wrote: > assign DeviceClass::vmsd instead of using vmstate_register function > > Signed-off-by: xiaoqiang zhao > --- > hw/timer/m48t59.c | 24 ++-- > 1 file changed, 22 insertions(+), 2 deletions(-) So, did you test this, and if s

Re: [Qemu-devel] [PATCH v5 0/8] QOM'ify hw/timer/*

2016-02-25 Thread Peter Maydell
On 25 February 2016 at 10:30, xiaoqiang zhao wrote: > This patch series QOM'ify timer code under hw/timer directory. > Main idea is to split the initfn's work, some to TypeInfo.instance_init > and some is placed in DeviceClass::realize. > Drop the use of SysBusDeviceClass::init if possible. > > Pa

Re: [Qemu-devel] [PATCH] Postcopy: Fix sync count in info migrate

2016-02-25 Thread Hailiang Zhang
On 2016/2/25 18:03, Dr. David Alan Gilbert (git) wrote: From: "Dr. David Alan Gilbert" I'd missed the sync count off in the postcopy case. Signed-off-by: Dr. David Alan Gilbert --- migration/migration.c | 1 + 1 file changed, 1 insertion(+) diff --git a/migration/migration.c b/migration/m

[Qemu-devel] [PATCH] MAINTAINERS: Add entry for the include/hw/vfio/ folder

2016-02-25 Thread Thomas Huth
The headers in include/hw/vfio/ should be listed in the VFIO section of the MAINTAINERS file. Signed-off-by: Thomas Huth --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 0f15d88..9179386 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -861,6 +861,

Re: [Qemu-devel] [PATCH v6 2/3] qmp event: Refactor QUORUM_REPORT_BAD

2016-02-25 Thread Alberto Garcia
On Thu 25 Feb 2016 06:33:08 AM CET, Changlong Xie wrote: > +Read/Write operation: > { "event": "QUORUM_REPORT_BAD", > - "data": { "node-name": "node0", "sector-num": 345435, "sectors-count": > 5 }, > + "data": { "node-name": "node0", "sector-num": 345435, "sectors-count": > 5, > +

Re: [Qemu-devel] [PATCH] Postcopy: Fix sync count in info migrate

2016-02-25 Thread Dr. David Alan Gilbert
* Hailiang Zhang (zhang.zhanghaili...@huawei.com) wrote: > On 2016/2/25 18:03, Dr. David Alan Gilbert (git) wrote: > >From: "Dr. David Alan Gilbert" > > > >I'd missed the sync count off in the postcopy case. > > > >Signed-off-by: Dr. David Alan Gilbert > >--- > > migration/migration.c | 1 + > >

[Qemu-devel] [PULL v2 5/8] linux-user: sync syscall numbers with kernel

2016-02-25 Thread riku . voipio
From: Riku Voipio Sync syscall numbers to match the linux v4.5-rc1 kernel. Reviewed-by: Laurent Vivier Signed-off-by: Riku Voipio --- linux-user/aarch64/syscall_nr.h| 2 +- linux-user/alpha/syscall_nr.h | 6 ++ linux-user/cris/syscall_nr.h | 24 l

[Qemu-devel] [PULL v2 0/8] linux-user updates and fixes

2016-02-25 Thread riku . voipio
iku.voipio/qemu.git tags/pull-linux-user-20160225 for you to fetch changes up to f894efd19917321844b31e5dd7a061fdc6fb322d: linux-user: add getrandom() syscall (2016-02-24 15:22:15 +0200) Second pull req with get

[Qemu-devel] [PULL v2 6/8] linux-user: remove unavailable syscalls from aarch64

2016-02-25 Thread riku . voipio
From: Riku Voipio QEMU lists deprecated system call numbers in for Aarch64. These are never enabled for Linux kernel, so don't define them in Qemu either. Remove the ifdef around host_to_target_stat64 since all architectures need it now. Reviewed-by: Peter Maydell Signed-off-by: Riku Voipio --

[Qemu-devel] [PULL v2 2/8] build: [linux-user] Rename "syscall.h" to "target_syscall.h" in target directories

2016-02-25 Thread riku . voipio
From: Lluís Vilanova This fixes double-definitions in linux-user builds when using the UST tracing backend (which indirectly includes the system's "syscall.h"). Signed-off-by: Lluís Vilanova Reviewed-by: Peter Maydell Signed-off-by: Riku Voipio --- linux-user/aarch64/syscall.h | 1

[Qemu-devel] [PULL v2 1/8] linux-user: fix realloc size of target_fd_trans.

2016-02-25 Thread riku . voipio
From: Laurent Vivier target_fd_trans is an array of "TargetFdTrans *": compute size accordingly. Use g_renew() as proposed by Paolo. Reported-by: Paolo Bonzini Signed-off-by: Laurent Vivier Signed-off-by: Riku Voipio --- linux-user/syscall.c | 4 ++-- 1 file changed, 2 insertions(+), 2 delet

[Qemu-devel] [PULL v2 3/8] linux-user: set ppc64/ppc64le default CPU to POWER8

2016-02-25 Thread riku . voipio
From: Laurent Vivier Set the default to the latest CPU version to have the largest set of available features. It is also really needed in little-endian mode because POWER7 is not really supported in this mode and some distros (at least debian) generate POWER8 code for their ppc64le target. Fixe

[Qemu-devel] [PULL v2 8/8] linux-user: add getrandom() syscall

2016-02-25 Thread riku . voipio
From: Laurent Vivier getrandom() has been introduced in kernel 3.17 and is now used during the boot sequence of Debian unstable (stretch/sid). Signed-off-by: Laurent Vivier Signed-off-by: Riku Voipio --- linux-user/syscall.c | 13 + 1 file changed, 13 insertions(+) diff --git a/l

[Qemu-devel] [PULL v2 4/8] linux-user: Don't assert if guest tries shmdt(0)

2016-02-25 Thread riku . voipio
From: Peter Maydell Our implementation of shmat() and shmdt() for linux-user was using "zero guest address" as its marker for "entry in the shm_regions[] array is not in use". This meant that if the guest did a shmdt(0) we would match on an unused array entry and call page_set_flags() with both s

[Qemu-devel] [Bug 1549654] Re: qemu-system-arm emulator

2016-02-25 Thread Peter Maydell
> the 1st instruction stores R3 at [fp, #-8] and the second instruction can not restores the value from the same address In bare metal code this usually means you're trying to store to an address which does not actually have any RAM in it. Here R13=00011088, and for the vexpress-a9 board that has

  1   2   3   4   >