mbox series

[v7,0/8] Add Marvell NETA PMD

Message ID 1538551336-19259-1-git-send-email-andrzej.ostruszka@gmail.com (mailing list archive)
Headers
Series Add Marvell NETA PMD |

Message

Andrzej Ostruszka Oct. 3, 2018, 7:22 a.m. UTC
  From: Andrzej Ostruszka <amo@semihalf.com>

This patch series introduces new PMD for Marvell NETA adapters (MVNETA).
See the documentation for more info.

It is split for easier reviewing.

v7:
  Addressed last round of review comments:
  * improved documentation by listing it in toc, removing unsupported
    MUSDK option and documenting run time option
  * removed variables that are not really used
  * documented devargs via REGISTER_PARAM_STRING
  * implemented better separation between mvneta_ethdev.c and
    mvneta_rxtx.c (now mvneta_rxtx.h includes only prototypes of
    functions needed by mvneta_ethdev.c)

v6:
  * add missing call to rte_eth_dev_probing_finish()
  * update doc to point to MUSDK 18.09
  * add mvep lib to LDLIBS in mvneta Makefile

v5:
  * fixed wrong order of clenup in mvneta_eth_dev_create()
  * inlined one auxilary function (mvneta_priv_create())

v4:
  * rebased on top of next-net (DEV_RX_OFFLOAD_CRC_STRIP removed)
  * Rx/Tx functionality moved to new mvneta_rxtx.c file
  * removed eth_mvneta alias (and docs updated accordingly)
  * fixed additional review comments

v3:
    No changes against v2, just resubmitting again to have clean patch
    set after faulty format-patch.  My apologies for the noise.

v2:
  * fixed couple of checkpatch warnings
  * removed '\n' from MVNETA_LOG invocations (appended by the macro)
  * removed unused MVNETA_MUSDK_DMA_MEMSIZE define
  * changed one printf into MVNETA_LOG
  * removed __func__ from one MVNETA_LOG invocation (inserted
    automatically by the macro)
  * minor grammar/spelling correction in comments
  * removed license text from file with SPDX tag (mvneta.rst)
  * removed misleading part of comment for mvneta_shadow_txq
  * changed authorship of the patches to the original author

Natalie Samsonov (1):
  net/mvneta: add reset statistics callback

Zyta Szpak (7):
  net/mvneta: add neta PMD skeleton
  net/mvneta: add Rx/Tx support
  net/mvneta: support for setting of MTU
  net/mvneta: add link update
  net/mvneta: support for promiscuous
  net/mvneta: add MAC filtering
  net/mvneta: add support for basic stats

 MAINTAINERS                                   |    8 +
 config/common_base                            |    5 +
 devtools/test-build.sh                        |    2 +
 doc/guides/nics/features/mvneta.ini           |   19 +
 doc/guides/nics/index.rst                     |    1 +
 doc/guides/nics/mvneta.rst                    |  171 +++++
 doc/guides/rel_notes/release_18_11.rst        |    4 +
 drivers/common/Makefile                       |    4 +-
 drivers/common/mvep/rte_mvep_common.h         |    1 +
 drivers/net/Makefile                          |    1 +
 drivers/net/meson.build                       |    1 +
 drivers/net/mvneta/Makefile                   |   42 +
 drivers/net/mvneta/meson.build                |   28 +
 drivers/net/mvneta/mvneta_ethdev.c            |  993 ++++++++++++++++++++++++
 drivers/net/mvneta/mvneta_ethdev.h            |   79 ++
 drivers/net/mvneta/mvneta_rxtx.c              | 1021 +++++++++++++++++++++++++
 drivers/net/mvneta/mvneta_rxtx.h              |   32 +
 drivers/net/mvneta/rte_pmd_mvneta_version.map |    3 +
 mk/rte.app.mk                                 |    7 +-
 19 files changed, 2419 insertions(+), 3 deletions(-)
 create mode 100644 doc/guides/nics/features/mvneta.ini
 create mode 100644 doc/guides/nics/mvneta.rst
 create mode 100644 drivers/net/mvneta/Makefile
 create mode 100644 drivers/net/mvneta/meson.build
 create mode 100644 drivers/net/mvneta/mvneta_ethdev.c
 create mode 100644 drivers/net/mvneta/mvneta_ethdev.h
 create mode 100644 drivers/net/mvneta/mvneta_rxtx.c
 create mode 100644 drivers/net/mvneta/mvneta_rxtx.h
 create mode 100644 drivers/net/mvneta/rte_pmd_mvneta_version.map
  

Comments

Ferruh Yigit Oct. 3, 2018, 1:25 p.m. UTC | #1
On 10/3/2018 8:22 AM, Andrzej Ostruszka wrote:
> From: Andrzej Ostruszka <amo@semihalf.com>
> 
> This patch series introduces new PMD for Marvell NETA adapters (MVNETA).
> See the documentation for more info.
> 
> It is split for easier reviewing.
> 
> v7:
>   Addressed last round of review comments:
>   * improved documentation by listing it in toc, removing unsupported
>     MUSDK option and documenting run time option
>   * removed variables that are not really used
>   * documented devargs via REGISTER_PARAM_STRING
>   * implemented better separation between mvneta_ethdev.c and
>     mvneta_rxtx.c (now mvneta_rxtx.h includes only prototypes of
>     functions needed by mvneta_ethdev.c)
> 
> v6:
>   * add missing call to rte_eth_dev_probing_finish()
>   * update doc to point to MUSDK 18.09
>   * add mvep lib to LDLIBS in mvneta Makefile
> 
> v5:
>   * fixed wrong order of clenup in mvneta_eth_dev_create()
>   * inlined one auxilary function (mvneta_priv_create())
> 
> v4:
>   * rebased on top of next-net (DEV_RX_OFFLOAD_CRC_STRIP removed)
>   * Rx/Tx functionality moved to new mvneta_rxtx.c file
>   * removed eth_mvneta alias (and docs updated accordingly)
>   * fixed additional review comments
> 
> v3:
>     No changes against v2, just resubmitting again to have clean patch
>     set after faulty format-patch.  My apologies for the noise.
> 
> v2:
>   * fixed couple of checkpatch warnings
>   * removed '\n' from MVNETA_LOG invocations (appended by the macro)
>   * removed unused MVNETA_MUSDK_DMA_MEMSIZE define
>   * changed one printf into MVNETA_LOG
>   * removed __func__ from one MVNETA_LOG invocation (inserted
>     automatically by the macro)
>   * minor grammar/spelling correction in comments
>   * removed license text from file with SPDX tag (mvneta.rst)
>   * removed misleading part of comment for mvneta_shadow_txq
>   * changed authorship of the patches to the original author
> 
> Natalie Samsonov (1):
>   net/mvneta: add reset statistics callback
> 
> Zyta Szpak (7):
>   net/mvneta: add neta PMD skeleton
>   net/mvneta: add Rx/Tx support
>   net/mvneta: support for setting of MTU
>   net/mvneta: add link update
>   net/mvneta: support for promiscuous
>   net/mvneta: add MAC filtering
>   net/mvneta: add support for basic stats

Series applied to dpdk-next-net/master, thanks.
  
Ferruh Yigit Oct. 3, 2018, 7:46 p.m. UTC | #2
On 10/3/2018 2:25 PM, Ferruh Yigit wrote:
> On 10/3/2018 8:22 AM, Andrzej Ostruszka wrote:
>> From: Andrzej Ostruszka <amo@semihalf.com>
>>
>> This patch series introduces new PMD for Marvell NETA adapters (MVNETA).
>> See the documentation for more info.
>>
>> It is split for easier reviewing.
>>
>> v7:
>>   Addressed last round of review comments:
>>   * improved documentation by listing it in toc, removing unsupported
>>     MUSDK option and documenting run time option
>>   * removed variables that are not really used
>>   * documented devargs via REGISTER_PARAM_STRING
>>   * implemented better separation between mvneta_ethdev.c and
>>     mvneta_rxtx.c (now mvneta_rxtx.h includes only prototypes of
>>     functions needed by mvneta_ethdev.c)
>>
>> v6:
>>   * add missing call to rte_eth_dev_probing_finish()
>>   * update doc to point to MUSDK 18.09
>>   * add mvep lib to LDLIBS in mvneta Makefile
>>
>> v5:
>>   * fixed wrong order of clenup in mvneta_eth_dev_create()
>>   * inlined one auxilary function (mvneta_priv_create())
>>
>> v4:
>>   * rebased on top of next-net (DEV_RX_OFFLOAD_CRC_STRIP removed)
>>   * Rx/Tx functionality moved to new mvneta_rxtx.c file
>>   * removed eth_mvneta alias (and docs updated accordingly)
>>   * fixed additional review comments
>>
>> v3:
>>     No changes against v2, just resubmitting again to have clean patch
>>     set after faulty format-patch.  My apologies for the noise.
>>
>> v2:
>>   * fixed couple of checkpatch warnings
>>   * removed '\n' from MVNETA_LOG invocations (appended by the macro)
>>   * removed unused MVNETA_MUSDK_DMA_MEMSIZE define
>>   * changed one printf into MVNETA_LOG
>>   * removed __func__ from one MVNETA_LOG invocation (inserted
>>     automatically by the macro)
>>   * minor grammar/spelling correction in comments
>>   * removed license text from file with SPDX tag (mvneta.rst)
>>   * removed misleading part of comment for mvneta_shadow_txq
>>   * changed authorship of the patches to the original author
>>
>> Natalie Samsonov (1):
>>   net/mvneta: add reset statistics callback
>>
>> Zyta Szpak (7):
>>   net/mvneta: add neta PMD skeleton
>>   net/mvneta: add Rx/Tx support
>>   net/mvneta: support for setting of MTU
>>   net/mvneta: add link update
>>   net/mvneta: support for promiscuous
>>   net/mvneta: add MAC filtering
>>   net/mvneta: add support for basic stats
> 
> Series applied to dpdk-next-net/master, thanks.

For series,
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>