Re: [Qemu-devel] [Qemu-ppc] [PATCH v2 0/3] Enable MTTCG on PPC64

2017-04-09 Thread Nikunj A Dadhania
Alex Bennée writes: > luigi burdo writes: > >> Hi David and Nikuji, >> >> can i suggest to remove the message: >> >> >> Guest not yet converted to MTTCG - you may get unexpected results >> where the mttcg is enabled? > > Have you declared the

Re: [Qemu-devel] [PATCH 15/21] ppc/pnv: add initial IPMI sensors for the BMC simulator

2017-04-09 Thread David Gibson
On Wed, Apr 05, 2017 at 02:41:40PM +0200, Cédric Le Goater wrote: > Skiboot, the firmware for the PowerNV platform, expects the BMC to > provide some specific IPMI sensors. These sensors are exposed in the > device tree and their values are updated by the firmware at boot time. > > Sensors of

Re: [Qemu-devel] [PATCH 16/21] ppc/pnv: generate an OEM SEL event on shutdown

2017-04-09 Thread David Gibson
On Wed, Apr 05, 2017 at 02:41:41PM +0200, Cédric Le Goater wrote: > OpenPOWER systems expect to be notified with such an event before a > shutdown or a reboot. An OEM SEL message is sent with specific > identifiers and a user data containing the request : OFF or REBOOT. > > Signed-off-by: Cédric

Re: [Qemu-devel] [PATCH 17/21] qdev: Add a hook for a bus to device if it can add devices

2017-04-09 Thread David Gibson
On Wed, Apr 05, 2017 at 02:41:42PM +0200, Cédric Le Goater wrote: > From: Benjamin Herrenschmidt > > This allows a bus class to tell whether a given bus has room for > any new device. max_dev isn't sufficient as the rules can depend > on some arguments or can differ

Re: [Qemu-devel] [PATCH 14/21] ppc/pnv: populate device tree for IPMI BT devices

2017-04-09 Thread David Gibson
On Wed, Apr 05, 2017 at 02:41:39PM +0200, Cédric Le Goater wrote: > When an ipmi-bt device [1] is defined on the ISA bus, we need to > populate the device tree with the object properties. Such devices are > created with the command line options : > >-device ipmi-bmc-sim,id=bmc0 -device

Re: [Qemu-devel] [PATCH 18/21] pci: Use the new pci_can_add_device() to enforce devfn_min/max

2017-04-09 Thread David Gibson
On Wed, Apr 05, 2017 at 02:41:43PM +0200, Cédric Le Goater wrote: > From: Benjamin Herrenschmidt > > This adds a devfn_max field to PCIBus and adds a pci_can_add_device() > function which, if no "addr" (aka devfn) is specified, will tell whether > there is any slot free

Re: [Qemu-devel] [PATCH 12/21] ppc/pnv: populate device tree for RTC devices

2017-04-09 Thread David Gibson
On Wed, Apr 05, 2017 at 02:41:37PM +0200, Cédric Le Goater wrote: > The code could be common to any ISA device but we are missing the IO > length. > > Signed-off-by: Cédric Le Goater Reviewed-by: David Gibson > --- > hw/ppc/pnv.c | 30

Re: [Qemu-devel] [PATCH 11/21] ppc/pnv: scan ISA bus to populate device tree

2017-04-09 Thread David Gibson
On Wed, Apr 05, 2017 at 02:41:36PM +0200, Cédric Le Goater wrote: > This is an empty shell that we will use to include nodes in the device > tree for ISA devices. We expect RTC, UART and IPMI BT devices. > > Signed-off-by: Cédric Le Goater This is so simple, I'd probably fold it

Re: [Qemu-devel] [PATCH 13/21] ppc/pnv: populate device tree for serial devices

2017-04-09 Thread David Gibson
On Wed, Apr 05, 2017 at 02:41:38PM +0200, Cédric Le Goater wrote: > Signed-off-by: Cédric Le Goater Reviewed-by: David Gibson > --- > hw/ppc/pnv.c | 33 + > 1 file changed, 33 insertions(+) > > diff --git

[Qemu-devel] [Bug 1662050] Re: qemu-img convert a overlay qcow2 image into a entire image

2017-04-09 Thread wayen
Hi Lidong Chen: I used QEMU 2.0.0 on Ubuntu 14.04. Do you mean your patch can make qemu-img convert qcow2 overlay into a new overlay? -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1662050

Re: [Qemu-devel] [PATCH v9 1/9] memory: add section range info for IOMMU notifier

2017-04-09 Thread David Gibson
On Fri, Apr 07, 2017 at 06:59:07PM +0800, Peter Xu wrote: > In this patch, IOMMUNotifier.{start|end} are introduced to store section > information for a specific notifier. When notification occurs, we not > only check the notification type (MAP|UNMAP), but also check whether the > notified iova

Re: [Qemu-devel] [Bug 1662050] Re: qemu-img convert a overlay qcow2 image into a entire image

2017-04-09 Thread 858585 jemmy
Hi wayen: Which version are you used? I also find this problem on old version qemu, and i write a patch for it. It works. I'm not sure the mainline version have solve this problem. what command are you used? On Mon, Apr 10, 2017 at 10:14 AM, wayen <1662...@bugs.launchpad.net>

[Qemu-devel] 答复: 转发: Re: [PATCH] vhost: skip RAM device memory sections

2017-04-09 Thread lu.zhipeng
i have been tested many times,i confirm ram device region references is not be dropped,when detaching vhost net. So far ,i do not have better idea to drop the references. the ram device momery is special . i think vhost listening the RAM devicemomery is not necessary. skip RAM device

[Qemu-devel] [PATCH v3 1/2] block: Make bdrv_parent_drained_begin/end public

2017-04-09 Thread Fam Zheng
Signed-off-by: Fam Zheng --- block/io.c| 4 ++-- include/block/block.h | 16 2 files changed, 18 insertions(+), 2 deletions(-) diff --git a/block/io.c b/block/io.c index 7321dda..9598646 100644 --- a/block/io.c +++ b/block/io.c @@ -44,7 +44,7 @@

[Qemu-devel] [Bug 1662050] Re: qemu-img convert a overlay qcow2 image into a entire image

2017-04-09 Thread wayen
Is there any way to remove holes from qcow2 overlay images? It's very important to me. I am looking forward to your reply. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1662050 Title: qemu-img

[Qemu-devel] [PATCH v3 2/2] block: Quiesce old aio context during bdrv_set_aio_context

2017-04-09 Thread Fam Zheng
The fact that the bs->aio_context is changing can confuse the dataplane iothread, because of the now fine granularity aio context lock. bdrv_drain should rather be a bdrv_drained_begin/end pair, but since bs->aio_context is changing, we can just use aio_disable_external and

[Qemu-devel] [PATCH v3 0/2] block: Quiesce old aio context during bdrv_set_aio_context

2017-04-09 Thread Fam Zheng
v3: Use bdrv_parent_drained_begin/end. [Kevin] Do it before releasing new_context. [Stefan] Fam Zheng (2): block: Make bdrv_parent_drained_begin/end public block: Quiesce old aio context during bdrv_set_aio_context block.c | 7 +-- block/io.c| 4 ++--

[Qemu-devel] [Bug 1662050] Re: qemu-img convert a overlay qcow2 image into a entire image

2017-04-09 Thread wayen
Is there any way to remove holes from sparse qcow2 overlay? It's very important to me. I am looking forward to your reply. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1662050 Title: qemu-img

Re: [Qemu-devel] [PATCH v2] migration/block: use blk_pwrite_zeroes for each zero cluster

2017-04-09 Thread 858585 jemmy
On Mon, Apr 10, 2017 at 9:47 AM, Fam Zheng wrote: > On Sat, 04/08 21:29, 858585 jemmy wrote: >> On Sat, Apr 8, 2017 at 12:52 PM, 858585 jemmy wrote: >> > On Fri, Apr 7, 2017 at 6:10 PM, Fam Zheng wrote: >> >> On Fri, 04/07 16:44,

[Qemu-devel] 答复: 转发: Re: [PATCH] vhost: skip RAM device memory sections

2017-04-09 Thread lu.zhipeng
i'm sorry ,it should be memory_region_is_ram_device in new version qemu , memory_region_is_skip_dump is in older version qemu. 芦志朋 luzhipeng IT开发工程师 IT Development Engineer 操作系统产品部/中心研究院/系统产品 OS Product Dept./Central R&D Institute/System Product 四川省成都市天府大道中段800号 E:

Re: [Qemu-devel] [PATCH v2] migration/block: use blk_pwrite_zeroes for each zero cluster

2017-04-09 Thread Fam Zheng
On Sat, 04/08 21:29, 858585 jemmy wrote: > On Sat, Apr 8, 2017 at 12:52 PM, 858585 jemmy wrote: > > On Fri, Apr 7, 2017 at 6:10 PM, Fam Zheng wrote: > >> On Fri, 04/07 16:44, jemmy858...@gmail.com wrote: > >>> From: Lidong Chen >

Re: [Qemu-devel] [PATCH v3] migration/block: use blk_pwrite_zeroes for each zero cluster

2017-04-09 Thread Fam Zheng
On Mon, 04/10 09:10, 858585 jemmy wrote: > >> +if (ret < 0) { > >> +g_free(buf); > >> +return ret; > >> +} > > > > This if block is not necessary because... > > Hi Fam: > It's necessary

Re: [Qemu-devel] [PATCH v2 5/6] coroutine: Explicitly specify AioContext when entering coroutine

2017-04-09 Thread Fam Zheng
On Mon, 04/10 08:56, Paolo Bonzini wrote: > > > On 07/04/2017 23:14, Kevin Wolf wrote: > > One part of the reason is that > > BDRV_POLL_WHILE() drops the lock since commit c9d1a561, so just calling > > aio_context_acquire() doesn't protect you any more if there is any > > chance that a nested

Re: [Qemu-devel] [PATCH v3] migration/block: use blk_pwrite_zeroes for each zero cluster

2017-04-09 Thread 858585 jemmy
On Mon, Apr 10, 2017 at 8:51 AM, Fam Zheng wrote: > On Sun, 04/09 20:37, jemmy858...@gmail.com wrote: >> From: Lidong Chen >> >> BLOCK_SIZE is (1 << 20), qcow2 cluster size is 65536 by default, >> this maybe cause the qcow2 file size is bigger after

Re: [Qemu-devel] [PATCH v2 5/6] coroutine: Explicitly specify AioContext when entering coroutine

2017-04-09 Thread Paolo Bonzini
On 07/04/2017 23:14, Kevin Wolf wrote: > One part of the reason is that > BDRV_POLL_WHILE() drops the lock since commit c9d1a561, so just calling > aio_context_acquire() doesn't protect you any more if there is any > chance that a nested function calls BDRV_POLL_WHILE(). Hi guys, sorry for the

Re: [Qemu-devel] [PATCH v3] migration/block: use blk_pwrite_zeroes for each zero cluster

2017-04-09 Thread Fam Zheng
On Sun, 04/09 20:37, jemmy858...@gmail.com wrote: > From: Lidong Chen > > BLOCK_SIZE is (1 << 20), qcow2 cluster size is 65536 by default, > this maybe cause the qcow2 file size is bigger after migration. > This patch check each cluster, use blk_pwrite_zeroes for each >

Re: [Qemu-devel] [Xen-devel] [RFC/BUG] xen-mapcache: buggy invalidate map cache?

2017-04-09 Thread Alexey G
On Mon, 10 Apr 2017 00:36:02 +0800 hrg wrote: Hi, > On Sun, Apr 9, 2017 at 11:55 PM, hrg wrote: > > On Sun, Apr 9, 2017 at 11:52 PM, hrg wrote: > >> Hi, > >> > >> In xen_map_cache_unlocked(), map to guest memory maybe in

Re: [Qemu-devel] [PATCH v2 0/3] Enable MTTCG on PPC64

2017-04-09 Thread Richard Henderson
On 04/07/2017 11:51 PM, David Gibson wrote: On Fri, Apr 07, 2017 at 11:37:49AM +0530, Nikunj A Dadhania wrote: The series enables Multi-Threaded TCG on PPC64 Patch 01: Use atomic_cmpxchg in store conditional 02: Handle first write to page during atomic operation 03: Generate memory

Re: [Qemu-devel] [RFC/BUG] xen-mapcache: buggy invalidate map cache?

2017-04-09 Thread hrg
On Sun, Apr 9, 2017 at 11:55 PM, hrg wrote: > On Sun, Apr 9, 2017 at 11:52 PM, hrg wrote: >> Hi, >> >> In xen_map_cache_unlocked(), map to guest memory maybe in entry->next >> instead of first level entry (if map to rom other than guest memory >> comes

Re: [Qemu-devel] [RFC/BUG] xen-mapcache: buggy invalidate map cache?

2017-04-09 Thread hrg
On Sun, Apr 9, 2017 at 11:52 PM, hrg wrote: > Hi, > > In xen_map_cache_unlocked(), map to guest memory maybe in entry->next > instead of first level entry (if map to rom other than guest memory > comes first), while in xen_invalidate_map_cache(), when VM ballooned > out

[Qemu-devel] [RFC/BUG] xen-mapcache: buggy invalidate map cache?

2017-04-09 Thread hrg
Hi, In xen_map_cache_unlocked(), map to guest memory maybe in entry->next instead of first level entry (if map to rom other than guest memory comes first), while in xen_invalidate_map_cache(), when VM ballooned out memory, qemu did not invalidate cache entries in linked list(entry->next), so when

Re: [Qemu-devel] [Qemu-ppc] [PATCH v2 0/3] Enable MTTCG on PPC64

2017-04-09 Thread Alex Bennée
luigi burdo writes: > Hi David and Nikuji, > > can i suggest to remove the message: > > > Guest not yet converted to MTTCG - you may get unexpected results > where the mttcg is enabled? Have you declared the memory ordering for the guest? > > another thing im finding

Re: [Qemu-devel] [Qemu-ppc] [PATCH v2 0/3] Enable MTTCG on PPC64

2017-04-09 Thread Alex Bennée
luigi burdo writes: > Hi, on info is mttcg using an amouth of ram for cpu caching and > translating operations like was did in past by emulators like > virtualpc,realpc, bluelabel or softwindows? It's a fixed buffer. You can see if it being flushed by running: info

Re: [Qemu-devel] [PATCH v3] migration/block:limit the time used for block migration

2017-04-09 Thread 858585 jemmy
On Fri, Apr 7, 2017 at 7:33 PM, Stefan Hajnoczi wrote: > On Fri, Apr 07, 2017 at 09:30:33AM +0800, 858585 jemmy wrote: >> On Thu, Apr 6, 2017 at 10:02 PM, Stefan Hajnoczi wrote: >> > On Wed, Apr 05, 2017 at 05:27:58PM +0800, jemmy858...@gmail.com wrote:

[Qemu-devel] [PATCH v3] migration/block: use blk_pwrite_zeroes for each zero cluster

2017-04-09 Thread jemmy858585
From: Lidong Chen BLOCK_SIZE is (1 << 20), qcow2 cluster size is 65536 by default, this maybe cause the qcow2 file size is bigger after migration. This patch check each cluster, use blk_pwrite_zeroes for each zero cluster. Signed-off-by: Lidong Chen

[Qemu-devel] [Qemu-devel RFC v2 2/4] msf2: Microsemi Smartfusion2 System Register block.

2017-04-09 Thread Subbaraya Sundeep
Added Sytem register block of Smartfusion2. This block has PLL registers which are accessed by guest. Signed-off-by: Subbaraya Sundeep --- hw/misc/Makefile.objs | 1 + hw/misc/msf2_sysreg.c | 168 ++ 2 files changed, 169

[Qemu-devel] [Qemu-devel RFC v2 4/4] msf2: Add Emcraft's Smartfusion2 SOM kit.

2017-04-09 Thread Subbaraya Sundeep
Emulated Emcraft's Smartfusion2 System On Module starter kit. Signed-off-by: Subbaraya Sundeep --- default-configs/arm-softmmu.mak | 1 + hw/arm/Makefile.objs| 2 +- hw/arm/msf2_soc.c | 141 3 files

[Qemu-devel] [Qemu-devel RFC v2 3/4] msf2: Add Smartfusion2 SPI controller

2017-04-09 Thread Subbaraya Sundeep
Modelled Microsemi's Smartfusion2 SPI controller. Signed-off-by: Subbaraya Sundeep --- hw/ssi/Makefile.objs | 1 + hw/ssi/msf2_spi.c| 449 +++ 2 files changed, 450 insertions(+) create mode 100644 hw/ssi/msf2_spi.c

[Qemu-devel] [Qemu-devel RFC v2 0/4] Add support for Smartfusion2 SoC

2017-04-09 Thread Subbaraya Sundeep
Hi Qemu-devel, I am trying to add Smartfusion2 SoC. SoC is from Microsemi and System on Module(SOM) board is from Emcraft systems. Smartfusion2 has hardened Microcontroller(Cortex-M3)based Sub System and FPGA fabric. At the moment only system timer, sysreg and SPI controller are modelled.

[Qemu-devel] [Qemu-devel RFC v2 1/4] msf2: Add Smartfusion2 System timer

2017-04-09 Thread Subbaraya Sundeep
Modelled System Timer in Microsemi's Smartfusion2 Soc. Timer has two 32bit down counters and two interrupts. Signed-off-by: Subbaraya Sundeep --- hw/timer/Makefile.objs | 1 + hw/timer/msf2_timer.c | 273 + 2 files

Re: [Qemu-devel] [Qemu-ppc] [PATCH v2 0/3] Enable MTTCG on PPC64

2017-04-09 Thread luigi burdo
Hi David and Nikuji, can i suggest to remove the message: Guest not yet converted to MTTCG - you may get unexpected results where the mttcg is enabled? another thing im finding is this message Guest expects a stronger memory ordering than the host provides This may cause strange/hard to debug

Re: [Qemu-devel] [PATCH v2 0/3] Enable MTTCG on PPC64

2017-04-09 Thread David Gibson
On Fri, Apr 07, 2017 at 11:37:49AM +0530, Nikunj A Dadhania wrote: > The series enables Multi-Threaded TCG on PPC64 > > Patch 01: Use atomic_cmpxchg in store conditional > 02: Handle first write to page during atomic operation > 03: Generate memory barriers for sync/isync and

Re: [Qemu-devel] What is the best commit for record-replay?

2017-04-09 Thread Pranith Kumar
On Thu, Mar 23, 2017 at 4:05 AM, Igor R wrote: > Hi, > > I'm trying to use the deterministic record/replay feature, and I would > like to know which commit I should take to get it work. > In RC0 it seems to be broken. I tried pre-MTTCG commit 2421f381dc, as Can you retry

Re: [Qemu-devel] [Bug 1020309] Re: qemu-system-ppc: no keyboard after savevm/loadvm

2017-04-09 Thread Thomas Eschenbacher
Thomas Huth wrote: > Triaging old bug tickets ... can you still reproduce this issue with the > latest version of QEMU (version 2.9)? > > ** Changed in: qemu >Status: New => Incomplete > Hello Thomas, here my answer per email: I re-tested that simple sequence above and it seems to