Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor GDB adding support for threading and tls packets #15

Merged
merged 6 commits into from
Nov 27, 2024

Conversation

andressbarajas
Copy link
Collaborator

@andressbarajas andressbarajas commented Nov 12, 2024

This PR enhances the GDB (GNU Debugger) stub in KallistiOS with threading and TLS (Thread-Local Storage) packet support, making debugging multi-threaded apps more reliable and efficient.

Threading Support:

  • Added basic threading commands in gdb_stub.c.
  • Updated the stub to recognize and handle multiple threads (full thread info is still in progress).

TLS Packets:

  • Added support for TLS packets to improve compatibility with GDB’s thread-local storage debugging.

Protocol Formatting:

  • Cleaned up and standardized protocol descriptions for easier readability.

Error Codes:

  • Introduced #define statements for common error codes to simplify error management in the stub.

@andressbarajas andressbarajas changed the title Gdb add Refactor gdb adding support for threading and tls packets Nov 12, 2024
@andressbarajas andressbarajas changed the title Refactor gdb adding support for threading and tls packets Refactor GDB adding support for threading and tls packets Nov 12, 2024
gyrovorbis and others added 6 commits November 12, 2024 19:37
- Cleaned up formatting and styling issues with the protocol
  descriptions
- Started implementing thread-specific commands
* Currently incomplete. Debugging fails without some of the other thread
  commands implemented. WIP
- Got the stub to at least execute properly as it used to with the new
  threading commands enabled. Still not displaying the extended info or
other thread ID's, though. WIP.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants