MKL with 1.2.0rc3

[Naveen Swamy <mn...@gmail.com>]

How do I build the latest release(1.2.0rc3) with MKL enabled, it looks like
its now defaulting to the experimental MKLDNN. I do not want to MKLDNN as
it is experimental.

-Naveen

Re: MKL with 1.2.0rc3

[Pedro Larroy <pe...@gmail.com>]

It would be great if we could improve the documentation of building with
MKL? Whenever I have done this it took quite some effort to get it right,
also add the different MKL related libraries.

there's also duplicated scripts:

./3rdparty/mkldnn/scripts/prepare_mkl.bat
./3rdparty/mkldnn/scripts/prepare_mkl.sh
./prepare_mkl.sh
./prepare_mkldnn.sh

And not documented for CMake. I have seen that now master downloads mkl by
itself. This is really cool. But I wish this was better documented and
clear specially with CMake.

Thanks a lot.

Pedro.


On Wed, May 16, 2018 at 9:13 PM, Naveen Swamy <mn...@gmail.com> wrote:

> I understand that MKLML was experimental but like you pointed out it has
> been around for a while, so I am presuming some bugs would have been fixed.
> Users have also seen about 30% performance boost with MKL, now with the
> latest release we are dropping support and forcing to use another
> experimental which we know(from all the discussion during the release) that
> its not stable. Also OpenBlas=MKL does not bring all the features for MKLML
> and MKLML will have a smaller sized library per this response by a Intel
> member https://github.com/intel/mkl-dnn/issues/102.
>
> IMHO this is not a good practice, I also wish that a discussion happen on
> dev list before we drop support, so more people provide feedback.
>
>
>
> On Wed, May 16, 2018 at 11:54 AM, Anirudh <an...@gmail.com> wrote:
>
> > Hi Naveen,
> >
> > There was an earlier flag for using MKLML which was a subset of MKL. I
> > think this flag was removed when MKLDNN was added.
> > I think the only replacement to the old MKLML is the MKLDNN feature that
> > was added. ( I am assuming this also installs the MKLML)
> > So if users don't want to use MKLDNN feature they will have to install
> the
> > full MKL library and set USE_BLAS=mkl if I am not wrong.
> >
> > I would also like to understand more from others about what is the best
> way
> > to transition for users who were using MKLML flag earlier but don't want
> to
> > use
> > MKLDNN flag now ? (Both flags were experimental, but MKLML flag had been
> > around for some time.)
> >
> > Anirudh
> >
> >
> > On Wed, May 16, 2018 at 11:38 AM, Naveen Swamy <mn...@gmail.com>
> wrote:
> >
> > > How do I build the latest release(1.2.0rc3) with MKL enabled, it looks
> > like
> > > its now defaulting to the experimental MKLDNN. I do not want to MKLDNN
> as
> > > it is experimental.
> > >
> > > -Naveen
> > >
> >
>

Re: MKL with 1.2.0rc3

[Naveen Swamy <mn...@gmail.com>]

I understand that MKLML was experimental but like you pointed out it has
been around for a while, so I am presuming some bugs would have been fixed.
Users have also seen about 30% performance boost with MKL, now with the
latest release we are dropping support and forcing to use another
experimental which we know(from all the discussion during the release) that
its not stable. Also OpenBlas=MKL does not bring all the features for MKLML
and MKLML will have a smaller sized library per this response by a Intel
member https://github.com/intel/mkl-dnn/issues/102.

IMHO this is not a good practice, I also wish that a discussion happen on
dev list before we drop support, so more people provide feedback.



On Wed, May 16, 2018 at 11:54 AM, Anirudh <an...@gmail.com> wrote:

> Hi Naveen,
>
> There was an earlier flag for using MKLML which was a subset of MKL. I
> think this flag was removed when MKLDNN was added.
> I think the only replacement to the old MKLML is the MKLDNN feature that
> was added. ( I am assuming this also installs the MKLML)
> So if users don't want to use MKLDNN feature they will have to install the
> full MKL library and set USE_BLAS=mkl if I am not wrong.
>
> I would also like to understand more from others about what is the best way
> to transition for users who were using MKLML flag earlier but don't want to
> use
> MKLDNN flag now ? (Both flags were experimental, but MKLML flag had been
> around for some time.)
>
> Anirudh
>
>
> On Wed, May 16, 2018 at 11:38 AM, Naveen Swamy <mn...@gmail.com> wrote:
>
> > How do I build the latest release(1.2.0rc3) with MKL enabled, it looks
> like
> > its now defaulting to the experimental MKLDNN. I do not want to MKLDNN as
> > it is experimental.
> >
> > -Naveen
> >
>

Re: MKL with 1.2.0rc3

[Anirudh <an...@gmail.com>]

Hi Naveen,

There was an earlier flag for using MKLML which was a subset of MKL. I
think this flag was removed when MKLDNN was added.
I think the only replacement to the old MKLML is the MKLDNN feature that
was added. ( I am assuming this also installs the MKLML)
So if users don't want to use MKLDNN feature they will have to install the
full MKL library and set USE_BLAS=mkl if I am not wrong.

I would also like to understand more from others about what is the best way
to transition for users who were using MKLML flag earlier but don't want to
use
MKLDNN flag now ? (Both flags were experimental, but MKLML flag had been
around for some time.)

Anirudh


On Wed, May 16, 2018 at 11:38 AM, Naveen Swamy <mn...@gmail.com> wrote:

> How do I build the latest release(1.2.0rc3) with MKL enabled, it looks like
> its now defaulting to the experimental MKLDNN. I do not want to MKLDNN as
> it is experimental.
>
> -Naveen
>