* 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* [dpdk-dev] [PATCH v3 0/6] make dpdk iova aware
2017-09-05 10:31 [dpdk-dev] [PATCH v2 0/5] " Santosh Shukla
@ 2017-10-20 12:31 ` Santosh Shukla
2017-10-23 14:58 ` Thomas Monjalon
0 siblings, 1 reply; 5+ messages in thread
From: Santosh Shukla @ 2017-10-20 12:31 UTC (permalink / raw)
To: dev
Cc: olivier.matz, thomas, jerin.jacob, hemant.agrawal,
anatoly.burakov, Santosh Shukla
v3:
Include v2 review comment and rebased on
top of upstream tip commit:6b9ed026a8704
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.
6th patch : Misc patch (Use RTE_PTR_DIFF in virt2iova())
Change History:
v2 --> v3:
- Replaced __KERNEL__ with __clang__ in kni header (by Anatoly)
- Added RTE_PTR_DIFF() change in rte_malloc_virt2iova() (Suggested by Anatolgy)
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 (6):
eal: rename phys addr to iova addr
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
eal/common/rte_malloc: use pointer diff in virt2iova
app/proc_info/main.c | 2 +-
app/test-crypto-perf/cperf_test_common.c | 6 ++--
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 | 30 +++++++++++++++-
drivers/bus/dpaa/base/qbman/qman.c | 2 +-
drivers/bus/dpaa/base/qbman/qman.h | 2 +-
drivers/bus/dpaa/rte_dpaa_bus.h | 4 +--
drivers/bus/fslmc/fslmc_vfio.c | 2 +-
drivers/bus/fslmc/mc/fsl_mc_cmd.h | 2 +-
drivers/bus/fslmc/portal/dpaa2_hw_pvt.h | 20 +++++------
drivers/crypto/dpaa_sec/dpaa_sec.c | 26 +++++++-------
drivers/crypto/qat/qat_adf/qat_algs.h | 6 ++--
drivers/crypto/qat/qat_crypto.c | 2 +-
drivers/crypto/qat/qat_crypto.h | 2 +-
drivers/crypto/qat/qat_qp.c | 2 +-
drivers/mempool/dpaa2/dpaa2_hw_mempool.h | 2 +-
drivers/mempool/octeontx/octeontx_fpavf.c | 4 +--
drivers/mempool/octeontx/rte_mempool_octeontx.c | 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 | 26 +++++++-------
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/dpaa/dpaa_rxtx.c | 6 ++--
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_rxtx_vec.c | 4 +--
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_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_mr.c | 2 +-
drivers/net/mlx5/mlx5_mr.c | 2 +-
drivers/net/nfp/nfp_net.c | 2 +-
drivers/net/octeontx/base/octeontx_pkovf.c | 4 +--
drivers/net/qede/base/bcm_osal.h | 2 +-
drivers/net/sfc/efsys.h | 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 +--
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 | 14 ++++----
lib/librte_eal/common/include/rte_memzone.h | 2 +-
lib/librte_eal/common/rte_malloc.c | 12 +++----
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 +++---
.../linuxapp/eal/include/exec-env/rte_kni_common.h | 23 ++++++++-----
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 | 38 ++++++++++----------
lib/librte_mempool/rte_mempool.h | 24 ++++++-------
lib/librte_mempool/rte_mempool_ops.c | 2 +-
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 +++++-----
122 files changed, 415 insertions(+), 389 deletions(-)
--
2.14.1
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [dpdk-dev] [PATCH v3 0/6] make dpdk iova aware
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
0 siblings, 1 reply; 5+ messages in thread
From: Thomas Monjalon @ 2017-10-23 14:58 UTC (permalink / raw)
To: Santosh Shukla
Cc: dev, olivier.matz, jerin.jacob, hemant.agrawal, anatoly.burakov
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [dpdk-dev] [PATCH v3 0/6] make dpdk iova aware
2017-10-23 14:58 ` Thomas Monjalon
@ 2017-10-24 5:12 ` santosh
2017-10-24 7:38 ` Thomas Monjalon
0 siblings, 1 reply; 5+ messages in thread
From: santosh @ 2017-10-24 5:12 UTC (permalink / raw)
To: Thomas Monjalon
Cc: dev, olivier.matz, jerin.jacob, hemant.agrawal, anatoly.burakov
Hi Thomas,
On Monday 23 October 2017 08:28 PM, Thomas Monjalon wrote:
> 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
>
Checkpatch warning coming from legacy code, not from iova change,
Same mentioned down in cover letter.
Will address them in v4.. After we conclude on your proposition on [5/6] patch of this series.
Thanks.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [dpdk-dev] [PATCH v3 0/6] make dpdk iova aware
2017-10-24 5:12 ` santosh
@ 2017-10-24 7:38 ` Thomas Monjalon
0 siblings, 0 replies; 5+ messages in thread
From: Thomas Monjalon @ 2017-10-24 7:38 UTC (permalink / raw)
To: santosh; +Cc: dev, olivier.matz, jerin.jacob, hemant.agrawal, anatoly.burakov
24/10/2017 07:12, santosh:
> Hi Thomas,
>
>
> On Monday 23 October 2017 08:28 PM, Thomas Monjalon wrote:
> > 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
> >
> Checkpatch warning coming from legacy code, not from iova change,
> Same mentioned down in cover letter.
> Will address them in v4.. After we conclude on your proposition on [5/6] patch of this series.
Sorry, I had overlooked checkpatch issues.
No need to fix the checkpatch issues if it makes code lines
formatted differently of their context.
^ 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).