[dpdk-dev,3/3] doc: add event octeontx Rx adapter limitation

Message ID 1508230393-27018-3-git-send-email-pbhagavatula@caviumnetworks.com (mailing list archive)
State Superseded, archived
Delegated to: Jerin Jacob
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation fail apply patch file failure

Commit Message

Pavan Nikhilesh Oct. 17, 2017, 8:53 a.m. UTC
  Add limitaion when using eth_octeontx as Rx adapter with event_octeontx.

Signed-off-by: Pavan Nikhilesh <pbhagavatula@caviumnetworks.com>
---
 doc/guides/eventdevs/octeontx.rst | 5 +++++
 1 file changed, 5 insertions(+)
  

Comments

John McNamara Oct. 18, 2017, 8:48 a.m. UTC | #1
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Pavan Nikhilesh
> Sent: Tuesday, October 17, 2017 9:53 AM
> To: santosh.shukla@caviumnetworks.com; jerin.jacob@caviumnetworks.com
> Cc: dev@dpdk.org; Pavan Nikhilesh <pbhagavatula@caviumnetworks.com>
> Subject: [dpdk-dev] [PATCH 3/3] doc: add event octeontx Rx adapter
> limitation
> 
> Add limitaion when using eth_octeontx as Rx adapter with event_octeontx.
> 
> Signed-off-by: Pavan Nikhilesh <pbhagavatula@caviumnetworks.com>
> ---
>  doc/guides/eventdevs/octeontx.rst | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/doc/guides/eventdevs/octeontx.rst
> b/doc/guides/eventdevs/octeontx.rst
> index b43d515..4412bfa 100644
> --- a/doc/guides/eventdevs/octeontx.rst
> +++ b/doc/guides/eventdevs/octeontx.rst
> @@ -149,3 +149,8 @@ Burst mode support
>  Burst mode is not supported. Dequeue and Enqueue functions accepts only
> single  event at a time.
> 
> +Rx adapter support
> +~~~~~~~~~~~~~~~~~~
> +
> +When eth_octeontx is used as Rx adapter event schedule type
> +RTE_SCHED_TYPE_PARALLEL is not supported.

Note, for future patches, it is best to add fixed formatting to variable
names and #defines like this: `` RTE_SCHED_TYPE_PARALLEL``. That isn't
worth a respin though, so:

Acked-by: John McNamara <john.mcnamara@intel.com>
  
Pavan Nikhilesh Oct. 18, 2017, 9:06 a.m. UTC | #2
On Wed, Oct 18, 2017 at 08:48:57AM +0000, Mcnamara, John wrote:
>
>
> > -----Original Message-----
> > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Pavan Nikhilesh
> > Sent: Tuesday, October 17, 2017 9:53 AM
> > To: santosh.shukla@caviumnetworks.com; jerin.jacob@caviumnetworks.com
> > Cc: dev@dpdk.org; Pavan Nikhilesh <pbhagavatula@caviumnetworks.com>
> > Subject: [dpdk-dev] [PATCH 3/3] doc: add event octeontx Rx adapter
> > limitation
> >
> > Add limitaion when using eth_octeontx as Rx adapter with event_octeontx.
> >
> > Signed-off-by: Pavan Nikhilesh <pbhagavatula@caviumnetworks.com>
> > ---
> >  doc/guides/eventdevs/octeontx.rst | 5 +++++
> >  1 file changed, 5 insertions(+)
> >
> > diff --git a/doc/guides/eventdevs/octeontx.rst
> > b/doc/guides/eventdevs/octeontx.rst
> > index b43d515..4412bfa 100644
> > --- a/doc/guides/eventdevs/octeontx.rst
> > +++ b/doc/guides/eventdevs/octeontx.rst
> > @@ -149,3 +149,8 @@ Burst mode support
> >  Burst mode is not supported. Dequeue and Enqueue functions accepts only
> > single  event at a time.
> >
> > +Rx adapter support
> > +~~~~~~~~~~~~~~~~~~
> > +
> > +When eth_octeontx is used as Rx adapter event schedule type
> > +RTE_SCHED_TYPE_PARALLEL is not supported.
>
> Note, for future patches, it is best to add fixed formatting to variable
> names and #defines like this: `` RTE_SCHED_TYPE_PARALLEL``. That isn't
> worth a respin though, so:
>

Thanks for the headsup. Will take care while sending future patches.
-Pavan.

> Acked-by: John McNamara <john.mcnamara@intel.com>
>
>
  

Patch

diff --git a/doc/guides/eventdevs/octeontx.rst b/doc/guides/eventdevs/octeontx.rst
index b43d515..4412bfa 100644
--- a/doc/guides/eventdevs/octeontx.rst
+++ b/doc/guides/eventdevs/octeontx.rst
@@ -149,3 +149,8 @@  Burst mode support
 Burst mode is not supported. Dequeue and Enqueue functions accepts only single
 event at a time.
 
+Rx adapter support
+~~~~~~~~~~~~~~~~~~
+
+When eth_octeontx is used as Rx adapter event schedule type
+RTE_SCHED_TYPE_PARALLEL is not supported.