MKLDNN dynamically linked

[Alex Zai <az...@gmail.com>]

Currently in mxnet-mkl the libmxnet.so is dynamically linked to to
libmkldnn.so.0. This is known to cause some issues if the wrong version of
mkldnn is linked. Can we static link this file instead?

Alex

Re: MKLDNN dynamically linked

[Hagay Lupesko <lu...@gmail.com>]

+1
Kellen made a good call about watching out for the license. Not an issue
for MKL-DNN though, which I believe has an Apache 2 license.

On Thu, Nov 8, 2018 at 3:51 PM Zhao, Patric <pa...@intel.com> wrote:

> +1 for static link :)
>
> Feel free to let us know if anything we can help.
>
> > -----Original Message-----
> > From: kellen sunderland [mailto:kellen.sunderland@gmail.com]
> > Sent: Friday, November 9, 2018 7:30 AM
> > To: dev@mxnet.incubator.apache.org
> > Cc: dev@mxnet.apache.org
> > Subject: Re: MKLDNN dynamically linked
> >
> > I think we should bias towards static linking.  It should make using
> mxnet
> > easier in a lot of cases for users.  As long as the license permits
> static linking
> > (i.e. is non-gpl) I'd +1 static linking for portability and ease of
> use.  The only
> > caveat would be in cases where the package size would cause grief for
> PyPi
> > maintainers.
> >
> > On Thu, Nov 8, 2018, 3:54 PM Sheng Zha <zhasheng@apache.org wrote:
> >
> > > +1. Ideally, MKLDNN can be statically linked. mxnet-mkl relies on Make
> > > +for
> > > building it so help is wanted on mxnet.
> > >
> > > -sz
> > >
> > > On 2018/11/08 21:28:50, Alex Zai <az...@gmail.com> wrote:
> > > > Currently in mxnet-mkl the libmxnet.so is dynamically linked to to
> > > > libmkldnn.so.0. This is known to cause some issues if the wrong
> > > > version
> > > of
> > > > mkldnn is linked. Can we static link this file instead?
> > > >
> > > > Alex
> > > >
> > >
>

RE: MKLDNN dynamically linked

["Zhao, Patric" <pa...@intel.com>]

+1 for static link :)

Feel free to let us know if anything we can help.

> -----Original Message-----
> From: kellen sunderland [mailto:kellen.sunderland@gmail.com]
> Sent: Friday, November 9, 2018 7:30 AM
> To: dev@mxnet.incubator.apache.org
> Cc: dev@mxnet.apache.org
> Subject: Re: MKLDNN dynamically linked
> 
> I think we should bias towards static linking.  It should make using mxnet
> easier in a lot of cases for users.  As long as the license permits static linking
> (i.e. is non-gpl) I'd +1 static linking for portability and ease of use.  The only
> caveat would be in cases where the package size would cause grief for PyPi
> maintainers.
> 
> On Thu, Nov 8, 2018, 3:54 PM Sheng Zha <zhasheng@apache.org wrote:
> 
> > +1. Ideally, MKLDNN can be statically linked. mxnet-mkl relies on Make
> > +for
> > building it so help is wanted on mxnet.
> >
> > -sz
> >
> > On 2018/11/08 21:28:50, Alex Zai <az...@gmail.com> wrote:
> > > Currently in mxnet-mkl the libmxnet.so is dynamically linked to to
> > > libmkldnn.so.0. This is known to cause some issues if the wrong
> > > version
> > of
> > > mkldnn is linked. Can we static link this file instead?
> > >
> > > Alex
> > >
> >

Re: MKLDNN dynamically linked

[kellen sunderland <ke...@gmail.com>]

I think we should bias towards static linking.  It should make using mxnet
easier in a lot of cases for users.  As long as the license permits static
linking (i.e. is non-gpl) I'd +1 static linking for portability and ease of
use.  The only caveat would be in cases where the package size would cause
grief for PyPi maintainers.

On Thu, Nov 8, 2018, 3:54 PM Sheng Zha <zhasheng@apache.org wrote:

> +1. Ideally, MKLDNN can be statically linked. mxnet-mkl relies on Make for
> building it so help is wanted on mxnet.
>
> -sz
>
> On 2018/11/08 21:28:50, Alex Zai <az...@gmail.com> wrote:
> > Currently in mxnet-mkl the libmxnet.so is dynamically linked to to
> > libmkldnn.so.0. This is known to cause some issues if the wrong version
> of
> > mkldnn is linked. Can we static link this file instead?
> >
> > Alex
> >
>

Re: MKLDNN dynamically linked

[Sheng Zha <zh...@apache.org>]

+1. Ideally, MKLDNN can be statically linked. mxnet-mkl relies on Make for building it so help is wanted on mxnet.

-sz

On 2018/11/08 21:28:50, Alex Zai <az...@gmail.com> wrote: 
> Currently in mxnet-mkl the libmxnet.so is dynamically linked to to
> libmkldnn.so.0. This is known to cause some issues if the wrong version of
> mkldnn is linked. Can we static link this file instead?
> 
> Alex
>