DPDK patches and discussions
 help / color / mirror / Atom feed
From: Jerin Jacob <jerin.jacob@caviumnetworks.com>
To: Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>
Cc: "Van Haaren, Harry" <harry.van.haaren@intel.com>,
	"dev@dpdk.org" <dev@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH] doc/event: improve eventdev library documentation
Date: Wed, 6 Jun 2018 05:15:05 +0530	[thread overview]
Message-ID: <20180605234503.GA5277@jerin> (raw)
In-Reply-To: <HE1PR0801MB19306F1C880D003C5F0CB17898660@HE1PR0801MB1930.eurprd08.prod.outlook.com>

-----Original Message-----
> Date: Tue, 5 Jun 2018 20:32:39 +0000
> From: Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>
> To: "Van Haaren, Harry" <harry.van.haaren@intel.com>, Jerin Jacob
>  <jerin.jacob@caviumnetworks.com>
> CC: "dev@dpdk.org" <dev@dpdk.org>
> Subject: RE: [PATCH] doc/event: improve eventdev library documentation
> 
> 
> 
> -----Original Message-----
> From: Van Haaren, Harry <harry.van.haaren@intel.com>
> Sent: Tuesday, June 5, 2018 11:31 AM
> To: Jerin Jacob <jerin.jacob@caviumnetworks.com>; Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>
> Cc: dev@dpdk.org
> Subject: RE: [PATCH] doc/event: improve eventdev library documentation
> 
> > From: Jerin Jacob [mailto:jerin.jacob@caviumnetworks.com]
> > Sent: Monday, June 4, 2018 5:42 AM
> > To: Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>
> > Cc: dev@dpdk.org; Van Haaren, Harry <harry.van.haaren@intel.com>
> > Subject: Re: [PATCH] doc/event: improve eventdev library documentation
> >
> > -----Original Message-----
> > > Date: Thu, 31 May 2018 15:23:42 -0500
> > > From: Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>
> > > To: jerin.jacob@caviumnetworks.com
> > > CC: dev@dpdk.org, Honnappa Nagarahalli
> > > <honnappa.nagarahalli@arm.com>
> > > Subject: [PATCH] doc/event: improve eventdev library documentation
> > > X-Mailer: git-send-email 2.7.4
> > >
> > > Add small amount of additional code, use consistent variable names
> > > across code blocks, change the image to represent queues and CPU
> > > cores intuitively. These help improve the eventdev library
> > > documentation.
> > >
> > > Signed-off-by: Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>
> > > Reviewed-by: Gavin Hu <gavin.hu@arm.com>
> > > ---
> >
> > Changes looks good to me.
> >
> > Adding Harry for this review as he is the original author of this file.
> 
> Thanks Jerin,
> 
> >> uint16_t nb_rx = rte_event_dequeue_burst(dev_id, worker_port_id1,
> >> events, BATCH_SIZE, timeout);
> 
> why is the worker port variable named "worker_port_id1"? Previously it was just worker_port_id.
> 
> I think just worker_port_id makes more sense, perhaps update the 2 occurances of id1 ?
> 
> Will change in V2
> 
> With that you have my
> Acked-by: Harry van Haaren <harry.van.haaren@intel.com>
> 
> 
> 
> > >  doc/guides/prog_guide/eventdev.rst           |   55 +-
> > >  doc/guides/prog_guide/img/eventdev_usage.svg | 1518
> > > +++++++++------------
> > -----
> > >  2 files changed, 570 insertions(+), 1003 deletions(-)
> > >
> > > diff --git a/doc/guides/prog_guide/eventdev.rst
> > b/doc/guides/prog_guide/eventdev.rst
> > > index ce19997..0203d9e 100644
> > > --- a/doc/guides/prog_guide/eventdev.rst
> > > +++ b/doc/guides/prog_guide/eventdev.rst
> > > @@ -1,5 +1,6 @@
> > >  ..  SPDX-License-Identifier: BSD-3-Clause
> > >      Copyright(c) 2017 Intel Corporation.
> > > +    Copyright(c) 2018 Arm Limited.
> > >
> > >  Event Device Library
> > >  ====================
> > > @@ -129,7 +130,7 @@ API Walk-through
> > >
> > >  This section will introduce the reader to the eventdev API, showing
> > > how
> > to
> > >  create and configure an eventdev and use it for a two-stage atomic
> > pipeline
> > > -with a single core for TX. The diagram below shows the final state
> > > of the
> > > +with one core each for RX and TX. The diagram below shows the final
> > > +state
> > of the
> >
> > I think, we can mention the usage of RX and TX core are API
> > illustration purpose only, In the real case, it is abstracted using
> > Eventdev - Ethdev adapters to hide the difference between various
> > eventdev capabilities to Rx or Tx the packets.
> > or something on similar lines.
> >
> 
> I think we should leave it as is currently, the code under the section 'Ingress of New Events' is tied into RX core. I think the API walk-through section as a whole has to change to introduce the usage of adaptors.

We already have Rx adapter documentation at
http://dpdk.org/doc/guides/prog_guide/event_ethernet_rx_adapter.html
My concern is reader gets confused about Rx core and think That way
to send the packets in eventdev from ethdev. I think, it will help reader if give
pointers about adapter here and mention the Rx and Tx core usage as
API illustration purpose only to clarify the existing architecture.


> 
> > >
> > >
> > >  Egress of Events
> > > diff --git a/doc/guides/prog_guide/img/eventdev_usage.svg
> > b/doc/guides/prog_guide/img/eventdev_usage.svg
> > > index 7765649..b0792dc 100644
> > > --- a/doc/guides/prog_guide/img/eventdev_usage.svg
> > > +++ b/doc/guides/prog_guide/img/eventdev_usage.svg
> > > @@ -1,994 +1,546 @@
> > >  <?xml version="1.0" encoding="UTF-8" standalone="no"?>
> >
> > License is missing
> >
> > see doc/guides/prog_guide/img/architecture-overview.svg file as reference.
> >
> > With above changes:
> 
> Will add in V2
> 
> >
> > Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
> 
> IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

  reply	other threads:[~2018-06-05 23:45 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-05-31 20:23 Honnappa Nagarahalli
2018-06-04  4:42 ` Jerin Jacob
2018-06-05 16:31   ` Van Haaren, Harry
2018-06-05 20:32     ` Honnappa Nagarahalli
2018-06-05 23:45       ` Jerin Jacob [this message]
2018-06-06  1:27         ` Honnappa Nagarahalli
2018-06-06  1:44           ` Honnappa Nagarahalli

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180605234503.GA5277@jerin \
    --to=jerin.jacob@caviumnetworks.com \
    --cc=Honnappa.Nagarahalli@arm.com \
    --cc=dev@dpdk.org \
    --cc=harry.van.haaren@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).