Re: [Qemu-devel] Potential to accelerate QEMU for specific architectures

2013-05-27 Thread Paolo Bonzini
Il 26/05/2013 18:35, Lior Vernia ha scritto: > What about no to the first bullet but yes to the second (just x86 on > ARM)? Any room for significant improvement in that case, starting from > the foundations of QEMU? You could write a target-specific translator, yes. But first of all I would answe

Re: [Qemu-devel] [RFC PATCH v3 00/11] qemu-ga: fsfreeze on Windows using VSS

2013-05-27 Thread Tomoki Sekiyama
Hi, Thank you for testing the patch. > {"error": {"class": "GenericError", "desc": "DoSnapshotSet. (Error: > 8004230f) "}} This is VSS_E_UNEXPECTED_PROVIDER_ERROR. I can't say what happens from this, but Detailed error log would be logged in Event Viewer. Could you send me the log in Event

Re: [Qemu-devel] [PATCH 7/8] pseries: savevm support for PAPR virtual SCSI

2013-05-27 Thread Paolo Bonzini
Il 27/05/2013 08:48, Alexey Kardashevskiy ha scritto: >> > >> > This is only true when the rerror and werror options have the values >> > "ignore" or "report". See virtio-scsi for an example of how to save the >> > requests using the save_request and load_request callbacks in SCSIBusInfo. > > Si

Re: [Qemu-devel] [PATCH v4] net: introduce command to query rx-filter information

2013-05-27 Thread Amos Kong
On Fri, May 24, 2013 at 06:26:40AM -0600, Eric Blake wrote: > On 05/24/2013 06:03 AM, Michael S. Tsirkin wrote: > > On Fri, May 24, 2013 at 02:44:06PM +0800, Amos Kong wrote: > >> We want to implement mac programming over macvtap through Libvirt. The > >> related rx-filter information of the nic co

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

2013-05-27 Thread Paolo Bonzini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Il 26/05/2013 11:02, Jan Kiszka ha scritto: >>> >>> Nice patches. Only one thing, how is .impl.unaligned different >>> from the existing .valid.unaligned? > See memory.h: valid controls is an unaligned access traps or gets > processed, impl manages

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

2013-05-27 Thread Jan Kiszka
On 2013-05-27 09:20, Paolo Bonzini wrote: > Il 26/05/2013 11:02, Jan Kiszka ha scritto: Nice patches. Only one thing, how is .impl.unaligned different from the existing .valid.unaligned? >> See memory.h: valid controls is an unaligned access traps or gets >> processed, impl manages

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

2013-05-27 Thread Paolo Bonzini
Il 26/05/2013 23:08, Christian Borntraeger ha scritto: > On 26/05/13 21:07, Paolo Bonzini wrote: >> Il 26/05/2013 16:14, Andreas Färber ha scritto: With the next patch, the memory API will complain if the TARGET_PHYS_ADDR_SPACE_BITS gets dangerously close to an overflow. s390x can h

Re: [Qemu-devel] [PATCH 3/9 v3] Move includes and struct definition to dump.h

2013-05-27 Thread Qiao Nuohan
On 05/22/2013 10:12 PM, Andreas Färber wrote: I fear that this patch is conflicting. Are really all of those headers actually needed for the struct you're moving? In particular I'm worried about cpu.h as well as cpu-all.h and kvm.h with indirect dependencies on cpu.h. Well, I cannot avoid conf

[Qemu-devel] Selecting device variant types based on bdrv size

2013-05-27 Thread Peter Crosthwaite
Hi All, I have a bit of a chicken and egg problem trying to refactor Jans AT24 I2C EEPROM model. I'm trying to migrate static class properties up to the class level rather than down on the device property level (as we did for EHCI in the sysbusification a while back). Problem is the device model h

Re: [Qemu-devel] [PULL 00/15] IOMMU patches for 1.6, part 1

2013-05-27 Thread Paolo Bonzini
Il 24/05/2013 19:02, Paolo Bonzini ha scritto: > Anthony, > > The following changes since commit 9ce0e9275434bacdeba42dd32e0e8269293fe2cf: > > Open up 1.6 development (2013-05-20 10:55:18 -0500) > > are available in the git repository at: > > git://github.com/bonzini/qemu.git iommu-for-anth

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

2013-05-27 Thread Paolo Bonzini
Il 27/05/2013 09:23, Jan Kiszka ha scritto: > On 2013-05-27 09:20, Paolo Bonzini wrote: >> Il 26/05/2013 11:02, Jan Kiszka ha scritto: > > Nice patches. Only one thing, how is .impl.unaligned different > from the existing .valid.unaligned? >>> See memory.h: valid controls is an unalign

Re: [Qemu-devel] [PATCH] rtl8139: flush queued packets when RxBufPtr is written

2013-05-27 Thread Stefan Hajnoczi
On Mon, May 27, 2013 at 08:15:42AM +0200, Peter Lieven wrote: > I ocassionally have seen a probably related problem in the past. It mainly > happend with rtl8139 under > WinXP where we most likely use rtl8139 due to lack of shipped e1000 drivers. > > My question is if you see increasing dropped p

Re: [Qemu-devel] [PATCH v3 2/2] net: introduce command to query rx-filter information

2013-05-27 Thread Stefan Hajnoczi
On Sun, May 26, 2013 at 10:38:14AM +0300, Michael S. Tsirkin wrote: > On Fri, May 24, 2013 at 04:00:42PM -0400, Luiz Capitulino wrote: > > On Fri, 24 May 2013 12:05:12 -0600 > > Eric Blake wrote: > > > > > On 05/24/2013 10:12 AM, Michael S. Tsirkin wrote: > > > > > > > > Event message con

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

2013-05-27 Thread Jan Kiszka
On 2013-05-27 10:19, Paolo Bonzini wrote: > Il 27/05/2013 09:23, Jan Kiszka ha scritto: >> On 2013-05-27 09:20, Paolo Bonzini wrote: >>> Il 26/05/2013 11:02, Jan Kiszka ha scritto: >> >> Nice patches. Only one thing, how is .impl.unaligned different >> from the existing .valid.unaligne

Re: [Qemu-devel] [PATCH 0/7] VNVRAM persistent storage

2013-05-27 Thread Stefan Hajnoczi
On Fri, May 24, 2013 at 11:39:09AM -0400, Corey Bryant wrote: > > > On 05/24/2013 08:36 AM, Stefan Hajnoczi wrote: > >On Fri, May 24, 2013 at 08:13:27AM -0400, Stefan Berger wrote: > >>On 05/24/2013 05:59 AM, Stefan Hajnoczi wrote: > >>>On Thu, May 23, 2013 at 01:44:40PM -0400, Corey Bryant wrote

Re: [Qemu-devel] [PATCH v4] net: introduce command to query rx-filter information

2013-05-27 Thread Amos Kong
On Fri, May 24, 2013 at 06:42:34AM -0600, Eric Blake wrote: > On 05/24/2013 12:44 AM, Amos Kong wrote: > > We want to implement mac programming over macvtap through Libvirt. The > > related rx-filter information of the nic contains main mac, rx-mode > > items. > > > > This patch adds a QMP event t

Re: [Qemu-devel] [PATCH 2/2] rtl8139: flush queued packets when RxBufPtr is written

2013-05-27 Thread Stefan Hajnoczi
On Fri, May 24, 2013 at 05:18:09PM +0200, Paolo Bonzini wrote: > Il 24/05/2013 16:38, Stefan Hajnoczi ha scritto: > > diff --git a/hw/net/rtl8139.c b/hw/net/rtl8139.c > > index 9369507..7993f9f 100644 > > --- a/hw/net/rtl8139.c > > +++ b/hw/net/rtl8139.c > > @@ -2575,6 +2575,9 @@ static void rtl813

Re: [Qemu-devel] [PATCH 2/4] acpi/gpe: expand bits of gpe register

2013-05-27 Thread Gerd Hoffmann
On 05/24/13 04:02, li guang wrote: > 在 2013-05-24五的 10:24 +0900,Isaku Yamahata写道: >> On Wed, May 22, 2013 at 01:37:41PM +0800, li guang wrote: >>> 在 2013-05-22三的 14:28 +0900,Isaku Yamahata写道: Why? And it breaks pointer operation like >>> >>> the fact is I can't guess why gpe->sts is defin

Re: [Qemu-devel] [PATCH 2/4] acpi/gpe: expand bits of gpe register

2013-05-27 Thread Gerd Hoffmann
On 05/24/13 04:02, li guang wrote: > 在 2013-05-24五的 10:24 +0900,Isaku Yamahata写道: >> On Wed, May 22, 2013 at 01:37:41PM +0800, li guang wrote: >>> 在 2013-05-22三的 14:28 +0900,Isaku Yamahata写道: Why? And it breaks pointer operation like >>> >>> the fact is I can't guess why gpe->sts is defin

[Qemu-devel] [Bug 1180924] Re: fails to handle a usb serial port with a specific vendorid

2013-05-27 Thread Rostislav Devyatov
I think the ability to specify a different vendorid + deviceid can be useful. Suppose there is a USB device such that the specifications are open and officially published, but the driver is proprietary. (As far as I know, this is similar to the situation with ATI video cards, but they are not USB d

Re: [Qemu-devel] [PATCH 2/4] tcg/aarch64: implement new TCG target for aarch64

2013-05-27 Thread Claudio Fontana
(removing Paolo from CC as agreed with him) On 24.05.2013 10:51, Claudio Fontana wrote: > On 23.05.2013 18:39, Peter Maydell wrote: >> On 23 May 2013 09:18, Claudio Fontana wrote: >>> >>> add preliminary support for TCG target aarch64. >> >> Richard's handling the technical bits of the review, so

Re: [Qemu-devel] [PATCH 0/4] debugcon: fix some bugs when DEBUG_DEBUGCON

2013-05-27 Thread Gerd Hoffmann
On 05/24/13 04:47, liguang wrote: > when enable DEBUG_DEBUGCON, there are some message > printing bugs, so fix them. I'd suggest to either simply remove the debug printfs or turn them into tracepoints. cheers, Gerd

Re: [Qemu-devel] snabbswitch integration with QEMU for userspace ethernet I/O

2013-05-27 Thread Stefan Hajnoczi
On Sun, May 26, 2013 at 11:32:49AM +0200, Luke Gorrie wrote: > Stefan put us onto the highly promising track of vhost/virtio. We have > implemented this between Snabb Switch and the Linux kernel, but not > directly between Snabb Switch and QEMU guests. The "roadblock" we have hit > is embarrasingly

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

2013-05-27 Thread Amos Kong
On Fri, May 24, 2013 at 08:51:36AM -0400, Luiz Capitulino wrote: > On Fri, 24 May 2013 15:10:16 +0300 > "Michael S. Tsirkin" wrote: > > > On Thu, May 23, 2013 at 01:26:33PM -0400, Luiz Capitulino wrote: > > > On Thu, 23 May 2013 20:18:34 +0300 > > > "Michael S. Tsirkin" wrote: > > > > > > > On

Re: [Qemu-devel] [PATCH 2/4] tcg/aarch64: implement new TCG target for aarch64

2013-05-27 Thread Laurent Desnogues
Hi, basically pointing out what I pointed for v1. On Thu, May 23, 2013 at 10:18 AM, Claudio Fontana wrote: > > add preliminary support for TCG target aarch64. > > Signed-off-by: Claudio Fontana > --- > include/exec/exec-all.h |5 +- > tcg/aarch64/tcg-target.c | 1185 > +++

Re: [Qemu-devel] [PATCH 2/4] tcg/aarch64: implement new TCG target for aarch64

2013-05-27 Thread Claudio Fontana
Hello, On 27.05.2013 11:47, Laurent Desnogues wrote: > Hi, > > basically pointing out what I pointed for v1. > > On Thu, May 23, 2013 at 10:18 AM, Claudio Fontana > wrote: >> >> add preliminary support for TCG target aarch64. >> >> Signed-off-by: Claudio Fontana >> --- >> include/exec/exec-al

Re: [Qemu-devel] [PATCH] rtl8139: flush queued packets when RxBufPtr is written

2013-05-27 Thread Peter Lieven
On 27.05.2013 10:32, Stefan Hajnoczi wrote: On Mon, May 27, 2013 at 08:15:42AM +0200, Peter Lieven wrote: I ocassionally have seen a probably related problem in the past. It mainly happend with rtl8139 under WinXP where we most likely use rtl8139 due to lack of shipped e1000 drivers. My questi

Re: [Qemu-devel] [PATCH v2 00/11] qemu: use virtio linux headers in portable code

2013-05-27 Thread Peter Maydell
On 26 May 2013 19:10, Michael S. Tsirkin wrote: > Ouch. Forgot to git-add them. Thanks. > > I'll send a fixed version - > could you please try this patch on top? With this extra patch MacOSX compiles. thanks -- PMM

Re: [Qemu-devel] [PATCH] qemu: fix out of tree cross compile

2013-05-27 Thread Peter Maydell
On 27 May 2013 07:13, Alexey Kardashevskiy wrote: > QEMU uses headers from source directory, however the "asm" symlink was > created in the build directory what broke KVM stuff compilation. > > Signed-off-by: Alexey Kardashevskiy > --- > configure |2 +- > 1 file changed, 1 insertion(+), 1 d

Re: [Qemu-devel] [PATCH 2/4] tcg/aarch64: implement new TCG target for aarch64

2013-05-27 Thread Laurent Desnogues
On Mon, May 27, 2013 at 12:13 PM, Claudio Fontana wrote: > Hello, > > On 27.05.2013 11:47, Laurent Desnogues wrote: >> Hi, >> >> basically pointing out what I pointed for v1. >> >> On Thu, May 23, 2013 at 10:18 AM, Claudio Fontana >> wrote: >>> >>> add preliminary support for TCG target aarch64.

[Qemu-devel] [PATCH 0/2] Trivial fixes for migration

2013-05-27 Thread Lei Li
This small series improves the document migration.txt and remove a duplicated setting of bandwidth_limit. Lei Li (2): docs: Fix typo and update file in migration migration: Remove duplicate bandwidth_limit set docs/migration.txt | 17 + migration.c|1 - 2 files ch

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

2013-05-27 Thread Peter Maydell
On 27 May 2013 09:19, Paolo Bonzini wrote: > Il 27/05/2013 09:23, Jan Kiszka ha scritto: >> Err, why? Will we emulate IOMMUs for TCG differently? > > Because IOMMUs should never be added to address_space_memory. > > TCG should only encounter an IOMMU during device emulation (DMA), not > because of

[Qemu-devel] [PATCH 1/2] docs: Fix typo and update file in migration

2013-05-27 Thread Lei Li
This patch fix some typo and update the file that already moved. Signed-off-by: Lei Li --- docs/migration.txt | 17 + 1 files changed, 9 insertions(+), 8 deletions(-) diff --git a/docs/migration.txt b/docs/migration.txt index 0719a55..0e0a1d4 100644 --- a/docs/migration.txt ++

[Qemu-devel] [PATCH 2/2] migration: Remove duplicate bandwidth_limit set

2013-05-27 Thread Lei Li
bandwidth_limit is double set in migrate_init(), remove one. Signed-off-by: Lei Li --- migration.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/migration.c b/migration.c index bfbc345..058f9e6 100644 --- a/migration.c +++ b/migration.c @@ -349,7 +349,6 @@ static Migra

Re: [Qemu-devel] [PATCH] linux-user: Fix typo in comment

2013-05-27 Thread Lei Li
On 05/20/2013 07:10 PM, Stefan Weil wrote: Am 20.05.2013 11:20, schrieb Lei Li: Signed-off-by: Lei Li --- linux-user/syscall.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/linux-user/syscall.c b/linux-user/syscall.c index 30e93bc..72c6866 100644 --- a/linux-user/

Re: [Qemu-devel] [PATCH 2/4] tcg/aarch64: implement new TCG target for aarch64

2013-05-27 Thread Peter Maydell
On 27 May 2013 10:10, Claudio Fontana wrote: > Would it be acceptable to put a comment at the beginning of the function > describing ext use, to avoiding a series of /* fall through */ comments? The 'fall through' comments are for the benefit of automatic checking and linting tools, not just huma

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

2013-05-27 Thread Paolo Bonzini
Il 27/05/2013 12:33, Peter Maydell ha scritto: > On 27 May 2013 09:19, Paolo Bonzini wrote: >> Il 27/05/2013 09:23, Jan Kiszka ha scritto: >>> Err, why? Will we emulate IOMMUs for TCG differently? >> >> Because IOMMUs should never be added to address_space_memory. >> >> TCG should only encounter a

Re: [Qemu-devel] [PATCH v2] linux-user: improve target_to_host_sock_type conversion

2013-05-27 Thread Petar Jovanovic
Can anyone take a look at this and commit it if there are no other change requests? Thank you. From: Petar Jovanovic Sent: Wednesday, May 08, 2013 1:16 AM To: riku.voi...@linaro.org; qemu-devel@nongnu.org Cc: Aurelien Jarno; Petar Jovanovic; blauwir...@gma

Re: [Qemu-devel] [PATCH] kvm: exclude ioeventfd from counting kvm_io_range limit

2013-05-27 Thread Stefan Hajnoczi
On Sat, May 25, 2013 at 06:44:15AM +0800, Amos Kong wrote: > We can easily reach the 1000 limit by start VM with a couple > hundred I/O devices (multifunction=on). The hardcode limit > already been adjusted 3 times (6 ~ 200 ~ 300 ~ 1000). > > In userspace, we already have maximum file descriptor t

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

2013-05-27 Thread Peter Maydell
On 27 May 2013 11:45, Paolo Bonzini wrote: > It is possible to do it. > As in other cases, I prefer no code to untested code. The design is > sane (it isn't mine, so I can say it :)) and we know it can be done. Agreed; we're a long way from being able to make use of this. > Almost, I have an a

[Qemu-devel] [PATCH 2/4] build: do not use TARGET_ARCH

2013-05-27 Thread Paolo Bonzini
TARGET_ARCH is generally wrong to use, there are better variables provided in config-target.mak. The right one is usually TARGET_NAME (previously TARGET_ARCH2), but for bsd-user we can also use TARGET_ABI_DIR for consistency with linux-user. Signed-off-by: Paolo Bonzini --- Makefile.target

[Qemu-devel] [PATCH 0/4] configure: TARGET_ARCH cleanups

2013-05-27 Thread Paolo Bonzini
We have three variables currently in config-target.h: - TARGET_ARCH is used to create a unique per-arch symbol, used in #ifdefs. It is also used as a string through config-target.h, but this is almost always wrong. - TARGET_ARCH2 is the name of the executable (minus the qemu-/qemu-system- p

[Qemu-devel] [PATCH 1/4] build: rename TARGET_ARCH2 to TARGET_NAME

2013-05-27 Thread Paolo Bonzini
Do not introduce any new use yet. Signed-off-by: Paolo Bonzini --- Makefile.target | 6 +++--- configure | 42 +- scripts/create_config | 2 +- 3 files changed, 25 insertions(+), 25 deletions(-) diff --git a/Makefile.target b/Makefile.

[Qemu-devel] [PATCH 4/4] build: drop TARGET_TYPE

2013-05-27 Thread Paolo Bonzini
Just use the TARGET_NAME free string. Signed-off-by: Paolo Bonzini --- arch_init.c | 2 +- configure| 1 - qapi-schema.json | 18 +- 3 files changed, 2 insertions(+), 19 deletions(-) diff --git a/arch_init.c b/arch_init.c index 22fbe96..ebea903 100644 --- a/arch_i

Re: [Qemu-devel] [PATCH 0/2] Trivial fixes for migration

2013-05-27 Thread Michael Tokarev
27.05.2013 14:33, Lei Li wrote: > This small series improves the document migration.txt > and remove a duplicated setting of bandwidth_limit. FWIW, please don't CC me directly, I'm subscribed to qemu-trivial@ obviously (being the one behind trivial- patches currently). Thanks, /mjt

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-27 Thread Stefan Hajnoczi
On Sun, May 26, 2013 at 07:08:02PM +0200, Andreas Färber wrote: > Am 21.05.2013 09:12, schrieb Gonglei (Arei): > > Through analysis, I found that because the system call the fdatasync > > command in the Qemu over 30s, > > after the Guest's kernel thread detects the io transferation is timeout, >

[Qemu-devel] [PATCH 3/4] main: use TARGET_ARCH only for the target-specific #define

2013-05-27 Thread Paolo Bonzini
Everything else needs to match the executable name, which is TARGET_NAME. Before: $ sh4eb-linux-user/qemu-sh4eb --help usage: qemu-sh4 [options] program [arguments...] Linux CPU emulator (compiled for sh4 emulation) After: $ sh4eb-linux-user/qemu-sh4eb --help usage: qemu-sh4eb

Re: [Qemu-devel] [PATCH 2/4] tcg/aarch64: implement new TCG target for aarch64

2013-05-27 Thread Claudio Fontana
On 24.05.2013 19:02, Richard Henderson wrote: > On 05/24/2013 01:53 AM, Claudio Fontana wrote: >>> No real need to special case zero; it's just an extra test slowing down the >>> compiler. >> >> Yes, we need to handle the special case zero. >> Otherwise no instruction at all would be emitted for va

Re: [Qemu-devel] [PATCH 0/4] add ACPI Embedded Controller

2013-05-27 Thread Igor Mammedov
On Mon, 27 May 2013 09:22:59 +0800 li guang wrote: > 在 2013-05-26日的 19:51 -0500,Anthony Liguori写道: > > li guang writes: > > > > > 在 2013-05-24五的 14:45 +0300,Michael S. Tsirkin写道: > > >> On Wed, May 22, 2013 at 11:46:33AM +0800, liguang wrote: > > >> > These patches try to add ACPI Embedded Cont

Re: [Qemu-devel] [RFC PATCH v3 02/11] Fix errors and warnings while compiling with c++ compilier

2013-05-27 Thread Stefan Hajnoczi
On Fri, May 24, 2013 at 05:25:21PM +0200, Markus Armbruster wrote: > Tomoki Sekiyama writes: > > > On 5/24/13 4:52 , "Stefan Hajnoczi" wrote: > > > >>On Thu, May 23, 2013 at 06:34:43PM +, Tomoki Sekiyama wrote: > >>> On 5/23/13 8:12 , "Stefan Hajnoczi" wrote: > >>> > >>> >On Tue, May 21, 2

Re: [Qemu-devel] [RFC PATCH 1/2] mem: make phys_section and phys_map_nodes prepared for RCU

2013-05-27 Thread Paolo Bonzini
Il 26/05/2013 15:02, liu ping fan ha scritto: > [...] > +static PhysPageTable *cur_pgtbl; > +static PhysPageTable *next_pgtbl; You shouldn't need cur_pgtbl. Instead, each AddressSpaceDispatch should have a pointer to its own cur_pgtbl. In the commit hook you can then t

Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read

2013-05-27 Thread Richard W.M. Jones
On Mon, May 27, 2013 at 10:25:21AM +0800, Fam Zheng wrote: > On Fri, 05/24 09:07, Richard W.M. Jones wrote: > > On Fri, May 24, 2013 at 01:36:55PM +0800, Fam Zheng wrote: > > > Changes from v5: > > > 05: Rename bs to s for BDRVCURLState. > > > 06: Use int64_t for offsets. > > > Fix printf

Re: [Qemu-devel] [RFC PATCH v3 11/11] QMP/qmp.py: set locale for exceptions to display non-ascii messages correctly

2013-05-27 Thread Stefan Hajnoczi
On Fri, May 24, 2013 at 06:55:21PM +, Tomoki Sekiyama wrote: > On 5/23/13 8:30 , "Stefan Hajnoczi" wrote: > >Either or both bugs could be present. Once they are fixed you shouldn't > >see encoding problems. > > > >Stefan > > Anyway, as I understood this patch for qmp.py is not correct way to

Re: [Qemu-devel] [PATCH] target-i386: Disable CPUID_EXT_MONITOR when KVM is enabled

2013-05-27 Thread Eduardo Habkost
On Sat, May 25, 2013 at 08:25:49AM +0200, Paolo Bonzini wrote: > Il 25/05/2013 03:21, Bandan Das ha scritto: > > There is one user-visible effect: "-cpu ...,enforce" will stop failing > > because of missing KVM support for CPUID_EXT_MONITOR. But that's exactly > > the point: there's no point in hav

Re: [Qemu-devel] [PATCH] target-i386: Disable CPUID_EXT_MONITOR when KVM is enabled

2013-05-27 Thread Paolo Bonzini
Il 27/05/2013 14:09, Eduardo Habkost ha scritto: > On Sat, May 25, 2013 at 08:25:49AM +0200, Paolo Bonzini wrote: >> Il 25/05/2013 03:21, Bandan Das ha scritto: >>> There is one user-visible effect: "-cpu ...,enforce" will stop failing >>> because of missing KVM support for CPUID_EXT_MONITOR. But t

Re: [Qemu-devel] 'qemu-nbd' explicit flush

2013-05-27 Thread Stefan Hajnoczi
On Sat, May 25, 2013 at 09:42:08AM -0800, Mark Trumpold wrote: > On 5/24/13 1:05 AM, "Stefan Hajnoczi" wrote: > >On Thu, May 23, 2013 at 09:58:31PM +, Mark Trumpold wrote: > >One thing to be careful of is whether these operations are asynchronous. > >The signal is asynchronous, you have no way

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

2013-05-27 Thread Andreas Färber
Am 27.05.2013 09:28, schrieb Paolo Bonzini: > Il 26/05/2013 23:08, Christian Borntraeger ha scritto: >> On 26/05/13 21:07, Paolo Bonzini wrote: >>> Il 26/05/2013 16:14, Andreas Färber ha scritto: > With the next patch, the memory API will complain if the > TARGET_PHYS_ADDR_SPACE_BITS gets d

Re: [Qemu-devel] [Bug 1184089] [NEW] [Feature request] loadvm snapshot as read-only

2013-05-27 Thread Stefan Hajnoczi
On Sat, May 25, 2013 at 08:29:11AM -, Michael Coppola wrote: > There are many ways to take and manage snapshots in QEMU, but one main > feature that's missing is the ability to 'loadvm' a LIVE snapshot and > have all future changes redirected to a temporary file. This would > effectively be co

Re: [Qemu-devel] use O_DIRECT to open disk images for IDE failed under xen-4.1.2 and qemu upstream

2013-05-27 Thread Stefan Hajnoczi
On Sat, May 25, 2013 at 07:32:55AM +, Gonglei (Arei) wrote: > > > > -Original Message- > > From: Stefan Hajnoczi [mailto:stefa...@gmail.com] > > Sent: Friday, May 24, 2013 8:20 PM > > To: Gonglei (Arei) > > Cc: Stefano Stabellini; Hanweidong; Luonengjun; qemu-devel@nongnu.org; > > Wan

[Qemu-devel] SeaBIOS v1.7.2.2 stable release

2013-05-27 Thread Kevin O'Connor
A new stable release of SeaBIOS (version 1.7.2.2) has been tagged. This release contains bug fixes. The release is available via git: git clone git://git.seabios.org/seabios -b 1.7.2-stable -Kevin Asias He (2): virtio-scsi: Pack struct virtio_scsi_{req_cmd,resp_cmd} virtio-scsi: Set

Re: [Qemu-devel] [PATCH 2/2] vfio: hugepage support for vfio_iommu_type1

2013-05-27 Thread Sethi Varun-B16395
> -Original Message- > From: iommu-boun...@lists.linux-foundation.org [mailto:iommu- > boun...@lists.linux-foundation.org] On Behalf Of Alex Williamson > Sent: Friday, May 24, 2013 10:55 PM > To: alex.william...@redhat.com > Cc: io...@lists.linux-foundation.org; chegu_vi...@hp.com; qemu-

Re: [Qemu-devel] [PATCH] target-i386: Disable CPUID_EXT_MONITOR when KVM is enabled

2013-05-27 Thread Eduardo Habkost
On Mon, May 27, 2013 at 02:21:36PM +0200, Paolo Bonzini wrote: > Il 27/05/2013 14:09, Eduardo Habkost ha scritto: > > On Sat, May 25, 2013 at 08:25:49AM +0200, Paolo Bonzini wrote: > >> Il 25/05/2013 03:21, Bandan Das ha scritto: > >>> There is one user-visible effect: "-cpu ...,enforce" will stop

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

2013-05-27 Thread Luiz Capitulino
On Mon, 27 May 2013 17:34:25 +0800 Amos Kong wrote: > On Fri, May 24, 2013 at 08:51:36AM -0400, Luiz Capitulino wrote: > > On Fri, 24 May 2013 15:10:16 +0300 > > "Michael S. Tsirkin" wrote: > > > > > On Thu, May 23, 2013 at 01:26:33PM -0400, Luiz Capitulino wrote: > > > > On Thu, 23 May 2013 20

Re: [Qemu-devel] [SeaBIOS] SeaBIOS v1.7.2.2 stable release

2013-05-27 Thread Michael Tokarev
27.05.2013 16:59, Kevin O'Connor wrote: > A new stable release of SeaBIOS (version 1.7.2.2) has been tagged. > This release contains bug fixes. > > The release is available via git: > git clone git://git.seabios.org/seabios -b 1.7.2-stable Just.. curious -- why there are no tarballs for the dotdo

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

2013-05-27 Thread Paolo Bonzini
Il 27/05/2013 14:52, Andreas Färber ha scritto: >> > Ok, I'll drop this patch and the next one from the pull request. > It has already been merged, that's how I became aware of it: > > http://git.qemu.org/?p=qemu.git;a=commit;h=fd469df97ab4277411ecdd4032a2f045a3a87b2a > > Note that Alex is curren

Re: [Qemu-devel] [PATCH] target-i386: Disable CPUID_EXT_MONITOR when KVM is enabled

2013-05-27 Thread Paolo Bonzini
Il 27/05/2013 15:07, Eduardo Habkost ha scritto: >> Changing TCG to KVM should not change hardware >> > if you use "-cpu ...,enforce", so it is right that it fails when >> > starting with KVM. >> > > Changing between KVM and TCG _does_ change hardware, today (with or > without check/enforce). All

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

2013-05-27 Thread Luiz Capitulino
On Mon, 27 May 2013 09:10:11 -0400 Luiz Capitulino wrote: > > We use the QMP event to notify management about the mac changing. > > > > In this thread, we _wrongly_ considered to use qmp approach to delay > > the event for avoiding the flooding. > > > > eg: monitor_protocol_event_throttle(NIC

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

2013-05-27 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] virtfs fails after migration

2013-05-27 Thread Malte Starostik
Hello, trying to get migration work on a setup with several libvirt based hosts, I'm struggling with those VMs that use virtfs shares. I remember that in previous versions of qemu, the attempt to migrate those lead to chaos and mayhem, while now a *mounted* virtfs simply blocks migration. Fine

Re: [Qemu-devel] [PATCH] target-i386: Disable CPUID_EXT_MONITOR when KVM is enabled

2013-05-27 Thread Eduardo Habkost
On Mon, May 27, 2013 at 03:14:25PM +0200, Paolo Bonzini wrote: > Il 27/05/2013 15:07, Eduardo Habkost ha scritto: > >> Changing TCG to KVM should not change hardware > >> > if you use "-cpu ...,enforce", so it is right that it fails when > >> > starting with KVM. > >> > > > Changing between KVM an

Re: [Qemu-devel] [PATCH 2/2] vfio: hugepage support for vfio_iommu_type1

2013-05-27 Thread Alex Williamson
On Mon, 2013-05-27 at 08:41 +, Sethi Varun-B16395 wrote: > > > -Original Message- > > From: iommu-boun...@lists.linux-foundation.org [mailto:iommu- > > boun...@lists.linux-foundation.org] On Behalf Of Alex Williamson > > Sent: Friday, May 24, 2013 10:55 PM > > To: alex.william...@redha

Re: [Qemu-devel] Selecting device variant types based on bdrv size

2013-05-27 Thread Andreas Färber
Hi, Am 27.05.2013 09:50, schrieb Peter Crosthwaite: > I have a bit of a chicken and egg problem trying to refactor Jans AT24 > I2C EEPROM model. I'm trying to migrate static class properties up to > the class level rather than down on the device property level (as we > did for EHCI in the sysbusif

Re: [Qemu-devel] [PATCH] NVMe: Initial commit

2013-05-27 Thread Kevin Wolf
Am 10.04.2013 um 22:16 hat Keith Busch geschrieben: > Initial commit for emulated nvme pci storage device. Implements the > minimum from the specification to work with existing drivers. > > Cc: Keith Busch > Signed-off-by: Keith Busch > > A lot of people are very interested in this, so I should

Re: [Qemu-devel] [PATCH] rtl8139: flush queued packets when RxBufPtr is written

2013-05-27 Thread Oliver Francke
Well, Am 27.05.2013 um 08:15 schrieb Peter Lieven : > Hi all, > > I ocassionally have seen a probably related problem in the past. It mainly > happend with rtl8139 under > WinXP where we most likely use rtl8139 due to lack of shipped e1000 drivers. > > My question is if you see increasing drop

Re: [Qemu-devel] [PATCH] qemu: fix out of tree cross compile

2013-05-27 Thread Alexey Kardashevskiy
On 05/27/2013 08:24 PM, Peter Maydell wrote: > On 27 May 2013 07:13, Alexey Kardashevskiy wrote: >> QEMU uses headers from source directory, however the "asm" symlink was >> created in the build directory what broke KVM stuff compilation. >> >> Signed-off-by: Alexey Kardashevskiy >> --- >> confi

Re: [Qemu-devel] [Xen-devel] target-i386: Introduce ICC bus/device/bridge

2013-05-27 Thread Igor Mammedov
On Fri, 24 May 2013 08:56:14 -0600 jacek burghardt wrote: > I wonder if anyone has patch that allows to tlak to icc bus introduced in > qemu upstream ? > https://github.com/qemu/qemu/commit/f0513d2c0156799e0c75a108ab9a049eea4f9607 > > icc-bridge will serve as a parent for icc-bus and provide >

[Qemu-devel] [PATCH] qemu: fix out of tree cross compile

2013-05-27 Thread Alexey Kardashevskiy
The symlink to platform linux headers is made in the build tree by the configure script but gcc is not told to look for them there. The patch fixes this. Signed-off-by: Alexey Kardashevskiy --- configure |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure

Re: [Qemu-devel] [PATCH] rtl8139: flush queued packets when RxBufPtr is written

2013-05-27 Thread Peter Lieven
On 27.05.2013 16:07, Oliver Francke wrote: Well, Am 27.05.2013 um 08:15 schrieb Peter Lieven : Hi all, I ocassionally have seen a probably related problem in the past. It mainly happend with rtl8139 under WinXP where we most likely use rtl8139 due to lack of shipped e1000 drivers. My questi

Re: [Qemu-devel] [PATCH] qapi: pad GenericList value fields to 64 bits

2013-05-27 Thread mdroth
On Mon, May 27, 2013 at 06:38:35AM +0200, Stefan Weil wrote: > Am 27.05.2013 05:20, schrieb Michael Roth: > > With the introduction of native list types, we now have types such as > > int64List where the 'value' field is not a pointer, but the actual > > 64-bit value. > > > > On 32-bit architecture

Re: [Qemu-devel] [PATCH v2 00/11] qemu: use virtio linux headers in portable code

2013-05-27 Thread Michael S. Tsirkin
On Sun, May 26, 2013 at 07:55:25PM -0500, Anthony Liguori wrote: > "Michael S. Tsirkin" writes: > > > On Sun, May 26, 2013 at 03:49:53PM -0500, Anthony Liguori wrote: > >> Paolo Bonzini writes: > >> > >> > Il 26/05/2013 22:02, Michael S. Tsirkin ha scritto: > >> >> > My fault. I should have lo

Re: [Qemu-devel] [PATCH] qemu: fix out of tree cross compile

2013-05-27 Thread Paolo Bonzini
Il 27/05/2013 16:12, Alexey Kardashevskiy ha scritto: > The symlink to platform linux headers is made in the build tree by > the configure script but gcc is not told to look for them there. > The patch fixes this. > > Signed-off-by: Alexey Kardashevskiy > --- > configure |2 +- > 1 file chan

Re: [Qemu-devel] [PATCH V3 4/4] block: dump snapshot and image info to specified output

2013-05-27 Thread Luiz Capitulino
On Sat, 25 May 2013 11:09:45 +0800 Wenchao Xia wrote: > bdrv_snapshot_dump() and bdrv_image_info_dump() do not dump to a buffer now, > some internal buffers are still used for format control, which have no > chance to be truncated. As a result, these two functions have no more issue > of truncati

[Qemu-devel] [Bug 1091766] Re: Physical host crash with Mellanox IB PCI passthrough

2013-05-27 Thread Vlastimil Holer
My believe is that this bug can be closed. We have tested QEMU 1.4.x and 1.5.x series (with pci-assign) and at least these works fine. ** Changed in: qemu Status: New => Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to Q

Re: [Qemu-devel] snabbswitch integration with QEMU for userspace ethernet I/O

2013-05-27 Thread Michael S. Tsirkin
On Mon, May 27, 2013 at 11:34:09AM +0200, Stefan Hajnoczi wrote: > On Sun, May 26, 2013 at 11:32:49AM +0200, Luke Gorrie wrote: > > Stefan put us onto the highly promising track of vhost/virtio. We have > > implemented this between Snabb Switch and the Linux kernel, but not > > directly between Sna

Re: [Qemu-devel] [PATCH V3 1/4] block: drop bs_snapshots global variable

2013-05-27 Thread Kevin Wolf
Am 25.05.2013 um 05:09 hat Wenchao Xia geschrieben: > From: Stefan Hajnoczi > > The bs_snapshots global variable points to the BlockDriverState which > will be used to save vmstate. This is really a savevm.c concept but was > moved into block.c:bdrv_snapshots() when it became clear that hotplug

Re: [Qemu-devel] [PATCH v2] monitor: work around delayed CHR_EVENT_OPENED events

2013-05-27 Thread Luiz Capitulino
On Sun, 26 May 2013 10:33:39 -0500 Michael Roth wrote: > In the past, CHR_EVENT_OPENED events were emitted via a pre-expired > QEMUTimer. Due to timers being processing at the tail end of each main > loop iteration, this generally meant that such events would be emitted > within the same main loo

Re: [Qemu-devel] [PATCH V3 1/4] block: drop bs_snapshots global variable

2013-05-27 Thread Kevin Wolf
Am 25.05.2013 um 05:09 hat Wenchao Xia geschrieben: > From: Stefan Hajnoczi > > The bs_snapshots global variable points to the BlockDriverState which > will be used to save vmstate. This is really a savevm.c concept but was > moved into block.c:bdrv_snapshots() when it became clear that hotplug

Re: [Qemu-devel] [PATCH] rtl8139: flush queued packets when RxBufPtr is written

2013-05-27 Thread Stefan Hajnoczi
On Mon, May 27, 2013 at 04:24:59PM +0200, Peter Lieven wrote: > On 27.05.2013 16:07, Oliver Francke wrote: > >Well, > > > >Am 27.05.2013 um 08:15 schrieb Peter Lieven : > > > >>Hi all, > >> > >>I ocassionally have seen a probably related problem in the past. It mainly > >>happend with rtl8139 unde

Re: [Qemu-devel] [PATCH V3 0/4] qapi and snapshot code clean up in block layer

2013-05-27 Thread Kevin Wolf
Am 25.05.2013 um 05:09 hat Wenchao Xia geschrieben: > These patches are the common part of my hmp/qmp block query series and Pavel's > qmp snapshot command converion series. It mainly does following things: > 1 move snapshot related code to block/snapshot.c, qmp and info dumping code to > block/qap

Re: [Qemu-devel] [PATCH V3 4/4] block: dump snapshot and image info to specified output

2013-05-27 Thread Kevin Wolf
Am 27.05.2013 um 17:02 hat Luiz Capitulino geschrieben: > On Sat, 25 May 2013 11:09:45 +0800 > Wenchao Xia wrote: > > > bdrv_snapshot_dump() and bdrv_image_info_dump() do not dump to a buffer now, > > some internal buffers are still used for format control, which have no > > chance to be truncate

Re: [Qemu-devel] snabbswitch integration with QEMU for userspace ethernet I/O

2013-05-27 Thread Paolo Bonzini
Il 27/05/2013 11:34, Stefan Hajnoczi ha scritto: > On Sun, May 26, 2013 at 11:32:49AM +0200, Luke Gorrie wrote: >> Stefan put us onto the highly promising track of vhost/virtio. We have >> implemented this between Snabb Switch and the Linux kernel, but not >> directly between Snabb Switch and QEMU

Re: [Qemu-devel] [PATCH V3 4/4] block: dump snapshot and image info to specified output

2013-05-27 Thread Luiz Capitulino
On Mon, 27 May 2013 17:40:59 +0200 Kevin Wolf wrote: > Am 27.05.2013 um 17:02 hat Luiz Capitulino geschrieben: > > On Sat, 25 May 2013 11:09:45 +0800 > > Wenchao Xia wrote: > > > > > bdrv_snapshot_dump() and bdrv_image_info_dump() do not dump to a buffer > > > now, > > > some internal buffers

Re: [Qemu-devel] [PATCH v2 00/11] qemu: use virtio linux headers in portable code

2013-05-27 Thread Anthony Liguori
"Michael S. Tsirkin" writes: > On Sun, May 26, 2013 at 07:55:25PM -0500, Anthony Liguori wrote: >> "Michael S. Tsirkin" writes: >> >> > On Sun, May 26, 2013 at 03:49:53PM -0500, Anthony Liguori wrote: >> >> Paolo Bonzini writes: >> >> >> >> > Il 26/05/2013 22:02, Michael S. Tsirkin ha scritto

Re: [Qemu-devel] [PATCH v2] monitor: work around delayed CHR_EVENT_OPENED events

2013-05-27 Thread Anthony Liguori
Luiz Capitulino writes: > On Sun, 26 May 2013 10:33:39 -0500 > Michael Roth wrote: > >> In the past, CHR_EVENT_OPENED events were emitted via a pre-expired >> QEMUTimer. Due to timers being processing at the tail end of each main >> loop iteration, this generally meant that such events would be

Re: [Qemu-devel] snabbswitch integration with QEMU for userspace ethernet I/O

2013-05-27 Thread Anthony Liguori
Paolo Bonzini writes: > Il 27/05/2013 11:34, Stefan Hajnoczi ha scritto: >> On Sun, May 26, 2013 at 11:32:49AM +0200, Luke Gorrie wrote: >>> Stefan put us onto the highly promising track of vhost/virtio. We have >>> implemented this between Snabb Switch and the Linux kernel, but not >>> directly

Re: [Qemu-devel] snabbswitch integration with QEMU for userspace ethernet I/O

2013-05-27 Thread Paolo Bonzini
Il 27/05/2013 18:18, Anthony Liguori ha scritto: > Paolo Bonzini writes: > >> Il 27/05/2013 11:34, Stefan Hajnoczi ha scritto: >>> On Sun, May 26, 2013 at 11:32:49AM +0200, Luke Gorrie wrote: Stefan put us onto the highly promising track of vhost/virtio. We have implemented this between

Re: [Qemu-devel] snabbswitch integration with QEMU for userspace ethernet I/O

2013-05-27 Thread Anthony Liguori
Paolo Bonzini writes: > Il 27/05/2013 18:18, Anthony Liguori ha scritto: >> Paolo Bonzini writes: >> >>> Il 27/05/2013 11:34, Stefan Hajnoczi ha scritto: On Sun, May 26, 2013 at 11:32:49AM +0200, Luke Gorrie wrote: > Stefan put us onto the highly promising track of vhost/virtio. We hav

[Qemu-devel] [Bug 1184616] [NEW] undefined reference to `trace_qemu_anon_ram_alloc'

2013-05-27 Thread Nigel Horne
Public bug reported: The latest git version (commit 6a4e17711442849bf2cc731ccddef5a2a2d92d29) fails to compile: ... LINK qemu-ga libqemuutil.a(oslib-posix.o): In function `qemu_anon_ram_alloc': oslib-posix.c:(.text+0x154): undefined reference to `trace_qemu_anon_ram_alloc' libqemuutil.a(oslib-

Re: [Qemu-devel] [Bug 1180970] *** affects all x86_64 soft emulation

2013-05-27 Thread Luiz Capitulino
On Fri, 24 May 2013 23:23:02 +0200 Laszlo Ersek wrote: > --[ proposed fix ]-- > diff --git a/target-i386/translate.c b/target-i386/translate.c > index 0e0356f..4fbd6c0 100644 > --- a/target-i386/translate.c > +++ b/target-i386/translate.c > @@ -

Re: [Qemu-devel] [PATCH 2/4] tcg/aarch64: implement new TCG target for aarch64

2013-05-27 Thread Richard Henderson
On 2013-05-27 02:10, Claudio Fontana wrote: +case INDEX_op_mov_i64: ext = 1; Please don't put code on the same line as a case statement. Also fall-through cases should have an explicit /* fall through */ comment (except in the case where there is no code at all between one case statement an

  1   2   >