Richard Henderson writes:
> Many hosts pass and return 128-bit quantities like sequential
> 64-bit quantities. Treat this just like we currently break
> down 64-bit quantities for a 32-bit host.
>
> Signed-off-by: Richard Henderson
Reviewed-by: Alex Bennée
--
Alex Bennée
Virtualisation Te
From: Marc-André Lureau
Instead of listening for incoming connections with a SocketAddr, add a
new method open_with_socket() that accepts an existing socket.
Signed-off-by: Marc-André Lureau
Reviewed-by: Daniel P. Berrangé
---
python/qemu/qmp/protocol.py | 25 -
1 file
From: Marc-André Lureau
When no monitor address is given, establish the QMP communication through
a socketpair() (API is also supported on Windows since Python 3.5)
Signed-off-by: Marc-André Lureau
Reviewed-by: Daniel P. Berrangé
---
python/qemu/machine/machine.py | 24 ---
From: Marc-André Lureau
Hi,
As reported earlier by Richard Henderson ("virgl avocado hang" thread), avocado
tests may hang when QEMU exits before the QMP connection is established.
v3:
- after merge in https://gitlab.com/qemu-project/python-qemu-qmp
- resend as requested by John Snow
v2:
-
From: Marc-André Lureau
Teach QEMUMonitorProtocol to accept an exisiting socket.
Signed-off-by: Marc-André Lureau
Reviewed-by: Daniel P. Berrangé
---
python/qemu/qmp/legacy.py | 18 +++---
1 file changed, 15 insertions(+), 3 deletions(-)
diff --git a/python/qemu/qmp/legacy.py b/p
Hi Klaus
On Tue, Jan 10, 2023 at 7:49 PM Ripke, Klaus wrote:
>
> Signed-off-by: Klaus Ripke
>
> hw/usb/dev-smartcard-reader.c:
> Set static values from env vars QEMU_CCID_VENDOR/PRODUCT_ID and
> _DESCRIPTOR
We don't use environment variables to configure devices. Please add
properties to ccid_p
Automatic change running:
$ sed -i -E \
's/([^0][^x])%016" HWADDR_PRIx/\10x%016" HWADDR_PRIx/g' \
$(git grep -l -F '%016" HWADDR_PRIx')
Signed-off-by: Philippe Mathieu-Daudé
---
RFC: Maybe some non-0x prefix are justified in some cases,
i.e. 'info mtree' output, or generically
The 'hwaddr' type is defined in "exec/hwaddr.h" as:
hwaddr is the type of a physical address
(its size can be different from 'target_ulong').
All definitions use the 'HWADDR_' prefix, except TARGET_FMT_plx:
$ fgrep define include/exec/hwaddr.h
#define HWADDR_H
#define HWADDR_BITS 64
#
We are going to modify this code, fix its style first to avoid
the following checkpatch.pl violations:
ERROR: code indent should never use tabs
ERROR: space prohibited between function name and open parenthesis '('
Signed-off-by: Philippe Mathieu-Daudé
---
hw/dma/etraxfs_dma.c | 196 +++
Since v1:
- Fix checkpatch style violations
- Use HWADDR_PRIx instead of HWADDR_FMT_plx (Zoltan)
Supersedes: <20230110212947.34557-1-phi...@linaro.org>
"bulk: Rename TARGET_FMT_plx -> HWADDR_FMT_plx"
Philippe Mathieu-Daudé (4):
hw: Remove hardcoded tabs (code style)
bulk: Coding s
Fix the following checkpatch.pl violation on lines using the
TARGET_FMT_plx definition to avoid:
WARNING: line over 80 characters
Signed-off-by: Philippe Mathieu-Daudé
---
hw/block/pflash_cfi01.c | 5 +++--
hw/char/digic-uart.c | 8
hw/core/sysbus.c | 3
Hi Qiang,
On 11/1/23 08:05, Qiang Liu wrote:
This patch replaces hw_error to guest error log for [read|write]b
accesses when mode_16bit is enabled. This avoids aborting qemu.
Fixes: 1248f8d4cbc3 ("hw/lan9118: Add basic 16-bit mode support.")
Resolves: https://gitlab.com/qemu-project/qemu/-/issu
On Wed, Jan 11, 2023 at 4:44 PM Philippe Mathieu-Daudé
wrote:
> Hi Qiang,
>
> On 11/1/23 08:05, Qiang Liu wrote:
> > This patch replaces hw_error to guest error log for [read|write]b
> > accesses when mode_16bit is enabled. This avoids aborting qemu.
> >
> > Fixes: 1248f8d4cbc3 ("hw/lan9118: Add
Series fully reviewed.
Since v2:
- Use array of const pointers to const (Richard)
Since v1:
- Fixed overwritten RECEIVING_BIT7 entry (Richard)
- Picked R-b tags
- Remove unused dummy MemoryRegion
- Convert DPRINTF() to using trace events
(series used as base for follow-up, better if merged via
Define TYPE_GPIO_I2C in the public "hw/i2c/bitbang_i2c.h"
header and use it in hw/arm/musicpal.c.
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Richard Henderson
---
hw/arm/musicpal.c| 3 ++-
hw/i2c/bitbang_i2c.c | 1 -
include/hw/i2c/bitbang_i2c.h | 2 ++
3 files chang
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Richard Henderson
---
hw/i2c/bitbang_i2c.c | 23 +++
1 file changed, 15 insertions(+), 8 deletions(-)
diff --git a/hw/i2c/bitbang_i2c.c b/hw/i2c/bitbang_i2c.c
index e41cb63daa..bf4b781393 100644
--- a/hw/i2c/bitbang_i2c.c
++
Convert the remaining DPRINTF debug macro uses to tracepoints.
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Richard Henderson
---
hw/i2c/bitbang_i2c.c | 18 ++
hw/i2c/trace-events | 4
2 files changed, 10 insertions(+), 12 deletions(-)
diff --git a/hw/i2c/bitbang_i
Trace bitbang state machine changes with trace events.
Reviewed-by: Richard Henderson
Signed-off-by: Philippe Mathieu-Daudé
---
hw/i2c/bitbang_i2c.c | 33 -
hw/i2c/trace-events | 3 +++
2 files changed, 31 insertions(+), 5 deletions(-)
diff --git a/hw/i2c/bitb
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Richard Henderson
---
hw/i2c/bitbang_i2c.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/hw/i2c/bitbang_i2c.c b/hw/i2c/bitbang_i2c.c
index ac84bf0262..e41cb63daa 100644
--- a/hw/i2c/bitbang_i2c.c
+++ b/hw/i2c/bitbang
On 05/01/2023 15.53, Pierre Morel wrote:
The maximum nested topology entries is used by the guest to know
how many nested topology are available on the machine.
Currently, SCLP READ SCP INFO reports MNEST = 0, which is the
equivalent of reporting the default value of 2.
Let's use the default SCL
On 05/01/2023 15.53, Pierre Morel wrote:
During a subsystem reset the Topology-Change-Report is cleared
by the machine.
Let's ask KVM to clear the Modified Topology Change Report (MTCR)
bit of the SCA in the case of a subsystem reset.
Signed-off-by: Pierre Morel
---
...
diff --git a/target/s3
On 10/1/23 17:52, Peter Maydell wrote:
On Tue, 10 Jan 2023 at 16:45, Philippe Mathieu-Daudé wrote:
Replace the ARMCPU field in DigicState by a reference to
an allocated ARMCPU. Instead of initializing the field
with object_initialize(), allocate it with object_new().
As we don't access ARMCPU
On Tue, Jan 10, 2023 at 05:02:58PM -0500, Stefan Berger wrote:
>
>
> On 1/10/23 14:47, Stefan Berger wrote:
> >
> >
> > On 1/10/23 14:27, Daniel P. Berrangé wrote:
> > > On Tue, Jan 10, 2023 at 01:50:26PM -0500, Stefan Berger wrote:
> > > >
> > > >
> > > > On 1/6/23 10:16, Stefan Berger wrote
On Tue, Jan 10, 2023 at 06:18:29PM -0500, John Snow wrote:
> On Tue, Jan 10, 2023 at 3:34 AM Peter Delevoryas wrote:
> >
> > On macOS, private $TMPDIR's are the default. These $TMPDIR's are
> > generated from a user's unix UID and UUID [1], which can create a
> > relatively long path:
> >
> >
On 1/10/23 11:53, Peter Maydell wrote:
On Tue, 10 Jan 2023 at 09:33, Paolo Bonzini wrote:
A handful of header files in QEMU are wrapped with extern "C" blocks.
These are not necessary: there are C++ source files anymore in QEMU,
and even where there were some, they did not include most of thes
On 10/01/2023 15.29, Thomas Huth wrote:
On 05/01/2023 15.53, Pierre Morel wrote:
On interception of STSI(15.1.x) the System Information Block
(SYSIB) is built from the list of pre-ordered topology entries.
Signed-off-by: Pierre Morel
---
...
+void insert_stsi_15_1_x(S390CPU *cpu, int sel2, _
On 1/10/23 18:19, Eric Auger wrote:
Hi Paolo,
On 1/10/23 17:02, Paolo Bonzini wrote:
From: Eric Auger
To avoid compilation errors when -Werror=maybe-uninitialized is used,
replace 'case 3' by 'default'.
Otherwise we get:
../target/i386/ops_sse.h: In function ‘helper_vpermdq_ymm’:
../tar
On 1/10/23 17:40, Daniel P. Berrangé wrote:
Signed-off-by: Daniel P. Berrang??
This has mangled my name - think your client sending mail
needs to specify utf8 charset in the mail headers perhaps ?
I have a "Content-type: text/plain" header added from the mimecast
issues of last year, and git
On Tue, 2023-01-10 at 14:00 +0100, Thomas Huth wrote:
> On 05/01/2023 15.53, Pierre Morel wrote:
> > The topology information are attributes of the CPU and are
> > specified during the CPU device creation.
> >
> > On hot plug, we gather the topology information on the core,
> > creates a list of t
On 1/9/23 11:55, Michael S. Tsirkin wrote:
On Fri, Jan 06, 2023 at 03:21:43PM +0100, Laurent Vivier wrote:
Hi,
it seems this patch breaks vhost-user with DPDK.
See https://bugzilla.redhat.com/show_bug.cgi?id=2155173
it seems QEMU doesn't receive the expected commands sequence:
Received unexp
On 10/1/23 14:02, Kevin Wolf wrote:
Am 09.01.2023 um 23:54 hat Philippe Mathieu-Daudé geschrieben:
Hi,
There will always be a need to deprecate things. Here I'm
tackling the QOM (class) properties, since they can be set
from some CLI options (-object -device -global ...).
As an experiment, we
Hi
On Tue, Jan 10, 2023 at 9:49 PM Daniel P. Berrangé
wrote:
> It didn't appear that glibc was going to fix the problem
> breaking includes of linux/mount.h vs sys/mount.h, so
> QEMU applied a workaround copying the symbols/structs we
> need into a local header.
>
> Since then Linux modified lin
On Wed, Jan 11, 2023 at 10:55:47AM +0100, Philippe Mathieu-Daudé wrote:
> On 10/1/23 14:02, Kevin Wolf wrote:
> > Am 09.01.2023 um 23:54 hat Philippe Mathieu-Daudé geschrieben:
> > > Hi,
> > >
> > > There will always be a need to deprecate things. Here I'm
> > > tackling the QOM (class) properties
Richard Henderson writes:
> When allocating a temp to the stack frame, consider the
> base type and allocate all parts at once.
>
> Signed-off-by: Richard Henderson
> ---
> tcg/tcg.c | 30 ++
> 1 file changed, 22 insertions(+), 8 deletions(-)
>
> diff --git a/tcg/t
On 10/01/2023 18:19, Cédric Le Goater wrote:
External email: Use caution opening links or attachments
Hello Avihai
On 1/10/23 15:08, Avihai Horon wrote:
On 09/01/2023 20:36, Jason Gunthorpe wrote:
On Mon, Jan 09, 2023 at 06:27:21PM +0100, Cédric Le Goater wrote:
also, in vfio_migration_q
On 10/1/23 17:02, Paolo Bonzini wrote:
From: Eric Auger
To avoid compilation errors when -Werror=maybe-uninitialized is used,
replace 'case 3' by 'default'.
Otherwise we get:
../target/i386/ops_sse.h: In function ‘helper_vpermdq_ymm’:
../target/i386/ops_sse.h:2495:13: error: ‘r3’ may
On 05/01/2023 15.53, Pierre Morel wrote:
The KVM capability, KVM_CAP_S390_CPU_TOPOLOGY is used to
Remove the "," in above line?
activate the S390_FEAT_CONFIGURATION_TOPOLOGY feature and
the topology facility for the guest in the case the topology
I'd like to suggest to add "in the host CPU
On 11/1/23 10:59, Daniel P. Berrangé wrote:
On Wed, Jan 11, 2023 at 10:55:47AM +0100, Philippe Mathieu-Daudé wrote:
On 10/1/23 14:02, Kevin Wolf wrote:
Am 09.01.2023 um 23:54 hat Philippe Mathieu-Daudé geschrieben:
Hi,
There will always be a need to deprecate things. Here I'm
tackling the QOM
On 05/01/2023 15.53, Pierre Morel wrote:
The modification of the CPU attributes are done through a monitor
commands.
s/commands/command/
It allows to move the core inside the topology tree to optimise
the cache usage in the case the host's hypervizor previously
s/hypervizor/hypervisor/
mo
On 09.01.23 21:18, Richard Henderson wrote:
In db9aab5783a2 we broke the contract of s390_probe_access, in that it
no longer returned an exception code, nor set __excp_addr. Fix both.
Reported-by: David Hildenbrand
Signed-off-by: Richard Henderson
Should we add a Fixes tag?
Reviewed-by: Da
On 09.01.23 21:18, Richard Henderson wrote:
Passing a pointer from the caller down to access_prepare_nf
eliminates a structure copy.
Signed-off-by: Richard Henderson
---
Reviewed-by: David Hildenbrand
--
Thanks,
David / dhildenb
On 09.01.23 21:18, Richard Henderson wrote:
Assign to access struct immediately, rather than waiting
until the end of the function. This means we can pass
address of haddr struct members instead of allocating
extra space on the local stack.
Signed-off-by: Richard Henderson
Reviewed-by: David
On 09.01.23 21:18, Richard Henderson wrote:
The interface from probe_access_flags is void*, and matching
that will be helpful. We already rely on the gcc extension
for byte arithmetic on void*.
Signed-off-by: Richard Henderson
---
Reviewed-by: David Hildenbrand
--
Thanks,
David / dhildenb
On 11/1/23 10:20, Paolo Bonzini wrote:
On 1/10/23 17:40, Daniel P. Berrangé wrote:
Signed-off-by: Daniel P. Berrang??
This has mangled my name - think your client sending mail
needs to specify utf8 charset in the mail headers perhaps ?
I have a "Content-type: text/plain" header added from th
On 09.01.23 21:18, Richard Henderson wrote:
When this code was written, it was using tlb_vaddr_to_host,
which does not handle TLB_DIRTY. Since then, it has been
converted to probe_access_flags, which does.
Signed-off-by: Richard Henderson
---
It's been a long time ... I have to trust you on
On 09.01.23 21:18, Richard Henderson wrote:
Inline into the parent functions with a simple test
to select the page, and a new define to remove ifdefs.
Signed-off-by: Richard Henderson
---
Reviewed-by: David Hildenbrand
--
Thanks,
David / dhildenb
On 09.01.23 21:18, Richard Henderson wrote:
Ensure that the total length is in a local variable
across the byte loop. Compute size1 difference once.
Signed-off-by: Richard Henderson
---
target/s390x/tcg/mem_helper.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff -
On Wed, Jan 11, 2023 at 11:08:05AM +0100, Philippe Mathieu-Daudé wrote:
> On 11/1/23 10:59, Daniel P. Berrangé wrote:
> > On Wed, Jan 11, 2023 at 10:55:47AM +0100, Philippe Mathieu-Daudé wrote:
> > > On 10/1/23 14:02, Kevin Wolf wrote:
> > > > Am 09.01.2023 um 23:54 hat Philippe Mathieu-Daudé gesch
This patchset implements RISC-V Zc* extension v1.0.0.RC5.7 version instructions.
Specification:
https://github.com/riscv/riscv-code-size-reduction/tree/main/Zc-specification
The port is available here:
https://github.com/plctlab/plct-qemu/tree/plct-zce-upstream-v9
To test Zc* implementation, spe
Zcmp/Zcmt instructions will override disasm for c.fld*/c.fsd*
instructions currently
Signed-off-by: Weiwei Li
Signed-off-by: Junqiang Wang
Acked-by: Alistair Francis
---
disas/riscv.c | 228 +-
1 file changed, 227 insertions(+), 1 deletion(-)
di
Separate c_fld/c_fsd from fld/fsd to add additional check for
c.fld{sp}/c.fsd{sp} which is useful for zcmp/zcmt to reuse
their encodings
Signed-off-by: Weiwei Li
Signed-off-by: Junqiang Wang
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
---
target/riscv/insn16.decode
Add encode, trans* functions and helper functions support for Zcmt
instrutions
Add support for jvt csr
Signed-off-by: Weiwei Li
Signed-off-by: Junqiang Wang
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
---
target/riscv/cpu.h| 4 ++
target/riscv/cpu_bit
Expose zca,zcb,zcf,zcd,zcmp,zcmt properties
Signed-off-by: Weiwei Li
Signed-off-by: Junqiang Wang
Reviewed-by: Alistair Francis
---
target/riscv/cpu.c | 13 +
1 file changed, 13 insertions(+)
diff --git a/target/riscv/cpu.c b/target/riscv/cpu.c
index 39ab7e46d3..6df667805f 100644
Add properties for Zca,Zcb,Zcf,Zcd,Zcmp,Zcmt extension
Add check for these properties
Signed-off-by: Weiwei Li
Signed-off-by: Junqiang Wang
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
---
target/riscv/cpu.c | 43 +++
target/riscv/cpu.h
Add encode and trans* functions support for Zcb instructions
Signed-off-by: Weiwei Li
Signed-off-by: Junqiang Wang
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
---
target/riscv/insn16.decode| 24 ++
target/riscv/insn_trans/trans_rvzce.c.inc | 100 +
Modify the check for C extension to Zca (C implies Zca)
Signed-off-by: Weiwei Li
Signed-off-by: Junqiang Wang
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
Reviewed-by: Wilfred Mallawa
---
target/riscv/insn_trans/trans_rvi.c.inc | 4 ++--
target/riscv/translate.c
Separate c_flw/c_fsw from flw/fsw to add check for Zcf extension
Signed-off-by: Weiwei Li
Signed-off-by: Junqiang Wang
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
---
target/riscv/insn16.decode | 8
target/riscv/insn_trans/trans_rvf.c.inc | 18 +
Add encode, trans* functions for Zcmp instructions
Signed-off-by: Weiwei Li
Signed-off-by: Junqiang Wang
Reviewed-by: Richard Henderson
Reviewed-by: Alistair Francis
---
target/riscv/insn16.decode| 18 +++
target/riscv/insn_trans/trans_rvzce.c.inc | 189 +-
Daniel P. Berrangé writes:
> On Wed, Jan 11, 2023 at 11:08:05AM +0100, Philippe Mathieu-Daudé wrote:
>> On 11/1/23 10:59, Daniel P. Berrangé wrote:
>> > On Wed, Jan 11, 2023 at 10:55:47AM +0100, Philippe Mathieu-Daudé wrote:
>> > > On 10/1/23 14:02, Kevin Wolf wrote:
>> > > > Am 09.01.2023 um 23:
On 8/1/23 03:36, Richard Henderson wrote:
We expect the backend to require register pairs in
host-endian ordering, thus for big-endian the first
register of a pair contains the high part.
We were forcing R0 to contain the low part for calls.
Signed-off-by: Richard Henderson
---
tcg/tci.c | 21
On 8/1/23 03:36, Richard Henderson wrote:
Add code generation functions for data movement between
TCGv_i128 (mov) and to/from TCGv_i64 (concat, extract).
Signed-off-by: Richard Henderson
---
include/tcg/tcg-op.h | 4
tcg/tcg-internal.h | 13 +
tcg/tcg-op.c | 20 +
On Tue, Jan 10, 2023 at 10:55:35PM -0500, Emilio Cota wrote:
> The only reason to add this tree is to control the memory allocator
> used. Some users (e.g. TCG) cannot work reliably in multi-threaded
> environments (e.g. forking in user-mode) with GTree's allocator, GSlice.
> See https://gitlab.com
On 1/10/23 15:02, Peter Maydell wrote:
> On Tue, 10 Jan 2023 at 13:36, Fabiano Rosas wrote:
>>
>> Peter Maydell writes:
>>
>>> On Tue, 10 Jan 2023 at 13:00, Fabiano Rosas wrote:
Thomas Huth writes:
> On 09/01/2023 23.42, Fabiano Rosas wrote:
>> From: Claudio Fontana
On Tue, Jan 10, 2023 at 10:55:36PM -0500, Emilio Cota wrote:
> qemu-user can hang in a multi-threaded fork. One common
> reason is that when creating a TB, between fork and exec
> we manipulate a GTree whose memory allocator (GSlice) is
> not fork-safe.
>
> Although POSIX does not mandate it, the
On 11/01/2023 13.08, Claudio Fontana wrote:
On 1/10/23 15:02, Peter Maydell wrote:
On Tue, 10 Jan 2023 at 13:36, Fabiano Rosas wrote:
Peter Maydell writes:
On Tue, 10 Jan 2023 at 13:00, Fabiano Rosas wrote:
Thomas Huth writes:
On 09/01/2023 23.42, Fabiano Rosas wrote:
From: Claudio
On Tue, Jan 10, 2023 at 10:55:34PM -0500, Emilio Cota wrote:
> Context:
> https://gitlab.com/qemu-project/qemu/-/issues/285
>
> So far the only fix that we have had posted on the list is
> https://lists.gnu.org/archive/html/qemu-devel/2022-10/msg00391.html
> by Daniel. The approach that I'm fo
On Tue, Jan 10, 2023 at 10:55:36PM -0500, Emilio Cota wrote:
> qemu-user can hang in a multi-threaded fork. One common
> reason is that when creating a TB, between fork and exec
> we manipulate a GTree whose memory allocator (GSlice) is
> not fork-safe.
BTW, I just checked latest glib status
ht
Claudio Fontana writes:
> On 1/10/23 15:02, Peter Maydell wrote:
>> On Tue, 10 Jan 2023 at 13:36, Fabiano Rosas wrote:
>>>
>>> Peter Maydell writes:
>>>
On Tue, 10 Jan 2023 at 13:00, Fabiano Rosas wrote:
>
> Thomas Huth writes:
>
>> On 09/01/2023 23.42, Fabiano Rosas wrot
On 1/11/23 04:05, Daniel P. Berrangé wrote:
On Tue, Jan 10, 2023 at 05:02:58PM -0500, Stefan Berger wrote:
Yes, my ppc64 machine is also little endian. If the issue was not an
intermittent but a permanent
failure I would look for something like that. I would think it's more some sort
On 09/01/2023 11.58, Dr. David Alan Gilbert (git) wrote:
From: "Dr. David Alan Gilbert"
Fixup the migration compatibility for existing machine types
so that they do not enable msi-x.
Symptom:
(qemu) qemu: get_pci_config_device: Bad config data: i=0x34 read: 84 device: 98
cmask: ff wmask: 0 w
Emilio Cota writes:
> On Tue, Jan 10, 2023 at 20:58:01 +, Alex Bennée wrote:
>> Emilio Cota writes:
> (snip)
>> > +static inline void qht_do_if_first_in_stripe(const struct qht_map *map,
>> > + struct qht_bucket *b,
>> > +
On 11/1/23 12:29, Markus Armbruster wrote:
Daniel P. Berrangé writes:
On Wed, Jan 11, 2023 at 11:08:05AM +0100, Philippe Mathieu-Daudé wrote:
On 11/1/23 10:59, Daniel P. Berrangé wrote:
On Wed, Jan 11, 2023 at 10:55:47AM +0100, Philippe Mathieu-Daudé wrote:
On 10/1/23 14:02, Kevin Wolf wrot
Install abort handlers that close the TPM control and data channels in
case an abort occurs. The purpose of this is to have QEMU terminate
under abnormal test case failures to resolve intermittent hangs on s390x
hosts running TPM tests for QEMU/x86_64.
Signed-off-by: Stefan Berger
---
tests/qtes
On Wed, Jan 11, 2023 at 08:30:23AM -0500, Stefan Berger wrote:
> Install abort handlers that close the TPM control and data channels in
> case an abort occurs. The purpose of this is to have QEMU terminate
> under abnormal test case failures to resolve intermittent hangs on s390x
> hosts running TP
On 2022/12/23 17:01, Alexander Graf wrote:
Up to now, the finalize_gic_version() code open coded what is essentially
a support bitmap match between host/emulation environment and desired
target GIC type.
This open coding leads to undesirable side effects. For example, a VM with
KVM and -smp 10 w
Hi Alexander,
On 2022/12/23 17:01, Alexander Graf wrote:
Let's explicitly list out all accelerators that we support when trying to
determine the supported set of GIC versions. KVM was already separate, so
the only missing one is HVF which simply reuses all of TCG's emulation
code and thus has th
My sincere apologies for not replying sooner.
This needs a rebase now. But let me have a look at it first.
Install abort handlers that close the TPM control and data channels in
case an abort occurs. The purpose of this is to have QEMU terminate
under abnormal test case failures to resolve intermittent hangs on s390x
hosts running TPM tests for QEMU/x86_64.
Signed-off-by: Stefan Berger
---
tests/qtes
On 10.01.23 21:03, Peter Xu wrote:
On Tue, Jan 10, 2023 at 12:52:32PM +0100, David Hildenbrand wrote:
The following seems to work,
That looks much better at least from the diffstat pov (comparing to the
existing patch 1+5 and the framework changes), thanks.
but makes analyze-migration.py ang
On Wed, Jan 11, 2023 at 08:45:47AM -0500, Stefan Berger wrote:
> Install abort handlers that close the TPM control and data channels in
> case an abort occurs. The purpose of this is to have QEMU terminate
> under abnormal test case failures to resolve intermittent hangs on s390x
> hosts running TP
huang...@chinatelecom.cn writes:
> From: Hyman Huang(黄勇)
>
> Introduce "x-vcpu-dirty-limit-period" migration experimental
> parameter, which is in the range of 1 to 1000ms and used to
> make dirtyrate calculation period configurable.
>
> Currently with the "x-vcpu-dirty-limit-period" varies, the
> > > From an interface cleanliness point of view I'd rather see
> > > all the optional fields as optional. That's done by marking them
> > > with a * so
> > > '*channel': 'int16'
> > >
> > > Then the signature of the related qmp_cxl_inject_gen_media_event
> > > gains a boolean has_channel para
huang...@chinatelecom.cn writes:
> From: Hyman Huang(黄勇)
>
> Implement dirty-limit convergence algo for live migration,
> which is kind of like auto-converge algo but using dirty-limit
> instead of cpu throttle to make migration convergent.
>
> Enable dirty page limit if dirty_rate_high_cnt great
A small collection of misc fixes and tidying up pulled out from various
series. I've pulled this to the top of my queue of CXL related work
as they stand fine on their own and it will reduce the noise in
the larger patch sets if these go upstream first.
Gregory's patches were posted as part of his
msix_init_exclusive_bar() can fail, so if it does cleanup the address space.
Signed-off-by: Jonathan Cameron
---
hw/mem/cxl_type3.c | 12 ++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/hw/mem/cxl_type3.c b/hw/mem/cxl_type3.c
index dae4fd89ca..252822bd82 100644
--- a/
Fix capitalization difference between struct name and typedef.
Signed-off-by: Jonathan Cameron
---
hw/pci-bridge/cxl_downstream.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/pci-bridge/cxl_downstream.c b/hw/pci-bridge/cxl_downstream.c
index 3d4e6b59cd..54f507318f 10064
From: Gregory Price
Current code sets to STORAGE_EXPRESS and then overrides it.
Signed-off-by: Gregory Price
Reviewed-by: Davidlohr Bueso
Signed-off-by: Jonathan Cameron
---
hw/mem/cxl_type3.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/hw/mem/cxl_type3.c b/hw/mem/c
From: Gregory Price
Remove usage of magic numbers when accessing capacity fields and replace
with CXL_CAPACITY_MULTIPLIER, matching the kernel definition.
Signed-off-by: Gregory Price
Reviewed-by: Davidlohr Bueso
Signed-off-by: Jonathan Cameron
---
hw/cxl/cxl-mailbox-utils.c | 14 ---
Noticed as this prevents iASL disasembling the DSDT table.
Signed-off-by: Jonathan Cameron
---
hw/i386/acpi-build.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index 127c4e2d50..a584b62ae2 100644
--- a/hw/i386/acpi-build.c
+++ b/hw/i386/acpi-bui
* David Woodhouse (dw...@infradead.org) wrote:
> From: Joao Martins
>
> Specifically add listing, injection of event channels.
>
> Signed-off-by: Joao Martins
> Signed-off-by: David Woodhouse
> ---
> hmp-commands.hx | 29
> hw/i386/kvm/meson.build | 4 ++
> hw/i386/kvm/x
From: Ira Weiny
Gcc requires constant versions of cpu_to_le* calls.
Add a 64 bit version.
Reviewed-by: Peter Maydell
Signed-off-by: Ira Weiny
Signed-off-by: Jonathan Cameron
---
include/qemu/bswap.h | 10 ++
1 file changed, 10 insertions(+)
diff --git a/include/qemu/bswap.h b/inclu
On Fri, 2 Dec 2022 at 07:31, Alberto Faria wrote:
>
> On Mon, Sep 5, 2022 at 8:45 AM Paolo Bonzini wrote:
> > Thanks, I will queue it.
> >
> > Paolo
> >
> > Il lun 5 set 2022, 02:00 Alberto Faria ha scritto:
> >>
> >> flatview_(read|write)_continue() must be called with an address in the
> >> Fl
From: Ira Weiny
The cel_uuid was programatically generated previously because there was
no static initializer for network order UUIDs.
Use the new network order initializer for cel_uuid. Adjust
cxl_initialize_mailbox() because it can't fail now.
Update specification reference.
Signed-off-by:
From: Ira Weiny
UUID's are defined as network byte order fields. No static initializer
was available for UUID's in their standard big endian format.
Define a big endian initializer for UUIDs.
Signed-off-by: Ira Weiny
Signed-off-by: Jonathan Cameron
---
include/qemu/uuid.h | 12
Am 11.01.2023 um 08:55 hat Zhiyong Ye geschrieben:
> Hi Kevin,
>
> Thank you for your reply and detailed answers.
>
> In my scenario is the iSCSI SAN environment. The network storage device is
> connected to the physical machine via iSCSI, and LVM is used as the middle
> layer between the storage
huang...@chinatelecom.cn writes:
> From: Hyman Huang(黄勇)
>
> Export dirty limit throttle time and estimated ring full
> time, through which we can observe if dirty limit take
> effect during live migration.
Suggest something like "Extend query-migrate to provide ..." both here
and in subject.
>
This allows drivers to register a callback on a qemu_irq, which is
invoked when a level-triggered IRQ is acked on the irqchip.
This allows us to simulate level-triggered interrupts more efficiently,
by resampling the state of the interrupt only when it actually matters.
This can be used in two wa
On 11/1/23 15:24, Jonathan Cameron via wrote:
Fix capitalization difference between struct name and typedef.
Signed-off-by: Jonathan Cameron
---
hw/pci-bridge/cxl_downstream.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/pci-bridge/cxl_downstream.c b/hw/pci-bridge/c
On Wed, 2023-01-11 at 14:28 +, Dr. David Alan Gilbert wrote:
>
> Those \n's are hanging in for dear life :-)
Ah crap, that's in the *stubs* not the real code. Really really fixed
now!
> Other than that, I think fromt he HMP side we're OK, so
>
> Acked-by: Dr. David Alan Gilbert
>
> cc'in
On Wed, 2023-01-11 at 02:47 +0100, Ilya Leoshkevich wrote:
> Add ability to dump /tmp/perf-.map and jit-.dump.
> The first one allows the perf tool to map samples to each individual
> translation block. The second one adds the ability to resolve symbol
> names, line numbers and inspect JITed code.
1 - 100 of 204 matches
Mail list logo
