Re: [PATCH 0/3] virtio-gpu: Enable virglrenderer backend for rutabaga

Weifeng Liu writes: > Hi Alex, > > On Thu, 2024-06-06 at 11:43 +0100, Alex Bennée wrote: >> Weifeng Liu writes: >> >> > Greetings, >> > >> > I'd like to introduce you my attempt to enable virglrenderer backend for >> > rutabaga empo

[PULL 03/12] tests/vm: update centos.aarch64 image to 9

As Centos Stream 8 goes out of support we need to update. To do this powertools is replaced by crb and we don't over specify the python3 we want. Reviewed-by: Richard Henderson Reviewed-by: Thomas Huth Signed-off-by: Alex Bennée Message-Id: <20240603175328.3823123-4-alex.ben...@linaro.

[PULL 07/12] tests/lcitool: Delete obsolete centos-stream-8.yml file

: <20240601070543.37786-2-th...@redhat.com> Signed-off-by: Alex Bennée Message-Id: <20240603175328.3823123-8-alex.ben...@linaro.org> diff --git a/tests/lcitool/targets/centos-stream-8.yml b/tests/lcitool/targets/centos-stream-8.yml deleted file mode 100644 index 6b11160fd1..00

[PULL 10/12] tests/lcitool: Install mingw-w64-tools for the Windows cross-builds

-th...@redhat.com> Signed-off-by: Alex Bennée Message-Id: <20240603175328.3823123-11-alex.ben...@linaro.org> diff --git a/tests/docker/dockerfiles/fedora-win64-cross.docker b/tests/docker/dockerfiles/fedora-win64-cross.docker index fef846d5a6..007e1574bd 100644 --- a/tests/docker/dockerfiles/

[PULL 11/12] tests/lcitool: generate package lists for ansible

With the new ability to output YAML we can build the package list for our ansible setup scripts. We will integrate them in the next commit. Signed-off-by: Alex Bennée Message-Id: <20240603175328.3823123-12-alex.ben...@linaro.org> diff --git a/scripts/ci/setup/ubuntu/ubuntu-2204-aarch64.y

[PULL 12/12] scripts/ci: drive ubuntu/build-environment.yml from lcitool

Now lcitool can write the package list for us we no longer need to duplicate the information directly in build-environment. Signed-off-by: Alex Bennée Message-Id: <20240603175328.3823123-13-alex.ben...@linaro.org> diff --git a/scripts/ci/setup/ubuntu/build-environment.yml b/scripts/ci

[PULL 00/12] testing cleanups (ci, vm, lcitool, ansible)

Alex Bennée (8): ci: remove centos-steam-8 customer runner docs/devel: update references to centos to non-versioned container tests/vm: update centos.aarch64 image to 9 tests/vm: remove plain centos image scripts/ci: remove CentOS bits from common build

[PULL 01/12] ci: remove centos-steam-8 customer runner

-by: Richard Henderson Signed-off-by: Alex Bennée Message-Id: <20240603175328.3823123-2-alex.ben...@linaro.org> diff --git a/docs/devel/ci-jobs.rst.inc b/docs/devel/ci-jobs.rst.inc index be06322279..3756bbe355 100644 --- a/docs/devel/ci-jobs.rst.inc +++ b/docs/devel/ci-jobs.rst.inc @@ -182,13

[PULL 09/12] tests/lcitool: Bump to latest libvirt-ci and update Fedora and Alpine version

ork-around package that we can use until somebody updates our Avocado to a newer version. Signed-off-by: Thomas Huth Message-Id: <20240601070543.37786-3-th...@redhat.com> [AJB: regen on rebase] Reviewed-by: Daniel P. Berrangé Signed-off-by: Alex Bennée Message-Id: <20240603175328.

[PULL 06/12] docs/ci: clean-up references for consistency

Document we have split up build-environment by distro and update the references that exist in the code base to be correct. Reviewed-by: Richard Henderson Signed-off-by: Alex Bennée Message-Id: <20240603175328.3823123-7-alex.ben...@linaro.org> diff --git a/docs/devel/ci-runners.rst.inc

[PULL 04/12] tests/vm: remove plain centos image

This isn't really used and we have lighter weight docker containers for testing this stuff directly. Reviewed-by: Thomas Huth Reviewed-by: Richard Henderson Signed-off-by: Alex Bennée Message-Id: <20240603175328.3823123-5-alex.ben...@linaro.org> diff --git a/tests/vm/Makefile.include b

[PULL 02/12] docs/devel: update references to centos to non-versioned container

have to bump at the next update. Reviewed-by: Richard Henderson Reviewed-by: Thomas Huth Signed-off-by: Alex Bennée Message-Id: <20240603175328.3823123-3-alex.ben...@linaro.org> diff --git a/docs/devel/testing.rst b/docs/devel/testing.rst index fa28e3ecb2..23d3f44f52 100644 --- a/docs/devel/tes

[PULL 05/12] scripts/ci: remove CentOS bits from common build-environment

Although I've just removed the CentOS specific build-environment its probably a bit too confusing to have multiple distros mixed up in one place. Prior to moving clean-up what will be just for ubuntu. Reviewed-by: Richard Henderson Signed-off-by: Alex Bennée Message-Id: <20240603175328.3823

[PULL 08/12] .gitlab-ci.d/buildtest.yml: Use -fno-sanitize=function in the clang-system job

this off in the failing clang-system job until all spots in the QEMU sources have been tackled. Signed-off-by: Thomas Huth Reviewed-by: Daniel P. Berrangé Message-Id: <20240601070543.37786-4-th...@redhat.com> Signed-off-by: Alex Bennée Message-Id: <20240603175328.3823123-9

Re: [PATCH 0/3] virtio-gpu: Enable virglrenderer backend for rutabaga

gle.com/ > > Weifeng Liu (3): > virtio-gpu: rutabaga: Properly set stride when copying resources > virtio-gpu: rutabaga: Poll rutabaga upon events > virtio-gpu: rutabaga: Add options to enable virgl and venus contexts > > hw/display/virtio-gpu-rutabaga.c | 104 ++- > include/hw/virtio/virtio-gpu.h | 1 + > 2 files changed, 104 insertions(+), 1 deletion(-) -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: [PATCH v2 4/9] plugins: Introduce TCGCPUOps callbacks for mid-tb register reads

Alex Bennée writes: > Richard Henderson writes: > >> Certain target registers are not updated continuously within >> the translation block. For normal exception handling we use >> unwind info to re-generate the correct value when required. >> Leverage that same in

Re: [PATCH v2 7/9] target/i386: Implement TCGCPUOps for plugin register reads

Richard Henderson writes: > Reviewed-by: Pierrick Bouvier > Signed-off-by: Richard Henderson Reviewed-by: Alex Bennée -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: [PATCH v2 4/9] plugins: Introduce TCGCPUOps callbacks for mid-tb register reads

; > All targets will need updating for these new callbacks. > > Reviewed-by: Pierrick Bouvier > Signed-off-by: Richard Henderson Reviewed-by: Alex Bennée -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: [PATCH v2 3/9] accel/tcg: Return the TranslationBlock from cpu_unwind_state_data

Richard Henderson writes: > Adjust the i386 get_memio_eip function to use tb->cflags instead > of tcg_cflags_has, which is technically more correct. > > Reviewed-by: Pierrick Bouvier > Signed-off-by: Richard Henderson Reviewed-by: Alex Bennée -- Alex Bennée Virtua

Re: [PATCH v2 2/9] accel/tcg: Set CPUState.plugin_ra before all plugin callbacks

Richard Henderson writes: > Store a host code address to use with the tcg unwinder when called > from a plugin. Generate one such store per guest insn that uses > a plugin callback. > > Reviewed-by: Pierrick Bouvier > Signed-off-by: Richard Henderson Reviewed-by: Alex

Re: [PATCH v2 1/9] tcg: Introduce INDEX_op_plugin_pc

Richard Henderson writes: > Add an opcode to find a code address within the current insn, > for later use with unwinding. Generate the code generically > using tcg_reg_alloc_do_movi. > > Reviewed-by: Pierrick Bouvier > Signed-off-by: Richard Henderson Reviewed-by: Alex

Re: [PATCH v12 00/13] Support blob memory and venus on qemu

Dmitry Osipenko writes: > On 6/5/24 17:47, Alex Bennée wrote: > >> I'm guessing some sort of resource leak, if I run vkcube-wayland in the >> guest it complains about being stuck on a fence with the iterator going >> up. However on the host I see: >> >>

Re: [RFC PATCH] subprojects: add a wrapper for libvirglrenderer

Manos Pitsidianakis writes: > On Wed, 05 Jun 2024 16:35, Alex Bennée wrote: >>As the latest features for virtio-gpu need a pretty recent version of >>libvirglrenderer. When it is not available on the system we can use a >>meson wrapper and provide it when --download is spe

Re: [PATCH v12 00/13] Support blob memory and venus on qemu

Dmitry Osipenko writes: > On 5/22/24 12:00, Alex Bennée wrote: >> Dmitry Osipenko writes: >> >>> On 5/21/24 17:57, Alex Bennée wrote: >>>> Alex Bennée writes: >>>> >>>>> Dmitry Osipenko writes: >>>>> >>>

Re: [RFC PATCH] subprojects: add a wrapper for libvirglrenderer

Marc-André Lureau writes: > Hi > > On Wed, Jun 5, 2024 at 5:35 PM Alex Bennée wrote: > > As the latest features for virtio-gpu need a pretty recent version of > libvirglrenderer. When it is not available on the system we can use a > meson wrapper and provide it when --

[RFC PATCH] subprojects: add a wrapper for libvirglrenderer

fails to exec the render server. As the error isn't back propagated we make sure we at least test we have a path to an executable before tweaking the environment. Signed-off-by: Alex Bennée Cc: Manos Pitsidianakis Cc: Dmitry Osipenko Cc: Akihiko Odaki --- meson.build| 7

Re: [PATCH] docs, tests: do not specify scsi=off for virtio-blk-pci

Paolo Bonzini writes: > This has been the default forever. > > Signed-off-by: Paolo Bonzini Acked-by: Alex Bennée -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: [PATCH] mips64el-softmmu: Enable MTTCG

7d 100644 > --- a/configs/targets/mips64el-softmmu.mak > +++ b/configs/targets/mips64el-softmmu.mak > @@ -1,3 +1,4 @@ > TARGET_ARCH=mips64 > TARGET_BASE_ARCH=mips > +TARGET_SUPPORTS_MTTCG=y > TARGET_NEED_FDT=y > > --- > base-commit: 248f6f62df073a3b4158fd0093863ab885feabb5 > change-id: 20240511-mips_mttcg-47a6b19074b3 > > Best regards, -- Alex Bennée Virtualisation Tech Lead @ Linaro

[PATCH v3 03/12] tests/vm: update centos.aarch64 image to 9

As Centos Stream 8 goes out of support we need to update. To do this powertools is replaced by crb and we don't over specify the python3 we want. Reviewed-by: Richard Henderson Reviewed-by: Thomas Huth Message-Id: <20240521125314.1255403-4-alex.ben...@linaro.org> Signed-off-by: Alex

[PATCH v3 02/12] docs/devel: update references to centos to non-versioned container

have to bump at the next update. Reviewed-by: Richard Henderson Message-Id: <20240521125314.1255403-3-alex.ben...@linaro.org> Reviewed-by: Thomas Huth Signed-off-by: Alex Bennée Message-Id: <20240529160934.982373-3-alex.ben...@linaro.org> --- docs/devel/testing.rst | 8 1 file ch

[PATCH v3 05/12] scripts/ci: remove CentOS bits from common build-environment

Although I've just removed the CentOS specific build-environment its probably a bit too confusing to have multiple distros mixed up in one place. Prior to moving clean-up what will be just for ubuntu. Signed-off-by: Alex Bennée Message-Id: <20240529160934.982373-6-alex.ben...@linaro.

[PATCH v3 07/12] tests/lcitool: Delete obsolete centos-stream-8.yml file

-by: Alex Bennée Message-Id: <20240601070543.37786-2-th...@redhat.com> --- tests/lcitool/targets/centos-stream-8.yml | 3 --- 1 file changed, 3 deletions(-) delete mode 100644 tests/lcitool/targets/centos-stream-8.yml diff --git a/tests/lcitool/targets/centos-stream-8.yml b/tests/lcitool/t

[PATCH v3 04/12] tests/vm: remove plain centos image

This isn't really used and we have lighter weight docker containers for testing this stuff directly. Reviewed-by: Thomas Huth Reviewed-by: Richard Henderson Message-Id: <20240521125314.1255403-5-alex.ben...@linaro.org> Signed-off-by: Alex Bennée Message-Id: <20240529160934.982373-5

[PATCH v3 09/12] tests/lcitool: Bump to latest libvirt-ci and update Fedora and Alpine version

ork-around package that we can use until somebody updates our Avocado to a newer version. Signed-off-by: Thomas Huth Message-Id: <20240601070543.37786-3-th...@redhat.com> [AJB: regen on rebase] Signed-off-by: Alex Bennée Reviewed-by: Daniel P. Berrangé --- tests/docker/dockerfil

[PATCH v3 08/12] .gitlab-ci.d/buildtest.yml: Use -fno-sanitize=function in the clang-system job

this off in the failing clang-system job until all spots in the QEMU sources have been tackled. Signed-off-by: Thomas Huth Reviewed-by: Daniel P. Berrangé Message-Id: <20240601070543.37786-4-th...@redhat.com> Signed-off-by: Alex Bennée --- .gitlab-ci.d/buildtest.yml | 1 + 1 file chan

[PATCH v3 06/12] docs/ci: clean-up references for consistency

Document we have split up build-environment by distro and update the references that exist in the code base to be correct. Signed-off-by: Alex Bennée Message-Id: <20240529160934.982373-7-alex.ben...@linaro.org> --- docs/devel/ci-runners.rst.inc

[PATCH v3 00/12] testing/next: purging centos and more lcitool (pre-PR)

CentOS bits from common build-environment Alex Bennée (8): ci: remove centos-steam-8 customer runner docs/devel: update references to centos to non-versioned container tests/vm: update centos.aarch64 image to 9 tests/vm: remove plain centos image scripts/ci: remove CentOS bits from common

[PATCH v3 10/12] tests/lcitool: Install mingw-w64-tools for the Windows cross-builds

-th...@redhat.com> Signed-off-by: Alex Bennée --- tests/docker/dockerfiles/fedora-win64-cross.docker | 1 + tests/lcitool/projects/qemu-win-installer.yml | 1 + 2 files changed, 2 insertions(+) diff --git a/tests/docker/dockerfiles/fedora-win64-cross.docker b/tests/docker/dockerfiles/fedora

[PATCH v3 11/12] tests/lcitool: generate package lists for ansible

With the new ability to output YAML we can build the package list for our ansible setup scripts. We will integrate them in the next commit. Signed-off-by: Alex Bennée Message-Id: <20240529160934.982373-9-alex.ben...@linaro.org> --- v2 - MR now merged into libvirt-ci --- .../ci/setup/

[PATCH v3 12/12] scripts/ci: drive ubuntu/build-environment.yml from lcitool

Now lcitool can write the package list for us we no longer need to duplicate the information directly in build-environment. Signed-off-by: Alex Bennée Message-Id: <20240529160934.982373-10-alex.ben...@linaro.org> --- scripts/ci/setup/ubuntu/build-environment.yml | 131 +-

[PATCH v3 01/12] ci: remove centos-steam-8 customer runner

-by: Richard Henderson Message-Id: <20240521125314.1255403-2-alex.ben...@linaro.org> Signed-off-by: Alex Bennée Message-Id: <20240529160934.982373-2-alex.ben...@linaro.org> --- docs/devel/ci-jobs.rst.inc| 7 - .gitlab-ci.d/custom-runners.yml | 1 - .../cu

Re: [PATCH] io/channel-socket: Fix -fsanitize=undefined problem with latest Clang

Daniel P. Berrangé writes: > On Mon, Jun 03, 2024 at 03:47:55PM +0100, Alex Bennée wrote: >> Thomas Huth writes: >> >> > Casting function pointers from one type to another causes undefined >> > behavior errors when compiling with -fsanitize=undefined w

Re: [PATCH] io/channel-socket: Fix -fsanitize=undefined problem with latest Clang

Thomas Huth writes: > Casting function pointers from one type to another causes undefined > behavior errors when compiling with -fsanitize=undefined with Clang > v18: Queued to testing/next, thanks. -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: [PATCH 4/5] plugins: remove special casing for cpu->realized

Philippe Mathieu-Daudé writes: > On 30/5/24 21:42, Alex Bennée wrote: >> Now the condition variable is initialised early on we don't need to go >> through hoops to avoid calling async_run_on_cpu. >> Signed-off-by: Alex Bennée >> --- >> plugins/core.c

Re: [PATCH 5/7] target/i386: Split out gdb-internal.h

Richard Henderson writes: > Signed-off-by: Richard Henderson Reviewed-by: Alex Bennée -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: [PATCH 3/7] accel/tcg: Return the TranslationBlock from cpu_unwind_state_data

a[0]; this has a merge conflict with subsequent changes. > } else { > return data[0] - env->segs[R_CS].base; -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: [PATCH 2/7] accel/tcg: Set CPUState.plugin_ra before all plugin callbacks

{ > inject_cb( > -_array_index(cbs, struct qemu_plugin_dyn_cb, i)); > +_array_index(cbs, struct qemu_plugin_dyn_cb, i), > +_ra); > } > break; >

Re: [PATCH v3 0/6] Implement icount=auto using TCG Plugins

$ time ./build/qemu-x86_64 -plugin > ./build/contrib/plugins/libips,ips=$((120546/5)) /bin/true > real0m4.985s > > Tested in system mode by booting a full debian system, and using: > $ sysbench cpu run > Performance decrease linearly with the given number of ips. Queued to plugins/next, thanks. -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: [PATCH] plugins: Ensure register handles are not NULL

Akihiko Odaki writes: > Ensure register handles are not NULL so that a plugin can assume NULL is > invalid as a register handle. > > Signed-off-by: Akihiko Odaki Queued to plugins/next, thanks. -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: [RFC 0/6] scripts: Rewrite simpletrace printer in Rust

yet and doing so start implying externally visible versioning and deprecation policies for QEMU APIs and data formats. TCG plugins have a header based API but currently everyone builds against their local checkout and we are fairly liberal about bumping the API versions. > > > With regards, > Daniel -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: [PATCH 5/5] core/cpu-common: initialise plugin state before thread creation

Pierrick Bouvier writes: > On 5/30/24 12:42, Alex Bennée wrote: >> Originally I tried to move where vCPU thread initialisation to later >> in realize. However pulling that thread (sic) got gnarly really >> quickly. It turns out some steps of CPU realization need va

Re: [Semihosting Tests PATCH v2 1/3] .editorconfig: add code conventions for tooling

Brian Cain writes: > On 5/30/2024 6:23 AM, Alex Bennée wrote: >> It's a pain when you come back to a code base you haven't touched in a >> while and realise whatever indent settings you were using having >> carried over. Add an editorconfig and be done with it. >> &g

Re: [PATCH 5/5] core/cpu-common: initialise plugin state before thread creation

Philippe Mathieu-Daudé writes: > On 30/5/24 21:42, Alex Bennée wrote: >> Originally I tried to move where vCPU thread initialisation to later >> in realize. However pulling that thread (sic) got gnarly really >> quickly. It turns out some steps of CPU realization need va

[PATCH 1/5] hw/core: expand on the alignment of CPUState

Make the relationship between CPUState, ArchCPU and cpu_env a bit clearer in the kdoc comments. Signed-off-by: Alex Bennée --- include/hw/core/cpu.h | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/include/hw/core/cpu.h b/include/hw/core/cpu.h index bb398e8237

[PATCH 2/5] cpu: move Qemu[Thread|Cond] setup into common code

specialises than making the other accelerators jump through hoops. Signed-off-by: Alex Bennée --- include/hw/core/cpu.h | 4 accel/dummy-cpus.c| 3 --- accel/hvf/hvf-accel-ops.c | 4 accel/kvm/kvm-accel-ops.c | 3 --- accel/tcg/tcg-accel-ops

[PATCH 4/5] plugins: remove special casing for cpu->realized

Now the condition variable is initialised early on we don't need to go through hoops to avoid calling async_run_on_cpu. Signed-off-by: Alex Bennée --- plugins/core.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/plugins/core.c b/plugins/core.c index 0726bc7f25

[PATCH 5/5] core/cpu-common: initialise plugin state before thread creation

of the thread creation we can now queue work before the thread starts (at least for TCG guests) and avoid the race between vcpu_init and other vcpu states a plugin might subscribe to. Signed-off-by: Alex Bennée --- hw/core/cpu-common.c | 20 1 file changed, 12 insertions(+), 8

[PATCH 0/5] cpus: a few tweaks to CPU realization

in the vCPU threads can clearly be done earlier so I've moved enough from cpu_common_realizefn to cpu_common_initfn to allow plugins to queue work before the threads start solving the race. Please review. Alex Bennée (5): hw/core: expand on the alignment of CPUState cpu: move Qemu[Thread|Cond] setup

[PATCH 3/5] cpu-target: don't set cpu->thread_id to bogus value

The thread_id isn't valid until the threads are created. There is no point setting it here. The only thing that cares about the thread_id is qmp_query_cpus_fast. Signed-off-by: Alex Bennée --- cpu-target.c | 1 - 1 file changed, 1 deletion(-) diff --git a/cpu-target.c b/cpu-target.c index

Re: [RFC PATCH] cpus: split qemu_init_vcpu and delay vCPU thread creation

Pierrick Bouvier writes: > On 5/29/24 08:22, Alex Bennée wrote: >> This ensures we don't start the thread until cpu_common_realizefn has >> finished. This ensures that plugins will always run >> qemu_plugin_vcpu_init__async first before any other states. It doesn't >>

Re: [PATCH] hw/usb/hcd-ohci: Fix ohci_service_td: accept valid TDs

hat boots and executes a test, the issue can be seen: > > * OS that sends USB requests to a USB mass storage device > but sends td.cbp = td.be + 1 > * qemu 4.2 > * qemu HEAD (4e66a0854) > * Actual OHCI controller (hardware) -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: hw/usb/hcd-ohci: Fix #1510, #303: pid not IN or OUT

Cord Amfmgm writes: > On Thu, May 30, 2024 at 3:33 AM Alex Bennée wrote: > > Cord Amfmgm writes: > > > On Tue, May 28, 2024 at 11:32 AM Peter Maydell > wrote: > > > > On Tue, 28 May 2024 at 16:37, Cord Amfmgm wrote: > > > > >

[Semihosting Tests PATCH v2 3/3] add SYS_GET_CMDLINE test

was exposing the binary name into the program so we can validate the result. Signed-off-by: Alex Bennée --- v2 - add return leg on failing test - check returned length is sane - s/could/couldn't/ - just let array reference decay to the pointer - rewrite the commit message to be less terse

[Semihosting Tests PATCH v2 2/3] update includes for bare metal compiling

We shouldn't use for our own implementation. Also the base types we need live in as doesn't exist for the bare metal compilers. Signed-off-by: Alex Bennée Reviewed-by: Peter Maydell --- semihosting.c | 4 ++-- semihosting.h | 2 +- string.c | 2 +- usertest.c| 2 +- 4 files changed

[Semihosting Tests PATCH v2 0/3] add SYS_GET_CMDLINE test

Hi Peter, Looking at bug #2322 I wanted to make sure SYS_GET_CMDLINE works as I expected. While at it I needed to fix a compile error with headers which I guess we got away with on earlier compilers. I've added an editorconfig for good measure. v2 - addressed review comments Alex Bennée (3

[Semihosting Tests PATCH v2 1/3] .editorconfig: add code conventions for tooling

It's a pain when you come back to a code base you haven't touched in a while and realise whatever indent settings you were using having carried over. Add an editorconfig and be done with it. Signed-off-by: Alex Bennée --- v2 - drop mention of custom major modes (not needed here) - include

Re: hw/usb/hcd-ohci: Fix #1510, #303: pid not IN or OUT

d guest OS bugs, but only as > when we really have to. It's usually better to fix the > bug in the guest. > > It's not, and I've already demonstrated that real hardware is consistent with > the fix in this patch. > > Please check your tone. I don't think that is a particularly he

Re: [PATCH 3/3] semihosting: Restrict to TCG

fig >> @@ -1,6 +1,7 @@ >> >> config SEMIHOSTING >> bool >> + depends on TCG >> >> config ARM_COMPATIBLE_SEMIHOSTING >> bool >> -- >> 2.41.0 >> -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: [PATCH 0/3] semihosting: Restrict to TCG

sting/next, thanks. -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: [PATCH 07/10] tests/lcitool: bump to latest version

Thomas Huth writes: > On 29/05/2024 18.09, Alex Bennée wrote: >> We have to simultaneously update a few bits on our side as lcitool has >> already deprecated fedora-38, alpine-3.18 and centos-8-stream. However >> there is no change to the package list yet. >> Signed-of

Re: [RFC PATCH] cpus: split qemu_init_vcpu and delay vCPU thread creation

Philippe Mathieu-Daudé writes: > Hi Alex, > > On 29/5/24 17:22, Alex Bennée wrote: >> This ensures we don't start the thread until cpu_common_realizefn has >> finished. This ensures that plugins will always run >> qemu_plugin_vcpu_init__async first before any other st

[PATCH 00/10] testing/next: purging centos and more lcitool

-tool: bump to latest version docs/ci: clean-up references for consistency scripts/ci: remove CentOS bits from common build-environment Alex Bennée (9): ci: remove centos-steam-8 customer runner docs/devel: update references to centos to non-versioned container tests/vm: update centos.aarch64

[PATCH 03/10] tests/vm: update centos.aarch64 image to 9

As Centos Stream 8 goes out of support we need to update. To do this powertools is replaced by crb and we don't over specify the python3 we want. Reviewed-by: Richard Henderson Signed-off-by: Alex Bennée Reviewed-by: Thomas Huth Message-Id: <20240521125314.1255403-4-alex.ben...@linaro.

[PATCH 01/10] ci: remove centos-steam-8 customer runner

-by: Richard Henderson Signed-off-by: Alex Bennée Message-Id: <20240521125314.1255403-2-alex.ben...@linaro.org> --- docs/devel/ci-jobs.rst.inc| 7 - .gitlab-ci.d/custom-runners.yml | 1 - .../custom-runners/centos-stream-8-x86_64.yml | 24 --- .../org.centos/st

[PATCH 02/10] docs/devel: update references to centos to non-versioned container

have to bump at the next update. Reviewed-by: Richard Henderson Signed-off-by: Alex Bennée Message-Id: <20240521125314.1255403-3-alex.ben...@linaro.org> --- v2 - replaced centos with debian, tweaked commit message --- docs/devel/testing.rst | 8 1 file changed, 4 insertions(+), 4 delet

[PATCH 10/10] tests/avocado: update sbsa-ref firmware

-by: Marcin Juszkiewicz Reviewed-by: Leif Lindholm Message-Id: <20240528182917.91027-1-marcin.juszkiew...@linaro.org> Signed-off-by: Alex Bennée --- tests/avocado/machine_aarch64_sbsaref.py | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/tests/a

[PATCH 05/10] scripts/ci: remove CentOS bits from common build-environment

Although I've just removed the CentOS specific build-environment its probably a bit too confusing to have multiple distros mixed up in one place. Prior to moving clean-up what will be just for ubuntu. Signed-off-by: Alex Bennée --- scripts/ci/setup/build-environment.yml | 98

[PATCH 04/10] tests/vm: remove plain centos image

This isn't really used and we have lighter weight docker containers for testing this stuff directly. Reviewed-by: Thomas Huth Reviewed-by: Richard Henderson Signed-off-by: Alex Bennée Message-Id: <20240521125314.1255403-5-alex.ben...@linaro.org> --- tests/vm/Makefile.include | 1 - te

[PATCH 08/10] tests/lcitool: generate package lists for ansible

This includes updating libvirt-ci to the currently in-flight MR: https://gitlab.com/libvirt/libvirt-ci/-/merge_requests/488 With the new ability to output YAML we can build the package list for our ansible setup scripts. We will integrate them in the next commit. Signed-off-by: Alex Bennée

[PATCH 06/10] docs/ci: clean-up references for consistency

Document we have split up build-environment by distro and update the references that exist in the code base to be correct. Signed-off-by: Alex Bennée --- docs/devel/ci-runners.rst.inc | 13 ++--- .../custom-runners/ubuntu-22.04-aarch32.yml | 2

[PATCH 09/10] scripts/ci: drive ubuntu/build-environment.yml from lcitool

Now lcitool can write the package list for us we no longer need to duplicate the information directly in build-environment. Signed-off-by: Alex Bennée --- scripts/ci/setup/ubuntu/build-environment.yml | 131 +- 1 file changed, 7 insertions(+), 124 deletions(-) diff --git

[PATCH 07/10] tests/lcitool: bump to latest version

We have to simultaneously update a few bits on our side as lcitool has already deprecated fedora-38, alpine-3.18 and centos-8-stream. However there is no change to the package list yet. Signed-off-by: Alex Bennée --- v2 - fix typo s/lci-tool/lcitool/ --- tests/docker/dockerfiles

Re: [PATCH 5/5] contrib/plugins: add ips plugin example for cost modeling

Pierrick Bouvier writes: > On 5/29/24 05:13, Alex Bennée wrote: >> Pierrick Bouvier writes: >> (Added Philip to CC) >> >>> On 5/28/24 12:57, Alex Bennée wrote: >>>> Pierrick Bouvier writes: >>>> >>>>>

Re: [PATCH 1/1] tests/avocado: update sbsa-ref firmware

n 'bookworm' cross toolchain (gcc > 12.2.0). Queued to testing/next, thanks. -- Alex Bennée Virtualisation Tech Lead @ Linaro

[RFC PATCH] cpus: split qemu_init_vcpu and delay vCPU thread creation

in cpu_create() before continuing. Signed-off-by: Alex Bennée Cc: Pierrick Bouvier Cc: Philippe Mathieu-Daudé --- include/hw/core/cpu.h | 8 accel/tcg/user-exec-stub.c | 5 + hw/core/cpu-common.c | 7 ++- plugins/core.c | 5 + system/cpus.c

Re: [PATCH 5/5] contrib/plugins: add ips plugin example for cost modeling

Pierrick Bouvier writes: (Added Philip to CC) > On 5/28/24 12:57, Alex Bennée wrote: >> Pierrick Bouvier writes: >> >>> On 5/28/24 12:14, Alex Bennée wrote: >>>> Pierrick Bouvier writes: >>>> >>>>> This plugin uses the new

Re: [PATCH 5/5] contrib/plugins: add ips plugin example for cost modeling

Pierrick Bouvier writes: > On 5/28/24 12:14, Alex Bennée wrote: >> Pierrick Bouvier writes: >> >>> This plugin uses the new time control interface to make decisions >>> about the state of time during the emulation. The algorithm is >>> currently v

Re: [PATCH 5/5] contrib/plugins: add ips plugin example for cost modeling

tion)) { > +fprintf(stderr, "boolean argument parsing failed: %s\n", > opt); > +return -1; > +} > +} else { > +fprintf(stderr, "option parsing failed: %s\n", opt); > +return -1; > +} > +} > + > +vcpus = qemu_plugin_scoreboard_new(sizeof(vCPUTime)); > +insn_quantum = insn_per_second / NUM_TIME_UPDATE_PER_SEC; > + > +time_handle = qemu_plugin_request_time_control(); > +g_assert(time_handle); > + > +qemu_plugin_register_vcpu_tb_trans_cb(id, vcpu_tb_trans); > +qemu_plugin_register_vcpu_init_cb(id, vcpu_init); > +qemu_plugin_register_vcpu_idle_cb(id, vcpu_idle); > +qemu_plugin_register_vcpu_resume_cb(id, vcpu_resume); > +qemu_plugin_register_vcpu_exit_cb(id, vcpu_exit); > +qemu_plugin_register_atexit_cb(id, plugin_exit, NULL); > + > +return 0; > +} > diff --git a/contrib/plugins/Makefile b/contrib/plugins/Makefile > index 0b64d2c1e3a..449ead11305 100644 > --- a/contrib/plugins/Makefile > +++ b/contrib/plugins/Makefile > @@ -27,6 +27,7 @@ endif > NAMES += hwprofile > NAMES += cache > NAMES += drcov > +NAMES += ips > > ifeq ($(CONFIG_WIN32),y) > SO_SUFFIX := .dll -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: [PATCH 1/5] sysemu: add set_virtual_time to accel ops

Pierrick Bouvier writes: > From: Alex Bennée > > We are about to remove direct calls to individual accelerators for > this information and will need a central point for plugins to hook > into time changes. > > From: Alex Bennée > Signed-off-by: Alex Bennée > Re

Re: [PATCH 2/4] gdbstub: Add support for MTE in user mode

l/aarch64-fpu.xml > gdb-xml/aarch64-pauth.xml gdb-xml/aarch64-mte.xml Ahh there it is, this got missed from the commit -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: [PATCH 1/4] gdbstub: Add support for target-specific stubs

; + * > + * Currently supported delimiters: > + * '?' -> Stop at any delimiter (",;:=\0") > + * '0' -> Stop at "\0" > + * '.' -> Skip 1 char unless reached "\0" > + * Any other value is treated as the delimiter value itself > + */ > +typedef

Re: [PATCH 0/4] Add MTE stubs for aarch64 user mode

mu.git /home/alex/lsrc/qemu.git/builds/all make: *** [Makefile:167: run-ninja] Error 1 Compilation exited abnormally with code 2 at Tue May 28 16:59:05 -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: [PATCH 0/4] testing/next: purging remaining centos 8 bits

Philippe Mathieu-Daudé writes: > Hi Alex, > > On 21/5/24 14:53, Alex Bennée wrote: >> There are a few more bits referencing centos8 in the tree which needed >> cleaning up. After this we can remove the dedicated runner from the >> gitlab registration. If we want to kee

Re: Test scripts

; in the host copy of the script - there should be path to the directory > containing a copy of the script in VM. Execute the host script with > net arguments: > ./vsock_guest_exec_test.py srv Maybe these would best live in contrib/vsock with a README? -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: [PULL 10/38] tests/qtest/migration: Add a test for the analyze-migration script

Fabiano Rosas writes: > Alex Bennée writes: > >> Juan Quintela writes: >> >>> From: Fabiano Rosas >>> >>> Add a smoke test that migrates to a file and gives it to the >>> script. It should catch the most annoying errors such as changes in

Re: [PATCH v12 03/13] virtio-gpu: Use pkgconfig version to decide which virgl features are available

FO_EXT', 1) > endif If all the host side needs is 1.0.0 then: Reviewed-by: Alex Bennée -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: [PATCH v12 02/13] virtio-gpu: Handle virtio_gpu_virgl_init() failure

RS_RESET, /** failed initialisation */ RS_INIT_FAILED } RenderState; then for example you could flow virtio_gpu_gl_handle_ctrl() as: switch (gl->render_state) { case RS_START: { gl->render_state = virtio_gpu_virgl_init(g) ? RS_INIT_FAILED : RS_INITED; } case RS_RESET:

Re: [PATCH v12 01/13] virtio-gpu: Unrealize GL device

@ -336,6 +336,7 @@ void virtio_gpu_virgl_fence_poll(VirtIOGPU *g); > void virtio_gpu_virgl_reset_scanout(VirtIOGPU *g); > void virtio_gpu_virgl_reset(VirtIOGPU *g); > int virtio_gpu_virgl_init(VirtIOGPU *g); > +void virtio_gpu_virgl_deinit(VirtIOGPU *g); > int virtio_gpu_virgl_get_num_capsets(VirtIOGPU *g); > > #endif -- Alex Bennée Virtualisation Tech Lead @ Linaro

Re: [PATCH v12 00/13] Support blob memory and venus on qemu

Dmitry Osipenko writes: > On 5/21/24 17:57, Alex Bennée wrote: >> Alex Bennée writes: >> >>> Dmitry Osipenko writes: >>> >>>> Hello, >>>> >>>> This series enables Vulkan Venus context support on virtio-gpu. >>>>

Re: [PATCH V10 7/8] gdbstub: Add helper function to unregister GDB register space

Salil Mehta writes: > Hi Alex, > >> From: Alex Bennée >> Sent: Tuesday, May 21, 2024 1:45 PM >> To: Salil Mehta >> >> Salil Mehta writes: >> >> > Add common function to help unregister the GDB register space. This >

  1   2   3   4   5   6   7   8   9   10   >