From: Laurent Vivier
Signed-off-by: Laurent Vivier
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: David Gibson
---
hw/ppc/ppc4xx_pci.c | 13 +++--
hw/ppc/trace-events | 4
2 files changed, 7 insertions(+), 10 deletions(-)
diff --git a/hw/ppc/ppc4xx_pci.c b/hw/ppc/ppc4xx_pci
From: Peter Maydell
When performing clock calculations, the ppc405_uc code
has several places where it multiplies together two
32-bit variables and assigns the result to a 64-bit
variable. This doesn't quite do what is intended because
C will compute a 32-bit multiply result. Add casts to
ensure
From: Bharata B Rao
Implement float128_to_uint64() and use that to implement
float128_to_uint64_round_to_zero()
This is required by xscvqpudz instruction of PowerPC ISA 3.0.
Signed-off-by: Bharata B Rao
Reviewed-by: Peter Maydell
Signed-off-by: David Gibson
---
fpu/softfloat.c | 59
From: Suraj Jitindar Singh
The vpm0 bit was removed from the LPCR in POWER9, this bit controlled
whether ISI and DSI interrupts were directed to the hypervisor or the
partition. These interrupts now go to the hypervisor irrespective, thus
it is no longer necessary to check the vmp0 bit in the LPC
From: Igor Mammedov
Fill in CpuInstanceProperties once at board init time and
just copy them whenever query_hotpluggable_cpus() is called.
It will keep topology info always available without need
to recalculate it every time it's needed.
Considering it has NUMA node id, it will be used to keep
NU
From: Igor Mammedov
Signed-off-by: Igor Mammedov
Signed-off-by: David Gibson
---
hw/i386/pc.c | 16 +---
1 file changed, 9 insertions(+), 7 deletions(-)
diff --git a/hw/i386/pc.c b/hw/i386/pc.c
index 50ba977..3475174 100644
--- a/hw/i386/pc.c
+++ b/hw/i386/pc.c
@@ -1789,12 +1789,1
From: Laurent Vivier
Signed-off-by: Laurent Vivier
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: David Gibson
---
hw/ppc/mac_newworld.c | 15 +++
hw/ppc/trace-events | 4
2 files changed, 7 insertions(+), 12 deletions(-)
diff --git a/hw/ppc/mac_newworld.c b/hw/ppc/ma
From: Bharata B Rao
xsaddqpo: VSX Scalar Add Quad-Precision using round to Odd
xsmulqo: VSX Scalar Multiply Quad-Precision using round to Odd
xsdivqpo: VSX Scalar Divide Quad-Precision using round to Odd
xscvqpdpo: VSX Scalar round & Convert Quad-Precision format to
Double-Precisio
From: Suraj Jitindar Singh
The logical partitioning control register controls a threads operation
based on the partition it is currently executing. Add new definitions and
update the mask used when writing to the LPCR based on the POWER9 spec.
Signed-off-by: Suraj Jitindar Singh
Signed-off-by:
From: Thomas Huth
hw_error() is for CPU related errors only (it prints out a
register dump and calls abort()), so we should not use it
if we just failed to load the bios image. Apart from that,
realize() functions should not exit directly but always set
the errp with error_setg() in case of error
From: Alex Zuepke
The Book-E TLB matching process should bail out early when a TLB
entry matches, but the access permissions are wrong. The CPU
will then raise a DSI error instead of a Data TLB error, as
described for TLB matching in Freescale and IBM documents.
Signed-off-by: Alex Zuepke
Signe
From: Suraj Jitindar Singh
POWER9 processors implement the mmu as defined in version 3.00 of the ISA.
Add a definition for this mmu model and set the POWER9 cpu model to use
this mmu model.
Signed-off-by: Suraj Jitindar Singh
Signed-off-by: David Gibson
---
target/ppc/cpu-qom.h| 5 ++
From: Igor Mammedov
so it could be reused for SPAPR cores as well
Signed-off-by: Igor Mammedov
Signed-off-by: David Gibson
---
hw/acpi/cpu.c | 2 +-
hw/i386/pc.c| 8
include/hw/boards.h | 2 +-
3 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/hw/acpi/cpu.c
From: Igor Mammedov
All callbacks FOO_query_hotpluggable_cpus() are practically
the same except of setting vcpus_count to different values.
Convert them to a generic machine_query_hotpluggable_cpus()
callback by moving vcpus_count initialization to per machine
specific callback possible_cpu_arch_
From: Sam Bobroff
The spapr-vlan device in QEMU has always presented it's MAC address in
the device tree as an 8 byte value, even though PAPR requires it to be
6 bytes. This is because, at the time, AIX required the value to be 8
bytes. However, modern versions of AIX support the (correct) 6
by
From: Igor Mammedov
so that it would be possible to reuse it with
spapr/virt-aarch64 targets.
Signed-off-by: Igor Mammedov
Signed-off-by: David Gibson
---
hw/i386/pc.c | 57 ++--
include/hw/boards.h | 1 +
include/hw/i386/pc.h | 1 -
From: Thomas Huth
On POWER, the valid page sizes that the guest can use are bound
to the CPU and not to the memory region. QEMU already has some
fancy logic to find out the right maximum memory size to tell
it to the guest during boot (see getrampagesize() in the file
target/ppc/kvm.c for more in
From: Igor Mammedov
Replace SPAPR specific cores[] array with generic
machine->possible_cpus and store core objects there.
It makes cores bookkeeping similar to x86 cpus and
will allow to unify similar code.
It would allow to replace cpu_index based NUMA node
mapping with iproperty based one (for
From: Suraj Jitindar Singh
The DPFD field in the LPCR is 3 bits wide. This has always been defined
as 0x3 << shift which indicates a 2 bit field, which is incorrect.
Correct this.
Signed-off-by: Suraj Jitindar Singh
Signed-off-by: David Gibson
---
target/ppc/cpu.h | 2 +-
1 file changed, 1 in
On Thu, Feb 16, 2017 at 02:47:26PM +0100, Cédric Le Goater wrote:
> A list of ICS objects was introduced under the XICS object for the
> PowerNV machine, but, for the sPAPR machine, it brings extra complexity
> as there is only a single ICS. To simplify the code, let's add the ICS
> pointer under t
From: Igor Mammedov
Generic helper machine_query_hotpluggable_cpus() replaced
target specific query_hotpluggable_cpus() callbacks so
there is no need in it anymore. However inon NULL callback
value is used to detect/report hotpluggable cpus support,
therefore it can be removed completely.
Replace
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Subject: [Qemu-devel] [PULL 00/43] ppc-for-2.9 queue 20170222
Message-id: 20170222063348.32176-1-da...@gibson.dropbear.id.au
=== TEST SCRIPT BEGIN ===
#!/bin/bash
BASE=base
n=1
total=$(
on 2017/2/22 2:44, Alex Williamson wrote:
On Tue, 21 Feb 2017 18:09:04 +
Peter Maydell wrote:
On 21 February 2017 at 16:34, Paolo Bonzini wrote:
On 21/02/2017 17:21, Alex Williamson wrote:
On Tue, 21 Feb 2017 14:46:55 +0800
Yongji Xie wrote:
At the moment ram device's memory regi
On 02/22/2017 12:20 AM, Eric Blake wrote:
On 02/21/2017 08:07 AM, Markus Armbruster wrote:
Zhang Chen writes:
On 02/21/2017 07:15 PM, Markus Armbruster wrote:
Zhang Chen writes:
We can call this qmp command to do checkpoint outside of qemu.
Xen colo will need this function.
I know noth
This allows to specify display and head to use, simliar to vnc.
Signed-off-by: Gerd Hoffmann
---
ui/spice-core.c| 6 ++
ui/spice-display.c | 22 +-
2 files changed, 27 insertions(+), 1 deletion(-)
diff --git a/ui/spice-core.c b/ui/spice-core.c
index d613f1a..e18ad77
This way you can use -spice multiple times on the command line
and qemu will actually pick up all options.
Signed-off-by: Gerd Hoffmann
---
ui/spice-core.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/ui/spice-core.c b/ui/spice-core.c
index 39ccab7..d613f1a 100644
--- a/ui/spice-core.c
++
On Tue, Feb 21, 2017 at 6:49 AM Eric Blake wrote:
> From: Vladimir Sementsov-Ogievskiy
>
> Comparison symbol is misused. It may lead to memory corruption.
> Introduced in commit 7d3123e.
>
> Signed-off-by: Vladimir Sementsov-Ogievskiy
> Message-Id: <20170203154757.36140-6-vsement...@virtuozzo.c
On Tue, Feb 21, 2017 at 6:49 AM Eric Blake wrote:
> The NBD Protocol is introducing some additional information
> about exports, such as minimum request size and alignment, as
> well as an advertised maximum request size. It will be easier
> to feed this information back to the block layer if we
On Tue, Feb 21, 2017 at 6:45 AM Eric Blake wrote:
> The NBD protocol would like to advertise the optimal I/O
> size to the client; but it would be a layering violation to
> peek into blk_bs(blk)->bl, when we only have a BB.
>
> This copies the existing blk_get_max_transfer() in reading
> a value
Hi
On Tue, Feb 21, 2017 at 12:00 PM Gerd Hoffmann wrote:
> This way you can use -spice multiple times on the command line
> and qemu will actually pick up all options.
>
>
Is this actually a good thing to have? vnc doesn't have it for example.
Perhaps some day we may want to have -spice for diff
On Tue, Feb 21, 2017 at 6:54 AM Eric Blake wrote:
> The NBD protocol has several constants defined in various extensions
> that we are about to implement. Expose them to the code, along with
> an easy way to map various constants to strings during diagnostic
> messages.
>
> Doing this points out
In 'qemu_chr_open_spice_vmc' if the 'psubtype' is NULL, it will
call 'char_spice_finalize'. But as the SpiceChardev is not inserted
in the 'spice_chars' list, the 'QLIST_REMOVE' will cause a segfault.
Add a detect to avoid it.
Signed-off-by: Li Qiang
---
spice-qemu-char.c | 5 -
1 file chang
On Di, 2017-02-21 at 08:15 +, Marc-André Lureau wrote:
> Hi
>
> On Tue, Feb 21, 2017 at 12:00 PM Gerd Hoffmann
> wrote:
>
> This way you can use -spice multiple times on the command line
> and qemu will actually pick up all options.
>
>
>
> Is this actually a good
Hi
- Original Message -
> In 'qemu_chr_open_spice_vmc' if the 'psubtype' is NULL, it will
> call 'char_spice_finalize'. But as the SpiceChardev is not inserted
> in the 'spice_chars' list, the 'QLIST_REMOVE' will cause a segfault.
> Add a detect to avoid it.
>
> Signed-off-by: Li Qiang
On 20/02/2017 21:16, Dr. David Alan Gilbert wrote:
>> p.has_max_bandwidth = true;
>> ret = qemu_strtosz_mebi(valuestr, NULL, &valuebw);
>> -if (ret < 0 || (size_t)valuebw != valuebw) {
>> +if (ret < 0 || valuebw > INT64_MAX
>> +
Ping!
Would be nice for us if we can get this into 2.9.
Thanks,
Fred
Le 17/02/2017 à 21:17, fred.kon...@greensocs.com a écrit :
From: KONRAD Frederic
This series allows to execute code from mmio areas.
The main goal of this is to be able to run code for example from an SPI device.
The three
Hi
On Tue, Feb 21, 2017 at 11:58 AM Gerd Hoffmann wrote:
> This allows to specify display and head to use, simliar to vnc.
>
>
Does "head" in qemu always match the head for multi-monitor spice? (with a
single qxl/virtio). I don't clearly understand the relation, I would need
to do some research.
* Cornelia Huck [2017-02-20 19:59:01 +0100]:
> On Fri, 17 Feb 2017 09:29:39 +0100
> Dong Jia Shi wrote:
>
> > Although Linux does not use format-0 channel command words (CCW0)
> > these are a non-optional part of the platform spec, and for the sake
> > of platform compliance, and possibly some
Daniel Henrique Barboza writes:
> The previous error message was displaying the values in miliseconds,
> being misleading with the command that accepts the value in seconds:
>
> { "execute": "migrate_set_downtime", "arguments": {"value": 3000}}
> {"error": {"class": "GenericError", "desc": "Param
Vincenzo Maffione writes:
> In the vhost-user example, a chardev with id chr0 is referenced by the
> vhost-user net backend, but the id is not specified in the chardev option.
>
> Signed-off-by: Vincenzo Maffione
> ---
> qemu-options.hx | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Signed-off-by: Paolo Bonzini
---
scripts/kvm/vmxcap | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/scripts/kvm/vmxcap b/scripts/kvm/vmxcap
index 2220255..af8de15 100755
--- a/scripts/kvm/vmxcap
+++ b/scripts/kvm/vmxcap
@@ -27,9 +27,9 @@ MSR_IA32_VMX_VMFUNC = 0x
Signed-off-by: Paolo Bonzini
---
scripts/kvm/vmxcap | 8
1 file changed, 8 insertions(+)
diff --git a/scripts/kvm/vmxcap b/scripts/kvm/vmxcap
index af8de15..e0990c7 100755
--- a/scripts/kvm/vmxcap
+++ b/scripts/kvm/vmxcap
@@ -171,8 +171,11 @@ controls = [
13: 'Enable VM fun
On Di, 2017-02-21 at 08:52 +, Marc-André Lureau wrote:
> Hi
>
> On Tue, Feb 21, 2017 at 11:58 AM Gerd Hoffmann
> wrote:
>
> This allows to specify display and head to use, simliar to
> vnc.
>
>
>
> Does "head" in qemu always match the head for multi-monitor spice?
"Dr. David Alan Gilbert" writes:
> * Markus Armbruster (arm...@redhat.com) wrote:
>> Change the qemu_strtosz() & friends to return -EINVAL when @endptr is
>> null and the conversion doesn't consume the string completely.
>> Matches how qemu_strtol() & friends work.
>>
>> Only test_qemu_strtosz_s
* Markus Armbruster (arm...@redhat.com) wrote:
> "Dr. David Alan Gilbert" writes:
>
> > * Markus Armbruster (arm...@redhat.com) wrote:
> >> Change the qemu_strtosz() & friends to return -EINVAL when @endptr is
> >> null and the conversion doesn't consume the string completely.
> >> Matches how qe
Richard Henderson writes:
> On 02/20/2017 09:11 PM, Nikunj A Dadhania wrote:
>> tcg_temp_free(t0);
>> +tcg_gen_extract_tl(cpu_ov32, cpu_ov, 31, 1);
>> +tcg_gen_extract_tl(cpu_ov, cpu_ov, 63, 1);
>> if (NARROW_MODE(ctx)) {
>> -tcg_gen_ext32s_tl(cpu_ov, cpu_ov);
>> +
Hi
On Tue, Feb 21, 2017 at 1:11 PM Gerd Hoffmann wrote:
> On Di, 2017-02-21 at 08:52 +, Marc-André Lureau wrote:
> > Hi
> >
> > On Tue, Feb 21, 2017 at 11:58 AM Gerd Hoffmann
> > wrote:
> >
> > This allows to specify display and head to use, simliar to
> > vnc.
> >
> >
> >
>
Richard Henderson writes:
> On 02/20/2017 09:11 PM, Nikunj A Dadhania wrote:
>> static void gen_neg(DisasContext *ctx)
>> {
>> -gen_op_arith_neg(ctx, 0);
>> +tcg_gen_neg_tl(cpu_gpr[rD(ctx->opcode)], cpu_gpr[rA(ctx->opcode)]);
>> }
>>
>
> NACK. You're forgetting "neg.".
Oops, i totall
"Dr. David Alan Gilbert" writes:
> * Markus Armbruster (arm...@redhat.com) wrote:
>> This will permit its use in parse_option_size().
>>
>> Cc: Dr. David Alan Gilbert
>> Cc: Eduardo Habkost (maintainer:X86)
>> Cc: Kevin Wolf (supporter:Block layer core)
>> Cc: Max Reitz (supporter:Block laye
Richard Henderson writes:
> On 02/20/2017 09:11 PM, Nikunj A Dadhania wrote:
>> For 64-bit mode if the register RA contains 0x8000___, OV
>> and OV32 are set to 1.
>>
>> For 32-bit mode if the register RA contains 0x8000_, OV and OV32 are
>> set to 1.
>>
>> Use the tcg-ops for neg
On 20 February 2017 at 14:12, Gerd Hoffmann wrote:
> Hi,
>
> Here is the audio queue, bringing support for replay and fixing audio
> with SDL2.
>
> please pull,
> Gerd
>
> The following changes since commit 5d42ff913bb294c661aed8dfcd386fed9e185148:
>
> Merge remote-tracking branch 'remotes/h
On 20 February 2017 at 20:30, Yongbok Kim wrote:
> Hi,
>
> This is pull-req for target-mips.
>
> Regards,
> Yongbok
>
> The following changes since commit 56f9e46b841c7be478ca038d8d4085d776ab4b0d:
>
> Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2017-02-20'
> into staging (2017-0
Signed-off-by: Gerd Hoffmann
---
ui/spice-display.c | 36 +---
1 file changed, 21 insertions(+), 15 deletions(-)
diff --git a/ui/spice-display.c b/ui/spice-display.c
index b80a9f3..23ccf2a 100644
--- a/ui/spice-display.c
+++ b/ui/spice-display.c
@@ -928,6 +928,17
Helper function (and DisplayChangeListenerOps ptr) to disable scanouts.
Replaces using dpy_gl_scanout_texture with 0x0 size and no texture
specified.
Allows cleanups to make the io and gfx emulation code more readable.
Signed-off-by: Gerd Hoffmann
---
include/ui/console.h | 2 ++
ui/console.c
Hi,
Upcoming intel vgpu support will need some updates for the qemu opengl
support, specifically we will need support for importing dma-bufs.
The kernel support for this is still brewing though and thus the qemu
patches are still experimental and a moving target.
While working on it I did some
Signed-off-by: Gerd Hoffmann
---
include/ui/sdl2.h | 1 +
ui/sdl2-gl.c | 16 +++-
ui/sdl2.c | 1 +
3 files changed, 13 insertions(+), 5 deletions(-)
diff --git a/include/ui/sdl2.h b/include/ui/sdl2.h
index 2de792f..aaf226c 100644
--- a/include/ui/sdl2.h
+++ b/include/u
Signed-off-by: Gerd Hoffmann
---
hw/display/virtio-gpu-3d.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/hw/display/virtio-gpu-3d.c b/hw/display/virtio-gpu-3d.c
index b526b3f..f49b7fe 100644
--- a/hw/display/virtio-gpu-3d.c
+++ b/hw/display/virtio-gpu-3d.c
@@ -185,8 +1
Signed-off-by: Gerd Hoffmann
---
include/ui/gtk.h | 1 +
ui/gtk-egl.c | 15 ++-
ui/gtk.c | 1 +
3 files changed, 12 insertions(+), 5 deletions(-)
diff --git a/include/ui/gtk.h b/include/ui/gtk.h
index 408e21b..ca9a226 100644
--- a/include/ui/gtk.h
+++ b/include/ui/gtk.h
Eric Blake writes:
[...]
> 'git send-email -s' can also add Signed-off-by: lines, if you didn't add
> them earlier (but only if you use send-email, rather than attachments) :)
That's fine, as you *should* use git send-email.
If it looks unattractive to you because it needs some setup, then that
We'll add a variant which accepts dmabufs soon. Change
the name so we can easily disturgish the two variants.
Signed-off-by: Gerd Hoffmann
---
hw/display/virtio-gpu-3d.c | 14 +++---
include/ui/console.h | 19 +++
include/ui/gtk.h | 24 ++-
On 21 February 2017 at 09:41, Markus Armbruster wrote:
> Eric Blake writes:
>
> [...]
>> 'git send-email -s' can also add Signed-off-by: lines, if you didn't add
>> them earlier (but only if you use send-email, rather than attachments) :)
>
> That's fine, as you *should* use git send-email.
It h
On 02/21/17 02:43, Michael S. Tsirkin wrote:
> On Mon, Feb 20, 2017 at 09:55:40PM +0100, Laszlo Ersek wrote:
>> On 02/20/17 21:45, Eric Blake wrote:
>>> On 02/20/2017 02:19 PM, Dr. David Alan Gilbert wrote:
* Eric Blake (ebl...@redhat.com) wrote:
> On 02/20/2017 04:23 AM, Dr. David Alan Gi
On 21 February 2017 at 02:24, wrote:
> On Sat, Feb 18, 2017 at 10:41:41PM +, Peter Maydell wrote:
>>
>> Does the BE ppc64 risu build work for you? I tried
>> installing the cross-compiler, but the build fails
>> because there's no risu_ppc64.c. (ppc64le works fine.)
>> If we don't support the
* Alexey Perevalov (a.pereva...@samsung.com) wrote:
>
> Hello David,
Hi Alexey,
> On Tue, Feb 14, 2017 at 07:34:26PM +, Dr. David Alan Gilbert wrote:
> > * Alexey Perevalov (a.pereva...@samsung.com) wrote:
> > > Hi David,
> > >
> > > Thank your, now it's clear.
> > >
> > > On Mon, Feb 13,
Hi,
> It's always 0, except with virtio.
>
> But I think virtio linux driver multi-monitor code would work with a
> single qemu head (like qxl, fwiw last time I checked, multi-monitor
> virtio+spice worked, with 3d too).
> It seems to me the qemu "head" is quite different (lower-level) from
>
Applied to -trivial, thanks!
/mjt
Applied to -trivial, thanks!
/mjt
(restored cc:s)
On Mon, 20 Feb 2017 16:33:36 +0100
Thomas Huth wrote:
> On 20.02.2017 15:19, Cornelia Huck wrote:
> > From: Farhan Ali
> >
> > The current QEMU ROM infrastructure rejects late loading of ROMs.
> > And ELFs are currently loaded as ROM, this prevents delayed loading
> > of ELFs.
Hi,
First, sorry for such a long time delay on the AER job. I was on 12 days
holiday, and start to work on the patch 2 weeks ago, because I use a
newer version kernel(4.10 rc8) to start off the work, several tiny
problems slow me.
Now I meet a confusing issue on aer_inject module. When I use aer_
On 02/20/2017 04:33 PM, Thomas Huth wrote:
> On 20.02.2017 15:19, Cornelia Huck wrote:
>> From: Farhan Ali
>>
>> The current QEMU ROM infrastructure rejects late loading of ROMs.
>> And ELFs are currently loaded as ROM, this prevents delayed loading
>> of ELFs. So when loading ELF, allow the user
be478ca038d8d4085d776ab4b0d:
>
> Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2017-02-20'
> into staging (2017-02-20 17:42:47 +)
>
> are available in the git repository at:
>
>
> git://git.kraxel.org/qemu tags/pull-usb-20170221-1
>
> for
On Tue, Feb 21, 2017 at 1:38 PM Gerd Hoffmann wrote:
> Hi,
>
> Upcoming intel vgpu support will need some updates for the qemu opengl
> support, specifically we will need support for importing dma-bufs.
> The kernel support for this is still brewing though and thus the qemu
> patches are still
On Wed, Feb 15, 2017 at 9:47 PM, Alex Williamson wrote:
> On Thu, 16 Feb 2017 10:28:39 +0800
> Peter Xu wrote:
>
> > On Wed, Feb 15, 2017 at 11:15:52AM -0700, Alex Williamson wrote:
> >
> > [...]
> >
> > > > Alex, do you like something like below to fix above issue that
> Jintack
> > > > has enc
On 20/02/2017 19:21, Denis V. Lunev wrote:
> This is a set of followup patches requested by Eric Blake.
>
> Signed-off-by: Anton Nefedov
> Signed-off-by: Denis V. Lunev
> CC: Paolo Bonzini
> CC: Eric Blake
>
> Anton Nefedov (3):
> i386/cpu: release GuestPanicInformation memory
> qapi: f
qemu_system_guest_panicked was already using current_cpu implicitly,
so it makes sense for it to receive a CPUState. This lets the
function call cpu_get_crash_info and free the result.
Signed-off-by: Paolo Bonzini
---
include/sysemu/sysemu.h | 2 +-
kvm-all.c | 2 +-
vl.c
The functions simplify the handling of QOM properties whose type
is a QAPI struct. They go through a QObject just like the other
functions that access a QOM property through its C type.
Like QAPI_CLONE, the functions are wrapped by macros that take a
QAPI type name and use it to build the name of
This is an alternative approach to simplifying the crash information
patches.
Currently, crash information is exposed twice, through a QOM property
and through a method. This is because accessing QOM properties with
QAPI struct types is a huge pain in the neck. Patch 1 fixes this by
providing a
Provide a generic implementation for all CPU subclasses.
Signed-off-by: Paolo Bonzini
---
include/qom/cpu.h | 1 -
qom/cpu.c | 11 ---
target/i386/cpu.c | 2 +-
3 files changed, 5 insertions(+), 9 deletions(-)
diff --git a/include/qom/cpu.h b/include/qom/cpu.h
index 1bc3ad2..0
* Halil Pasic (pa...@linux.vnet.ibm.com) wrote:
> Add test for VMSTATE_ARRAY_OF_POINTER_TO_STRUCT with an array
> containing some null pointer.
>
> Signed-off-by: Halil Pasic
> Reviewed-by: Dr. David Alan Gilbert
> ---
> tests/test-vmstate.c | 51
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Subject: [Qemu-devel] [PATCH 0/3] simplify struct QOM properties and use the
result for GUEST_PANICKED
Message-id: 20170221104256.5153-1-pbonz...@redhat.com
=== TEST SCRIPT BEGIN ===
#!
Hi,
This series failed automatic build test. Please find the testing commands and
their output below. If you have docker installed, you can probably reproduce it
locally.
Type: series
Subject: [Qemu-devel] [PATCH 0/3] simplify struct QOM properties and use the
result for GUEST_PANICKED
Message-i
On Mon 20 Feb 2017 05:52:04 PM CET, Stefan Hajnoczi wrote:
> The disk I/O throttling options have been listed for a long time but
> never explained on the QEMU man page.
> +@item bps=@var{b},bps_rd=@var{r},bps_wr=@var{w}
> +Specify bandwidth throttling limits in bytes per second, either for all
>
On Mon, Feb 20, 2017 at 03:34:57AM -0800, ashish mittal wrote:
> On Mon, Feb 20, 2017 at 3:02 AM, Stefan Hajnoczi wrote:
> > On Sat, Feb 18, 2017 at 12:30:31AM +, Ketan Nilangekar wrote:
> >> On 2/17/17, 1:42 PM, "Jeff Cody" wrote:
> >>
> >> On Thu, Feb 16, 2017 at 02:24:19PM -0800, ashis
On 20 February 2017 at 17:34, Clement Deschamps
wrote:
> This patches add the Arasan SD controller to the BCM2835/36 platforms.
Cool!
> On the real hardware, both SD controllers are wired to the same SD card.
> Selection is done via multi-function pins through the GPIO controller. I'm
> not
>
Drop unused QIV_STACK_SIZE and unused qobject_input_start_struct()
parameter errp.
Signed-off-by: Markus Armbruster
---
qapi/qobject-input-visitor.c | 14 +++---
1 file changed, 3 insertions(+), 11 deletions(-)
diff --git a/qapi/qobject-input-visitor.c b/qapi/qobject-input-visitor.c
ind
The improvement is nice for QMP, but my real motivation is to avoid
embarrassingly horrible error messages for command line options once
we use the qobject input visitor there, as we do in "[PATCH RFC v2
0/2] block: Crude initial implementation of -blockdev".
Markus Armbruster (2):
qapi: Clean u
Error messages refer to nodes of the QObject being visited by name.
Trouble is the names are sometimes less than helpful:
* The name of the root QObject is whatever @name argument got passed
to the visitor, except NULL gets mapped to "null". We commonly pass
NULL. Not good.
Avoiding error
On 02/21/2017 11:49 AM, Dr. David Alan Gilbert wrote:
>> +static void test_arr_ptr_str_0_save(void)
>> +{
>> +TestStructTriv ar[AR_SIZE] = {{.i = 0}, {.i = 1}, {.i = 2}, {.i = 3} };
>> +TestArrayOfPtrToStuct sample = {.ar = {&ar[0], NULL, &ar[2], &ar[3]} };
>> +uint8_t wire_sample[] =
On Mon, Feb 20, 2017 at 03:59:42PM +0100, Peter Lieven wrote:
> Am 20.02.2017 um 15:50 schrieb Stefan Hajnoczi:
> > On Fri, Feb 17, 2017 at 05:00:24PM +0100, Peter Lieven wrote:
> > > +if (s->wr_in_order) {
> > > +/* reenter the coroutine that might have waited
> > > +
* Halil Pasic (pa...@linux.vnet.ibm.com) wrote:
> Let's have a test for ptr arrays to some primitive type with some
> not-null and null ptrs intermixed.
>
> Signed-off-by: Halil Pasic
>
> ---
>
> Mainly for the sake of completeness and also to demonstrate that it works
> since in the previous v
Zhang Chen writes:
> We can call this qmp command to do checkpoint outside of qemu.
> Xen colo will need this function.
I know nothing about "Xen colo", but I'll try anyway.
I *guess* "Xen colo" is a long-running activity, and the new commands
interact with it. Correct?
>
> Signed-off-by: Zha
On Mon, Feb 20, 2017 at 01:37:58PM +, Peter Maydell wrote:
> On 20 February 2017 at 09:32, Stefan Hajnoczi wrote:
> > The following changes since commit 5dae13cd71f0755a1395b5a4cde635b8a6ee3f58:
> >
> > Merge remote-tracking branch 'remotes/rth/tags/pull-or-20170214' into
> > staging (2017-
On Mon, Feb 20, 2017 at 05:29:19PM +0100, Alberto Garcia wrote:
> On Mon, Feb 20, 2017 at 04:45:54PM +0100, Kevin Wolf wrote:
> > > I can imagine two solutions that do not need these parameters in
> > > blockdev-add:
> > >
> > > 1. I/O throttling is implemented by a BlockDriver. Users are expecte
Zhang Chen writes:
> Hi~ Markus.
>
> This patch set has been reviewed for a long time, can you pick up it?
>
> [PATCH V7 0/2] Add new qmp commands to suppurt Xen COLO
I think this should go through a COLO maintainer chain, or perhaps the
Xen maintainer, but not the QAPI maintainer (me). QAPI gu
On Mon, Feb 20, 2017 at 01:40:21PM -0600, Eric Blake wrote:
> On 02/20/2017 10:52 AM, Stefan Hajnoczi wrote:
> > The disk I/O throttling options have been listed for a long time but
> > never explained on the QEMU man page.
> >
> > Suggested-by: Nini Gu
> > Cc: Alberto Garcia
> > Signed-off-by:
On Mon, Feb 20, 2017 at 9:45 AM, Philippe Mathieu-Daudé wrote:
> On 02/20/2017 12:36 PM, Peter Maydell wrote:
>>
>> Switch the stm32f205 SoC to create the armv7m object directly
>> rather than via the armv7m_init() wrapper. This fits better
>> with the SoC model's very QOMified design.
>>
>> In pa
On Tue, Feb 21, 2017 at 10:59:18AM +, Stefan Hajnoczi wrote:
> On Mon, Feb 20, 2017 at 03:34:57AM -0800, ashish mittal wrote:
> > On Mon, Feb 20, 2017 at 3:02 AM, Stefan Hajnoczi wrote:
> > > On Sat, Feb 18, 2017 at 12:30:31AM +, Ketan Nilangekar wrote:
> > >> On 2/17/17, 1:42 PM, "Jeff Co
On Mon, Feb 20, 2017 at 9:46 AM, Philippe Mathieu-Daudé wrote:
> On 02/20/2017 12:36 PM, Peter Maydell wrote:
>>
>> The local variable 'nvic' in stm32f205_soc_realize() no longer
>> holds a direct pointer to the NVIC device; it is a pointer to
>> the ARMv7M container object. Rename it 'armv7m' acc
On Mon, Feb 20, 2017 at 7:35 AM, Peter Maydell wrote:
> Make the legacy armv7m_init() function use the newly QOMified
> armv7m object rather than doing everything by hand.
>
> We can return the armv7m object rather than the NVIC from
> armv7m_init() because its interface to the rest of the
> board
On Mon, Feb 20, 2017 at 8:23 AM, Philippe Mathieu-Daudé wrote:
> On 02/20/2017 12:35 PM, Peter Maydell wrote:
>>
>> Abstract the "load kernel" code out of armv7m_init() into its own
>> function. This includes the registration of the CPU reset function,
>> to parallel how we handle this for A prof
1 - 100 of 544 matches
Mail list logo
