Hello Xiaoyao,
I did not find a better name at the time. The meaning of 'host' there has
nothing to do with the cpu type called "host",
but rather identifies common code between kvm and hvf, which both use the host
cpuid(), See accel_uses_host_cpuid(), host_cpuid(), host_cpu_vendor_fms().
Maybe
line, and that gets
me to:
2972153@1743002351.233130:vdagent_open
2972192@1743002362.649762:vdagent_close
2972192@1743002362.649771:vdagent_disconnect
Apparently the vdagent connection is closed after 11 seconds, but I have no
idea why..
Thanks for any suggestions,
Claudio
--
Claudio Fontana
Engineering Manager Virtualization, SUSE Labs Core
SUSE Software Solutions Italy Srl
ping, adding Richard.
We will need to include this downstream because of the breakage its lack causes.
It is already reviewed by me, but some TCG maintainer indicating it will be
included in some queue would help,
Thanks,
Claudio
On 9/18/24 17:11, Claudio Fontana wrote:
> Adding Ilya
Adding Ilya FYI.
Ciao,
Claudio
On 9/11/24 18:19, Claudio Fontana wrote:
> On 9/11/24 16:16, Fabiano Rosas wrote:
>> The XT check for the lxvx/stxvx instructions is currently
>> inverted. This was introduced during the move to decodetree.
>>
>> From the ISA:
&g
On 9/9/24 12:07, Claudio Fontana wrote:
> On 9/9/24 11:54, Daniel Henrique Barboza wrote:
>>
>>
>> On 9/9/24 12:17 AM, Andrew.Yuan wrote:
>>> The code 'ops = ACCEL_OPS_CLASS(module_object_class_by_name(ops_name));' is
>>> unnecessary;
>
tx, arg_X *a, bool store, bool paired)
> {
> - if (paired || a->rt >= 32) {
> +if (paired || a->rt < 32) {
> REQUIRE_VSX(ctx);
> } else {
> REQUIRE_VECTOR(ctx);
Reviewed-by: Claudio Fontana
On 9/9/24 11:54, Daniel Henrique Barboza wrote:
>
>
> On 9/9/24 12:17 AM, Andrew.Yuan wrote:
>> The code 'ops = ACCEL_OPS_CLASS(module_object_class_by_name(ops_name));' is
>> unnecessary;
>>
>> And, the following code :
>> 1.has the same functionality;
>> 2.includes error checking;
>>
>> Signed-
Reviewed-by: Claudio Fontana
On 8/12/24 10:57, Alexander Ivanov wrote:
> Move local_err initialization inside the loop in the module_load_qom_all()
> function. This change ensures that the error variable is reset to NULL for
> each iteration of the loop. This prevents triggering ass
(module_load("", modinfo->name, &local_err) < 0) {
> error_report_err(local_err);
> +local_err = NULL;
> }
> }
> module_loaded_qom_all = true;
Reviewed-by: Claudio Fontana
gltm, thanks!
Reviewed-by: Claudio Fontana
On 8/9/24 07:10, Ani Sinha wrote:
> Refactoring the core logic around KVM_CREATE_VM into its own separate function
> so that it can be called from other functions in subsequent patches. There is
> no functional change in this patch.
>
Hello,
as a suggestion you could adjust the names of the functions to match the
existing pattern in this module.
It is modulename_method ie kvm_* , so:
On 8/8/24 13:38, Ani Sinha wrote:
> Refactoring the core logic around KVM_CREATE_VM into its own separate function
> so that it can be called f
a
partial match ...
Add a check for this. While at it, fold a number of explicitly handled
special cases into the generic logic.
Cc: qemu-sta...@nongnu.org
Reported-by: Claudio Fontana
Closes: https://lists.gnu.org/archive/html/qemu-devel/2023-08/msg00633.htm
On 5/6/24 13:20, Thomas Huth wrote:
> The old "-runas" option has the disadvantage that it is not visible
> in the QAPI schema, so it is not available via the normal introspection
> mechanisms. We've recently introduced the "-run-with" option for exactly
> this purpose, which is meant to handle the
Thanks Philippe, I fear I am unable to really be uptodate,
but it's great to see all the cleanup you and others have done!
Ciao,
Claudio
On 3/19/24 16:48, Philippe Mathieu-Daudé wrote:
> (Forgot to Cc Claudio to keep him updated)
>
> On 19/3/24 16:42, Philippe Mathieu-Daudé wrote:
>> Hi,
>>
>>
Hello,
pretty much the $Subject.
We had reports of very bad performance with the default options when running:
$ time qemu-img convert -c -p -O qcow2 file.qcow2 file.disk
real7m13.220s
user6m56.682s
sys 0m13.038s
I studied this and noticed that for every coroutine a thread was creat
On 3/15/24 17:59, Nina Schoetterl-Glausch wrote:
> On Thu, 2024-03-14 at 22:37 +0100, Claudio Fontana wrote:
>> some users were confused by this message showing under TCG:
>>
>> Selected CPU generation is too new. Maximum supported model
>> in the configuration: &
est trying
different accel, QEMU and kernel versions.
Signed-off-by: Claudio Fontana
---
target/s390x/cpu_models.c | 22 +++---
1 file changed, 15 insertions(+), 7 deletions(-)
diff --git a/target/s390x/cpu_models.c b/target/s390x/cpu_models.c
index 1a1c096122..8ed3bb6a27 100644
--
On 3/14/24 21:10, Claudio Fontana wrote:
> On 3/14/24 20:44, Nina Schoetterl-Glausch wrote:
>> On Thu, 2024-03-14 at 20:00 +0100, Claudio Fontana wrote:
>>> some users were confused by this message showing under TCG:
>>>
>>> Selected CPU generation is too new
On 3/14/24 20:44, Nina Schoetterl-Glausch wrote:
> On Thu, 2024-03-14 at 20:00 +0100, Claudio Fontana wrote:
>> some users were confused by this message showing under TCG:
>>
>> Selected CPU generation is too new. Maximum supported model
>> in the configuration: 'xy
eration is too new. Maximum supported model
in the accelerator 'tcg' configuration: 'xyz'
Signed-off-by: Claudio Fontana
---
target/s390x/cpu_models.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/target/s390x/cpu_models.c b/target/s390x
On 3/14/24 17:44, Nina Schoetterl-Glausch wrote:
> On Thu, 2024-03-14 at 16:54 +0100, Thomas Huth wrote:
>> On 14/03/2024 16.49, Claudio Fontana wrote:
>>> Hello Pierre, Ilya,
>>>
>>> I have a question on the s390x "topology" feature and examples.
>
-by: Claudio Fontana
---
docs/system/introduction.rst | 2 ++
docs/system/s390x/cpu-topology.rst | 14 --
2 files changed, 10 insertions(+), 6 deletions(-)
diff --git a/docs/system/introduction.rst b/docs/system/introduction.rst
index 51ac132d6c..746707eb00 100644
--- a/docs
0,core-id=14 \
\
-device gen16b-s390x-cpu,core-id=4,dedicated=on,entitlement=high
"
We received questions about this, so I hope you can shed some light, maybe it
would be good to just update the web page to include -accel kvm or -enable-kvm
everywhere for clarity?
Thanks for your hel
On 2/23/24 03:59, Peter Xu wrote:
> On Tue, Feb 20, 2024 at 07:41:04PM -0300, Fabiano Rosas wrote:
>> Latest numbers
>> ==
>>
>> => guest: 128 GB RAM - 120 GB dirty - 1 vcpu in tight loop dirtying memory
>> => host: 128 CPU AMD EPYC 7543 - 2 NVMe disks in RAID0 (8586 MiB/s) - xfs
>> =>
wrote:
> On 05.02.24 11:14, Claudio Fontana wrote:
>> Hi,
>
> Hi Claudio,
>
>>
>> turning pages back in time,
>>
>> noticed that in recent qemu-img binaries we include an ELF dependency on
>> libnuma.so that seems unused.
>&
Hi,
turning pages back in time,
noticed that in recent qemu-img binaries we include an ELF dependency on
libnuma.so that seems unused.
I think it stems from this commit:
commit 10218ae6d006f76410804cc4dc690085b3d008b5
Author: David Hildenbrand
Date: Fri Oct 14 15:47:17 2022 +0200
util:
On 11/30/23 18:51, Daniel P. Berrangé wrote:
> On Thu, Nov 30, 2023 at 05:54:26PM +0100, David Hildenbrand wrote:
>> On 30.11.23 17:01, Sean Christopherson wrote:
>>> On Thu, Nov 30, 2023, David Hildenbrand wrote:
On 30.11.23 08:32, Xiaoyao Li wrote:
> On 11/20/2023 5:26 PM, David Hildenbr
On 11/30/23 14:31, Philippe Mathieu-Daudé wrote:
> Hi Claudio,
>
> On 30/11/23 13:48, Claudio Fontana wrote:
>> Hi Philippe,
>>
>> took a quick look with
>>
>> grep -R can_do_io
>>
>> and this seems to be in include/hw/core/cpu.h as well as cpu-
Hi Philippe,
took a quick look with
grep -R can_do_io
and this seems to be in include/hw/core/cpu.h as well as cpu-common.c,
maybe there is more meat to address to fully solve this?
Before we had stuff for reset in cpu-common.c under a
if (tcg_enabled()) {
}
but now we have cpu_exec_reset_ho
Reviewed-by: Claudio Fontana
On 11/10/23 18:08, Richard Henderson wrote:
> In cpu_exec_step_atomic, we did not set CF_LAST_IO, which can
> lead to a loop with cpu_io_recompile.
>
> But since 18a536f1f8 ("Always require can_do_io") we no longer need
> a flag to indicate
#x27;t handle a very high
> PCI space, then a workaround in the PCI code might make it more clear
> what is occurring.)
>
> Cheers,
> -Kevin
>
>
>>
>> Cc: Claudio Fontana
>> Signed-off-by: Gerd Hoffmann
Hi,
I thought about this, and I am not sure it&#
On 11/7/23 12:49, Gerd Hoffmann wrote:
>> Hi, thanks for the response,
>>
>> just to be sure, this is a regression (it worked fine prior to the seabios
>> update).
>
> Yes.
>
>> Address sizes: 43 bits physical, 48 bits virtual
>
> OK.
>
>> [0.881472] pci :00:01.0:
On 11/7/23 11:35, Gerd Hoffmann wrote:
> Hi,
>
>> Trying to install a simple centos7 iso for testing as in:
>>
>> IMAGESDIR=/images
>> ISO=${IMAGESDIR}/CentOS-7-x86_64-Minimal-2009.iso
>
> Oh, you are running a software museum ;)
>
>> ===
On 10/4/23 12:05, Philippe Mathieu-Daudé wrote:
> Hi,
>
> I'm getting a bunch of errors for 'optarg' declared in :
>
> NAME
> getopt – get option character from command line argument list
>
> LIBRARY
> Standard C Library (libc, -lc)
>
> SYNOPSIS
> #include
>
> extern c
On 10/4/23 14:37, Thomas Huth wrote:
> On 04/10/2023 14.33, Daniel P. Berrangé wrote:
>> On Wed, Oct 04, 2023 at 11:06:15AM +0200, Philippe Mathieu-Daudé wrote:
>>> This series finishes the cleanup which remove the confusion
>>> of using 'softmmu' when we really mean 'system emulation',
>>> as oppo
On 10/3/23 16:04, Richard Henderson wrote:
> On 10/2/23 23:44, Philippe Mathieu-Daudé wrote:
>> On 15/9/23 21:00, Philippe Mathieu-Daudé wrote:
>>> - Add missing accel_cpu_unrealize()
>>> - Add AccelClass::[un]realize_cpu handlers
>>> - Use tcg_exec_[un]realizefn as AccelClass handlers
>>>
>>> Phil
Nice. I build-tested and make checked this as well,
Reviewed-by: Claudio Fontana
Tested-by: Claudio Fontana
On 10/3/23 14:30, Philippe Mathieu-Daudé wrote:
> Since v1:
> - Use 'target'/'common' in function names (Claudio)
> - Added Claudio R-b tags
-Daudé
Reviewed-by: Claudio Fontana
> ---
> accel/tcg/internal.h | 3 +++
> include/exec/cpu-all.h | 2 --
> accel/tcg/tcg-all.c| 2 ++
> cpu.c | 8
> 4 files changed, 5 insertions(+), 10 deletions(-)
>
> diff --git a/accel/tcg/internal
thieu-Daudé
Reviewed-by: Claudio Fontana
> ---
> include/exec/cpu-all.h | 2 +-
> accel/tcg/cpu-exec.c | 4 +++-
> 2 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/include/exec/cpu-all.h b/include/exec/cpu-all.h
> index c2c62160c6..1e5c530ee1 100644
>
On 9/15/23 21:00, Philippe Mathieu-Daudé wrote:
> Currently accel_cpu_realize() only performs target-specific
> realization. Introduce the [un]realize_cpu fields in the
> base AccelClass to be able to perform target-agnostic
> [un]realization of vCPUs.
>
> Signed-off-by: Philippe Mathieu-Daudé
J
Reviewed-by: Claudio Fontana
On 9/15/23 21:00, Philippe Mathieu-Daudé wrote:
> We use the '*fn' suffix for handlers, this is a public method.
> Drop the suffix.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> include/qemu/accel.h | 4 ++--
> accel/acce
On 9/13/23 15:00, Claudio Fontana wrote:
> On 8/30/23 15:09, Claudio Fontana wrote:
>> On 8/22/23 15:25, Philippe Mathieu-Daudé wrote:
>>> Hi Claudio,
>>>
>>> On 22/8/23 14:00, Claudio Fontana wrote:
>>>> Hello,
>>>>
>>>> th
Hi,
this is the first step towards faster migration to disk that we care about the
most,
can it be merged?
Thanks,
Claudio
On 9/8/23 16:22, Steve Sistare wrote:
> Add the migration URI "file:filename[,offset=offset]".
>
> Fabiano Rosas has submitted the unit tests in the series
> migration:
On 8/30/23 15:09, Claudio Fontana wrote:
> On 8/22/23 15:25, Philippe Mathieu-Daudé wrote:
>> Hi Claudio,
>>
>> On 22/8/23 14:00, Claudio Fontana wrote:
>>> Hello,
>>>
>>> this series is all reviewed,
>>>
>>> and is needed a
On 9/4/23 13:07, Andrew Jones wrote:
> On Mon, Sep 04, 2023 at 10:18:05AM +0200, Claudio Fontana wrote:
>> Hi,
>>
>> I think this discussion from ~2015 could potentially be be historically
>> relevant for context,
>> at the time we had the problem with CNTV
Hi,
I think this discussion from ~2015 could potentially be be historically
relevant for context,
at the time we had the problem with CNTVOFF IIRC so KVM_REG_ARM_TIMER_CNT being
read and rewritten causing time warps in the guest:
https://patchwork.kernel.org/project/linux-arm-kernel/patch/14351
On 8/22/23 15:25, Philippe Mathieu-Daudé wrote:
> Hi Claudio,
>
> On 22/8/23 14:00, Claudio Fontana wrote:
>> Hello,
>>
>> this series is all reviewed,
>>
>> and is needed as a precondition for further work to improve dramatically the
>> performance o
Hello,
this series is all reviewed,
and is needed as a precondition for further work to improve dramatically the
performance of virsh save, virsh restore
when migrating to disk, can it be merged?
Thanks,
Claudio
On 8/16/23 20:28, Fabiano Rosas wrote:
> For v2 I fixed patch 3 which had a hunk
Apologies, already reviewed, will ping for the merge of the series momentarily,
Claudio
On 8/22/23 14:18, Claudio Fontana wrote:
> Hello,
>
> this patch would still need a review,
>
> and is needed as a precondition for further work to improve dramatically the
> perform
Hello,
this patch would still need a review,
and is needed as a precondition for further work to improve dramatically the
performance of virsh save, virsh restore
when migrating to disk, since Peter Xu already reviewed PATCH 1, 2, 3, maybe it
makes sense to look at 4 too?
Thanks,
Claudio
On
Hello,
this patch is reviewed,
and is needed as a precondition for further work to improve dramatically the
performance of virsh save, virsh restore
when migrating to disk, can it be merged?
Thanks,
Claudio
On 7/25/23 15:26, Fabiano Rosas wrote:
> From: Nikolay Borisov
>
> Extract the rambl
Hello,
this series is all reviewed,
and is needed as a precondition for further work to improve dramatically the
performance of virsh save, virsh restore
when migrating to disk, can it be merged?
Thanks,
Claudio
On 7/12/23 21:07, Fabiano Rosas wrote:
> Based-on:
> [PATCH V4 0/2] migration fil
Hello,
this series is all reviewed,
and is needed as a precondition for further work to improve dramatically the
performance of virsh save, virsh restore
when migrating to disk, can it be merged?
Thanks,
Claudio
On 6/30/23 16:25, Steve Sistare wrote:
> Add the migration URI "file:filename[,of
Hi all,
we currently have to maintain something downstream for this, since the current
behavior can compound problems on top of existing bad NFS latency,
could someone continue to help reviewing this work?
Thanks,
Claudio
On 6/9/23 22:19, Fabiano Rosas wrote:
> Hi,
>
> The major change from
On 8/5/23 01:03, Ilya Leoshkevich wrote:
> Add a small test to prevent regressions.
>
> Signed-off-by: Ilya Leoshkevich
Something seems off in the wiring of the make check target?
I built with:
./configure --target-list=s390x-linux-user,s390x-softmmu
make -j
make -j check-help
...
Individua
_VECTOR, HWCAP_S390_VXRS);
> GET_FEATURE(S390_FEAT_VECTOR_ENH, HWCAP_S390_VXRS_EXT);
> +GET_FEATURE(S390_FEAT_VECTOR_ENH2, HWCAP_S390_VXRS_EXT2);
>
> return hwcap;
> }
Reviewed-by: Claudio Fontana
On 8/8/23 19:08, Stefan Hajnoczi wrote:
> On Mon, Jul 31, 2023 at 05:33:38PM -0300, Fabiano Rosas wrote:
>> We can fail the blk_insert_bs() at init_blk_migration(), leaving the
>> BlkMigDevState without a dirty_bitmap and BlockDriverState. Account
>> for the possibly missing elements when doing cle
added Kevin and Hanna for block, since this seems still untouched?
Thanks,
Claudio
On 7/31/23 22:33, Fabiano Rosas wrote:
> We can fail the blk_insert_bs() at init_blk_migration(), leaving the
> BlkMigDevState without a dirty_bitmap and BlockDriverState. Account
> for the possibly missing elemen
On 8/7/23 10:08, Ilya Leoshkevich wrote:
> On Sun, 2023-08-06 at 13:05 +0200, Claudio Fontana wrote:
>> On 8/5/23 01:03, Ilya Leoshkevich wrote:
>>> Add a small test to prevent regressions.
>>>
>>> Signed-off-by: Ilya Leoshkevich
>>> ---
>>>
re can not be a
> partial match ...
>
> Add a check for this. While at it, fold a number of explicitly handled
> special cases into the generic logic.
>
> Cc: qemu-sta...@nongnu.org
> Reported-by: Claudio Fontana
> Closes: https://lists.gnu.org/archive/html/qemu-devel/2023
On 8/5/23 01:03, Ilya Leoshkevich wrote:
> Add a small test to prevent regressions.
>
> Signed-off-by: Ilya Leoshkevich
> ---
> tests/tcg/s390x/Makefile.target | 1 +
> tests/tcg/s390x/vxeh2_vstrs.c | 88 +
> 2 files changed, 89 insertions(+)
> create mode 100
On 8/4/23 11:20, Thomas Huth wrote:
> On 04/08/2023 11.00, Claudio Fontana wrote:
>> Hi,
>>
>> On 7/21/23 11:08, Claudio Fontana wrote:
>>>
>>> Hello Cornelia, Richard,
>>>
>>> I had some strange behavior in an s390x TCG VM that I am de
Hi,
On 7/21/23 11:08, Claudio Fontana wrote:
>
> Hello Cornelia, Richard,
>
> I had some strange behavior in an s390x TCG VM that I am debugging,
>
> and configured latest upstream QEMU with --enable-debug --enable-debug-tcg
>
> and I am running the qemu binary wi
On 7/31/23 18:21, Xiaoyao Li wrote:
> From: Chao Peng
>
> Switch to KVM_SET_USER_MEMORY_REGION2 when supported by KVM.
>
> With KVM_SET_USER_MEMORY_REGION2, QEMU can set up memory region that
> backen'ed both by hva-based shared memory and gmem fd based private
> memory.
>
> Signed-off-by: Chao
On 8/1/23 18:48, Claudio Fontana wrote:
> On 7/31/23 18:21, Xiaoyao Li wrote:
>> Signed-off-by: Xiaoyao Li
>> ---
>> include/exec/memory.h | 9 +
>> softmmu/memory.c | 5 +
>> 2 files changed, 14 insertions(+)
>>
>> diff --git a/
On 7/31/23 18:21, Xiaoyao Li wrote:
> Signed-off-by: Xiaoyao Li
> ---
> include/exec/memory.h | 9 +
> softmmu/memory.c | 5 +
> 2 files changed, 14 insertions(+)
>
> diff --git a/include/exec/memory.h b/include/exec/memory.h
> index 61e31c7b9874..e119d3ce1a1d 100644
> --- a/inc
On 7/31/23 13:31, Claudio Fontana wrote:
> On 7/21/23 11:08, Claudio Fontana wrote:
>>
>> Hello Cornelia, Richard,
>>
>> I had some strange behavior in an s390x TCG VM that I am debugging,
>>
>> and configured latest upstream QEMU with --enable-debug --enab
On 7/21/23 11:08, Claudio Fontana wrote:
>
> Hello Cornelia, Richard,
>
> I had some strange behavior in an s390x TCG VM that I am debugging,
>
> and configured latest upstream QEMU with --enable-debug --enable-debug-tcg
>
> and I am running the qemu binary with -d uni
e exception has been raised,
> we will have unwound the pages locked for translation and the
> problem does not arise. There are plenty of other updates to
> LowCore while delivering an interrupt/exception; trans_exc_code
> does not need to be special.
>
> Reported-by: Claudio
On 7/28/23 16:28, Richard Henderson wrote:
> On 7/28/23 07:23, Claudio Fontana wrote:
>>> It is a test environment for building packages, so the guest at the time of
>>> the error is running ./configure for the package swig-v4.1.1.tar.gz
>>>
>>> checking bui
On 7/28/23 16:40, Claudio Fontana wrote:
> On 7/28/23 16:28, Richard Henderson wrote:
>> On 7/28/23 07:23, Claudio Fontana wrote:
>>>> It is a test environment for building packages, so the guest at the time
>>>> of the error is running ./configure f
On 7/28/23 15:45, Claudio Fontana wrote:
> On 7/28/23 15:33, Richard Henderson wrote:
>> On 7/28/23 06:29, Claudio Fontana wrote:
>>> On 7/27/23 19:41, Richard Henderson wrote:
>>>> On 7/21/23 02:08, Claudio Fontana wrote:
>>>>> Thread 3 "
On 7/27/23 19:41, Richard Henderson wrote:
> On 7/21/23 02:08, Claudio Fontana wrote:
>> Thread 3 "qemu-system-s39" received signal SIGABRT, Aborted.
>> [Switching to Thread 0x753516c0 (LWP 215975)]
>> (gdb) bt
>> #0 0x7730dabc in __pthread_kill_im
On 7/28/23 15:33, Richard Henderson wrote:
> On 7/28/23 06:29, Claudio Fontana wrote:
>> On 7/27/23 19:41, Richard Henderson wrote:
>>> On 7/21/23 02:08, Claudio Fontana wrote:
>>>> Thread 3 "qemu-system-s39" received signal SIGABRT, Aborted.
>>>
.. adding Alex, maybe something related to multithreaded TCG?
On 7/21/23 11:08, Claudio Fontana wrote:
>
> Hello Cornelia, Richard,
>
> I had some strange behavior in an s390x TCG VM that I am debugging,
>
> and configured latest upstream QEMU with --enable-debug --enable-de
Hello Cornelia, Richard,
I had some strange behavior in an s390x TCG VM that I am debugging,
and configured latest upstream QEMU with --enable-debug --enable-debug-tcg
and I am running the qemu binary with -d unimp,guest_errors .
I get:
/usr/bin/qemu-system-s390x -nodefaults -no-reboot -nogr
On 7/17/23 12:37, Igor Mammedov wrote:
> On Mon, 17 Jul 2023 10:32:33 +0200
> Claudio Fontana wrote:
>
>> Hello Igor,
>>
>> thanks for getting back to me on this,
>>
>> On 7/14/23 11:51, Igor Mammedov wrote:
>>> On Wed, 5 Jul 2023 10:12:40 +020
On 7/17/23 10:32, Claudio Fontana wrote:
> Hello Igor,
>
> thanks for getting back to me on this,
>
> On 7/14/23 11:51, Igor Mammedov wrote:
>> On Wed, 5 Jul 2023 10:12:40 +0200
>> Claudio Fontana wrote:
>>
>>> Hi all, partially resurrecting an
Hello Igor,
thanks for getting back to me on this,
On 7/14/23 11:51, Igor Mammedov wrote:
> On Wed, 5 Jul 2023 10:12:40 +0200
> Claudio Fontana wrote:
>
>> Hi all, partially resurrecting an old thread.
>>
>> I've seen how for Epyc something special was done in
Hi all, partially resurrecting an old thread.
I've seen how for Epyc something special was done in the past in terms of
apicid assignments based on topology, which was then reverted apparently,
but I wonder if something more general would be useful to all?
The QEMU apicid assignments first of al
On 6/20/23 09:48, Philippe Mathieu-Daudé wrote:
> Although the PPC target only supports the TCG and KVM
> accelerators, QEMU supports more. We can no assume that
> '!kvm == tcg', so test for the correct accelerator. This
> also eases code review, because here we don't care about
> KVM, we really wa
On 5/24/23 11:24, Lin Ma wrote:
> The query-named-block-nodes is only availabe for qmp, not support hmp yet.
>
> Lin
Ok, makes sense.
AIO_ZONE_MGMT0x0200
> #define QEMU_AIO_ZONE_APPEND 0x0400
> +#define QEMU_AIO_FSTAT0x0800
> #define QEMU_AIO_TYPE_MASK \
> (QEMU_AIO_READ | \
> QEMU_AIO_WRITE | \
> @@ -42,7 +43,8 @@
> QEMU_AIO_TRUNCATE | \
> QEMU_AIO_ZONE_REPORT | \
> QEMU_AIO_ZONE_MGMT | \
> - QEMU_AIO_ZONE_APPEND)
> + QEMU_AIO_ZONE_APPEND | \
> + QEMU_AIO_FSTAT)
>
> /* AIO flags */
> #define QEMU_AIO_MISALIGNED 0x1000
Reviewed-by: Claudio Fontana
On 5/23/23 23:39, Fabiano Rosas wrote:
> From: Lin Ma
>
> Sometimes the query-block performs time-consuming I/O(say waiting for
> the fstat of NFS complete), So let's make this QMP handler runs in a
> coroutine.
>
> The following patch moves the fstat() into a thread pool.
>
> Signed-off-by: Li
On 5/23/23 23:39, Fabiano Rosas wrote:
> From: Lin Ma
>
> Sometimes the query-block performs time-consuming I/O(say waiting for
> the fstat of NFS complete), So let's make this QMP handler runs in a
> coroutine.
>
> The following patch moves the fstat() into a thread pool.
Hi, this message tal
Error **errp);
> void bdrv_query_image_info(BlockDriverState *bs,
> ImageInfo **p_info,
> bool flat,
Reviewed-by: Claudio Fontana
On 4/6/23 18:46, Fabiano Rosas wrote:
> Peter Xu writes:
>
>> On Tue, Apr 04, 2023 at 05:10:52PM +0200, Claudio Fontana wrote:
>>> On 4/4/23 16:53, Peter Xu wrote:
>>>> On Tue, Apr 04, 2023 at 10:00:16AM +0200, Claudio Fontana wrote:
>>>>> Hi Peter
On 4/4/23 16:53, Peter Xu wrote:
> On Tue, Apr 04, 2023 at 10:00:16AM +0200, Claudio Fontana wrote:
>> Hi Peter,
>
> Hi, Claudio,
>
>>
>> On 4/3/23 21:26, Peter Xu wrote:
>>> Hi, Claudio,
>>>
>>> Thanks for the context.
>>>
&
Hi Peter,
On 4/3/23 21:26, Peter Xu wrote:
> Hi, Claudio,
>
> Thanks for the context.
>
> On Mon, Apr 03, 2023 at 09:47:26AM +0200, Claudio Fontana wrote:
>> Hi, not sure if what is asked here is context in terms of the previous
>> upstream discussions or our specifi
On 3/31/23 23:52, Peter Xu wrote:
> On Fri, Mar 31, 2023 at 03:18:37PM -0300, Fabiano Rosas wrote:
>> Peter Xu writes:
>>
>>> On Fri, Mar 31, 2023 at 05:10:16PM +0100, Daniel P. Berrangé wrote:
On Fri, Mar 31, 2023 at 11:55:03AM -0400, Peter Xu wrote:
> On Fri, Mar 31, 2023 at 12:30:45PM
On 3/20/23 16:34, Philippe Mathieu-Daudé wrote:
> On 20/3/23 16:23, Claudio Fontana wrote:
>> Hi Alex, all,
>>
>> again, this moves TCG-only code to common code, no?
>
> Oh, good point.
>
>> Even if this happens to work, the idea is to avoid adding unnee
Hi Alex, all,
again, this moves TCG-only code to common code, no?
Even if this happens to work, the idea is to avoid adding unneeded accel TCG
code to a KVM-only binary.
We need to keep in mind all dimensions when we do refactorings:
user-mode vs sysemu,
the architecture,
the accel, in particu
On 3/20/23 14:32, Alex Bennée wrote:
>
> Claudio Fontana writes:
>
>> How is this conditional on CONFIG_TCG? To me it looks like this breaks
>> !CONFIG_TCG.
>> Careful, the meson.build in accel/tcg/meson.build is always recursed.
>
> Surely it shouldn't b
How is this conditional on CONFIG_TCG? To me it looks like this breaks
!CONFIG_TCG.
Careful, the meson.build in accel/tcg/meson.build is always recursed.
This code was in tcg_ss before, why not simply add it to tcg_ss and then to
specific_ss along with the other tcg pieces?
Ciao,
C
On 3/20/2
(adding Fabiano to the thread)
On 2/10/23 16:35, Daniel P. Berrangé wrote:
> On Thu, Feb 09, 2023 at 02:32:01PM +0100, Claudio Fontana wrote:
>> Hello Daniel and all,
>>
>> resurrecting this series from end of last year,
>>
>> do we think that this is the right
Added Fabiano to the thread,
CLaudio
On 3/20/23 12:05, Claudio Fontana wrote:
> On 2/10/23 18:11, Daniel P. Berrangé wrote:
>> On Fri, Oct 28, 2022 at 01:39:10PM +0300, Nikolay Borisov wrote:
>>> Implement 'fixed-ram' feature. The core of the feature is to ensure t
On 2/10/23 18:11, Daniel P. Berrangé wrote:
> On Fri, Oct 28, 2022 at 01:39:10PM +0300, Nikolay Borisov wrote:
>> Implement 'fixed-ram' feature. The core of the feature is to ensure that
>> each ram page of the migration stream has a specific offset in the
>> resulting migration stream. The reason
On 1/30/23 12:44, Thomas Huth wrote:
> Testing 32-bit host OS support takes a lot of precious time during the QEMU
> contiguous integration tests, and considering that many OS vendors stopped
> shipping 32-bit variants of their OS distributions and most hardware from
> the past >10 years is capable
On 2/17/23 11:36, Markus Armbruster wrote:
> I feel the discussion petered out without a conclusion.
>
> I don't think letting the status quo win by inertia is a good outcome
> here.
>
> Which 32-bit hosts are still useful, and why?
Hi Markus,
if the question is very very general, my opinion i
On 1/17/23 12:55, Chuang Xu wrote:
> In this version:
>
> - rename rcu_read_locked() to rcu_read_is_locked().
> - adjust the sanity check in address_space_to_flatview().
> - improve some comments.
>
> The duration of loading non-iterable vmstate accounts for a significant
> portion of downtime (s
1 - 100 of 1005 matches
Mail list logo
