On Fri, May 16, 2014 at 12:00 PM, Peter Crosthwaite
peter.crosthwa...@xilinx.com wrote:
Dynamically allocate Memory Region pointers and set them up as QOM
links. Plug these dynamic links to the sysbus_init_mmio() and
sysbus_mmio_get_region APIs.
This allows for removal of the Sysbus Memory
To Stefan:
yes, I can reproduce this on qemu.git/master.
Actually, I've found the cause of this bug and sent a patch to qemu-devel
mailing list a few days ago:
http://lists.nongnu.org/archive/html/qemu-devel/2014-05/msg05777.html
It's very kind of you helping review the patch and give some
Il 28/05/2014 18:53, Fabio Fantoni ha scritto:
2014-05-28 18:41 GMT+02:00 Stefano Stabellini
stefano.stabell...@eu.citrix.com
mailto:stefano.stabell...@eu.citrix.com:
On Wed, 28 May 2014, Stefano Stabellini wrote:
On Wed, 28 May 2014, Fabio Fantoni wrote:
Il 26/05/2014 10:00,
Hi all
I'm new to the list, and recently I'm digging in Qemu's source code, I've
got something confused me much, simply list the items:
1. Any benchmarks paying attention to TCG code generate quality
measured by code expansion ratio? Of course I've got some news said that the
ratio
Il 29/05/2014 05:23, Chunyan Liu ha scritto:
[support xen HVM direct kernel boot]
qemu side patch: if -kernel exists, calls xen_load_linux(), which
will read kernel/initrd and add a linuxboot.bin or multiboot.bin
option rom. The linuxboot.bin/multiboot.bin will load kernel/initrd
and jump to
On Fri, May 23, 2014 at 1:40 AM, Stefan Hajnoczi stefa...@redhat.com wrote:
Sometimes an object needs to present a property which is actually on
another object, or it needs to provide an alias name for an existing
property.
Examples:
a.foo - b.foo
a.old_name - a.new_name
The new
On Fri, May 23, 2014 at 1:40 AM, Stefan Hajnoczi stefa...@redhat.com wrote:
It becomes unwiedly to duplicate all virtio-blk qdev property
definitions due to an #ifdef. The C preprocessor syntax makes it a
little hard to resolve this cleanly but we can extract the #ifdef and
call a macro it
On Fri, May 23, 2014 at 1:40 AM, Stefan Hajnoczi stefa...@redhat.com wrote:
Move the x-data-plane property. Originally it was outside since not
every transport may wish to support dataplane. But that makes little
sense when we have a dedicated CONFIG_VIRTIO_BLK_DATA_PLANE ifdef
already.
On Fri, May 23, 2014 at 1:40 AM, Stefan Hajnoczi stefa...@redhat.com wrote:
virtio-blk-pci, virtio-blk-s390, and virtio-blk-ccw all duplicate the
qdev properties of their VirtIOBlock child. This approach does not work
well with string or pointer properties since we must be careful about
On Fri, May 23, 2014 at 1:40 AM, Stefan Hajnoczi stefa...@redhat.com wrote:
This function is no longer used since parent objects now use child
aliases to set the VirtIOBlkConf directly.
Signed-off-by: Stefan Hajnoczi stefa...@redhat.com
Reviewed-by: Peter Crosthwaite
On 29 May 2014 08:58, Chaos Shu chaos.s...@live.com wrote:
1. Any benchmarks paying attention to TCG code generate quality
measured by code expansion ratio? Of course I’ve got some news said that the
ratio maybe 4 or 5 in X86 to MIPS, that is to say 1 x86 insn to 4 or 5 mips
insns, Does
On Fri, May 23, 2014 at 1:40 AM, Stefan Hajnoczi stefa...@redhat.com wrote:
object_initialize() leaves the object with a refcount of 1.
object_property_add_child() adds its own reference which is dropped
again when the property is deleted.
The upshot of this is that we always have a refcount
On Fri, May 23, 2014 at 1:40 AM, Stefan Hajnoczi stefa...@redhat.com wrote:
There is no need to make DEFINE_VIRTIO_BLK_PROPERTIES() public. Inline
it into virtio-blk.c so it cannot be used by mistake from other source
files.
Signed-off-by: Stefan Hajnoczi stefa...@redhat.com
Reviewed-by:
On Tue, 27 May 2014 17:01:38 +0200
Paolo Bonzini pbonz...@redhat.com wrote:
Il 14/05/2014 17:42, Greg Kurz ha scritto:
+{ .name = virtio/is_big_endian,
+ .version_id = 1,
+ .save = virtio_save_device_endian,
+ .load = virtio_load_device_endian,
+},
{ .name
On Wed, 28 May 2014 18:38:13 +0200
Vasilis Liaskovitis vasilis.liaskovi...@profitbricks.com wrote:
On Wed, May 28, 2014 at 03:26:42PM +0200, Igor Mammedov wrote:
On Wed, 28 May 2014 14:23:13 +0200
Vasilis Liaskovitis vasilis.liaskovi...@profitbricks.com wrote:
On Wed, May 28, 2014 at
Hi Andreas, Edgar,
Another 4 users of SysBusDevice::init bite the dust!
A round of styling cleanup for Microblaze devices as per QOM
conventions. Tested with published MB test images.
Regards,
Peter
Peter Crosthwaite (6):
timer: xilinx_timer: Convert to realize()
net: xilinx_ethlite:
Hi Andreas, Edgar,
Another 4 users of SysBusDevice::init bite the dust!
A round of styling cleanup for Microblaze devices as per QOM
conventions. Tested with published MB test images.
Regards,
Peter
Peter Crosthwaite (6):
timer: xilinx_timer: Convert to realize()
net: xilinx_ethlite:
This zeroing-out of the rxbuf variable (ping pong state) is a reset
side effect. Extract into a proper reset.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
---
hw/net/xilinx_ethlite.c | 9 -
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git
SysBusDevice::init is depracated. Convert to Object::init and
Device::realize as prescribed by QOM conventions.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
---
hw/timer/xilinx_timer.c | 20
1 file changed, 12 insertions(+), 8 deletions(-)
diff --git
SysBusDevice::init is depracated. Convert to Object::init and
Device::realize as prescribed by QOM conventions.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
---
hw/net/xilinx_ethlite.c | 25 ++---
1 file changed, 14 insertions(+), 11 deletions(-)
diff --git
This refresh of the device state is intended to be a reset side
effect. Move it to a proper reset handler rather than do it at
init time.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
---
hw/char/xilinx_uartlite.c | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
SysBusDevice::init is depracated. Convert to Object::init and
Device::realize as prescribed by QOM conventions.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
---
hw/char/xilinx_uartlite.c | 24 ++--
1 file changed, 14 insertions(+), 10 deletions(-)
diff
SysBusDevice::init is depracated. Convert to Object::init
as prescribed by QOM conventions.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
---
hw/intc/xilinx_intc.c | 17 +++--
1 file changed, 7 insertions(+), 10 deletions(-)
diff --git a/hw/intc/xilinx_intc.c
Sorry about the dup mail.
Sent this one with the wrong SMTP server then aborted it. Proper send on list.
Regards,
Peter
On Thu, May 29, 2014 at 7:21 PM, Peter Crosthwaite
peter.crosthwa...@xilinx.com wrote:
Hi Andreas, Edgar,
Another 4 users of SysBusDevice::init bite the dust!
A round of
SysBusDevice::init is depracated. Convert to Object::init and
Device::realize as prescribed by QOM conventions.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
---
hw/timer/xilinx_timer.c | 20
1 file changed, 12 insertions(+), 8 deletions(-)
diff --git
This zeroing-out of the rxbuf variable (ping pong state) is a reset
side effect. Extract into a proper reset.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
---
hw/net/xilinx_ethlite.c | 9 -
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git
... fixes freeing constant from vl.c by machine_finalize()
Signed-off-by: Igor Mammedov imamm...@redhat.com
---
hw/core/machine.c |3 +++
vl.c |7 +--
2 files changed, 4 insertions(+), 6 deletions(-)
diff --git a/hw/core/machine.c b/hw/core/machine.c
index
On 29 May 2014 08:58, Chaos Shu chaos.s...@live.com wrote:
1. Any benchmarks paying attention to TCG code generate quality
measured by code expansion ratio? Of course I’ve got some news said
that the ratio maybe 4 or 5 in X86 to MIPS, that is to say 1 x86 insn
to 4 or 5 mips insns,
Il 29/05/2014 11:12, Greg Kurz ha scritto:
int virtio_load(VirtIODevice *vdev, QEMUFile *f)
{
[...]
nheads = vring_avail_idx(vdev-vq[i]) - vdev-vq[i].last_avail_idx;
^^^
/* Check it isn't doing very strange things with descriptor
numbers.
Il 29/05/2014 11:11, Peter Crosthwaite ha scritto:
object_initialize() leaves the object with a refcount of 1.
object_property_add_child() adds its own reference which is dropped
again when the property is deleted.
The upshot of this is that we always have a refcount = 1. Upon hot
unplug
On Thu, 2014-05-29 at 11:47 +0200, Igor Mammedov wrote:
... fixes freeing constant from vl.c by machine_finalize()
Nice cleanup, thanks!
Reviewed-by: Marcel Apfelbaum marce...@redhat.com
Signed-off-by: Igor Mammedov imamm...@redhat.com
---
hw/core/machine.c |3 +++
vl.c
Am 29.05.2014 11:47, schrieb Igor Mammedov:
... fixes freeing constant from vl.c by machine_finalize()
Signed-off-by: Igor Mammedov imamm...@redhat.com
Did you check whether there are any others in need of changes? I could
imagine kernel_irqchip does, and I see that we forgot to fix the
The zero_ethaddr[] array is never used; delete it.
Signed-off-by: Peter Maydell peter.mayd...@linaro.org
---
clang 3.4 warns about unused const variables like this and
also about unused functions; we have over 400 such warnings
currently, of which some are simple forgot to clean up
redundant code
The variables parallel_io and parallel_irq are unused; delete them.
Signed-off-by: Peter Maydell peter.mayd...@linaro.org
---
hw/i386/pc.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/hw/i386/pc.c b/hw/i386/pc.c
index e6369d5..32d1632 100644
--- a/hw/i386/pc.c
+++ b/hw/i386/pc.c
@@
Fix a typo in the ppce500_pci vmstate definition which meant that
we were migrating the struct pci_inbound using the vmstate for
pci_outbound. Fortunately the two structures have exactly the same
format at the moment (four uint32_ts) so this was harmless, and
we can correcting the typo without a
Peter Maydell peter.mayd...@linaro.org writes:
On 29 May 2014 08:58, Chaos Shu chaos.s...@live.com wrote:
1. Any benchmarks paying attention to TCG code generate quality
measured by code expansion ratio? Of course I’ve got some news said that the
ratio maybe 4 or 5 in X86 to MIPS, that
On Thu, 29 May 2014 12:47:45 +0200
Andreas Färber afaer...@suse.de wrote:
Am 29.05.2014 11:47, schrieb Igor Mammedov:
... fixes freeing constant from vl.c by machine_finalize()
Signed-off-by: Igor Mammedov imamm...@redhat.com
Did you check whether there are any others in need of
On 05/28/2014 07:15 PM, Paolo Bonzini wrote:
Using the file-backed smartcard backend is black magic, but it can
be useful if your only smartcard bricks itself if it is accessed
the wrong way too many times.
Complete the documentation to include the art of creating certificates
and using
On Thu, May 29, 2014 at 12:01:49PM +0100, Peter Maydell wrote:
The variables parallel_io and parallel_irq are unused; delete them.
Signed-off-by: Peter Maydell peter.mayd...@linaro.org
Thanks, applied.
Reviewed-by: Michael S. Tsirkin m...@redhat.com
---
hw/i386/pc.c | 3 ---
1 file
On Thu, 2014-05-29 at 12:47 +0200, Andreas Färber wrote:
Am 29.05.2014 11:47, schrieb Igor Mammedov:
... fixes freeing constant from vl.c by machine_finalize()
Signed-off-by: Igor Mammedov imamm...@redhat.com
Did you check whether there are any others in need of changes? I could
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu
---
scripts/tracetool/transform.py | 166
1 file changed, 166 insertions(+)
create mode 100644 scripts/tracetool/transform.py
diff --git a/scripts/tracetool/transform.py b/scripts/tracetool/transform.py
NOTE: TCG code for execution-time event tracing is always generated, regardless
of wether the event has been dynamically disabled or not (unless the event
has the static disable property).
This approach keeps this series simple, and a future series will handle
the case of
Transforms event:
tcg name(...) ..., ...
into two internal events:
tcg-trans name_trans(...) ...
tcg-exec name_exec(...) ...
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu
---
scripts/tracetool/__init__.py| 64 +++---
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu
---
scripts/tracetool/__init__.py | 30 +-
1 file changed, 29 insertions(+), 1 deletion(-)
diff --git a/scripts/tracetool/__init__.py b/scripts/tracetool/__init__.py
index e8e8edc..bd3fd85 100644
---
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu
---
docs/tracing.txt | 40
1 file changed, 40 insertions(+)
diff --git a/docs/tracing.txt b/docs/tracing.txt
index c6ab1c1..2e035a5 100644
--- a/docs/tracing.txt
+++ b/docs/tracing.txt
@@ -307,3 +307,43
Generate header trace/generated-helpers.h with the necessary TCG helper
declarations for tracing events in guest code at execution time:
* gen_helper_trace_${event}_exec
Routine to transform mixed native and TCG argument types to TCG types and call
TCG helper
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu
---
include/exec/helper-gen.h |3 +++
include/exec/helper-proto.h |1 +
include/exec/helper-tcg.h |1 +
3 files changed, 5 insertions(+)
diff --git a/include/exec/helper-gen.h b/include/exec/helper-gen.h
index a04a034..ba5a379
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu
---
target-alpha/translate.c |3 +++
target-arm/translate-a64.c|2 ++
target-arm/translate.c|3 +++
target-cris/translate.c |3 +++
target-i386/translate.c |3 +++
target-lm32/translate.c |
From: Grant Likely grant.lik...@linaro.org
Right now to run firmware inside the QEMU VExpress model requires
padding out the firmware image to the size of the virtual flash and
passing it in via the -pflash argument. If the firmware image is passed
without padding, then QEMU will fail. Also, when
Am 29.05.2014 14:21, schrieb Marcel Apfelbaum:
On Thu, 2014-05-29 at 12:47 +0200, Andreas Färber wrote:
Am 29.05.2014 11:47, schrieb Igor Mammedov:
... fixes freeing constant from vl.c by machine_finalize()
Signed-off-by: Igor Mammedov imamm...@redhat.com
Did you check whether there are any
On Thu, 29 May 2014 14:25:31 +0200
Andreas Färber afaer...@suse.de wrote:
Am 29.05.2014 14:21, schrieb Marcel Apfelbaum:
On Thu, 2014-05-29 at 12:47 +0200, Andreas Färber wrote:
Am 29.05.2014 11:47, schrieb Igor Mammedov:
... fixes freeing constant from vl.c by machine_finalize()
Generate file trace/generated-helpers.c with the necessary TCG helper routines
for tracing events in guest code at execution time:
* helper_trace_${event}_exec_proxy
TCG helper implementation to cast TCG-compatible argument types to native
types and call tracing routine
Alexey Kardashevskiy writes:
This replaces fprintf(stderr) with error_report.
This prints line number of the trace which does not exist or is not
traceable.
A little nit pick; it shows an error when some of the events in the list of
events to enable (not the trace) does not exist or is not
On Thu, May 29, 2014 at 10:24 PM, Peter Maydell
peter.mayd...@linaro.org wrote:
From: Grant Likely grant.lik...@linaro.org
Right now to run firmware inside the QEMU VExpress model requires
padding out the firmware image to the size of the virtual flash and
passing it in via the -pflash
On Thu, May 29, 2014 at 02:58:20AM +0200, Petar Jovanovic wrote:
From: Petar Jovanovic petar.jovano...@imgtec.com
From MIPS documentation (Volume III):
UserLocal Register (CP0 Register 4, Select 2)
Compliance Level: Recommended.
The UserLocal register is a read-write register that is
Otherwise the user has to explicitly include an auto-generated header.
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu
---
include/trace.h |1 +
1 file changed, 1 insertion(+)
diff --git a/include/trace.h b/include/trace.h
index c15f498..44a1f1f 100644
--- a/include/trace.h
+++
Am 29.05.2014 13:00, schrieb Peter Maydell:
The zero_ethaddr[] array is never used; delete it.
Signed-off-by: Peter Maydell peter.mayd...@linaro.org
---
clang 3.4 warns about unused const variables like this and
also about unused functions; we have over 400 such warnings
currently, of
On Wed, May 28, 2014 at 7:30 AM, Peter Maydell peter.mayd...@linaro.org wrote:
cpu64.c contains a reginfo list for the impdef registers on
the Cortex-A57; however we forgot to actually call define_arm_cp_regs(),
so it was sitting there doing nothing. Remedy this omission.
Signed-off-by: Peter
Generate header trace/generated-tcg-tracers.h with the necessary routines for
tracing events in guest code:
* trace_${event}_tcg
Convenience wrapper that calls the translation-time tracer
'trace_${event}_trans', and calls 'gen_helper_trace_${event}_exec to
generate the TCG code to later
On Thu, 2014-05-29 at 14:40 +0200, Igor Mammedov wrote:
On Thu, 29 May 2014 14:25:31 +0200
Andreas Färber afaer...@suse.de wrote:
Am 29.05.2014 14:21, schrieb Marcel Apfelbaum:
On Thu, 2014-05-29 at 12:47 +0200, Andreas Färber wrote:
Am 29.05.2014 11:47, schrieb Igor Mammedov:
...
While adding MIPS_HFLAG_HWRENA_ULR is a good idea for performance
reasons, I don't think we should add MIPS_HFLAG_CP0UL. This value is
always the same (true or false) during all the run time of the
qemu-system-mips binary, and thus we don't need to take care of code
generated with it being
On 29 May 2014 13:52, Stefan Weil s...@weilnetz.de wrote:
I can add some more:
* Missing 'static' attributes for local variables and functions
Those seem worth fixing.
* Use of 0 instead of NULL for pointers (do we want to fix those?)
I wouldn't bother personally.
There's also still a tail
On Thu, May 29, 2014 at 01:01:38PM +, Petar Jovanovic wrote:
While adding MIPS_HFLAG_HWRENA_ULR is a good idea for performance
reasons, I don't think we should add MIPS_HFLAG_CP0UL. This value is
always the same (true or false) during all the run time of the
qemu-system-mips binary,
On Thu, 29 May 2014 15:56:16 +0300
Marcel Apfelbaum marce...@redhat.com wrote:
On Thu, 2014-05-29 at 14:40 +0200, Igor Mammedov wrote:
On Thu, 29 May 2014 14:25:31 +0200
Andreas Färber afaer...@suse.de wrote:
Am 29.05.2014 14:21, schrieb Marcel Apfelbaum:
On Thu, 2014-05-29 at
On 28 May 2014 19:00, Sean Bruno sbr...@freebsd.org wrote:
From: Stacey Son s...@freebsd.org
Update FreeBSD system call numbers in freebsd/syscall_nr.h.
Signed-off bY: Stacey Son s...@freebsd.org
(stray uppercase Y)
Signed-off-by: Sean Bruno sbr...@freebsd.org
Reviewed-by: Ed Maste
On Thu, 2014-05-29 at 15:08 +0200, Igor Mammedov wrote:
On Thu, 29 May 2014 15:56:16 +0300
Marcel Apfelbaum marce...@redhat.com wrote:
On Thu, 2014-05-29 at 14:40 +0200, Igor Mammedov wrote:
On Thu, 29 May 2014 14:25:31 +0200
Andreas Färber afaer...@suse.de wrote:
Am 29.05.2014
On 29/05/14 14:08, Aurelien Jarno wrote:
On Thu, May 29, 2014 at 01:01:38PM +, Petar Jovanovic wrote:
While adding MIPS_HFLAG_HWRENA_ULR is a good idea for performance
reasons, I don't think we should add MIPS_HFLAG_CP0UL. This value is
always the same (true or false) during all the run
On Thu, 2014-05-29 at 14:25 +0200, Andreas Färber wrote:
Am 29.05.2014 14:21, schrieb Marcel Apfelbaum:
On Thu, 2014-05-29 at 12:47 +0200, Andreas Färber wrote:
Am 29.05.2014 11:47, schrieb Igor Mammedov:
... fixes freeing constant from vl.c by machine_finalize()
Signed-off-by: Igor
On 29 May 2014 14:22, James Hogan james.ho...@imgtec.com wrote:
loadvm/migration will write Config3, possibly with a different value if
the snapshot being loaded is an older one without userlocal support.
Migration load always happens with a freshly reset system,
so this isn't a problem.
Am 29.05.2014 15:21, schrieb Marcel Apfelbaum:
On Thu, 2014-05-29 at 15:08 +0200, Igor Mammedov wrote:
On Thu, 29 May 2014 15:56:16 +0300
Marcel Apfelbaum marce...@redhat.com wrote:
On Thu, 2014-05-29 at 14:40 +0200, Igor Mammedov wrote:
Maybe we can hack QemuOpts to do s/foo_moo/foo-moo/ ?
change as CP0_Config3 is a read-only register. If I am right, probably
the best is to check directly env-CP0_Config3.
If you take a look at v1 of the patch, that's what was done.
In the code review, this was marked as unacceptable because it
required [1] passing env within the translator.
[1]
Am 29.05.2014 15:35, schrieb Petar Jovanovic:
change as CP0_Config3 is a read-only register. If I am right, probably
the best is to check directly env-CP0_Config3.
If you take a look at v1 of the patch, that's what was done.
In the code review, this was marked as unacceptable because it
On Thu, May 29, 2014 at 01:35:17PM +, Petar Jovanovic wrote:
change as CP0_Config3 is a read-only register. If I am right, probably
the best is to check directly env-CP0_Config3.
If you take a look at v1 of the patch, that's what was done.
In the code review, this was marked as
On 29 May 2014 14:35, Petar Jovanovic petar.jovano...@imgtec.com wrote:
change as CP0_Config3 is a read-only register. If I am right, probably
the best is to check directly env-CP0_Config3.
If you take a look at v1 of the patch, that's what was done.
In the code review, this was marked as
On 5/28/2014 5:59 PM, Alexander Graf wrote:
On 28.05.14 22:34, Tom Musta wrote:
Look at ELF header to determine ABI version on PPC64. This is required
for executing the first instruction correctly. Also print correct machine
name in uname() system call.
Signed-off-by: Doug Kwan
On Tue, May 27, 2014 at 11:01 PM, Igor Mammedov imamm...@redhat.com wrote:
Provides framework for splitting host RAM allocation/
policies into a separate backend that could be used
by devices.
Initially only legacy RAM backend is provided, which
uses memory_region_init_ram() allocator and
On Tue, May 27, 2014 at 11:01 PM, Igor Mammedov imamm...@redhat.com wrote:
Add following parameters:
slots - total number of hotplug memory slots
maxmem - maximum possible memory
slots and maxmem should go in pair and maxmem should be greater
than mem for memory hotplug to be enabled.
On Thu, May 29, 2014 at 02:48:28PM +0100, Peter Maydell wrote:
On 29 May 2014 14:35, Petar Jovanovic petar.jovano...@imgtec.com wrote:
change as CP0_Config3 is a read-only register. If I am right, probably
the best is to check directly env-CP0_Config3.
If you take a look at v1 of the
This is a follow up to the patch series initiated by Doug Kwan
http://lists.nongnu.org/archive/html/qemu-devel/2014-05/msg01936.html
The resubmission of V4 cleans up some attribution issues (corrects Doug's
email address in the Signed-off-by and also cites Doug as the author
of the first three
From: Doug Kwan dougk...@google.com
Look at ELF header to determine ABI version on PPC64. This is required
for executing the first instruction correctly. Also print correct machine
name in uname() system call.
Signed-off-by: Doug Kwan dougk...@google.com
Signed-off-by: Tom Musta
From: Doug Kwan dougk...@google.com
Signed-off-by: Doug Kwan dougk...@google.com
Signed-off-by: Tom Musta tommu...@gmail.com
---
V3: Addressing comment from Peter Maydell. Adding libdecnumber enable.
configure |6 ++
1 files changed, 6 insertions(+), 0 deletions(-)
diff --git
The existing code does a check to ensure that a .bss region is properly
mmap'd. When additional mmap is required, the (guest) pages are also
validated. However, this code has a bug: when host page size is larger
than target page size, it is possible for the .bss pages to already be
(host) mapped
The size and register information are encoded into the reserve_info field
of CPU state in the store conditional translation code. Specifically, the
size is shifted left by 5 bits (see target-ppc/translate.c
gen_conditional_store).
The user-mode store conditional code erroneously extracts the
Some modern tool chains use VSX instructions. Therefore attempt to enable the
VSX MSR
bit by default, just like similar bits (FP, VEC, SPE, etc.).
Signed-off-by: Tom Musta tommu...@gmail.com
---
V3: new patch
target-ppc/translate_init.c |1 +
1 files changed, 1 insertions(+), 0
From: Doug Kwan dougk...@google.com
This allows running PPC64 little-endian in user mode if target is configured
that way. In PPC64 LE user mode we set MSR.LE during initialization.
Signed-off-by: Doug Kwan dougk...@google.com
Signed-off-by: Tom Musta tommu...@gmail.com
---
V2: Overhaul
On Tue, May 27, 2014 at 11:01 PM, Igor Mammedov imamm...@redhat.com wrote:
From: Vasilis Liaskovitis vasilis.liaskovi...@profitbricks.com
Each hotplug-able memory slot is a DimmDevice.
A hot-add operation for a DIMM:
- creates a new DimmDevice and makes hotplug controller to map it into
In order to avoid access to the CPUMIPSState structure in the
translator, keep a copy of CP0_Config1 into DisasContext. The whole
register is read-only so it can be copied as a single value.
Signed-off-by: Aurelien Jarno aurel...@aurel32.net
---
target-mips/translate.c | 20
On Thu, May 29, 2014 at 04:06:56PM +0200, Aurelien Jarno wrote:
On Thu, May 29, 2014 at 02:48:28PM +0100, Peter Maydell wrote:
On 29 May 2014 14:35, Petar Jovanovic petar.jovano...@imgtec.com wrote:
change as CP0_Config3 is a read-only register. If I am right, probably
the best is to
On 29/05/14 14:26, Peter Maydell wrote:
On 29 May 2014 14:22, James Hogan james.ho...@imgtec.com wrote:
loadvm/migration will write Config3, possibly with a different value if
the snapshot being loaded is an older one without userlocal support.
Migration load always happens with a freshly
On Tue, May 27, 2014 at 11:01 PM, Igor Mammedov imamm...@redhat.com wrote:
which allows to check if MemoryRegion is already mapped.
Signed-off-by: Igor Mammedov imamm...@redhat.com
---
include/exec/memory.h |8
memory.c | 15 ++-
2 files changed, 22
On Tue, May 27, 2014 at 11:00 PM, Igor Mammedov imamm...@redhat.com wrote:
NOTE to commiter:
* update precompiled ACPI hex files for iasl-less hosts
and ACPI tables test to match new ACPI tables.
What's new since v9:
* drop usage of error_is_set()
* exit QEMU if it's started with
CC'ing Drew, Eric and Michal.
On 05/29/14 14:45, Peter Crosthwaite wrote:
On Thu, May 29, 2014 at 10:24 PM, Peter Maydell
peter.mayd...@linaro.org wrote:
From: Grant Likely grant.lik...@linaro.org
Right now to run firmware inside the QEMU VExpress model requires
padding out the firmware
Il 29/05/2014 16:05, Peter Crosthwaite ha scritto:
+path = object_get_canonical_path_component(OBJECT(backend));
+memory_region_init_ram(backend-mr, OBJECT(backend), path,
Passing the full canonical path as the name of memory region is
redundant as that information is already passed
On Fri, May 30, 2014 at 12:46 AM, Laszlo Ersek ler...@redhat.com wrote:
CC'ing Drew, Eric and Michal.
On 05/29/14 14:45, Peter Crosthwaite wrote:
On Thu, May 29, 2014 at 10:24 PM, Peter Maydell
peter.mayd...@linaro.org wrote:
From: Grant Likely grant.lik...@linaro.org
Right now to run
Il 29/05/2014 16:22, Peter Crosthwaite ha scritto:
+bool memory_region_is_mapped(MemoryRegion *mr)
+{
Is it not enough to just return mr-parent? Memory mapping assertion
will happen if you try and map the same twice, even if one of the
mappings is not contained within an AddressSpace.
On 29 May 2014 15:29, James Hogan james.ho...@imgtec.com wrote:
On 29/05/14 14:26, Peter Maydell wrote:
On 29 May 2014 14:22, James Hogan james.ho...@imgtec.com wrote:
loadvm/migration will write Config3, possibly with a different value if
the snapshot being loaded is an older one without
On 29 May 2014 15:46, Laszlo Ersek ler...@redhat.com wrote:
(2) Can anyone post / link a concise, *up-to-date* command line for
upstream qemu, in connection with *upstream* edk2 build instructions, so
that I can test aarch64 UEFI stuff on an x86_64 host (with TCG)?
Note that AArch64 isn't
Il 29/05/2014 17:39, Peter Crosthwaite ha scritto:
It's not the full canonical path, it's basically the id.
Fair enough. Although its still redundant though isn't it?
Should we ever clean up messages etc in memory api to include both
owner and name this is going to come out:
foo/bar/baz:baz
On Fri, May 30, 2014 at 1:25 AM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 29/05/2014 16:05, Peter Crosthwaite ha scritto:
+path = object_get_canonical_path_component(OBJECT(backend));
+memory_region_init_ram(backend-mr, OBJECT(backend), path,
Passing the full canonical path as
Il 29/05/2014 15:26, Andreas Färber ha scritto:
Am 29.05.2014 15:21, schrieb Marcel Apfelbaum:
On Thu, 2014-05-29 at 15:08 +0200, Igor Mammedov wrote:
On Thu, 29 May 2014 15:56:16 +0300
Marcel Apfelbaum marce...@redhat.com wrote:
On Thu, 2014-05-29 at 14:40 +0200, Igor Mammedov wrote:
Maybe
1 - 100 of 172 matches
Mail list logo