[Qemu-devel] IDE disk FLUSH take more than 30 secs, the SUSE guest reports "lost interrupt and the file system becomes read-only"

2013-05-21 Thread Gonglei (Arei)
In the case of physical hard disk's speed which processing IO (when grouping RAID) is very slow, I encountered a problem. I dd big file in SUSE virtual machine, the command is linux:/ # dd if=/dev/zero of=./info bs=1M count=5000;sync but finally I get those message: linux:~ # dmesg [ 174.804114]

Re: [Qemu-devel] [PATCH v3 0/8] block: drive-backup live backup command

2013-05-21 Thread Stefan Hajnoczi
On Mon, May 20, 2013 at 09:23:43AM +0200, Paolo Bonzini wrote: > Il 20/05/2013 08:24, Stefan Hajnoczi ha scritto: > >> > You only need to fdatasync() before every guest flush, no? > > No, you need to set the dirty bit before issuing the write on the > > host. Otherwise the image data may be modifi

Re: [Qemu-devel] [PATCH v2 2/2] net: introduce command to query mac-table information

2013-05-21 Thread Stefan Hajnoczi
On Tue, May 21, 2013 at 12:46:09PM +0800, Amos Kong wrote: > On Fri, May 17, 2013 at 09:39:31AM +0200, Stefan Hajnoczi wrote: > > On Thu, May 16, 2013 at 07:07:25PM +0800, Amos Kong wrote: > > Hi Stefan, > > > > @@ -961,6 +961,44 @@ void print_net_client(Monitor *mon, NetClientState > > > *nc) >

Re: [Qemu-devel] [PATCH v3 00/10] curl: fix curl read

2013-05-21 Thread Richard W.M. Jones
On Tue, May 21, 2013 at 09:54:15AM +0800, Fam Zheng wrote: > On Mon, 05/20 09:49, Richard W.M. Jones wrote: > > On Mon, May 20, 2013 at 09:41:06AM +0100, Richard W.M. Jones wrote: > > > On Mon, May 20, 2013 at 03:03:34PM +0800, Fam Zheng wrote: > > > > CURL library API has changed, the current curl

Re: [Qemu-devel] [PATCH v3 0/8] block: drive-backup live backup command

2013-05-21 Thread Stefan Hajnoczi
On Tue, May 21, 2013 at 11:25:01AM +0800, Wenchao Xia wrote: > 于 2013-5-17 17:14, Stefan Hajnoczi 写道: > >On Fri, May 17, 2013 at 02:58:57PM +0800, Wenchao Xia wrote: > >>于 2013-5-16 15:47, Stefan Hajnoczi 写道: > >>>On Thu, May 16, 2013 at 02:16:20PM +0800, Wenchao Xia wrote: > After checking

[Qemu-devel] [Bug 1182344] [NEW] ARM: invalid code execution after subs instruction

2013-05-21 Thread Sebastian Huber
Public bug reported: I use Qemu (Git 1239b472bb0dba8060f1af29d40dafbc1b2860d4) to test a SMP application on ARM. I use the following command line: qemu-system-arm -S -s -net none -nographic -M realview-pbx-a9 -kernel app.exe -m 256M -no-reboot -smp 2 -icount 8 The application crashes because Qe

Re: [Qemu-devel] [PATCH] hw/9pfs: use O_NOFOLLOW for mapped readlink operation

2013-05-21 Thread Stefan Hajnoczi
On Mon, May 20, 2013 at 11:04:29PM +0530, Aneesh Kumar K.V wrote: > From: "Aneesh Kumar K.V" > > With mapped security models like mapped-xattr and mapped-file, we save the > symlink target as file contents. Now if we ever expose a normal directory > with mapped security model and find real symlin

Re: [Qemu-devel] [PATCH 0/2] coroutine: dataplane support

2013-05-21 Thread Stefan Hajnoczi
On Fri, May 17, 2013 at 04:08:36PM +0100, Peter Maydell wrote: > On 17 May 2013 14:51, Stefan Hajnoczi wrote: > > There is ongoing work to enable multiple event loop threads. This will > > allow > > QEMU itself to take advantage of SMP and reduce Big QEMU Lock (BQL) > > contention. > > This ser

Re: [Qemu-devel] [libvirt] Default machine type setting for ppc64

2013-05-21 Thread Li Zhang
Oops, correct libvirt mailing list address. :) Thanks. On 2013年05月21日 16:19, Li Zhang wrote: Hi all, Sorry to bring this problem up again. We encounter this problem in openstack which always use default machine type. Currently, QEMU sets mac99 as default setting for ppc64 but it doesn't work

[Qemu-devel] [qemu-devel][libvirt] Default machine type setting for ppc64

2013-05-21 Thread Li Zhang
Hi all, Sorry to bring this problem up again. We encounter this problem in openstack which always use default machine type. Currently, QEMU sets mac99 as default setting for ppc64 but it doesn't work on our platform at all. I tried to fix this in libvirt which it is not acceptable because libvi

Re: [Qemu-devel] [PATCH v3 0/8] block: drive-backup live backup command

2013-05-21 Thread Paolo Bonzini
Il 21/05/2013 09:31, Stefan Hajnoczi ha scritto: > On Mon, May 20, 2013 at 09:23:43AM +0200, Paolo Bonzini wrote: >> Il 20/05/2013 08:24, Stefan Hajnoczi ha scritto: > You only need to fdatasync() before every guest flush, no? >>> No, you need to set the dirty bit before issuing the write on th

[Qemu-devel] [Bug 1182344] Re: ARM: invalid code execution after subs instruction

2013-05-21 Thread Peter Maydell
Can you attach or provide a link to your app.exe test case? Otherwise this is going to be harder to track down and test. Thanks! -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1182344 Title: ARM: in

Re: [Qemu-devel] [qemu-devel][libvirt] Default machine type setting for ppc64

2013-05-21 Thread Peter Maydell
On 21 May 2013 09:19, Li Zhang wrote: > We encounter this problem in openstack which always use > default machine type. Currently, QEMU sets mac99 as default > setting for ppc64 but it doesn't work on our platform at all. > > I tried to fix this in libvirt which it is not acceptable because > libv

Re: [Qemu-devel] [PATCH] ps2: add support of auto-repeat

2013-05-21 Thread Paolo Bonzini
Il 21/05/2013 10:33, Amos Kong ha scritto: > On Thu, May 16, 2013 at 05:20:37PM +0200, Paolo Bonzini wrote: >> Il 16/05/2013 17:17, Peter Maydell ha scritto: >>> On 16 May 2013 16:09, Paolo Bonzini wrote: ... In XWindows, you get a KeyRelease for every KeyPress Event. In X, it looks som

Re: [Qemu-devel] [qemu-devel][libvirt] Default machine type setting for ppc64

2013-05-21 Thread Daniel P. Berrange
On Tue, May 21, 2013 at 09:31:26AM +0100, Peter Maydell wrote: > On 21 May 2013 09:19, Li Zhang wrote: > > We encounter this problem in openstack which always use > > default machine type. Currently, QEMU sets mac99 as default > > setting for ppc64 but it doesn't work on our platform at all. > > >

Re: [Qemu-devel] [PATCH] ps2: add support of auto-repeat

2013-05-21 Thread Amos Kong
On Thu, May 16, 2013 at 05:20:37PM +0200, Paolo Bonzini wrote: > Il 16/05/2013 17:17, Peter Maydell ha scritto: > > On 16 May 2013 16:09, Paolo Bonzini wrote: > >> ... In XWindows, you get a KeyRelease for every KeyPress Event. In X, > >> it looks something like this: > >> > >> PRPRPRPRPRPRPRPR >

Re: [Qemu-devel] [PATCH v2 1/2] net: introduce MAC_TABLE_CHANGED event

2013-05-21 Thread Michael S. Tsirkin
On Tue, May 21, 2013 at 01:04:55PM +0800, Amos Kong wrote: > On Thu, May 16, 2013 at 03:17:45PM +0300, Michael S. Tsirkin wrote: > > On Thu, May 16, 2013 at 07:07:24PM +0800, Amos Kong wrote: > > > Introduce this new QMP event to notify management after guest changes > > > mac-table configuration.

Re: [Qemu-devel] [qemu-devel][libvirt] Default machine type setting for ppc64

2013-05-21 Thread Peter Maydell
On 21 May 2013 09:39, Daniel P. Berrange wrote: > Libvirt has always had support for specifying what machine type to use. > This discussion is simply about what machine type to default to, if the > user hasn't explicitly asked for one. > > QEMU has the notion of a default machine for each target,

Re: [Qemu-devel] [qemu-devel][libvirt] Default machine type setting for ppc64

2013-05-21 Thread Li Zhang
On 2013年05月21日 16:31, Peter Maydell wrote: On 21 May 2013 09:19, Li Zhang wrote: We encounter this problem in openstack which always use default machine type. Currently, QEMU sets mac99 as default setting for ppc64 but it doesn't work on our platform at all. I tried to fix this in libvirt whic

Re: [Qemu-devel] [PATCH 2/2] pcie: Add more ASPM support

2013-05-21 Thread Michael S. Tsirkin
On Mon, May 20, 2013 at 02:44:25PM -0600, Bjorn Helgaas wrote: > On Mon, May 20, 2013 at 12:32 PM, Michael S. Tsirkin wrote: > > On Mon, May 20, 2013 at 11:09:56AM -0600, Bjorn Helgaas wrote: > >> Indicate ASPM L0s and L1 support in Link Capabilities and make the ASPM > >> bits in Link Control wri

Re: [Qemu-devel] [qemu-devel][libvirt] Default machine type setting for ppc64

2013-05-21 Thread Li Zhang
On 2013年05月21日 16:45, Peter Maydell wrote: On 21 May 2013 09:39, Daniel P. Berrange wrote: Libvirt has always had support for specifying what machine type to use. This discussion is simply about what machine type to default to, if the user hasn't explicitly asked for one. QEMU has the notion o

Re: [Qemu-devel] [PATCH] ps2: add support of auto-repeat

2013-05-21 Thread Amos Kong
On Tue, May 21, 2013 at 10:38:00AM +0200, Paolo Bonzini wrote: > Il 21/05/2013 10:33, Amos Kong ha scritto: > > On Thu, May 16, 2013 at 05:20:37PM +0200, Paolo Bonzini wrote: > >> Il 16/05/2013 17:17, Peter Maydell ha scritto: > >>> On 16 May 2013 16:09, Paolo Bonzini wrote: > ... In XWindow

[Qemu-devel] [Bug 1182344] Re: ARM: invalid code execution after subs instruction

2013-05-21 Thread Sebastian Huber
** Attachment added: "ELF file of the application." https://bugs.launchpad.net/qemu/+bug/1182344/+attachment/3682743/+files/app.exe -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1182344 Title:

Re: [Qemu-devel] [qemu-devel][libvirt] Default machine type setting for ppc64

2013-05-21 Thread Peter Maydell
On 21 May 2013 10:02, Li Zhang wrote: > On 2013年05月21日 16:45, Peter Maydell wrote: >> On 21 May 2013 09:39, Daniel P. Berrange wrote: >>> Libvirt has always had support for specifying what machine type to use. >> OK, that makes sense. So is the problem here just configuration >> or is it the nex

Re: [Qemu-devel] [qemu-devel][libvirt] Default machine type setting for ppc64

2013-05-21 Thread Daniel P. Berrange
On Tue, May 21, 2013 at 05:02:51PM +0800, Li Zhang wrote: > On 2013年05月21日 16:45, Peter Maydell wrote: > >On 21 May 2013 09:39, Daniel P. Berrange wrote: > >>Libvirt has always had support for specifying what machine type to use. > >>This discussion is simply about what machine type to default to,

Re: [Qemu-devel] [PATCH arm-devs v1 1/5] sd/sd.c: Fix "inquiry" ACMD41

2013-05-21 Thread Peter Maydell
On 21 May 2013 07:50, wrote: > From: Peter Crosthwaite > > the SD command ACMD41 can be used in a read only mode to query device > state without doing the SD card initialisation. This is valid even > which the device is already initialised. Fix the command to be > responsive when in the ready st

Re: [Qemu-devel] [PATCH] ps2: add support of auto-repeat

2013-05-21 Thread Amos Kong
On Tue, May 21, 2013 at 05:04:30PM +0800, Amos Kong wrote: > On Tue, May 21, 2013 at 10:38:00AM +0200, Paolo Bonzini wrote: > > > Please correct me if something is wrong, thanks. > > > > > > When we use VNC/SPICE/SDL, vm Window will captured the key events, > > > then qemu process the events and

Re: [Qemu-devel] [PATCH arm-devs v1 4/5] sd/sdhci.c: Fix bdata_read DPRINT message

2013-05-21 Thread Peter Maydell
On 21 May 2013 07:52, wrote: > From: Peter Crosthwaite > > This message was printing out the data in decimal only, which is not > very friendly to the debugging developer. Add hex variant in > parenthesis to make it consistent with other similar messages in this > module. > > Signed-off-by: Pete

Re: [Qemu-devel] [PATCH] ps2: add support of auto-repeat

2013-05-21 Thread Paolo Bonzini
Il 21/05/2013 11:51, Amos Kong ha scritto: > On Tue, May 21, 2013 at 05:04:30PM +0800, Amos Kong wrote: >> On Tue, May 21, 2013 at 10:38:00AM +0200, Paolo Bonzini wrote: Please correct me if something is wrong, thanks. When we use VNC/SPICE/SDL, vm Window will captured the key events

Re: [Qemu-devel] [qemu-devel][libvirt] Default machine type setting for ppc64

2013-05-21 Thread Daniel P. Berrange
On Tue, May 21, 2013 at 07:55:27PM +1000, Paul Mackerras wrote: > On Tue, May 21, 2013 at 09:39:53AM +0100, Daniel P. Berrange wrote: > > On Tue, May 21, 2013 at 09:31:26AM +0100, Peter Maydell wrote: > > > On 21 May 2013 09:19, Li Zhang wrote: > > > > We encounter this problem in openstack which

Re: [Qemu-devel] [qemu-devel][libvirt] Default machine type setting for ppc64

2013-05-21 Thread Peter Maydell
On 21 May 2013 11:01, Daniel P. Berrange wrote: > On Tue, May 21, 2013 at 07:55:27PM +1000, Paul Mackerras wrote: >> I think libvirt needs some more sensible way to ask qemu what its >> capabilities are. Currently it has no way to ask qemu "what machines >> can you emulate with kvm acceleration?"

Re: [Qemu-devel] [for 1.5? Qemu-devel] [PATCH 2/3] chardev: Make the name of ringbuf device consistent

2013-05-21 Thread Lei Li
On 05/20/2013 11:15 PM, Paolo Bonzini wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Il 20/05/2013 17:05, Eric Blake ha scritto: On 05/20/2013 04:59 AM, Paolo Bonzini wrote: Il 20/05/2013 12:43, Paolo Bonzini ha scritto: Il 20/05/2013 08:51, Lei Li ha scritto: Now we have ringbuf char

[Qemu-devel] [PATCH 2/2] chardev: Get filename for new qapi backend

2013-05-21 Thread Lei Li
This patch sets the filename when the new qapi backend init from opts. The previous patch and discussions as link below: http://patchwork.ozlabs.org/patch/243896/ If anyone who have better idea to fix this please let me know your suggestions. Signed-off-by: Lei Li --- qemu-char.c |2 ++ 1

Re: [Qemu-devel] [PATCH v3 0/8] block: drive-backup live backup command

2013-05-21 Thread Stefan Hajnoczi
On Tue, May 21, 2013 at 10:30:22AM +0200, Paolo Bonzini wrote: > Il 21/05/2013 09:31, Stefan Hajnoczi ha scritto: > > On Mon, May 20, 2013 at 09:23:43AM +0200, Paolo Bonzini wrote: > >> Il 20/05/2013 08:24, Stefan Hajnoczi ha scritto: > > You only need to fdatasync() before every guest flush, n

Re: [Qemu-devel] [PATCH v3 0/8] block: drive-backup live backup command

2013-05-21 Thread Paolo Bonzini
Il 21/05/2013 12:34, Stefan Hajnoczi ha scritto: > On Tue, May 21, 2013 at 10:30:22AM +0200, Paolo Bonzini wrote: >> Il 21/05/2013 09:31, Stefan Hajnoczi ha scritto: >>> On Mon, May 20, 2013 at 09:23:43AM +0200, Paolo Bonzini wrote: Il 20/05/2013 08:24, Stefan Hajnoczi ha scritto: >>> You

[Qemu-devel] [PATCH 0/2 v2] Chardev related fixes

2013-05-21 Thread Lei Li
This series tries to fix the consistency of char devices ringbuf and a regression of chardev filename. Changes since v1: - Rename the struct related to new qapi ringbuf to memory, and fix the wrong description in qemu-option from Paolo. Lei Li (2): chardev: Make the name of memory device

[Qemu-devel] [PATCH 1/2] chardev: Make the name of memory device consistent

2013-05-21 Thread Lei Li
Now we have memory char device, but the backend name of it is a little confusion. We actually register it by 'memory', but the description in qemu-option, the name of open functions and the new api backend called it 'ringbuf'. It should keep consistent. This patch named it all to 'memory'. Signed-

[Qemu-devel] [PATCH 00/30] Introduction of IOMMUs into the memory API

2013-05-21 Thread Paolo Bonzini
This is part 1 of the memory API updates for 1.6. (Part 2 is the introduction of get/set_owner, part 3 is the introduction of the RCU-style flatview). Compared to the first submissions, there are a few extra cleanup patches, and address_space_rw/read/write grew a return value so that the existing

[Qemu-devel] [PATCH 01/30] exec: remove obsolete comment

2013-05-21 Thread Paolo Bonzini
See how we call memory_region_section_addr two lines below to convert a physical address to a base address in the region. Signed-off-by: Paolo Bonzini --- exec.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/exec.c b/exec.c index aec65c5..197625c 100644 --- a/exec.c +++ b/exec.c @@ -6

[Qemu-devel] [PATCH 02/30] exec: eliminate qemu_put_ram_ptr

2013-05-21 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- exec.c| 8 hw/pci/pci.c | 2 -- hw/scsi/megasas.c | 1 - include/exec/cpu-common.h | 1 - trace-events | 3 --- 5 files changed, 15 deletions(-) diff --git a/exec.c b/exec.c index 197625c..fa5f9c3 100

[Qemu-devel] [PATCH 05/30] memory: assert that PhysPageEntry's ptr does not overflow

2013-05-21 Thread Paolo Bonzini
While sized to 15 bits in PhysPageEntry, the ptr field is ORed into the iotlb entries together with a page-aligned pointer. The ptr field must not overflow into this page-aligned value, assert that it is smaller than the page size. Signed-off-by: Paolo Bonzini --- exec.c | 2 ++ 1 file changed,

[Qemu-devel] [PATCH 04/30] exec: eliminate stq_phys_notdirty

2013-05-21 Thread Paolo Bonzini
It is not used anywhere. Signed-off-by: Paolo Bonzini --- exec.c| 27 --- include/exec/cpu-common.h | 1 - include/exec/poison.h | 1 - 3 files changed, 29 deletions(-) diff --git a/exec.c b/exec.c index fa5f9c3..1355661 100644 --- a/exec.c +++

[Qemu-devel] [PATCH 08/30] memory: Rename readable flag to romd_mode

2013-05-21 Thread Paolo Bonzini
From: Jan Kiszka "Readable" is a very unfortunate name for this flag because even a rom_device region will always be readable from the guest POV. What differs is the mapping, just like the comments had to explain already. Also, readable could currently be understood as being a generic region flag

[Qemu-devel] [PATCH 03/30] exec: make qemu_get_ram_ptr private

2013-05-21 Thread Paolo Bonzini
It is a private interface between exec.c and memory.c. Signed-off-by: Paolo Bonzini --- include/exec/cpu-common.h | 2 -- include/exec/memory-internal.h | 1 + 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/include/exec/cpu-common.h b/include/exec/cpu-common.h index cafc3c2..

[Qemu-devel] [PATCH 11/30] memory: fix address space initialization/destruction

2013-05-21 Thread Paolo Bonzini
From: Avi Kivity A couple of fields were left uninitialized. This was not observed earlier because all address spaces were statically allocated. Also free allocation for those fields. Signed-off-by: Avi Kivity Signed-off-by: Paolo Bonzini --- memory.c | 3 +++ 1 file changed, 3 insertions(+

[Qemu-devel] [PATCH 07/30] memory: Replace open-coded memory_region_is_romd

2013-05-21 Thread Paolo Bonzini
From: Jan Kiszka Improves readability. Signed-off-by: Jan Kiszka --- translate-all.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/translate-all.c b/translate-all.c index da93608..0d84b0d 100644 --- a/translate-all.c +++ b/translate-all.c @@ -1359,7 +1359,7 @@ void tb_inv

[Qemu-devel] [PATCH 15/30] memory: add address_space_valid

2013-05-21 Thread Paolo Bonzini
The old-style IOMMU lets you check whether an access is valid in a given DMAContext. There is no equivalent for AddressSpace in the memory API, implement it with a lookup of the dispatch tree. Signed-off-by: Paolo Bonzini --- dma-helpers.c | 5 + exec.c| 25

[Qemu-devel] [PATCH 06/30] memory: allow memory_region_find() to run on non-root memory regions

2013-05-21 Thread Paolo Bonzini
memory_region_find() is similar to registering a MemoryListener and checking for the MemoryRegionSections that come from a particular region. There is no reason for this to be limited to a root memory region. Signed-off-by: Paolo Bonzini --- include/exec/memory.h | 13 +++-- memory.c

[Qemu-devel] [PATCH 16/30] memory: clean up phys_page_find

2013-05-21 Thread Paolo Bonzini
Remove the goto. Signed-off-by: Paolo Bonzini --- exec.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/exec.c b/exec.c index 8f1b507..82da067 100644 --- a/exec.c +++ b/exec.c @@ -187,19 +187,15 @@ MemoryRegionSection *phys_page_find(AddressSpaceDispatch *d, hwaddr

[Qemu-devel] [PATCH 10/30] memory: make memory_global_sync_dirty_bitmap take an AddressSpace

2013-05-21 Thread Paolo Bonzini
Suggested-by: Peter Maydell Signed-off-by: Paolo Bonzini --- arch_init.c | 2 +- include/exec/memory.h | 7 +++ memory.c | 3 +-- 3 files changed, 5 insertions(+), 7 deletions(-) diff --git a/arch_init.c b/arch_init.c index 49c5dc2..5d32ecf 100644 --- a/arch_init.c ++

[Qemu-devel] [PATCH 18/30] memory: add return value to address_space_rw/read/write

2013-05-21 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- exec.c| 23 ++- include/exec/memory.h | 12 +--- 2 files changed, 19 insertions(+), 16 deletions(-) diff --git a/exec.c b/exec.c index e5ee8ff..4cc5ecb 100644 --- a/exec.c +++ b/exec.c @@ -1901,7 +1901,7 @@ static void

[Qemu-devel] [PATCH 09/30] memory: do not duplicate memory_region_destructor_none

2013-05-21 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- memory.c | 5 - 1 file changed, 5 deletions(-) diff --git a/memory.c b/memory.c index 013464b..5431463 100644 --- a/memory.c +++ b/memory.c @@ -768,10 +768,6 @@ static void memory_region_destructor_ram_from_ptr(MemoryRegion *mr) qemu_ram_free_from_ptr(m

[Qemu-devel] [PATCH 19/30] memory: Introduce address_space_lookup_region

2013-05-21 Thread Paolo Bonzini
From: Jan Kiszka This introduces a wrapper for phys_page_find (before we complicate address_space_translate with IOMMU translation). This function will also encapsulate locking and reference counting when we introduce BQL-free dispatching. Signed-off-by: Jan Kiszka Signed-off-by: Paolo Bonzini

[Qemu-devel] [PATCH 23/30] spapr: convert TCE API to use an opaque type

2013-05-21 Thread Paolo Bonzini
The TCE table is currently returned as a DMAContext, and non-type-safe APIs are called later passing back the DMAContext. Since we want to move away from DMAContext, use an opaque type instead, and add an accessor to retrieve the DMAContext from it. Acked-by: David Gibson Signed-off-by: Paolo Bo

[Qemu-devel] [PATCH 22/30] vfio: abort if an emulated iommu is used

2013-05-21 Thread Paolo Bonzini
From: Avi Kivity vfio doesn't support guest iommus yet, indicate it to the user by gently depositing a core on their disk. Signed-off-by: Avi Kivity Signed-off-by: Paolo Bonzini --- hw/misc/vfio.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/misc/vfio.c b/hw/misc/vfio.c index 693a

[Qemu-devel] [PATCH 24/30] spapr: make IOMMU translation go through IOMMUTLBEntry

2013-05-21 Thread Paolo Bonzini
The next step is to introduce the translation code that will be used for IOMMU MemoryRegions, but still do the actual translation in a DMAContext. Acked-by: David Gibson Signed-off-by: Paolo Bonzini --- hw/ppc/spapr_iommu.c | 59 ++-- 1 file chang

[Qemu-devel] [PATCH 30/30] memory: give name to every AddressSpace

2013-05-21 Thread Paolo Bonzini
From: Alexey Kardashevskiy The "info mtree" command in QEMU console prints only "memory" and "I/O" address spaces while there are actually a lot more other AddressSpace structs created by PCI and VIO devices. Those devices do not normally have names and therefore not present in "info mtree" outpu

[Qemu-devel] [PATCH 14/30] memory: create FlatView for new address spaces

2013-05-21 Thread Paolo Bonzini
Even a new address space might have a non-empty FlatView. In order to initialize it properly, address_space_init should (a) call memory_region_transaction_commit after the address space is inserted into the list; (b) force memory_region_transaction_commit to do something. This bug was latent so f

[Qemu-devel] [PATCH 27/30] pci: use memory core for iommu support

2013-05-21 Thread Paolo Bonzini
From: Avi Kivity Use the new iommu support in the memory core for iommu support. The only user, spapr, is also converted, but it still provides a DMAContext interface until the non-PCI bits switch to AddressSpace. Cc: Michael S. Tsirkin Signed-off-by: Avi Kivity [ Do not calls memory_region_d

[Qemu-devel] [PATCH 17/30] memory: add address_space_translate

2013-05-21 Thread Paolo Bonzini
Using phys_page_find to translate an AddressSpace to a MemoryRegionSection is unwieldy. It requires to pass the page index rather than the address, and later memory_region_section_addr has to be called. Replace memory_region_section_addr with a function that does all of it: call phys_page_find, c

[Qemu-devel] [PATCH 29/30] dma: eliminate DMAContext

2013-05-21 Thread Paolo Bonzini
The DMAContext is a simple pointer to an AddressSpace that is now always already available. Make everyone hold the address space directly, and clean up the DMA API to use the AddressSpace directly. Signed-off-by: Paolo Bonzini --- dma-helpers.c | 24 +-- exec.c

[Qemu-devel] [PATCH 26/30] dma: eliminate old-style IOMMU support

2013-05-21 Thread Paolo Bonzini
The translate function in the DMAContext is now always NULL. Remove every reference to it. Signed-off-by: Paolo Bonzini --- dma-helpers.c| 178 +++ exec.c | 3 +- hw/pci/pci.c | 3 +- hw/ppc/spapr_iommu.c | 2 +-

[Qemu-devel] [PATCH 20/30] memory: iommu support

2013-05-21 Thread Paolo Bonzini
From: Avi Kivity Add a new memory region type that translates addresses it is given, then forwards them to a target address space. This is similar to an alias, except that the mapping is more flexible than a linear translation and trucation, and also less efficient since the translation happens

Re: [Qemu-devel] [PATCH v3 0/8] block: drive-backup live backup command

2013-05-21 Thread Dietmar Maurer
> >> True, but that would happen only in case the host crashes. Even for > >> a QEMU crash the changes would be safe, I think. They would be > >> written back when the persistent dirty bitmap's mmap() area is > >> unmapped, during process exit. > > > > I'd err on the side of caution, mark the per

[Qemu-devel] [PATCH 21/30] memory: Add iommu map/unmap notifiers

2013-05-21 Thread Paolo Bonzini
From: David Gibson This patch adds a NotifierList to MemoryRegions which represent IOMMUs allowing other parts of the code to register interest in mappings or unmappings from the IOMMU. All IOMMU implementations will need to call memory_region_notify_iommu() to inform those waiting on the notifi

[Qemu-devel] [PATCH 28/30] spapr_vio: take care of creating our own AddressSpace/DMAContext

2013-05-21 Thread Paolo Bonzini
Fetch the root region from the sPAPRTCETable, and use it to build an AddressSpace and DMAContext. Now, everywhere we have a DMAContext we also have access to the corresponding AddressSpace (either because we create it just before the DMAContext, or because dma_context_memory's AddressSpace is triv

[Qemu-devel] Get Guest Virtual to Host Physical address mapping

2013-05-21 Thread Shehbaz Jaffer
Hi, I am trying to find out the Guest VA to Host PA mapping in QEMU. Can someone give ideas about the data-structures that are used for storing the same? From the code, It looks like the CPU_X86 state contains control registers, and from the cr[3] register, I can get the page table address. but ca

[Qemu-devel] [PATCH 25/30] spapr: use memory core for iommu support

2013-05-21 Thread Paolo Bonzini
Now we can stop using a "translating" DMAContext, but we do not yet modify the sPAPRTCETable users to get an AddressSpace; they keep using the table via a DMAContext. Acked-by: David Gibson Signed-off-by: Paolo Bonzini --- hw/ppc/spapr_iommu.c | 49 +++-

Re: [Qemu-devel] [PATCH] Fix mask of pte index

2013-05-21 Thread Andreas Färber
Nuohan, Am 21.05.2013 08:16, schrieb Jesse Larrew: > On 05/20/2013 10:00 PM, Qiao Nuohan wrote: >> Function walk_pte needs pte index to calculate virtual address. However, pte >> index of PAE paging or IA-32e paging is 9 bit, so the mask should be 0x1ff. >> --- >> target-i386/arch_memory_mapping.

[Qemu-devel] (Another) 1.4.1 -> 1.5.0 migration failure

2013-05-21 Thread Nicholas Thomas
Hi all, Migrating from: /opt/qemu-1.4.1/bin/qemu-system-x86_64 -M pc -watchdog i6300esb -watchdog-action reset [...] to: /opt/qemu-1.5.0/bin/qemu-system-x86_64 -M pc-i440fx-1.4 -watchdog i6300esb -watchdog-action reset [...] I get: qemu: warning: error while loading state for instance 0x0

Re: [Qemu-devel] [PATCH 01/30] exec: remove obsolete comment

2013-05-21 Thread Peter Maydell
On 21 May 2013 11:57, Paolo Bonzini wrote: > See how we call memory_region_section_addr two lines below to > convert a physical address to a base address in the region. > > Signed-off-by: Paolo Bonzini Reviewed-by: Peter Maydell (I'm sure I've reviewed this patch already in a previous version)

Re: [Qemu-devel] [PATCH v1 1/1] qom/object: Don't poll cast cache for NULL objects

2013-05-21 Thread Andreas Färber
Am 21.05.2013 08:05, schrieb peter.crosthwa...@xilinx.com: > From: Peter Crosthwaite > > object_dymamic_cast_assert used to be tolerant of NULL objects and not "dynamic" > assert. Its clear from the implementation that this is the expected "It's" > behavior. > > The preceding check of the ca

Re: [Qemu-devel] [PATCH 02/30] exec: eliminate qemu_put_ram_ptr

2013-05-21 Thread Peter Maydell
On 21 May 2013 11:57, Paolo Bonzini wrote: > Signed-off-by: Paolo Bonzini Reviewed-by: Peter Maydell -- PMM

Re: [Qemu-devel] [PATCH 03/30] exec: make qemu_get_ram_ptr private

2013-05-21 Thread Peter Maydell
On 21 May 2013 11:57, Paolo Bonzini wrote: > It is a private interface between exec.c and memory.c. > > Signed-off-by: Paolo Bonzini Reviewed-by: Peter Maydell -- PMM

Re: [Qemu-devel] IDE disk FLUSH take more than 30 secs, the SUSE guest reports "lost interrupt and the file system becomes read-only"

2013-05-21 Thread Andreas Färber
Hi, Am 21.05.2013 09:12, schrieb Gonglei (Arei): > In the case of physical hard disk's speed which processing IO (when grouping > RAID) is very slow, I encountered a problem. > I dd big file in SUSE virtual machine, the command is > linux:/ # dd if=/dev/zero of=./info bs=1M count=5000;sync > > b

[Qemu-devel] [PATCH 13/30] memory: limit sections in the radix tree to the actual address space size

2013-05-21 Thread Paolo Bonzini
From: Avi Kivity The radix tree is statically sized to fit TARGET_PHYS_ADDR_SPACE_BITS. If a larger memory region is registered, it will overflow. Fix by limiting any section in the radix tree to the supported size. This problem was not observed earlier since artificial regions (containers and

Re: [Qemu-devel] [PATCH 07/30] memory: Replace open-coded memory_region_is_romd

2013-05-21 Thread Peter Maydell
On 21 May 2013 11:57, Paolo Bonzini wrote: > From: Jan Kiszka > > Improves readability. > > Signed-off-by: Jan Kiszka Reviewed-by: Peter Maydell for about the third time. -- PMM

Re: [Qemu-devel] (Another) 1.4.1 -> 1.5.0 migration failure

2013-05-21 Thread Andreas Färber
Hi, Am 21.05.2013 13:33, schrieb Nicholas Thomas: > Migrating from: > > /opt/qemu-1.4.1/bin/qemu-system-x86_64 -M pc -watchdog i6300esb > -watchdog-action reset [...] > > to: > > /opt/qemu-1.5.0/bin/qemu-system-x86_64 -M pc-i440fx-1.4 -watchdog > i6300esb -watchdog-action reset [...] > > I g

Re: [Qemu-devel] [PATCH 09/30] memory: do not duplicate memory_region_destructor_none

2013-05-21 Thread Peter Maydell
On 21 May 2013 11:57, Paolo Bonzini wrote: > Signed-off-by: Paolo Bonzini Reviewed-by: Peter Maydell -- PMM

Re: [Qemu-devel] [PATCH 10/30] memory: make memory_global_sync_dirty_bitmap take an AddressSpace

2013-05-21 Thread Peter Maydell
On 21 May 2013 11:57, Paolo Bonzini wrote: > Suggested-by: Peter Maydell > Signed-off-by: Paolo Bonzini Reviewed-by: Peter Maydell -- PMM

Re: [Qemu-devel] [PATCH 11/30] memory: fix address space initialization/destruction

2013-05-21 Thread Peter Maydell
On 21 May 2013 11:57, Paolo Bonzini wrote: > From: Avi Kivity > > A couple of fields were left uninitialized. This was not observed earlier > because all address spaces were statically allocated. Also free allocation > for those fields. > > Signed-off-by: Avi Kivity > Signed-off-by: Paolo Bonz

[Qemu-devel] [PATCH 12/30] s390x: reduce TARGET_PHYS_ADDR_SPACE_BITS to 62

2013-05-21 Thread Paolo Bonzini
With the next patch, the memory API will complain if the TARGET_PHYS_ADDR_SPACE_BITS gets dangerously close to an overflow. s390x can handle up to 64 bit of physical address space from its page tables, but we never use that much. Just decrease the value. Cc: Alexander Graf Signed-off-by: Paolo

Re: [Qemu-devel] [PATCH 13/30] memory: limit sections in the radix tree to the actual address space size

2013-05-21 Thread Peter Maydell
On 21 May 2013 11:57, Paolo Bonzini wrote: > From: Avi Kivity > > The radix tree is statically sized to fit TARGET_PHYS_ADDR_SPACE_BITS. > If a larger memory region is registered, it will overflow. > > Fix by limiting any section in the radix tree to the supported size. > > This problem was not o

Re: [Qemu-devel] [PATCH 14/30] memory: create FlatView for new address spaces

2013-05-21 Thread Peter Maydell
On 21 May 2013 11:57, Paolo Bonzini wrote: > Even a new address space might have a non-empty FlatView. In order > to initialize it properly, address_space_init should (a) call > memory_region_transaction_commit after the address space is inserted > into the list; (b) force memory_region_transacti

Re: [Qemu-devel] [PATCH v1 1/1] qom/object: Don't poll cast cache for NULL objects

2013-05-21 Thread Anthony Liguori
peter.crosthwa...@xilinx.com writes: > From: Peter Crosthwaite > > object_dymamic_cast_assert used to be tolerant of NULL objects and not > assert. Its clear from the implementation that this is the expected > behavior. > > The preceding check of the cast cache dereferences obj however causing >

Re: [Qemu-devel] IDE disk FLUSH take more than 30 secs, the SUSE guest reports "lost interrupt and the file system becomes read-only"

2013-05-21 Thread Gonglei (Arei)
Hi, Andreas > -Original Message- > From: Andreas Färber [mailto:afaer...@suse.de] > Sent: Tuesday, May 21, 2013 7:50 PM > To: Gonglei (Arei) > Cc: kw...@redhat.com; qemu-devel@nongnu.org; Wangzhenguo; Luonengjun; > Huangweidong (Hardware); Bo Yang > Subject: Re: [Qemu-devel] IDE disk FLUSH

Re: [Qemu-devel] [qemu-devel][libvirt] Default machine type setting for ppc64

2013-05-21 Thread Anthony Liguori
"Daniel P. Berrange" writes: > On Tue, May 21, 2013 at 07:55:27PM +1000, Paul Mackerras wrote: >> On Tue, May 21, 2013 at 09:39:53AM +0100, Daniel P. Berrange wrote: >> > QEMU has the notion of a default machine for each target, and that is >> > what libvirt uses if the user hasn't specified a ma

Re: [Qemu-devel] [PATCH] Fix mask of pte index

2013-05-21 Thread Qiao Nuohan
On 05/21/2013 07:32 PM, Andreas Färber wrote: This patch is missing a Signed-off-by! Ha! I forgot it. If you can reply with one, I'd fix up the subject for you (it should indicate where you are fixing it) and queue it together with my CPU'ish memory_mapping refactorings, if no one objects.

Re: [Qemu-devel] [qemu-devel][libvirt] Default machine type setting for ppc64

2013-05-21 Thread Peter Maydell
On 21 May 2013 13:04, Anthony Liguori wrote: > We've talked in the past about having an accelerator specific machine > default. I think this is a perfectly reasonable thing to do and would > solve the problem for ARM and for PPC. For ARM I would prefer not to have a default at all, and make the

Re: [Qemu-devel] KVM call agenda for 2013-05-21

2013-05-21 Thread Anthony Liguori
"Michael S. Tsirkin" writes: > On Mon, May 20, 2013 at 12:57:47PM +0200, Juan Quintela wrote: >> >> Hi >> >> Please, send any topic that you are interested in covering. >> >> Thanks, Juan. > > Generating acpi tables. > > Cc'd a bunch of people who might be interested in this topic. Unfortunat

Re: [Qemu-devel] (Another) 1.4.1 -> 1.5.0 migration failure

2013-05-21 Thread Nicholas Thomas
On Tue, 2013-05-21 at 13:55 +0200, Andreas Färber wrote: > Hi, > > Am 21.05.2013 13:33, schrieb Nicholas Thomas: > > Migrating from: > > > > /opt/qemu-1.4.1/bin/qemu-system-x86_64 -M pc -watchdog i6300esb > > -watchdog-action reset [...] > > > > to: > > > > /opt/qemu-1.5.0/bin/qemu-system-x86_

Re: [Qemu-devel] [PATCH 1/2] chardev: Make the name of memory device consistent

2013-05-21 Thread Lei Li
On 05/21/2013 06:34 PM, Paolo Bonzini wrote: Il 21/05/2013 12:27, Lei Li ha scritto: Now we have memory char device, but the backend name of it is a little confusion. We actually register it by 'memory', but the description in qemu-option, the name of open functions and the new api backend calle

Re: [Qemu-devel] [PATCH 0/2] block: fix spurious DEVICE_TRAY_MOVED events on shutdown

2013-05-21 Thread Luiz Capitulino
On Fri, 17 May 2013 16:23:51 +0200 Pavel Hrdina wrote: > On 25.4.2013 16:31, Luiz Capitulino wrote: > > On Thu, 25 Apr 2013 16:29:45 +0200 > > Stefan Hajnoczi wrote: > > > >> On Thu, Apr 25, 2013 at 09:51:47AM -0400, Luiz Capitulino wrote: > >>> On Mon, 22 Apr 2013 15:53:43 +0200 > >>> Stefan Ha

Re: [Qemu-devel] KVM call agenda for 2013-05-21

2013-05-21 Thread Michael S. Tsirkin
On Tue, May 21, 2013 at 07:18:58AM -0500, Anthony Liguori wrote: > "Michael S. Tsirkin" writes: > > > On Mon, May 20, 2013 at 12:57:47PM +0200, Juan Quintela wrote: > >> > >> Hi > >> > >> Please, send any topic that you are interested in covering. > >> > >> Thanks, Juan. > > > > Generating acp

Re: [Qemu-devel] [PATCH 1/2] chardev: Make the name of memory device consistent

2013-05-21 Thread Eric Blake
On 05/21/2013 04:27 AM, Lei Li wrote: > Now we have memory char device, but the backend name of it > is a little confusion. We actually register it by 'memory', but > the description in qemu-option, the name of open functions > and the new api backend called it 'ringbuf'. It should keep > consisten

[Qemu-devel] [PATCH v2] tests: set MALLOC_PERTURB_ to expose memory bugs

2013-05-21 Thread Stefan Hajnoczi
glibc wipes malloc(3) memory when the MALLOC_PERTURB_ environment variable is set. The value of the environment variable determines the bit pattern used to wipe memory. For more information, see http://udrepper.livejournal.com/11429.html. Set MALLOC_PERTURB_ for gtester and qemu-iotests. Note w

[Qemu-devel] [PATCH scsi] scsi-generic: fix sign extension of READ CAPACITY(10) data

2013-05-21 Thread Paolo Bonzini
Issuing the READ CAPACITY(10) command in the guest will cause QEMU to update its knowledge of the maximum accessible LBA in the disk. The recorded maximum LBA will be wrong if the disk is bigger than 1TB, because ldl_be_p returns a signed int. When this is fixed, a latent bug will be unmasked. If

Re: [Qemu-devel] [PATCH 1/2] chardev: Make the name of memory device consistent

2013-05-21 Thread Paolo Bonzini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Il 21/05/2013 14:38, Eric Blake ha scritto: > On 05/21/2013 04:27 AM, Lei Li wrote: >> Now we have memory char device, but the backend name of it is a >> little confusion. We actually register it by 'memory', but the >> description in qemu-option, the

Re: [Qemu-devel] [PATCH v2] tests: set MALLOC_PERTURB_ to expose memory bugs

2013-05-21 Thread Eric Blake
On 05/21/2013 06:43 AM, Stefan Hajnoczi wrote: > glibc wipes malloc(3) memory when the MALLOC_PERTURB_ environment > variable is set. The value of the environment variable determines the > bit pattern used to wipe memory. For more information, see > http://udrepper.livejournal.com/11429.html. >

Re: [Qemu-devel] [PATCH 0/2] block: fix spurious DEVICE_TRAY_MOVED events on shutdown

2013-05-21 Thread Pavel Hrdina
On 21.5.2013 14:26, Luiz Capitulino wrote: On Fri, 17 May 2013 16:23:51 +0200 Pavel Hrdina wrote: On 25.4.2013 16:31, Luiz Capitulino wrote: On Thu, 25 Apr 2013 16:29:45 +0200 Stefan Hajnoczi wrote: On Thu, Apr 25, 2013 at 09:51:47AM -0400, Luiz Capitulino wrote: On Mon, 22 Apr 2013 15:53

  1   2   3   >