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

Jit uni reorders give incorrect results with certain 4d matrices and src zero point != 0 #2213

Closed
Ryo-not-rio opened this issue Nov 12, 2024 · 2 comments
Labels
platform:cpu-aarch64 Codeowner: @oneapi-src/onednn-cpu-aarch64 sighting Suspicious library behavior. Should be promoted to a bug when confirmed

Comments

@Ryo-not-rio
Copy link
Contributor

Jit:uni reorders for certain 4d matrices are returning incorrect results when the src zero point != 0. A guard has been added to skip jit uni reorders so the simple:any implementation will be used in #2207, however a fix for jit:uni should be created to improve performance.

From my initial investigation, it seems to happen for some shapes when dim[2] > 16 and dim[3] != multiple of 2 but I'm sure there are other shapes that would cause the same issue.

Version

oneDNN v3.7.0 (commit 2fc6ff9)

Environment

oneDNN includes hardware-specific optimizations and may behave
differently on depending on the compiler and build environment. Include
the following information to help reproduce the issue:

  • CPU make and model: aarch64, graviton 3
  • OS version: Linux 22.04.1-Ubuntu
  • Compiler version: gcc 11.4.0
  • CMake version: 3.30.2
  • git hash: 1712f97

Steps to reproduce

./tests/benchdnn/benchdnn --reorder  --sdt=u8 --ddt=f32 --stag=adbc --attr-zero-points=src:common:1 1x32x128x33

Observed behavior

benchdnn test fails.

Expected behavior

benchdnn test passes.

@Ryo-not-rio Ryo-not-rio added the sighting Suspicious library behavior. Should be promoted to a bug when confirmed label Nov 12, 2024
@Ryo-not-rio
Copy link
Contributor Author

@Shreyas-fuj I noticed you contributed to Jit:uni recently. Could you have a look at this or tag the appropriate person for this? Thank you

@shu1chen shu1chen added the platform:cpu-aarch64 Codeowner: @oneapi-src/onednn-cpu-aarch64 label Nov 13, 2024
@Shreyas-fuj
Copy link
Contributor

Hi @Ryo-not-rio,
I will have a look at this. Thanks for the temporary fix : #2207.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform:cpu-aarch64 Codeowner: @oneapi-src/onednn-cpu-aarch64 sighting Suspicious library behavior. Should be promoted to a bug when confirmed
Projects
None yet
Development

No branches or pull requests

3 participants