Skip to content

Kernel overlap verification stage, initial version - #124

Merged
ppnaik1890 merged 8 commits into
mainfrom
overlap_verify
Aug 26, 2026
Merged

Kernel overlap verification stage, initial version#124
ppnaik1890 merged 8 commits into
mainfrom
overlap_verify

Conversation

@lasch

@lasch lasch commented Aug 12, 2026

Copy link
Copy Markdown
Member

introducing initial version of verification stage for kernel overlap detection.

Main functionality for overlap verification: src/aiu_trace_analyzer/verification/overlap_verify.py

On top of the derived context, a few tweaks were needed to support the requirements:

  • strict mode: needed to flag regular function call (fully nested overlaps) as actual overlaps
  • more flexible hash-keys: hard-coded pid+tid needs to be more configurable to cover actual 'streams'
  • issue_warning required an override-capable extension to allow derived classes to issue 'warning instances' for the verification report

Also included the following cleanup:

  • verification functions/context imports moved to their own __init__.py
  • removed the template/example verify stage from the verification pipeline
  • verification pipeline needed global sorting stage

@nicksunka @ppnaik1890 I did not yet scrutinize the event filter - right now, any event that comes with indication of 'ran on accelerator' is being checked for overlap within its stream.

lasch added 7 commits August 18, 2026 15:07
…s for verification

Signed-off-by: Lars Schneidenbach <schneidenbach@us.ibm.com>
Signed-off-by: Lars Schneidenbach <schneidenbach@us.ibm.com>
Signed-off-by: Lars Schneidenbach <schneidenbach@us.ibm.com>
Signed-off-by: Lars Schneidenbach <schneidenbach@us.ibm.com>
Signed-off-by: Lars Schneidenbach <schneidenbach@us.ibm.com>
Signed-off-by: Lars Schneidenbach <schneidenbach@us.ibm.com>
Signed-off-by: Lars Schneidenbach <schneidenbach@us.ibm.com>

@ppnaik1890 ppnaik1890 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

minor clarification questions

Comment thread src/aiu_trace_analyzer/core/acelyzer.py
Comment thread src/aiu_trace_analyzer/pipeline/overlap.py
Comment thread src/aiu_trace_analyzer/verification/overlap_verify.py
Comment thread tests/aiu_trace_analyzer/pipeline/test_overlap.py
@lasch

lasch commented Aug 25, 2026

Copy link
Copy Markdown
Member Author

@ppnaik1890 are the provided explanations sufficient?

@ppnaik1890
ppnaik1890 merged commit 45efaf3 into main Aug 26, 2026
2 checks passed
@ppnaik1890
ppnaik1890 deleted the overlap_verify branch August 26, 2026 12:07
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