DPDK patches and discussions
 help / color / mirror / Atom feed
* Re: [dpdk-dev] [PATCH v3 0/6] make dpdk iova aware
@ 2017-10-23 15:16 sshukla
  0 siblings, 0 replies; 5+ messages in thread
From: sshukla @ 2017-10-23 15:16 UTC (permalink / raw)
  To: Thomas Monjalon, Santosh Shukla
  Cc: dev, olivier.matz, jerin.jacob, hemant.agrawal, anatoly.burakov





Sent from my Samsung Galaxy smartphone.
-------- Original message --------From: Thomas Monjalon <thomas@monjalon.net> Date: 10/23/17  20:28  (GMT+05:30) To: Santosh Shukla <santosh.shukla@caviumnetworks.com> Cc: dev@dpdk.org, olivier.matz@6wind.com, jerin.jacob@caviumnetworks.com, hemant.agrawal@nxp.com, anatoly.burakov@intel.com Subject: Re: [dpdk-dev] [PATCH v3 0/6] make dpdk iova aware 
20/10/2017 14:31, Santosh Shukla:
> v3:
> Include v2 review comment and rebased on
> top of upstream tip commit:6b9ed026a8704
> 
> 
> v2:
> Include build fixes reported in patchworks.

Please fix checkpatch issues, thanks

Santosh>
[Excuse me for font and typo..Travelling and has limited access to internet]
IIRC, those checkpatch warning coming from legacy code base, not because of iova change.
Though I tried to address most in v2.. and left those which I wasn't quite sure..if you insist then will address them in v4 series.
Thanks

^ permalink raw reply	[flat|nested] 5+ messages in thread
* [dpdk-dev] [PATCH v2 0/5] make dpdk iova aware
@ 2017-09-05 10:31 Santosh Shukla
  2017-10-20 12:31 ` [dpdk-dev] [PATCH v3 0/6] " Santosh Shukla
  0 siblings, 1 reply; 5+ messages in thread
From: Santosh Shukla @ 2017-09-05 10:31 UTC (permalink / raw)
  To: dev; +Cc: thomas, jerin.jacob, hemant.agrawal, olivier.matz, Santosh Shukla

v2:
Include build fixes reported in patchworks.

Changeset based on deprecation notice[1], planned for v17.11 release.
Patches are based commit: 
(c42021fe56 : ethdev: rename map file to match library name)

Summary:
Renaming memory address translation api/ datatypes
and memory struct members to iova types.

1st patch : rename phys_addr_t to iova_addr_t
2nd patch : rename dma var mainly buf_physaddr to buf_iovaaddr
3rd patch : rename rte_memseg {.phys_addr} to {.iova_addr}.
4rd patch : rename memory translation api to _iova types.
5th patch : remove deprecation notice for dpdk iova aware.

v1 --> v2:
- Includes build fixes reported in v1 [2]
- aded separate patch for rte_memseg's phys_addr to iova_addr renaming.

Checkpatch warning:
- Noticed warning in changeset comining from legacy code. unreleated
  with iova changes.

Thanks.

[1]
http://dpdk.org/browse/dpdk/commit/doc/guides/rel_notes?id=caa570db61307e07efc461cf558ec291a3e71b29
[2] http://dpdk.org/ml/archives/test-report/2017-August/027020.html

Santosh Shukla (5):
  eal: rename phys_addr_t to iova_addr_t
  eal/memory: rename buf_physaddr to buf_iovaaddr
  eal/memory: rename memseg member phys to iova addr
  eal/memory: rename memory api to iova types
  doc: remove dpdk iova aware notice

 app/proc_info/main.c                               |  2 +-
 app/test-crypto-perf/cperf_test_vector_parsing.c   |  4 +--
 app/test-crypto-perf/cperf_test_vectors.c          |  6 ++--
 app/test-crypto-perf/cperf_test_vectors.h          |  4 +--
 app/test-pmd/cmdline.c                             |  2 +-
 doc/guides/contributing/documentation.rst          |  4 +--
 doc/guides/prog_guide/cryptodev_lib.rst            |  6 ++--
 doc/guides/prog_guide/img/mbuf1.svg                |  2 +-
 doc/guides/rel_notes/deprecation.rst               |  7 ----
 doc/guides/rel_notes/release_17_11.rst             | 27 ++++++++++++++
 drivers/bus/fslmc/fslmc_vfio.c                     |  2 +-
 drivers/bus/fslmc/portal/dpaa2_hw_pvt.h            | 20 +++++------
 drivers/crypto/qat/qat_adf/qat_algs.h              |  6 ++--
 drivers/crypto/qat/qat_crypto.h                    |  2 +-
 drivers/crypto/qat/qat_qp.c                        |  2 +-
 drivers/mempool/dpaa2/dpaa2_hw_mempool.h           |  2 +-
 drivers/net/ark/ark_ddm.c                          |  2 +-
 drivers/net/ark/ark_ddm.h                          |  4 +--
 drivers/net/ark/ark_ethdev_rx.c                    | 24 ++++++-------
 drivers/net/ark/ark_ethdev_tx.c                    |  6 ++--
 drivers/net/ark/ark_mpu.c                          |  2 +-
 drivers/net/ark/ark_mpu.h                          |  4 +--
 drivers/net/ark/ark_udm.c                          |  2 +-
 drivers/net/ark/ark_udm.h                          |  4 +--
 drivers/net/avp/avp_ethdev.c                       |  2 +-
 drivers/net/avp/rte_avp_common.h                   | 20 +++++------
 drivers/net/bnx2x/bnx2x.c                          | 40 ++++++++++-----------
 drivers/net/bnx2x/bnx2x.h                          | 22 ++++++------
 drivers/net/bnx2x/bnx2x_rxtx.c                     |  8 ++---
 drivers/net/bnx2x/bnx2x_stats.c                    |  2 +-
 drivers/net/bnx2x/bnx2x_vfpf.c                     |  2 +-
 drivers/net/bnx2x/ecore_sp.h                       |  2 +-
 drivers/net/bnxt/bnxt.h                            | 10 +++---
 drivers/net/bnxt/bnxt_cpr.h                        |  4 +--
 drivers/net/bnxt/bnxt_ethdev.c                     | 10 +++---
 drivers/net/bnxt/bnxt_hwrm.c                       | 14 ++++----
 drivers/net/bnxt/bnxt_ring.c                       |  6 ++--
 drivers/net/bnxt/bnxt_ring.h                       |  4 +--
 drivers/net/bnxt/bnxt_rxr.h                        |  4 +--
 drivers/net/bnxt/bnxt_txr.h                        |  2 +-
 drivers/net/bnxt/bnxt_vnic.c                       |  6 ++--
 drivers/net/bnxt/bnxt_vnic.h                       |  6 ++--
 drivers/net/cxgbe/sge.c                            |  4 +--
 drivers/net/e1000/em_rxtx.c                        |  4 +--
 drivers/net/e1000/igb_rxtx.c                       |  4 +--
 drivers/net/ena/ena_ethdev.c                       |  6 ++--
 drivers/net/enic/enic_main.c                       |  2 +-
 drivers/net/enic/enic_rxtx.c                       |  6 ++--
 drivers/net/fm10k/fm10k.h                          |  4 +--
 drivers/net/fm10k/fm10k_ethdev.c                   |  4 +--
 drivers/net/fm10k/fm10k_rxtx_vec.c                 |  4 +--
 drivers/net/i40e/i40e_ethdev.c                     |  2 +-
 drivers/net/i40e/i40e_fdir.c                       |  2 +-
 drivers/net/i40e/i40e_rxtx.c                       |  8 ++---
 drivers/net/i40e/i40e_rxtx_vec_altivec.c           |  4 +--
 drivers/net/i40e/i40e_rxtx_vec_neon.c              |  6 ++--
 drivers/net/i40e/i40e_rxtx_vec_sse.c               |  6 ++--
 drivers/net/ixgbe/ixgbe_rxtx.c                     |  4 +--
 drivers/net/ixgbe/ixgbe_rxtx_vec_neon.c            |  6 ++--
 drivers/net/ixgbe/ixgbe_rxtx_vec_sse.c             |  6 ++--
 drivers/net/liquidio/lio_rxtx.c                    |  4 +--
 drivers/net/liquidio/lio_rxtx.h                    |  4 +--
 drivers/net/mlx4/mlx4.c                            |  2 +-
 drivers/net/mlx5/mlx5_mr.c                         |  2 +-
 drivers/net/nfp/nfp_net.c                          |  2 +-
 drivers/net/qede/base/bcm_osal.h                   |  2 +-
 drivers/net/sfc/efsys.h                            |  2 +-
 drivers/net/sfc/sfc.c                              |  2 +-
 drivers/net/sfc/sfc_ef10_rx.c                      |  2 +-
 drivers/net/sfc/sfc_ef10_tx.c                      |  4 +--
 drivers/net/sfc/sfc_tso.c                          |  2 +-
 drivers/net/thunderx/base/nicvf_hw.c               |  2 +-
 drivers/net/thunderx/base/nicvf_hw.h               |  2 +-
 drivers/net/thunderx/base/nicvf_hw_defs.h          |  6 ++--
 drivers/net/thunderx/nicvf_ethdev.c                |  4 +--
 drivers/net/thunderx/nicvf_ethdev.h                |  4 +--
 drivers/net/thunderx/nicvf_struct.h                |  6 ++--
 drivers/net/virtio/virtio_ethdev.c                 |  2 +-
 drivers/net/virtio/virtio_rxtx.h                   |  4 +--
 drivers/net/virtio/virtqueue.h                     |  4 +--
 drivers/net/xenvirt/rte_eth_xenvirt.c              |  2 +-
 drivers/net/xenvirt/rte_mempool_gntalloc.c         |  6 ++--
 drivers/net/xenvirt/rte_xen_lib.c                  |  6 ++--
 drivers/net/xenvirt/rte_xen_lib.h                  |  8 ++---
 examples/l2fwd-crypto/main.c                       |  4 +--
 lib/librte_cryptodev/rte_crypto.h                  |  2 +-
 lib/librte_cryptodev/rte_crypto_sym.h              |  6 ++--
 lib/librte_cryptodev/rte_cryptodev.c               |  2 +-
 lib/librte_cryptodev/rte_cryptodev.h               |  2 +-
 lib/librte_eal/bsdapp/eal/eal.c                    |  2 +-
 lib/librte_eal/bsdapp/eal/eal_memory.c             |  8 ++---
 lib/librte_eal/bsdapp/eal/rte_eal_version.map      | 12 +++----
 lib/librte_eal/common/eal_common_memory.c          |  8 ++---
 lib/librte_eal/common/eal_common_memzone.c         |  4 +--
 lib/librte_eal/common/eal_private.h                |  2 +-
 lib/librte_eal/common/include/rte_malloc.h         |  4 +--
 lib/librte_eal/common/include/rte_memory.h         | 24 ++++++-------
 lib/librte_eal/common/include/rte_memzone.h        |  2 +-
 lib/librte_eal/common/rte_malloc.c                 |  9 ++---
 lib/librte_eal/linuxapp/eal/eal.c                  |  2 +-
 lib/librte_eal/linuxapp/eal/eal_memory.c           | 24 ++++++-------
 lib/librte_eal/linuxapp/eal/eal_pci.c              |  4 +--
 lib/librte_eal/linuxapp/eal/eal_vfio.c             | 10 +++---
 lib/librte_eal/linuxapp/eal/eal_xen_memory.c       |  4 +--
 .../linuxapp/eal/include/exec-env/rte_kni_common.h | 22 +++++++-----
 lib/librte_eal/linuxapp/eal/rte_eal_version.map    | 12 +++----
 lib/librte_eal/linuxapp/kni/kni_net.c              |  6 ++--
 lib/librte_kni/rte_kni.c                           |  2 +-
 lib/librte_mbuf/rte_mbuf.c                         |  6 ++--
 lib/librte_mbuf/rte_mbuf.h                         | 18 +++++-----
 lib/librte_mempool/rte_mempool.c                   | 42 +++++++++++-----------
 lib/librte_mempool/rte_mempool.h                   | 18 +++++-----
 lib/librte_mempool/rte_mempool_version.map         |  4 +--
 lib/librte_vhost/vhost.h                           |  2 +-
 lib/librte_vhost/vhost_user.c                      |  4 +--
 lib/librte_vhost/virtio_net.c                      |  2 +-
 test/test/commands.c                               |  2 +-
 test/test/test_cryptodev.h                         |  2 +-
 test/test/test_malloc.c                            |  4 +--
 test/test/test_mbuf.c                              |  2 +-
 test/test/test_memory.c                            |  6 ++--
 test/test/test_mempool.c                           |  4 +--
 test/test/test_memzone.c                           | 18 +++++-----
 123 files changed, 406 insertions(+), 381 deletions(-)

-- 
2.11.0

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

end of thread, other threads:[~2017-10-24  7:38 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-23 15:16 [dpdk-dev] [PATCH v3 0/6] make dpdk iova aware sshukla
  -- strict thread matches above, loose matches on Subject: below --
2017-09-05 10:31 [dpdk-dev] [PATCH v2 0/5] " Santosh Shukla
2017-10-20 12:31 ` [dpdk-dev] [PATCH v3 0/6] " Santosh Shukla
2017-10-23 14:58   ` Thomas Monjalon
2017-10-24  5:12     ` santosh
2017-10-24  7:38       ` Thomas Monjalon

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).