<feed xmlns='http://www.w3.org/2005/Atom'>
<title>qemu/tests/guest-debug, branch master</title>
<subtitle>QEMU development tree</subtitle>
<id>https://git.zx2c4.com/qemu/atom/tests/guest-debug?h=master</id>
<link rel='self' href='https://git.zx2c4.com/qemu/atom/tests/guest-debug?h=master'/>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/qemu/'/>
<updated>2024-05-17T14:49:04Z</updated>
<entry>
<title>tests: Gently exit from GDB when tests complete</title>
<updated>2024-05-17T14:49:04Z</updated>
<author>
<name>Gustavo Romero</name>
<email>gustavo.romero@linaro.org</email>
</author>
<published>2024-05-15T17:31:31Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/qemu/commit/?id=93a3048dcf4565c73f2aa1d751f7197e296f1f1f'/>
<id>urn:sha1:93a3048dcf4565c73f2aa1d751f7197e296f1f1f</id>
<content type='text'>
GDB commit a207f6b3a38 ('Rewrite "python" command exception handling')
changed how exit() called from Python scripts loaded by GDB behave,
turning it into an exception instead of a generic error code that is
returned. This change caused several QEMU tests to crash with the
following exception:

Python Exception &lt;class 'SystemExit'&gt;: 0
Error occurred in Python: 0

This happens because in tests/guest-debug/test_gdbstub.py exit is
called after the tests have completed.

This commit fixes it by politely asking GDB to exit via gdb.execute,
passing the proper fail_count to be reported to 'make', instead of
abruptly calling exit() from the Python script.

Signed-off-by: Gustavo Romero &lt;gustavo.romero@linaro.org&gt;
Reviewed-by: Philippe Mathieu-Daudé &lt;philmd@linaro.org&gt;
Message-ID: &lt;20240515173132.2462201-4-gustavo.romero@linaro.org&gt;
Signed-off-by: Philippe Mathieu-Daudé &lt;philmd@linaro.org&gt;
</content>
</entry>
<entry>
<title>tests/tcg: Factor out gdbstub test functions</title>
<updated>2024-01-29T10:48:49Z</updated>
<author>
<name>Ilya Leoshkevich</name>
<email>iii@linux.ibm.com</email>
</author>
<published>2024-01-29T09:32:15Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/qemu/commit/?id=4d48c1bc079147a02ec1a3df9a41497c515d2e61'/>
<id>urn:sha1:4d48c1bc079147a02ec1a3df9a41497c515d2e61</id>
<content type='text'>
Both the report() function as well as the initial gdbstub test sequence
are copy-pasted into ~10 files with slight modifications. This
indicates that they are indeed generic, so factor them out. While
at it, add a few newlines to make the formatting closer to PEP-8.

Signed-off-by: Ilya Leoshkevich &lt;iii@linux.ibm.com&gt;
Message-Id: &lt;20240129093410.3151-3-iii@linux.ibm.com&gt;
Signed-off-by: Richard Henderson &lt;richard.henderson@linaro.org&gt;
</content>
</entry>
<entry>
<title>tests/tcg: clean-up gdb confirm/pagination settings</title>
<updated>2023-08-30T13:57:50Z</updated>
<author>
<name>Alex Bennée</name>
<email>alex.bennee@linaro.org</email>
</author>
<published>2023-08-29T16:15:23Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/qemu/commit/?id=a8fea70f656416b2ed7d388fbcc7cc6cda126a82'/>
<id>urn:sha1:a8fea70f656416b2ed7d388fbcc7cc6cda126a82</id>
<content type='text'>
We can do this all in the run-test.py script so remove the extraneous
bits from the individual tests which got copied from the original
non-CI gdb tests.

Acked-by: Ilya Leoshkevich &lt;iii@linux.ibm.com&gt;
Reviewed-by: Philippe Mathieu-Daudé &lt;philmd@linaro.org&gt;
Reviewed-by: Richard Henderson &lt;richard.henderson@linaro.org&gt;
Signed-off-by: Alex Bennée &lt;alex.bennee@linaro.org&gt;
Message-Id: &lt;20230829161528.2707696-8-alex.bennee@linaro.org&gt;
</content>
</entry>
<entry>
<title>tests: remove test-gdbstub.py</title>
<updated>2023-08-30T13:57:50Z</updated>
<author>
<name>Alex Bennée</name>
<email>alex.bennee@linaro.org</email>
</author>
<published>2023-08-29T16:15:22Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/qemu/commit/?id=d84842be6c53c03c9498101b509a25961ed5856f'/>
<id>urn:sha1:d84842be6c53c03c9498101b509a25961ed5856f</id>
<content type='text'>
This isn't directly called by our CI and because it doesn't run via
our run-test.py script does things slightly differently. Lets remove
it as we have plenty of working in-tree tests now for various aspects
of gdbstub.

Reviewed-by: Richard Henderson &lt;richard.henderson@linaro.org&gt;
Signed-off-by: Alex Bennée &lt;alex.bennee@linaro.org&gt;
Message-Id: &lt;20230829161528.2707696-7-alex.bennee@linaro.org&gt;
</content>
</entry>
<entry>
<title>tests/tcg: ensure system-mode gdb tests start stopped</title>
<updated>2023-08-10T18:04:34Z</updated>
<author>
<name>Alex Bennée</name>
<email>alex.bennee@linaro.org</email>
</author>
<published>2023-08-10T15:36:38Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/qemu/commit/?id=dad1036f43b55c68eb76e73ef771b252ec764b14'/>
<id>urn:sha1:dad1036f43b55c68eb76e73ef771b252ec764b14</id>
<content type='text'>
Without -S we run into potential races with tests starting before the
gdbstub attaches. We don't need to worry about user-mode as enabling
the gdbstub implies we wait for the initial connection.

Signed-off-by: Alex Bennée &lt;alex.bennee@linaro.org&gt;
Reviewed-by: Richard Henderson &lt;richard.henderson@linaro.org&gt;
Reviewed-by: Philippe Mathieu-Daudé &lt;philmd@linaro.org&gt;
Message-Id: &lt;20230810153640.1879717-7-alex.bennee@linaro.org&gt;
Signed-off-by: Richard Henderson &lt;richard.henderson@linaro.org&gt;
</content>
</entry>
<entry>
<title>gdbstub: add test for untimely stop-reply packets</title>
<updated>2023-05-18T19:40:52Z</updated>
<author>
<name>Matheus Tavares Bernardino</name>
<email>quic_mathbern@quicinc.com</email>
</author>
<published>2023-05-04T15:37:32Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/qemu/commit/?id=dae66a3f669eaf0e33447ed42a36ad4ee2f178f2'/>
<id>urn:sha1:dae66a3f669eaf0e33447ed42a36ad4ee2f178f2</id>
<content type='text'>
In the previous commit, we modified gdbstub.c to only send stop-reply
packets as a response to GDB commands that accept it. Now, let's add a
test for this intended behavior. Running this test before the fix from
the previous commit fails as QEMU sends a stop-reply packet
asynchronously, when GDB was in fact waiting an ACK.

Signed-off-by: Matheus Tavares Bernardino &lt;quic_mathbern@quicinc.com&gt;
Acked-by: Alex Bennée &lt;alex.bennee@linaro.org&gt;
Signed-off-by: Taylor Simpson &lt;tsimpson@quicinc.com&gt;
Message-Id: &lt;a30d93b9a8d66e9d9294354cfa2fc3af35f00202.1683214375.git.quic_mathbern@quicinc.com&gt;
</content>
</entry>
<entry>
<title>tests/guest-debug: better handle gdb crashes</title>
<updated>2022-04-20T15:04:20Z</updated>
<author>
<name>Alex Bennée</name>
<email>alex.bennee@linaro.org</email>
</author>
<published>2022-04-19T09:10:20Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/qemu/commit/?id=caccf599181e2ea5f236345de9d9957a4c23e5ec'/>
<id>urn:sha1:caccf599181e2ea5f236345de9d9957a4c23e5ec</id>
<content type='text'>
There are a number of GDB's on various distros which fail fairly hard
when attempting to talk to a cross-arch guest. The previous attempt to
catch this was incorrect as the shell will deliver signals as 128+n.
Fix the detection and while we are it improve the logging we dump into
the test output.

Signed-off-by: Alex Bennée &lt;alex.bennee@linaro.org&gt;
Reported-by: Gautam Agrawal &lt;gautamnagrawal@gmail.com&gt;
Reviewed-by: Richard Henderson &lt;richard.henderson@linaro.org&gt;
Message-Id: &lt;20220419091020.3008144-26-alex.bennee@linaro.org&gt;
</content>
</entry>
<entry>
<title>chardev: do not use short form boolean options in non-QemuOpts character device descriptions</title>
<updated>2021-02-25T14:41:53Z</updated>
<author>
<name>Paolo Bonzini</name>
<email>pbonzini@redhat.com</email>
</author>
<published>2021-02-25T11:05:22Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/qemu/commit/?id=9d902d51154bcbd305ae7138ef31e2843bb3427e'/>
<id>urn:sha1:9d902d51154bcbd305ae7138ef31e2843bb3427e</id>
<content type='text'>
Options such as "-gdb" or "-serial" accept a part-QemuOpts part-parsed-by-hand
character device description.  Do not use short form boolean options in the
QemuOpts part.

Signed-off-by: Paolo Bonzini &lt;pbonzini@redhat.com&gt;
</content>
</entry>
<entry>
<title>gdbstub: implement a softmmu based test</title>
<updated>2021-01-18T10:04:31Z</updated>
<author>
<name>Alex Bennée</name>
<email>alex.bennee@linaro.org</email>
</author>
<published>2021-01-08T22:42:41Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/qemu/commit/?id=c00506aa26e975918483d0d1fe17a2192d19098a'/>
<id>urn:sha1:c00506aa26e975918483d0d1fe17a2192d19098a</id>
<content type='text'>
This adds a new tests that allows us to test softmmu only features
including watchpoints. To do achieve this we need to:

  - add _exit: labels to the boot codes
  - write a memory.py test case
  - plumb the test case into the build system
  - tweak the run_test script to:
    - re-direct output when asked
    - use socket based connection for all tests
    - add a small pause before connection

Signed-off-by: Alex Bennée &lt;alex.bennee@linaro.org&gt;
Reviewed-by: Philippe Mathieu-Daudé &lt;f4bug@amsat.org&gt;
Message-Id: &lt;20210108224256.2321-6-alex.bennee@linaro.org&gt;
</content>
</entry>
<entry>
<title>test/guest-debug: echo QEMU command as well</title>
<updated>2021-01-18T10:04:31Z</updated>
<author>
<name>Alex Bennée</name>
<email>alex.bennee@linaro.org</email>
</author>
<published>2021-01-08T22:42:38Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/qemu/commit/?id=5662ab11f6e70157521913e38e49d4d1b135c4d5'/>
<id>urn:sha1:5662ab11f6e70157521913e38e49d4d1b135c4d5</id>
<content type='text'>
This helps with debugging.

Signed-off-by: Alex Bennée &lt;alex.bennee@linaro.org&gt;
Reviewed-by: Philippe Mathieu-Daudé &lt;philmd@redhat.com&gt;
Message-Id: &lt;20210108224256.2321-3-alex.bennee@linaro.org&gt;
</content>
</entry>
</feed>
