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

[GPU] Improve reference kernel coverage. #2366

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dyoussif
Copy link
Contributor

@dyoussif dyoussif commented Jan 9, 2025

Addresses https://jira.devtools.intel.com/browse/MFDNN-12444.
Broaden ocl:ref coverage and add some smoke tests for key compute bound primitives.

@dyoussif dyoussif requested review from a team as code owners January 9, 2025 23:36
@github-actions github-actions bot added platform:gpu-intel Codeowner: @oneapi-src/onednn-gpu-intel component:tests Codeowner: @oneapi-src/onednn-arch labels Jan 9, 2025
@dyoussif
Copy link
Contributor Author

dyoussif commented Jan 9, 2025

make test
disable device_cpu
disable run_scans
disable benchdnn_all
enable benchdnn_nightly
enable benchdnn_matmul
enable benchdnn_conv
enable benchdnn_ip
enable benchdnn_deconv
enable arch_xe-lp
enable arch_xe2-lpg
enable arch_xe2-hpg-bmg

Copy link
Contributor

@mgouicem mgouicem left a comment

Choose a reason for hiding this comment

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

My 2 cents on targeting specific implementations in our ci coverage: it would increase ci testing scope with no clear value since ref impls should be tested by HW configuration that do not dispatch optimized ones.

For that reason, I would keep --global-impl flag out of the input files, and have it used for development purpose only.

@@ -33,3 +33,19 @@
--attr-post-ops=,linear:2:1
--runtime_dims_masks=1:0,3:3
--batch=shapes_2d_ci

# Decompression
Copy link
Contributor

Choose a reason for hiding this comment

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

these do not seem to cover decompression as fp-mathmode is not set here. Am I missing something?

@@ -689,7 +689,7 @@ void def_eltwise_alg_kinds(compute::kernel_ctx_t &kernel_ctx);

bool post_ops_with_binary_ok(const primitive_attr_t *attr,
const data_type_t dst_dt, const int max_ndims_supported = 2,
const int prelu_mask_supported = 3);
const int prelu_mask_supported = 0xFFFF);
Copy link
Contributor

Choose a reason for hiding this comment

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

should the default value just be dropped if it doesnt restrict any value?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:tests Codeowner: @oneapi-src/onednn-arch platform:gpu-intel Codeowner: @oneapi-src/onednn-gpu-intel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants