Pierrick Bouvier writes:
> On 7/18/24 07:59, Alex Bennée wrote:
>> This is a simple control flow tracking plugin that uses the latest
>> inline and conditional operations to detect and track control flow
>> changes. It is currently an exercise at seeing how useful the cha
a-f]+\)."
>
> Double up the \ to pass one through to the pattern.
>
> Signed-off-by: Richard Henderson
Queued to maintainer/for-9.1-rc0, thanks.
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
ro
Cc: Pierrick Bouvier
Signed-off-by: Alex Bennée
Message-Id: <20240311153432.1395190-1-alex.ben...@linaro.org>
---
v2
- only need a single call back
- drop need for INSN_WIDTH
- still don't understand the early exits
v3
- move initial STORE ops to first instruction to avo
: <20240717105723.58965-7-phi...@linaro.org>
Signed-off-by: Alex Bennée
---
target/riscv/Kconfig | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/target/riscv/Kconfig b/target/riscv/Kconfig
index 5f30df22f2..c332616d36 100644
--- a/target/riscv/Kconfig
+++ b/target/riscv/K
: <20240717105723.58965-6-phi...@linaro.org>
Signed-off-by: Alex Bennée
---
target/mips/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/target/mips/Kconfig b/target/mips/Kconfig
index eb19c94c7d..876048b150 100644
--- a/target/mips/Kconfig
+++ b/target/mips/Kconfig
@@ -1,
complete_cb complete,
^
Signed-off-by: Philippe Mathieu-Daudé
Message-Id: <20240717105723.58965-2-phi...@linaro.org>
Signed-off-by: Alex Bennée
---
include/semihosting/syscalls.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/include/semiho
From: Philippe Mathieu-Daudé
Semihosting currently uses the TCG probe_access API.
It is pointless to have it in the binary when TCG isn't.
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Richard Henderson
Message-Id: <20240717105723.58965-9-phi...@linaro.org>
Signed-off-by: Alex
Using bare printf's in plugins is perfectly acceptable but they do
rather mess up the output of "make check-tcg". Convert the printfs to
use g_string and then output with the plugin output helper which will
already be captured to .pout files by the test harness.
Signed-off-by: A
Reported-by: Anton Johansson
Signed-off-by: Philippe Mathieu-Daudé
Message-Id: <20240717105723.58965-5-phi...@linaro.org>
Signed-off-by: Alex Bennée
---
target/m68k/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/target/m68k/Kconfig b/target/m68k/Kconfig
Bouvier
Message-Id: <20240715081521.19122-2-simon.hame...@grenoble-inp.org>
Signed-off-by: Alex Bennée
---
docs/devel/tcg-plugins.rst| 22 +
contrib/plugins/stoptrigger.c | 151 ++
contrib/plugins/Makefile | 1 +
3 files changed, 174 inse
From: Philippe Mathieu-Daudé
Since the SEMIHOSTING feature is optional, we need
a stub to link when it is disabled.
Suggested-by: Paolo Bonzini
Signed-off-by: Philippe Mathieu-Daudé
Message-Id: <20240717105723.58965-3-phi...@linaro.org>
Signed-off-by: Alex Bennée
---
target/m68k/semih
From: Frédéric Pétrot
Register values are dumped as 'sz' chunks of two nibbles in the execlog
plugin, sz was 1 too big.
Signed-off-by: Frédéric Pétrot
Reviewed-by: Pierrick Bouvier
Message-Id: <20240620083805.73603-1-frederic.pet...@univ-grenoble-alpes.fr>
Signed-off-by: Alex
Reported-by: Anton Johansson
Signed-off-by: Philippe Mathieu-Daudé
Message-Id: <20240717105723.58965-8-phi...@linaro.org>
Signed-off-by: Alex Bennée
---
target/xtensa/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/target/xtensa/Kconfig b/target/xten
qemu_plugin_outs for inline stats
testing: bump to latest libvirt-ci
Alex Bennée (3):
testing: bump to latest libvirt-ci
gdbstub: Re-factor gdb command extensions
tests/plugins: use qemu_plugin_outs for inline stats
Frédéric Pétrot (1):
plugins/execlog.c: correct dump of registers values
From: Pierrick Bouvier
data was correctly copied, but size of array was not set
(g_array_sized_new only reserves memory, but does not set size).
As a result, callbacks were not called for code path relying on
plugin_register_vcpu_mem_cb().
Found when trying to trigger mem access callbacks for
From: Philippe Mathieu-Daudé
Since the SEMIHOSTING feature is optional, we need
a stub to link when it is disabled.
Signed-off-by: Philippe Mathieu-Daudé
Message-Id: <20240717105723.58965-4-phi...@linaro.org>
Signed-off-by: Alex Bennée
---
target/mips/tcg/sysemu/semihosting-stub.
This brings in the latest python mappings for the BSD updates.
Signed-off-by: Alex Bennée
---
.gitlab-ci.d/cirrus/freebsd-13.vars | 2 +-
tests/lcitool/libvirt-ci| 2 +-
tests/vm/generated/freebsd.json | 14 +++---
3 files changed, 9 insertions(+), 9 deletions(-)
diff
complications of transferring ownership of
arrays and keeps the final command entries as static entries in the
target code.
Cc: Akihiko Odaki
Cc: Gustavo Bueno Romero
Cc: Peter Maydell
Signed-off-by: Alex Bennée
Reviewed-by: Gustavo Romero
---
v2
- don't use strrstr, instead maintain a strv
Chigot
Acked-by: Alex Bennée
Message-Id: <20240710111755.60584-1-th...@redhat.com>
[AJB: fix MAINTAINERS]
Signed-off-by: Alex Bennée
---
MAINTAINERS | 1 -
tests/avocado/machine_sparc_leon3.py | 37
2 files changed, 38 deletions(-)
delet
Alex Bennée writes:
> There is some confusion about when you should use one over the other.
> Lets try and address that by adding some kdoc comments.
>
> Suggested-by: Paolo Bonzini
> Signed-off-by: Alex Bennée
ping?
> ---
> include/hw/core/cpu.h | 19
nted for other accelerators, but
> work need to be done. Meanwhile, do not enable it
> unless TCG is available.
Queued to semihosting/next, thanks.
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
Frédéric Pétrot writes:
> Register values are dumped as 'sz' chunks of two nibbles in the execlog
> plugin, sz was 1 too big.
>
> Signed-off-by: Frédéric Pétrot
>
Queued to plugins/next, thanks.
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
Richard Henderson writes:
> On 7/17/24 02:55, Alex Bennée wrote:
>>> Are you expecting the same GdbCmdParseEntry object to be registered
>>> multiple times? Can we fix that at a higher level?
>> Its basically a hack to deal with the fact everything is tied to the
Richard Henderson writes:
> On 7/16/24 21:42, Alex Bennée wrote:
>> void gdb_extend_qsupported_features(char *qsupported_features)
>> {
>> -/*
>> - * We don't support different sets of CPU gdb features on different
>> CPUs yet
>> - * so
armcpu = ARM_CPU(cpuobj);
pa_bits = arm_pamax(armcpu);
object_unref(cpuobj);
Anyway:
Tested-by: Alex Bennée
> ---
> include/hw/core/cpu.h | 6 ++
> cpu-common.c | 11 +++
> hw/core/cpu-common.c | 1 +
> 3 files changed, 18 insertions(+)
to trigger mem access callbacks for atomic
> instructions.
>
> Reviewed-by: Xingtao Yao
> Reviewed-by: Richard Henderson
> Signed-off-by: Pierrick Bouvier
I'm queuing this patch to plugins/next as it is a fix.
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
Gustavo Romero writes:
> Hi Alex,
>
> On 7/16/24 8:42 AM, Alex Bennée wrote:
>> Coverity reported a memory leak (CID 1549757) in this code and its
>> admittedly rather clumsy handling of extending the command table.
>> Instead of handing over a full array of the comma
melin
> Signed-off-by: Alexandre Iooss
Queued to plugins/next, thanks.
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
complications of transferring ownership of
arrays and keeps the final command entries as static entries in the
target code.
Cc: Akihiko Odaki
Cc: Gustavo Bueno Romero
Cc: Peter Maydell
Signed-off-by: Alex Bennée
---
include/gdbstub/commands.h | 19 --
target/arm/internals.h | 4
GArray and its contents").
>
> Also, there is a second leak here if you have more than one
> CPU -- when the second CPU calls gdb_extend_query_table() etc,
> the function will leak the first CPU's data. Having the function
> API be clearly either "always takes ownership" or "never takes
> ownership" would make it easier to fix this leak too.
I'm working on cleaning this API up to make it easier to use. I'll send
a patch once its tested.
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
Richard Henderson writes:
> From: Daniyal Khan
>
> Signed-off-by: Daniyal Khan
> Message-Id: 172090222034.13953.1688870870882292209...@git.sr.ht
> [rth: Split test cases to separate patch, tidy assembly.]
> Signed-off-by: Richard Henderson
Reviewed-by: Alex Bennée
n
Reviewed-by: Alex Bennée
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
nt FMOPA, FMOPS (non-widening)")
> Signed-off-by: Daniyal Khan
> [rth: Split from a larger patch]
> Signed-off-by: Richard Henderson
Reviewed-by: Alex Bennée
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
a-cflags=-fno-sanitize=function
> TARGETS: alpha-softmmu arm-softmmu m68k-softmmu mips64-softmmu
> s390x-softmmu
> MAKE_CHECK_ARGS: check-qtest check-tcg
>
>
> ---
> base-commit: f2cb4026fccfe073f84a4b440e41d3ed0c3134f6
> change-id: 20240714-function-7d32c723abbc
>
> Best regards,
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
ore calling it.
>
> Signed-off-by: Peter Maydell
Reviewed-by: Alex Bennée
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
Pierrick Bouvier writes:
> On 7/8/24 12:15, Alex Bennée wrote:
>> Pierrick Bouvier writes:
>>
>>> Add an explicit test to check expected memory values are read/written.
>>> For sizes 8, 16, 32, 64 and 128, we generate a load/store operation.
>>&g
gt; few more indirectly used, but worst case we just clone the bits we need
> into the QEMU tree.
Is Avocado still actively developed? I thought you guys used it quite
widely within RedHat?
>
> With regards,
> Daniel
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
Manos Pitsidianakis writes:
> Hi Daniel, Alex,
>
> I will pick this patch up with all the reviewed-by trailers for my
> next Rust RFC series if that's alright with you,
Absolutely - I made it for you ;-)
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
e cachable?
>
> (1) Asset download should not count against test timeout.
> (2) Running tests while disconnected should skip unavailable assets.
>
> Avocado kinda does this, but still generates errors instead of skips.
>
>
> r~
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
omas Huth
Queued to testing/next, thanks.
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
bsystem in QEMU and we
> are very happy to welcome him to the team. His knowledge and expertise
> with QEMU internals will be of great help.
>
> Signed-off-by: Stefano Stabellini
Reviewed-by: Alex Bennée
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
here its execution flow went.
> +
> +Two types of triggers can be configured: a count of instructions to stop at,
> +or an address to stop at. Multiple triggers can be set at once.
> +
> +By default, QEMU will exit with return code 0. A custom return code can be
> +configured for each trigger using ``:CODE`` syntax.
> +
> +For example, to stop at the 20-th instruction with return code 41, at address
> +0xd4 with return code 0 or at address 0xd8 with return code 42::
> +
> + $ qemu-system-aarch64 $(QEMU_ARGS) \
> +-plugin
> ./contrib/plugins/libstoptrigger.so,icount=20:41,addr=0xd4,addr=0xd8:42 -d
> plugin
> +
> +The plugin will log the reason of exit, for example::
> +
> + 0xd4 reached, exiting
> +
> Plugin API
> ==
Otherwise it looks good to me. Unless you want to tackle additional exit
modes?
What is your current use case for this?
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
For rust development we need cargo, rustc and bindgen in our various
development environments. Update the libvirt-ci project to (!495) and
regenerate the containers and other dependency lists.
Signed-off-by: Alex Bennée
---
NB:
- this is currently waiting on the upstream MR, but if you
homas Huth
Acked-by: Alex Bennée
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
ecution flow went.
> +
> +Two types of triggers can be configured: a count of instructions to stop at,
> +or an address to stop at. Multiple triggers can be set at once.
> +
> +By default, QEMU will exit with return code 0. A custom return code can be
> +configured for each trigger using ``:CODE`` syntax.
So I can see this being useful for general testing of "did the code get
to this point". However would it be worth considering other cases like
simply stopping the VM or triggering a gdbstub exception?
> +
> +For example, to stop at the 20-th instruction with return code 41, at address
> +0xd4 with return code 0 or at address 0xd8 with return code 42::
> +
> + $ qemu-system-aarch64 $(QEMU_ARGS) \
> +-plugin
> ./contrib/plugins/libstoptrigger.so,icount=20:41,addr=0xd4,addr=0xd8:42 -d
> plugin
> +
> +The plugin will log the reason of exit, for example::
> +
> + 0xd4 reached, exiting
> +
> Plugin API
> ==
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
Generating bindings for Rust
rustmod-bindgen-aarch64-softmmu_wrapper.h
Failed to run rustfmt: cannot find binary path (non-fatal, continuing)
[3041/3041] Linking target tests/qtest/qos-test
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
Manos Pitsidianakis writes:
> Set rust source code to diff=rust (built-in with new git versions)
> and merge=binary for Cargo.lock files (they should not be merged but
> auto-generated by cargo)
>
> Signed-off-by: Manos Pitsidianakis
Reviewed-by: Alex Bennée
--
Alex Bennée
Alex Bennée writes:
> Manos Pitsidianakis writes:
>
>> Add mechanism to generate rust hw targets that depend on a custom
>> bindgen target for rust bindings to C.
>>
>> This way bindings will be created before the rust crate is compiled.
>>
>> Th
m we have.
I was going to say Bookworm has now superseded Bullseye which will reach
its release + 3 year support point in August. However the version you
mention in the Bookworm one!
>
> Paolo
>
>> At that point we have more information to figure out what
>> if any tradeoff we want to make.
>>
>> thanks
>> -- PMM
>>
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
ing edge (it even tripped up on the
.cargo/bin/cargo I have installed). This needs to be set to the
baseline which from:
https://wiki.qemu.org/RustInQemu/2022
Looks to be 1.24.0 for rustc and I guess even lower for cargo (Debian
says 0.66.0). While it might make sense to delay merging if we are
waiting for one distro to produce a new LTS we shouldn't be needing
rustup by default.
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
8,load,0x42
> +,atomic_op_u8,.*,8,store,0xf1
> +,load_u8,.*,8,load,0xf1
> +,store_u16,.*,16,store,0xf123
> +,atomic_op_u16,.*,16,load,0x0042
> +,atomic_op_u16,.*,16,store,0xf123
> +,load_u16,.*,16,load,0xf123
> +,store_u32,.*,32,store,0xff112233
> +,atomic_op_u32,.*,32,load,0x0042
> +,atomic_op_u32,.*,32,store,0xff112233
> +,load_u32,.*,32,load,0xff112233
> +,store_u64,.*,64,store,0xf123456789abcdef
> +,atomic_op_u64,.*,64,load,0x0042
> +,atomic_op_u64,.*,64,store,0xf123456789abcdef
> +,load_u64,.*,64,load,0xf123456789abcdef
> +,store_u128,.*,128,store,0xf122334455667788f123456789abcdef
> +,load_u128,.*,128,load,0xf122334455667788f123456789abcdef
> +EOF
> +}
> +
> +expected | while read line; do
> +check "$plugin_out" "$line"
> +done
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
Manos Pitsidianakis writes:
> Hello Alex, I thought It was tomorrow? QEMU Project Calendar says "
> Tuesday, July 9⋅4:00 – 5:00pm
> Every 2 weeks on Tuesday
> "
Sorry yes - I really should script my invite rather than copy/paste edit
each time.
>
> On Mon, 8 Ju
tract-plugin,$@) output \
+ with $(call strip-plugin,$<)))
--8<---cut here---end--->8---
> endif
>
> gdb-%: %
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
BEGIN:VCALENDAR
PRODID:-//Google Inc//Google Calendar 70.9054//EN
VERSION:2.0
CALSCALE:GREGORIAN
METHOD:REQUEST
BEGIN:VTIMEZONE
TZID:America/New_York
X-LIC-LOCATION:America/New_York
BEGIN:DAYLIGHT
TZOFFSETFROM:-0500
TZOFFSETTO:-0400
TZNAME:EDT
DTSTART:19700308T02
BEGIN:VCALENDAR
PRODID:-//Google Inc//Google Calendar 70.9054//EN
VERSION:2.0
CALSCALE:GREGORIAN
METHOD:REQUEST
BEGIN:VTIMEZONE
TZID:America/New_York
X-LIC-LOCATION:America/New_York
BEGIN:DAYLIGHT
TZOFFSETFROM:-0500
TZOFFSETTO:-0400
TZNAME:EDT
DTSTART:19700308T02
Hi,
The KVM/QEMU community call is at:
https://meet.jit.si/kvmcallmeeting
@
8/7/2024 14:00 UTC
Are there any agenda items for the sync-up?
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
.c
and aside from the inline asm I don't see why this couldn't be a
multi-arch test. Could we not use the atomic primitives to make it multiarch?
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
ct/qemu/-/issues/1719
> Resolves: https://gitlab.com/qemu-project/qemu/-/issues/2152
> Reviewed-by: Richard Henderson
> Reviewed-by: Xingtao Yao
> Signed-off-by: Pierrick Bouvier
Reviewed-by: Alex Bennée
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
will support wider operations, we'll be able to
> reconsider this.
>
> Reviewed-by: Richard Henderson
> Signed-off-by: Pierrick Bouvier
Reviewed-by: Alex Bennée
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
to trigger mem access callbacks for atomic
> instructions.
>
> Reviewed-by: Xingtao Yao
> Reviewed-by: Richard Henderson
> Signed-off-by: Pierrick Bouvier
Reviewed-by: Alex Bennée
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
"Edgar E. Iglesias" writes:
> On Thu, Jul 4, 2024 at 9:48 PM Edgar E. Iglesias
> wrote:
>
> On Thu, Jul 04, 2024 at 05:44:52PM +0100, Alex Bennée wrote:
> > Anthony PERARD writes:
> >
> > > On Tue, Jul 02, 2024 at 12:44:21AM +0200, Edga
In fact any other accelerator would be pointless as the point is to
exercise the TCI accelerator anyway.
Reviewed-by: Thomas Huth
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-21-alex.ben...@linaro.org>
diff --git a/.gitlab-ci.d/buildtest.yml b/.gitlab-ci.d/buildtest.yml
to gdb_get_cmd_param.
This commit also makes gdb_put_packet public since is used in gdbstub
command handling.
Signed-off-by: Gustavo Romero
Reviewed-by: Alex Bennée
Message-Id: <20240628050850.536447-3-gustavo.rom...@linaro.org>
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-32
From: Gustavo Romero
If page in 'ptr_access' is inaccessible and probe is 'true'
allocation_tag_mem_probe should not throw an exception, but currently it
does, so fix it.
Signed-off-by: Gustavo Romero
Reviewed-by: Alex Bennée
Reviewed-by: Richard Henderson
Message-Id: <20240628050850.536
While the match functionality is useful lets make the verbosity
optional while we are actually running.
Reviewed-by: Manos Pitsidianakis
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-22-alex.ben...@linaro.org>
diff --git a/tests/plugin/insn.c b/tests/plugin/insn.c
From: Gustavo Romero
cmd_startswith is a boolean so use 'true' to set it instead of 1.
Signed-off-by: Gustavo Romero
Message-Id: <20240628050850.536447-10-gustavo.rom...@linaro.org>
Reviewed-by: Manos Pitsidianakis
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-39
We were repeating information which wasn't super clear. As we already
will have dumped the last failing PC just note the divergence and dump
the previous instruction log.
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-27-alex.ben...@linaro.org>
diff --git a/contrib/p
We are interested in the particular instruction so we should use a
stable record for it. We could bring this down to physical address but
for now vaddr + disas seems to do the trick.
Reviewed-by: Manos Pitsidianakis
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-23-alex.
org>
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-29-alex.ben...@linaro.org>
diff --git a/include/qemu/plugin.h b/include/qemu/plugin.h
index bc5aef979e..af5f9db469 100644
--- a/include/qemu/plugin.h
+++ b/include/qemu/plugin.h
@@ -149,6 +149,9 @@ struct CP
wed-by: Akihiko Odaki
Message-Id: <20240630190050.160642-15-richard.hender...@linaro.org>
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-20-alex.ben...@linaro.org>
diff --git a/linux-user/main.c b/linux-user/main.c
index 94c99a1366..7d3cf45fa9 100644
--- a/linux-user/main.c
+
functions as interfaces
to extend the qSupported string, the query handler table, and the set
handler table, allowing target-specific stub implementations.
Signed-off-by: Gustavo Romero
Reviewed-by: Alex Bennée
Message-Id: <20240628050850.536447-4-gustavo.rom...@linaro.org>
Signed-off-by
From: Gustavo Romero
Change 'process_string_cmd' to return true on success and false on
failure, instead of 0 and -1.
Signed-off-by: Gustavo Romero
Reviewed-by: Alex Bennée
Reviewed-by: Richard Henderson
Reviewed-by: Philippe Mathieu-Daudé
Message-Id: <20240628050850.536447-2-gustavo.
The ExecState is shared across the socket and if we want to compare
say 64 bit and 32 bit binaries we need the two to use the same sizes
for things.
Reviewed-by: Richard Henderson
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-25-alex.ben...@linaro.org>
diff --git a/c
We can't assign sock_path directly from the autofree'd GStrv, take a
copy.
Reviewed-by: Manos Pitsidianakis
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-24-alex.ben...@linaro.org>
diff --git a/contrib/plugins/lockstep.c b/contrib/plugins/lockstep.c
index 237543b43a..67a7
From: Gustavo Romero
Make gdb_hextomem non-internal so it's not confined to use only in
gdbstub.c.
Signed-off-by: Gustavo Romero
Reviewed-by: Richard Henderson
Message-Id: <20240628050850.536447-8-gustavo.rom...@linaro.org>
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.
', used to control the MTE fault type at runtime.
Signed-off-by: Gustavo Romero
Message-Id: <20240628050850.536447-11-gustavo.rom...@linaro.org>
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-40-alex.ben...@linaro.org>
diff --git a/configs/targets/aarch64-linux-user.ma
mnemonic is clearer.
Signed-off-by: Richard Henderson
Reviewed-by: Akihiko Odaki
Message-Id: <20240630190050.160642-14-richard.hender...@linaro.org>
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-19-alex.ben...@linaro.org>
diff --git a/tests/tcg/arm/fcvt.c b/tests
This really helps with lockstep although its super slow on big jobs.
Reviewed-by: Manos Pitsidianakis
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-26-alex.ben...@linaro.org>
diff --git a/contrib/plugins/lockstep.c b/contrib/plugins/lockstep.c
index 8b90b37f67..1765
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-35-alex.ben...@linaro.org>
diff --git a/target/arm/tcg/mte_helper.h b/target/arm/tcg/mte_helper.h
new file mode 100644
index 00..1f471fb69b
--- /dev/null
+++ b/target/arm/tcg/mte_helper.h
@@ -0,0 +1,66 @@
+/*
+ * ARM MemT
From: Philippe Mathieu-Daudé
Since vCPUs are hashed by their index, this index can't
be uninitialized (UNASSIGNED_CPU_INDEX).
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Pierrick Bouvier
Message-Id: <20240606124010.2460-2-phi...@linaro.org>
Signed-off-by: Alex Bennée
Mess
phi...@linaro.org>
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-30-alex.ben...@linaro.org>
diff --git a/hw/core/cpu-common.c b/hw/core/cpu-common.c
index 8f6cb64da3..b19e1fdacf 100644
--- a/hw/core/cpu-common.c
+++ b/hw/core/cpu-common.c
@@ -192,13 +192,6 @@
org>
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-41-alex.ben...@linaro.org>
diff --git a/configure b/configure
index 8b6a2f16ce..019fcbd0ef 100755
--- a/configure
+++ b/configure
@@ -1673,6 +1673,10 @@ for target in $target_list; do
echo "GDB=$gdb_bin"
.
Replace -u $(UID) with --userns keep-id, which fixes the UID remapping.
This change is limited to Podman because Docker does not support
--userns keep-id.
Signed-off-by: Akihiko Odaki
Message-Id: <20240626-podman-v1-1-f8c8daf2b...@daynix.com>
Signed-off-by: Alex Bennée
Mess
From: Richard Henderson
Clang requires the architecture to be set properly
in order to assemble the half-precision instructions.
Signed-off-by: Richard Henderson
Reviewed-by: Akihiko Odaki
Message-Id: <20240630190050.160642-13-richard.hender...@linaro.org>
Signed-off-by: Alex Bennée
M
I guess we never noticed and tried to build with this cross image. Fix
the toolchain prefix so we actually build 32 bit images.
Reviewed-by: Richard Henderson
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-2-alex.ben...@linaro.org>
diff --git a/tests/docker/dockerfiles/
133...@daynix.com>
Reviewed-by: Akihiko Odaki
Message-Id: <20240630190050.160642-5-richard.hender...@linaro.org>
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-10-alex.ben...@linaro.org>
diff --git a/tests/tcg/aarch64/bti-1.c b/tests/tcg/aarch64/bti-1.c
index 99a8
;
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-12-alex.ben...@linaro.org>
diff --git a/tests/tcg/arm/fcvt.c b/tests/tcg/arm/fcvt.c
index 7ac47b564e..f631197287 100644
--- a/tests/tcg/arm/fcvt.c
+++ b/tests/tcg/arm/fcvt.c
@@ -126,7 +126,7 @@ static void convert_single_to_half(
.160642-6-richard.hender...@linaro.org>
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-11-alex.ben...@linaro.org>
diff --git a/tests/tcg/aarch64/mte-1.c b/tests/tcg/aarch64/mte-1.c
index 88dcd617ad..146cad4a04 100644
--- a/tests/tcg/aarch64/mte-1.c
+++ b/tests/tcg/aarch64/mte-1.c
@@
fcvt test cases for AArch32/64")
Message-Id: <20240627-tcg-v2-1-1690a8133...@daynix.com>
[rth: Update arm ref file as well]
Signed-off-by: Richard Henderson
Reviewed-by: Akihiko Odaki
Message-Id: <20240630190050.160642-9-richard.hender...@linaro.org>
Signed-off-by: Alex Bennée
Me
in the way and its TCG issues we want to
catch I've added --disable-kvm to the build.
Reported-by: Richard Henderson
Suggested-by: Thomas Huth
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-3-alex.ben...@linaro.org>
diff --git a/.gitlab-ci.d/crossbuilds.yml b/.gitla
across the
board.
Signed-off-by: Gustavo Romero
Message-Id: <20240628050850.536447-7-gustavo.rom...@linaro.org>
[AJB: clean-up includes, move MTE defines]
Reviewed-by: Manos Pitsidianakis
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-36-alex.ben...@linaro.org>
diff -
From: Richard Henderson
Clang does not support IWMXT instructions.
Fall back to the external assembler.
Signed-off-by: Richard Henderson
Reviewed-by: Akihiko Odaki
Message-Id: <20240630190050.160642-11-richard.hender...@linaro.org>
Signed-off-by: Alex Bennée
Mess
2-8-richard.hender...@linaro.org>
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-13-alex.ben...@linaro.org>
diff --git a/tests/tcg/aarch64/Makefile.target
b/tests/tcg/aarch64/Makefile.target
index 11ccde5579..ad1774c2ce 100644
--- a/tests/tcg/aarch64/Makefile.target
+++
e x constraint
tests/tcg/arm: Fix fcvt result messages
tests/tcg/arm: Manually register allocate half-precision numbers
Alex Bennée (11):
tests/lcitool: fix debian-i686-cross toolchain prefix
testing: restore some testing for i686
tracepoints: move physmem trace poi
From: Gustavo Romero
Allow passing the current CPU context to command handlers via user_ctx
when the handler requires it.
Signed-off-by: Gustavo Romero
Message-Id: <20240628050850.536447-9-gustavo.rom...@linaro.org>
Reviewed-by: Manos Pitsidianakis
Signed-off-by: Alex Bennée
Mess
From: Richard Henderson
This option is not supported by clang, and is not required
in order to get sve code generation with gcc 12.
Signed-off-by: Richard Henderson
Reviewed-by: Akihiko Odaki
Message-Id: <20240630190050.160642-4-richard.hender...@linaro.org>
Signed-off-by: Alex
They don't need to be in the global trace-events file and can have a
local trace header. Also add address_space_map tracepoint for tracking
mapping behaviour.
Reviewed-by: Richard Henderson
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-4-alex.ben...@linaro.org>
diff
))
| ^
../../hw/core/loader-fit.c:270:12: note: ‘kernel_end’ was declared here
270 | hwaddr kernel_end;
|^~
Reviewed-by: Manos Pitsidianakis
Signed-off-by: Alex Bennée
Reviewed-by: Aleksandar Rikalo
Message-Id
From: Richard Henderson
Define the variable to the compiler flag used, not "y".
This avoids replication of the compiler flag itself.
Signed-off-by: Richard Henderson
Reviewed-by: Akihiko Odaki
Message-Id: <20240630190050.160642-3-richard.hender...@linaro.org>
Signed-off
Message-Id: <20240630190050.160642-2-richard.hender...@linaro.org>
Signed-off-by: Alex Bennée
Message-Id: <20240705084047.857176-7-alex.ben...@linaro.org>
diff --git a/tests/tcg/minilib/printf.c b/tests/tcg/minilib/printf.c
index 10472b4f58..fb0189c2bb 100644
--- a/tests/tcg/minilib/pr
1 - 100 of 18729 matches
Mail list logo