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.

Sign up for GitHub

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

Jump to bottom

[Flaky Test] tests/python/unittest/test_roofline.py::test_estimate_peak_flops[llvm] #12955

Open
driazati opened this issue Sep 30, 2022 · 1 comment
Open

[Flaky Test] tests/python/unittest/test_roofline.py::test_estimate_peak_flops[llvm] #12955

driazati opened this issue Sep 30, 2022 · 1 comment
Assignees
@tkonolige
Labels
byoc:ethosu test: flaky

Comments

@driazati
Copy link
Member

Test(s)

  • tests/python/unittest/test_roofline.py::test_estimate_peak_flops[llvm]

Jenkins Links
@driazati driazati mentioned this issue Sep 30, 2022
Merged
driazati added a commit to driazati/tvm that referenced this issue Sep 30, 2022
@driazati
[ci] Disable flaky ethosu + roofline tests
6f8a420 
These are all segfaulting in main (see apache#12955, apache#12933, and apache#12841) so
they need to be skipped until a fix is merged.
driazati added a commit to driazati/tvm that referenced this issue Sep 30, 2022
@driazati
[ci] Disable flaky ethosu + roofline tests
411109a 
These are all segfaulting in main (see apache#12955, apache#12933, and apache#12841) so
they need to be skipped until a fix is merged.
driazati added a commit to driazati/tvm that referenced this issue Sep 30, 2022
@driazati
[ci] Disable flaky ethosu + roofline tests
5f6ef00 
These are all segfaulting in main (see apache#12955, apache#12933, and apache#12841) so
they need to be skipped until a fix is merged.
driazati added a commit to driazati/tvm that referenced this issue Sep 30, 2022
@driazati
[ci] Disable flaky ethosu + roofline tests
9748664 
These are all segfaulting in main (see apache#12955, apache#12933, and apache#12841) so
they need to be skipped until a fix is merged.
driazati added a commit to driazati/tvm that referenced this issue Sep 30, 2022
@driazati
[ci] Disable flaky ethosu + roofline tests
b81f79e 
These are all segfaulting in main (see apache#12955, apache#12933, and apache#12841) so
they need to be skipped until a fix is merged.
@tkonolige tkonolige self-assigned this Sep 30, 2022
@tkonolige tkonolige mentioned this issue Sep 30, 2022
Merged
driazati added a commit to driazati/tvm that referenced this issue Oct 3, 2022
@driazati
[ci] Disable flaky ethosu + roofline tests
c1965ad 
These are all segfaulting in main (see apache#12955, apache#12933, and apache#12841) so
they need to be skipped until a fix is merged.
junrushao pushed a commit that referenced this issue Oct 6, 2022
@driazati
[ci] Disable flaky ethosu + roofline tests (#12956)
3d5e440 
These are all segfaulting in main (see #12955, #12933, and #12841) so
they need to be skipped until a fix is merged.

Co-authored-by: driazati <driazati@users.noreply.github.com>
driazati pushed a commit that referenced this issue Oct 6, 2022
[LOGGING] Optionally print backtrace on segfault (#12959)
78df9c1 
Add BACKTRACE_ON_SEGFAULT cmake option to install a signal handler to print a backtrace on segfault. Disabled by default because existing signal handlers may be overriden by this signal handler. Creating a backtrace may allocate, which should not happen in a signal handler, but the program is crashing so it shouldn't make any difference.

Hopefully this will help diagnose segfaults like in #12955.
@tkonolige
Copy link
Contributor

Given that #12959 is merged, can we reenable the roofline test to get a stack trace for debugging?

@driazati driazati mentioned this issue Oct 7, 2022
Merged
driazati added a commit to driazati/tvm that referenced this issue Oct 7, 2022
@driazati
[ci] Re-enable roofline test
9c9f6c1 
After apache#12959, this re-enables the test disabled in apache#12955 to get a
backtrace next time it fails.
tkonolige pushed a commit that referenced this issue Oct 7, 2022
@driazati
[ci] Re-enable roofline test (#13007)
20aa0cf 
After #12959, this re-enables the test disabled in #12955 to get a
backtrace next time it fails.

Co-authored-by: driazati <driazati@users.noreply.github.com>
@areusch areusch added the needs-triage PRs or issues that need to be investigated by maintainers to find the right assignees to address it label Oct 19, 2022
@janetsc janetsc added byoc:ethosu and removed needs-triage PRs or issues that need to be investigated by maintainers to find the right assignees to address it labels Oct 19, 2022
xinetzone pushed a commit to daobook/tvm that referenced this issue Nov 25, 2022
@driazati @xinetzone
[ci] Disable flaky ethosu + roofline tests (apache#12956)
99d00ef 
These are all segfaulting in main (see apache#12955, apache#12933, and apache#12841) so
they need to be skipped until a fix is merged.

Co-authored-by: driazati <driazati@users.noreply.github.com>
xinetzone pushed a commit to daobook/tvm that referenced this issue Nov 25, 2022
@xinetzone
[LOGGING] Optionally print backtrace on segfault (apache#12959)
50dee8a 
Add BACKTRACE_ON_SEGFAULT cmake option to install a signal handler to print a backtrace on segfault. Disabled by default because existing signal handlers may be overriden by this signal handler. Creating a backtrace may allocate, which should not happen in a signal handler, but the program is crashing so it shouldn't make any difference.

Hopefully this will help diagnose segfaults like in apache#12955.
xinetzone pushed a commit to daobook/tvm that referenced this issue Nov 25, 2022
@driazati @xinetzone
[ci] Re-enable roofline test (apache#13007)
85b3397 
After apache#12959, this re-enables the test disabled in apache#12955 to get a
backtrace next time it fails.

Co-authored-by: driazati <driazati@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tkonolige tkonolige

Labels
byoc:ethosu test: flaky
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@areusch @tkonolige @driazati @janetsc