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

Another bucket of fixes (and future bugs...) #2917

Merged
merged 12 commits into from
Mar 24, 2025
Merged

Conversation

dzarukin
Copy link
Contributor

@dzarukin dzarukin commented Mar 18, 2025

MFDNN-13351 - improve filling for matmul so that benchdnn doesn't report false-positives with relatively big zero-point values.
MFDNN-13377 - Update benchdnn and the library to restrict int4 buffers for byte-alignment only on the most dense dimension.
MFDNN-13360 - Fixup of previous change reducing the scratchpad space - the execute part was misaligned with the updated logic.
MFDNN-13368 - Updated benchdnn filling for reorder to avoid underflow in case of s32 validation with zero-points.
MFDNN-13376 - The product of refactor + a victim of a bug in benchdnn which didn't report cases becoming unimplemented properly. Reporting was fixed, case started returning unimplemented as expected. Unimplemented case was fixed in jit_uni_reorder, and also verbose messages were added.
MFDNN-11868 - Use a string object to carry the content for parsing as it sometimes somehow can be destroyed.

@dzarukin dzarukin requested review from a team as code owners March 18, 2025 23:31
@github-actions github-actions bot added platform:cpu-x64 Intel64/AMD64 processors. Codeowner: @oneapi-src/onednn-cpu-x64 component:tests Codeowner: @oneapi-src/onednn-arch labels Mar 18, 2025
@dzarukin
Copy link
Contributor Author

make test
set test_scope=NIGHTLY

@dzarukin dzarukin force-pushed the dzarukin/allow_int4_odd branch from 5f0373c to a14aff9 Compare March 19, 2025 04:55
@dzarukin
Copy link
Contributor Author

make test
set test_scope=NIGHTLY

@dzarukin dzarukin force-pushed the dzarukin/allow_int4_odd branch 2 times, most recently from b34fab9 to 2e2c013 Compare March 19, 2025 19:43
@dzarukin
Copy link
Contributor Author

make test
set test_scope=NIGHTLY

@dzarukin dzarukin force-pushed the dzarukin/allow_int4_odd branch 3 times, most recently from 8e14b33 to f4d86c8 Compare March 19, 2025 23:00
@dzarukin
Copy link
Contributor Author

make test
set test_scope=NIGHTLY

@dzarukin dzarukin force-pushed the dzarukin/allow_int4_odd branch from f4d86c8 to 8e04316 Compare March 20, 2025 04:43
@dzarukin
Copy link
Contributor Author

make test

dzarukin added 11 commits March 20, 2025 18:09

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
… on it

Case with different mask is not supported if only both scales were
specified.
Temporary "const char *" objects can disappear while getting to the
parser internals. Moving strings to parse into a permanent container
solves the problem.
@dzarukin dzarukin force-pushed the dzarukin/allow_int4_odd branch from 8e04316 to 608baa3 Compare March 21, 2025 18:27
@dzarukin
Copy link
Contributor Author

make test
set test_scope=NIGHTLY

@vpirogov vpirogov force-pushed the dzarukin/allow_int4_odd branch from 608baa3 to f933c1e Compare March 24, 2025 17:50
@vpirogov vpirogov merged commit 67c3042 into main Mar 24, 2025
22 checks passed
@vpirogov vpirogov deleted the dzarukin/allow_int4_odd branch March 24, 2025 19:24
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:cpu-x64 Intel64/AMD64 processors. Codeowner: @oneapi-src/onednn-cpu-x64
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants