Hi

-----Original Message-----
From: fengchengwen <redacted>
Sent: Saturday, September 4, 2021 15:02
To: Maxime Coquelin <redacted>; Richardson, Bruce
[off-list ref]; Li, Xiaoyun [off-list ref]
Cc: thomas@monjalon.net; Yigit, Ferruh <redacted>;
jerinj@marvell.com; jerinjacobk@gmail.com; andrew.rybchenko@oktetlabs.ru;
dev@dpdk.org; mb@smartsharesystems.com; nipun.gupta@nxp.com;
hemant.agrawal@nxp.com; honnappa.nagarahalli@arm.com;
david.marchand@redhat.com; sburla@marvell.com; pkapoor@marvell.com;
Ananyev, Konstantin [off-list ref]; Walsh, Conor
[off-list ref]; Xia, Chenbo [off-list ref]
Subject: Re: [PATCH v18 8/8] maintainers: add for dmadev

On 2021/9/3 20:59, Maxime Coquelin wrote:

quoted

Hi,

On 9/2/21 3:39 PM, fengchengwen wrote:

quoted

Fix in v19

I think there many patches wait for dmadev framework upstream, so
could you help review unreviewd patches (like dma/skeleton and app/test)?

Thanks for all the work, it looks really promising!

quoted

Also, we have developed the dma driver for hisilicon, and
corresponding test app (like examples/vhost and testpmd) is being developed.

examples/vhost: will extend support dmadev type.

I think Sunil has posted RFC for Vhost lib and example, you might want
to have a look to avoid duplicate work.

Got it, thanks

quoted

quoted

testpmd: will introduces some dma control commands and the dma

forwarding mode.

quoted

quoted

         the dma forwarding mode process:
              // 1st: call rte_eth_rx_burst
              // 2st: post received packet to dma, move data to another packet
              // 3st: set newcopyed rte_mbuf header
              // 4st: free received packet
              // 5st: get dma completed request and associate it rte_mbuf
              // 6st: send 5st's rte_mbuf to nic

@Maxime @Chenbo @Xiaoyun Like to hear your opinion.

What're some dma control commands? Can you give some examples here?

And the fwding mode in testpmd sounds like a function test to verify functions of dmadev.
Why don't you just test in app/test like test_eventdev or test_cryptodev? What's the purpose of this dma forwarding mode since it's just dma-copy from one mbuf to another?

quoted

We might also think of adding async support to Vhost PMD, that would
be another way to test dmadev with testpmd.

+1 on this. This makes more sense.

OK

quoted

Thanks,
Maxime

quoted

On 2021/9/2 19:51, Bruce Richardson wrote:

quoted

On Thu, Sep 02, 2021 at 06:54:17PM +0800, Chengwen Feng wrote:

quoted

This patch add myself as dmadev's maintainer and update release notes.

Signed-off-by: Chengwen Feng <redacted> ---

Just in case you are doing any further revisions of this patchset,
the maintainers entry, and RN entry, generally is added in the first
patch, so squash this patch into patch #1.
.

.