On Tue, Jul 06, 2021 at 11:56:03AM +0800, fengchengwen wrote: > Many thanks, mostly OK, a few comment inline > > On 2021/7/4 22:57, Andrew Rybchenko wrote: > > On 7/2/21 4:18 PM, Chengwen Feng wrote: > >> This patch introduces 'dmadevice' which is a generic type of DMA > >> device. <snip> > > Do we really need both stats and xstats from the very > > beginning? I think it is better to start from just > > generic stats and add xstats when it is really required. > > OK, but I think we should add one dump ops, which could be useful to > find the problem. > +1 to both suggestions - dropping xstats (for now) and adding dump fn.
- Re: [dpdk-dev] [PATCH] dmadev: introduce DMA dev... Jerin Jacob
- Re: [dpdk-dev] [PATCH] dmadev: introduce DMA... Bruce Richardson
- Re: [dpdk-dev] [PATCH] dmadev: introduce... Jerin Jacob
- Re: [dpdk-dev] [PATCH] dmadev: intr... Bruce Richardson
- Re: [dpdk-dev] [PATCH] dmadev: introduce DMA dev... fengchengwen
- Re: [dpdk-dev] [PATCH] dmadev: introduce DMA... Bruce Richardson
- Re: [dpdk-dev] [PATCH] dmadev: introduce DMA device ... fengchengwen
- Re: [dpdk-dev] [PATCH] dmadev: introduce DMA dev... Bruce Richardson
- Re: [dpdk-dev] [PATCH] dmadev: introduce DMA device libra... Andrew Rybchenko
- Re: [dpdk-dev] [PATCH] dmadev: introduce DMA device ... fengchengwen
- Re: [dpdk-dev] [PATCH] dmadev: introduce DMA dev... Bruce Richardson
- Re: [dpdk-dev] [PATCH] dmadev: introduce DMA device libra... Matan Azrad
- Re: [dpdk-dev] [PATCH] dmadev: introduce DMA device ... fengchengwen
- Re: [dpdk-dev] [PATCH] dmadev: introduce DMA dev... Matan Azrad
- Re: [dpdk-dev] [PATCH] dmadev: introduce DMA... fengchengwen
- Re: [dpdk-dev] [PATCH] dmadev: introduce... Matan Azrad
- [dpdk-dev] [RFC UPDATE PATCH 0/9] dmadev rfc suggested up... Bruce Richardson
- [dpdk-dev] [RFC UPDATE PATCH 2/9] dmadev: change vir... Bruce Richardson
- [dpdk-dev] [RFC UPDATE PATCH 1/9] dmadev: add missin... Bruce Richardson
- Re: [dpdk-dev] [RFC UPDATE PATCH 1/9] dmadev: ad... David Marchand
- Re: [dpdk-dev] [RFC UPDATE PATCH 1/9] dmadev... Bruce Richardson