diff mbox series

doc: announce changes to event vector structure

Message ID 20220628112852.1397-1-pbhagavatula@marvell.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers
Series doc: announce changes to event vector structure |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/github-robot: build success github build: passed
ci/iol-aarch64-unit-testing success Testing PASS
ci/iol-x86_64-unit-testing success Testing PASS
ci/iol-x86_64-compile-testing success Testing PASS
ci/iol-aarch64-compile-testing success Testing PASS
ci/iol-abi-testing success Testing PASS

Commit Message

Pavan Nikhilesh Bhagavatula June 28, 2022, 11:28 a.m. UTC 
  From: Pavan Nikhilesh <pbhagavatula@marvell.com>

The structure ``rte_event_vector`` will be modified to include
``elem_offset:12`` bits taken from ``rsvd:15``.
The ``elem_offset`` defines the offset into the vector array from
which valid elements are present.

Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
---
 doc/guides/rel_notes/deprecation.rst | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Jerin Jacob July 11, 2022, 2:56 p.m. UTC | #1 
On Tue, Jun 28, 2022 at 5:01 PM <pbhagavatula@marvell.com> wrote:
>
> From: Pavan Nikhilesh <pbhagavatula@marvell.com>
>
> The structure ``rte_event_vector`` will be modified to include
> ``elem_offset:12`` bits taken from ``rsvd:15``.
> The ``elem_offset`` defines the offset into the vector array from
> which valid elements are present.
>
> Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>


Acked-by: Jerin Jacob <jerinj@marvell.com>


> ---
>  doc/guides/rel_notes/deprecation.rst | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index 4e5b23c53d..d7933629f2 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -125,3 +125,10 @@ Deprecation Notices
>    applications should be updated to use the ``dmadev`` library instead,
>    with the underlying HW-functionality being provided by the ``ioat`` or
>    ``idxd`` dma drivers
> +
> +* eventdev: The structure ``rte_event_vector`` will be modified to include
> +  ``elem_offset:12`` bits taken from ``rsvd:15``. The ``elem_offset`` defines
> +  the offset into the vector array from which valid elements are present.
> +  The difference between ``rte_event_vector::nb_elem`` and
> +  ``rte_event_vector::elem_offset`` gives the number of valid elements left to
> +  process from the ``rte_event_vector::elem_offset``.
> --
> 2.25.1
>
Thomas Monjalon July 12, 2022, 2:57 p.m. UTC | #2 
Tip to get more reviews: add interested maintainers in Cc.
Adding Jay, Erik, Abhinandan, Timothy, Hemant, Harry, Mattias and Liang.


11/07/2022 16:56, Jerin Jacob:
> On Tue, Jun 28, 2022 at 5:01 PM <pbhagavatula@marvell.com> wrote:
> >
> > From: Pavan Nikhilesh <pbhagavatula@marvell.com>
> >
> > The structure ``rte_event_vector`` will be modified to include
> > ``elem_offset:12`` bits taken from ``rsvd:15``.
> > The ``elem_offset`` defines the offset into the vector array from
> > which valid elements are present.
> >
> > Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
> 
> 
> Acked-by: Jerin Jacob <jerinj@marvell.com>
> 
> 
> > ---
> >  doc/guides/rel_notes/deprecation.rst | 7 +++++++
> >  1 file changed, 7 insertions(+)
> >
> > diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> > index 4e5b23c53d..d7933629f2 100644
> > --- a/doc/guides/rel_notes/deprecation.rst
> > +++ b/doc/guides/rel_notes/deprecation.rst
> > @@ -125,3 +125,10 @@ Deprecation Notices
> >    applications should be updated to use the ``dmadev`` library instead,
> >    with the underlying HW-functionality being provided by the ``ioat`` or
> >    ``idxd`` dma drivers
> > +
> > +* eventdev: The structure ``rte_event_vector`` will be modified to include
> > +  ``elem_offset:12`` bits taken from ``rsvd:15``. The ``elem_offset`` defines
> > +  the offset into the vector array from which valid elements are present.
> > +  The difference between ``rte_event_vector::nb_elem`` and
> > +  ``rte_event_vector::elem_offset`` gives the number of valid elements left to
> > +  process from the ``rte_event_vector::elem_offset``.
> > --
> > 2.25.1
> >
>
Timothy McDaniel July 12, 2022, 4:06 p.m. UTC | #3 
> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Tuesday, July 12, 2022 9:57 AM
> To: Pavan Nikhilesh <pbhagavatula@marvell.com>
> Cc: dev@dpdk.org; Jerin Jacob <jerinj@marvell.com>; Ray Kinsella
> <mdr@ashroe.eu>; Jerin Jacob <jerinjacobk@gmail.com>; Jayatheerthan, Jay
> <jay.jayatheerthan@intel.com>; Carrillo, Erik G <erik.g.carrillo@intel.com>;
> Gujjar, Abhinandan S <abhinandan.gujjar@intel.com>; McDaniel, Timothy
> <timothy.mcdaniel@intel.com>; hemant.agrawal@nxp.com; Van Haaren, Harry
> <harry.van.haaren@intel.com>; mattias.ronnblom
> <mattias.ronnblom@ericsson.com>; liangma@liangbit.com
> Subject: Re: [PATCH] doc: announce changes to event vector structure
> 
> Tip to get more reviews: add interested maintainers in Cc.
> Adding Jay, Erik, Abhinandan, Timothy, Hemant, Harry, Mattias and Liang.
> 
> 
> 11/07/2022 16:56, Jerin Jacob:
> > On Tue, Jun 28, 2022 at 5:01 PM <pbhagavatula@marvell.com> wrote:
> > >
> > > From: Pavan Nikhilesh <pbhagavatula@marvell.com>
> > >
> > > The structure ``rte_event_vector`` will be modified to include
> > > ``elem_offset:12`` bits taken from ``rsvd:15``.
> > > The ``elem_offset`` defines the offset into the vector array from
> > > which valid elements are present.
> > >
> > > Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
> >
> >
> > Acked-by: Jerin Jacob <jerinj@marvell.com>
> >
> >
> > > ---
> > >  doc/guides/rel_notes/deprecation.rst | 7 +++++++
> > >  1 file changed, 7 insertions(+)
> > >
> > > diff --git a/doc/guides/rel_notes/deprecation.rst
> b/doc/guides/rel_notes/deprecation.rst
> > > index 4e5b23c53d..d7933629f2 100644
> > > --- a/doc/guides/rel_notes/deprecation.rst
> > > +++ b/doc/guides/rel_notes/deprecation.rst
> > > @@ -125,3 +125,10 @@ Deprecation Notices
> > >    applications should be updated to use the ``dmadev`` library instead,
> > >    with the underlying HW-functionality being provided by the ``ioat`` or
> > >    ``idxd`` dma drivers
> > > +
> > > +* eventdev: The structure ``rte_event_vector`` will be modified to include
> > > +  ``elem_offset:12`` bits taken from ``rsvd:15``. The ``elem_offset`` defines
> > > +  the offset into the vector array from which valid elements are present.
> > > +  The difference between ``rte_event_vector::nb_elem`` and
> > > +  ``rte_event_vector::elem_offset`` gives the number of valid elements left
> to
> > > +  process from the ``rte_event_vector::elem_offset``.
> > > --
> > > 2.25.1
> > >

Looks good to me.

Acked-by: Timothy McDaniel <timothy.mcdaniel@intel.com>
Gujjar, Abhinandan S July 13, 2022, 6:53 a.m. UTC | #4 
> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Tuesday, July 12, 2022 8:27 PM
> To: Pavan Nikhilesh <pbhagavatula@marvell.com>
> Cc: dev@dpdk.org; Jerin Jacob <jerinj@marvell.com>; Ray Kinsella
> <mdr@ashroe.eu>; Jerin Jacob <jerinjacobk@gmail.com>; Jayatheerthan, Jay
> <jay.jayatheerthan@intel.com>; Carrillo, Erik G <erik.g.carrillo@intel.com>;
> Gujjar, Abhinandan S <abhinandan.gujjar@intel.com>; McDaniel, Timothy
> <timothy.mcdaniel@intel.com>; hemant.agrawal@nxp.com; Van Haaren, Harry
> <harry.van.haaren@intel.com>; mattias.ronnblom
> <mattias.ronnblom@ericsson.com>; liangma@liangbit.com
> Subject: Re: [PATCH] doc: announce changes to event vector structure
> 
> Tip to get more reviews: add interested maintainers in Cc.
> Adding Jay, Erik, Abhinandan, Timothy, Hemant, Harry, Mattias and Liang.
> 
> 
> 11/07/2022 16:56, Jerin Jacob:
> > On Tue, Jun 28, 2022 at 5:01 PM <pbhagavatula@marvell.com> wrote:
> > >
> > > From: Pavan Nikhilesh <pbhagavatula@marvell.com>
> > >
> > > The structure ``rte_event_vector`` will be modified to include
> > > ``elem_offset:12`` bits taken from ``rsvd:15``.
> > > The ``elem_offset`` defines the offset into the vector array from
> > > which valid elements are present.
> > >
> > > Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
> >
> >
> > Acked-by: Jerin Jacob <jerinj@marvell.com>
> >
> >
> > > ---
> > >  doc/guides/rel_notes/deprecation.rst | 7 +++++++
> > >  1 file changed, 7 insertions(+)
> > >
> > > diff --git a/doc/guides/rel_notes/deprecation.rst
> > > b/doc/guides/rel_notes/deprecation.rst
> > > index 4e5b23c53d..d7933629f2 100644
> > > --- a/doc/guides/rel_notes/deprecation.rst
> > > +++ b/doc/guides/rel_notes/deprecation.rst
> > > @@ -125,3 +125,10 @@ Deprecation Notices
> > >    applications should be updated to use the ``dmadev`` library instead,
> > >    with the underlying HW-functionality being provided by the ``ioat`` or
> > >    ``idxd`` dma drivers
> > > +
> > > +* eventdev: The structure ``rte_event_vector`` will be modified to
> > > +include
> > > +  ``elem_offset:12`` bits taken from ``rsvd:15``. The
> > > +``elem_offset`` defines
> > > +  the offset into the vector array from which valid elements are present.
> > > +  The difference between ``rte_event_vector::nb_elem`` and
> > > +  ``rte_event_vector::elem_offset`` gives the number of valid
> > > +elements left to
> > > +  process from the ``rte_event_vector::elem_offset``.
> > > --
> > > 2.25.1
> > >
> >
> 
Acked-by: Abhinandan Gujjar <abhinandan.gujjar@intel.com>
Thomas Monjalon July 17, 2022, 12:48 p.m. UTC | #5 
28/06/2022 13:28, pbhagavatula@marvell.com:
> From: Pavan Nikhilesh <pbhagavatula@marvell.com>
> 
> The structure ``rte_event_vector`` will be modified to include
> ``elem_offset:12`` bits taken from ``rsvd:15``.
> The ``elem_offset`` defines the offset into the vector array from
> which valid elements are present.
> 
> Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>

    Acked-by: Jerin Jacob <jerinj@marvell.com>
    Acked-by: Timothy McDaniel <timothy.mcdaniel@intel.com>
    Acked-by: Abhinandan Gujjar <abhinandan.gujjar@intel.com>

Applied, thanks.
diff mbox series

Patch

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 4e5b23c53d..d7933629f2 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -125,3 +125,10 @@  Deprecation Notices
   applications should be updated to use the ``dmadev`` library instead,
   with the underlying HW-functionality being provided by the ``ioat`` or
   ``idxd`` dma drivers
+
+* eventdev: The structure ``rte_event_vector`` will be modified to include
+  ``elem_offset:12`` bits taken from ``rsvd:15``. The ``elem_offset`` defines
+  the offset into the vector array from which valid elements are present.
+  The difference between ``rte_event_vector::nb_elem`` and
+  ``rte_event_vector::elem_offset`` gives the number of valid elements left to
+  process from the ``rte_event_vector::elem_offset``.