Orit Wasserman wrote:
> On 09/21/2012 11:47 AM, Juan Quintela wrote:
>> It always have that type, just change it.
>> -QEMUFile *qemu_fopen_ops_buffered(void *opaque, size_t xfer_limit,
>> +QEMUFile *qemu_fopen_ops_buffered(MigrationState *migration_state,
>> + size
On 09/28/2012 09:33 AM, Juan Quintela wrote:
> Orit Wasserman wrote:
>> On 09/21/2012 11:47 AM, Juan Quintela wrote:
>>> It always have that type, just change it.
>>> -QEMUFile *qemu_fopen_ops_buffered(void *opaque, size_t xfer_limit,
>>> +QEMUFile *qemu_fopen_ops_buffered(MigrationState *migratio
Il 28/09/2012 05:00, Wenchao Xia ha scritto:
>>
>>
>> uint64_t qblock_get_virt_size(QBlockFmtInfo *fmt);
>> QBlockProtInfo *qblock_get_backing_loc(QBlockFmtInfo *fmt);
>> bool qblock_get_encrypt(QBlockFmtInfo *fmt);
>>
>> etc. that include the switch statement.
>>
> I think it is reasonable use g
Il 28/09/2012 05:04, Wenchao Xia ha scritto:
> +$(LIBQBLOCK_TEST_DIR):
> +@echo "Make libqblock test directory"
> +mkdir $(LIBQBLOCK_TEST_DIR)
You can leave the files in tests/ directly, and avoid this as well.
>>>Having a new directory will make clean easier, othe
On 2012-09-28 01:21, Satoru Moriya wrote:
> This is a first time for me to post a patch to qemu-devel.
> If there is something missing/wrong, please let me know.
>
> We have some plans to migrate old enterprise systems which require
> low latency (msec order) to kvm virtualized environment. Usuall
On Thu, Sep 27, 2012 at 05:55:15PM +0200, Paolo Bonzini wrote:
> Il 27/09/2012 16:36, Daniel P. Berrange ha scritto:
> >> > qemu: URI parsing library
> >> >
> >> > From: Paolo Bonzini
> >> >
> >> > Add a new URI parsing library to QEMU. The code has been borrowed from
> >> > libxml2 and libvirt.
Old qemu versions required that 1st s/g entry is the header.
My recent patchset titled "virtio-net: iovec handling cleanup"
removed this limitation but a feature
bit is needed so guests know it's safe to lay out
header differently.
This patch applies on top and adds such a feature bit.
virtio net
Il 28/09/2012 10:39, Daniel P. Berrange ha scritto:
>>> > > IMHO, you should also be importing the URI code test suite from libvirt
>>> > > to verify that the way you merged/changed the codebases did not break
>>> > > anything.
>> >
>> > Yes, can be done separately though.
> In this case I disagre
于 2012-9-28 15:52, Paolo Bonzini 写道:
Il 28/09/2012 05:04, Wenchao Xia ha scritto:
+$(LIBQBLOCK_TEST_DIR):
+@echo "Make libqblock test directory"
+mkdir $(LIBQBLOCK_TEST_DIR)
You can leave the files in tests/ directly, and avoid this as well.
Having a new directory will make clean
Il 06/09/2012 09:21, Gerd Hoffmann ha scritto:
> +#ifndef CONFIG_QXL_IO_MONITORS_CONFIG_ASYNC
> +/* spice-protocol is too old, add missing definitions */
> +#define QXL_IO_MONITORS_CONFIG_ASYNC (QXL_IO_FLUSH_RELEASE + 1)
> +#endif
> +
I don't see the need for this, why can't you just bump the mini
hi everybody,
I've decided to bother you guys for user list seems.. quiet,
no, really.
so, novice type of question, where/how do I get hold of the
values for below options?
I don't read the code, ain't no programmer, I can't figure
out how to get hold of the values for
pci-assign.host=pci
On Tue, Sep 25, 2012 at 12:05:13AM +0530, Amit Shah wrote:
> This is a rebased version of the patchset sent earlier.
>
> The current virtqueue_avail_bytes() is a weird API: it's oddly-named:
> doesn't tell us what the API is going to do, and also suits just one
> use-case (that in virtio-net.c).
>
On Thu, Sep 27, 2012 at 07:53:47AM +0200, Gerd Hoffmann wrote:
> Hi,
>
> > Understood, but I'd really prefer a file in docs/. We should be
> > rigorous about having formal specs for all of our paravirtual devices.
> > The code shouldn't be the spec.
>
> Well, pci-serial and pci-bridge are *not
On Wed, Sep 26, 2012 at 08:01:39AM -0500, Anthony Liguori wrote:
> Gerd Hoffmann writes:
>
> > On 09/26/12 01:43, Anthony Liguori wrote:
> >> Gerd Hoffmann writes:
> >>
> >>> Hi,
> >>>
> >>> Two patches, first split up serial.c a bit,
> >>> then actually add the pci-based serial device.
> >>
On 27/09/12 21:28, Luiz Capitulino wrote:
Sorry the delay review.
Before the qapi conversion, the sendkey command could be used to
send key codes in hex directly to the guest. In HMP, this would
be like:
(qemu) sendkey 0xdc
'0xdc' is not a available keycode, I didn't find '0xdc' in old
The following changes since commit 6f8fd2530e9a530f237240daf1c981fa5df7f978:
Merge branch 'arm-devs.for-upstream' of
git://git.linaro.org/people/pmaydell/qemu-arm (2012-09-27 21:48:56 +0200)
are available in the git repository at:
git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for
于 2012-9-28 15:51, Paolo Bonzini 写道:
Il 28/09/2012 05:00, Wenchao Xia ha scritto:
uint64_t qblock_get_virt_size(QBlockFmtInfo *fmt);
QBlockProtInfo *qblock_get_backing_loc(QBlockFmtInfo *fmt);
bool qblock_get_encrypt(QBlockFmtInfo *fmt);
etc. that include the switch statement.
I think it
Il 28/09/2012 12:52, Wenchao Xia ha scritto:
>> No problem for the implementation, but please make this function static.
>
> Do you mean declaring it as static function? I think user may need
> call it to get information so it need to be public.
The user can use the accessors. Let's not add a f
Il 27/09/2012 22:31, Jeff Cody ha scritto:
>
> block_job_completed(): cleans up when a job is done
It also raises a BLOCK_JOB_COMPLETED event.
> block_job_complete(): requests that a block job be completed
>
> How about renaming both of them, respectively, to:
>
> block_job_cleanup(): cleans
Hi,
>> For pci-serial windows needs a "driver" (which is just a inf file, the
>> driver itself is shipped by windows). So in that case it is easier to
>> go with our own ids I think, as we can simply ship a inf file then.
>> When picking the IDs of other cards, existing as real hardware, users
Jan Kiszka writes:
> On 2012-09-28 01:21, Satoru Moriya wrote:
>> This is a first time for me to post a patch to qemu-devel.
>> If there is something missing/wrong, please let me know.
>>
>> We have some plans to migrate old enterprise systems which require
>> low latency (msec order) to kvm vir
On 09/28/2012 04:36 AM, Amos Kong wrote:
> On 27/09/12 21:28, Luiz Capitulino wrote:
>>
>
> Sorry the delay review.
>
>
> hex isn't support when using qmp monitor.
>
>> +{ 'union': 'KeyValue',
>> + 'data': {
>> +'number': 'int',
>
> It's 'int' not hex format.
Indeed - I just re-read th
On 2012-09-28 14:33, Anthony Liguori wrote:
> Jan Kiszka writes:
>
>> On 2012-09-28 01:21, Satoru Moriya wrote:
>>> This is a first time for me to post a patch to qemu-devel.
>>> If there is something missing/wrong, please let me know.
>>>
>>> We have some plans to migrate old enterprise systems
On Fri, 28 Sep 2012 18:36:53 +0800
Amos Kong wrote:
> On 27/09/12 21:28, Luiz Capitulino wrote:
> >
>
> Sorry the delay review.
>
> > Before the qapi conversion, the sendkey command could be used to
> > send key codes in hex directly to the guest. In HMP, this would
> > be like:
> >
> > (qemu
On Fri, 28 Sep 2012 07:01:31 -0600
Eric Blake wrote:
> On 09/28/2012 04:36 AM, Amos Kong wrote:
> > On 27/09/12 21:28, Luiz Capitulino wrote:
> >>
> >
> > Sorry the delay review.
> >
>
> >
> > hex isn't support when using qmp monitor.
> >
>
> >> +{ 'union': 'KeyValue',
> >> + 'data': {
> >
- Original Message -
> On Fri, 28 Sep 2012 18:36:53 +0800
> Amos Kong wrote:
>
> > On 27/09/12 21:28, Luiz Capitulino wrote:
> > >
> >
> > Sorry the delay review.
> >
> > > Before the qapi conversion, the sendkey command could be used to
> > > send key codes in hex directly to the guest
Il 26/09/2012 14:14, Gerd Hoffmann ha scritto:
> Add multiport serial card implementation, with two variants,
> one featuring two and one featuring four ports.
>
> Signed-off-by: Gerd Hoffmann
> ---
> docs/qemupciserial.inf |2 +
> hw/serial-pci.c| 157
> +++
On 28 September 2012 02:12, Richard Henderson wrote:
> The interface to normalizeRoundAndPackFloat64 requires that the
> high bit be clear. Perform one shift-right-and-jam if needed.
>
> Signed-off-by: Richard Henderson
> ---
> fpu/softfloat.c | 9 +++--
> 1 file changed, 7 insertions(+), 2
On 28 September 2012 02:13, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> fpu/softfloat.c | 6 ++
> fpu/softfloat.h | 1 +
> 2 files changed, 7 insertions(+)
>
> diff --git a/fpu/softfloat.c b/fpu/softfloat.c
> index 45f105a..8df3a32 100644
> --- a/fpu/softfloat.c
> +++
Use the recently introduced tcg_out_mov_reg() function rather than
the equivalent inline code.
Signed-off-by: Peter Maydell
---
This is a minor follow-on cleanup patch after commit 9716ef3b1.
tcg/arm/tcg-target.c | 15 +++
1 file changed, 3 insertions(+), 12 deletions(-)
diff --g
The uint64_to_float32() conversion function was incorrectly always
returning numbers with the sign bit set (ie negative numbers). Correct
this so we return positive numbers instead.
Signed-off-by: Peter Maydell
---
As far as I can see we use this function only in the three PPC SPE
insns efscfuf,
On 28 September 2012 16:05, Peter Maydell wrote:
> Use the recently introduced tcg_out_mov_reg() function rather than
> the equivalent inline code.
>
> Signed-off-by: Peter Maydell
Whoops, used git send-email on wrong file; please ignore this email.
Correct patch should arrive shortly.
-- PMM
The DeviceNotActive text is not a particularly good match, add
a separate text while keeping the same class.
Signed-off-by: Paolo Bonzini
---
blockdev.c | 4 ++--
qerror.h | 3 +++
2 file modificati, 5 inserzioni(+), 2 rimozioni(-)
diff --git a/blockdev.c b/blockdev.c
index e5d450f..de5457d 1
Kevin,
the following changes since commit ac05f3492421caeb05809ffa02c6198ede179e43:
add a boot parameter to set reboot timeout (2012-09-25 20:05:04 -0500)
are available in the git repository at:
git://github.com/bonzini/qemu.git blkmirror-job-1.3-part1
for you to fetch changes up to ed306a
Extract it out of the implementation of info block-jobs.
Signed-off-by: Paolo Bonzini
---
blockdev.c | 15 ++-
blockjob.c | 11 +++
blockjob.h | 8
3 file modificati, 21 inserzioni(+), 13 rimozioni(-)
diff --git a/blockdev.c b/blockdev.c
index 7ab7d5e..5772c11 1006
Signed-off-by: Paolo Bonzini
---
Makefile.objs | 5 +-
block.c | 128 +
block.h | 2 +
block/Makefile.objs | 3 +-
block/stream.c | 1 +
block_int.h | 151
blockdev.
The imperative will be used for the QMP command.
Signed-off-by: Paolo Bonzini
---
block/stream.c | 4 ++--
blockjob.c | 2 +-
blockjob.h | 4 ++--
3 file modificati, 5 inserzioni(+), 5 rimozioni(-)
diff --git a/block/stream.c b/block/stream.c
index 57e4be7..a8f585a 100644
--- a/block/st
Add QMP commands matching the functionality.
Paused jobs cannot be canceled without first resuming them. This
ensures that I/O errors are never missed by management. However, an
optional force argument can be specified to allow that.
Signed-off-by: Paolo Bonzini
---
blockdev.c | 35
Add a test for each of report/ignore/stop. The tests use blkdebug
to generate an error in the middle of a script. The error is
recoverable (once = "on") so that we can test resuming a job after
stopping for an error.
Signed-off-by: Paolo Bonzini
---
tests/qemu-iotests/030| 220
Currently it is impossible to write a blkdebug script that ping-pongs
between two states, because the second set-state rule will use the
state that is set in the first. If you have
[set-state]
event = "..."
state = "1"
new_state = "2"
[set-state]
event = "..."
state =
The following behaviors are possible:
'report': The behavior is the same as in 1.1. An I/O error,
respectively during a read or a write, will complete the job immediately
with an error code.
'ignore': An I/O error, respectively during a read or a write, will be
ignored. For streaming, the job w
Move the common part of IDE/SCSI/virtio error handling to the block
layer. The new function bdrv_error_action subsumes all three of
bdrv_emit_qmp_error_event, vm_stop, bdrv_iostatus_set_err.
The same scheme will be used for errors in block jobs.
Signed-off-by: Paolo Bonzini
---
block.c
On 28 September 2012 16:17, Peter Maydell wrote:
> The uint64_to_float32() conversion function was incorrectly always
> returning numbers with the sign bit set (ie negative numbers). Correct
> this so we return positive numbers instead.
>
> Signed-off-by: Peter Maydell
> ---
> As far as I can see
Jan Kiszka writes:
> On 2012-09-28 14:33, Anthony Liguori wrote:
>> Jan Kiszka writes:
>>
>>> On 2012-09-28 01:21, Satoru Moriya wrote:
This is a first time for me to post a patch to qemu-devel.
If there is something missing/wrong, please let me know.
We have some plans to m
On 09/28/2012 09:22 AM, Paolo Bonzini wrote:
> Kevin,
>
> the following changes since commit ac05f3492421caeb05809ffa02c6198ede179e43:
>
> add a boot parameter to set reboot timeout (2012-09-25 20:05:04 -0500)
>
> are available in the git repository at:
>
> git://github.com/bonzini/qemu.git
Do this while we are touching this part of the code, before introducing
more uses of "int is_read".
Signed-off-by: Paolo Bonzini
---
block.c | 4 ++--
block.h | 4 ++--
blockdev.c | 2 +-
hw/ide/core.c | 2 +-
hw/ide/pci.c| 4 ++--
hw/scsi-disk.c | 2 +-
hw/virtio-blk
On Fri, 2012-09-28 at 10:54 +0100, lejeczek wrote:
> hi everybody,
>
> I've decided to bother you guys for user list seems.. quiet,
> no, really.
>
> so, novice type of question, where/how do I get hold of the
> values for below options?
> I don't read the code, ain't no programmer, I can't fig
Because pausing a job is asynchronous, we need to know whether it has
completed. This is described by the "busy" field of BlockJob; copy it
to BlockJobInfo.
Signed-off-by: Paolo Bonzini
---
blockjob.c | 1 +
qapi-schema.json | 5 -
2 file modificati, 5 inserzioni(+). 1 rimozione(-)
d
From: Jeff Cody
Signed-off-by: Jeff Cody
Signed-off-by: Paolo Bonzini
---
blockdev.c | 6 +++---
trace-events | 2 +-
2 file modificati, 4 inserzioni(+), 4 rimozioni(-)
diff --git a/blockdev.c b/blockdev.c
index de5457d..fa338fb 100644
--- a/blockdev.c
+++ b/blockdev.c
@@ -1065,12 +1065,12
iotests.py provides a convenience function that uses Python keyword
arguments to represent QMP command arguments. However, almost all
QMP commands use dashes for argument names (the sole exception is
block_set_io_throttle), and dashes are not allowed in a keyword
argument name. Hence provide auto
We want to remove knowledge of BLOCK_ERR_STOP_ENOSPC from drivers;
drivers should only be told whether to stop/report/ignore the error.
On the other hand, we want to keep using the nicer BlockErrorAction
name in the drivers. So rename the enums, while leaving aside the
names of the enum values for
Job pausing reuses the existing support for cancellable sleeps. A pause
happens at the next sleeping point and lasts until the coroutine is
re-entered explicitly. Cancellation was already doing a forced resume,
so implement it explicitly in terms of resume.
Paused jobs cannot be canceled without
This patch adds support for error management to streaming.
Signed-off-by: Paolo Bonzini
---
block/stream.c | 28 +++-
block_int.h | 3 ++-
blockdev.c | 11 ---
hmp.c| 3 ++-
qapi-schema.json | 9 +++--
qmp-commands.hx | 2 +-
6 fi
Do this in a separate commit before we move the functions to
blockjob.h.
Signed-off-by: Paolo Bonzini
---
block_int.h | 12 +---
1 file modificato, 5 inserzioni(+), 7 rimozioni(-)
diff --git a/block_int.h b/block_int.h
index ac4245c..b025fa0 100644
--- a/block_int.h
+++ b/block_int.h
@@
This will let block-stream reuse the enum. Places that used the enums
are renamed accordingly.
Signed-off-by: Paolo Bonzini
---
block.c | 6 +++---
block.h | 5 -
blockdev.c| 12 ++--
hw/fdc.c | 4 ++--
hw/ide/core.c | 6 +++---
hw/scsi-d
These check that a paused streaming job does not advance its offset.
Sometimes the new test fails; the map is different between the source
and the destination of the streaming because qemu-io does not always
pack adjacent clusters that have the same allocated/unallocated state.
However, this also
setsockopt needs a type cast for MinGW. That type cast is missing in
a recent commit which results in a compiler warning.
Like for other socket related functions which have the same problem,
we add a 'qemu_setsockopt' macro which provides that type cast where
needed and use the new macro to avoid
On 09/27/2012 07:28 AM, Luiz Capitulino wrote:
> Today, it's necessary to specify the protocol you want to use
> when dumping the guest memory, for example:
>
> (qemu) dump-guest-memory file:/tmp/guest-memory
>
> This has a few issues:
>
> 1. It's cumbersome to type
> 2. We loose file path au
Very easy to reproduce:
1) Build the latest qemu.git (we've captured this on internal automated
testing, verified manually), the commit for reference is:
14:07:02 INFO | git commit ID is
6f8fd2530e9a530f237240daf1c981fa5df7f978 (tag v1.2.0-461-g6f8fd25)
2) Install a linux guest in it (caugh
Public bug reported:
Very easy to reproduce:
1) Build the latest qemu.git (we've captured this on internal automated
testing, verified manually), the commit for reference is:
14:07:02 INFO | git commit ID is
6f8fd2530e9a530f237240daf1c981fa5df7f978 (tag v1.2.0-461-g6f8fd25)
2) Install a linux g
From: Bharata B Rao
This patch adds gluster as the new block backend in QEMU. This gives
QEMU the ability to boot VM images from gluster volumes. Its already
possible to boot from VM images on gluster volumes using FUSE mount, but
this patchset provides the ability to boot VM images from gluster
From: Jeff Cody
This is a simple helper function, that will return the base image
of a given image chain.
Signed-off-by: Jeff Cody
Reviewed-by: Eric Blake
Signed-off-by: Kevin Wolf
---
block.c | 16
block.h |1 +
2 files changed, 17 insertions(+), 0 deletions(-)
diff
From: Jeff Cody
This adds the live commit coroutine. This iteration focuses on the
commit only below the active layer, and not the active layer itself.
The behaviour is similar to block streaming; the sectors are walked
through, and anything that exists above 'base' is committed back down
into
From: Paolo Bonzini
Extract it out of the implementation of info block-jobs.
Signed-off-by: Paolo Bonzini
Reviewed-by: Eric Blake
Signed-off-by: Kevin Wolf
---
blockdev.c | 15 ++-
blockjob.c | 11 +++
blockjob.h |8
3 files changed, 21 insertions(+), 13
From: Paolo Bonzini
Because pausing a job is asynchronous, we need to know whether it has
completed. This is described by the "busy" field of BlockJob; copy it
to BlockJobInfo.
Signed-off-by: Paolo Bonzini
Reviewed-by: Eric Blake
Signed-off-by: Kevin Wolf
---
blockjob.c |1 +
qapi
From: Paolo Bonzini
Add a test for each of report/ignore/stop. The tests use blkdebug
to generate an error in the middle of a script. The error is
recoverable (once = "on") so that we can test resuming a job after
stopping for an error.
Signed-off-by: Paolo Bonzini
Reviewed-by: Eric Blake
Si
From: Paolo Bonzini
This patch adds support for error management to streaming.
Signed-off-by: Paolo Bonzini
Reviewed-by: Eric Blake
Signed-off-by: Kevin Wolf
---
block/stream.c | 28 +++-
block_int.h |3 ++-
blockdev.c | 11 ---
hmp.c
The algorithm was named after Martin E. Hellman.
Signed-off-by: Stefan Weil
---
ui/vnc-tls.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ui/vnc-tls.c b/ui/vnc-tls.c
index 3aaa939..a7f7d07 100644
--- a/ui/vnc-tls.c
+++ b/ui/vnc-tls.c
@@ -49,7 +49,7 @@ static int vnc_tls
Am 27.09.2012 19:29, schrieb Jeff Cody:
> This series adds the basic case, of a live commit between two
> images below the active layer, e.g.:
>
> [base] <--- [snp-1] <--- [snp-2] <--- [snp-3] <--- [active]
>
> can be collapsed down via commit, into:
>
> [base] <--- [active]
>
> or,
>
> [base]
Am 28.09.2012 17:22, schrieb Paolo Bonzini:
> Kevin,
>
> the following changes since commit ac05f3492421caeb05809ffa02c6198ede179e43:
>
> add a boot parameter to set reboot timeout (2012-09-25 20:05:04 -0500)
>
> are available in the git repository at:
>
> git://github.com/bonzini/qemu.git
Am 27.09.2012 15:55, schrieb Bharata B Rao:
> Hi,
>
> This is v10 of the patchset to support GlusterFS backend from QEMU.
>
> Changes in v10
> --
> - Included another fix to qemu_aio_wait() from Paolo in the series.
> - A few more fixes to the volume and image parsing code.
> - Remove
From: Paolo Bonzini
The DeviceNotActive text is not a particularly good match, add
a separate text while keeping the same class.
Signed-off-by: Paolo Bonzini
Reviewed-by: Eric Blake
Signed-off-by: Kevin Wolf
---
blockdev.c |4 ++--
qerror.h |3 +++
2 files changed, 5 insertions(+),
From: Paolo Bonzini
Signed-off-by: Paolo Bonzini
Reviewed-by: Eric Blake
Signed-off-by: Kevin Wolf
---
Makefile.objs |5 +-
block.c | 128 +
block.h |2 +
block/Makefile.objs |5 +-
block/commit.c |1 +
bl
From: Paolo Bonzini
Add QMP commands matching the functionality.
Paused jobs cannot be canceled without first resuming them. This
ensures that I/O errors are never missed by management. However, an
optional force argument can be specified to allow that.
Signed-off-by: Paolo Bonzini
Reviewed-
thanks Alex for your patience, appreciate it I do
what would be the droids I need?
I'm experiencing guests' "puzzling" behavior and was
suggested that command line arguments were wrong/incomplete.
same box/hardware and radeon 5450 and ...
- winXP-32bit -> OK - I assumed getting the guest on a
From: Jeff Cody
Currently, after a live snapshot of a drive, the image that has
been 'demoted' to be below the new active layer remains r/w.
This patch reopens it read-only.
Note that we do not check for error on the reopen(), because we
will not abort the snapshots if the reopen fails.
This pa
From: Paolo Bonzini
The AIO dispatch loop will call QLIST_REMOVE and g_free even if there
are other pending calls to qemu_aio_wait outside the current one.
Signed-off-by: Paolo Bonzini
Signed-off-by: Bharata B Rao
Signed-off-by: Kevin Wolf
---
aio.c | 10 +-
1 files changed, 5 inse
From: Paolo Bonzini
Currently it is impossible to write a blkdebug script that ping-pongs
between two states, because the second set-state rule will use the
state that is set in the first. If you have
[set-state]
event = "..."
state = "1"
new_state = "2"
[set-state]
eve
From: Paolo Bonzini
Do this while we are touching this part of the code, before introducing
more uses of "int is_read".
Signed-off-by: Paolo Bonzini
Reviewed-by: Eric Blake
Signed-off-by: Kevin Wolf
---
block.c |4 ++--
block.h |4 ++--
blockdev.c |2 +-
hw/i
From: Paolo Bonzini
This will let block-stream reuse the enum. Places that used the enums
are renamed accordingly.
Signed-off-by: Paolo Bonzini
Reviewed-by: Eric Blake
Signed-off-by: Kevin Wolf
---
block.c |6 +++---
block.h |5 -
block/commit.c| 14 +++
From: Paolo Bonzini
Fix qemu_aio_wait() to ensure that registered aio handlers don't get
deleted when they are still active. This is ensured by maintaning the
right count of walking_handlers.
Signed-off-by: Paolo Bonzini
Signed-off-by: Bharata B Rao
Signed-off-by: Kevin Wolf
---
aio.c |8
From: Paolo Bonzini
Job pausing reuses the existing support for cancellable sleeps. A pause
happens at the next sleeping point and lasts until the coroutine is
re-entered explicitly. Cancellation was already doing a forced resume,
so implement it explicitly in terms of resume.
Paused jobs cann
From: Paolo Bonzini
The following behaviors are possible:
'report': The behavior is the same as in 1.1. An I/O error,
respectively during a read or a write, will complete the job immediately
with an error code.
'ignore': An I/O error, respectively during a read or a write, will be
ignored. Fo
From: Bharata B Rao
GlusterFS support in QEMU depends on libgfapi, libgfrpc and
libgfxdr provided by GlusterFS.
Signed-off-by: Bharata B Rao
Signed-off-by: Kevin Wolf
---
configure | 35 +++
1 files changed, 35 insertions(+), 0 deletions(-)
diff --git a/conf
From: Paolo Bonzini
Move the common part of IDE/SCSI/virtio error handling to the block
layer. The new function bdrv_error_action subsumes all three of
bdrv_emit_qmp_error_event, vm_stop, bdrv_iostatus_set_err.
The same scheme will be used for errors in block jobs.
Signed-off-by: Paolo Bonzini
From: Paolo Bonzini
Do this in a separate commit before we move the functions to
blockjob.h.
Signed-off-by: Paolo Bonzini
Reviewed-by: Eric Blake
Signed-off-by: Kevin Wolf
---
block_int.h | 12 +---
1 files changed, 5 insertions(+), 7 deletions(-)
diff --git a/block_int.h b/block_
From: Paolo Bonzini
iotests.py provides a convenience function that uses Python keyword
arguments to represent QMP command arguments. However, almost all
QMP commands use dashes for argument names (the sole exception is
block_set_io_throttle), and dashes are not allowed in a keyword
argument nam
From: Jeff Cody
The command for live block commit is added, which has the following
arguments:
device: the block device to perform the commit on (mandatory)
base: the base image to commit into; optional (if not specified,
it is the underlying original image)
top:the top image of th
When cancelling block migration, all in-flight requests of the block
migration must be completed before the data can be freed. This was
visible as failing assertions and segfaults.
Reported-by: Peter Lieven
Signed-off-by: Kevin Wolf
---
block-migration.c |2 ++
1 files changed, 2 insertions
From: Jeff Cody
Signed-off-by: Jeff Cody
Reviewed-by: Eric Blake
Signed-off-by: Kevin Wolf
---
blockdev.c |6 +++---
trace-events |2 +-
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/blockdev.c b/blockdev.c
index 0267fa3..f910ac5 100644
--- a/blockdev.c
+++ b/blockd
From: Jeff Cody
Derived from the streaming test cases (030), this adds the
following 9 tests:
1. For the following image chain, commit [mid] into [backing],
and use qemu-io to verify [backing] has its original data, as
well as the data from [mid]
[backing] <-- [mid] <-- [test]
From: Paolo Bonzini
We want to remove knowledge of BLOCK_ERR_STOP_ENOSPC from drivers;
drivers should only be told whether to stop/report/ignore the error.
On the other hand, we want to keep using the nicer BlockErrorAction
name in the drivers. So rename the enums, while leaving aside the
names
From: Jeff Cody
Add bdrv_find_overlay(), and bdrv_drop_intermediate().
bdrv_find_overlay(): given 'bs' and the active (topmost) BDS of an image chain,
find the image that is the immediate top of 'bs'
bdrv_drop_intermediate():
Given 3 BDS (active, top, ba
From: Paolo Bonzini
These check that a paused streaming job does not advance its offset.
Sometimes the new test fails; the map is different between the source
and the destination of the streaming because qemu-io does not always
pack adjacent clusters that have the same allocated/unallocated stat
The following changes since commit 6f8fd2530e9a530f237240daf1c981fa5df7f978:
Merge branch 'arm-devs.for-upstream' of
git://git.linaro.org/people/pmaydell/qemu-arm (2012-09-27 21:48:56 +0200)
are available in the git repository at:
git://repo.or.cz/qemu/kevin.git for-anthony
Bharata B Rao (
On Fri, 2012-09-28 at 19:46 +0100, lejeczek wrote:
> thanks Alex for your patience, appreciate it I do
>
> what would be the droids I need?
> I'm experiencing guests' "puzzling" behavior and was
> suggested that command line arguments were wrong/incomplete.
>
> same box/hardware and radeon 5450
On 09/28/2012 02:52:15 PM, folkert wrote:
On Tue, Sep 25, 2012 at 09:49:08AM +0200, Alexander Graf wrote:
> > While investigating dtb pad issues, I noticed that initrd_base
wasn't taking
> > loadaddr into account the way dt_base was. This seems wrong.
> >
> > Signed-off-by: Scott Wood
>
> Ah
QEMU by default configures -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE
so there is not need to mention O_LARGEFILE in the block layer.
Signed-off-by: Richa Marwaha
---
block/raw-posix.c |2 +-
block/vdi.c |2 +-
block/vmdk.c |6 +++---
block/vvfat.c |2 +-
4 files c
On Tue, Sep 25, 2012 at 09:49:08AM +0200, Alexander Graf wrote:
> > While investigating dtb pad issues, I noticed that initrd_base wasn't taking
> > loadaddr into account the way dt_base was. This seems wrong.
> >
> > Signed-off-by: Scott Wood
>
> Ah, all the images I tested start off from addr
On 09/28/2012 10:45 AM, Richa Marwaha wrote:
> QEMU by default configures -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE
> so there is not need to mention O_LARGEFILE in the block layer.
>
> Signed-off-by: Richa Marwaha
> ---
> block/raw-posix.c |2 +-
> block/vdi.c |2 +-
> block/vmdk
1 - 100 of 103 matches
Mail list logo
