Open verdurin opened 6 years ago
@verdurin That issue is fixed in a more recent version of mkl-dnn
, see https://github.com/intel/mkl-dnn/issues/53 + https://github.com/intel/mkl-dnn/commit/ab84d723f1988c4b11877e320182aa59428839b3 .
Can you try the patch (see https://github.com/intel/mkl-dnn/commit/ab84d723f1988c4b11877e320182aa59428839b3.patch)?
@boegel looks like that patch is already included in the 0.13
release?:
https://github.com/intel/mkl-dnn/blob/v0.13/tests/gtests/gtest/src/gtest.cc#L3868-L3870
@verdurin Maybe in the tests, but you need a similar construction in src/cpu/jit_generator.hpp
for the failing line?
@boegel yes. For reference, v0.16
builds and runs all the tests fine. Maybe easier and cleaner to add that instead of patching the old one, then update the PyTorch
easyconfig, or would you prefer the less disruptive change?
@verdurin I prefer not updating (dependencies in) existing easyconfig files when we can avoid it, it often ends up being more painful than expected. Unless those easyconfig file are only in develop
currently, then it's worth considering.
I had to modify my src/cpu/jit_generator.hpp
in a bit different manner than it is presented in that patch. It may not be obvious, because patch uses pfile
which in my case is defined as fp
, so I'll just paste corrected lines that works for me:
717 // Failure to dump code is not fatal
718 if (fp) {
719 size_t unused = fwrite("0", 1, 1, fp);
720 (void)unused;
721 fclose(fp);
Trying to build
mkl-dnn/0.13-intel-2018a
as a requirement ofPyTorch
:Build host is Ivy Bridge, hence no
avx2
,