'd', 'e', 'x', and 'y'. No longer following that makes us select
the wrong register index and the HVX store new instruction does not
update the memory like expected.
Signed-off-by: Matheus Tavares Bernardino
---
tests/tcg/hexagon/hvx_misc.c | 23 +++
target/hexagon/gen_t
The Hexagon Programmer's Reference Manual says that the exception 0x1e
should be raised upon an unaligned program counter. Let's implement that
and also add some tests.
Signed-off-by: Matheus Tavares Bernardino
Reviewed-by: Richard Henderson
Reviewed-by: Taylor Simpson
---
Changes in v6
On Thu, 2 May 2024 13:00:34 -0700 Richard Henderson
wrote:
>
> On 5/2/24 12:20, Matheus Tavares Bernardino wrote:
> >
> > +
> > +void test_multi_cof(void)
> > +{
> > +asm volatile(
> > +"p0 = cmp.eq(r0, r0)\n"
> > +&qu
The Hexagon Programmer's Reference Manual says that the exception 0x1e
should be raised upon an unaligned program counter. Let's implement that
and also add some tests.
Signed-off-by: Matheus Tavares Bernardino
Reviewed-by: Richard Henderson
Reviewed-by: Taylor Simpson
---
Changes in v5
The Hexagon Programmer's Reference Manual says that the exception 0x1e
should be raised upon an unaligned program counter. Let's implement that
and also add some tests.
Signed-off-by: Matheus Tavares Bernardino
Reviewed-by: Richard Henderson
---
v3:
https://lore.kernel.org/qemu-devel
On Tue, 30 Apr 2024 08:52:36 -0700 Richard Henderson
wrote:
>
> On 4/30/24 07:25, Matheus Tavares Bernardino wrote:
> > +void test_multi_cof(void)
> > +{
> > +asm volatile(
> > +"p0 = cmp.eq(r0, r0)\n"
> > +"{\n"
&
The Hexagon Programmer's Reference Manual says that the exception 0x1e
should be raised upon an unaligned program counter. Let's implement that
and also add some tests.
Signed-off-by: Matheus Tavares Bernardino
---
v2:
https://lore.kernel.org/qemu-devel
The Hexagon Programmer's Reference Manual says that the exception 0x1e
should be raised upon an unaligned program counter. Let's implement that
and also add tests for both the most common case as well as packets with
multiple change-of-flow instructions.
Signed-off-by: Matheus Tavares Bernardino
s
> complete.
Will do, thanks.
> > -Original Message-
> > From: ltaylorsimp...@gmail.com
> >
> >
> >
> > > -Original Message-
> > > From: Matheus Tavares Bernardino
> > >
> > >
> > > --- a/target/
The Hexagon Programmer's Reference Manual says that the exception 0x1e
should be raised upon an unaligned program counter. Let's implement that
and also add tests for both the most common case as well as packets with
multiple change-of-flow instructions.
Signed-off-by: Matheus Tavares Bernardino
On Fri, 12 Apr 2024 10:02:54 +0200 Paolo Bonzini wrote:
>
> On Thu, Apr 11, 2024 at 5:46 PM Matheus Tavares Bernardino
> wrote:
> > +$(if $(filter -j, $(MAKEFLAGS)) \
> > +,, \
> > +$(or \
> > +
Hi, Philippe
On Thu, 11 Apr 2024 17:29:58 +0200 =?UTF-8?Q?Philippe_Mathieu-Daud=C3=A9?=
wrote:
>
> Hi Matheus,
>
> On 11/4/24 17:09, Matheus Tavares Bernardino wrote:
> > Our Makefile massages the given make arguments to invoke ninja
> > accordingly. One key
t's fix that and indent the touched code for better readability.
Signed-off-by: Matheus Tavares Bernardino
---
Makefile | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/Makefile b/Makefile
index 183756018f..d299c14dab 100644
--- a/Makefile
+++ b/Makefile
@@ -142,8 +142,1
On Mon, 26 Feb 2024 13:17:17 -0700 Taylor Simpson
wrote:
>
> diff --git a/target/hexagon/gen_trans_funcs.py
> b/target/hexagon/gen_trans_funcs.py
> index 07292e0170..f1972fd2dd 100755
> --- a/target/hexagon/gen_trans_funcs.py
> +++ b/target/hexagon/gen_trans_funcs.py
> @@ -86,6 +86,7 @@ def
On Mon, 26 Feb 2024 13:17:16 -0700 Taylor Simpson
wrote:
>
> diff --git a/target/hexagon/gen_trans_funcs.py
> b/target/hexagon/gen_trans_funcs.py
> index 79475b2946..07292e0170 100755
> --- a/target/hexagon/gen_trans_funcs.py
> +++ b/target/hexagon/gen_trans_funcs.py
> @@ -85,6 +85,7 @@ def
On Mon, 26 Feb 2024 13:17:15 -0700 Taylor Simpson
wrote:
>
> diff --git a/target/hexagon/gen_trans_funcs.py
> b/target/hexagon/gen_trans_funcs.py
> index 53e844a44b..79475b2946 100755
> --- a/target/hexagon/gen_trans_funcs.py
> +++ b/target/hexagon/gen_trans_funcs.py
> @@ -84,14 +84,15 @@ def
On Sun, 14 Jan 2024 15:23:54 -0800 Brian Cain wrote:
>
> This update includes support for privileged instructions.
>
> Signed-off-by: Brian Cain
> ---
Reviewed-by: Matheus Tavares Bernardino
Tested-by: Matheus Tavares Bernardino
> diff --git a/tests/docker/dockerfi
On Sun, 14 Jan 2024 15:44:53 -0800 Brian Cain wrote:
>
> This is intended to address a coverity finding: CID 1527408.
>
> Signed-off-by: Brian Cain
> ---
Reviewed-by: Matheus Tavares Bernardino
Taylor Simpson wrote:
>
> RFC - This patch handles gen_tcg_funcs.py. I'd like to get comments
> on the general approach before working on the other Python scripts.
>
> The generators are generally a bunch of Python if-then-else
> statements based on the regtype and regid. Encapsulate
[resending as it looks like there was some delivery issue with the first
msg]
Hi,
It seems that we lose the install prefix option when regenerating config
for a Windows cross-build. Looks like this behavior appeared with c36dd41ba2
(configure: move target-specific defaults to an external machine
Hi,
It seems that we lose the install prefix option when regenerating config
for a Windows cross-build. Looks like this behavior appeared with c36dd41ba2
(configure: move target-specific defaults to an external machine file,
2023-10-16), but I couldn't find the specific root cause yet.
To
We duplicate "cmd" as strtok may modify its argument, but we forgot
to free it later. Furthermore, add_semihosting_arg doesn't take
responsibility for this memory either (it strdup's the argument).
Signed-off-by: Matheus Tavares Bernardino
Reviewed-by: Philippe Mathieu-Daudé
---
s
We duplicate "cmd" as strtok may modify its argument, but we forgot
to free it later. Furthermore, add_semihosting_arg doesn't take
responsibility for this memory either (it strdup's the argument).
Signed-off-by: Matheus Tavares Bernardino
---
semihosting/config.c | 6 --
1 file
at virtio_add_resource() itself are not strictly necessary
for the memleak fix, but they make it more obvious that, on an error
return, the passed object is not added to the hash.
Signed-off-by: Matheus Tavares Bernardino
---
hw/display/virtio-dmabuf.c | 12 ++--
1 file changed, 10
Hi, Richard
Richard Henderson wrote:
>
> On 5/18/23 13:04, Taylor Simpson wrote:
> > From: Matheus Tavares Bernardino
> >
> > In the previous commit, we modified gdbstub.c to only send stop-reply
> > packets as a response to GDB commands that accept
Ilya Leoshkevich wrote:
>
> On Tue, 2023-08-01 at 12:37 -0300, Matheus Tavares Bernardino wrote:
> > Previously, qemu-user would always report PID 1 to GDB. This was
> > changed
> > at dc14a7a6e9 (gdbstub: Report the actual qemu-user pid, 2023-06-30),
> > but read_t
Hi, Nick.
> Nicholas Piggin wrote:
>
> On Tue Jul 11, 2023 at 9:03 PM AEST, Matheus Tavares Bernardino wrote:
> > > Nicholas Piggin wrote:
> > >
> > > diff --git a/gdbstub/gdbstub.c b/gdbstub/gdbstub.c
> > > index 6911b73c07..ce8b42eb15 100644
>
PID,
or PID 0.
This issue was found with hexagon-lldb, which sends a "Hq" packet with
only the thread-id, but no process-id, leading to the invalid usage of
"PID 1" by qemu-hexagon and a subsequent "E22" reply.
Signed-off-by: Matheus Tavares Bernardino
---
gdb
Matheus Tavares Bernardino wrote:
>
> Subject: [PATCH v3] Hexagon: move GETPC() calls to top level helpers
Apologies, I had some problems with my send-email and server
configuration, thus ended up sending this v3 multiple times. Please
ignore the others and consider only this one (i.e.
,
unifying the mem_load*() functions.
- HELPER(probe_hvx_stores), when called from another helper, ends up
using its own GETPC() expansion instead of the top level caller.
Signed-off-by: Matheus Tavares Bernardino
---
v2:
https://lore.kernel.org/qemu-devel/93a2ca786530cbc8a94f7c7a6451f4f1f47c8a9b
,
unifying the mem_load*() functions.
Signed-off-by: Matheus Tavares Bernardino
Reviewed-by: Taylor Simpson
---
v2:
https://lore.kernel.org/qemu-devel/93a2ca786530cbc8a94f7c7a6451f4f1f47c8a9b.1688581908.git.quic_mathb...@quicinc.com/
Changes in v3: refactored fLOAD macro with 'do {...} while(0
,
unifying the mem_load*() functions.
- HELPER(probe_hvx_stores), when called from another helper, ends up
using its own GETPC() expansion instead of the top level caller.
Signed-off-by: Matheus Tavares Bernardino
---
v2:
https://lore.kernel.org/qemu-devel/93a2ca786530cbc8a94f7c7a6451f4f1f47c8a9b
> Nicholas Piggin wrote:
>
> diff --git a/gdbstub/gdbstub.c b/gdbstub/gdbstub.c
> index 6911b73c07..ce8b42eb15 100644
> --- a/gdbstub/gdbstub.c
> +++ b/gdbstub/gdbstub.c
> @@ -2051,8 +2051,17 @@ void gdb_read_byte(uint8_t ch)
> return;
> }
> if (runstate_is_running()) {
> -
,
unifying the mem_load*() functions.
- HELPER(probe_hvx_stores), when called from another helper, ends up
using its own GETPC() expansion instead of the top level caller.
Signed-off-by: Matheus Tavares Bernardino
Reviewed-by: Taylor Simpson
---
v2:
https://lore.kernel.org/qemu-devel
> ltaylorsimp...@gmail.com wrote:
>
> > -Original Message-
> > From: Matheus Tavares Bernardino
> > Sent: Wednesday, July 5, 2023 12:35 PM
> > To: qemu-devel@nongnu.org
> > Cc: quic_mathb...@quicinc.com; bc...@quicinc.com;
> > ltaylors
,
unifying the mem_load*() functions.
Signed-off-by: Matheus Tavares Bernardino
---
v1:
d40fabcf9d6e92e4cd8d6a144e9b2a9acf4580dc.1688420966.git.quic_mathb...@quicinc.com
Changes in v2:
- Fixed wrong cpu_ld* unification from previous version.
- Passed retaddr down to check_noshuf() and further
> Taylor wrote:
>
> > Matheus Tavares Bernardino wrote:
> >
> > diff --git a/target/hexagon/macros.h b/target/hexagon/macros.h index
> > 5451b061ee..efb8013912 100644
> > --- a/target/hexagon/macros.h
> > +++ b/target/hexagon/macr
> Matheus Tavares Bernardino wrote:
>
> diff --git a/target/hexagon/macros.h b/target/hexagon/macros.h
> index 5451b061ee..efb8013912 100644
> --- a/target/hexagon/macros.h
> +++ b/target/hexagon/macros.h
> @@ -173,14 +173,20 @@
> #define MEM_STORE8(VA, DATA, SLOT) \
>
,
unifying the mem_load*() functions.
Signed-off-by: Matheus Tavares Bernardino
---
target/hexagon/macros.h| 22 ++---
target/hexagon/op_helper.h | 11 ++---
target/hexagon/op_helper.c | 49 +++---
3 files changed, 25 insertions(+), 57 deletions
Peter Maydell wrote:
>
> On Fri, 23 Jun 2023 at 13:19, Matheus Tavares Bernardino
> wrote:
> >
> > Ah, it's interesting that [1] doesn't include 'bc' and 'bs' in the list
> > of cmds that may respond with a stop-reply packet:
> >
> > "The '
'vStopped', and
'?' packets can receive any of the below as a reply."
But their definitions at [2] do say the following:
'bc' (and 'bc')
[...]
Reply: See Stop Reply Packets, for the reply specifications.
So I guess the list from [1] is not exhaustive. Anyway, thanks for the
fi
This fixes compiler messages like "warning: format specifies type
'unsigned int' but the argument has type 'uint32_t' (aka 'unsigned
long') [-Wformat]".
Signed-off-by: Matheus Tavares Bernardino
---
tests/tcg/hexagon/hex_test.h | 14 ++
1 file changed, 10 insertions(+), 4
This series includes two fixes on hexagon test files: one for a
non-porable printf specifier, and the other for the use of an
uninitialized register.
Marco Liebel (1):
Hexagon (hvx_misc test): fix uninitialized regs at test_load_tmp2
Matheus Tavares Bernardino (1):
Hexagon (tests
From: Marco Liebel
This test case was using some vector registers which were not properly
initialized.
Signed-off-by: Marco Liebel
Signed-off-by: Matheus Tavares Bernardino
---
tests/tcg/hexagon/hvx_misc.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/tests/tcg
-by: Matheus Tavares Bernardino
---
target/hexagon/gen_analyze_funcs.py | 10 +++---
target/hexagon/gen_helper_funcs.py | 30
target/hexagon/gen_helper_protos.py | 22 ++--
target/hexagon/gen_idef_parser_funcs.py | 4 +--
target/hexagon/gen_op_regs.py
: replaced outdated comments with pseudocode
v1:
https://lore.kernel.org/qemu-devel/cover.1684873957.git.quic_mathb...@quicinc.com/
Matheus Tavares Bernardino (2):
target/hexagon/*.py: clean up used 'toss' and 'numregs' vars
Hexagon: fix outdated `hex_new_*` comments
target/hexagon/genptr.c
name change in the future, let's
replace them with pseudocode.
Suggested-by: Taylor Simpson
Signed-off-by: Matheus Tavares Bernardino
---
target/hexagon/genptr.c| 26 --
target/hexagon/translate.c | 2 +-
2 files changed, 13 insertions(+), 15 deletions(-)
diff --git
Some calls to `hex_common.bad_register()` in Hexagon python files are
passing undefined variables. Let's remove those.
Signed-off-by: Matheus Tavares Bernardino
---
target/hexagon/gen_helper_funcs.py | 8
target/hexagon/gen_tcg_funcs.py| 6 +++---
2 files changed, 7 insertions
to assist developers, we can replace the old names with some
pseudocode when convenient.
Signed-off-by: Matheus Tavares Bernardino
---
target/hexagon/genptr.c| 10 +-
target/hexagon/translate.c | 2 +-
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/target/hexagon/genptr.c b
These are two minor follow ups to the last Hexagon pull, updating some
stale code comments and removing undefined variables from error messages
at python scripts.
Matheus Tavares Bernardino (2):
target/hexagon/*.py: remove undef vars from bad_register()
Hexagon: fix outdated `hex_new_
r to debug possible future errors caused
by missing updates to `find_iclass_slots()` rules in
target/hexagon/iclass.c.
Co-authored-by: Taylor Simpson
Signed-off-by: Taylor Simpson
Signed-off-by: Matheus Tavares Bernardino
---
Changes in v2:
- Only call decode_set_slot_number() with !disas_only, fix
r to debug possible future errors caused
by missing updates to `find_iclass_slots()` rules in
target/hexagon/iclass.c.
Signed-off-by: Matheus Tavares Bernardino
Reviewed-by: Taylor Simpson
---
I extracted this patch from [1]. There are more changes needed to
support v73 in qemu, so the f
r did it read but does not know such version?").
And running `qemu-hexagon -cpu help` doesn't help either, as it just
errors out with no output. This patchset tries to improve this process.
https://lore.kernel.org/qemu-devel/cover.1673616964.git.quic_mathb...@quicinc.com/
Matheus Tavares Ber
CPU model 'unknown (0x69)'
Signed-off-by: Matheus Tavares Bernardino
---
linux-user/hexagon/target_elf.h | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/linux-user/hexagon/target_elf.h b/linux-user/hexagon/target_elf.h
index b4e9f40527..f47e130537 100644
--- a/linux-user
e printing. This is to avoid confusing
situations like the following:
$ qemu-hexagon -cpu help
Available CPUs:
v67-hexagon-cpu
$ qemu-hexagon -cpu v67-hexagon-cpu ./prog
qemu-hexagon: unable to find CPU model 'v67-hexagon-cpu'
Signed-off-by: Matheus Tavares Bernardino
-
.
This patch was generated with:
git grep -l "Bad register" *hexagon* | \
xargs sed -i "" -e 's/print("Bad register parse: "[,
]*\([^)]*\))/hex_common.bad_register(\1)/g'
Plus the bad_register() helper added to hex_common.py.
Signed-off-by: Matheus Tavares B
This enables LLDB to work with hexagon linux-user mode through the GDB
remote protocol.
Helped-by: Richard Henderson
Signed-off-by: Matheus Tavares Bernardino
---
linux-user/hexagon/cpu_loop.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/linux-user/hexagon/cpu_loop.c b/linux-user
Signed-off-by: Matheus Tavares Bernardino
---
MAINTAINERS| 1 +
configs/targets/hexagon-linux-user.mak | 1 +
target/hexagon/cpu.c | 3 +-
gdb-xml/hexagon-core.xml | 84 ++
4 files changed, 88 insertions(+), 1
From: Brian Cain
Signed-off-by: Brian Cain
Co-authored-by: Sid Manning
Signed-off-by: Sid Manning
Co-authored-by: Matheus Tavares Bernardino
Signed-off-by: Matheus Tavares Bernardino
Reviewed-by: Taylor Simpson
---
target/hexagon/gdbstub.c | 16
1 file changed, 16
From: Taylor Simpson
Signed-off-by: Taylor Simpson
Co-authored-by: Brian Cain
Signed-off-by: Brian Cain
Co-authored-by: Matheus Tavares Bernardino
Signed-off-by: Matheus Tavares Bernardino
---
configs/targets/hexagon-linux-user.mak | 2 +-
target/hexagon/internal.h | 2
was in fact waiting an ACK.
Signed-off-by: Matheus Tavares Bernardino
Acked-by: Alex Bennée
---
tests/guest-debug/run-test.py| 16
.../tcg/multiarch/system/Makefile.softmmu-target | 16 +++-
2 files changed, 27 insertions(+), 5 deletions(-)
diff
ic_mathb...@quicinc.com/
v1:
https://lore.kernel.org/qemu-devel/cover.1680808943.git.quic_mathb...@quicinc.com/
Brian Cain (1):
Hexagon (gdbstub): fix p3:0 read and write via stub
Matheus Tavares Bernardino (4):
gdbstub: only send stop-reply packets when allowed to
gdbstub: add test for untim
-lldb. Let's
change the gdbstub to send stop messages only as a response to a
previous GDB command that accepts such a reply.
Signed-off-by: Matheus Tavares Bernardino
Acked-by: Alex Bennée
---
gdbstub/internals.h | 5 +
gdbstub/gdbstub.c | 37 -
gdbstub
Philippe Mathieu-Daudé wrote:
>
> > Matheus Tavares wrote:
> >
> > diff --git a/target/hexagon/gdb_qreginfo.h b/target/hexagon/gdb_qreginfo.h
> > new file mode 100644
> > index 00..64631ddd58
> > --- /dev/null
> > +++ b/target/hexagon/gdb_qreginfo.h
>
> This should be
Alex Bennée wrote:
>
> > Matheus Tavares wrote:
> >
> > diff --git a/gdbstub/gdbstub.c b/gdbstub/gdbstub.c
> > index be18568d0a..f19f8c58c3 100644
> > --- a/gdbstub/gdbstub.c
> > +++ b/gdbstub/gdbstub.c
> > @@ -1578,6 +1599,12 @@ static const GdbCmdParseEntry gdb_gen_query_table[]
> > = {
> >
Alex Bennée wrote:
>
> > Matheus Tavares wrote:
> >
> > diff --git a/gdbstub/user.c b/gdbstub/user.c
> > index 80488b6bb9..bb03622c83 100644
> > --- a/gdbstub/user.c
> > +++ b/gdbstub/user.c
> > @@ -174,12 +177,14 @@ void gdb_signalled(CPUArchState *env, int sig)
> > {
> > char buf[4];
> >
From: Brian Cain
Co-authored-by: Sid Manning
Signed-off-by: Sid Manning
Signed-off-by: Brian Cain
Co-authored-by: Matheus Tavares Bernardino
Signed-off-by: Matheus Tavares Bernardino
Reviewed-by: Taylor Simpson
---
target/hexagon/gdbstub.c | 16
1 file changed, 16
was in fact waiting an ACK.
Signed-off-by: Matheus Tavares Bernardino
---
tests/guest-debug/run-test.py| 16
.../tcg/multiarch/system/Makefile.softmmu-target | 16 +++-
2 files changed, 27 insertions(+), 5 deletions(-)
diff --git a/tests/guest
From: Taylor Simpson
Co-authored-by: Brian Cain
Signed-off-by: Brian Cain
Signed-off-by: Taylor Simpson
Co-authored-by: Matheus Tavares Bernardino
Signed-off-by: Matheus Tavares Bernardino
---
target/hexagon/gdbstub.c | 60
1 file changed, 60
From: Brian Cain
Signed-off-by: Brian Cain
Signed-off-by: Matheus Tavares Bernardino
---
target/hexagon/gdb_qreginfo.h | 124 ++
target/hexagon/cpu.c | 3 +
2 files changed, 127 insertions(+)
create mode 100644 target/hexagon/gdb_qreginfo.h
diff
From: Brian Cain
Signed-off-by: Brian Cain
Signed-off-by: Matheus Tavares Bernardino
---
include/hw/core/cpu.h | 4
gdbstub/gdbstub.c | 27 +++
2 files changed, 31 insertions(+)
diff --git a/include/hw/core/cpu.h b/include/hw/core/cpu.h
index 397fd3ac68
-lldb. Let's
change the gdbstub to send stop messages only as a response to a
previous GDB command that accepts such a reply.
Signed-off-by: Matheus Tavares Bernardino
---
gdbstub/internals.h | 5 +
gdbstub/gdbstub.c | 37 -
gdbstub/softmmu.c | 13
From: Taylor Simpson
This enables LLDB to work with hexagon linux-user mode through the GDB
remote protocol.
Signed-off-by: Taylor Simpson
Signed-off-by: Matheus Tavares Bernardino
---
linux-user/hexagon/cpu_loop.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/linux-user
and write via stub
Matheus Tavares Bernardino (2):
gdbstub: only send stop-reply packets when allowed to
gdbstub: add test for untimely stop-reply packets
Taylor Simpson (2):
Hexagon (gdbstub): add HVX support
Hexagon (linux-user/hexagon): handle breakpoints
gdbstub/internals.h
From: Taylor Simpson
This enables LLDB to work with hexagon linux-user mode through the GDB
remote protocol.
Signed-off-by: Taylor Simpson
Signed-off-by: Matheus Tavares Bernardino
---
linux-user/hexagon/cpu_loop.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/linux-user
From: Brian Cain
Signed-off-by: Brian Cain
Signed-off-by: Matheus Tavares Bernardino
---
target/hexagon/gdb_qreginfo.h | 124 ++
target/hexagon/cpu.c | 3 +
2 files changed, 127 insertions(+)
create mode 100644 target/hexagon/gdb_qreginfo.h
diff
From: Taylor Simpson
Co-authored-by: Brian Cain
Signed-off-by: Brian Cain
Signed-off-by: Taylor Simpson
Co-authored-by: Matheus Tavares Bernardino
Signed-off-by: Matheus Tavares Bernardino
---
target/hexagon/gdbstub.c | 60
1 file changed, 60
-lldb. Let's
change the gdbstub to send stop messages only as a response to a
previous GDB command that accepts such a reply.
Signed-off-by: Matheus Tavares Bernardino
---
gdbstub/internals.h | 5 +
gdbstub/gdbstub.c | 37 -
gdbstub/softmmu.c | 13
From: Brian Cain
Co-authored-by: Sid Manning
Signed-off-by: Sid Manning
Signed-off-by: Brian Cain
Co-authored-by: Matheus Tavares Bernardino
Signed-off-by: Matheus Tavares Bernardino
---
target/hexagon/gdbstub.c | 16
1 file changed, 16 insertions(+)
diff --git a/target
From: Brian Cain
Signed-off-by: Brian Cain
Signed-off-by: Matheus Tavares Bernardino
---
include/hw/core/cpu.h | 4
gdbstub/gdbstub.c | 27 +++
2 files changed, 31 insertions(+)
diff --git a/include/hw/core/cpu.h b/include/hw/core/cpu.h
index 821e937020
was in fact waiting an ACK.
Signed-off-by: Matheus Tavares Bernardino
---
tests/guest-debug/run-test.py| 16
.../tcg/multiarch/system/Makefile.softmmu-target | 16 +++-
2 files changed, 27 insertions(+), 5 deletions(-)
diff --git a/tests/guest
Matheus Tavares Bernardino (2):
gdbstub: only send stop-reply packets when allowed to
gdbstub: add test for untimely stop-reply packets
Taylor Simpson (2):
Hexagon (gdbstub): add HVX support
Hexagon (linux-user/hexagon): handle breakpoints
gdbstub/internals.h
the additional "move_to_pc" flag instead of just
avoiding the update whenever `dest == ctx->next_PC`, as that could
potentially skip updates from a COF with met condition, whose
ctx->branch_dest just happens to be equal to ctx->next_PC.
Signed-off-by: Matheus Tavares Bernardino
--
0
(/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x57ef0)
Fix that by performing the overflow check in the inner loop, instead.
Signed-off-by: Matheus Tavares Bernardino
---
migration/xbzrle.c | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/migration/xbzrle.c b/migration/
which properly
handles 0.
Signed-off-by: Matheus Tavares Bernardino
---
migration/xbzrle.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/migration/xbzrle.c b/migration/xbzrle.c
index 05366e86c0..21b92d4eae 100644
--- a/migration/xbzrle.c
+++ b/migration/xbzrle.c
@@ -12,6
org/qemu-devel/cover.1678199971.git.quic_mathb...@quicinc.com/
(No changes since v1, only rebased on current master)
Matheus Tavares Bernardino (2):
migration/xbzrle: use ctz64 to avoid undefined result
migration/xbzrle: fix out-of-bounds write with axv512
migration/xbzrle.c | 12 ++--
0
(/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x57ef0)
Fix that by performing the overflow check in the inner loop, instead.
Signed-off-by: Matheus Tavares Bernardino
---
migration/xbzrle.c | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/migration/xbzrle.c b/migration/
which properly
handles 0.
Signed-off-by: Matheus Tavares Bernardino
---
migration/xbzrle.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/migration/xbzrle.c b/migration/xbzrle.c
index 05366e86c0..21b92d4eae 100644
--- a/migration/xbzrle.c
+++ b/migration/xbzrle.c
@@ -12,6
Matheus Tavares Bernardino (2):
migration/xbzrle: use ctz64 to avoid undefined result
migration/xbzrle: fix out-of-bounds write with axv512
migration/xbzrle.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
--
2.39.1
/unit/test-io-channel-tls shows no leakages.
Signed-off-by: Matheus Tavares Bernardino
---
io/channel-tls.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/io/channel-tls.c b/io/channel-tls.c
index 8052945ba0..5a7a3d48d6 100644
--- a/io/channel-tls.c
+++ b/io/channel-tls.c
@@ -44
Alex Bennée writes:
>
> Fabiano Rosas writes:
>
> >
> > I think we might need to add DOCKER_BUILDKIT=1 $(RUNC) ... to properly
> > make use of caching with docker. As for podman, I'm seeing conflicting
> > information on the web as to whether it even supports caching.
>
> As opposed to
Hi,
We noticed that local `make check-tcg` is rebuilding the docker image
for qemu-hexagon at every run, whereas previously it would say "Image is
up to date" and move on.
This was changed at 0b1a649047 (tests/docker: use direct RUNC call to
build containers, 2023-02-28), where we started to no
Thomas Huth wrote:
>
> Out of curiosity: Where did you encounter this problem? After having a quick
> look at https://repology.org/project/bison/versions it seems to me that all
> our supported OS distros should already ship bison 3.0 or newer...
I actually noticed this when compiling our
0025define-parse_002eerror
Signed-off-by: Matheus Tavares Bernardino
---
target/hexagon/meson.build | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/target/hexagon/meson.build b/target/hexagon/meson.build
index c9d31d095c..42b03c81e6 100644
--- a/target/hexagon/meson.build
++
during decoding time and
throw an invalid packet exception. That will be done in the subsequent
commit.
Signed-off-by: Matheus Tavares Bernardino
---
target/hexagon/iclass.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/target/hexagon/iclass.c b/target/hexagon/iclass.c
index
r to debug possible future errors caused
by missing updates to `find_iclass_slots()` rules in
target/hexagon/iclass.c.
Signed-off-by: Matheus Tavares Bernardino
---
target/hexagon/decode.c | 30 +++---
tests/tcg/hexagon/Makefile.target | 10 ++
tests/t
This small series basically fix an outdated hexagon slot rule for
instruction hintjr and makes the decode machinery more resilient against
any possible outdated slot constraints in the future.
Matheus Tavares Bernardino (2):
Hexagon (iclass): update J4_hintjumpr slot constraints
Hexagon
Signed-off-by: Matheus Tavares Bernardino
---
target/hexagon/idef-parser/README.rst | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/target/hexagon/idef-parser/README.rst
b/target/hexagon/idef-parser/README.rst
index 65e6bf4ee5..ff6d14150a 100644
--- a/target/hexagon/idef
Signed-off-by: Matheus Tavares Bernardino
---
accel/tcg/tcg-accel-ops-rr.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/accel/tcg/tcg-accel-ops-rr.c b/accel/tcg/tcg-accel-ops-rr.c
index cc8adc2380..cc912df108 100644
--- a/accel/tcg/tcg-accel-ops-rr.c
+++ b/accel/tcg/tcg
Taylor Simpson wrote:
>
> diff --git a/target/hexagon/translate.h b/target/hexagon/translate.h
> index eae358cf33..e60dbf0e7a 100644
> --- a/target/hexagon/translate.h
> +++ b/target/hexagon/translate.h
> @@ -54,6 +54,9 @@ typedef struct DisasContext {
> bool qreg_is_predicated[NUM_QREGS];
>
1 - 100 of 113 matches
Mail list logo