A command is a query if it has no side effect and yields a result.
Such commands are typically named query-FOO, but there are exceptions.
The basic idea is to find candidates with query-qmp-schema, filter out
the ones that aren't queries with an explicit blacklist, and test the
remaining ones agai
From: Marc-André Lureau
Fix code style issues while at it, to please checkpatch.
Signed-off-by: Marc-André Lureau
Reviewed-by: Markus Armbruster
Message-Id: <20170825105913.4060-3-marcandre.lur...@redhat.com>
Signed-off-by: Markus Armbruster
---
include/qapi/qmp/qlit.h | 49 +
Signed-off-by: Markus Armbruster
Message-Id: <1503564371-26090-2-git-send-email-arm...@redhat.com>
Reviewed-by: Marc-André Lureau
---
docs/devel/qapi-code-gen.txt | 47 +++-
1 file changed, 38 insertions(+), 9 deletions(-)
diff --git a/docs/devel/qapi-cod
The following changes since commit 223cd0e13f2e46078d7b573f0b8402bfbee339be:
Merge remote-tracking branch 'remotes/elmarco/tags/tidy-pull-request' into
staging (2017-08-31 15:52:43 +0100)
are available in the git repository at:
git://repo.or.cz/qemu/armbru.git tags/pull-qapi-2017-09-01
for
Cc: Alberto Garcia
Signed-off-by: Markus Armbruster
Message-Id: <1503602048-12268-13-git-send-email-arm...@redhat.com>
Reviewed-by: Alberto Garcia
Reviewed-by: Marc-André Lureau
---
qapi/block.json | 68 +
qapi/event.json | 68 ---
From: Marc-André Lureau
Rename from LiteralQ to QLit.
Signed-off-by: Marc-André Lureau
Reviewed-by: Markus Armbruster
Message-Id: <20170825105913.4060-4-marcandre.lur...@redhat.com>
Signed-off-by: Markus Armbruster
---
include/qapi/qmp/qlit.h | 24
qobject/qlit.c
From: Marc-André Lureau
The error message on invalid parameter name changes from
Invalid parameter "NAME"
to
invalid parameter value: NAME
Slight degratation, perhaps.
Signed-off-by: Marc-André Lureau
Message-Id: <20170822132255.23945-10-marcandre.lur...@redhat.com>
Reviewed-by: Mar
Bug: introspection documentation is in section "Tracing commands".
Cause: sub-schema qapi/introspect.json lacks a section header, and
therefore goes into whatever section precedes its include.
Fix: add a section header.
Signed-off-by: Markus Armbruster
Reviewed-by: Marc-André Lureau
Message-Id
Sadly, we don't have a TPM maintainer, not even a MAINTAINERS entry.
Create one, and mark it orphaned.
Signed-off-by: Markus Armbruster
Message-Id: <1503602048-12268-12-git-send-email-arm...@redhat.com>
Reviewed-by: Marc-André Lureau
---
MAINTAINERS | 8
Makefile | 1 +
qa
Bug: section "Rocker switch device" starts with the rocker stuff, but
then has unrelated stuff, like ReplayMode, xen-load-devices-state, ...
Cause: rocker.json is included in the middle of section "QMP commands".
Fix: include it in a sane place, namely next to the other sub-schemas.
Signed-off-b
From: Marc-André Lureau
Bonus: rids us of a side effect in an assertion.
Signed-off-by: Marc-André Lureau
Reviewed-by: Markus Armbruster
Message-Id: <20170825105913.4060-10-marcandre.lur...@redhat.com>
Signed-off-by: Markus Armbruster
---
qobject/qlit.c | 5 +
1 file changed, 1 insertion
From: Marc-André Lureau
Signed-off-by: Marc-André Lureau
Reviewed-by: Markus Armbruster
Message-Id: <20170825105913.4060-8-marcandre.lur...@redhat.com>
Signed-off-by: Markus Armbruster
---
include/qapi/qmp/qlit.h | 2 +-
qobject/qlit.c | 2 +-
2 files changed, 2 insertions(+), 2 dele
From: Marc-André Lureau
We check that all members of the QLit list are also in the QList. We
neglect to check the other direction. Fix that.
While there, use QLIST_FOREACH_ENTRY() to simplify the code and break
the loop on the first mismatch.
Signed-off-by: Marc-André Lureau
Message-Id: <201
From: Marc-André Lureau
As they are going to be used in the following patches.
Signed-off-by: Marc-André Lureau
Reviewed-by: Markus Armbruster
Message-Id: <20170825105913.4060-9-marcandre.lur...@redhat.com>
Signed-off-by: Markus Armbruster
---
include/qapi/qmp/qlit.h | 5 +
qobject/qlit.
Signed-off-by: Markus Armbruster
Message-Id: <1503564371-26090-11-git-send-email-arm...@redhat.com>
Reviewed-by: Marc-André Lureau
---
qapi/qapi-visit-core.c | 13 -
1 file changed, 4 insertions(+), 9 deletions(-)
diff --git a/qapi/qapi-visit-core.c b/qapi/qapi-visit-core.c
index ed
We have a strict separation between enum TpmModel and tpm_models[]:
* TpmModel may have any number of members. It just happens to have one.
* tpm_register_model() uses the first empty slot in tpm_models[].
If you register more than tpm_models[] has space,
tpn_register_model() fails. Its ca
Signed-off-by: Markus Armbruster
Message-Id: <1503602048-12268-14-git-send-email-arm...@redhat.com>
Reviewed-by: Marc-André Lureau
---
Makefile | 2 +-
qapi-schema.json | 134 -
qapi/event.json | 138 -
Cc: "Daniel P. Berrange"
Signed-off-by: Markus Armbruster
Message-Id: <1503564371-26090-10-git-send-email-arm...@redhat.com>
Reviewed-by: Marc-André Lureau
Acked-by: Daniel P. Berrange
---
crypto/block-luks.c | 19 +++
1 file changed, 7 insertions(+), 12 deletions(-)
diff --gi
query-version and query-commands are in common.json for no good
reason. Several similar queries are in qapi-schema.json. Move them
there.
Signed-off-by: Markus Armbruster
Message-Id: <1503602048-12268-16-git-send-email-arm...@redhat.com>
Reviewed-by: Marc-André Lureau
---
qapi-schema.json | 1
From: Marc-André Lureau
The error message on invalid capability name changes from
Invalid parameter "NAME"
to
invalid parameter value: NAME
No worse than before.
Signed-off-by: Marc-André Lureau
Message-Id: <20170822132255.23945-9-marcandre.lur...@redhat.com>
Reviewed-by: Markus Arm
Cc: Paolo Bonzini
Signed-off-by: Markus Armbruster
Message-Id: <1503602048-12268-6-git-send-email-arm...@redhat.com>
Reviewed-by: Marc-André Lureau
---
MAINTAINERS | 1 +
Makefile| 1 +
qapi-schema.json| 165 +---
qapi/event.json | 182 ---
Signed-off-by: Markus Armbruster
Message-Id: <1503564371-26090-15-git-send-email-arm...@redhat.com>
Reviewed-by: Marc-André Lureau
---
hw/core/qdev-properties.c | 7 +--
qapi/qapi-visit-core.c| 2 +-
2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/hw/core/qdev-properties.c
Cc: Jason Wang
Signed-off-by: Markus Armbruster
Message-Id: <1503602048-12268-8-git-send-email-arm...@redhat.com>
Reviewed-by: Marc-André Lureau
---
MAINTAINERS | 1 +
Makefile | 1 +
qapi-schema.json | 675 +---
qapi/event.json
From: Marc-André Lureau
Make it more obvious about the expected return values.
Signed-off-by: Marc-André Lureau
Reviewed-by: Markus Armbruster
Message-Id: <20170825105913.4060-7-marcandre.lur...@redhat.com>
Signed-off-by: Markus Armbruster
---
include/qapi/qmp/qlit.h | 2 +-
qobject/qlit.c
Cc: Paolo Bonzini
Cc: Marc-André Lureau
Signed-off-by: Markus Armbruster
Message-Id: <1503602048-12268-7-git-send-email-arm...@redhat.com>
Reviewed-by: Marc-André Lureau
---
MAINTAINERS | 1 +
Makefile | 1 +
qapi-schema.json | 511 +
From: Marc-André Lureau
The error message on invalid blkdebug events changes from
qemu-system-x86_64: LOCATION: Invalid event name "VALUE"
to
qemu-system-x86_64: LOCATION: invalid parameter value: VALUE
Slight degradation, but the message is sub-par even before the patch.
When complai
Signed-off-by: Markus Armbruster
Message-Id: <1503602048-12268-11-git-send-email-arm...@redhat.com>
Reviewed-by: Marc-André Lureau
---
MAINTAINERS | 1 +
Makefile | 1 +
qapi-schema.json | 151 +--
qapi/transaction.json
From: Marc-André Lureau
We have a strict separation between enum TpmType and be_drivers[]:
* TpmType may have any number of members. It just happens to have one.
* tpm_register_driver() uses the first empty slot in be_drivers[].
If you register more than tpm_models[] has space,
tpm_regist
Cc: "Daniel P. Berrange"
Cc: Gerd Hoffmann
Cc: Paolo Bonzini
Signed-off-by: Markus Armbruster
Message-Id: <1503602048-12268-5-git-send-email-arm...@redhat.com>
Reviewed-by: Marc-André Lureau
---
MAINTAINERS | 1 +
Makefile | 4 +-
qapi-schema.json | 152 +-
vga display update mis-calculated the region for the dirty bitmap
snapshot in case split screen mode is used. This can trigger an
assert in cpu_physical_memory_snapshot_get_dirty().
Impact: DoS for privileged guest users.
Fixes: CVE-2017-13673
Fixes: fec5e8c92becad223df9d972770522f64aafdb72
Cc:
From: Marc-André Lureau
Now that all usages have been converted to user lookup helpers.
Signed-off-by: Marc-André Lureau
Message-Id: <20170822132255.23945-14-marcandre.lur...@redhat.com>
[Rebased, superfluous local variable dropped, missing
check-qom-proplist.c update added]
Signed-off-by: Mark
From: Marc-André Lureau
compare_litqobj_to_qobj() lacks a qlit_ prefix. Moreover, "compare"
suggests -1, 0, +1 for less than, equal and greater than. The
function actually returns non-zero for equal, zero for unequal.
Rename to qlit_equal_qobject().
Its return type will be cleaned up in the ne
Signed-off-by: Gerd Hoffmann
Message-id: 20170828123933.30323-3-kra...@redhat.com
---
hw/display/qxl-render.c | 36 +---
1 file changed, 29 insertions(+), 7 deletions(-)
diff --git a/hw/display/qxl-render.c b/hw/display/qxl-render.c
index e1b3f05ecb..90e0865618 10
From: Marc-André Lureau
Signed-off-by: Marc-André Lureau
Message-Id: <20170822132255.23945-12-marcandre.lur...@redhat.com>
Reviewed-by: Markus Armbruster
[Rebased, qemu_opt_get() factored out, commit message tweaked]
Cc: Alberto Garcia
Reviewed-by: Alberto Garcia
Signed-off-by: Markus Armbrus
The next commit will put it to use. May look pointless now, but we're
going to change the FOO_lookup's type, and then it'll help.
Signed-off-by: Markus Armbruster
Message-Id: <1503564371-26090-13-git-send-email-arm...@redhat.com>
Reviewed-by: Marc-André Lureau
---
block.c
From: "Dr. David Alan Gilbert"
After migration the chain4 alias mapping added by 80763888 (in 2011)
might be missing, since there's no call to vga_update_memory_access
in the post_load after the registers are updated. Add it back.
Signed-off-by: Dr. David Alan Gilbert
Reviewed-by: Juan Quintel
The following changes since commit 223cd0e13f2e46078d7b573f0b8402bfbee339be:
Merge remote-tracking branch 'remotes/elmarco/tags/tidy-pull-request' into
staging (2017-08-31 15:52:43 +0100)
are available in the git repository at:
git://git.kraxel.org/qemu tags/vga-20170901-pull-re
The chunk size sanity check in qxl_render_cursor works for
SPICE_CURSOR_TYPE_ALPHA cursors only. So support for
SPICE_CURSOR_TYPE_MONO cursors must be broken for ages without anyone
noticing. Most likely it simply isn't used any more by guest drivers.
Drop the dead code.
Signed-off-by: Gerd Hoff
From: Marc-André Lureau
Currently, a FOO_lookup is an array of strings terminated by a NULL
sentinel.
A future patch will generate enums with "holes". NULL-termination
will cease to work then.
To prepare for that, store the length in the FOO_lookup by wrapping it
in a struct and adding a membe
Hello Alexey,
On 09/01/2017 03:34 PM, Alexey Perevalov wrote:
Hello David,
You wrote in previous version:
We've had a postcopy migrate work now, with a few hacks we're still
cleaning up, both on vhost-user-bridge and dpdk; so I'll get this
updated and reposted.
I want to know more about DPD
Eduardo Otubo writes:
> When not available, isa-fdc falls into assert instead of proper error
> exit. This patch fixes this behavior.
When what exactly isn't available?
>
> Signed-off-by: Eduardo Otubo
> ---
> hw/block/fdc.c | 6 +-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> dif
On Fri, Sep 01, 2017 at 11:05:11AM +0200, Markus Armbruster wrote:
> Eduardo Habkost writes:
>
> > On Thu, Aug 24, 2017 at 12:33:38PM +0200, Marc-André Lureau wrote:
> >> Fix code style issues while at it, to please check-patch.
> >>
> >> Signed-off-by: Marc-André Lureau
> >> ---
> >> include/
Currently, the FOO_lookup[] generated for QAPI enum types are
terminated by a NULL sentinel.
A future patch will generate enums with "holes". NULL-termination
will cease to work then.
To prepare for that, replace "have we reached the sentinel?"
predicates by "have we reached the FOO__MAX value?"
On 08/22/2017 10:04 PM, Michael S. Tsirkin wrote:
> On Tue, Aug 22, 2017 at 03:13:57PM +, Diana Madalina Craciun wrote:
>> On 08/11/2017 06:50 PM, Edgar E. Iglesias wrote:
>>> On Fri, Aug 11, 2017 at 02:35:28PM +, Diana Madalina Craciun wrote:
Hi Edgar,
On 07/31/2017 06:16 PM
Reported-by: niuguoxi...@huawei.com
Signed-off-by: Gerd Hoffmann
---
hw/audio/wm8750.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/hw/audio/wm8750.c b/hw/audio/wm8750.c
index f8b5bebfc2..d2bf2e1da1 100644
--- a/hw/audio/wm8750.c
+++ b/hw/audio/wm8750.c
@@ -680,8 +680,
Signed-off-by: Markus Armbruster
Message-Id: <1503564371-26090-14-git-send-email-arm...@redhat.com>
Reviewed-by: Marc-André Lureau
---
backends/hostmem.c | 2 +-
block/backup.c | 2 +-
block/file-posix.c | 4 +--
block/file-win32.c
UI stuff is remote desktop stuff (Spice, VNC) and input stuff (mouse,
keyboard).
Cc: Gerd Hoffmann
Signed-off-by: Markus Armbruster
Message-Id: <1503602048-12268-9-git-send-email-arm...@redhat.com>
Reviewed-by: Marc-André Lureau
Reviewed-by: Gerd Hoffmann
---
MAINTAINERS | 2 +
Makefil
Hello David,
You wrote in previous version:
We've had a postcopy migrate work now, with a few hacks we're still
cleaning up, both on vhost-user-bridge and dpdk; so I'll get this
updated and reposted.
I want to know more about DPDK work, do you know, is somebody assigned to that
task?
On 0
Cc: Juan Quintela
Cc: Dr. David Alan Gilbert
Signed-off-by: Markus Armbruster
Message-Id: <1503602048-12268-10-git-send-email-arm...@redhat.com>
Reviewed-by: Marc-André Lureau
Reviewed-by: Dr. David Alan Gilbert
---
MAINTAINERS |1 +
Makefile|1 +
qapi-schema.json
On Thu, 31 Aug 2017 16:40:08 +0200
Thomas Huth wrote:
> We can use the drive_del test on s390x, too, to check that adding and
> deleting also works fine with the virtio-ccw bus. But we have to make
> sure that we use the devices with the "-ccw" suffix instead of the
> "-pci" suffix for the virtio
Instead pass around the address (aka offset into vga memory).
Add vga_read_* helper functions which apply vbe_size_mask to
the address, to make sure the address stays within the valid
range, similar to the cirrus blitter fixes (commits ffaf857778
and 026aeffcb4).
Impact: DoS for privileged guest
On 11.08.2017 11:08, Markus Armbruster wrote:
> Eric Blake writes:
>
>> On 08/10/2017 01:30 PM, Markus Armbruster wrote:
[...]
>>> We get basic test coverage for the following commands:
>>
>> Cool!
>>
>>>
>>> qom-list-types
>>> query-acpi-ospm-status
>>> query-balloon (e
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.
Message-id: 20170901125611.29295-1-arm...@redhat.com
Subject: [Qemu-devel] [PULL 00/47] QAPI patches for 2017-09-01
Type: seri
On 09/01/2017 02:06 AM, Christian Ehrhardt wrote:
> On Thu, Aug 31, 2017 at 7:44 PM, Michael Roth
>
> wrote:
>
>> Quoting Thomas Huth (2017-08-29 04:35:22)
>>> On 28.08.2017 09:18, Christian Borntraeger wrote:
On 08/25/2017 10:29 AM, Cornelia Huck wrote:
> On Fri, 25 Aug 2017 1
Hi,
> I think it makes sense to create a separate project for the firmware
> blobs. Move over the firmware binaries and source submodules to the
> new project. This way updating both firmware sources and binaries
> can
> be done with a single commit, like we handle this today, just in the
> ne
On 09/01/2017 05:54 AM, Daniel P. Berrange wrote:
> The LUKS driver requires extra args to QEMU to setup passwords.
> The _launch_qemu function takes care of this, so convert the
> test to use this function and use correct -drive syntax
>
> Signed-off-by: Daniel P. Berrange
> ---
> tests/qemu-io
On 09/01/2017 05:54 AM, Daniel P. Berrange wrote:
> The 194 test has alot of code that assumes a simple image file. Rewriting
s/alot/a lot/
> this to work with luks is possible, but non-trivial, so blacklist the
> luks format for now.
>
> Signed-off-by: Daniel P. Berrange
> ---
> tests/qemu-io
On 09/01/2017 05:54 AM, Daniel P. Berrange wrote:
> Two recently added iotests are currently breaking 'check -luks',
> fix the easy one and disable the hard one.
>
> Daniel P. Berrange (2):
> iotests: rewrite 192 to use _launch_qemu to fix LUKS support
> iotests: blacklist 194 with the luks dr
i82374 is compiled in only on ppc and sh4, so I'm CCing the
maintainers for those architectures.
On Fri, Sep 01, 2017 at 01:03:32PM +0200, Eduardo Otubo wrote:
> When used with the following command line:
>
> ./ppc64-softmmu/qemu-system-ppc64 -S -machine 40p,accel=tcg -device i82374
>
> QEMU wi
On 08/23/2017 11:29 PM, Edgar E. Iglesias wrote:
> On Tue, Aug 22, 2017 at 10:04:25PM +0300, Michael S. Tsirkin wrote:
>> On Tue, Aug 22, 2017 at 03:13:57PM +, Diana Madalina Craciun wrote:
>>> On 08/11/2017 06:50 PM, Edgar E. Iglesias wrote:
On Fri, Aug 11, 2017 at 02:35:28PM +, Diana
On Fri, Sep 01, 2017 at 09:26:30AM -0500, Eric Blake wrote:
> On 09/01/2017 05:54 AM, Daniel P. Berrange wrote:
> > Two recently added iotests are currently breaking 'check -luks',
> > fix the easy one and disable the hard one.
> >
> > Daniel P. Berrange (2):
> > iotests: rewrite 192 to use _lau
Thomas Huth writes:
> On 11.08.2017 11:08, Markus Armbruster wrote:
>> Eric Blake writes:
>>
>>> On 08/10/2017 01:30 PM, Markus Armbruster wrote:
> [...]
We get basic test coverage for the following commands:
>>>
>>> Cool!
>>>
qom-list-types
query-acpi-ospm-status
>>
no-re...@patchew.org writes:
> 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.
What? How did this tree survive my testing? *sigh* v2 coming.
On Jul 9, 2017 2:04 PM, "Eric Auger" wrote:
This series implements INTx to gsi routing for ARM VIRT/Gpex. This is
a respin of [1] which was lost in limbo.
ARM virt uses GPEX PCIe bridge. This latter does not implement INTx
to GSI routing. PCIe/INTx assignment works but the consequence is
irqfd i
On Fri, 09/01 11:54, Daniel P. Berrange wrote:
> Two recently added iotests are currently breaking 'check -luks',
> fix the easy one and disable the hard one.
I can add -luks to the test-block series with this, thanks for the quick fix!
Tested-by: Fam Zheng
The following changes since commit 223cd0e13f2e46078d7b573f0b8402bfbee339be:
Merge remote-tracking branch 'remotes/elmarco/tags/tidy-pull-request' into
staging (2017-08-31 15:52:43 +0100)
are available in the git repository at:
git://github.com/ehabkost/qemu.git tags/x86-and-machine-pull-re
The helper can be used for CPU object lookup using the CPU's
arch-specific ID (the one returned by CPUClass::get_arch_id()).
Signed-off-by: Eduardo Habkost
[Yi Wang: Added documentation comments]
Signed-off-by: Yi Wang
Signed-off-by: Yun Liu
[ehabkost: extracted cpu_by_arch_id() to a separate p
The errp argument is ignored by all implementations of the
method, and user_creatable_del() would break if any
implementation set an error (because it calls error_setg(errp) if
the function returns false). Remove the unused parameter.
Signed-off-by: Eduardo Habkost
Message-Id: <20170829220337.23
From: Igor Mammedov
since commit ( 9262685b cpu: Factor out cpu_generic_init() )
features parsed by it were truncated only to the 1st feature
after CPU name due to fact that
featurestr = strtok(NULL, ",");
cc->parse_features(cpu, featurestr, &err);
would extract exactly one feature and pa
From: Igor Mammedov
QOMfy cpu models handling introducing propper cpu types
for each cpu model.
Signed-off-by: Igor Mammedov
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
Message-Id: <1503592308-93913-3-git-send-email-imamm...@redhat.com>
Signed-off-by: Eduardo Habkost
From: Yi Wang
Add [apic-id] support for hmp command "info lapic", which is
useful when debugging ipi and so on. Current behavior is not
changed when the parameter isn't specified.
Signed-off-by: Yi Wang
Signed-off-by: Yun Liu
Message-Id: <1501049917-4701-3-git-send-email-wang.y...@zte.com.cn>
From: Brijesh Singh
Add a new base CPU model called 'EPYC' to model processors from AMD EPYC
family (which includes EPYC 76xx,75xx,74xx, 73xx and 72xx).
The following features bits have been added/removed compare to Opteron_G5
Added: monitor, movbe, rdrand, mmxext, ffxsr, rdtscp, cr8legacy, osv
From: Igor Mammedov
drop custom cpu_hppa_init() in favor of cpu_generic_init(),
to make cpu_generic_init() work all we need is to provide
cc->class_by_name callback that would resolve any cpu_model
to the sole TYPE_HPPA_CPU to match current behaviour.
Signed-off-by: Igor Mammedov
Acked-by: Rich
From: Igor Mammedov
Make CPUSPARCState::def embedded so it would be allocated as part
of cpu instance and we won't have to worry about cleaning def pointer
up mannualy on cpu destruction.
Signed-off-by: Igor Mammedov
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
Messag
From: Igor Mammedov
cpu_alpha_init() used to provide default fallback if invalid
(i.e. non existent) cpu_model were provided.
dp264 machine provides its own default so sole user of fallback
is [bsd|linux]-user targets which specifies 'any' cpu model that
fallbacks to "ev67" in cpu_alpha_init().
From: Igor Mammedov
SPARC is the last target that uses legacy way of parsing
and initializing cpu features, drop legacy approach and
convert features to properties so that SPARC could as minimum
benefit from generic cpu_generic_init(), common with
x86 +-feat parser
PS:
the main purpose is to rem
From: Igor Mammedov
it's just a wrapper, drop it and use cpu_generic_init() directly
Signed-off-by: Igor Mammedov
Reviewed-by: Eduardo Habkost
Message-Id: <1503592308-93913-17-git-send-email-imamm...@redhat.com>
Signed-off-by: Eduardo Habkost
---
target/tricore/cpu.h | 4 +---
hw/t
From: Igor Mammedov
it's just a wrapper, drop it and use cpu_generic_init() directly
Signed-off-by: Igor Mammedov
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
Message-Id: <1503592308-93913-8-git-send-email-imamm...@redhat.com>
Signed-off-by: Eduardo Habkost
---
targ
From: Igor Mammedov
it's just a wrapper, drop it and use cpu_generic_init() directly
Signed-off-by: Igor Mammedov
Reviewed-by: Eduardo Habkost
Message-Id: <1503592308-93913-21-git-send-email-imamm...@redhat.com>
Signed-off-by: Eduardo Habkost
---
target/i386/cpu.h | 3 +--
target/i386/cpu.c
From: Igor Mammedov
SPARCCPU::env was initialized from previously set properties
(with help of sparc_cpu_parse_features) in cpu_sparc_register().
However there is not reason to keep it there as this task is
typically done at realize time. So move post properties
initialization into sparc_cpu_real
From: Igor Mammedov
it's just a wrapper, drop it and use cpu_generic_init() directly
Signed-off-by: Igor Mammedov
Reviewed-by: Eduardo Habkost
Message-Id: <1503592308-93913-18-git-send-email-imamm...@redhat.com>
Signed-off-by: Eduardo Habkost
---
target/sh4/cpu.h | 3 +--
hw/sh4/r2d.c |
From: Igor Mammedov
call xtensa_irq_init() at realize time which makes
cpu_xtensa_init() like generic cpu creation function.
As result we can replace it with cpu_generic_init()
which does the same job, reducing code duplication a bit.
Signed-off-by: Igor Mammedov
Reviewed-by: Eduardo Habkost
R
From: Igor Mammedov
it's just a wrapper, drop it and use cpu_generic_init() directly
Signed-off-by: Igor Mammedov
Reviewed-by: Eduardo Habkost
Message-Id: <1503592308-93913-25-git-send-email-imamm...@redhat.com>
Signed-off-by: Eduardo Habkost
---
target/unicore32/cpu.h| 4 +---
hw/unicor
From: Igor Mammedov
with features converted to properties we can use the same
approach as x86 for features parsing and drop legacy
approach that manipulated CPU instance directly.
New sparc_cpu_parse_features() will allow only +-feat
and explicitly disable feat=on|off syntax for now.
With that i
From: Igor Mammedov
it's just a wrapper, drop it and use cpu_generic_init() directly
Signed-off-by: Igor Mammedov
Reviewed-by: Eduardo Habkost
Message-Id: <1503592308-93913-23-git-send-email-imamm...@redhat.com>
Signed-off-by: Eduardo Habkost
---
target/moxie/cpu.h | 3 +--
hw/moxie/moxiesi
From: Igor Mammedov
cpu_tilegx_init() always falls back to TYPE_TILEGX_CPU object
regardless of cpu_model. Put fallback logic into
tilegx_cpu_class_by_name() which would translate any cpu_model
into TYPE_TILEGX_CPU class and replace cpu_tilegx_init()
with cpu_generic_init().
Signed-off-by: Igor
On Fri, Sep 01, 2017 at 04:11:44PM +0200, Gerd Hoffmann wrote:
> Hi,
>
> > I think it makes sense to create a separate project for the firmware
> > blobs. Move over the firmware binaries and source submodules to the
> > new project. This way updating both firmware sources and binaries
> > can
From: Igor Mammedov
cpu_s390x_init() is used only *-user targets indirectly
via cpu_init() macro and has a hack to assign ids to created
cpus (I'm not sure if 'id' really matters to *-user emulation).
So to on safe side, instead of having custom wrapper to do numbering
replace it with cpu_generi
From: Igor Mammedov
it's just a wrapper, drop it and use cpu_generic_init() directly
Signed-off-by: Igor Mammedov
Acked-by: Stafford Horne
Reviewed-by: Eduardo Habkost
Message-Id: <1503592308-93913-24-git-send-email-imamm...@redhat.com>
Signed-off-by: Eduardo Habkost
---
target/openrisc/cpu
On Wed, Aug 30, 2017 at 07:52:18PM -0300, Philippe Mathieu-Daudé wrote:
> Hi,
>
> This series is based on Igor's "complete cpu QOMification" [1] but only modify
> the MIPS part. Igor posted an updated series [2], both series should apply
> separately.
>
> Igor suggested on IRC this series could e
"Daniel P. Berrange" wrote:
> On Fri, Sep 01, 2017 at 01:54:59PM +0200, Juan Quintela wrote:
>> We were using -1 instead of the real size because the functions check
>> what is bigger, size in bytes or the size of the iov. Recent gcc's
>> barf at this.
>>
>> Signed-off-by: Juan Quintela
>> Revi
From: Igor Mammedov
call register_m68k_insns() at realize time which makes
cpu_m68k_init() typical object creation function.
As result we can replace it with cpu_generic_init()
which does the same job, reducing code duplication a bit.
Signed-off-by: Igor Mammedov
Tested-by: Thomas Huth
Acked-b
From: Igor Mammedov
it's just a wrapper, drop it and use cpu_generic_init() directly
Signed-off-by: Igor Mammedov
Acked-by: David Gibson
Reviewed-by: Hervé Poussineau
Reviewed-by: Eduardo Habkost
Message-Id: <1503592308-93913-26-git-send-email-imamm...@redhat.com>
Signed-off-by: Eduardo Habk
On 31.08.2017 16:21, Cornelia Huck wrote:
> On Wed, 30 Aug 2017 19:05:51 +0200
> David Hildenbrand wrote:
>
>> All but two, namely exec.c and dump.c, include exec/exec-all.h via
>> cpu.h only. as these files already include cpu.h, let's just drop the
>> additional include.
>>
>> Signed-off-by: Da
From: Igor Mammedov
cpu_nios2_init() always falls back to TYPE_NIOS2_CPU object
regardless of cpu_model. Put fallback logic into
nios2_cpu_class_by_name() which would translate any cpu_model
into TYPE_NIOS2_CPU class and replace cpu_nios2_init()
with cpu_generic_init()
Signed-off-by: Igor Mammed
A few tweaks to the docker images and running commands allow us to run
qemu-iotests in the fedora container. This will be added to the patchew test
cases later.
v3: Add nbd, luks and limit to quick group.
v2: Add nettle-devel because a few iotest cases blindly assumes the support is
there.
Fam Z
From: Igor Mammedov
cpu_mb_init() always falls back to TYPE_MICROBLAZE_CPU object
regardless of cpu_model. Put fallback logic into
mb_cpu_class_by_name() which would translate any cpu_model
into TYPE_MICROBLAZE_CPU class and replace cpu_mb_init()
with cpu_generic_init().
Signed-off-by: Igor Mamm
Signed-off-by: Fam Zheng
---
tests/docker/test-block | 22 ++
1 file changed, 22 insertions(+)
create mode 100755 tests/docker/test-block
diff --git a/tests/docker/test-block b/tests/docker/test-block
new file mode 100755
index 00..efc77b4b91
--- /dev/null
+++ b/test
From: Igor Mammedov
it's just a wrapper, drop it and use cpu_generic_init() directly
Signed-off-by: Igor Mammedov
Reviewed-by: Eduardo Habkost
Message-Id: <1503592308-93913-20-git-send-email-imamm...@redhat.com>
Signed-off-by: Eduardo Habkost
---
target/cris/cpu.h| 3 +--
hw/cris/axis_de
The LUKS cases in qemu-iotests requires this.
Signed-off-by: Fam Zheng
---
tests/docker/dockerfiles/fedora.docker | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/docker/dockerfiles/fedora.docker
b/tests/docker/dockerfiles/fedora.docker
index 4eaa8ed2a5..27e8201c54 100644
--- a/tests/d
101 - 200 of 381 matches
Mail list logo