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

debugger: fibers debugging tests #8554

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tsafin
Copy link
Contributor

@tsafin tsafin commented Apr 11, 2023

As we have found it harder way we do already perfectly support fiber debugging in the console debugger. The purpose of this PR to make that fact more visible and measurable.

  • now we visualize the moment of fiber switch occurred during debugging session;
  • we implemented special debugging scenario to keep fiber debugging functionality in a good shape;

As a bonus track (to simplify current debugging efforts of a debugger implemented) we have added ability to inject debugger commands automatically upon debugger start via .luadebugrc script. (Similar to .gdbinit functionality we have in gdb).

Followup to #8072

@tsafin tsafin requested a review from ligurio April 11, 2023 18:31
@coveralls
Copy link

coveralls commented Apr 11, 2023

Coverage Status

Coverage: 85.579% (-0.02%) from 85.599% when pulling 5dd64f9 on tsafin:tsafin/debugger-6 into 5ab5ce2
on tarantool:master
.

@tsafin tsafin marked this pull request as ready for review April 12, 2023 08:39
@tsafin tsafin marked this pull request as draft April 12, 2023 08:39
Use `.luadebugrc` as a way to inject set of debugger commands,
which will be executed before all interactive commands to be executed.

This works similarly to `.gdbinit` script.

NO_CHANGELOG=yet
NO_DOC=yet
NO_TEST=yet
Monitor fiber changes from debugger hook, and display
if there is something changed since prior step.

NO_CHANGELOG=internal
NO_DOC=internal
@tsafin tsafin marked this pull request as ready for review April 14, 2023 09:14
@ligurio ligurio removed their request for review July 24, 2023 10:08
@ligurio ligurio assigned igormunkin and unassigned ligurio Jul 24, 2023
@igormunkin igormunkin assigned Lord-KA and unassigned igormunkin Mar 4, 2024
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.

None yet

5 participants