On Thu, Apr 19, 2012 at 22:56, Xin Tong wrote:
> On Thu, Apr 19, 2012 at 1:03 PM, Blue Swirl wrote:
>> On Thu, Apr 19, 2012 at 01:55, Xin Tong wrote:
>>> but should not the address be within 1 - 4G-1 even with PAE. is not
>>> the PAE just using 64bits addresses as supposed to 32 bit ? what does
On Thu, Apr 19, 2012 at 20:48, Peter Maydell wrote:
> On 19 April 2012 21:27, Blue Swirl wrote:
>> @@ -119,7 +120,9 @@ CPUSPARCState *cpu_sparc_init(const char *cpu_model)
>> cpu = SPARC_CPU(object_new(TYPE_SPARC_CPU));
>> env = &cpu->env;
>>
>> - gen_intermediate_code_init(env);
>> +
Am 15.04.2012 05:10, schrieb Andreas Färber:
> Hello,
>
> This series splits up my m68k QOM'ification patch from the qom-cpu-others RFC
> series.
> CPU models are now modelled as subclasses with their own initfns, leaving the
> setting of feature flags imperative.
>
> Please review and apply.
P
Am 12.04.2012 01:00, schrieb Andreas Färber:
> Hello Edgar,
>
> This series strips down my CRIS QOM'ification patch from the qom-cpu-others.v1
> RFC series and splits it into multiple steps. I still think the code is
> calling for subclasses for crisv8 etc. but, given the timing, my priority is
>
Am 12.04.2012 13:20, schrieb Andreas Färber:
> Hi Edgar,
>
> Here's a rebased version of MicroBlaze CPU QOM'ification.
Ping! Any problem or just lack of time? Hard Freeze is Monday in a
week... I'd have one additional optional change to cpu_mb_init() for a
v4, so let me know if I need to change a
Hi Peter,
Am 19.04.2012 04:33, schrieb Peter Crosthwaite:
> So is there any standard policy on setting maintainer-ships for device
> models? When I pushed the cadence IP device models (hw/cadence*) for
> xilinx zynq there was insistence that they be in the MAINTAINERS,
Yes, that was my request.
Callers are changed to use qerror_report_err() to keep their QError
semantics.
Signed-off-by: Luiz Capitulino
---
hw/pci-hotplug.c |8 ++--
hw/usb/dev-network.c |7 +--
net.c| 53 +++---
net.h|2
Signed-off-by: Luiz Capitulino
---
qemu-option.c | 20 +++-
1 file changed, 11 insertions(+), 9 deletions(-)
diff --git a/qemu-option.c b/qemu-option.c
index a8b50af..61354af 100644
--- a/qemu-option.c
+++ b/qemu-option.c
@@ -203,7 +203,8 @@ static void parse_option_number(cons
This is not a full QAPI conversion, but an intermediate step.
In essence, do_netdev_add() is split into three functions:
1. netdev_add(): performs the actual work. This function is fully
converted to Error (thus, it's "qapi-friendly")
2. qmp_netdev_add(): the QMP front-end for netdev_add()
The functions qemu_opt_set() and opts_do_parse() both call opt_set(),
but their callers expect QError semantics. Thus, both functions call
qerro_report_err() to keep the expected semantics.
Signed-off-by: Luiz Capitulino
---
qemu-option.c | 31 ---
1 file changed, 2
net_client_init() propagates the error up by calling qerror_report_err(),
because its users expect QError semantics.
Signed-off-by: Luiz Capitulino
---
net.c |6 +-
qemu-option.c | 13 +
qemu-option.h |2 +-
3 files changed, 11 insertions(+), 10 deletions(-)
di
Signed-off-by: Anthony Liguori
Signed-off-by: Luiz Capitulino
---
hmp-commands.hx |3 +--
hmp.c|9 +
hmp.h|1 +
net.c| 11 +--
net.h|1 -
qapi-schema.json | 14 ++
qmp-commands.hx |5 +
7 fi
The functions opt_set() and qemu_opts_validate() both call qemu_opt_parse(),
but their callers expect QError semantics. Thus, both functions call
qerro_report_err() to keep the expected semantics.
Signed-off-by: Luiz Capitulino
---
qemu-option.c | 36 +---
1 fil
Note that set_option_parameter() callers still expect automatic error
reporting with QError, so set_option_parameter() calls
qerror_report_err() to keep the same semantics.
Signed-off-by: Luiz Capitulino
---
qemu-option.c | 22 +++---
1 file changed, 15 insertions(+), 7 deletio
This is like qemu_find_opts(), except that it takes an Error argument.
This new function allows for a incremental conversion of code using
qemu_find_opts().
Signed-off-by: Luiz Capitulino
---
qemu-config.c |5 +
qemu-config.h |3 +++
2 files changed, 8 insertions(+)
diff --git a/qe
do_device_add() and do_netdev_add() call qerror_report_err() to maintain
their QError semantics.
Signed-off-by: Luiz Capitulino
---
hw/qdev-monitor.c |7 +--
net.c |5 -
qemu-option.c | 31 ---
qemu-option.h |3 ++-
4 files ch
This is like qemu_opt_set(), except that it takes an Error argument.
This new function allows for a incremental conversion of code using
qemu_opt_set().
Signed-off-by: Luiz Capitulino
---
qemu-option.c |6 ++
qemu-option.h |2 ++
2 files changed, 8 insertions(+)
diff --git a/qemu-o
The only backend that really uses it is the socket one, which calls
monitor_get_fd(). But it can use 'cur_mon' instead.
Signed-off-by: Luiz Capitulino
---
hw/pci-hotplug.c |2 +-
hw/usb/dev-network.c |2 +-
net.c| 18 +++---
net.h|2 +
Signed-off-by: Luiz Capitulino
---
qerror.c |4
qerror.h |3 +++
2 files changed, 7 insertions(+)
diff --git a/qerror.c b/qerror.c
index 96fbe71..5d92096 100644
--- a/qerror.c
+++ b/qerror.c
@@ -156,6 +156,10 @@ static const QErrorStringTable qerror_table[] = {
.desc =
This commit converts qemu_opts_create() from qerror_report() to
error_set().
Currently, most calls to qemu_opts_create() can't fail, so most
callers don't need any changes.
The two cases where code checks for qemu_opts_create() erros are:
1. Initialization code in vl.c. All of them print their
Note that qemu_find_opts() and qemu_config_parse() need to call
error_report() to maintain their semantics on error.
Signed-off-by: Luiz Capitulino
---
qemu-config.c | 32 +---
1 file changed, 25 insertions(+), 7 deletions(-)
diff --git a/qemu-config.c b/qemu-confi
Note that qemu_opt_parse() callers still expect automatic error reporting
with QError, so qemu_opts_parse() calls qerror_report_err() to keep the
same semantics.
Signed-off-by: Luiz Capitulino
---
qemu-option.c | 26 +++---
1 file changed, 19 insertions(+), 7 deletions(-)
v2
- convert netdev_add by using gen=no as suggested by Paolo
- drop unneeded patches (qemu-option & code generator changes)
- re-work some error handling changes
- improve some changelogs
blockdev.c |2 +-
hmp-commands.hx |6 +-
hmp.c| 30 +
hmp.
On Thu, 19 Apr 2012 19:00:39 +0800
Wen Congyang wrote:
> At 04/18/2012 08:49 PM, Wen Congyang Wrote:
> > Hi, all
> >
> > 'virsh dump' can not work when host pci device is used by guest. We have
> > discussed this issue here:
> > http://lists.nongnu.org/archive/html/qemu-devel/2011-10/msg00736.ht
On 04/19/2012 05:47 AM, Laurent Vivier wrote:
> Hi Rob,
>
> you need to do a fresh clone because I rebase the branch frequently on
> the qemu/master instead of merging it. It is easier to manage for me.
Ok. (Did that.)
> BTW, qemu-system-m68k is not working currently.
Did it ever? :)
> I'm wo
On Fri, Apr 20, 2012 at 09:39:59PM +0200, Paolo Bonzini wrote:
> Il 20/04/2012 19:26, Michael Roth ha scritto:
> > Do we have a way of differentiating a guest shutting down as a result of
> > shutdown vs. suspend-to-disk? AFAIK they both have the same ending state. If
> > so I'm not sure we can say
So my next question is, does qemu have a way that features can be sponsored?
Aaron Stover
Celestech Inc.
480-940-1640 x205
- Reply message -
From: "Brian Jackson"
To: , "Aaron Stover (Celestech)"
Subject: [Qemu-devel] Fault Tolerance (Kemari) Status?
Date: Fri, Apr 20, 2012 9:44 am
> /* QEMU Synchronous Serial Interface support. */
>
> -/* In principle SSI is a point-point interface. As such the qemu
> - implementation has a single slave device on a "bus".
> +/* In principle SSI is a point-point interface.
> However it is fairly common for boards to have multiple sl
At Fri, 20 Apr 2012 20:05:48 +0200,
Christoph Hellwig wrote:
>
> On Tue, Apr 03, 2012 at 01:35:50AM +0800, Liu Yuan wrote:
> > From: Liu Yuan
> >
> > Flush operation is supposed to flush the write-back cache of
> > sheepdog cluster.
> >
> > By issuing flush operation, we can assure the Guest of
Public bug reported:
When the GNOME 3 screensaver activates with the mouse cursor over the
SDL window, the screensaver will not unlock unless the gnome-shell
process is killed and restarted manually. This seems to be related to
the fact that the key strokes are grabbed by SDL, but the screensaver
Il 20/04/2012 19:26, Michael Roth ha scritto:
> Do we have a way of differentiating a guest shutting down as a result of
> shutdown vs. suspend-to-disk? AFAIK they both have the same ending state. If
> so I'm not sure we can say anything other than "the machine may or may not
> resume when you star
If do_alloc_cluster_offset() fails, the error handling code tried to
remove the request from the in-flight queue, to which it wasn't added
yet, resulting in a NULL pointer dereference.
m->nb_clusters really only becomes != 0 when the request is in the list.
Signed-off-by: Kevin Wolf
---
block/q
From: Paolo Bonzini
Both unused after the previous patch.
Signed-off-by: Paolo Bonzini
Signed-off-by: Kevin Wolf
---
aio.c | 29 ++---
block/curl.c | 10 --
block/iscsi.c |4 ++--
block/nbd.c|8
block/rbd.c
With this change, reading from a qcow2 image ignores all reserved bits
that are set in an L1 or L2 table entry.
Now get_cluster_offset() assigns *cluster_offset only the offset without
any other flags. The cluster type is not longer encoded in the offset,
but a positive return value in case of suc
qcow2_alloc_compressed_cluster_offset() already fails if the copied flag
is set, because qcow2_write_compressed() doesn't perform COW as it would
have to do to allow this.
However, what we really want to check here is whether the cluster is
allocated or not. With internal snapshots the copied flag
Signed-off-by: Kevin Wolf
---
tests/qemu-iotests/034 | 113
tests/qemu-iotests/034.out | 81 +++
tests/qemu-iotests/group |1 +
3 files changed, 195 insertions(+), 0 deletions(-)
create mode 100755 tests/qemu-i
From: Stefan Hajnoczi
The IDE PIO interface currently uses bdrv_read() to perform reads
synchronously. Synchronous I/O in the vcpu thread is bad because it
prevents the guest from executing code - it makes the guest
unresponsive.
This patch converts IDE PIO to use bdrv_aio_readv(). We simply n
To be used as in 'qemu-io -t writeback test.img'
Signed-off-by: Kevin Wolf
Reviewed-by: Paolo Bonzini
---
qemu-io.c | 10 +-
1 files changed, 9 insertions(+), 1 deletions(-)
diff --git a/qemu-io.c b/qemu-io.c
index e6fcd77..3095a22 100644
--- a/qemu-io.c
+++ b/qemu-io.c
@@ -1784,6 +1
Also don't infer the cluster type directly from the L2 entries, but use
qcow2_get_cluster_type() to keep everything in a single place.
Signed-off-by: Kevin Wolf
---
block/qcow2-refcount.c | 98 ++-
1 files changed, 54 insertions(+), 44 deletions(-)
Some image formats do have a cluster size, others don't, but there are
tests that work with both sets of images and currently we get failures
because the qemu-img create output doesn't mention the cluster size for
some formats.
Signed-off-by: Kevin Wolf
Reviewed-by: Paolo Bonzini
---
tests/qemu
Instead of printing an ugly bitmask, qemu can now print a more helpful
string even for yet unknown features.
Signed-off-by: Kevin Wolf
---
block/qcow2.c | 66 +---
block/qcow2.h | 12 ++
2 files changed, 69 insertions(+), 9 deletion
From: Dong Xu Wang
According comment, we should not read again, we will write.
Signed-off-by: Dong Xu Wang
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Kevin Wolf
---
tests/qemu-iotests/005 |2 +-
tests/qemu-iotests/005.out |2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
From: Stefan Weil
Signed-off-by: Stefan Weil
Signed-off-by: Kevin Wolf
---
block/cow.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/block/cow.c b/block/cow.c
index 8d3c9f8..a5a00eb 100644
--- a/block/cow.c
+++ b/block/cow.c
@@ -103,7 +103,7 @@ static int cow_open(B
This adds support for reading zero clusters in version 3 images.
Signed-off-by: Kevin Wolf
---
block/qcow2-cluster.c | 17 +
block/qcow2-refcount.c |7 +++
block/qcow2.c |8
block/qcow2.h |5 +
4 files changed, 33 insertions(+), 4
This updates the qcow2 specification to cover version 3. It contains the
following changes:
- Added compatible/incompatible/auto-clear feature bits plus an optional
feature name table to allow useful error messages even if an older
version doesn't know some feature at all.
- Configurable refc
This adds the basic infrastructure to qcow2 to handle version 3 images.
It includes code to create v3 images, allow header updates for v3 images
and checks feature bits.
It still misses support for zero clusters, so this is not a fully
compliant implementation of v3 yet.
The default for creating
From: Paolo Bonzini
This will let timers run during aio_read and aio_write commands,
though not during synchronous commands.
Signed-off-by: Paolo Bonzini
Signed-off-by: Kevin Wolf
---
cmd.c | 10 +-
qemu-io.c |7 ---
2 files changed, 9 insertions(+), 8 deletions(-)
diff
This adds an -o option to qemu-iotests, which is an option string that
is passed through to qemu-img create -o... This allows testing different
subformat with a command like './check -qcow2 -o compat=0.10'.
For qcow2, if no compat option is specified, compat=1.1 is the new
default.
Signed-off-by:
From: Andreas Färber
`hostname -s` may output an errror:
hostname: Name or service not known
This causes all tests to fail for `make check-block`.
Suppress such error messages, letting the tests succeed.
Signed-off-by: Andreas Färber
Signed-off-by: Kevin Wolf
---
tests/qemu-iotests/common.co
count_cow_clusters() tries to reuse existing functions, and all it
achieves is to make things much more complicated than they really are:
Everything needs COW, unless it's a normal cluster with refcount 1.
This patch implements the obvious way of doing this, and by using
qcow2_get_cluster_type() i
From: Paolo Bonzini
Signed-off-by: Paolo Bonzini
Signed-off-by: Kevin Wolf
---
tests/qemu-iotests/033 | 73
tests/qemu-iotests/033.out | 29 +
tests/qemu-iotests/group |1 +
3 files changed, 103 insertions(+), 0 deletio
Signed-off-by: Kevin Wolf
---
block.c | 14 +++--
block/qcow2-cluster.c | 72 +
block/qcow2.c | 21 ++
block/qcow2.h |1 +
4 files changed, 105 insertions(+), 3 deletions(-)
diff --git a/block
This changes the still existing places that assume that the only flags
are QCOW_OFLAG_COPIED and QCOW_OFLAG_COMPRESSED to properly mask out
reserved bits.
It does not convert bdrv_check yet.
Signed-off-by: Kevin Wolf
---
block/qcow2-cluster.c | 26 +-
block/qcow2-refc
From: Paolo Bonzini
The definition of when qemu_aio_flush should loop is much simpler
than it looks. It just has to call qemu_aio_wait until it makes
no progress and all flush callbacks return false. qemu_aio_wait
is the logical place to tell the caller about this.
Signed-off-by: Paolo Bonzini
Signed-off-by: Kevin Wolf
---
block/qcow2-refcount.c |2 +-
block/qcow2.h |2 ++
2 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/block/qcow2-refcount.c b/block/qcow2-refcount.c
index 909d615..e0854dc 100644
--- a/block/qcow2-refcount.c
+++ b/block/qcow2-refcount.c
From: Liu Yuan
The 'qemu-img convert -h' advertise that the default cache mode is
'writeback', while in fact it is 'unsafe'.
This patch 1) fix the help manual and 2) let bdrv_close() call bdrv_flush()
2) is needed because some backend storage doesn't have a self-flush
mechanism(for e.g., sheepd
qcow2.py must be updated to work with version 3 images at all, the
output has changed since the feature table extension has been added, and
version 2 and version 3 images can't possibly have the same test output.
Change the test case to completely ignore IMGOPTS and run the test for
both compat=1.
From: Paolo Bonzini
In the next patch we want to reenter the coroutine from
block_job_cancel_sync and cancel the timer.
Signed-off-by: Paolo Bonzini
Signed-off-by: Kevin Wolf
---
qemu-coroutine-sleep.c |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/qemu-coroutine-s
Until now, count_contiguous_clusters() has an argument that allowed to
specify flags that should be ignored in the comparison, i.e. that are
allowed to change between contiguous clusters.
This patch changes the function so that it ignores all flags by default
now and you need to pass the flags on
On Tue, Apr 03, 2012 at 01:35:50AM +0800, Liu Yuan wrote:
> From: Liu Yuan
>
> Flush operation is supposed to flush the write-back cache of
> sheepdog cluster.
>
> By issuing flush operation, we can assure the Guest of data
> reaching the sheepdog cluster storage.
How does qemu know that the ca
This allows that different snapshots of an image can have different
sizes, which is a requirement for enabling image resizing even with
images that have internal snapshots.
We don't do the actual support for it now, but make sure that the
additional field is present and not completely ignored in a
From: Stefan Hajnoczi
The IDE PIO write sector code path uses bdrv_write() and hence can make
the guest unresponsive while the I/O request is in progress. This patch
converts ide_sector_write() to use bdrv_aio_writev() by using the
BUSY_STAT bit to tell the guest that the request is in progress.
On 04/03/2012 06:15 AM, Paolo Bonzini wrote:
Signed-off-by: Paolo Bonzini
---
hw/qdev.c | 35 ---
1 file changed, 20 insertions(+), 15 deletions(-)
diff --git a/hw/qdev.c b/hw/qdev.c
index e674248..45f1133 100644
--- a/hw/qdev.c
+++ b/hw/qdev.c
@@ -601,27 +60
From: Paolo Bonzini
The do...while loop can never loop, because select will just not return
0 when invoked with infinite timeout.
Signed-off-by: Paolo Bonzini
Signed-off-by: Kevin Wolf
---
aio.c | 133 +++-
1 files changed, 64 inser
Refcount block allocation and refcount table growth rely on
s->free_cluster_index pointing to somewhere after the current
allocation. Change qcow2_alloc_cluster_at() to fulfill this
assumption.
Without this change it could happen that a newly allocated refcount
block and the allocated data block p
From: Paolo Bonzini
posix_aio_read already calls qemu_aio_process_queue, and dually
qemu_aio_process_queue is always followed by a select loop that calls
posix_aio_read.
No races are possible, so there is no need for a separate process_queue
callback.
Signed-off-by: Paolo Bonzini
Signed-off-by
From: Oskar Andero
Add ds1338 rtc attached on i2c.
Signed-off-by: Oskar Andero
Signed-off-by: Peter Maydell
---
hw/versatilepb.c |6 ++
1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/hw/versatilepb.c b/hw/versatilepb.c
index 25afb1e..d011554 100644
--- a/hw/versatilepb
From: Oskar Andero
The versatile i2c controller implementation was separated to
its own file called versatile_i2c.c. This is done as a preparation
for adding i2c support to the versatilepb board.
Signed-off-by: Oskar Andero
Signed-off-by: Peter Maydell
---
Makefile.target|1 +
hw/real
On reset of the mpcore timer/watchdog block we need to
delete the qemu_timer in case it was running.
Signed-off-by: Peter Maydell
---
hw/arm_mptimer.c |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/hw/arm_mptimer.c b/hw/arm_mptimer.c
index df7fb4c..fe43cbb 100644
---
From: Eric Benard
- add support for the 64MB NOR CFI01 flash available at
0x3400 on the versatilepb board
http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.dui0225d/BBAJIHEC.html
- tested with barebox bootloader
Signed-off-by: Eric Bénard
Signed-off-by: Peter Maydell
---
hw/ver
Hi; this is a pullreq for the arm-devs queue, fairly minor stuff.
Please pull.
thanks
-- PMM
The following changes since commit 51006bbc45bc74977ae538190a53df2af534acb9:
Merge remote-tracking branch 'origin/master' into staging (2012-04-18
10:06:09 -0500)
are available in the git repository
From: Peter Maydell
Move the A9 config_base_address cp15 register reset value to
ARMCPU. This should become a QOM property so that the Highbank
board can set it without having to pull in cpu-qom.h, but at
least this avoids the implicit dependency on reset ordering
that the previous workaround had
Make cpu_arm_init() return a QOM ARMCPU, so that we don't need to
obtain an ARMCPU through arm_env_get_cpu() in machine init code.
This requires to adjust the inclusion site of cpu-qom.h and in turn,
forward-looking, to homogenize its include order.
cpu_init() must still return a CPUARMState for b
From: Andreas Färber
Hello Peter,
This mini-series redoes your drop-reset-model-init patch 14/14 v2.
I've investigated doing it this way for sh4 and plan to adjust all targets.
Note that I haven't tested Highbank, just the Integrator/CP using old
cpu_init().
Regards,
Andreas
Cc: Peter Maydel
On Fri, Apr 20, 2012 at 12:26:10PM -0500, Michael Roth wrote:
> On Fri, Apr 20, 2012 at 10:36:38AM -0300, Luiz Capitulino wrote:
> > On Fri, 20 Apr 2012 14:07:16 +0200
> > Michal Privoznik wrote:
> >
> > > >> But, I think if we tell users we'll *only* send response on error,
> > > >> we should do
Zero clusters will add another cluster type. Refactor the open-coded
cluster type detection into a switch of QCOW2_CLUSTER_* options so that
the detection is in a single place. This makes it easier to add new
cluster types.
Signed-off-by: Kevin Wolf
---
block/qcow2-refcount.c | 41
From: Paolo Bonzini
QED uses vm_clock timers so that images are not touched during and after
migration. This however does not apply to qemu-io and qemu-img.
Treat vm_clock as a synonym for rt_clock there, and enable it.
Signed-off-by: Paolo Bonzini
Signed-off-by: Kevin Wolf
---
qemu-tool.c |
On Fri, Apr 20, 2012 at 10:36:38AM -0300, Luiz Capitulino wrote:
> On Fri, 20 Apr 2012 14:07:16 +0200
> Michal Privoznik wrote:
>
> > >> But, I think if we tell users we'll *only* send response on error,
> > >> we should do our part to *not* send the responses, rather than relying
> > >> on them
On 20 April 2012 18:01, Andreas Färber wrote:
> Am 14.04.2012 18:42, schrieb Peter Maydell:
>> @@ -213,14 +207,17 @@ static void highbank_init(ram_addr_t ram_size,
>> }
>>
>> for (n = 0; n < smp_cpus; n++) {
>> + ARMCPU *cpu;
>> env = cpu_init(cpu_model);
>> if (
Someone forgot something in commit 29c1a730... Documenting the right
return value is not enough, you also need to actually return it in the
code.
This bug sometimes causes error return values even when everything has
succeeded: The new offset of the refcount block is truncated to 32 bits
and inter
Am 14.04.2012 18:42, schrieb Peter Maydell:
> Move the A9 config_base_address cp15 register reset value to
> ARMCPU. This should become a QOM property so that the Highbank
> board can set it without having to pull in cpu-qom.h, but at
> least this avoids the implicit dependency on reset ordering
>
If the BlockDriverState is closed/freed without draining the AIO
requests first, the request coroutines may work on invalid data and file
descriptors or have some dangling pointers that cause segfaults.
Signed-off-by: Kevin Wolf
Reviewed-by: Paolo Bonzini
---
tests/qemu-iotests/032 | 69 +
The following changes since commit 51006bbc45bc74977ae538190a53df2af534acb9:
Merge remote-tracking branch 'origin/master' into staging (2012-04-18
10:06:09 -0500)
are available in the git repository at:
git://repo.or.cz/qemu/kevin.git for-anthony
Andreas Färber (1):
iotests: Resolve
If an AIO request is in flight that refers to a BlockDriverState that
has been closed and possibly even freed, more or less anything could
happen. I have seen segfaults, -EBADF return values and qcow2 sometimes
actually catches the situation in bdrv_close() and abort()s.
Signed-off-by: Kevin Wolf
Am 14.04.2012 18:42, schrieb Peter Maydell:
> Now that cpu_reset_model_id() has gone we can move the
> reset code over to the class reset function and have cpu_state_reset
> simply do a reset on the CPU QOM object.
>
> Signed-off-by: Peter Maydell
Acked-by: Andreas Färber
/-F
--
SUSE LINUX P
On 20.4.2012 12:17, Jakub Jermar wrote:
> I am observing an interesting phenomenon on the latest QEMU git and the
> latest HelenOS mainline (the problem is likely guest-neutral though).
> The QEMU command line is as follows:
>
> $ qemu-system-i386 -device rtl8139,vlan=0 -net user -boot d -redir \
Hi,
I am observing an interesting phenomenon on the latest QEMU git and the
latest HelenOS mainline (the problem is likely guest-neutral though).
The QEMU command line is as follows:
$ qemu-system-i386 -device rtl8139,vlan=0 -net user -boot d -redir \
udp:8080::8080 -redir udp:8081::8081 -redir
On Thu, 19 Apr 2012 14:59:25 -0500, Aaron Stover (Celestech)
wrote:
Hi,
I'm curious as to the current status of the fault tolerance features of
QEMU? From reading what I could find on the wiki, it appears as though
the code was ready to be merged into version 0.14 but it does appear to
On Fri, Apr 20, 2012 at 12:33:57PM +0200, Igor Mammedov wrote:
> On 04/19/2012 04:08 PM, Vasilis Liaskovitis wrote:
> >-numa_fw_cfg = g_malloc0((1 + max_cpus + nb_numa_nodes) * 8);
> >+numa_fw_cfg = g_malloc0((2 + max_cpus + nb_numa_nodes + 3 *
> >nb_hp_memslots) * 8);
> > numa_fw_cfg
Am 20.04.2012 18:05, schrieb Peter Maydell:
> On 20 April 2012 17:00, Andreas Färber wrote:
>> Am 14.04.2012 18:42, schrieb Peter Maydell:
>>> +cpu->clidr = (1 << 27) | (2 << 24) | 3;
>>
>> Copy&paste, should be (1 << 27) | (1 << 24) | 3.
>
> Fixed and pushed, sigh.
Acked-by: Andreas Färber
On 20 April 2012 17:14, Andreas Färber wrote:
> Am 14.04.2012 18:42, schrieb Peter Maydell:
>> /* TODO Move contents into arm_cpu_reset() in cpu.c,
>> * once cpu_reset_model_id() is eliminated,
>
> ...if you were to drop this comment line it would be cleaner. There is
> no cpu_reset_model_
Il 20/04/2012 12:03, Stefan Weil ha scritto:
> These patches remove timer code which is no longer needed
> and try to improve the remaining code.
>
> [PATCH 1/5] qemu-timer: Remove redundant include statements
> [PATCH 2/5] qemu-timer: Remove unused function qemu_alarm_pending
> [PATCH 3/5] qemu-t
Am 14.04.2012 18:42, schrieb Peter Maydell:
> cpu_reset_model_id() is now empty and we can remove it.
>
> Signed-off-by: Peter Maydell
Acked-by: Andreas Färber
However, ...
> diff --git a/target-arm/helper.c b/target-arm/helper.c
> index 5cbc7e0..653885a 100644
> --- a/target-arm/helper.c
> +
On 20 April 2012 17:00, Andreas Färber wrote:
> Am 14.04.2012 18:42, schrieb Peter Maydell:
>> + cpu->clidr = (1 << 27) | (2 << 24) | 3;
>
> Copy&paste, should be (1 << 27) | (1 << 24) | 3.
Fixed and pushed, sigh.
-- PMM
Am 14.04.2012 18:42, schrieb Peter Maydell:
> Move cache ID register reset out of cpu_reset_model_id() by
> creating a field for the reset value in ARMCPU and setting it
> up in the cpu specific init functions.
>
> Signed-off-by: Peter Maydell
> ---
> target-arm/cpu-qom.h |5 +
> target-
On 20 April 2012 16:50, Andreas Färber wrote:
> Missing cpu->id_mmfr3 = 0x11;
[etc]
Whoops, that was a bit sloppy. Fixed and new version pushed
to git://git.linaro.org/people/pmaydell/qemu-arm.git drop-reset-model-id
-- PMM
Am 14.04.2012 18:42, schrieb Peter Maydell:
> Move feature register value setup to per-CPU init functions.
>
> Signed-off-by: Peter Maydell
> ---
> target-arm/cpu-qom.h | 14
> target-arm/cpu.c | 91
> ++
> target-arm/helper.c |
Public bug reported:
Looks like there is a simple misspelling of the env parameter, as "emv",
in hw/spapr_hcall.c:485:
static target_ulong deregister_dtl(CPUPPCState *emv, target_ulong
addr)
** Affects: qemu
Importance: Undecided
Status: New
--
You received this bug notificatio
Signed-off-by: Peter Portante
---
hw/spapr_hcall.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/hw/spapr_hcall.c b/hw/spapr_hcall.c
index 634763e..94bb504 100644
--- a/hw/spapr_hcall.c
+++ b/hw/spapr_hcall.c
@@ -482,7 +482,7 @@ static target_ulong register_dtl(CPUPPCS
On Fri, Apr 20, 2012 at 05:19:17PM +0200, Jan Kiszka wrote:
> On 2012-04-20 17:00, Eduardo Habkost wrote:
> > On Fri, Apr 20, 2012 at 12:12:38PM +0200, Jan Kiszka wrote:
> >> On 2012-04-19 22:03, Eduardo Habkost wrote:
> >>> Jan/Avi: ping?
> >>>
> >>> I would like to get this ABI detail clarified s
1 - 100 of 188 matches
Mail list logo
