DPDK patches and discussions
 help / color / mirror / Atom feed
From: Thomas Monjalon <thomas@monjalon.net>
To: Pavan Nikhilesh <pbhagavatula@marvell.com>,
	Jerin Jacob <jerinj@marvell.com>
Cc: "Gujjar, Abhinandan S" <abhinandan.gujjar@intel.com>,
	"Erik Gabriel Carrillo" <erik.g.carrillo@intel.com>,
	"Van Haaren, Harry" <harry.van.haaren@intel.com>,
	"Hemant Agrawal" <hemant.agrawal@nxp.com>,
	"McDaniel, Timothy" <timothy.mcdaniel@intel.com>,
	"Liang Ma" <liang.j.ma@intel.com>,
	"Jayatheerthan, Jay" <jay.jayatheerthan@intel.com>,
	dev@dpdk.org, "Ray Kinsella" <mdr@ashroe.eu>,
	"Mattias Rönnblom" <mattias.ronnblom@ericsson.com>,
	"Jerin Jacob" <jerinjacobk@gmail.com>
Subject: Re: [dpdk-dev] [PATCH v2] doc: announce changes to eventdev library
Date: Thu, 05 Aug 2021 16:22:13 +0200	[thread overview]
Message-ID: <1966296.uGCnGVyLxp@thomas> (raw)
In-Reply-To: <CALBAE1Negy7jcziw+6Z4KgEFsMvz5t9ct3Cj1XUJMfniNKzk_g@mail.gmail.com>

03/08/2021 06:12, Jerin Jacob:
> On Tue, Aug 3, 2021 at 2:46 AM <pbhagavatula@marvell.com> wrote:
> >
> > From: Pavan Nikhilesh <pbhagavatula@marvell.com>
> >
> > Make driver layer as internal, remove unnecessary rte_ prefix for
> > structures and functions that are not a part of public API.
> > Promote experimental trace and vector APIs to stable.
> > Add reserved field to `rte_event_timer` structure.
> >
> > Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
> 
> Acked-by: Jerin Jacob <jerinj@marvell.com>
> 
> 
> ++ Eventdev driver Maintainers.
> 
> This list is based on items identified for 21.11 ABI improvement at
> https://docs.google.com/spreadsheets/d/1betlC000ua5SsSiJIcC54mCCCJnW6voH5Dqv9UxeyfE/edit#gid=0

    Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
    Acked-by: Mattias Rönnblom <mattias.ronnblom@ericsson.com>
    Acked-by: Jay Jayatheerthan <jay.jayatheerthan@intel.com>
    Acked-by: Abhinandan Gujjar <abhinandan.gujjar@intel.com>


> > +* eventdev: The file ``rte_eventdev_pmd.h`` will be renamed to ``eventdev_driver.h``
> > +  to make the driver interface as internal and the structures ``rte_eventdev_data``,
> > +  ``rte_eventdev`` and ``rte_eventdevs`` will be moved to a new file named
> > +  ``rte_eventdev_core.h`` in DPDK 21.11.
> > +  The ``rte_`` prefix for internal structures and functions will be removed across the
> > +  library.

If a function is used outside of the library (in drivers),
it is better to keep rte_ prefix to avoid possible clash
with some driver dependencies.

> > +  The experimental eventdev trace APIs and ``rte_event_vector_pool_create``,
> > +  ``rte_event_eth_rx_adapter_vector_limits_get`` will be promoted to stable.
> > +  An 8byte reserved field will be added to the structure ``rte_event_timer`` to
> > +  support future extensions.

Applied, thanks.



      parent reply	other threads:[~2021-08-05 14:22 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-02 17:32 [dpdk-dev] [PATCH] " pbhagavatula
2021-08-02 21:09 ` [dpdk-dev] [PATCH v2] " pbhagavatula
2021-08-03  4:12   ` Jerin Jacob
2021-08-03  6:47     ` Hemant Agrawal
2021-08-03  8:32     ` Mattias Rönnblom
2021-08-04  5:57     ` Jayatheerthan, Jay
2021-08-04  6:06     ` Gujjar, Abhinandan S
2021-08-05 14:22     ` Thomas Monjalon [this message]

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=1966296.uGCnGVyLxp@thomas \
    --to=thomas@monjalon.net \
    --cc=abhinandan.gujjar@intel.com \
    --cc=dev@dpdk.org \
    --cc=erik.g.carrillo@intel.com \
    --cc=harry.van.haaren@intel.com \
    --cc=hemant.agrawal@nxp.com \
    --cc=jay.jayatheerthan@intel.com \
    --cc=jerinj@marvell.com \
    --cc=jerinjacobk@gmail.com \
    --cc=liang.j.ma@intel.com \
    --cc=mattias.ronnblom@ericsson.com \
    --cc=mdr@ashroe.eu \
    --cc=pbhagavatula@marvell.com \
    --cc=timothy.mcdaniel@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).