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

Differences with MKL 2017 DNN #34

Closed
Maratyszcza opened this issue Mar 15, 2017 · 8 comments
Closed

Differences with MKL 2017 DNN #34

Maratyszcza opened this issue Mar 15, 2017 · 8 comments

Comments

@Maratyszcza
Copy link

What are the differences between this version, and DNN primitives in MKL 2017? I see that the interfaces are different.

@emfomenk
Copy link

Hi @Maratyszcza,

For the description of current differences please see this comment.

MKL-DNN interface was re-thought, in particular layout (mkldnn_memory_desc_t) become transparent, while in Intel MKL it is opaque for user. For the rest Intel MKL (DNN primitives) and MKL-DNN are pretty close.

@Maratyszcza
Copy link
Author

Thank you. Will Intel MKL adapt interfaces of MKL-DNN in the future, or there will be two parallel sets of interfaces?

@emfomenk
Copy link

We definitely want to use one single API for both Intel MKL and MKL-DNN. This single API will be MKL-DNN one.
Alas, right now I can't estimate when this will happen. Also since Intel MKL has to provide backward compatibility, DNN interface of Intel MKL will also remain supported for time being (at least for several years after deprecation notice).

@bhack
Copy link

bhack commented Mar 30, 2017

@Maratyszcza Do you plan to integrate in NNPACK?

@Maratyszcza
Copy link
Author

@bhack Integrate what?

@bhack
Copy link

bhack commented Mar 31, 2017

I mean using NNPACK internal implementation and MKL-DNN with the same NNPACK API..

@Maratyszcza
Copy link
Author

No, this wouldn't be efficient. MKL-DNN uses a different layout for layers.

@bhack
Copy link

bhack commented Mar 31, 2017

@Maratyszcza So.. We need to interface another engine in tiny 😉

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

No branches or pull requests

3 participants