KEMBAR78
[INTEL oneDNN] Disable oneDNN blocked format support by gzmkl · Pull Request #53288 · tensorflow/tensorflow · GitHub
Skip to content

Conversation

@gzmkl
Copy link
Contributor

@gzmkl gzmkl commented Dec 2, 2021

This PR intends to disable oneDNN blocked format support in TensorFlow 2.8.

We will refactor related code in TensorFlow 2.9 (Note: refactoring work has been done but PR is BIG;
we do not submit it because TF 2.8 cutting date is coming).

@google-ml-butler google-ml-butler bot added the size:S CL Change Size: Small label Dec 2, 2021
@google-cla google-cla bot added the cla: yes label Dec 2, 2021
@gbaned gbaned added the comp:mkl MKL related issues label Dec 3, 2021
@gbaned gbaned requested a review from penpornk December 3, 2021 04:30
@google-ml-butler google-ml-butler bot added the awaiting review Pull request awaiting review label Dec 3, 2021
@google-ml-butler google-ml-butler bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Dec 5, 2021
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Dec 5, 2021
@penpornk
Copy link
Member

penpornk commented Dec 6, 2021

@gzmkl Just curious, do you plan to document somewhere (e.g. on your website) that this environment variable is no longer supported? I wonder if we should make the code print a warning if users have this var set for 2.8 (that setting the var has no effect) and remove the print in 2.9. We can also add this change to TF release note as a minor change note, I guess.

@gbaned gbaned added ready to pull PR ready for merge process and removed awaiting review Pull request awaiting review ready to pull PR ready for merge process labels Dec 6, 2021
@gzmkl
Copy link
Contributor Author

gzmkl commented Dec 8, 2021

@penpornk Hi Penporn, thank you for the approval.

Yes, it is proper to add this change in TF release notes. Please help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes comp:mkl MKL related issues size:S CL Change Size: Small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants