diff mbox

[dpdk-dev,v2,1/4] app: link the whole rte_cfgfile library

Message ID 1506594158-15721-2-git-send-email-tdu@semihalf.com (mailing list archive)
State Superseded, archived
Delegated to: Ferruh Yigit
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Tomasz Duszynski Sept. 28, 2017, 10:22 a.m. UTC 
  MRVL net pmd needs rte_cfgfile to parse QoS configuration file thus
librte_pmd_mrvl.a contains undefined symbols from librte_cfgfile.a.

As a result linking applications under app/ directory will fail
because librte_cfgfile.a comes before librte_pmd_mrvl.a during
the linking stage.

Linking the whole librte_cfgfile.a solves the issue.

Signed-off-by: Jacek Siuda <jck@semihalf.com>
Signed-off-by: Tomasz Duszynski <tdu@semihalf.com>
---
 v2:
 * Changed commit message to explain problem better.

 mk/rte.app.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
2.7.4

Comments

Tomasz Duszynski Oct. 3, 2017, 11:51 a.m. UTC | #1 
Hello,

This patch series introduces the net driver for Marvell Armada 7k/8k
SoCs along with documentation.

Below you can find the list of features which net pmd supports:
* Speed capabilities
* Link status
* Queue start/stop
* MTU update
* Jumbo frame
* Promiscuous mode
* Allmulticast mode
* Unicast MAC filter
* Multicast MAC filter
* RSS hash
* VLAN filter
* CRC offload
* L3 checksum offload
* L4 checksum offload
* Packet type parsing
* Basic stats
* Stats per queue

Changes since v2:
* Removed LINE_SPACING, MULTILINE_DEREFERENCE and SPLIT_STRING
  checkpatch warnings.
* Removed unnecessary forward declarations.
* Fixed whitespace warnings.

Changes since v1:
* Changed commit message to explain problem better.
* Removed bunch of checkpatch warnings about unnecessary parentheses.

Tomasz Duszynski (4):
  app: link the whole rte_cfgfile library
  net/mrvl: add mrvl net pmd driver
  doc: add mrvl net pmd documentation
  maintainers: add maintainers for the mrvl net pmd

 MAINTAINERS                               |   10 +
 config/common_base                        |    7 +
 doc/guides/nics/features/mrvl.ini         |   24 +
 doc/guides/nics/index.rst                 |    1 +
 doc/guides/nics/mrvl.rst                  |  151 ++
 drivers/net/Makefile                      |    2 +
 drivers/net/mrvl/Makefile                 |   69 +
 drivers/net/mrvl/mrvl_ethdev.c            | 2277 +++++++++++++++++++++++++++++
 drivers/net/mrvl/mrvl_ethdev.h            |  114 ++
 drivers/net/mrvl/mrvl_qos.c               |  628 ++++++++
 drivers/net/mrvl/mrvl_qos.h               |  112 ++
 drivers/net/mrvl/rte_pmd_mrvl_version.map |    3 +
 mk/rte.app.mk                             |    3 +-
 13 files changed, 3400 insertions(+), 1 deletion(-)
 create mode 100644 doc/guides/nics/features/mrvl.ini
 create mode 100644 doc/guides/nics/mrvl.rst
 create mode 100644 drivers/net/mrvl/Makefile
 create mode 100644 drivers/net/mrvl/mrvl_ethdev.c
 create mode 100644 drivers/net/mrvl/mrvl_ethdev.h
 create mode 100644 drivers/net/mrvl/mrvl_qos.c
 create mode 100644 drivers/net/mrvl/mrvl_qos.h
 create mode 100644 drivers/net/mrvl/rte_pmd_mrvl_version.map

--
2.7.4
Tomasz Duszynski Oct. 9, 2017, 3 p.m. UTC | #2 
Hello,

This patch series introduces the net driver for Marvell Armada 7k/8k
SoCs along with documentation.

Below you can find the list of features which net pmd supports:
* Speed capabilities
* Link status
* MTU update
* Jumbo frame
* Promiscuous mode
* Allmulticast mode
* Unicast MAC filter
* Multicast MAC filter
* RSS hash
* VLAN filter
* CRC offload
* L3 checksum offload
* L4 checksum offload
* Packet type parsing
* Basic stats
* QoS

Changes since v3:
* Split driver into skeleton, rx/tx, features, documentation parts
* Added speed capabilities flags.
* Added missing rx offload flags: VLAN/JUMBOFRAME
* Updated release notes.
* Updated documentation.

Changes since v2:
* Removed LINE_SPACING, MULTILINE_DEREFERENCE and SPLIT_STRING
  checkpatch warnings.
* Removed unnecessary forward declarations.
* Fixed whitespace warnings.

Changes since v1:
* Changed commit message to explain problem better.
* Removed bunch of checkpatch warnings about unnecessary parentheses.

Tomasz Duszynski (4):
  app: link the whole rte_cfgfile library
  net/mrvl: add mrvl net pmd driver skeleton
  net/mrvl: add rx/tx support
  net/mrvl: add link update
  net/mrvl: add link speed capabilities
  net/mrvl: add support for updating mtu
  net/mrvl: add jumbo frame support
  net/mrvl: add support for promiscuous and allmulticast modes
  net/mrvl: add support for mac filtering
  net/mrvl: add rss hashing support
  net/mrvl: add support for vlan filtering
  net/mrvl: add crc, l3 and l4 offloads support
  net/mrvl: add packet type parsing support.
  net/mrvl: add basic stats support
  maintainers: add maintainers for the mrvl net pmd
  doc: add mrvl net pmd documentation

 MAINTAINERS                               |   10 +
 config/common_base                        |    7 +
 doc/guides/nics/features/mrvl.ini         |   23 +
 doc/guides/nics/index.rst                 |    1 +
 doc/guides/nics/mrvl.rst                  |  256 ++++
 doc/guides/rel_notes/release_17_11.rst    |    6 +
 drivers/net/Makefile                      |    8 +
 drivers/net/mrvl/Makefile                 |   63 +
 drivers/net/mrvl/mrvl_ethdev.c            | 2279 +++++++++++++++++++++++++++++
 drivers/net/mrvl/mrvl_ethdev.h            |  114 ++
 drivers/net/mrvl/mrvl_qos.c               |  633 ++++++++
 drivers/net/mrvl/mrvl_qos.h               |  112 ++
 drivers/net/mrvl/rte_pmd_mrvl_version.map |    3 +
 mk/rte.app.mk                             |    3 +-
 14 files changed, 3517 insertions(+), 1 deletion(-)
 create mode 100644 doc/guides/nics/features/mrvl.ini
 create mode 100644 doc/guides/nics/mrvl.rst
 create mode 100644 drivers/net/mrvl/Makefile
 create mode 100644 drivers/net/mrvl/mrvl_ethdev.c
 create mode 100644 drivers/net/mrvl/mrvl_ethdev.h
 create mode 100644 drivers/net/mrvl/mrvl_qos.c
 create mode 100644 drivers/net/mrvl/mrvl_qos.h
 create mode 100644 drivers/net/mrvl/rte_pmd_mrvl_version.map

--
2.7.4
Ferruh Yigit Oct. 9, 2017, 8:59 p.m. UTC | #3 
On 10/9/2017 4:00 PM, Tomasz Duszynski wrote:
> Hello,
> 
> This patch series introduces the net driver for Marvell Armada 7k/8k
> SoCs along with documentation.
> 
> Below you can find the list of features which net pmd supports:
> * Speed capabilities
> * Link status
> * MTU update
> * Jumbo frame
> * Promiscuous mode
> * Allmulticast mode
> * Unicast MAC filter
> * Multicast MAC filter
> * RSS hash
> * VLAN filter
> * CRC offload
> * L3 checksum offload
> * L4 checksum offload
> * Packet type parsing
> * Basic stats
> * QoS
> 
> Changes since v3:
> * Split driver into skeleton, rx/tx, features, documentation parts
> * Added speed capabilities flags.
> * Added missing rx offload flags: VLAN/JUMBOFRAME
> * Updated release notes.
> * Updated documentation.
> 
> Changes since v2:
> * Removed LINE_SPACING, MULTILINE_DEREFERENCE and SPLIT_STRING
>   checkpatch warnings.
> * Removed unnecessary forward declarations.
> * Fixed whitespace warnings.
> 
> Changes since v1:
> * Changed commit message to explain problem better.
> * Removed bunch of checkpatch warnings about unnecessary parentheses.
> 
> Tomasz Duszynski (4):
>   app: link the whole rte_cfgfile library
>   net/mrvl: add mrvl net pmd driver skeleton
>   net/mrvl: add rx/tx support
>   net/mrvl: add link update
>   net/mrvl: add link speed capabilities
>   net/mrvl: add support for updating mtu
>   net/mrvl: add jumbo frame support
>   net/mrvl: add support for promiscuous and allmulticast modes
>   net/mrvl: add support for mac filtering
>   net/mrvl: add rss hashing support
>   net/mrvl: add support for vlan filtering
>   net/mrvl: add crc, l3 and l4 offloads support
>   net/mrvl: add packet type parsing support.
>   net/mrvl: add basic stats support
>   maintainers: add maintainers for the mrvl net pmd
>   doc: add mrvl net pmd documentation

Series applied to dpdk-next-net/master, thanks.

(I can't compile because of missing musdk library, but since PMD is
disabled by default this is not so bad, lets get this for rc1 and for
crypto dependency, later I can have my environment set and test)
Ferruh Yigit Oct. 10, 2017, 12:25 a.m. UTC | #4 
On 10/9/2017 9:59 PM, Ferruh Yigit wrote:
> On 10/9/2017 4:00 PM, Tomasz Duszynski wrote:
>> Hello,
>>
>> This patch series introduces the net driver for Marvell Armada 7k/8k
>> SoCs along with documentation.
>>
>> Below you can find the list of features which net pmd supports:
>> * Speed capabilities
>> * Link status
>> * MTU update
>> * Jumbo frame
>> * Promiscuous mode
>> * Allmulticast mode
>> * Unicast MAC filter
>> * Multicast MAC filter
>> * RSS hash
>> * VLAN filter
>> * CRC offload
>> * L3 checksum offload
>> * L4 checksum offload
>> * Packet type parsing
>> * Basic stats
>> * QoS
>>
>> Changes since v3:
>> * Split driver into skeleton, rx/tx, features, documentation parts
>> * Added speed capabilities flags.
>> * Added missing rx offload flags: VLAN/JUMBOFRAME
>> * Updated release notes.
>> * Updated documentation.
>>
>> Changes since v2:
>> * Removed LINE_SPACING, MULTILINE_DEREFERENCE and SPLIT_STRING
>>   checkpatch warnings.
>> * Removed unnecessary forward declarations.
>> * Fixed whitespace warnings.
>>
>> Changes since v1:
>> * Changed commit message to explain problem better.
>> * Removed bunch of checkpatch warnings about unnecessary parentheses.
>>
>> Tomasz Duszynski (4):
>>   app: link the whole rte_cfgfile library
>>   net/mrvl: add mrvl net pmd driver skeleton
>>   net/mrvl: add rx/tx support
>>   net/mrvl: add link update
>>   net/mrvl: add link speed capabilities
>>   net/mrvl: add support for updating mtu
>>   net/mrvl: add jumbo frame support
>>   net/mrvl: add support for promiscuous and allmulticast modes
>>   net/mrvl: add support for mac filtering
>>   net/mrvl: add rss hashing support
>>   net/mrvl: add support for vlan filtering
>>   net/mrvl: add crc, l3 and l4 offloads support
>>   net/mrvl: add packet type parsing support.
>>   net/mrvl: add basic stats support
>>   maintainers: add maintainers for the mrvl net pmd
>>   doc: add mrvl net pmd documentation
> 
> Series applied to dpdk-next-net/master, thanks.

Can you also add a web patch [1] to list NICs as supported NICs [2] ?

[1]
http://dpdk.org/browse/tools/dpdk-web/

[2]
http://dpdk.org/doc/nics


> 
> (I can't compile because of missing musdk library, but since PMD is
> disabled by default this is not so bad, lets get this for rc1 and for
> crypto dependency, later I can have my environment set and test)
>
Tomasz Duszynski Oct. 10, 2017, 5:55 a.m. UTC | #5 
On Mon, Oct 09, 2017 at 09:59:27PM +0100, Ferruh Yigit wrote:
> On 10/9/2017 4:00 PM, Tomasz Duszynski wrote:
> > Hello,
> >
> > This patch series introduces the net driver for Marvell Armada 7k/8k
> > SoCs along with documentation.
> >
> > Below you can find the list of features which net pmd supports:
> > * Speed capabilities
> > * Link status
> > * MTU update
> > * Jumbo frame
> > * Promiscuous mode
> > * Allmulticast mode
> > * Unicast MAC filter
> > * Multicast MAC filter
> > * RSS hash
> > * VLAN filter
> > * CRC offload
> > * L3 checksum offload
> > * L4 checksum offload
> > * Packet type parsing
> > * Basic stats
> > * QoS
> >
> > Changes since v3:
> > * Split driver into skeleton, rx/tx, features, documentation parts
> > * Added speed capabilities flags.
> > * Added missing rx offload flags: VLAN/JUMBOFRAME
> > * Updated release notes.
> > * Updated documentation.
> >
> > Changes since v2:
> > * Removed LINE_SPACING, MULTILINE_DEREFERENCE and SPLIT_STRING
> >   checkpatch warnings.
> > * Removed unnecessary forward declarations.
> > * Fixed whitespace warnings.
> >
> > Changes since v1:
> > * Changed commit message to explain problem better.
> > * Removed bunch of checkpatch warnings about unnecessary parentheses.
> >
> > Tomasz Duszynski (4):
> >   app: link the whole rte_cfgfile library
> >   net/mrvl: add mrvl net pmd driver skeleton
> >   net/mrvl: add rx/tx support
> >   net/mrvl: add link update
> >   net/mrvl: add link speed capabilities
> >   net/mrvl: add support for updating mtu
> >   net/mrvl: add jumbo frame support
> >   net/mrvl: add support for promiscuous and allmulticast modes
> >   net/mrvl: add support for mac filtering
> >   net/mrvl: add rss hashing support
> >   net/mrvl: add support for vlan filtering
> >   net/mrvl: add crc, l3 and l4 offloads support
> >   net/mrvl: add packet type parsing support.
> >   net/mrvl: add basic stats support
> >   maintainers: add maintainers for the mrvl net pmd
> >   doc: add mrvl net pmd documentation
>
> Series applied to dpdk-next-net/master, thanks.
>
> (I can't compile because of missing musdk library, but since PMD is
> disabled by default this is not so bad, lets get this for rc1 and for
> crypto dependency, later I can have my environment set and test)

Thanks!

--
- Tomasz Duszyński
Tomasz Duszynski Oct. 10, 2017, 7:07 a.m. UTC | #6 
On Tue, Oct 10, 2017 at 01:25:42AM +0100, Ferruh Yigit wrote:
> On 10/9/2017 9:59 PM, Ferruh Yigit wrote:
> > On 10/9/2017 4:00 PM, Tomasz Duszynski wrote:
> >> Hello,
> >>
> >> This patch series introduces the net driver for Marvell Armada 7k/8k
> >> SoCs along with documentation.
> >>
> >> Below you can find the list of features which net pmd supports:
> >> * Speed capabilities
> >> * Link status
> >> * MTU update
> >> * Jumbo frame
> >> * Promiscuous mode
> >> * Allmulticast mode
> >> * Unicast MAC filter
> >> * Multicast MAC filter
> >> * RSS hash
> >> * VLAN filter
> >> * CRC offload
> >> * L3 checksum offload
> >> * L4 checksum offload
> >> * Packet type parsing
> >> * Basic stats
> >> * QoS
> >>
> >> Changes since v3:
> >> * Split driver into skeleton, rx/tx, features, documentation parts
> >> * Added speed capabilities flags.
> >> * Added missing rx offload flags: VLAN/JUMBOFRAME
> >> * Updated release notes.
> >> * Updated documentation.
> >>
> >> Changes since v2:
> >> * Removed LINE_SPACING, MULTILINE_DEREFERENCE and SPLIT_STRING
> >>   checkpatch warnings.
> >> * Removed unnecessary forward declarations.
> >> * Fixed whitespace warnings.
> >>
> >> Changes since v1:
> >> * Changed commit message to explain problem better.
> >> * Removed bunch of checkpatch warnings about unnecessary parentheses.
> >>
> >> Tomasz Duszynski (4):
> >>   app: link the whole rte_cfgfile library
> >>   net/mrvl: add mrvl net pmd driver skeleton
> >>   net/mrvl: add rx/tx support
> >>   net/mrvl: add link update
> >>   net/mrvl: add link speed capabilities
> >>   net/mrvl: add support for updating mtu
> >>   net/mrvl: add jumbo frame support
> >>   net/mrvl: add support for promiscuous and allmulticast modes
> >>   net/mrvl: add support for mac filtering
> >>   net/mrvl: add rss hashing support
> >>   net/mrvl: add support for vlan filtering
> >>   net/mrvl: add crc, l3 and l4 offloads support
> >>   net/mrvl: add packet type parsing support.
> >>   net/mrvl: add basic stats support
> >>   maintainers: add maintainers for the mrvl net pmd
> >>   doc: add mrvl net pmd documentation
> >
> > Series applied to dpdk-next-net/master, thanks.
>
> Can you also add a web patch [1] to list NICs as supported NICs [2] ?
>
> [1]
> http://dpdk.org/browse/tools/dpdk-web/
>
> [2]
> http://dpdk.org/doc/nics
>
>
> >
> > (I can't compile because of missing musdk library, but since PMD is
> > disabled by default this is not so bad, lets get this for rc1 and for
> > crypto dependency, later I can have my environment set and test)
> >
>

ACK

--
- Tomasz Duszyński
Ferruh Yigit Oct. 12, 2017, 1:51 a.m. UTC | #7 
On 10/9/2017 9:59 PM, Ferruh Yigit wrote:
> On 10/9/2017 4:00 PM, Tomasz Duszynski wrote:
>> Hello,
>>
>> This patch series introduces the net driver for Marvell Armada 7k/8k
>> SoCs along with documentation.
>>
>> Below you can find the list of features which net pmd supports:
>> * Speed capabilities
>> * Link status
>> * MTU update
>> * Jumbo frame
>> * Promiscuous mode
>> * Allmulticast mode
>> * Unicast MAC filter
>> * Multicast MAC filter
>> * RSS hash
>> * VLAN filter
>> * CRC offload
>> * L3 checksum offload
>> * L4 checksum offload
>> * Packet type parsing
>> * Basic stats
>> * QoS
>>
>> Changes since v3:
>> * Split driver into skeleton, rx/tx, features, documentation parts
>> * Added speed capabilities flags.
>> * Added missing rx offload flags: VLAN/JUMBOFRAME
>> * Updated release notes.
>> * Updated documentation.
>>
>> Changes since v2:
>> * Removed LINE_SPACING, MULTILINE_DEREFERENCE and SPLIT_STRING
>>   checkpatch warnings.
>> * Removed unnecessary forward declarations.
>> * Fixed whitespace warnings.
>>
>> Changes since v1:
>> * Changed commit message to explain problem better.
>> * Removed bunch of checkpatch warnings about unnecessary parentheses.
>>
>> Tomasz Duszynski (4):
>>   app: link the whole rte_cfgfile library
>>   net/mrvl: add mrvl net pmd driver skeleton
>>   net/mrvl: add rx/tx support
>>   net/mrvl: add link update
>>   net/mrvl: add link speed capabilities
>>   net/mrvl: add support for updating mtu
>>   net/mrvl: add jumbo frame support
>>   net/mrvl: add support for promiscuous and allmulticast modes
>>   net/mrvl: add support for mac filtering
>>   net/mrvl: add rss hashing support
>>   net/mrvl: add support for vlan filtering
>>   net/mrvl: add crc, l3 and l4 offloads support
>>   net/mrvl: add packet type parsing support.
>>   net/mrvl: add basic stats support
>>   maintainers: add maintainers for the mrvl net pmd
>>   doc: add mrvl net pmd documentation
> 
> Series applied to dpdk-next-net/master, thanks.
> 
> (I can't compile because of missing musdk library, but since PMD is
> disabled by default this is not so bad, lets get this for rc1 and for
> crypto dependency, later I can have my environment set and test)

Did able to compile but I have questions :)

1- Used the "arm64-armv8a-linuxapp-gcc" config, can you please confirm.
Does it make sense to document this?

2- I used different toolchain than documented in musdk, which uses a
marvel one. Can you please confirm any aarch64-linux-gnu-gcc is OK?

3- Used following command:
CROSS=<toolchain>/aarch64-linux-gnu- make
EXTRA_CFLAGS="-I.../musdk-marvell/src/include
-L.../musdk-marvell/src/.libs/"

Do you also need to use EXTRA_CFLAGS? Otherwise I can't compile. If you
also use it please document it, if not please share with us how to do?

4- musk generated a static library, can you please confirm there is a
way to generate a shared musdk library as well?

5- Still not tested building kernel modules, and building musdk with
./configure options, I will do later.

Thanks,
ferruh
Tomasz Duszynski Oct. 12, 2017, 6:07 a.m. UTC | #8 
On Thu, Oct 12, 2017 at 02:51:19AM +0100, Ferruh Yigit wrote:
> On 10/9/2017 9:59 PM, Ferruh Yigit wrote:
> > On 10/9/2017 4:00 PM, Tomasz Duszynski wrote:
> >> Hello,
> >>
> >> This patch series introduces the net driver for Marvell Armada 7k/8k
> >> SoCs along with documentation.
> >>
> >> Below you can find the list of features which net pmd supports:
> >> * Speed capabilities
> >> * Link status
> >> * MTU update
> >> * Jumbo frame
> >> * Promiscuous mode
> >> * Allmulticast mode
> >> * Unicast MAC filter
> >> * Multicast MAC filter
> >> * RSS hash
> >> * VLAN filter
> >> * CRC offload
> >> * L3 checksum offload
> >> * L4 checksum offload
> >> * Packet type parsing
> >> * Basic stats
> >> * QoS
> >>
> >> Changes since v3:
> >> * Split driver into skeleton, rx/tx, features, documentation parts
> >> * Added speed capabilities flags.
> >> * Added missing rx offload flags: VLAN/JUMBOFRAME
> >> * Updated release notes.
> >> * Updated documentation.
> >>
> >> Changes since v2:
> >> * Removed LINE_SPACING, MULTILINE_DEREFERENCE and SPLIT_STRING
> >>   checkpatch warnings.
> >> * Removed unnecessary forward declarations.
> >> * Fixed whitespace warnings.
> >>
> >> Changes since v1:
> >> * Changed commit message to explain problem better.
> >> * Removed bunch of checkpatch warnings about unnecessary parentheses.
> >>
> >> Tomasz Duszynski (4):
> >>   app: link the whole rte_cfgfile library
> >>   net/mrvl: add mrvl net pmd driver skeleton
> >>   net/mrvl: add rx/tx support
> >>   net/mrvl: add link update
> >>   net/mrvl: add link speed capabilities
> >>   net/mrvl: add support for updating mtu
> >>   net/mrvl: add jumbo frame support
> >>   net/mrvl: add support for promiscuous and allmulticast modes
> >>   net/mrvl: add support for mac filtering
> >>   net/mrvl: add rss hashing support
> >>   net/mrvl: add support for vlan filtering
> >>   net/mrvl: add crc, l3 and l4 offloads support
> >>   net/mrvl: add packet type parsing support.
> >>   net/mrvl: add basic stats support
> >>   maintainers: add maintainers for the mrvl net pmd
> >>   doc: add mrvl net pmd documentation
> >
> > Series applied to dpdk-next-net/master, thanks.
> >
> > (I can't compile because of missing musdk library, but since PMD is
> > disabled by default this is not so bad, lets get this for rc1 and for
> > crypto dependency, later I can have my environment set and test)
>
> Did able to compile but I have questions :)
>
> 1- Used the "arm64-armv8a-linuxapp-gcc" config, can you please confirm.
> Does it make sense to document this?

Right, that makes sense.

>
> 2- I used different toolchain than documented in musdk, which uses a
> marvel one. Can you please confirm any aarch64-linux-gnu-gcc is OK?
>

I am not sure that every toolchain out there will work but those
available here
https://releases.linaro.org/components/toolchain/binaries/*/aarch64-linux-gnu/
should work fine.

> 3- Used following command:
> CROSS=<toolchain>/aarch64-linux-gnu- make
> EXTRA_CFLAGS="-I.../musdk-marvell/src/include
> -L.../musdk-marvell/src/.libs/"
>
> Do you also need to use EXTRA_CFLAGS? Otherwise I can't compile. If you
> also use it please document it, if not please share with us how to do?

As for MUSDK I build it as follows:

export CROSS_COMPILE=/home/tdu/workspace/gcc-linaro-5.4.1-2017.01-x86_64_aarch64-linux-gnu/bin/aarch64-linux-gnu-

# in case of the first build
./bootstrap

# later on that will do
make clean
./configure \
	--host=aarch64-linux-gnu \
	--prefix=$(pwd)/musdk-install-dir \
	--enable-sam \
	--enable-bpool-dma=64 \
	--disable-shared

make -j8
make install

As for building DPDK that works for me:

export RTE_KERNELDIR=/home/tdu/workspace/tmp2/linux-marvell
export LIBMUSDK_PATH=/home/tdu/workspace/tmp2/musdk-marvell/musdk-install-dir
export CROSS=/home/tdu/workspace/tmp2/gcc-linaro-5.4.1-2017.01-x86_64_aarch64-linux-gnu/bin/aarch64-linux-gnu-

make -j8

>
> 4- musk generated a static library, can you please confirm there is a
> way to generate a shared musdk library as well?
>

To generate shared just remove `--disable-shared` from ./configure
command line. Then you will have both *.so and *.a generated.

> 5- Still not tested building kernel modules, and building musdk with
> ./configure options, I will do later.

Let me know in case you have any sort of difficulties or something is
unclear.

>
> Thanks,
> ferruh

--
- Tomasz Duszyński
diff mbox

Patch

diff --git a/mk/rte.app.mk b/mk/rte.app.mk
index c25fdd9..94568a8 100644
--- a/mk/rte.app.mk
+++ b/mk/rte.app.mk
@@ -81,10 +81,10 @@  _LDLIBS-$(CONFIG_RTE_LIBRTE_POWER)          += -lrte_power

 _LDLIBS-$(CONFIG_RTE_LIBRTE_TIMER)          += -lrte_timer
 _LDLIBS-$(CONFIG_RTE_LIBRTE_EFD)            += -lrte_efd
-_LDLIBS-$(CONFIG_RTE_LIBRTE_CFGFILE)        += -lrte_cfgfile

 _LDLIBS-y += --whole-archive

+_LDLIBS-$(CONFIG_RTE_LIBRTE_CFGFILE)        += -lrte_cfgfile
 _LDLIBS-$(CONFIG_RTE_LIBRTE_HASH)           += -lrte_hash
 _LDLIBS-$(CONFIG_RTE_LIBRTE_VHOST)          += -lrte_vhost
 _LDLIBS-$(CONFIG_RTE_LIBRTE_KVARGS)         += -lrte_kvargs