Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-23 Thread fengchengwen
On 2021/9/23 1:29, Jerin Jacob wrote: > On Wed, Sep 22, 2021 at 10:06 PM Bruce Richardson > wrote: >> >> On Wed, Sep 22, 2021 at 08:56:50AM +0100, Bruce Richardson wrote: >>> On Wed, Sep 22, 2021 at 09:51:42AM +0800, fengchengwen wrote: On 2021/9/22 2:11, Jerin Jacob wrote: > On Tue, Sep

Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-22 Thread Jerin Jacob
On Wed, Sep 22, 2021 at 10:06 PM Bruce Richardson wrote: > > On Wed, Sep 22, 2021 at 08:56:50AM +0100, Bruce Richardson wrote: > > On Wed, Sep 22, 2021 at 09:51:42AM +0800, fengchengwen wrote: > > > On 2021/9/22 2:11, Jerin Jacob wrote: > > > > On Tue, Sep 21, 2021 at 10:42 PM Pai G, Sunil > > >

Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-22 Thread Bruce Richardson
On Wed, Sep 22, 2021 at 08:56:50AM +0100, Bruce Richardson wrote: > On Wed, Sep 22, 2021 at 09:51:42AM +0800, fengchengwen wrote: > > On 2021/9/22 2:11, Jerin Jacob wrote: > > > On Tue, Sep 21, 2021 at 10:42 PM Pai G, Sunil > > > wrote: > > >> > > >> Hi Jerin, > > >> > > >> > > >> > > > To u

Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-22 Thread Bruce Richardson
On Wed, Sep 22, 2021 at 09:51:42AM +0800, fengchengwen wrote: > On 2021/9/22 2:11, Jerin Jacob wrote: > > On Tue, Sep 21, 2021 at 10:42 PM Pai G, Sunil wrote: > >> > >> Hi Jerin, > >> > >> > >> > > To understand it better, Could you share more details on feedback > > mechanism on your app

Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-21 Thread fengchengwen
On 2021/9/22 2:11, Jerin Jacob wrote: > On Tue, Sep 21, 2021 at 10:42 PM Pai G, Sunil wrote: >> >> Hi Jerin, >> >> >> > To understand it better, Could you share more details on feedback > mechanism on your application enqueue > > app_enqueue_v1(.., nb_seg) > { >

Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-21 Thread Jerin Jacob
On Tue, Sep 21, 2021 at 10:42 PM Pai G, Sunil wrote: > > Hi Jerin, > > > > > > > To understand it better, Could you share more details on feedback > > > > mechanism on your application enqueue > > > > > > > > app_enqueue_v1(.., nb_seg) > > > > { > > > > /* Not enough space, Let appli

Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-21 Thread Pai G, Sunil
Hi Jerin, > > > To understand it better, Could you share more details on feedback > > > mechanism on your application enqueue > > > > > > app_enqueue_v1(.., nb_seg) > > > { > > > /* Not enough space, Let application handle it by > > > dropping or resubmitting */ > > >

Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-21 Thread Jerin Jacob
On Tue, Sep 21, 2021 at 9:05 PM Pai G, Sunil wrote: > > Hi Jerin, > > > > > > > > The fact is that it's very hard for apps to calculate the > > > > > available space of a > > > > DMA ring. > > > > > > > > Yes, I agree. > > > > > > > > My question is more why to calculate the space per burst and

Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-21 Thread Pai G, Sunil
Hi Jerin, > > > > The fact is that it's very hard for apps to calculate the > > > > available space of a > > > DMA ring. > > > > > > Yes, I agree. > > > > > > My question is more why to calculate the space per burst and > > > introduce yet another fastpath API. > > > For example, the applicatio

Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-21 Thread Jerin Jacob
On Tue, Sep 21, 2021 at 7:46 PM Pai G, Sunil wrote: > > Hi Jerin, Hi Sunil, > > > > > > > From: Kevin Laatz <[2]kevin.la...@intel.com> > > > > > > Add a burst capacity check API to the dmadev library. This API > > > > > > is > > > > > > useful to > > > > > > applications wh

Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-21 Thread Pai G, Sunil
Hi Jerin, > > > > > From: Kevin Laatz <[2]kevin.la...@intel.com> > > > > > Add a burst capacity check API to the dmadev library. This API is > > > > > useful to > > > > > applications which need to how many descriptors can be enqueued > in > > > > > the > > > > > cur

Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-18 Thread Jerin Jacob
On Fri, Sep 17, 2021 at 8:07 PM Pai G, Sunil wrote: > > Hi Jerin, HI Sunil, > > > > > > > Add a burst capacity check API to the dmadev library. This API is > > > > useful to > > > > applications which need to how many descriptors can be enqueued in > > > > the > > > > c

Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-18 Thread Jerin Jacob
ngwen > > ; Jerin Jacob ; > > Satananda Burla ; Radha Mohan Chintakuntla > > ; Hu, Jiayu ; Pai G, Sunil > > > > Subject: Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API > > > > On Thu, Sep 9, 2021 at 1:46 PM Bruce Richardson > > wrote:

Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-17 Thread Hu, Jiayu
; Pai G, Sunil > > Subject: Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API > > On Thu, Sep 9, 2021 at 1:46 PM Bruce Richardson > wrote: > > > > On Wed, Sep 08, 2021 at 11:47:59PM +0530, Jerin Jacob wrote: > > >On Tue, 7 Sep 2021, 10:25 pm B

Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-17 Thread Pai G, Sunil
Hi Jerin, > > > Add a burst capacity check API to the dmadev library. This API is > > > useful to > > > applications which need to how many descriptors can be enqueued in > > > the > > > current batch. For example, it could be used to determine whether > > > all >

Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-17 Thread Jerin Jacob
On Thu, Sep 9, 2021 at 1:46 PM Bruce Richardson wrote: > > On Wed, Sep 08, 2021 at 11:47:59PM +0530, Jerin Jacob wrote: > >On Tue, 7 Sep 2021, 10:25 pm Bruce Richardson, > ><[1]bruce.richard...@intel.com> wrote: > > > > From: Kevin Laatz <[2]kevin.la...@intel.com> > > Add a burst

Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-09 Thread Bruce Richardson
On Wed, Sep 08, 2021 at 11:47:59PM +0530, Jerin Jacob wrote: >On Tue, 7 Sep 2021, 10:25 pm Bruce Richardson, ><[1]bruce.richard...@intel.com> wrote: > > From: Kevin Laatz <[2]kevin.la...@intel.com> > Add a burst capacity check API to the dmadev library. This API is > useful

Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-08 Thread Jerin Jacob
On Tue, 7 Sep 2021, 10:25 pm Bruce Richardson, wrote: > From: Kevin Laatz > > Add a burst capacity check API to the dmadev library. This API is useful to > applications which need to how many descriptors can be enqueued in the > current batch. For example, it could be used to determine whether a

Re: [dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-08 Thread Walsh, Conor
> Subject: [PATCH v3 2/8] dmadev: add burst capacity API > > From: Kevin Laatz > > Add a burst capacity check API to the dmadev library. This API is useful to > applications which need to how many descriptors can be enqueued in the > current batch. For example, it could be used to determine wh

[dpdk-dev] [PATCH v3 2/8] dmadev: add burst capacity API

2021-09-07 Thread Bruce Richardson
From: Kevin Laatz Add a burst capacity check API to the dmadev library. This API is useful to applications which need to how many descriptors can be enqueued in the current batch. For example, it could be used to determine whether all segments of a multi-segment packet can be enqueued in the same