DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] net/mrvl: update MUSDK library build instructions
@ 2018-01-26  8:02 Tomasz Duszynski
  2018-01-26 16:37 ` Ferruh Yigit
  2018-01-26 18:14 ` Ferruh Yigit
  0 siblings, 2 replies; 5+ messages in thread
From: Tomasz Duszynski @ 2018-01-26  8:02 UTC (permalink / raw)
  To: dev; +Cc: mw, dima, nsamsono, Jianbo.liu, Tomasz Duszynski

By default both static and shared libraries should be created while
building MUSDK library. It turns out that this will not happen if
host parameter is not explicitly passed to the configure script.

Specifying host makes sure configure will detect support for shared
libraries.

Signed-off-by: Tomasz Duszynski <tdu@semihalf.com>
---
 doc/guides/nics/mrvl.rst | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/doc/guides/nics/mrvl.rst b/doc/guides/nics/mrvl.rst
index 892097d..77b2d66 100644
--- a/doc/guides/nics/mrvl.rst
+++ b/doc/guides/nics/mrvl.rst
@@ -221,14 +221,13 @@ Usage example
 Building DPDK
 -------------
 
-Driver needs precompiled MUSDK library during compilation. Please consult
-``doc/musdk_get_started.txt`` for the detailed build instructions.
+Driver needs precompiled MUSDK library during compilation.
 
 .. code-block:: console
 
    export CROSS_COMPILE=<toolchain>/bin/aarch64-linux-gnu-
    ./bootstrap
-   ./configure --enable-bpool-dma=64
+   ./configure --host=aarch64-linux-gnu --enable-bpool-dma=64
    make install
 
 MUSDK will be installed to `usr/local` under current directory.
-- 
2.7.4

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH] net/mrvl: update MUSDK library build instructions
  2018-01-26  8:02 [dpdk-dev] [PATCH] net/mrvl: update MUSDK library build instructions Tomasz Duszynski
@ 2018-01-26 16:37 ` Ferruh Yigit
  2018-01-26 16:46   ` Tomasz Duszynski
  2018-01-26 18:14 ` Ferruh Yigit
  1 sibling, 1 reply; 5+ messages in thread
From: Ferruh Yigit @ 2018-01-26 16:37 UTC (permalink / raw)
  To: Tomasz Duszynski, dev; +Cc: mw, dima, nsamsono, Jianbo.liu

On 1/26/2018 8:02 AM, Tomasz Duszynski wrote:
> By default both static and shared libraries should be created while
> building MUSDK library. It turns out that this will not happen if
> host parameter is not explicitly passed to the configure script.
> 
> Specifying host makes sure configure will detect support for shared
> libraries.
> 
> Signed-off-by: Tomasz Duszynski <tdu@semihalf.com>
> ---
>  doc/guides/nics/mrvl.rst | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/doc/guides/nics/mrvl.rst b/doc/guides/nics/mrvl.rst
> index 892097d..77b2d66 100644
> --- a/doc/guides/nics/mrvl.rst
> +++ b/doc/guides/nics/mrvl.rst
> @@ -221,14 +221,13 @@ Usage example
>  Building DPDK
>  -------------
>  
> -Driver needs precompiled MUSDK library during compilation. Please consult
> -``doc/musdk_get_started.txt`` for the detailed build instructions.
> +Driver needs precompiled MUSDK library during compilation.
>  
>  .. code-block:: console
>  
>     export CROSS_COMPILE=<toolchain>/bin/aarch64-linux-gnu-
>     ./bootstrap
> -   ./configure --enable-bpool-dma=64
> +   ./configure --host=aarch64-linux-gnu --enable-bpool-dma=64

One more flag is required for crypto PMD, right? Is it documented somewhere?

>     make install
>  
>  MUSDK will be installed to `usr/local` under current directory.
> 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH] net/mrvl: update MUSDK library build instructions
  2018-01-26 16:37 ` Ferruh Yigit
@ 2018-01-26 16:46   ` Tomasz Duszynski
  2018-01-26 17:18     ` Ferruh Yigit
  0 siblings, 1 reply; 5+ messages in thread
From: Tomasz Duszynski @ 2018-01-26 16:46 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: Tomasz Duszynski, dev, mw, dima, nsamsono, Jianbo.liu

On Fri, Jan 26, 2018 at 04:37:27PM +0000, Ferruh Yigit wrote:
> On 1/26/2018 8:02 AM, Tomasz Duszynski wrote:
> > By default both static and shared libraries should be created while
> > building MUSDK library. It turns out that this will not happen if
> > host parameter is not explicitly passed to the configure script.
> >
> > Specifying host makes sure configure will detect support for shared
> > libraries.
> >
> > Signed-off-by: Tomasz Duszynski <tdu@semihalf.com>
> > ---
> >  doc/guides/nics/mrvl.rst | 5 ++---
> >  1 file changed, 2 insertions(+), 3 deletions(-)
> >
> > diff --git a/doc/guides/nics/mrvl.rst b/doc/guides/nics/mrvl.rst
> > index 892097d..77b2d66 100644
> > --- a/doc/guides/nics/mrvl.rst
> > +++ b/doc/guides/nics/mrvl.rst
> > @@ -221,14 +221,13 @@ Usage example
> >  Building DPDK
> >  -------------
> >
> > -Driver needs precompiled MUSDK library during compilation. Please consult
> > -``doc/musdk_get_started.txt`` for the detailed build instructions.
> > +Driver needs precompiled MUSDK library during compilation.
> >
> >  .. code-block:: console
> >
> >     export CROSS_COMPILE=<toolchain>/bin/aarch64-linux-gnu-
> >     ./bootstrap
> > -   ./configure --enable-bpool-dma=64
> > +   ./configure --host=aarch64-linux-gnu --enable-bpool-dma=64
>
> One more flag is required for crypto PMD, right? Is it documented somewhere?

You mean --enable-sam? It's mentioned in the crypto PMD docs.

>
> >     make install
> >
> >  MUSDK will be installed to `usr/local` under current directory.
> >
>

--
- Tomasz Duszyński

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH] net/mrvl: update MUSDK library build instructions
  2018-01-26 16:46   ` Tomasz Duszynski
@ 2018-01-26 17:18     ` Ferruh Yigit
  0 siblings, 0 replies; 5+ messages in thread
From: Ferruh Yigit @ 2018-01-26 17:18 UTC (permalink / raw)
  To: Tomasz Duszynski; +Cc: dev, mw, dima, nsamsono, Jianbo.liu

On 1/26/2018 4:46 PM, Tomasz Duszynski wrote:
> On Fri, Jan 26, 2018 at 04:37:27PM +0000, Ferruh Yigit wrote:
>> On 1/26/2018 8:02 AM, Tomasz Duszynski wrote:
>>> By default both static and shared libraries should be created while
>>> building MUSDK library. It turns out that this will not happen if
>>> host parameter is not explicitly passed to the configure script.
>>>
>>> Specifying host makes sure configure will detect support for shared
>>> libraries.
>>>
>>> Signed-off-by: Tomasz Duszynski <tdu@semihalf.com>
>>> ---
>>>  doc/guides/nics/mrvl.rst | 5 ++---
>>>  1 file changed, 2 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/doc/guides/nics/mrvl.rst b/doc/guides/nics/mrvl.rst
>>> index 892097d..77b2d66 100644
>>> --- a/doc/guides/nics/mrvl.rst
>>> +++ b/doc/guides/nics/mrvl.rst
>>> @@ -221,14 +221,13 @@ Usage example
>>>  Building DPDK
>>>  -------------
>>>
>>> -Driver needs precompiled MUSDK library during compilation. Please consult
>>> -``doc/musdk_get_started.txt`` for the detailed build instructions.
>>> +Driver needs precompiled MUSDK library during compilation.
>>>
>>>  .. code-block:: console
>>>
>>>     export CROSS_COMPILE=<toolchain>/bin/aarch64-linux-gnu-
>>>     ./bootstrap
>>> -   ./configure --enable-bpool-dma=64
>>> +   ./configure --host=aarch64-linux-gnu --enable-bpool-dma=64
>>
>> One more flag is required for crypto PMD, right? Is it documented somewhere?
> 
> You mean --enable-sam? It's mentioned in the crypto PMD docs.

Yes I was mentioning it, and confirmed that it is documented, thanks.

> 
>>
>>>     make install
>>>
>>>  MUSDK will be installed to `usr/local` under current directory.
>>>
>>
> 
> --
> - Tomasz Duszyński
> 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH] net/mrvl: update MUSDK library build instructions
  2018-01-26  8:02 [dpdk-dev] [PATCH] net/mrvl: update MUSDK library build instructions Tomasz Duszynski
  2018-01-26 16:37 ` Ferruh Yigit
@ 2018-01-26 18:14 ` Ferruh Yigit
  1 sibling, 0 replies; 5+ messages in thread
From: Ferruh Yigit @ 2018-01-26 18:14 UTC (permalink / raw)
  To: Tomasz Duszynski, dev; +Cc: mw, dima, nsamsono, Jianbo.liu

On 1/26/2018 8:02 AM, Tomasz Duszynski wrote:

   doc: update MUSDK library build instructions

> By default both static and shared libraries should be created while
> building MUSDK library. It turns out that this will not happen if
> host parameter is not explicitly passed to the configure script.
> 
> Specifying host makes sure configure will detect support for shared
> libraries.
> 
> Signed-off-by: Tomasz Duszynski <tdu@semihalf.com>

Applied to dpdk-next-net/master, thanks.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2018-01-26 18:14 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-01-26  8:02 [dpdk-dev] [PATCH] net/mrvl: update MUSDK library build instructions Tomasz Duszynski
2018-01-26 16:37 ` Ferruh Yigit
2018-01-26 16:46   ` Tomasz Duszynski
2018-01-26 17:18     ` Ferruh Yigit
2018-01-26 18:14 ` Ferruh Yigit

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).