Re: [Qemu-devel] qemu 1.6.1

2013-10-26 Thread Paolo Bonzini
Il 26/10/2013 11:51, Stefan Weil ha scritto: > Am 24.10.2013 23:47, schrieb Paolo Bonzini: >> Il 24/10/2013 17:37, Stefan Weil ha scritto: >>> Yes, that works, too. It also fixes the problem with the assertion >>> (tested with Wine). >>> >>> No, we cannot remove from_, because the same interface is

[Qemu-devel] [PATCH] hw/9pfs: fix P9_STATS_GEN handling

2013-10-26 Thread Kirill A. Shutemov
From: "Kirill A. Shutemov" Currently we have few issues with P9_STATS_GEN: - We don't try to read st_gen anything except files or directories, but still set P9_STATS_GEN bit in st_result_mask. It may mislead client: we present garbage as valid st_gen. - If we failed to get valid st_gen

[Qemu-devel] [PATCH] hw/9pfs: fix P9_STATS_GEN handling

2013-10-26 Thread Kirill A. Shutemov
Currently we have few issues with P9_STATS_GEN: - We don't try to read st_gen anything except files or directories, but still set P9_STATS_GEN bit in st_result_mask. It may mislead client: we present garbage as valid st_gen. - If we failed to get valid st_gen with ENOTTY, we ignore error,

[Qemu-devel] [PATCH v2] rdma: rename 'x-rdma' => 'rdma'

2013-10-26 Thread mrhines
From: "Michael R. Hines" As far as we can tell, all known bugs have been fixed: 1. Parallel RDMA migrations are working 2. IPv6 migration is working 3. Libvirt patches are ready 4. virt-test is working Signed-off-by: Michael R. Hines --- docs/rdma.txt| 24 ++-- migra

Re: [Qemu-devel] fix clearing i8259 IRQ lines (Was: Should the i8259 devices remain no-user?)

2013-10-26 Thread Matthew Ogilvie
On Wed, Oct 16, 2013 at 06:23:11PM +0200, Paolo Bonzini wrote: > Il 16/10/2013 18:21, BALATON Zoltan ha scritto: > > A bit off topic but this reminded me of these patches: > > > > http://patchwork.ozlabs.org/patch/206753/ > > http://patchwork.ozlabs.org/patch/208252/ > > > > which never got merged.

Re: [Qemu-devel] About VM fork in QEMU

2013-10-26 Thread Xinyang Ge
> External snapshots (via the blockdev-snapshot-sync QMP command) can be > taken in a matter of milliseconds if you only care about disk state. > Furthermore, if you want to take a snapshot of both memory and disk > state, such that the clone can be resumed from the same time, you can do > that wit

Re: [Qemu-devel] About VM fork in QEMU

2013-10-26 Thread Xinyang Ge
> External snapshots (via the blockdev-snapshot-sync QMP command) can be > taken in a matter of milliseconds if you only care about disk state. > Furthermore, if you want to take a snapshot of both memory and disk > state, such that the clone can be resumed from the same time, you can do > that wit

Re: [Qemu-devel] [PATCH] rdma: rename 'x-rdma' => 'rdma'

2013-10-26 Thread Paolo Bonzini
Il 26/10/2013 10:45, Michael R. Hines ha scritto: >> Sure, x-rdma => rdma *is* stable. I'm not sure about x-rdma-pin-all though. > > Use of this capability (which is disabled by default) is actually > "safer" than migration without it - you would get an early warning well > in advance of actually

Re: [Qemu-devel] [PATCHv1 0/4] Timers: add timer debugging through -timer-debug-log

2013-10-26 Thread Paolo Bonzini
Il 26/10/2013 10:24, Alex Bligh ha scritto: > How do timer_file and timer_line get filled in? If those still > need to be put in through code changes, how about taking > patch 1 of the series (that fills in timer->dbg with this > information), save for the unnecessary additions to the structure, >

Re: [Qemu-devel] Bug#727756: qemu: Broken -smb with latest SAMBA package. (Unsupported security=share option)

2013-10-26 Thread Michael Büsch
On Sat, 26 Oct 2013 13:19:29 +0400 Michael Tokarev wrote: > Thank you for the report and the patch Michael. Are you sure the result > is equivalent? Well, I am far from being an SMB expert. So I can't really say whether this is equivalent. I also posted this patch to the qemu-devel list, but di

[Qemu-devel] [PATCH] block: Don't copy backing file name on error

2013-10-26 Thread Max Reitz
bdrv_open_backing_file() tries to copy the backing file name using pstrcpy directly after calling bdrv_open() to open the backing file without checking whether that was actually successful. If it was not, ps->backing_hd->file will probably be NULL and qemu will crash. Fix this by moving pstrcpy af

Re: [Qemu-devel] [PATCH] Fix COR by disabling BDRV_O_COPY_ON_READ before opening the backing_file.

2013-10-26 Thread Max Reitz
Am 25.10.2013 02:15, schrieb Thibaut LAURENT: > Since commit 0ebd24e0a203cf2852c310b59fbe050190dc6c8c, > bdrv_open_common will throw an error when trying to open a file > read-only with the BDRV_O_COPY_ON_READ flag set. > Although BDRV_O_RDWR is unset for the backing files, > BDRV_O_COPY_ON_READ is

Re: [Qemu-devel] [PATCH v5 0/8] Add metadata overlap checks

2013-10-26 Thread Max Reitz
Am 26.10.2013 15:03, schrieb Max Reitz: > Am 20.09.2013 12:32, schrieb Stefan Hajnoczi: >> On Thu, Sep 19, 2013 at 05:07:56PM +0200, Max Reitz wrote: >>> As far as I understand, the I/O speed (the duration of an I/O >>> operation) should be pretty much the same for all scenarios, >>> however, the l

Re: [Qemu-devel] [PATCH v5 0/8] Add metadata overlap checks

2013-10-26 Thread Max Reitz
Am 20.09.2013 12:32, schrieb Stefan Hajnoczi: > On Thu, Sep 19, 2013 at 05:07:56PM +0200, Max Reitz wrote: >> As far as I understand, the I/O speed (the duration of an I/O >> operation) should be pretty much the same for all scenarios, >> however, the latency is the value in question (since the ove

Re: [Qemu-devel] qemu 1.6.1

2013-10-26 Thread Stefan Weil
Am 24.10.2013 23:47, schrieb Paolo Bonzini: > Il 24/10/2013 17:37, Stefan Weil ha scritto: >> Yes, that works, too. It also fixes the problem with the assertion >> (tested with Wine). >> >> No, we cannot remove from_, because the same interface is also used >> for Linux and other hosts which don't

[Qemu-devel] [PULL 6/7] misc: Spelling and grammar fixes in comments

2013-10-26 Thread Michael Tokarev
From: Stefan Weil * it's -> its * grammar fix in ui/vnc-enc-zywrle.h Signed-off-by: Stefan Weil Reviewed-by: Don Koch Reviewed-by: Peter Maydell Signed-off-by: Michael Tokarev --- qapi-schema.json|2 +- slirp/if.c |2 +- ui/vnc-enc-zywrle.h |2 +- 3 files changed, 3

[Qemu-devel] [PULL 5/7] docs/ccid.txt: fix the typo

2013-10-26 Thread Michael Tokarev
From: WengFan Signed-off-by: WengFan Reviewed-by: Eric Blake Signed-off-by: Michael Tokarev --- docs/ccid.txt |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/ccid.txt b/docs/ccid.txt index 8bbaa94..83c174d 100644 --- a/docs/ccid.txt +++ b/docs/ccid.txt @@ -52,7 +52

[Qemu-devel] [PULL 7/7] audio/mixeng_template.h: fix inline declaration

2013-10-26 Thread Michael Tokarev
From: Alex Bligh Fix error: ‘inline’ is not at beginning of declaration [-Werror=old-style-declaration] Signed-off-by: Alex Bligh Signed-off-by: Michael Tokarev --- audio/mixeng_template.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/audio/mixeng_template.h b/aud

[Qemu-devel] [PULL 4/7] qapi: fix documentation example

2013-10-26 Thread Michael Tokarev
From: Eric Blake The QMP wire format uses "", not '', around strings. * docs/qapi-code-gen.txt: Fix typo. Signed-off-by: Eric Blake Signed-off-by: Michael Tokarev --- docs/qapi-code-gen.txt |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/qapi-code-gen.txt b/docs/q

[Qemu-devel] [PULL 0/7 for-1.7] Trivial patches for 2013-10-26

2013-10-26 Thread Michael Tokarev
This is a 2-week pull request, I haven't asked for pull a week ago since we had not much stuff happening on "trivial front". Please consider pulling. Thanks, /mjt The following changes since commit fc8ead74674b7129e8f31c2595c76658e5622197: Merge remote-tracking branch 'qemu-kvm/uq/master' in

[Qemu-devel] [PULL 2/7] misc: New spelling fixes in comments

2013-10-26 Thread Michael Tokarev
From: Stefan Weil compatiblity -> compatibility continously -> continuously existance -> existence usefull -> useful shoudl -> should Signed-off-by: Stefan Weil Signed-off-by: Michael Tokarev --- block/iscsi.c|2 +- hw/ppc/spapr.c |2 +- target-alpha/translate.c

[Qemu-devel] [PULL 3/7] .gitignore: ignore qmp-commands.txt

2013-10-26 Thread Michael Tokarev
From: Fam Zheng This file is moved out from QMP/ to BUILD dir, change the ignore file too. Signed-off-by: Fam Zheng Reviewed-by: Luiz Capitulino Signed-off-by: Michael Tokarev --- .gitignore |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.gitignore b/.gitignore index

[Qemu-devel] [PULL 1/7] configure: create fsdev/ directory

2013-10-26 Thread Michael Tokarev
In some cases when building with parallelism (make -jN), build fails because the directory where output files are supposed to be does not exist. In particular, when make decides to build virtfs-proxy-helper.1 before other files in fsdev/, build will fail with the following error: perl -Ww -- BUIL

Re: [Qemu-devel] Bug#727756: qemu: Broken -smb with latest SAMBA package. (Unsupported security=share option)

2013-10-26 Thread Michael Tokarev
26.10.2013 13:07, Michael Büsch wrote: Package: qemu Version: 1.6.0+dfsg-2 Severity: normal Tags: patch The smb.conf automatically generated by qemu's -smb option fails on current samba, because smbd rejects the security=share option with the following warning: WARNING: Ignoring invalid va

Re: [Qemu-devel] [Qemu-trivial] [PATCH] audio/mixeng_template.h: fix inline declaration

2013-10-26 Thread Michael Tokarev
22.10.2013 13:26, Alex Bligh пишет: ping I think this is should go to qemu-trivial --On 6 August 2013 19:45:13 +0100 Alex Bligh wrote: Fix error: ‘inline’ is not at beginning of declaration [-Werror=old-style-declaration] Thanks, applied to the trivial patches queue. /mjt

Re: [Qemu-devel] [Qemu-trivial] [PATCH v3] misc: Spelling and grammar fixes in comments

2013-10-26 Thread Michael Tokarev
11.10.2013 23:34, Stefan Weil wrote: * it's -> its * grammar fix in ui/vnc-enc-zywrle.h Thanks, applied to the trivial patches queue. /mjt

Re: [Qemu-devel] [PATCH 1/1] docs/ccid.txt: fix the typo

2013-10-26 Thread Michael Tokarev
25.10.2013 07:36, Weng Fan wrote: From: WengFan Date: Wed, 25 Oct 2013 11:18:22 -0400 Subject: [PATCH 1/1] fix the typo Signed-off-by: WengFan --- qemu-master/docs/ccid.txt |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) I had to fix the filename (qemu git tree does not have

Re: [Qemu-devel] [Qemu-ppc] [PULL 00/29] ppc patch queue 2013-10-25

2013-10-26 Thread Mark Cave-Ayland
On 26/10/13 05:43, Alexander Graf wrote: Did you get my repost of the PPC PCI configuration space patch to qemu-devel here: http://lists.gnu.org/archive/html/qemu-devel/2013-10/msg01491.html? Or should that go via someone else's tree? Thanks a lot for the reminder. There is absolutely nothib

Re: [Qemu-devel] [PATCH 0/2] sun4m: Implement Sun CG3 framebuffer for QEMU

2013-10-26 Thread Mark Cave-Ayland
On 15/10/13 21:19, Mark Cave-Ayland wrote: This patchset follows on from the previous "Add FCode ROM for TCX framebuffer" patch and provides QEMU with an implementation of the Sun CG3 8-bit framebuffer. It is based upon Bob Breuer's original work which has been rebased onto git master, and is no

Re: [Qemu-devel] [PATCH] rdma: rename 'x-rdma' => 'rdma'

2013-10-26 Thread Michael R. Hines
On 10/25/2013 11:14 AM, Paolo Bonzini wrote: Il 25/10/2013 16:03, Michael R. Hines ha scritto: Well, I tried posting libvirt support with this naming scheme, but they didn't accepted. Their reason (Daniel, I think) is valid: experimental implies that it shouldn't be exposed in the management so

[Qemu-devel] Atmel AT91RM920 (ARM920) support and workload estimate

2013-10-26 Thread Alexandre Nabais
Hi, I've got a linux source code running on a real target based on AT91RM9200. I would like to make it working on qemu for demonstration and possible evolutions. I've read on a forum that this system-on-chip is not supported by QEMU. Is it correct ? Is it a hard work to make the porting ? Filip Na

Re: [Qemu-devel] [PATCH v4] integrator: fix Linux boot failure by emulating dbg region

2013-10-26 Thread Alex Bennée
peter.mayd...@linaro.org writes: > On 22 October 2013 15:16, wrote: >> +typedef struct { >> +SysBusDevice parent_obj; >> + >> +MemoryRegion iomem; >> + >> +uint32_t alpha; >> +uint32_t leds; >> +uint32_t switches; >> +} IntegratorDebugState > > You forgot to remove these unu

Re: [Qemu-devel] [PATCHv1 0/4] Timers: add timer debugging through -timer-debug-log

2013-10-26 Thread Alex Bligh
Paolo, On 26 Oct 2013, at 08:18, Paolo Bonzini wrote: > The simple trace backend includes a nanosecond value that is the same as > rt_clock, so you can correlate the last timer_mod with the next > iteration in timerlist_run_timers (I would put the tracepoint inside the > loop, just before the cal

Re: [Qemu-devel] [PATCHv1 0/4] Timers: add timer debugging through -timer-debug-log

2013-10-26 Thread Paolo Bonzini
Il 26/10/2013 06:52, Alex Bligh ha scritto: > > On 26 Oct 2013, at 00:00, Paolo Bonzini wrote: > >> his is a bug in the distro, if it is Linux. There is no reason not to >> enable the stap trace format when running on Linux (Fedora does for >> other packages than QEMU, too---most notably glib an