Skip to content

[SYCL][ESIMD][E2E] Fix esimd emulator timeout #9143

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

Merged
merged 1 commit into from
Apr 21, 2023

Conversation

sarnex
Copy link
Contributor

@sarnex sarnex commented Apr 20, 2023

This test is computationally intensive and when running with the esimd emulator backend on a slow CPU it may hit the timeout.

Add a 512x512 case and lower the existing esimd emulator limit to 512.

This brings the testing time from 4 minutes to 1 minute.

When run on a real GPU it runs in 1 second even on Gen9 so the extra 512 case shouldn't cause any issues there.

This test is computationally intensive and when running with the
esimd emulator backend on a slow CPU it may hit the timeout.

Add a 512x512 case and lower the existing esimd emulator limit to 512.

This brings the testing time from 4 minutes to 1 minute.

When run on a real GPU it runs in 1 second even on Gen9 so the extra 512 case shouldn't cause any issues there.

Signed-off-by: Sarnie, Nick <nick.sarnie@intel.com>
@sarnex sarnex marked this pull request as ready for review April 20, 2023 19:35
@sarnex sarnex requested a review from a team as a code owner April 20, 2023 19:35
@sarnex sarnex temporarily deployed to aws April 20, 2023 22:05 — with GitHub Actions Inactive
@sarnex sarnex temporarily deployed to aws April 21, 2023 01:34 — with GitHub Actions Inactive
@sarnex
Copy link
Contributor Author

sarnex commented Apr 21, 2023

@intel/llvm-gatekeepers This one is ready too, thx

@steffenlarsen steffenlarsen merged commit 615bed4 into intel:sycl Apr 21, 2023
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.

3 participants