On 5/3/24 06:51, Nicholas Piggin wrote:
On Thu May 2, 2024 at 6:47 PM AEST, Cédric Le Goater wrote:
On 5/1/24 14:39, Nicholas Piggin wrote:
On Wed Apr 17, 2024 at 9:25 PM AEST, Cédric Le Goater wrote:
Hello Nick,
On 4/17/24 13:02, Nicholas Piggin wrote:
This implements a framework for an
On 02/05/2024 07.44, Richard Henderson wrote:
Signed-off-by: Richard Henderson
---
target/s390x/cpu.h | 23 ++-
target/s390x/cpu.c | 22 ++
2 files changed, 24 insertions(+), 21 deletions(-)
Reviewed-by: Thomas Huth
On 02/05/2024 22.08, Michael Tokarev wrote:
02.05.2024 23:05, Richard Henderson wrote:
On 5/2/24 12:40, Michael Tokarev wrote:
Revert "gitlab-ci: Disable the riscv64-debian-cross-container by default"
This reverts commit f51f90c65ed7706c3c4f7a889ce3d6b7ab75ef6a.
Hopefully it wont be very
On Sat Apr 27, 2024 at 12:32 AM AEST, Cédric Le Goater wrote:
> On 4/26/24 13:00, Aditya Gupta wrote:
> > Make Power11 as default cpu type for 'pseries' and 'powernv' machine type,
> > with Power11 being the newest supported Power processor in QEMU.
>
> This is too early. We should merge Power11
On Thu May 2, 2024 at 6:47 PM AEST, Cédric Le Goater wrote:
> On 5/1/24 14:39, Nicholas Piggin wrote:
> > On Wed Apr 17, 2024 at 9:25 PM AEST, Cédric Le Goater wrote:
> >> Hello Nick,
> >>
> >> On 4/17/24 13:02, Nicholas Piggin wrote:
> >>> This implements a framework for an ADU unit model.
> >>>
On Wed Apr 24, 2024 at 7:30 PM AEST, Cédric Le Goater wrote:
> This helper routine uses the machine definition, sockets, cores and
> threads, to loop on all CPUs of the machine. Replace CPU_FOREACH()
> with it.
>
> Signed-off-by: Cédric Le Goater
Reviewed-by: Nicholas Piggin
> ---
>
On Thu May 2, 2024 at 6:32 PM AEST, Cédric Le Goater wrote:
> Hello Nick,
>
>
> >>> diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c
> >>> index 5869aac89a..eb9dbc62dd 100644
> >>> --- a/hw/ppc/pnv.c
> >>> +++ b/hw/ppc/pnv.c
> >>> @@ -1642,6 +1642,8 @@ static void pnv_chip_power9_realize(DeviceState
>
This new operation can store an immediate u64 value to a given
scoreboard.
Reviewed-by: Richard Henderson
Signed-off-by: Pierrick Bouvier
---
include/qemu/plugin.h | 1 +
include/qemu/qemu-plugin.h | 4 ++--
accel/tcg/plugin-gen.c | 13 +
plugins/core.c | 6
Count number of tb and insn executed using a conditional callback. We
ensure the callback has been called expected number of time (per vcpu).
Reviewed-by: Richard Henderson
Signed-off-by: Pierrick Bouvier
---
tests/plugin/inline.c | 89 +--
1 file
Plugin operations can access a scoreboard. This function factorizes code
generation for accessing entry associated to a given vcpu.
Reviewed-by: Richard Henderson
Signed-off-by: Pierrick Bouvier
---
accel/tcg/plugin-gen.c | 27 ++-
1 file changed, 18 insertions(+), 9
Reviewed-by: Richard Henderson
Signed-off-by: Pierrick Bouvier
---
tests/plugin/inline.c | 41 +
1 file changed, 37 insertions(+), 4 deletions(-)
diff --git a/tests/plugin/inline.c b/tests/plugin/inline.c
index 0163e9b51c5..103c3a22f6e 100644
---
Extend plugins API to support callback called with a given criteria
(evaluated inline).
Added functions:
- qemu_plugin_register_vcpu_tb_exec_cond_cb
- qemu_plugin_register_vcpu_insn_exec_cond_cb
They expect as parameter a condition, a qemu_plugin_u64_t (op1) and an
immediate (op2). Callback is
Factorizes function to access current cpu index for a given vcpu.
Reviewed-by: Richard Henderson
Signed-off-by: Pierrick Bouvier
---
accel/tcg/plugin-gen.c | 28 +---
1 file changed, 13 insertions(+), 15 deletions(-)
diff --git a/accel/tcg/plugin-gen.c
On 5/2/24 13:48, Gustavo Romero wrote:
Hi Pierrick,
On 5/2/24 5:09 PM, Gustavo Romero wrote:
On 5/2/24 4:45 PM, Pierrick Bouvier wrote:
Hi Gustavo,
On 5/2/24 12:16, Gustavo Romero wrote:
Hi Pierrick,
On 5/2/24 3:08 PM, Pierrick Bouvier wrote:
This series implement two new operations for
This series implement two new operations for plugins:
- Store inline allows to write a specific value to a scoreboard.
- Conditional callback executes a callback only when a given condition is true.
The condition is evaluated inline.
It's possible to mix various inline operations (add, store)
Until now, only add_u64 was available, and all functions assumed this or
were named uniquely.
Reviewed-by: Richard Henderson
Signed-off-by: Pierrick Bouvier
---
include/qemu/plugin.h | 2 +-
accel/tcg/plugin-gen.c | 6 +++---
plugins/core.c | 14 --
3 files changed, 16
To prevent errors when writing new types of callbacks or inline
operations, we split callbacks data to distinct types.
Reviewed-by: Richard Henderson
Signed-off-by: Pierrick Bouvier
---
include/qemu/plugin.h | 46 ++---
plugins/plugin.h | 2 +-
This field is not needed as the callback type already holds this
information.
Reviewed-by: Richard Henderson
Signed-off-by: Pierrick Bouvier
---
include/qemu/plugin.h | 1 -
plugins/plugin.h | 4 +++-
plugins/core.c| 13 +++--
3 files changed, 10 insertions(+), 8
02.05.2024 23:41, Richard Henderson wrote:
On 5/2/24 13:08, Michael Tokarev wrote:
This thing works now with sid/unstable, this is what this patch is about.
It worked when first introduced, then failed, then worked...
Sid is too unstable for our usage as a gating test.
unstable can be
Hi Pierrick,
On 5/2/24 5:09 PM, Gustavo Romero wrote:
On 5/2/24 4:45 PM, Pierrick Bouvier wrote:
Hi Gustavo,
On 5/2/24 12:16, Gustavo Romero wrote:
Hi Pierrick,
On 5/2/24 3:08 PM, Pierrick Bouvier wrote:
This series implement two new operations for plugins:
- Store inline allows to write a
On 5/2/24 13:08, Michael Tokarev wrote:
This thing works now with sid/unstable, this is what this patch is about.
It worked when first introduced, then failed, then worked...
Sid is too unstable for our usage as a gating test.
r~
Peter Xu writes:
> On Fri, Apr 26, 2024 at 11:20:33AM -0300, Fabiano Rosas wrote:
>> If the user is not passing in a file name which QEMU can open at will,
>> we must then require that the user pass the two file descriptors with
>> the flags already properly set. We'll use the already existing
From: Vladimir Sementsov-Ogievskiy
Cover more cases by trace-point.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: Fabiano Rosas
Reviewed-by: Peter Xu
Signed-off-by: Fabiano Rosas
---
migration/migration.c | 4 +++-
migration/trace-events | 2 +-
2 files changed, 4
The following changes since commit 4977ce198d2390bff8c71ad5cb1a5f6aa24b56fb:
Merge tag 'pull-tcg-20240501' of https://gitlab.com/rth7680/qemu into staging
(2024-05-01 15:15:33 -0700)
are available in the Git repository at:
https://gitlab.com/farosas/qemu.git tags/migration-20240502-pull
The block incremental option for block migration has been deprecated
in 8.2 in favor of using the block-mirror feature. Remove it now.
Deprecation commit 40101f320d ("migration: migrate 'inc' command
option is deprecated.").
Reviewed-by: Markus Armbruster
Reviewed-by: Peter Xu
Signed-off-by:
The fd: URI can currently trigger two different types of migration, a
TCP migration using sockets and a file migration using a plain
file. This is in conflict with the recently introduced (8.2) QMP
migrate API that takes structured data as JSON-like format. We cannot
keep the same backend for both
The block migration is considered obsolete and has been deprecated in
8.2. Remove the migrate command option that enables it. This only
affects the QMP and HMP commands, the feature can still be accessed by
setting the migration 'block' capability. The whole feature will be
removed in a future
From: Vladimir Sementsov-Ogievskiy
It's bad idea to leave critical section with error object freed, but
s->error still set, this theoretically may lead to use-after-free
crash. Let's avoid it.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: Fabiano Rosas
Reviewed-by: Peter Xu
From: Vladimir Sementsov-Ogievskiy
Unify error reporting in the function. This simplifies the following
commit, which will not-exit-on-error behavior variant to the function.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Peter Xu
Signed-off-by:
The 'compress' migration capability enables the old compression code
which has shown issues over the years and is thought to be less stable
and tested than the more recent multifd-based compression. The old
compression code has been deprecated in 8.2 and now is time to remove
it.
Deprecation
From: Will Gyda
migration/ram.c: API Conversion qemu_mutex_lock(),
and qemu_mutex_unlock() to WITH_QEMU_LOCK_GUARD macro
Signed-off-by: Will Gyda
Reviewed-by: Peter Xu
Signed-off-by: Fabiano Rosas
---
migration/ram.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff
From: Peter Xu
Peter missed the Sphinx HMP document for the "resume/-r" flag in commit
7a4da28b26 ("qmp: hmp: add migrate "resume" option"). Add it. Avoid
adding a Fixes to make life easier for the stable maintainer.
When at it, slightly cleanup the lines, move "detach/-d" to a separate
From: Vladimir Sementsov-Ogievskiy
Make call to migration_incoming_state_destroy(), instead of doing only
partial of it.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: Fabiano Rosas
Reviewed-by: Peter Xu
Signed-off-by: Fabiano Rosas
---
migration/migration.c | 5 +
1 file
From: Vladimir Sementsov-Ogievskiy
Now we do set MIGRATION_FAILED state, but don't give a chance to
orchestrator to query migration state and get the error.
Let's provide a possibility for QMP-based orchestrators to get an error
like with outgoing migration.
For hmp_migrate_incoming(), let's
The block migration has been considered obsolete since QEMU 8.2 in
favor of the more flexible storage migration provided by the
blockdev-mirror driver. Two releases have passed so now it's time to
remove it.
Deprecation commit 66db46ca83 ("migration: Deprecate block
migration").
Reviewed-by:
The 'skipped' field of the MigrationStats struct has been deprecated
in 8.1. Time to remove it.
Deprecation commit 7b24d32634 ("migration: skipped field is really
obsolete.").
Reviewed-by: Markus Armbruster
Reviewed-by: Peter Xu
Signed-off-by: Fabiano Rosas
---
docs/about/deprecated.rst
On 5/2/24 4:45 PM, Pierrick Bouvier wrote:
Hi Gustavo,
On 5/2/24 12:16, Gustavo Romero wrote:
Hi Pierrick,
On 5/2/24 3:08 PM, Pierrick Bouvier wrote:
This series implement two new operations for plugins:
- Store inline allows to write a specific value to a scoreboard.
- Conditional callback
02.05.2024 23:05, Richard Henderson wrote:
On 5/2/24 12:40, Michael Tokarev wrote:
Revert "gitlab-ci: Disable the riscv64-debian-cross-container by default"
This reverts commit f51f90c65ed7706c3c4f7a889ce3d6b7ab75ef6a.
Hopefully it wont be very unstable.
Since riscv64 is now a release
On 5/2/24 12:40, Michael Tokarev wrote:
Revert "gitlab-ci: Disable the riscv64-debian-cross-container by default"
This reverts commit f51f90c65ed7706c3c4f7a889ce3d6b7ab75ef6a.
Hopefully it wont be very unstable.
Since riscv64 is now a release architecture in debian, we can try switching
to
On Thu, 2 May 2024, Edgar E. Iglesias wrote:
> On Thu, May 2, 2024 at 8:53 PM Stefano Stabellini
> wrote:
> >
> > +Xenia
> >
> > On Thu, 2 May 2024, Edgar E. Iglesias wrote:
> > > On Wed, May 1, 2024 at 11:24 PM Stefano Stabellini
> > > wrote:
> > > >
> > > > On Tue, 30 Apr 2024, Edgar E.
On Fri, Apr 26, 2024 at 11:20:33AM -0300, Fabiano Rosas wrote:
> If the user is not passing in a file name which QEMU can open at will,
> we must then require that the user pass the two file descriptors with
> the flags already properly set. We'll use the already existing fdset +
> QMP add-fd
On 5/2/24 12:20, Matheus Tavares Bernardino wrote:
The Hexagon Programmer's Reference Manual says that the exception 0x1e
should be raised upon an unaligned program counter. Let's implement that
and also add some tests.
Signed-off-by: Matheus Tavares Bernardino
Reviewed-by: Richard Henderson
https://gitlab.com/mjt0k/qemu/-/jobs/6768635797 fwiw.
/mjt
--
GPG Key transition (from rsa2048 to rsa4096) since 2024-04-24.
New key: rsa4096/61AD3D98ECDF2C8E 9D8B E14E 3F2A 9DD7 9199 28F1 61AD 3D98
ECDF 2C8E
Old key: rsa2048/457CE0A0804465C5 6EE1 95D1 886E 8FFB 810D 4324 457C E0A0
8044
Revert "gitlab-ci: Disable the riscv64-debian-cross-container by default"
This reverts commit f51f90c65ed7706c3c4f7a889ce3d6b7ab75ef6a.
Hopefully it wont be very unstable.
Since riscv64 is now a release architecture in debian, we can try switching
to debian testing instead of debian sid. Also,
On Thu, May 2, 2024 at 9:18 PM Stefano Stabellini
wrote:
>
> On Tue, 30 Apr 2024, Edgar E. Iglesias wrote:
> > From: "Edgar E. Iglesias"
> >
> > Add a second mapcache for grant mappings. The mapcache for
> > grants needs to work with XC_PAGE_SIZE granularity since
> > we can't map larger ranges
Hi Gustavo,
On 5/2/24 12:16, Gustavo Romero wrote:
Hi Pierrick,
On 5/2/24 3:08 PM, Pierrick Bouvier wrote:
This series implement two new operations for plugins:
- Store inline allows to write a specific value to a scoreboard.
- Conditional callback executes a callback only when a given
On Thu, May 2, 2024 at 8:53 PM Stefano Stabellini
wrote:
>
> +Xenia
>
> On Thu, 2 May 2024, Edgar E. Iglesias wrote:
> > On Wed, May 1, 2024 at 11:24 PM Stefano Stabellini
> > wrote:
> > >
> > > On Tue, 30 Apr 2024, Edgar E. Iglesias wrote:
> > > > From: "Edgar E. Iglesias"
> > > >
> > > > The
Ping.
On 4/24/24 10:09, Richard Henderson wrote:
One patch to allow two output operands from gvec expansion,
to be used by target/arm for updating QC.
One patch to record the result of the generic breakpoint
search so target translators do not need to repeat it.
Three small optimization
The Hexagon Programmer's Reference Manual says that the exception 0x1e
should be raised upon an unaligned program counter. Let's implement that
and also add some tests.
Signed-off-by: Matheus Tavares Bernardino
Reviewed-by: Richard Henderson
Reviewed-by: Taylor Simpson
---
Changes in v5:
-
On Tue, 30 Apr 2024, Edgar E. Iglesias wrote:
> From: "Edgar E. Iglesias"
>
> Add a second mapcache for grant mappings. The mapcache for
> grants needs to work with XC_PAGE_SIZE granularity since
> we can't map larger ranges than what has been granted to us.
>
> Like with foreign mappings
Hi Pierrick,
On 5/2/24 3:08 PM, Pierrick Bouvier wrote:
This series implement two new operations for plugins:
- Store inline allows to write a specific value to a scoreboard.
- Conditional callback executes a callback only when a given condition is true.
The condition is evaluated inline.
+Xenia
On Thu, 2 May 2024, Edgar E. Iglesias wrote:
> On Wed, May 1, 2024 at 11:24 PM Stefano Stabellini
> wrote:
> >
> > On Tue, 30 Apr 2024, Edgar E. Iglesias wrote:
> > > From: "Edgar E. Iglesias"
> > >
> > > The current mapcache assumes that all memory is mapped
> > > in a single RAM MR
Contrary to what the cover letter says, all patches have been reviewed
in the series since v4.
On 5/2/24 11:08, Pierrick Bouvier wrote:
This series implement two new operations for plugins:
- Store inline allows to write a specific value to a scoreboard.
- Conditional callback executes a
On Thu, May 02, 2024 at 12:13:17PM -0400, Steven Sistare wrote:
> On 4/29/2024 11:55 AM, Steve Sistare wrote:
> > This patch series adds the live migration cpr-exec mode.
>
> Here is the text I plan to add to docs/devel/migration/CPR.rst. It is
> premature for me to submit this as a patch,
Reviewed-by: Richard Henderson
Signed-off-by: Pierrick Bouvier
---
accel/tcg/plugin-gen.c | 27 ++-
1 file changed, 18 insertions(+), 9 deletions(-)
diff --git a/accel/tcg/plugin-gen.c b/accel/tcg/plugin-gen.c
index 8028ae76c3a..8e2c3ef94f6 100644
---
Reviewed-by: Richard Henderson
Signed-off-by: Pierrick Bouvier
---
tests/plugin/inline.c | 41 +
1 file changed, 37 insertions(+), 4 deletions(-)
diff --git a/tests/plugin/inline.c b/tests/plugin/inline.c
index 0163e9b51c5..103c3a22f6e 100644
---
Count number of tb and insn executed using a conditional callback. We
ensure the callback has been called expected number of time (per vcpu).
Reviewed-by: Richard Henderson
Signed-off-by: Pierrick Bouvier
---
tests/plugin/inline.c | 89 +--
1 file
To prevent errors when writing new types of callbacks or inline
operations, we split callbacks data to distinct types.
Reviewed-by: Richard Henderson
Signed-off-by: Pierrick Bouvier
---
include/qemu/plugin.h | 46 ++---
plugins/plugin.h | 2 +-
Extend plugins API to support callback called with a given criteria
(evaluated inline).
Added functions:
- qemu_plugin_register_vcpu_tb_exec_cond_cb
- qemu_plugin_register_vcpu_insn_exec_cond_cb
They expect as parameter a condition, a qemu_plugin_u64_t (op1) and an
immediate (op2). Callback is
Reviewed-by: Richard Henderson
Signed-off-by: Pierrick Bouvier
---
accel/tcg/plugin-gen.c | 28 +---
1 file changed, 13 insertions(+), 15 deletions(-)
diff --git a/accel/tcg/plugin-gen.c b/accel/tcg/plugin-gen.c
index b829a959398..7b73520e788 100644
---
Reviewed-by: Richard Henderson
Signed-off-by: Pierrick Bouvier
---
include/qemu/plugin.h | 1 +
include/qemu/qemu-plugin.h | 4 ++--
accel/tcg/plugin-gen.c | 13 +
plugins/core.c | 6 ++
4 files changed, 22 insertions(+), 2 deletions(-)
diff --git
Until now, only add_u64 was available, and all functions assumed this or
were named uniquely.
Reviewed-by: Richard Henderson
Signed-off-by: Pierrick Bouvier
---
include/qemu/plugin.h | 2 +-
accel/tcg/plugin-gen.c | 6 +++---
plugins/core.c | 14 --
3 files changed, 16
Reviewed-by: Richard Henderson
Signed-off-by: Pierrick Bouvier
---
include/qemu/plugin.h | 1 -
plugins/plugin.h | 4 +++-
plugins/core.c| 13 +++--
3 files changed, 10 insertions(+), 8 deletions(-)
diff --git a/include/qemu/plugin.h b/include/qemu/plugin.h
index
This series implement two new operations for plugins:
- Store inline allows to write a specific value to a scoreboard.
- Conditional callback executes a callback only when a given condition is true.
The condition is evaluated inline.
It's possible to mix various inline operations (add, store)
Peter Xu writes:
> Peter missed the Sphinx HMP document for the "resume/-r" flag in commit
> 7a4da28b26 ("qmp: hmp: add migrate "resume" option"). Add it. Avoid
> adding a Fixes to make life easier for the stable maintainer.
>
> When at it, slightly cleanup the lines, move "detach/-d" to a
On 02/05/2024 15.15, Cédric Le Goater wrote:
get_sclp_device() scans the whole machine to find a TYPE_SCLP object.
Now that the SCLPDevice instance is available under the machine state,
use it to simplify the lookup. While at it, remove the inline to let
the compiler decide on how to optimize.
On 02/05/2024 15.15, Cédric Le Goater wrote:
sclp_get_event_facility_bus() scans the whole machine to find a
TYPE_SCLP_EVENTS_BUS object. The SCLPDevice instance is now available
under the machine state, use it to simplify the lookup and adjust the
creation of the consoles.
Signed-off-by:
Hi Peter
On Thu, May 2, 2024 at 6:20 PM Peter Xu wrote:
>
> On Thu, May 02, 2024 at 03:30:58PM +0200, Jinpu Wang wrote:
> > Hi Michael, Hi Peter,
> >
> >
> > On Thu, May 2, 2024 at 3:23 PM Michael Galaxy wrote:
> > >
> > > Yu Zhang / Jinpu,
> > >
> > > Any possibility (at your lesiure, and
On 02/05/2024 15.15, Cédric Le Goater wrote:
Initialize directly SCLPDevice from the machine init handler and
remove s390_sclp_init(). We will use the SCLPDevice pointer later to
create the consoles.
Signed-off-by: Cédric Le Goater
---
Reviewed-by: Thomas Huth
These instructions have f32 inputs, which changes the decode
of the register numbers. While we're fixing things, use a
common helper for both insns, extracting the 16-bit scalar
in tcg beforehand.
Signed-off-by: Richard Henderson
---
target/sparc/helper.h | 3 +--
target/sparc/translate.c
This instruction has f32 as source1, which alters the
decoding of the register number, which means we've been
passing the wrong data for odd register numbers.
Signed-off-by: Richard Henderson
---
target/sparc/helper.h | 2 +-
target/sparc/translate.c | 21 -
Supply HWCAP_SPARC_V8PLUS, HWCAP_SPARC_MUL32, HWCAP_SPARC_DIV32,
HWCAP_SPARC_POPC, HWCAP_SPARC_FSMULD, HWCAP_SPARC_VIS, HWCAP_SPARC_VIS2.
Signed-off-by: Richard Henderson
---
linux-user/elfload.c | 48 +++-
1 file changed, 34 insertions(+), 14
Split out from my vis4 patch set, with just the bug fixes.
I've fixed the issue in patch 6, as noticed by Mark, but
include the follow-up that cleans up all of the macros by
removing them.
r~
Richard Henderson (7):
linux-user/sparc: Add more hwcap bits for sparc64
target/sparc: Fix FEXPAND
This is a 2-operand instruction, not 3-operand.
Worse, we took the source from the wrong operand.
Signed-off-by: Richard Henderson
---
target/sparc/helper.h | 2 +-
target/sparc/insns.decode | 2 +-
target/sparc/translate.c | 20 +++-
target/sparc/vis_helper.c | 6 +++---
This instruction has f32 inputs, which changes the decode
of the register numbers.
Signed-off-by: Richard Henderson
---
target/sparc/helper.h | 2 +-
target/sparc/translate.c | 2 +-
target/sparc/vis_helper.c | 27 ++-
3 files changed, 16 insertions(+), 15
The unit operation for fmul8x16 and friends is described in the
manual as "MS16b". Split that out for clarity. Improve rounding
with an unconditional addition of 0.5 as a fixed-point integer.
Signed-off-by: Richard Henderson
---
target/sparc/vis_helper.c | 78
Not only do these instructions have f32 inputs, they also do not
perform rounding. Since these are relatively simple, implement
them properly inline.
Signed-off-by: Richard Henderson
---
target/sparc/helper.h | 2 --
target/sparc/translate.c | 48 +++
On 02/05/2024 16.57, Eric Blake wrote:
On Wed, May 01, 2024 at 07:27:46PM +0100, Daniel P. Berrangé wrote:
The various targets which define versioned machine types have
a bunch of obfuscated macro code for defining unique function
and variable names using string concatenation.
This addes a
> -Original Message-
> From: Matheus Tavares Bernardino
> Sent: Thursday, May 2, 2024 9:55 AM
> To: qemu-devel@nongnu.org
> Cc: bc...@quicinc.com; sidn...@quicinc.com; a...@rev.ng; a...@rev.ng;
> ltaylorsimp...@gmail.com; richard.hender...@linaro.org; Laurent Vivier
>
> Subject:
On Thu, May 02, 2024 at 03:30:58PM +0200, Jinpu Wang wrote:
> Hi Michael, Hi Peter,
>
>
> On Thu, May 2, 2024 at 3:23 PM Michael Galaxy wrote:
> >
> > Yu Zhang / Jinpu,
> >
> > Any possibility (at your lesiure, and within the disclosure rules of
> > your company, IONOS) if you could share any
On 4/29/2024 11:55 AM, Steve Sistare wrote:
This patch series adds the live migration cpr-exec mode.
Here is the text I plan to add to docs/devel/migration/CPR.rst. It is
premature for me to submit this as a patch, because it includes all
the functionality I plan to add in this and future
On Thu, May 02, 2024 at 12:35:38PM +0200, Yu Zhang wrote:
> As the links [1][2] below stated, QEMU development community is currently
> having some difficulties in maintaining the RDMA migration subsystem due
> to the lack of resources (maintainers, test cases, test environment etc.)
> and
Hi Paolo, thank you for looking.
On Thu, May 2, 2024 at 8:19 AM Paolo Bonzini wrote:
>
> Anyway, just out of curiosity I tried to see what it would take to
> compile edu.c as C++ code, which I think lets us give a more informed
> answer.
>
> There were a bunch of conflicts with C++ keyword,
On 5/2/2024 8:23 AM, Markus Armbruster wrote:
Steve Sistare writes:
Add the cpr-exec migration mode. Usage:
qemu-system-$arch -machine memfd-alloc=on ...
migrate_set_parameter mode cpr-exec
migrate_set_parameter cpr-exec-args \
... -incoming
migrate -d
The migrate
On 5/2/2024 8:23 AM, Markus Armbruster wrote:
Steve Sistare writes:
Create the cpr-exec-args migration parameter, defined as a list of
strings. It will be used for cpr-exec migration mode in a subsequent
patch.
No functional change, except that cpr-exec-args is shown by the
'info migrate'
hw/core/cpu.h is already using struct forward declarations in some cases
to avoid inclusions, and otherwise CPUAddressSpace and CPUJumpCache
are only used together with their definition. CPUTLBEntryFull is
always used when their definition is available. Remove all three
from typedefs.h.
They are needed in very few places, which already depends on other generated
QAPI
files. The benefit of having these types in typedefs.h is small.
Signed-off-by: Paolo Bonzini
---
include/hw/core/cpu.h | 1 +
include/qemu/typedefs.h | 2 --
2 files changed, 1 insertion(+), 2 deletions(-)
It is needed in very few places, which already depend on other parts of
qdev-core.h files. The benefit of having it in typedefs.h is small.
Signed-off-by: Paolo Bonzini
---
include/hw/qdev-core.h | 1 +
include/qemu/typedefs.h | 1 -
2 files changed, 1 insertion(+), 1 deletion(-)
diff --git
Exactly nobody needs them there. Place the typedef in the header
that defines the struct.
Signed-off-by: Paolo Bonzini
---
include/qemu/typedefs.h | 2 --
include/sysemu/numa.h | 8
2 files changed, 4 insertions(+), 6 deletions(-)
diff --git a/include/qemu/typedefs.h
For types that are embedded in structs defined by pci.h, the definition
is pretty much required to be available. Remove them from typedefs.h.
Signed-off-by: Paolo Bonzini
---
include/hw/pci/pcie.h | 3 +++
include/hw/pci/pcie_aer.h | 38 ++---
Only FWCfgState is used as part of APIs such as acpi_ghes_add_fw_cfg.
Everything else need not be in typedefs.h.
Signed-off-by: Paolo Bonzini
---
include/hw/nvram/fw_cfg.h | 2 ++
include/qemu/typedefs.h | 3 ---
2 files changed, 2 insertions(+), 3 deletions(-)
diff --git
Using QemuLockable almost always requires going through QEMU_MAKE_LOCKABLE().
Therefore, there is little point in having the typedef always present. Move
it to lockable.h, with only a small adjustment to coroutine.h (which has
a tricky co-dependency with lockable.h due to defining CoMutex *and*
Exactly nobody needs it there. Place the typedef in the header
that defines the struct.
Signed-off-by: Paolo Bonzini
---
include/net/announce.h | 4 ++--
include/qemu/typedefs.h | 1 -
2 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/include/net/announce.h
Basically all uses of GraphicHwOps are defining an instance of it, which
requires the
full definition of the struct. It is pointless to have it in typedefs.h.
Signed-off-by: Paolo Bonzini
---
hw/display/vga_int.h| 1 +
include/qemu/typedefs.h | 1 -
2 files changed, 1 insertion(+), 1
Move it to the existing "PIC related things" header, hw/intc/i8259.h.
Signed-off-by: Paolo Bonzini
---
include/hw/intc/i8259.h | 2 ++
include/qemu/typedefs.h | 1 -
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/include/hw/intc/i8259.h b/include/hw/intc/i8259.h
index
QemuOpt is basically an internal data structure. It has no business
being defined except if you need functions from include/qemu/option.h.
Signed-off-by: Paolo Bonzini
---
include/qemu/option.h | 2 ++
include/qemu/typedefs.h | 1 -
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git
It is defined and referred to exclusively from a .c file.
Signed-off-by: Paolo Bonzini
---
include/qemu/typedefs.h | 1 -
migration/postcopy-ram.c | 4 ++--
2 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/include/qemu/typedefs.h b/include/qemu/typedefs.h
index
MonitorDef is defined by hmp-target.h, and all users except one already
include it; the reason why the stubs do not include it, is because
hmp-target.h currently can only be used in files that are compiled
per target. However, that is easily fixed. Because the benefit of
having MonitorDef in
The main reason for typedefs.h to exist is to avoid the need for
"struct Foo;" forward declarations of types that are needed in function
prototypes. It avoids the infamous "struct declared inside parameter list
will not be visible outside of this definition or declaration" warning.
Another less
Let hw/hyperv/hyperv.c and hw/intc/s390_flic.c handle (respectively)
SynIC and adapter routes, removing the code from target-independent
files. This also removes the only occurrence of AdapterInfo outside
s390 code, so remove that from typedefs.h.
Signed-off-by: Paolo Bonzini
---
1 - 100 of 232 matches
Mail list logo
