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
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
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
) 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
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
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
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
) 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
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
: 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
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/
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
__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
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 |
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
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
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
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
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
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
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 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
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
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(-)
>
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
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:
/
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
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
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
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
35 matches
Mail list logo
