On 2021/9/6 10:03, Xia, Chenbo wrote: > Hi, > >> -----Original Message----- >> From: Maxime Coquelin <[email protected]> >> Sent: Friday, September 3, 2021 8:59 PM >> To: fengchengwen <[email protected]>; Richardson, Bruce >> <[email protected]>; Li, Xiaoyun <[email protected]> >> Cc: [email protected]; Yigit, Ferruh <[email protected]>; >> [email protected]; [email protected]; [email protected]; >> [email protected]; [email protected]; [email protected]; >> [email protected]; [email protected]; >> [email protected]; [email protected]; [email protected]; Ananyev, >> Konstantin <[email protected]>; Walsh, Conor >> <[email protected]>; Xia, Chenbo <[email protected]> >> Subject: Re: [PATCH v18 8/8] maintainers: add for dmadev >> >> Hi, >> >> On 9/2/21 3:39 PM, fengchengwen wrote: >>> 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! >> >>> 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. >> >>> testpmd: will introduces some dma control commands and the dma forwarding >> mode. >>> 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. >> >> We might also think of adding async support to Vhost PMD, that would be >> another way to test dmadev with testpmd. > > +1 for this. I would prefer support in vhost pmd so that it could be tested > with > testpmd. > > About the testpmd patches, if it's for testing dma device itself, usually we > create a new > app (now testpmd only tests ethdev). If vhost pmd also needs some user input, > just > make it a devarg. Make sense to you?
Got it, thanks > > Thanks, > Chenbo > >> >> Thanks, >> Maxime >> >>> >>> On 2021/9/2 19:51, Bruce Richardson wrote: >>>> On Thu, Sep 02, 2021 at 06:54:17PM +0800, Chengwen Feng wrote: >>>>> This patch add myself as dmadev's maintainer and update release notes. >>>>> >>>>> Signed-off-by: Chengwen Feng <[email protected]> --- >>>> >>>> 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. >>>> . >>>> >>> >
