Hello,
To write a qtest for an SoC using ARM GICv3 as interrupt controller the
qtest infrastructure is short on IRQs.
File tests/qtest/libqtest.c defines:
#define MAX_IRQ 256
Which is the number of elements in the array that is used to keep track of
the value of the irq gpio:
bool irq_leve
c
https:sourceware.org/git/?p=binutils-gdb.git;a=blob;f=gdb/breakpoint.c;h=17bd627f867cf3d4dc81322ed1919ba40cbb237d;hb=HEAD#l5550
The proposed fix inserts or removes a breakpoint to all the cpus
sharing the same process ID as the one selected with the latest
received 'Hg' packet.
Roque
rivate function
gdb_get_cpu_pid to the TCG and also expose the value of
gdbserver_state.multiprocess. The PID filtering will only be
applicable to multiprocess gdb because the PIDs are only defined in
that context.
Google-Bug-Id: 355027002
Signed-off-by: Roque Arcudia Hernandez
---
accel/tcg
he GDB source code file gdb/remote.c.
Functions remote_target::insert_breakpoint and
remote_target::remove_breakpoint will eventually call
remote_target::set_general_thread if it needs to change the process
focus and not remote_target::set_continue_thread.
This can be seen around a comment that says:
/
Hello,
I'm dealing with a problem in which I need to add support for some
neoverse-v2 registers defined as implementation defined in the TRM of the
core.
In file helper.c I can see the decision of whether or not a register is
implemented is based mainly in function calls arm_feature(env,
ARM_FEAT
ure the remote is pointing at the right process, if
> > necessary. */
> >
> > Google-Bug-Id: 355027002
>
> Where can we find more information on this bug ID?
> I tried various query in the Google public tracker but
> couldn't find anything.
> (i.e. https://issuetracker.google.com/issues?q=canonicalid:355027002)
>
> > Signed-off-by: Roque Arcudia Hernandez
> > ---
> > gdbstub/gdbstub.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
>
Thanks to 72d277a7, 1ed2cb32, and others, EDID (Extended Display
Identification Data) is propagated by QEMU such that a virtual
display presents legitimate metadata (e.g., name, serial number,
preferred resolutions, etc.) to its connected guest.
This change adds the ability to specify the EDID nam
vga,head=0,name=AAA \
-display vnc=localhost:1,id=bbb,display=vga,head=1,name=BBB \
-device virtio-vga,max_outputs=2,id=vga,edid_name=true
Signed-off-by: Andrew Keesler
Signed-off-by: Roque Arcudia Hernandez
---
hw/display/virtio-gpu-base.c | 4
include/hw/virtio/virtio-gpu.h | 5
ned-off-by: Andrew Keesler
Signed-off-by: Roque Arcudia Hernandez
---
include/ui/console.h | 1 +
ui/console-priv.h| 1 +
ui/console.c | 8
ui/vnc.c | 8 +++-
4 files changed, 17 insertions(+), 1 deletion(-)
diff --git a/include/ui/console.h b/include/ui/conso
From: Nabih Estefan
Converting error to trace message to avoid spamming of message when
there is nothing to transmit.
Signed-off-by: Nabih Estefan
Signed-off-by: Roque Arcudia Hernandez
---
hw/net/npcm_gmac.c | 5 ++---
hw/net/trace-events | 1 +
2 files changed, 3 insertions(+), 3
This contains a series of small compiler fixes to enable the compilation
of qemu using clang-cl.
It mainly involves adding some missing header files and updating #ifdefs
to handle clang-cl specific things.
Signed-off-by: Erwin Jansen
Signed-off-by: Roque Arcudia Hernandez
---
include/qemu
Hello,
Is there any feedback for this patch?
Thanks
Roque
On Mon, Oct 14, 2024 at 11:49 AM Roque Arcudia Hernandez
wrote:
>
> From: Nabih Estefan
>
> Converting error to trace message to avoid spamming of message when
> there is nothing to transmit.
>
> Signed-off-by: Na
Hello,
This is the second version of the patch. I have taken into
consideration the feedback from the first discussion thread. Is there
anything else to do?
Thanks
Roque
On Thu, Oct 17, 2024 at 8:37 AM Roque Arcudia Hernandez
wrote:
>
> This patch series modifies the gdbstub to address
l use a '/'
(forward) slash.
Signed-off-by: Erwin Jansen
Signed-off-by: Roque Arcudia Hernandez
---
scripts/tracetool/__init__.py | 3 ++-
scripts/tracetool/backend/ftrace.py | 5 +++--
scripts/tracetool/backend/log.py| 5 +++--
scripts/tracetool/backend/syslog.py |
__packed is non standard and is not present in clang-cl.
__attribute__((packed)) has the same semantics.
Signed-off-by: Erwin Jansen
Signed-off-by: Roque Arcudia Hernandez
---
include/hw/usb/dwc2-regs.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/hw/usb/dwc2
DF(bus, devfn) ((bus << 8) | (devfn))
| ^~~
cc1: all warnings being treated as errors
Signed-off-by: Roque Arcudia Hernandez
Reviewed-by: Nabih Estefan
---
include/hw/pci/pci.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/hw/pci/
: Roque Arcudia Hernandez
Reviewed-by: Stephen Longfield
---
tests/qtest/cmsdk-apb-watchdog-test.c | 214 ++
1 file changed, 214 insertions(+)
diff --git a/tests/qtest/cmsdk-apb-watchdog-test.c
b/tests/qtest/cmsdk-apb-watchdog-test.c
index fe535a553c..3777b7bd59 100644
--- a
abled, after previously being disabled.
Source of the time of writing:
https://developer.arm.com/documentation/ddi0479/d/apb-components/apb-watchdog/programmers-model
Signed-off-by: Roque Arcudia Hernandez
Reviewed-by: Stephen Longfield
Reviewed-by: Joe Komlodi
---
hw/watchdog/cm
which has a special behavior where it cannot
be stopped.
The MPS2 machine on the other hand uses the standard CMSDK watchdog
and could expose the problems we were dealing with.
Roque Arcudia Hernandez (3):
hw/watchdog/cmsdk_apb_watchdog: Fix INTEN issues
tests/qtest/cmsdk-apb-watchdog
) while still having CONFIG_CMSDK_APB_WATCHDOG and the test
will fail. Due to the addition of the MPS2 machine CONFIG_MPS2
becomes also a dependency for the test compilation.
Signed-off-by: Roque Arcudia Hernandez
Reviewed-by: Stephen Longfield
---
tests/qtest/cmsdk-apb-watchdog-test.c | 115
I'm reimplementing this in a new patchset with a new function
gdb_cpu_in_source_group instead of making public the PID and
multiptocess functions.
On Mon, Oct 7, 2024 at 3:15 AM Alex Bennée wrote:
>
> Roque Arcudia Hernandez writes:
>
> > In the context of using the remo
c
https:sourceware.org/git/?p=binutils-gdb.git;a=blob;f=gdb/breakpoint.c;h=17bd627f867cf3d4dc81322ed1919ba40cbb237d;hb=HEAD#l5550
The proposed fix inserts or removes a breakpoint to all the cpus
sharing the same process ID as the one selected with the latest
received 'Hg' packet.
Roq
entify the CPUs that should be
affected by the breakpoint insertion/deletion operations based on the
thread-id sent in the last 'Hg' packet. When multiprocess gdb is not
used it falls back to applying the breakpoints to all cpus.
Signed-off-by: Roque Arcudia Hernandez
---
accel/tcg/tcg-accel-o
he GDB source code file gdb/remote.c.
Functions remote_target::insert_breakpoint and
remote_target::remove_breakpoint will eventually call
remote_target::set_general_thread if it needs to change the process
focus and not remote_target::set_continue_thread.
This can be seen around a comment that says:
I'm adding extra documentation to those fields in a new patchset.
On Fri, Oct 4, 2024 at 1:46 PM Alex Bennée wrote:
>
> Roque Arcudia Hernandez writes:
>
> > In the context of using the remote gdb with multiple
> > processes/inferiors (multiple cluster machine) a
I'll be removing the extra line in the comment and adding a new patch
modifying the link in version 2.
On Thu, Nov 14, 2024 at 4:53 AM Peter Maydell wrote:
>
> On Fri, 8 Nov 2024 at 19:10, Roque Arcudia Hernandez
> wrote:
> >
> > Current watchdog is free running o
The patch changes the comments to point to the latest Design Kit
Technical Reference Manual.
Signed-off-by: Roque Arcudia Hernandez
---
hw/watchdog/cmsdk-apb-watchdog.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/watchdog/cmsdk-apb-watchdog.c b/hw/watchdog/cmsdk
: Roque Arcudia Hernandez
Reviewed-by: Stephen Longfield
---
tests/qtest/cmsdk-apb-watchdog-test.c | 215 ++
1 file changed, 215 insertions(+)
diff --git a/tests/qtest/cmsdk-apb-watchdog-test.c
b/tests/qtest/cmsdk-apb-watchdog-test.c
index fe535a553c..53538f98c9 100644
--- a
Hello,
I forgot to add the -v 2 to this patch series. This is the second
version after Peter's feedback. Shall I resend it again or is it fine
like this?
Thanks
Roque
On Fri, Nov 15, 2024 at 8:03 AM Roque Arcudia Hernandez
wrote:
>
> The following patchset tries to address an issu
I'll edit the g_test_set_nonfatal_assertions in a separate patch
explaining the reasoning behind it in version 2.
On Thu, Nov 14, 2024 at 4:55 AM Peter Maydell wrote:
>
> On Fri, 8 Nov 2024 at 19:10, Roque Arcudia Hernandez
> wrote:
> >
> > Currently the CMSDK AP
Thanks for pointing this out. For now I'll be adding the extra line in
the version 2 of this patch.
On Thu, Nov 14, 2024 at 5:01 AM Peter Maydell wrote:
>
> On Fri, 8 Nov 2024 at 19:10, Roque Arcudia Hernandez
> wrote:
> >
> > The following tests focus on making sure
) while still having CONFIG_CMSDK_APB_WATCHDOG and the test
will fail. Due to the addition of the MPS2 machine CONFIG_MPS2
becomes also a dependency for the test compilation.
Signed-off-by: Roque Arcudia Hernandez
Reviewed-by: Stephen Longfield
---
tests/qtest/cmsdk-apb-watchdog-test.c | 112
abled, after previously being disabled.
Source of the time of writing:
https://developer.arm.com/documentation/ddi0479/d/apb-components/apb-watchdog/programmers-model
Signed-off-by: Roque Arcudia Hernandez
Reviewed-by: Stephen Longfield
Reviewed-by: Joe Komlodi
---
hw/watchdog/cm
which has a special behavior where it cannot
be stopped.
The MPS2 machine on the other hand uses the standard CMSDK watchdog
and could expose the problems we were dealing with.
Roque Arcudia Hernandez (5):
hw/watchdog/cmsdk_apb_watchdog: Fix broken link
hw/watchdog/cmsdk_apb_watchdog: Fix
behavior in order to let the test finish other tests and
report the ones that pass and fail:
# ERROR:../tests/qtest/cmsdk-apb-watchdog-test.c:101:test_watchdog:
assertion failed ...
not ok 1 /arm/cmsdk-apb-watchdog/watchdog
Signed-off-by: Roque Arcudia Hernandez
---
tests/qtest/cmsdk-apb-watchdog
35 matches
Mail list logo
