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

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

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

Comments

@driazati
Copy link
Member

Test(s)

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

Jenkins Links

driazati added a commit to driazati/tvm that referenced this issue Sep 30, 2022
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
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
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
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
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
driazati added a commit to driazati/tvm that referenced this issue Oct 3, 2022
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
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
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 added a commit to driazati/tvm that referenced this issue Oct 7, 2022
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
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
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
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
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>
@tqchen tqchen closed this as completed Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants