Hi, Is there a document describe vm migration with nbd?
I can't now find it on google, please help me, thanks!
Hello,
sorry for high-jacking this thread, but I have a questing regarding VHD
(without the X): Xen-4.3 switched to upstream QEMUs support for VHD. The
dropped their own VHD implementation in blktap2, which supported backing
chains, which QEMU currently doesn't. The Xen implementation
(xen/too
Hi all,
the below patch is needed to compile qemu trunk on FreeBSD with gcc48,
clang will fail ;). Host x84_64-freebsd.
Installation will fail due to hardcoded /usr/bin/python in
scripts/acpi_extract.py
Thanks,
Andreas
Signed-off-by: Andreas Tobler
[neon:src/qemu/qemu-master] andreast% gi
Normally CPUState::cpu_index is used to pick the right CPU for various
operations. However default consecutive numbering does not always work
for POWERPC.
For example, on POWER7 (which supports 4 threads per core),
"-smp 8,threads=4" should create CPUs with indexes 0,1,2,3,4,5,6,7 and
"-smp 8,thre
于 2013/10/11 10:33, Wenchao Xia 写道:
> This series allow user to read internal snapshot's contents without qemu-img
> convert.
>
> V2:
>Address Stefan's comments:
>02: add 'fall through' comments in the case statement.
>03: add doc about the difference of internal snapshot and backing c
于 2013/10/15 5:52, Wenchao Xia 写道:
> V2:
>1: all fail case will goto fail section.
>2: add the goto code.
> v3:
>Address Stefan's comments:
>2: don't goto fail after allocation failure.
>3: use sn->l1size correctly in qcow2_free_cluster().
>4-7: add test case to verify the e
On 17 October 2013 17:48, Peter Maydell wrote:
> Add 'virt' platform support corresponding to arch/arm/mach-virt
> in the Linux kernel tree. This has no platform-specific code but
> can use any device whose kernel driver is is able to work purely
> from a device tree node. We use this to instantia
On 11.10.2013, at 05:57, Tom Musta wrote:
> This patch adds the Load VSX Scalar Doubleowrd Indexed (lxsdx)
> instruction.
>
> The lower 8 bytes of the target register are undefined; this
> implementation leaves those bytes unaltered.
>
> Signed-off-by: Tom Musta
The diff seems to be broken.
On Thu, Oct 31, 2013 at 03:18:41PM +0100, Andreas Färber wrote:
> Hi,
>
> Am 31.10.2013 15:02, schrieb Edgar E. Iglesias:
> > On Fri, Oct 25, 2013 at 07:07:23PM +0100, Peter Maydell wrote:
> >> The following changes since commit
> >> fc8ead74674b7129e8f31c2595c76658e5622197:
> >>
> >> Merge rem
On Tue, 2013-10-22 at 06:03 +, Liu, Jinsong wrote:
> Ian Campbell wrote:
> > On Mon, 2013-09-09 at 03:29 +, Liu, Jinsong wrote:
> >> From 18344216b432648605726b137b348f28ef64a4ef Mon Sep 17 00:00:00
> >> 2001
> >> From: Liu Jinsong
> >> Date: Fri, 23 Aug 2013 23:30:23 +0800
> >> Subject:
Il 31/10/2013 21:50, Anthony Liguori ha scritto:
> /x86_64/ide/bmdma/setup: OK
> /x86_64/ide/bmdma/simple_rw: OK
> /x86_64/ide/bmdma/short_prdt:OK
> /x86_64/ide/bmdma/
I will test this with 10.8.3
Since the move away from the global areg0, we're no longer globally
reserving areg0. Which means our use of R7 clobbers a call-saved
register. Shift areg0 into the windowed registers. Indeed, choose
the incoming parameter register that it comes to us by.
This requires moving the register holdin
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 24
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c
index 985e213..b62337e 100644
--- a/tcg/ia64/tcg-target.c
+++ b/tcg/ia64/tcg-target.c
@@ -1563,7
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 35 ---
1 file changed, 16 insertions(+), 19 deletions(-)
diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c
index e45b571..c40e32f 100644
--- a/tcg/ia64/tcg-target.c
+++ b/tcg/ia64/tcg-target.c
There was a misconception that a stop bit is required between a compare
and the branch that uses the predicate set by the compare. This lead to
the usage of an extra bundle in which to perform the compare. The extra
bundle left room for constants to be loaded for use with the compare insn.
If we
On Thu, Oct 31, 2013 at 3:37 PM, Andreas Färber wrote:
> Hello Anthony,
>
> This is my updated QOM devices patch queue. Please pull or provide more
> details
> of what is not working in your build environment.
Andreas,
If you cannot be bothered to adequately test your pull requests, then
don't
This problem also affects qemu-system-i386.
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1246890
Title:
AC97 sound card crashes QEMU
Status in QEMU:
New
Bug description:
The AC97 sound card
Public bug reported:
The AC97 sound card does not work. It stops QEMU on startup. The cause
appears to be some kind of deadlock.
Steps to reproduce:
Just add -soundhw ac97 to QEMU's arguments. Example: qemu-system-ppc -soundhw
ac97
The example above is all it takes to reproduce the problem.
Th
Saves one bundle for the common case of exit_tb 0.
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 12 +---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c
index a9dd153..6483fa0 100644
--- a/tcg/ia64/tcg-target.c
+
On Thu, Oct 31, 2013 at 4:48 PM, Kevin Wolf wrote:
> The following changes since commit fc8ead74674b7129e8f31c2595c76658e5622197:
>
> Merge remote-tracking branch 'qemu-kvm/uq/master' into staging (2013-10-18
> 10:03:24 -0700)
>
> are available in the git repository at:
make check is failing f
We say we support python 2.4, but python 2.4.3 does not
support the "expr if test else expr" syntax used here.
This allows QEMU to compile on RHEL 5.3, the last release for ia64.
Signed-off-by: Richard Henderson
---
scripts/qapi-visit.py | 17 ++---
1 file changed, 14 insertions(+),
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 176 --
1 file changed, 114 insertions(+), 62 deletions(-)
diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c
index c881a67..a9dd153 100644
--- a/tcg/ia64/tcg-target.c
+++ b/tcg/i
This sequencing requires 5 stop bits instead of 6, and has room left
over to pre-load the tlb addend, and bswap data prior to being stored.
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 77 ---
1 file changed, 54 insertions(+), 23 de
The only differences were in the bswap insns emitted.
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 60 ---
1 file changed, 23 insertions(+), 37 deletions(-)
diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c
index b62337e..
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 63 +++
1 file changed, 28 insertions(+), 35 deletions(-)
diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c
index ca38608..9ed587a 100644
--- a/tcg/ia64/tcg-target.c
+++ b/tcg/ia
Being able to "extend" from 64-bits (with a mov) simplifies
a few places where the conditional breaks the train of thought.
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 54 +++
1 file changed, 24 insertions(+), 30 deletions(-)
diff
We can subtract from more small constants that just 0 with one insn,
and we can add the negative for most small constants.
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 25 +++--
1 file changed, 23 insertions(+), 2 deletions(-)
diff --git a/tcg/ia64/tcg-target
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 32
1 file changed, 16 insertions(+), 16 deletions(-)
diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c
index c40e32f..c134034 100644
--- a/tcg/ia64/tcg-target.c
+++ b/tcg/ia64/tcg-target.c
@@
Still inline, but updated to the new routines. Always use the LE
helpers, reusing the bswap between the fast and slot paths.
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 140 --
1 file changed, 79 insertions(+), 61 deletions(-)
di
Avoids a wasted cycle loading up small constants.
Simplify the code assuming the tcg optimizer is going to work
and don't expect the first operand of the add to be constant.
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 20
1 file changed, 16 insertions(+), 4
When performing an operation with two input registers, we'd leave
the stop bit (and thus an extra cycle) that's only needed when one
or the other input is a constant.
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 17 ++---
1 file changed, 6 insertions(+), 11 deletions(
We can and/or/xor/andcm small constants, saving one cycle.
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 64 +++
1 file changed, 34 insertions(+), 30 deletions(-)
diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c
index d7bc
Saving at least two cycles per store, and cleaning up the code.
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 96 +--
1 file changed, 32 insertions(+), 64 deletions(-)
diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c
index
There's no need to go through the full opcode-to-insn function call
to generate nops. This makes the source a bit more readable.
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 251 +-
1 file changed, 127 insertions(+), 124 deletions(
This is an update and rebase of a patch set I sent at the
beginning of September. If I get no comments on this round
I'll just send a pull request next week.
r~
Richard Henderson (20):
tcg-ia64: Use TCGMemOp within qemu_ldst routines
tcg-ia64: Use shortcuts for nop insns
tcg-ia64: Handle
Using only indirect calls results in 3 bundles (one to load the
descriptor address), and 4 stop bits. By looking through the
descriptor to the constants, we can perform the call with 2
bundles and only 1 stop bit.
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 38 +
Signed-off-by: Richard Henderson
---
tcg/ia64/tcg-target.c | 171 ++
1 file changed, 90 insertions(+), 81 deletions(-)
diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c
index 0656d39..e09ad24 100644
--- a/tcg/ia64/tcg-target.c
+++ b/tcg/ia
Am 31.10.2013 20:41, schrieb Jan Kiszka:
> On 2013-10-31 20:31, Stefan Weil wrote:
>> Reloading of local variables after sigsetjmp is only needed for some
>> buggy compilers.
>>
>> The code which should reload these variables causes compiler warnings
>> with gcc 4.7 when compiler optimizations are
On 2013-10-31 20:31, Stefan Weil wrote:
> Reloading of local variables after sigsetjmp is only needed for some
> buggy compilers.
>
> The code which should reload these variables causes compiler warnings
> with gcc 4.7 when compiler optimizations are enabled:
>
> cpu-exec.c:204:15: error:
> vari
Reloading of local variables after sigsetjmp is only needed for some
buggy compilers.
The code which should reload these variables causes compiler warnings
with gcc 4.7 when compiler optimizations are enabled:
cpu-exec.c:204:15: error:
variable ‘cpu’ might be clobbered by ‘longjmp’ or ‘vfork’ [-
On Wed, 30 Oct 2013 17:10:29 +0800
"Zhou Yuan" wrote:
> From: zhouy
> To: qemu-devel@nongnu.org
> Date: Wed, 30 Oct 2013 12:37:51 -0400
> Subject: [PATCH] fix the monitor_user_noop function for hmp-commands calling
> user_print conveniently
> If a new hmp command added,just using the function
On Thu, Oct 31, 2013 at 6:14 PM, Andreas Färber wrote:
> Am 31.10.2013 16:16, schrieb Peter Maydell:
>> On 31 October 2013 14:36, Andreas Färber wrote:
>>> Am 31.10.2013 15:31, schrieb Peter Maydell:
On 31 October 2013 14:18, Andreas Färber wrote:
> Peter, since I had picked up the firs
On Thu, Oct 31, 2013 at 5:52 PM, Andreas Färber wrote:
> Am 31.10.2013 16:04, schrieb Anthony Liguori:
>> On Thu, Oct 31, 2013 at 3:45 PM, Andreas Färber wrote:
>>> Am 31.10.2013 15:39, schrieb Anthony Liguori:
On Thu, Oct 31, 2013 at 3:36 PM, Andreas Färber wrote:
> Am 31.10.2013 15:31
Il 31/10/2013 18:18, Michael S. Tsirkin ha scritto:
> On Thu, Oct 31, 2013 at 06:10:36PM +0100, Paolo Bonzini wrote:
>> Il 31/10/2013 18:01, Michael S. Tsirkin ha scritto:
Yes, that's what my patch (posted the link before) does:
-{ RUN_STATE_GUEST_PANICKED, RUN_STATE_PAUSED },
>>
On 10/31/2013 09:57 AM, Jeff Cody wrote:
> There were still instances of $TEST_IMG not being properly quoted.
> This was in the usage of a string built up for a 'for' loop; modify
> the loop so we can quote $TEST_IMG properly.
>
> Signed-off-by: Jeff Cody
> ---
> tests/qemu-iotests/019 | 6 +++--
On 31 October 2013 17:27, Andreas Färber wrote:
> Am 31.10.2013 18:18, schrieb Peter Maydell:
>> On 31 October 2013 17:14, Andreas Färber wrote:
>>> Am 31.10.2013 16:16, schrieb Peter Maydell:
On 31 October 2013 14:36, Andreas Färber wrote:
> It was clearly stated that a Reviewed-by
>>>
Am 31.10.2013 18:18, schrieb Peter Maydell:
> On 31 October 2013 17:14, Andreas Färber wrote:
>> Am 31.10.2013 16:16, schrieb Peter Maydell:
>>> On 31 October 2013 14:36, Andreas Färber wrote:
It was clearly stated that a Reviewed-by
needs to be explicitly sent as reply but that "looks
From: "Jason J. Herne"
s390_new_cpu is created to encapsulate the creation of a new QOM S390CPU
object given a cpuid and a model string.
All actual cpu initialization code is moved from boot time specific functions to
s390_cpu_initfn (qom init routine) or to s390_new_cpu. This is done to allow u
From: "Jason J. Herne"
Introduces global access to storage key data so we can set it for each cpu in
the S390 cpu initialization routine.
Signed-off-by: Jason J. Herne
---
hw/s390x/s390-virtio-ccw.c | 3 +--
hw/s390x/s390-virtio.c | 6 +++---
hw/s390x/s390-virtio.h | 2 +-
target-s390x
From: "Jason J. Herne"
Modify s390_cpu_addr2state to allow fetching state information for cpu addresses
above smp_cpus. Hotplug requires this capability.
Also add s390_cpu_set_state function to allow modification of ipi_state entries
during hotplug.
Signed-off-by: Jason J. Herne
---
hw/s390x
From: "Jason J. Herne"
Implement hot_add_cpu for S390 to allow hot plugging of cpus.
Signed-off-by: Jason J. Herne
---
hw/s390x/s390-virtio-ccw.c | 1 +
target-s390x/cpu.c | 29 +
target-s390x/cpu.h | 2 ++
3 files changed, 32 insertions(+)
diff -
From: "Jason J. Herne"
Implement the CPU data in SCLP "Read SCP Info". And implement "Read CPU Info"
SCLP command. This data will be used by the guest to get information about hot
plugged cpus.
Signed-off-by: Jason J. Herne
---
hw/s390x/sclp.c | 51
From: "Jason J. Herne"
Add HMP cpu-add wrapper to allow cpu hot plugging via monitor.
Signed-off-by: Jason J. Herne
---
hmp-commands.hx | 13 +
hmp.c | 10 ++
hmp.h | 1 +
3 files changed, 24 insertions(+)
diff --git a/hmp-commands.hx b/hmp-commands.hx
From: "Jason J. Herne"
Add an sclp event for "cpu was hot plugged". This allows Qemu to deliver an
SCLP interrupt to the guest stating that the requested cpu hotplug was
completed.
Signed-off-by: Jason J. Herne
---
hw/s390x/Makefile.objs| 1 +
hw/s390x/event-facility.c |
From: "Jason J. Herne"
Define new SCLP codes to improve code readability.
Signed-off-by: Jason J. Herne
---
hw/s390x/sclp.c | 2 +-
include/hw/s390x/sclp.h | 8
2 files changed, 9 insertions(+), 1 deletion(-)
diff --git a/hw/s390x/sclp.c b/hw/s390x/sclp.c
index 86d6ae0..cb035
From: "Jason J. Herne"
Latest code for cpu Hotplug on S390 architecture.
The last version can be found here:
http://lists.gnu.org/archive/html/qemu-devel/2013-08/msg00076.html
There is also a patch in this series to add cpu-add to the Qemu monitor
interface.
Hotplugged cpus are created in the
On 31 October 2013 17:14, Andreas Färber wrote:
> Am 31.10.2013 16:16, schrieb Peter Maydell:
>> On 31 October 2013 14:36, Andreas Färber wrote:
>>> It was clearly stated that a Reviewed-by
>>> needs to be explicitly sent as reply but that "looks okay" should in
>>> exactly such a case where send
On Thu, Oct 31, 2013 at 06:10:36PM +0100, Paolo Bonzini wrote:
> Il 31/10/2013 18:01, Michael S. Tsirkin ha scritto:
> > > Yes, that's what my patch (posted the link before) does:
> > >
> > > -{ RUN_STATE_GUEST_PANICKED, RUN_STATE_PAUSED },
> > > +{ RUN_STATE_GUEST_PANICKED, RUN_STATE_RUNN
On 31 October 2013 17:02, Peter Maydell wrote:
> I told you to queue the patches because you needed them as prereqs
> and I was expecting the timing to work out such that you'd get a pullreq
> taken so they'd get upstream while I was away.
> Since it didn't and I wanted them in 1.7 I put them in m
Am 31.10.2013 16:16, schrieb Peter Maydell:
> On 31 October 2013 14:36, Andreas Färber wrote:
>> Am 31.10.2013 15:31, schrieb Peter Maydell:
>>> On 31 October 2013 14:18, Andreas Färber wrote:
Peter, since I had picked up the first two patches into my still pending
qom-next pull, as per
Am 31.10.2013 17:54, schrieb Anthony Liguori:
> On Thu, Oct 31, 2013 at 5:52 PM, Andreas Färber wrote:
>> The new rule you made was: no patch without Reviewed-by.
>> Peter sending that PULL and Edgar merging it both violate that rule.
>
> I never said anything like that.
I could've sworn you did
Il 31/10/2013 18:01, Michael S. Tsirkin ha scritto:
> > Yes, that's what my patch (posted the link before) does:
> >
> > -{ RUN_STATE_GUEST_PANICKED, RUN_STATE_PAUSED },
> > +{ RUN_STATE_GUEST_PANICKED, RUN_STATE_RUNNING },
> > { RUN_STATE_GUEST_PANICKED, RUN_STATE_FINISH_MIGRATE },
>
Il 31/10/2013 18:00, Michael S. Tsirkin ha scritto:
> Interesting. Why do we have
> -{ RUN_STATE_INTERNAL_ERROR, RUN_STATE_PAUSED },
> then?
It's only for non-resumable states (such as pvpanic right now).
It's used here:
if (qemu_reset_requested()) {
pause_all_vcpus();
c
The main goal is that qemu doesn't crash.
Signed-off-by: Kevin Wolf
---
tests/ide-test.c | 26 ++
1 file changed, 26 insertions(+)
diff --git a/tests/ide-test.c b/tests/ide-test.c
index 7307f1d..bc824a8 100644
--- a/tests/ide-test.c
+++ b/tests/ide-test.c
@@ -81,6 +81,7
Il 31/10/2013 16:45, Michael S. Tsirkin ha scritto:
> On Thu, Oct 31, 2013 at 04:26:13PM +0100, Paolo Bonzini wrote:
>> Il 31/10/2013 16:09, Michael S. Tsirkin ha scritto:
>>> On Thu, Oct 31, 2013 at 03:56:42PM +0100, Paolo Bonzini wrote:
Il 31/10/2013 15:52, Michael S. Tsirkin ha scritto:
>>>
On 31 October 2013 16:52, Andreas Färber wrote:
> I *asked* about what to do with my QEMU CPU patches that only get a
> "looks okay" and got only positive answers for whether that should be an
> Acked-by and no objection, including none from you.
I agreed with that because IMHO you may treat a "l
On 10/31/2013 09:57 AM, Jeff Cody wrote:
> There were still a couple of instances of unquoted usage of
> $TEST_IMG and $TEST_IMG.orig. Quoted these so they will not fail
> on pathnames with spaces in them.
>
> Signed-off-by: Jeff Cody
> ---
> tests/qemu-iotests/051 | 8
> 1 file change
From: MORITA Kazutaka
This patch tries to cancel aio requests in pending queue and failed
queue. When the sheepdog driver cannot cancel the requests, it waits
for them to be completed.
Signed-off-by: MORITA Kazutaka
Tested-by: Liu Yuan
Reviewed-by: Liu Yuan
Signed-off-by: Kevin Wolf
---
bl
On Thu, Oct 31, 2013 at 05:38:40PM +0100, Paolo Bonzini wrote:
> Il 31/10/2013 17:26, Michael S. Tsirkin ha scritto:
> > On Thu, Oct 31, 2013 at 05:17:24PM +0100, Paolo Bonzini wrote:
> >> Il 31/10/2013 17:14, Michael S. Tsirkin ha scritto:
> PANICKED->DEBUG was added by commit bc7d0e667. Tha
On Thu, Oct 31, 2013 at 05:52:11PM +0100, Paolo Bonzini wrote:
> Il 31/10/2013 17:48, Michael S. Tsirkin ha scritto:
> > But code duplication is bad.
>
> So should we make a table of IO_ERROR-like states to avoid code
> duplication? You have to draw a line somewhere...
>
> > I think IO error for
From: Max Reitz
Add a test for saving a VM state from a qcow2 image and loading it back
(with having restarted qemu in between); this should work without any
problems.
Signed-off-by: Max Reitz
Signed-off-by: Kevin Wolf
---
tests/qemu-iotests/068 | 65 ++
On Thu, Oct 31, 2013 at 5:52 PM, Andreas Färber wrote:
> Am 31.10.2013 16:04, schrieb Anthony Liguori:
>> On Thu, Oct 31, 2013 at 3:45 PM, Andreas Färber wrote:
>>> Am 31.10.2013 15:39, schrieb Anthony Liguori:
On Thu, Oct 31, 2013 at 3:36 PM, Andreas Färber wrote:
> Am 31.10.2013 15:31
Il 31/10/2013 17:48, Michael S. Tsirkin ha scritto:
> But code duplication is bad.
So should we make a table of IO_ERROR-like states to avoid code
duplication? You have to draw a line somewhere...
> I think IO error for example
> is broken in that you can't pause but can run then pause.
> Seems
Am 31.10.2013 16:04, schrieb Anthony Liguori:
> On Thu, Oct 31, 2013 at 3:45 PM, Andreas Färber wrote:
>> Am 31.10.2013 15:39, schrieb Anthony Liguori:
>>> On Thu, Oct 31, 2013 at 3:36 PM, Andreas Färber wrote:
Am 31.10.2013 15:31, schrieb Peter Maydell:
> On 31 October 2013 14:18, Andre
From: Alexander Graf
When AHCI executes an asynchronous IDE command, it checked DRDY without
checking either DRQ or BSY. This sometimes caused interrupt to be sent
before command is actually completed.
This resulted in a race condition: if guest then managed to access the
device before command
On Thu, Oct 31, 2013 at 05:38:40PM +0100, Paolo Bonzini wrote:
> Il 31/10/2013 17:26, Michael S. Tsirkin ha scritto:
> > On Thu, Oct 31, 2013 at 05:17:24PM +0100, Paolo Bonzini wrote:
> >> Il 31/10/2013 17:14, Michael S. Tsirkin ha scritto:
> PANICKED->DEBUG was added by commit bc7d0e667. Tha
The usage of $TEST_IMG was not properly quoted everywhere in
common.pattern.
Signed-off-by: Jeff Cody
---
tests/qemu-iotests/common.pattern | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/tests/qemu-iotests/common.pattern
b/tests/qemu-iotests/common.pattern
inde
Il 31/10/2013 17:26, Michael S. Tsirkin ha scritto:
> On Thu, Oct 31, 2013 at 05:17:24PM +0100, Paolo Bonzini wrote:
>> Il 31/10/2013 17:14, Michael S. Tsirkin ha scritto:
PANICKED->DEBUG was added by commit bc7d0e667. That commit can be
reverted if the panicked state is removed from run
From: MORITA Kazutaka
After reconnection happens, all the inflight requests are moved to the
failed request list. As a result, sd_co_rw_vector() can send another
create request before resend_aioreq() resends a create request from
the failed list.
This patch adds a helper function check_simultan
On 10/31/2013 09:57 AM, Jeff Cody wrote:
> When creating images with backing files in the test, the backing
> file argument was not quoted properly. This caused the test to fail
> when run from a pathname with a space. Pass the backing argument in
> with the -b option to _make_test_img, so it can
From: MORITA Kazutaka
This helper function behaves similarly to co_sleep_ns(), but the
sleeping coroutine will be resumed when using qemu_aio_wait().
Signed-off-by: MORITA Kazutaka
Tested-by: Liu Yuan
Reviewed-by: Liu Yuan
Signed-off-by: Kevin Wolf
---
include/block/coroutine.h | 9 +++
On 10/31/2013 09:57 AM, Jeff Cody wrote:
> Test 039 had $TEST_IMG with duplicate double quotes - remove duplicate.
>
> Signed-off-by: Jeff Cody
> ---
> tests/qemu-iotests/039 | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Eric Blake
--
Eric Blake eblake redhat com
On Thu, Oct 31, 2013 at 05:17:24PM +0100, Paolo Bonzini wrote:
> Il 31/10/2013 17:14, Michael S. Tsirkin ha scritto:
> >> PANICKED->DEBUG was added by commit bc7d0e667. That commit can be
> >> reverted if the panicked state is removed from runstate_needs_reset.
> >
> > Okay so let's drop the code
On 10/31/2013 09:57 AM, Jeff Cody wrote:
> $TEST_IMG.base is used unquoted. Add quotes so that pathnames with
> spaces are supported.
>
> Signed-off-by: Jeff Cody
> ---
> tests/qemu-iotests/017 | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Eric Blake
--
Eric Blake eb
On 10/31/2013 09:57 AM, Jeff Cody wrote:
> The usage of $TEST_IMG was not properly quoted everywhere in
> common.pattern.
>
> Signed-off-by: Jeff Cody
> ---
> tests/qemu-iotests/common.pattern | 12 ++--
> 1 file changed, 6 insertions(+), 6 deletions(-)
Reviewed-by: Eric Blake
--
Eri
On Thu, Oct 31, 2013 at 05:17:24PM +0100, Paolo Bonzini wrote:
> Il 31/10/2013 17:14, Michael S. Tsirkin ha scritto:
> >> PANICKED->DEBUG was added by commit bc7d0e667. That commit can be
> >> reverted if the panicked state is removed from runstate_needs_reset.
> >
> > Okay so let's drop the code
On 10/31/2013 09:57 AM, Jeff Cody wrote:
> _make_test_img() currently works with spaced pathnames only when not
> specifying a backing file. This fixes it so that the backing file
> argument is properly quoted.
>
> Signed-off-by: Jeff Cody
> ---
> tests/qemu-iotests/common.rc | 13 +++--
Am 31.10.2013 um 17:13 hat Eric Blake geschrieben:
> On 10/31/2013 09:48 AM, Kevin Wolf wrote:
> > From: Fam Zheng
> >
> > Implement .bdrv_get_specific_info to return the extent information.
> >
> > Signed-off-by: Fam Zheng
> > Signed-off-by: Kevin Wolf
> > ---
>
> > +++ b/qapi-schema.json
>
Il 31/10/2013 17:14, Michael S. Tsirkin ha scritto:
>> PANICKED->DEBUG was added by commit bc7d0e667. That commit can be
>> reverted if the panicked state is removed from runstate_needs_reset.
>
> Okay so let's drop the code duplication and explicitly make
> them the same?
>
> Signed-off-by: Mic
On 10/31/2013 09:48 AM, Kevin Wolf wrote:
> From: Fam Zheng
>
> Implement .bdrv_get_specific_info to return the extent information.
>
> Signed-off-by: Fam Zheng
> Signed-off-by: Kevin Wolf
> ---
> +++ b/qapi-schema.json
> @@ -225,6 +225,27 @@
>} }
>
> ##
> +# @ImageInfoSpecificVmdk:
>
On 10/30/2013 08:06 PM, Fam Zheng wrote:
> Implement .bdrv_get_specific_info to return the extent information.
>
> Signed-off-by: Fam Zheng
>
> ---
> v5: Changed 3 lines:
> - Save cid in vmdk_open. (Kevin)
> - Report parent_cid in vmdk_get_specific_info.
> - Rename create_type to cre
On Thu, Oct 31, 2013 at 04:56:07PM +0100, Paolo Bonzini wrote:
> Il 31/10/2013 16:45, Michael S. Tsirkin ha scritto:
> > On Thu, Oct 31, 2013 at 04:26:13PM +0100, Paolo Bonzini wrote:
> >> Il 31/10/2013 16:09, Michael S. Tsirkin ha scritto:
> >>> On Thu, Oct 31, 2013 at 03:56:42PM +0100, Paolo Bonz
From: Fam Zheng
Signed-off-by: Fam Zheng
Reviewed-by: Eric Blake
Signed-off-by: Kevin Wolf
---
qapi-schema.json | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/qapi-schema.json b/qapi-schema.json
index 60f3fd1..add97e2 100644
--- a/qapi-schema.json
+++ b/qapi-schema.jso
From: Fam Zheng
Case 030 occasionally fails because of block job compltes too fast to be
captured by script, and 'unexpected qmp event' of job completion causes
the test failure.
Simply fill in some data to the test image to make this false alarm less
likely to happen.
(For other benefits to pr
From: MORITA Kazutaka
This prepares for using resend_aioreq() after reconnecting to the
sheepdog server.
Signed-off-by: MORITA Kazutaka
Tested-by: Liu Yuan
Reviewed-by: Liu Yuan
Signed-off-by: Kevin Wolf
---
block/sheepdog.c | 33 +++--
1 file changed, 19 inserti
When creating images with backing files in the test, the backing
file argument was not quoted properly. This caused the test to fail
when run from a pathname with a space. Pass the backing argument in
with the -b option to _make_test_img, so it can be properly quoted.
Signed-off-by: Jeff Cody
-
From: Max Reitz
Add a test case for trying to open an image file where it is impossible
to open its backing file (in this case, because it was deleted). When
doing this, qemu (or qemu-io in this case) should not crash but rather
print an appropriate error message.
Signed-off-by: Max Reitz
Revie
There were still a couple of instances of unquoted usage of
$TEST_IMG and $TEST_IMG.orig. Quoted these so they will not fail
on pathnames with spaces in them.
Signed-off-by: Jeff Cody
---
tests/qemu-iotests/051 | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tests/qe
From: Fam Zheng
There's a same common function in iotests.py
Signed-off-by: Fam Zheng
Signed-off-by: Kevin Wolf
---
tests/qemu-iotests/040 | 14 ++
1 file changed, 2 insertions(+), 12 deletions(-)
diff --git a/tests/qemu-iotests/040 b/tests/qemu-iotests/040
index aad535a..a2e18c5
1 - 100 of 172 matches
Mail list logo
