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

detect, unify: Add basic idea of how detect interacts with unify #10

Merged
merged 2 commits into from
Jun 15, 2022

Conversation

linD026
Copy link
Owner

@linD026 linD026 commented Jun 13, 2022

The detect part will call report_set_info() to register the task
information when the watchpoint found out.
And, it will call the report function in detect::setup_watchpoint()
to summarize the data race information.

Also, complete the implementation of the detect subsystem.

Signed-off-by: Chih-En Lin shiyn.lin@gmail.com

The detect will call report_set_info() to register the task information
when the watchpoint found out.

In the future, it also will call the report function in
detect::setup_watchpoint() to summarize the data race information.

Signed-off-by: Chih-En Lin <shiyn.lin@gmail.com>
@linD026 linD026 changed the title detect, report: Add basic idea of how detect interacts with report detect, unify: Add basic idea of how detect interacts with unify Jun 13, 2022
@linD026
Copy link
Owner Author

linD026 commented Jun 13, 2022

@ziyuan1135 @celizon @ZitingLi
Please check the concept of how the detect interact with the unify.

This was referenced Jun 13, 2022
int dummy;
};

struct cached_info cached_info[NR_UCSAN_SLOT * NR_UCSAN_WP];
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we want to discuss the container of how to maintain the task information, please create the issue. Not here.
Thanks.

@linD026
Copy link
Owner Author

linD026 commented Jun 14, 2022

@jack80806
Our basic implementation of the data race detect subsystem will complete at this PR.
Please give us some feedback.
Thanks.

Signed-off-by: Chih-En Lin <shiyn.lin@gmail.com>
@linD026 linD026 merged commit 2f9db49 into main Jun 15, 2022
@linD026 linD026 deleted the dev-watchpoint branch June 15, 2022 19:41
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

2 participants