Hi Drew,
I got a question below, and hope your reply. Thanks!
On 2021/4/13 16:07, Yanan Wang wrote:
Add the Processor Properties Topology Table (PPTT) to present
CPU topology information to ACPI guests. Note, while a DT boot
Linux guest with a non-flat CPU topology will see socket and
core IDs
On Wed, May 12, 2021 at 03:54:15PM -0300, matheus.fe...@eldorado.org.br wrote:
> From: Richard Henderson
>
> Since POWERPC_EXCP_TRAP is raised by gen_exception_err,
> we will have also set DISAS_NORETURN.
>
> Signed-off-by: Richard Henderson
> Reviewed-by: Luis Pires
> Signed-off-by: Matheus
On Wed, May 12, 2021 at 03:54:19PM -0300, matheus.fe...@eldorado.org.br wrote:
> From: Richard Henderson
>
> Since ba3e7926691ed33, we switched the implementation of icount
> to always reset can_do_io at the start of the following TB.
> Most of them were removed in 9e9b10c64911, but some were
On Wed, May 12, 2021 at 12:08:52PM -0500, Richard Henderson wrote:
> On 5/12/21 9:08 AM, Bruno Larsen (billionai) wrote:
> > Updated the code in machine.c to use the generic ppc_{store,get}_vscr
> > instead of helper style functions, so it can build without TCG
> >
> > Signed-off-by: Bruno Larsen
On Wed, May 12, 2021 at 03:54:18PM -0300, matheus.fe...@eldorado.org.br wrote:
> From: Richard Henderson
>
> Remove the synthetic "exception" after no more uses.
>
> Signed-off-by: Richard Henderson
> Signed-off-by: Matheus Ferst
Applied to ppc-for-6.1, thanks.
> ---
> target/ppc/cpu.h
On Wed, May 12, 2021 at 03:54:13PM -0300, matheus.fe...@eldorado.org.br wrote:
> From: Richard Henderson
>
> There are other valid settings for is_jmp besides
> DISAS_NEXT and DISAS_NORETURN, so eliminating that
> dichotomy from ppc_tr_translate_insn is helpful.
>
> Signed-off-by: Richard
On Wed, May 12, 2021 at 03:54:12PM -0300, matheus.fe...@eldorado.org.br wrote:
> From: Richard Henderson
>
> Signed-off-by: Richard Henderson
> Reviewed-by: Luis Pires
> Signed-off-by: Matheus Ferst
Applied to ppc-for-6.1.
> ---
> target/ppc/translate.c | 115
On Wed, May 12, 2021 at 03:54:10PM -0300, matheus.fe...@eldorado.org.br wrote:
> From: Matheus Ferst
>
> This series provides the basic infrastructure for adding the new 32/64-bit
> instructions in Power ISA 3.1 to target/ppc.
I've applied the first chunk of these. After that I get conflicts
On Wed, May 12, 2021 at 03:54:17PM -0300, matheus.fe...@eldorado.org.br wrote:
> From: Richard Henderson
>
> Rewrite ppc_tr_tb_stop to handle these new codes.
>
> Convert ctx->exception into these new codes at the end of
> ppc_tr_translate_insn, prior to pushing the change back
> throughout
On Wed, May 12, 2021 at 03:54:11PM -0300, matheus.fe...@eldorado.org.br wrote:
> From: Richard Henderson
>
> Signed-off-by: Richard Henderson
> Reviewed-by: Bruno Larsen (billionai)
> Reviewed-by: Luis Pires
> Signed-off-by: Matheus Ferst
Applied to ppc-for-6.1, thanks.
> ---
>
On Wed, May 12, 2021 at 03:54:16PM -0300, matheus.fe...@eldorado.org.br wrote:
> From: Richard Henderson
>
> Two of the call sites that use gen_debug_exception have already
> updated NIP. Only ppc_tr_breakpoint_check requires the update.
>
> Signed-off-by: Richard Henderson
> Signed-off-by:
On Wed, May 12, 2021 at 03:54:14PM -0300, matheus.fe...@eldorado.org.br wrote:
> From: Richard Henderson
>
> Since POWERPC_SYSCALL is raised by gen_exception_err,
> we will have also set DISAS_NORETURN.
>
> Signed-off-by: Richard Henderson
> Reviewed-by: Luis Pires
> Signed-off-by: Matheus
On Wed, May 12, 2021 at 04:21:13PM -0300, Matheus K. Ferst wrote:
> On 12/05/2021 15:54, matheus.fe...@eldorado.org.br wrote:
> > From: Matheus Ferst
> >
> > Create a function to handle the details for interacting with icount.
> >
> > Force the exit from the tb via DISAS_TOO_MANY, which allows
On Wed, May 12, 2021 at 11:08:12AM -0300, Bruno Larsen (billionai) wrote:
> Created a file with stubs needed to compile disabling TCG.
>
> We're not sure about keeping the softmmu stubs in this file. if there is
> a better place to put them, please let us know.
>
> The other 3 functions have
On Wed, May 12, 2021 at 11:08:04AM -0300, Bruno Larsen (billionai) wrote:
> Moved this function that is required in !TCG cases into a
> common code file
The reasons it's needed by !TCG are kind of bogus, related to
weirdness in the way KVM PR works. But it's fair not to care about
that right
On Wed, May 12, 2021 at 11:08:05AM -0300, Bruno Larsen (billionai) wrote:
> This function was forgotten in the cpu_init code motion series, but it
> seems to be used regardless of TCG, and so needs to be moved to support
> disabling TCG.
>
> Signed-off-by: Bruno Larsen (billionai)
>
Applied to
On Wed, May 12, 2021 at 12:05:04PM -0500, Richard Henderson wrote:
> On 5/12/21 9:08 AM, Bruno Larsen (billionai) wrote:
> > This function is used by !TCG cases, so it was moved to a common code
> > file. We chose gdbstub.c since it was the one giving us grief over it.
> >
> > Signed-off-by:
On Thu, May 06, 2021 at 08:19:24AM +0530, Vaibhav Jain wrote:
> Add support for H_SCM_PERFORMANCE_STATS described at [1] for
> spapr nvdimms. This enables guest to fetch performance stats[2] like
> expected life of an nvdimm ('MemLife ') etc and display them to the
> user. Linux kernel support for
On Wed, May 12, 2021 at 10:01:20AM +0200, Paolo Bonzini wrote:
> On 12/05/21 04:27, David Gibson wrote:
> > On Tue, May 11, 2021 at 05:53:52PM +0200, Philippe Mathieu-Daudé wrote:
> > > Since commit fea35ca4b8e ("ppc/spapr: Receive and store device
> > > tree blob from SLOF") the pSeries machine
On Wed, May 05, 2021 at 08:18:27PM +0530, Mahesh Salgaonkar wrote:
> With upstream kernel, especially after commit 98ba956f6a389
> ("powerpc/pseries/eeh: Rework device EEH PE determination") we see that KVM
> guest isn't able to enable EEH option for PCI pass-through devices anymore.
>
>
On Wed, May 12, 2021 at 11:08:03AM -0300, Bruno Larsen (billionai) wrote:
> Some functions unrelated to TCG use helper_m{t,f}vscr, so generic versions
> of those functions were added to cpu.c, in preparation for compilation
> without TCG
>
> Signed-off-by: Bruno Larsen (billionai)
>
Applied to
On Wed, May 12, 2021 at 11:08:07AM -0300, Bruno Larsen (billionai) wrote:
> This function is used in !TCG cases, so it has been moved into a file
> that is compiled when --disable-tcg is selected.
>
> Signed-off-by: Bruno Larsen (billionai)
>
Reviewed-by: David Gibson
> ---
>
On Wed, May 12, 2021 at 09:59:00AM +0200, Paolo Bonzini wrote:
> On 12/05/21 04:30, David Gibson wrote:
> > On Tue, May 11, 2021 at 05:53:53PM +0200, Philippe Mathieu-Daudé wrote:
> > > hw/ppc/fdt.c defines the ppc_create_page_sizes_prop() function,
> > > which is unrelated to the libfdt. Remove
On Wed, Apr 28, 2021 at 10:29 PM Changbin Du wrote:
>
> This dumps the CSR mscratch/sscratch/satp and meanwhile aligns
> the output of CSR mtval/stval.
>
> Signed-off-by: Changbin Du
Reviewed-by: Alistair Francis
Alistair
> ---
> target/riscv/cpu.c | 7 +--
> 1 file changed, 5
On Tue, May 4, 2021 at 6:45 AM Jose Martins wrote:
>
> VS interrupts (2, 6, 10) were not correctly forwarded to hs-mode when
> not delegated in hideleg (which was not being taken into account). This
> was mainly because hs level sie was not always considered enabled when
> it should. The spec
This was fixed by commit:
commit 65606f21243a796537bfe4708720a9bf4bb50169
Author: LIU Zhiwei
Date: Fri Feb 12 23:02:21 2021 +0800
target/riscv: Fixup saturate subtract function
The overflow predication ((a - b) ^ a) & (a ^ b) & INT64_MIN is right.
However, when the
On Wed, May 12, 2021 at 06:53:22AM +0200, Thomas Huth wrote:
> On 12/05/2021 05.57, Philippe Mathieu-Daudé wrote:
> > On 5/12/21 4:24 AM, David Gibson wrote:
> > > On Tue, May 11, 2021 at 05:53:50PM +0200, Philippe Mathieu-Daudé wrote:
> > > > Per the kconfig.rst:
> > > >
> > > >A device
On 4/29/21 9:40 AM, marcandre.lur...@redhat.com wrote:
From: Marc-André Lureau
Signed-off-by: Marc-André Lureau
I see why the docs are here at the end now -- you change the AST first,
and then the meaning of the conditional string second. You didn't wanna
write two versions.
I'm fine
On 4/29/21 9:40 AM, marcandre.lur...@redhat.com wrote:
From: Marc-André Lureau
Change the 'if' condition strings to be C-agnostic and be simple
identifiers.
Oh, the answer to my last question. We strip off the defined(...) stuff
here and now.
Seems fine to me, as I like removing
Inside of gdb, I'm getting 0xd6ad000 for memory allocations made with mmap()
I run qemu with: qemu-system-x86_64 -enable-kvm -m 512M -s -S -drive
format=raw,file=ker.img -kernel /boot/vmlinuz-5.8.0-50-generic -append
"root=/dev/sda init=/sbin/x64-ker nokaslr"
The mmap() call succeeds, yet the
On 4/29/21 9:40 AM, marcandre.lur...@redhat.com wrote:
From: Marc-André Lureau
Signed-off-by: Marc-André Lureau
---
qapi/machine-target.json | 20
qapi/misc-target.json| 13 -
2 files changed, 28 insertions(+), 5 deletions(-)
diff --git
On 4/29/21 9:40 AM, marcandre.lur...@redhat.com wrote:
From: Marc-André Lureau
Modify check_if() to build an IfPredicate tree (the schema
documentation is updated in a following patch).
I'm wondering if check_if() is the right place to do this. It's
certainly convenient, but we don't build
On Wed, May 12, 2021 at 4:16 PM Alistair Francis wrote:
>
> On Tue, May 11, 2021 at 8:07 PM Ruinland Chuan-Tzu Tsai
> wrote:
> >
> > Introduce ax25 and custom CSR handling mechanism to RISC-V platform.
> > This is just a POC in which we add Andes custom CSR table directly
> > into the generic
On Thu, May 13, 2021 at 12:47 AM Bin Meng wrote:
>
> On Wed, May 12, 2021 at 2:03 PM Alistair Francis wrote:
> >
> > On Tue, May 11, 2021 at 8:07 PM Ruinland Chuan-Tzu Tsai
> > wrote:
> > >
> > > Hi all,
> > >
> > > My sincere apology that I missed the patch to include our own CSR table
> > >
On 4/29/21 9:40 AM, marcandre.lur...@redhat.com wrote:
From: Marc-André Lureau
Introduce IfNot predicate class, for 'not' condition expressions.
Signed-off-by: Marc-André Lureau
---
scripts/qapi/common.py | 22 ++
1 file changed, 22 insertions(+)
diff --git
Add "make venv" to create the pipenv-managed virtual environment that
contains our explicitly pinned dependencies.
Add "make check" to run the python linters [in the host execution
environment].
Add "make venv-check" which combines the above two: create/update the
venv, then run the linters in
On 4/29/21 9:40 AM, marcandre.lur...@redhat.com wrote:
From: Marc-André Lureau
Refactor IfAll class, to introduce a base class IfPredicateList and add
IfAny for the 'any' conditions.
Signed-off-by: Marc-André Lureau
---
scripts/qapi/common.py | 32 +++-
1 file
Add python3.6 to the fedora container image: we need it to run the
linters against that explicit version to make sure we don't break our
minimum version promise.
Add pipenv so that we can fetch precise versions of pip packages we need
to guarantee test reproducability.
Signed-off-by: John Snow
Try using avocado to manage our various tests; even though right now
they're only invoking shell scripts and not really running any
python-native code.
Create tests/, and add shell scripts which call out to mypy, flake8,
pylint and isort to enforce the standards in this directory.
Add
0.730 appears to be about the oldest version that works with the
features we want, including nice human readable output (to make sure
iotest 297 passes), and type-parameterized Popen generics.
0.770, however, supports adding 'strict' to the config file, so require
at least 0.770.
Now that we are
Ignore *Python* build and package output (build, dist, qemu.egg-info);
these files are not created as part of a QEMU build.
Ignore miscellaneous cached python confetti (__pycache__, *.pyc,
.mypy_cache).
Ignore .idea (pycharm) .vscode, and .venv (pipenv et al).
Signed-off-by: John Snow
---
isort 5.0.0 through 5.0.4 has a bug that causes it to misinterpret
certain "from ..." clauses that are not related to imports.
isort < 5.1.1 has a bug where it does not handle comments near import
statements correctly.
Require 5.1.2 or greater.
isort can be run with 'isort -c qemu' from the
This is intended to be a manually run, non-CI script.
Use tox to test the linters against all python versions from 3.6 to
3.9. This will only work if you actually have those versions installed
locally, but Fedora makes this easy:
> sudo dnf install python36 python37 python38 python39
Unlike the
Signed-off-by: John Snow
Reviewed-by: Cleber Rosa
---
python/.isort.cfg | 7 ---
python/setup.cfg | 8
2 files changed, 8 insertions(+), 7 deletions(-)
delete mode 100644 python/.isort.cfg
diff --git a/python/.isort.cfg b/python/.isort.cfg
deleted file mode 100644
index
mypy supports reading its configuration values from a central project
configuration file; do so.
Signed-off-by: John Snow
---
python/mypy.ini | 4
python/setup.cfg | 5 +
2 files changed, 5 insertions(+), 4 deletions(-)
delete mode 100644 python/mypy.ini
diff --git a/python/mypy.ini
setuptools doesn't have a formal understanding of development requires,
but it has an optional feataures section. Fine; add a "devel" feature
and add the requirements to it.
To avoid duplication, we can modify pipenv to install qemu[devel]
instead. This enables us to run invocations like "pip
This adds the python qemu packages themselves to the pipenv manifest.
'pipenv sync' will create a virtual environment sufficient to use the SDK.
'pipenv sync --dev' will create a virtual environment sufficient to use
and test the SDK (with pylint, mypy, isort, flake8, etc.)
The qemu packages are
flake8 3.5.x does not support the --extend-ignore syntax used in the
.flake8 file to gracefully extend default ignores, so 3.6.x is our
minimum requirement. There is no known upper bound.
flake8 can be run from the python/ directory with no arguments.
Signed-off-by: John Snow
Reviewed-by:
Instruct flake8 to avoid certain well-known directories created by
python tooling that it ought not check.
Note that at-present, nothing actually creates a ".venv" directory; but
it is in such widespread usage as a de-facto location for a developer's
virtual environment that it should be excluded
Update the comment concerning the flake8 exception to match commit
42c0dd12, whose commit message stated:
A note on the flake8 exception: flake8 will warn on *any* bare except,
but pylint's is context-aware and will suppress the warning if you
re-raise the exception.
Signed-off-by: John Snow
When creating a source distribution via 'python3 setup.py sdist', the
VERSION and PACKAGE.rst files aren't bundled by default. Create a
MANIFEST.in file that instructs the build tools to include these so that
installation from source dists won't fail.
(This invocation is required by 'tox', as
Add short readmes to python/, python/qemu/, python/qemu/machine,
python/qemu/qmp, and python/qemu/utils that explain the directory
hierarchy. These readmes are visible when browsing the source on
e.g. gitlab/github and are designed to help new developers/users quickly
make sense of the source
Pylint 2.5.x - 2.7.x have regressions that make import checking
inconsistent, see:
https://github.com/PyCQA/pylint/issues/3609
https://github.com/PyCQA/pylint/issues/3624
https://github.com/PyCQA/pylint/issues/3651
Pinning to 2.4.4 is worse, because it mandates versions of shared
dependencies
We are specifying >= pylint 2.7.x for several reasons:
1. For setup.cfg support, added in pylint 2.5.x
2. To specify a version that has incompatibly dropped
bad-whitespace checks (2.6.x)
3. 2.7.x fixes "unsubscriptable" warnings in Python 3.9
Signed-off-by: John Snow
---
python/Pipfile
Python infrastructure as it exists today is not capable reliably of
single-sourcing a package version from a parent directory. The authors
of pip are working to correct this, but as of today this is not possible.
The problem is that when using pip to build and install a python
package, it copies
pipenv is a tool used for managing virtual environments with pinned,
explicit dependencies. It is used for precisely recreating python
virtual environments.
pipenv uses two files to do this:
(1) Pipfile, which is similar in purpose and scope to what setup.cfg
lists. It specifies the requisite
Delete the empty settings now that it's sharing a home with settings for
other tools.
pylint can now be run from this folder as "pylint qemu".
Signed-off-by: John Snow
Reviewed-by: Cleber Rosa
Tested-by: Cleber Rosa
---
python/qemu/machine/pylintrc | 58
Create a space for miscellaneous things that don't belong strictly in
"qemu.machine" nor "qemu.qmp" packages.
Signed-off-by: John Snow
---
python/qemu/machine/__init__.py | 8
python/qemu/utils/__init__.py | 23 +++
python/qemu/{machine =>
Add setup.cfg and setup.py, necessary for installing a package via
pip. Add a ReST document (PACKAGE.rst) explaining the basics of what
this package is for and who to contact for more information. This
document will be used as the landing page for the package on PyPI.
I am not yet using a
move python/qemu/*.py to python/qemu/[machine, qmp]/*.py and update import
directives across the tree.
This is done to create a PEP420 namespace package, in which we may
create subpackages. To do this, the namespace directory ("qemu") should
not have any modules in it. Those files will go into
Based-on: <20210512214642.2803189-1-js...@redhat.com>
CI: https://gitlab.com/jsnow/qemu/-/pipelines/302010131
GitLab: https://gitlab.com/jsnow/qemu/-/tree/python-package-mk3
MR: https://gitlab.com/jsnow/qemu/-/merge_requests/4
I invite you to leave review comments on my mock merge request on
mypy is kind of weird about how it handles imports. For legacy reasons,
it won't load PEP 420 namespaces, because of logic implemented prior to
that becoming a standard.
So, if you plan on using any, you have to pass
--namespace-packages. Alright, fine.
Signed-off-by: John Snow
Reviewed-by:
Move RAM_SAVE_FLAG_xxx defines from migration/ram.c to migration/ram.h
Signed-off-by: Andrey Gruzdev
---
migration/ram.c | 16
migration/ram.h | 16
2 files changed, 16 insertions(+), 16 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index
Includes code to parse incoming migration stream, dispatch data to
section handlers and deal with complications of open-coded migration
format without introducing strong dependencies on QEMU migration code.
Signed-off-by: Andrey Gruzdev
---
include/qemu-snapshot.h | 34 +-
qemu-snapshot-vm.c
The commit enables asynchronous snapshot loading using standard postcopy
migration mechanism on destination VM.
The point of switchover to postcopy is trivially selected based on
percentage of non-zero pages loaded in precopy.
Signed-off-by: Andrey Gruzdev
---
include/qemu-snapshot.h | 12 +
Execution environment, command-line argument parsing, usage/version info etc.
Signed-off-by: Andrey Gruzdev
---
include/qemu-snapshot.h | 59 ++
meson.build | 2 +
qemu-snapshot-vm.c | 57 ++
qemu-snapshot.c | 439
4
This part implements snapshot loading in precopy mode.
Signed-off-by: Andrey Gruzdev
---
include/qemu-snapshot.h | 24 +-
qemu-snapshot-vm.c | 588 +++-
qemu-snapshot.c | 47 +++-
3 files changed, 654 insertions(+), 5 deletions(-)
diff --git
This commit enables asynchronous block layer I/O for qemu-snapshot tool.
Implementation provides in-order request completion delivery to simplify
migration code.
Several file utility routines are introduced as well.
Signed-off-by: Andrey Gruzdev
---
include/qemu-snapshot.h | 30 +
Changes v1 -> v2:
* Fixed CI checks
Changes v0 -> v1:
* Changed command-line format, now use blockdev specification to
define vmstate image.
* Don't deal with image creation in the tool, create externally.
* Better block layer AIO handling in the load path.
* Reduced fragmentation of the
In qemu-snapshot it is needed to retrieve current QEMUFile offset as a
number of bytes read by qemu_get_byte()/qemu_get_buffer().
The existing qemu_ftell() routine would give read position as a number
of bytes fetched from underlying IOChannel which is not the same.
Signed-off-by: Andrey Gruzdev
More on the lines of quieting pylint 2.8.x, though to make it obvious
that we definitely handle the cleanup here, I elected to bolster the
close() method here.
1. Check for the process having terminated early more rigorously by
checking poll() directly.
2. Change the prompt read into an
More pylint 2.8.x warning hushing: use open's context manager where it's
applicable to do so to avoid a warning.
Signed-off-by: John Snow
---
tests/qemu-iotests/iotests.py | 13 ++---
1 file changed, 6 insertions(+), 7 deletions(-)
diff --git a/tests/qemu-iotests/iotests.py
In a few cases, we can't use 'with ...' because they belong to
long-running classes that need those items to stay open at the end of
the block. We're handling it, so tell pylint to shush.
Signed-off-by: John Snow
---
tests/qemu-iotests/iotests.py| 2 +-
tests/qemu-iotests/testrunner.py | 1
One less file resource to manage, and it helps quiet some pylint >=
2.8.0 warnings about not using a with-context manager for the open call.
Signed-off-by: John Snow
---
python/qemu/machine.py | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/python/qemu/machine.py
use run() instead of Popen() -- to assert to pylint that we are not
forgetting to close a long-running program.
Signed-off-by: John Snow
---
python/qemu/machine.py | 15 +--
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/python/qemu/machine.py b/python/qemu/machine.py
Shift the open() call later so that the pylint pragma applies *only* to
that one open() call. Add a note that suggests why this is safe: the
resource is unconditionally cleaned up in _post_shutdown().
_post_shutdown is called after failed launches (see launch()), and
unconditionally after every
pylint 2.8.x adds warnings whenever we use Popen calls without using
'with', so it's desirable to convert synchronous calls to run()
invocations where applicable.
(Though, this trades one pylint warning for another due to a pylint bug,
which I've silenced with a pragma and a link to the bug.)
We handle this resource rather meticulously in
shutdown/kill/wait/__exit__ et al, through the laborious mechanisms in
_do_shutdown().
Quiet this pylint warning here.
Signed-off-by: John Snow
---
python/qemu/machine.py | 3 +++
1 file changed, 3 insertions(+)
diff --git
We manage cleaning up this resource ourselves. Pylint should shush.
Signed-off-by: John Snow
---
python/qemu/console_socket.py | 1 +
1 file changed, 1 insertion(+)
diff --git a/python/qemu/console_socket.py b/python/qemu/console_socket.py
index 87237bebef7..8c4ff598ad7 100644
---
Fixes pylint warnings.
Signed-off-by: John Snow
Reviewed-by: Cleber Rosa
---
python/qemu/console_socket.py | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/python/qemu/console_socket.py b/python/qemu/console_socket.py
index ac21130e446..87237bebef7 100644
---
gitlab CI: https://gitlab.com/jsnow/qemu/-/pipelines/301924893
branch: https://gitlab.com/jsnow/qemu/-/commits/python-package-pre-cleanup
This series serves as a pre-requisite for packaging the python series
and getting the linters running via CI. The first patch fixes a linter
error we've had
On 4/29/21 9:40 AM, marcandre.lur...@redhat.com wrote:
From: Marc-André Lureau
The following patches are going to express schema 'if' conditions in a
target language agnostic way. For that, let's start building a predicate
tree of the configuration options.
This intermediary steps still uses
On 4/29/21 9:40 AM, marcandre.lur...@redhat.com wrote:
From: Marc-André Lureau
Move the generating function to the QAPISchemaIfCond class.
Signed-off-by: Marc-André Lureau
---
scripts/qapi/common.py | 20 +---
scripts/qapi/gen.py| 6 ++
On 4/29/21 9:40 AM, marcandre.lur...@redhat.com wrote:
From: Marc-André Lureau
Wrap the 'if' condition in a higher-level object. Not only this allows
more type safety but also further refactoring without too much churn.
Would have done it myself if I had gotten to it first. I like having a
** Changed in: qemu
Status: Incomplete => New
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1913969
Title:
unable to migrate non shared storage when TLS is used
Status in QEMU:
New
Bug
On Tue, May 11, 2021 at 11:37 PM Philippe Mathieu-Daudé
wrote:
> Add system/user emulation jobs on FreeBSD host.
>
> To build these jobs, you need to add a FreeBSD runner and
> add 'freebsd' to the QEMU_CUSTOM_RUNNER variable in your
> GitLab project.
>
> Signed-off-by: Philippe Mathieu-Daudé
>
On 5/12/21 7:07 PM, Thomas Huth wrote:
> On 12/05/2021 18.37, Philippe Mathieu-Daudé wrote:
>> On 5/12/21 6:24 PM, Markus Armbruster wrote:
>>> From: Thomas Huth
>>>
>>> There are no known users of this CPU anymore, and there are no
>>> binaries available online which could be used for regression
On 5/12/21 7:15 PM, Thomas Huth wrote:
> Clang versions before v11.0 insist on having the %rX or %cX register
> names instead of just a number. Since our Travis-CI is currently
> still using Clang v6.0, we have to fix this to avoid failing jobs.
>
> Signed-off-by: Thomas Huth
> ---
>
Patchew URL:
https://patchew.org/QEMU/20210512192619.537268-1-andrey.gruz...@virtuozzo.com/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 20210512192619.537268-1-andrey.gruz...@virtuozzo.com
Subject: [RFC PATCH v1
Hi
On Wed, May 12, 2021 at 11:51 PM Doug Evans wrote:
>
> Btw, can you elaborate on "should rather be a diff of the commits that are
> new"?
> Up until now I've been told to provide "git shortlog old..new" output.
> The patch itself is just a one-liner to update the subproject sha1.
>
git
On Tue, 2021-05-11 at 17:11 +0200, Cornelia Huck wrote:
> hsch and csch basically have two parts: execute the command,
> and perform the halt/clear function. For fully emulated
> subchannels, it is pretty clear how it will work: check the
> subchannel state, and actually 'perform the halt/clear
On 5/12/21 1:32 PM, Alex Bennée wrote:
But it's needed right? It's perfectly fine to pull someone elses patch
into your series if it's needed for the review. The alternative is to
add a "Based-on" tag to your cover letter so it's easy to find the patch
or series we need to apply before yours.
On 12/05/2021 15:54, matheus.fe...@eldorado.org.br wrote:
From: Richard Henderson
The special logging is unnecessary. It will have been done
immediately before in the log file.
Signed-off-by: Richard Henderson
Signed-off-by: Matheus Ferst
---
target/ppc/translate.c | 6 +-
1 file
Includes code to parse incoming migration stream, dispatch data to
section handlers and deal with complications of open-coded migration
format without introducing strong dependencies on QEMU migration code.
Signed-off-by: Andrey Gruzdev
---
include/qemu-snapshot.h | 34 +-
qemu-snapshot-vm.c
On 12/05/2021 15:54, matheus.fe...@eldorado.org.br wrote:
From: Richard Henderson
Remove the synthetic "exception" after no more uses.
Signed-off-by: Richard Henderson
Signed-off-by: Matheus Ferst
---
target/ppc/cpu.h | 1 -
target/ppc/translate.c | 27 +--
This part implements snapshot loading in precopy mode.
Signed-off-by: Andrey Gruzdev
---
include/qemu-snapshot.h | 24 +-
qemu-snapshot-vm.c | 588 +++-
qemu-snapshot.c | 47 +++-
3 files changed, 654 insertions(+), 5 deletions(-)
diff --git
Execution environment, command-line argument parsing, usage/version info etc.
Signed-off-by: Andrey Gruzdev
---
include/qemu-snapshot.h | 59 ++
meson.build | 2 +
qemu-snapshot-vm.c | 57 ++
qemu-snapshot.c | 439
4
The commit enables asynchronous snapshot loading using standard postcopy
migration mechanism on destination VM.
The point of switchover to postcopy is trivially selected based on
percentage of non-zero pages loaded in precopy.
Signed-off-by: Andrey Gruzdev
---
include/qemu-snapshot.h | 12 +
Changes v0 -> v1:
* Changed command-line format, now use blockdev specification to
define vmstate image.
* Don't deal with image creation in the tool, create externally.
* Better block layer AIO handling in the load path.
* Reduced fragmentation of the image backing file by using
In qemu-snapshot it is needed to retrieve current QEMUFile offset as a
number of bytes read by qemu_get_byte()/qemu_get_buffer().
The existing qemu_ftell() routine would give read position as a number
of bytes fetched from underlying IOChannel which is not the same.
Signed-off-by: Andrey Gruzdev
1 - 100 of 438 matches
Mail list logo
