On 02/06/2013 01:45 PM, Kuo-Jung Su wrote:
From: Kuo-Jung Su
The FTDDRII030 is a DDRII SDRAM controller which is responsible for
SDRAM initialization.
In QEMU we simply emualte the SDRAM enable function, neither timing parameter
nor bank setup is handled.
Signed-off-by: Kuo-Jung Su
---
hw/a
On Wed, Feb 06, 2013 at 01:15:05PM -0700, Alex Williamson wrote:
> On Wed, 2013-02-06 at 14:13 -0500, Laine Stump wrote:
> > 2) Are there other issues aside from implicit controller devices I
> > need to consider for q35? For example, are there any devices that (as
> > I recall is the case for some
On Thu, Feb 07, 2013 at 04:22:45PM +0100, Stefan Hajnoczi wrote:
> On Thu, Feb 7, 2013 at 3:43 PM, Laszlo Ersek wrote:
> > Instead, what about
> >
> > #define endof(container, field) \
> > (offsetof(container, field) + sizeof ((container *)0)->field)
>
> As mentioned in my reply, I think endo
"Michael S. Tsirkin" writes:
> On Thu, Feb 07, 2013 at 09:45:54AM -0600, Anthony Liguori wrote:
>> "Michael S. Tsirkin" writes:
>>
>> This is about bug-for-bug compatibility with older QEMU.
>
> Sorry, with which version?
>
> It looks like if I run qemu 1.3 with .status=off
> windows drivers wo
On Thu, 07 Feb 2013 18:07:01 +0100
Laszlo Ersek wrote:
> On 02/07/13 18:01, Eduardo Habkost wrote:
> > On Tue, Feb 05, 2013 at 09:39:15PM +0100, Laszlo Ersek wrote:
> >> Conversion status (call chains covered or substituted by error propagation
> >> marked with square brackets):
> >>
> >> do_devi
On Thu, 7 Feb 2013 15:04:54 -0200
Eduardo Habkost wrote:
> On Tue, Feb 05, 2013 at 09:39:16PM +0100, Laszlo Ersek wrote:
> > Error handling is not changed yet.
> >
> > Signed-off-by: Laszlo Ersek
>
> The extra parameter seems useless and even misleading (because Error
> info won't be set even
Hi Senthil,
On Mon, Feb 04, 2013 at 04:17:30PM +0530, kumaran wrote:
> Hi,
>
> I am Senthil, doing post graduation in IIT Bombay,India.
>
> I am looking for some problems related to KVM as part of my research
> work. I read about memory hotplug and other open issues in KVM.
>
> Can i be part of
On Thu, 7 Feb 2013 15:12:22 -0200
Eduardo Habkost wrote:
> On Tue, Feb 05, 2013 at 09:39:18PM +0100, Laszlo Ersek wrote:
> > Problems are now reported via Error only.
> >
> > Signed-off-by: Laszlo Ersek
> > ---
> > hw/qdev-properties.h |4 ++--
> > hw/qdev-monitor.c|3 ++-
> > hw/q
On 02/06/2013 05:31 AM, Benoît Canet wrote:
> This fix the sub cluster sized writes race conditions while waiting
s/fix/fixes/
> for a more faster solution.
s/more//
Would it be worth describing the race condition in more detail?
--
Eric Blake eblake redhat com+1-919-301-3266
Libvirt vi
Am 07.02.2013 18:46, schrieb Eduardo Habkost:
> On Thu, Feb 07, 2013 at 06:20:52PM +0100, Andreas Färber wrote:
>> Am 07.02.2013 17:40, schrieb Eduardo Habkost:
>>> On Sat, Feb 02, 2013 at 04:04:11PM +0100, Andreas Färber wrote:
In comparison to DeviceClass::vmsd, CPU VMState is split in two,
On Thu, 7 Feb 2013 15:18:41 -0200
Eduardo Habkost wrote:
> On Tue, Feb 05, 2013 at 09:39:19PM +0100, Laszlo Ersek wrote:
> >
> > Signed-off-by: Laszlo Ersek
> > ---
> > hw/qdev-monitor.c | 21 -
> > 1 files changed, 16 insertions(+), 5 deletions(-)
> >
> > diff --git a/h
On 02/06/2013 05:32 AM, Benoît Canet wrote:
> ---
> block/qcow2-dedup.c | 11 +++
> block/qcow2.h | 18 ++
> 2 files changed, 29 insertions(+)
>
> diff --git a/block/qcow2-dedup.c b/block/qcow2-dedup.c
> index 308b4f6..72ce1d7 100644
> --- a/block/qcow2-dedup.c
>
On Thu, 2013-02-07 at 17:31 +, Daniel P. Berrange wrote:
> On Wed, Feb 06, 2013 at 01:15:05PM -0700, Alex Williamson wrote:
> > On Wed, 2013-02-06 at 14:13 -0500, Laine Stump wrote:
> > > 2) Are there other issues aside from implicit controller devices I
> > > need to consider for q35? For exam
Am 07.02.2013 17:55, schrieb Eduardo Habkost:
> On Sat, Feb 02, 2013 at 04:04:12PM +0100, Andreas Färber wrote:
>> Expose vmstate_cpu as vmstate_x86_cpu and hook it up to CPUClass::vmsd.
>> Adapt opaques and VMState fields to X86CPU. Drop cpu_{save,load}().
>>
>> Signed-off-by: Andreas Färber
>> -
On 02/07/2013 09:15 AM, Stefan Hajnoczi wrote:
> Applies on kevin/block-next (2700d627e4d73bb2aacb832a93ffac8fec78b1fd).
>
> This series adds fragmentation info support to qcow2 and then adds compression
> info. Previously only QED supported fragmentation info. Note that
> fragmentation info onl
On Thu, Feb 07, 2013 at 07:02:32PM +0100, Andreas Färber wrote:
> Am 07.02.2013 18:46, schrieb Eduardo Habkost:
> > On Thu, Feb 07, 2013 at 06:20:52PM +0100, Andreas Färber wrote:
> >> Am 07.02.2013 17:40, schrieb Eduardo Habkost:
> >>> On Sat, Feb 02, 2013 at 04:04:11PM +0100, Andreas Färber wrote
On Sun, Jan 20, 2013 at 08:22:30AM +0100, Andreas Färber wrote:
> Introduce realizefn and set realized = true from cpu_cris_init().
>
> Signed-off-by: Andreas Färber
Reviewed-by: Eduardo Habkost
> ---
> target-cris/cpu-qom.h |2 ++
> target-cris/cpu.c | 15 +++
> tar
Am 07.02.2013 19:22, schrieb Eduardo Habkost:
> On Thu, Feb 07, 2013 at 07:02:32PM +0100, Andreas Färber wrote:
>> Am 07.02.2013 18:46, schrieb Eduardo Habkost:
>>> On Thu, Feb 07, 2013 at 06:20:52PM +0100, Andreas Färber wrote:
Am 07.02.2013 17:40, schrieb Eduardo Habkost:
> On Sat, Feb 0
Laszlo Ersek writes:
> On 02/07/13 09:55, Michael S. Tsirkin wrote:
>> On Tue, Feb 05, 2013 at 05:47:16PM -0600, Jesse Larrew wrote:
>>> Currently, the config size for virtio devices is hard coded. When a new
>>> feature is added that changes the config size, drivers that assume a static
>>> conf
On Tue, Feb 05, 2013 at 09:39:15PM +0100, Laszlo Ersek wrote:
> Conversion status (call chains covered or substituted by error propagation
> marked with square brackets):
>
> do_device_add -> [qemu_find_opts -> error_report]
> do_device_add -> qdev_device_add -> qerror_report
> do_device_add -> qd
Stefan Hajnoczi writes:
> On Thu, Feb 7, 2013 at 3:43 PM, Laszlo Ersek wrote:
>> Instead, what about
>>
>> #define endof(container, field) \
>> (offsetof(container, field) + sizeof ((container *)0)->field)
>
> As mentioned in my reply, I think endof() isn't necessary.
>
> Just use offsetof()
The check_refcounts_l1/l2() functions have a check_copied argument to
check that the QCOW_O_COPIED flag is consistent with refcount == 1.
This should be a bool, not an int.
However, the next patch introduces qcow2 fragmentation statistics and
also needs to pass an option to check_refcounts_l1/l2()
From: Petar Jovanovic
Correct sign-propagation before multiplication in MULQ_W helper.
The change also fixes previously incorrect expected values in the
tests for MULQ_RS.W and MULQ_S.W.
Signed-off-by: Petar Jovanovic
---
target-mips/dsp_helper.c|2 +-
tests/tcg/mips/mips32
On Tue, Feb 05, 2013 at 09:39:14PM +0100, Laszlo Ersek wrote:
>
> Signed-off-by: Laszlo Ersek
Reviewed-by: Eduardo Habkost
> ---
> hw/qdev-monitor.c |2 +-
> vl.c |8
> 2 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/hw/qdev-monitor.c b/hw/qdev
Am 02.02.2013 16:04, schrieb Andreas Färber:
> Hello,
>
> In response to patches from Juan [1], this series implements a declarative way
> for CPUs to register their CPU-specific VMStateDescription.
Ping? OK to go ahead with patches 1-3? Any thoughts on how to proceed
with the not-yet-migratable
Am 07.02.2013 17:31, schrieb Eduardo Habkost:
> On Sun, Jan 20, 2013 at 08:22:26AM +0100, Andreas Färber wrote:
>> Turn arm_cpu_realize() into a QOM realize function, no longer called
>> via cpu.h prototype. To maintain the semantics of cpu_init(), set
>> realized = true explicitly in cpu_arm_init(
Am 05.02.2013 14:02, schrieb Jeff Cody:
Hi Kevin,
I can verify the issue he is seeing (see my comments on Bug 1105670
(https://bugs.launchpad.net/qemu/+bug/1105670/comments/3), at least
using Hyper-V. I haven't tried it against VPC from Win7.
Jeff
Hi Kevin,
a rebased version of this patch
On Tue, 5 Feb 2013 21:39:20 +0100
Laszlo Ersek wrote:
> The return type can't be changed to void, because "set_property" must have
> type "qemu_opt_loopfunc".
>
> Conversion status (call chains covered or substituted by error propagation
> marked with square brackets):
This is fine to be squas
On 7 February 2013 16:08, Stefan Hajnoczi wrote:
> On Thu, Feb 7, 2013 at 3:12 PM, harryxiyou wrote:
>> The above debug info show jni will be used in coroutine. I do not
>> familay with coroutine, however I guess if coroutine not allow to
>> spawn a new thread? ,or it's stack corrupte with java j
So, this SDHCI implementation is working and it couldn't lock-up QEMU,
It'd be really great to have it merged during next development window.
Any issues we have we can address later.
On 02/07/2013 10:50 AM, Peter Crosthwaite wrote:
[Original cover by Igor]
First patch introduces standard SD
On 7 February 2013 18:44, Andreas Färber wrote:
> Am 07.02.2013 17:31, schrieb Eduardo Habkost:
>> My first question I had when I saw this was: are you really sure it is
>> safe to call cpu_reset() and qemu_init_vcpu() before
>> arm_translate_init()?
>>
>> But I see that you change this on commit
On 7 February 2013 18:57, Igor Mitsyanko wrote:
> So, this SDHCI implementation is working and it couldn't lock-up QEMU, It'd
> be really great to have it merged during next development window. Any issues
> we have we can address later.
Well, our quality bar should be higher than "it doesn't hang
From: Stefan Weil
The size calculated from the CHS values is not the real image (disk) size,
but usually a smaller value. This is caused by rounding effects.
Only older operating systems use CHS. Such guests won't be able to use
the whole disk. All modern operating systems use the real size.
Th
Clarify the definition of endof() by using the offsetof() and sizeof()
macros to handle the pointer magic.
Signed-off-by: Jesse Larrew
---
hw/virtio-net.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/virtio-net.c b/hw/virtio-net.c
index 8f521b3..663f33c 100644
--- a/hw/
On Thu, Feb 07, 2013 at 11:46:55AM -0600, Anthony Liguori wrote:
> "Michael S. Tsirkin" writes:
>
> > On Thu, Feb 07, 2013 at 09:45:54AM -0600, Anthony Liguori wrote:
> >> "Michael S. Tsirkin" writes:
> >>
> >> This is about bug-for-bug compatibility with older QEMU.
> >
> > Sorry, with which v
On 02/06/2013 05:32 AM, Benoît Canet wrote:
> Signed-off-by: Benoit Canet
> ---
> tests/qemu-iotests/common.rc |3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tests/qemu-iotests/common.rc b/tests/qemu-iotests/common.rc
> index aef5f52..72e746d 100644
> --- a/tests/q
"Michael S. Tsirkin" writes:
> On Thu, Feb 07, 2013 at 11:46:55AM -0600, Anthony Liguori wrote:
>
> So why do we add extra code who's sole effect is breaking old
> windows drivers?
A little dramatic, no?
> I like the idea but why add code for status and mac features?
Because it's correct and t
On Thu, Feb 07, 2013 at 06:20:52PM +0100, Andreas Färber wrote:
> Am 07.02.2013 17:40, schrieb Eduardo Habkost:
> > On Sat, Feb 02, 2013 at 04:04:11PM +0100, Andreas Färber wrote:
> >> In comparison to DeviceClass::vmsd, CPU VMState is split in two,
> >> "cpu_common" and "cpu", and uses cpu_index a
Since the new config_size member of struct VirtIONet specifies a size, it
should be declared as size_t.
Signed-off-by: Jesse Larrew
---
hw/virtio-net.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/virtio-net.c b/hw/virtio-net.c
index 663f33c..573c669 100644
--- a/hw/vir
On Thu, Feb 07, 2013 at 03:15:42PM -0600, Anthony Liguori wrote:
> "Michael S. Tsirkin" writes:
>
> > On Thu, Feb 07, 2013 at 11:46:55AM -0600, Anthony Liguori wrote:
> >
> > So why do we add extra code who's sole effect is breaking old
> > windows drivers?
>
> A little dramatic, no?
>
> > I li
Hello Klaus, or anyone else affected,
Accepted qemu-kvm into quantal-proposed. The package will build now and
be available at http://launchpad.net/ubuntu/+source/qemu-kvm/1.2.0
+noroms-0ubuntu2.12.10.3 in a few hours, and then in the -proposed
repository.
Please help us by testing this new packag
On Thu, Feb 7, 2013 at 11:13 AM, Kang Hua wrote:
> HI all :
> I write a new block driver based on hdfs of hadoop, it need to connect
> to hdfs when create step (called by bdrv_create in block.c) by use libhdfs (
> hdfs c++ api) 。
Please don't send duplicate emails to the list.
Stefan
Hello,
there was a proposed patch by Paolo Bonzini some time ago about adding an
option to qemu to disable tcg:
https://lists.gnu.org/archive/html/qemu-devel/2012-09/msg02568.html
Is there some plan to integrate it?
Paolo, do you have a newer version that applies cleanly to mainline?
Thanks,
On Thu, Feb 7, 2013 at 4:55 PM, Anthony Liguori wrote:
> Stefan Hajnoczi writes:
>
>> On Thu, Feb 7, 2013 at 3:43 PM, Laszlo Ersek wrote:
>>> Instead, what about
>>>
>>> #define endof(container, field) \
>>> (offsetof(container, field) + sizeof ((container *)0)->field)
>>
>> As mentioned in
On Sun, Jan 20, 2013 at 08:22:29AM +0100, Andreas Färber wrote:
> Adapt ppc_cpu_realize() signature, hook it up to DeviceClass and set
> realized = true in cpu_ppc_init().
>
> Signed-off-by: Andreas Färber
Reviewed-by: Eduardo Habkost
> ---
> target-ppc/cpu-qom.h|2 ++
> target-p
"Michael S. Tsirkin" writes:
> On Tue, Feb 05, 2013 at 05:47:16PM -0600, Jesse Larrew wrote:
>> Currently, the config size for virtio devices is hard coded. When a new
>> feature is added that changes the config size, drivers that assume a static
>> config size will break. For purposes of backwar
On Thu, Feb 7, 2013 at 10:13 PM, Andreas Färber wrote:
> Am 07.02.2013 13:04, schrieb Peter Maydell:
>> On 7 February 2013 11:50, Andreas Färber wrote:
>>> Am 07.02.2013 07:59, schrieb Kuo-Jung Su:
>> +/**
>> + *
This patch set includes some minor cleanups requested by the community for
the virtio-net config size solution.
[PATCH 1/2] hw/virtio-net: use existing macros to implement endof
[PATCH 2/2] hw/virtio-net: fix config_size data type
On 02/06/2013 05:32 AM, Benoît Canet wrote:
> ---
> blockdev.c | 20
> qapi-schema.json | 19 +++
> qmp-commands.hx | 25 +
> 3 files changed, 64 insertions(+)
>
> +++ b/qapi-schema.json
> @@ -812,6 +812,25 @@
> ##
> { '
On Thu, Feb 7, 2013 at 6:56 PM, Stefan Hajnoczi wrote:
> On Thu, Feb 07, 2013 at 04:22:02PM +1000, Peter Crosthwaite wrote:
>> diff --git a/util/hexdump.c b/util/hexdump.c
>> new file mode 100644
>> index 000..38d64ca
>> --- /dev/null
>> +++ b/util/hexdump.c
>> @@ -0,0 +1,23 @@
>
> License/cop
On Wed, Jan 23, 2013 at 03:19:22PM +0800, Hu Tao wrote:
> This patch enables preservation of cpu runstate during save/load vm.
> So when a vm is restored from snapshot, the cpu runstate is restored,
> too.
>
> See following example:
>
> # save two vms: one is running, the other is paused
> (qemu)
On Thu, Feb 7, 2013 at 9:24 PM, Andreas Färber wrote:
> Am 07.02.2013 07:08, schrieb Peter Crosthwaite:
>> These two patches add and use dtc as a submodule as per the RFC:
>>
>> http://lists.gnu.org/archive/html/qemu-devel/2013-01/msg05000.html
>>
>> There is a remaining action item to mandate lib
Hi,
On Wed, Jan 23, 2013 at 03:19:21PM +0800, Hu Tao wrote:
> We can know the guest is panicked when the guest runs on xen.
> But we do not have such feature on kvm.
>
> Another purpose of this feature is: management app(for example:
> libvirt) can do auto dump when the guest is panicked. If mana
On Thu, Feb 7, 2013 at 8:40 PM, Andreas Färber wrote:
> Am 07.02.2013 06:51, schrieb Peter Crosthwaite:
>> Currently, M25P80 uses an object property to differentiate between flash
>> parts.
>> Changed this over to use QOM sub-classes - the actual names of the different
>> parts
>> are used to cr
On Wed, Jan 23, 2013 at 03:19:23PM +0800, Hu Tao wrote:
> From: Wen Congyang
>
> The guest should run after resetting it, but it does not run if its
> old state is RUN_STATE_INTERNAL_ERROR or RUN_STATE_PAUSED.
>
> We don't set runstate to RUN_STATE_PAUSED when resetting the guest,
> so the runst
These patches add support for the Primcell PL330 DMA controller and add it to
the Xilinx Zynq machine model. Patch 2 is the device model. Patch 3 is the
machine model update. Patch 1 is a helper function for debug mode.
The Device model was originally contributed by Kirill Batuzov / Samsung, as
Factor out the hexdumper functionality from iov for all to use. Useful for
creating verbose debug printfery that dumps packet data.
Signed-off-by: Peter Crosthwaite
---
include/qemu/hexdump.h |3 +++
util/Makefile.objs |1 +
util/hexdump.c | 45
Device model for Primecell PL330 dma controller.
Signed-off-by: Peter Crosthwaite
Signed-off-by: Kirill Batuzov
Tested-by: Igor Mitsyanko
---
changed from v9:
s/sysbus_from_qdev/SYS_BUS_DEVICE (rebase)
Fixed include names (rebase)
Reworked debug instrumentation
Added level 2 debug hexdumper (du
Signed-off-by: Peter Crosthwaite
---
hw/xilinx_zynq.c | 24
1 files changed, 24 insertions(+), 0 deletions(-)
diff --git a/hw/xilinx_zynq.c b/hw/xilinx_zynq.c
index 0ac33b5..f826bff 100644
--- a/hw/xilinx_zynq.c
+++ b/hw/xilinx_zynq.c
@@ -35,6 +35,10 @@
#define IRQ
Patches 1-3 and 5 are trivial code cleanup and may be candidate to go via
trivial queue inpdependent of review of this work. Patch 4 unifies the MPTimer
and WDT into one device. My motivation for doing this is so I can dynamically
create one without the other (Im throwing away MPCore altogether
Trivial find replace on type names "timerblock" and "arm_mptimer_state" to
conform with QEMU coding style.
Signed-off-by: Peter Crosthwaite
---
hw/arm_mptimer.c | 56 +++---
1 files changed, 28 insertions(+), 28 deletions(-)
diff --git a/hw/arm
Split the SCU in a9mpcore out into its own object definition. mpcore is now
just a container for the mpcore components.
Signed-off-by: Peter Crosthwaite
---
hw/a9mpcore.c| 119 +++--
hw/a9scu.c | 162 +++
The DeviceState *mptimer var in a9mp_priv_state was only used by the init
function and had no reason for persistence. Made a local variable and removed
from state struct.
Signed-off-by: Peter Crosthwaite
---
hw/a9mpcore.c | 10 +-
1 files changed, 5 insertions(+), 5 deletions(-)
diff
In QEMU emulation, there is no functional difference between the ARM mpcore
private timers and watchdogs. Removed all the distinction between the two from
arm_mptimer.c and converted it to be just the mptimer. a9mpcore and arm11mpcore
just instantiate the same mptimer object twice to get both timer
To conform with QEMU coding style.
Signed-off-by: Peter Crosthwaite
---
hw/a9mpcore.c| 26 +-
hw/arm11mpcore.c | 20 ++--
2 files changed, 23 insertions(+), 23 deletions(-)
diff --git a/hw/a9mpcore.c b/hw/a9mpcore.c
index 1f6c985..6ad7c6d 100644
This field was write only and thus unused. Removed.
Signed-off-by: Peter Crosthwaite
---
hw/a9mpcore.c | 11 +++
1 files changed, 3 insertions(+), 8 deletions(-)
diff --git a/hw/a9mpcore.c b/hw/a9mpcore.c
index 6520c24..4523074 100644
--- a/hw/a9mpcore.c
+++ b/hw/a9mpcore.c
@@ -16,7
On Fri, Feb 8, 2013 at 12:11 AM, Peter Maydell wrote:
> On 7 February 2013 16:08, Stefan Hajnoczi wrote:
>> On Thu, Feb 7, 2013 at 3:12 PM, harryxiyou wrote:
>>> The above debug info show jni will be used in coroutine. I do not
>>> familay with coroutine, however I guess if coroutine not allow t
On Thu, Feb 07, 2013 at 11:26:03AM +0200, Michael S. Tsirkin wrote:
> mq feature is not needed: we can look at the number of queues and set
> the flag accordingly.
> Removing this feature removes ambiguity (what does it mean to have
> queues=2 with mq=off?).
>
> Changes from v1:
> - split the patc
101 - 168 of 168 matches
Mail list logo
