[v3] doc: announce change in event queue conf structure

Message ID d130273ad42ed5af264fe0509fae5b93eda68a76.1652610228.git.sthotton@marvell.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers
Series [v3] doc: announce change in event queue conf structure |

Checks

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

Commit Message

Shijith Thotton May 15, 2022, 10:24 a.m. UTC
  Structure rte_event_queue_conf will be extended to include fields to
support weight and affinity attribute. Once it gets added in DPDK 22.11,
eventdev internal op, queue_attr_get can be removed.

Signed-off-by: Shijith Thotton <sthotton@marvell.com>
---
 doc/guides/rel_notes/deprecation.rst | 3 +++
 1 file changed, 3 insertions(+)
  

Comments

Jerin Jacob July 12, 2022, 2:05 p.m. UTC | #1
On Sun, May 15, 2022 at 3:56 PM Shijith Thotton <sthotton@marvell.com> wrote:
>
> Structure rte_event_queue_conf will be extended to include fields to
> support weight and affinity attribute. Once it gets added in DPDK 22.11,
> eventdev internal op, queue_attr_get can be removed.
>
> Signed-off-by: Shijith Thotton <sthotton@marvell.com>


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

@Van Haaren, Harry  @Mattias Rönnblom  @Ray Kinsella  @Pavan Nikhilesh
Please Ack if you are OK.

> ---
>  doc/guides/rel_notes/deprecation.rst | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index 4e5b23c53d..04125db681 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -125,3 +125,6 @@ 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: New fields to represent event queue weight and affinity will be
> +  added to ``rte_event_queue_conf`` structure in DPDK 22.11.
> --
> 2.25.1
>
  
Pavan Nikhilesh Bhagavatula July 13, 2022, 6:52 a.m. UTC | #2
+Cc
timothy.mcdaniel@intel.com;
hemant.agrawal@nxp.com;
sachin.saxena@oss.nxp.com;
mattias.ronnblom@ericsson.com;
jerinj@marvell.com;
liangma@liangbit.com;
peter.mccarthy@intel.com;
harry.van.haaren@intel.com;
erik.g.carrillo@intel.com;
abhinandan.gujjar@intel.com;
jay.jayatheerthan@intel.com;
mdr@ashroe.eu;
anatoly.burakov@intel.com;


> -----Original Message-----
> From: Jerin Jacob <jerinjacobk@gmail.com>
> Sent: Tuesday, July 12, 2022 7:35 PM
> To: Shijith Thotton <sthotton@marvell.com>
> Cc: dpdk-dev <dev@dpdk.org>; Jerin Jacob Kollanukkaran
> <jerinj@marvell.com>; Pavan Nikhilesh Bhagavatula
> <pbhagavatula@marvell.com>; Van Haaren, Harry
> <harry.van.haaren@intel.com>; Mattias Rönnblom
> <mattias.ronnblom@ericsson.com>; Ray Kinsella <mdr@ashroe.eu>
> Subject: [EXT] Re: [PATCH v3] doc: announce change in event queue conf
> structure
> 
> External Email
> 
> ----------------------------------------------------------------------
> On Sun, May 15, 2022 at 3:56 PM Shijith Thotton <sthotton@marvell.com>
> wrote:
> >
> > Structure rte_event_queue_conf will be extended to include fields to
> > support weight and affinity attribute. Once it gets added in DPDK 22.11,
> > eventdev internal op, queue_attr_get can be removed.
> >
> > Signed-off-by: Shijith Thotton <sthotton@marvell.com>
> 
> 
> Acked-by: Jerin Jacob <jerinj@marvell.com>

Acked-by: Pavan Nikhilesh <pbhagavatula@marvell.com>

> @Van Haaren, Harry  @Mattias Rönnblom  @Ray Kinsella  @Pavan Nikhilesh
> Please Ack if you are OK.
> 
> > ---
> >  doc/guides/rel_notes/deprecation.rst | 3 +++
> >  1 file changed, 3 insertions(+)
> >
> > diff --git a/doc/guides/rel_notes/deprecation.rst
> b/doc/guides/rel_notes/deprecation.rst
> > index 4e5b23c53d..04125db681 100644
> > --- a/doc/guides/rel_notes/deprecation.rst
> > +++ b/doc/guides/rel_notes/deprecation.rst
> > @@ -125,3 +125,6 @@ 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: New fields to represent event queue weight and affinity will
> be
> > +  added to ``rte_event_queue_conf`` structure in DPDK 22.11.
> > --
> > 2.25.1
> >
  
Mattias Rönnblom July 13, 2022, 8:55 a.m. UTC | #3
On 2022-07-12 16:05, Jerin Jacob wrote:
> On Sun, May 15, 2022 at 3:56 PM Shijith Thotton <sthotton@marvell.com> wrote:
>>
>> Structure rte_event_queue_conf will be extended to include fields to
>> support weight and affinity attribute. Once it gets added in DPDK 22.11,
>> eventdev internal op, queue_attr_get can be removed.
>>
>> Signed-off-by: Shijith Thotton <sthotton@marvell.com>
> 
> 
> Acked-by: Jerin Jacob <jerinj@marvell.com>
> 
> @Van Haaren, Harry  @Mattias Rönnblom  @Ray Kinsella  @Pavan Nikhilesh
> Please Ack if you are OK.
> 

Will there be new capabilities to go with those new fields? To let the 
application know if they will be honored, or not.

Acked-by: Mattias Rönnblom <mattias.ronnblom@ericsson.com>

>> ---
>>   doc/guides/rel_notes/deprecation.rst | 3 +++
>>   1 file changed, 3 insertions(+)
>>
>> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
>> index 4e5b23c53d..04125db681 100644
>> --- a/doc/guides/rel_notes/deprecation.rst
>> +++ b/doc/guides/rel_notes/deprecation.rst
>> @@ -125,3 +125,6 @@ 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: New fields to represent event queue weight and affinity will be
>> +  added to ``rte_event_queue_conf`` structure in DPDK 22.11.
>> --
>> 2.25.1
>>
  
Pavan Nikhilesh Bhagavatula July 13, 2022, 9:56 a.m. UTC | #4
> On 2022-07-12 16:05, Jerin Jacob wrote:
> > On Sun, May 15, 2022 at 3:56 PM Shijith Thotton <sthotton@marvell.com>
> wrote:
> >>
> >> Structure rte_event_queue_conf will be extended to include fields to
> >> support weight and affinity attribute. Once it gets added in DPDK 22.11,
> >> eventdev internal op, queue_attr_get can be removed.
> >>
> >> Signed-off-by: Shijith Thotton <sthotton@marvell.com>
> >
> >
> > Acked-by: Jerin Jacob <jerinj@marvell.com>
> >
> > @Van Haaren, Harry  @Mattias Rönnblom  @Ray Kinsella  @Pavan
> Nikhilesh
> > Please Ack if you are OK.
> >
> 
> Will there be new capabilities to go with those new fields? To let the
> application know if they will be honored, or not.
> 

I think this capability got covered under 
RTE_EVENT_DEV_CAP_QUEUE_QOS.

> Acked-by: Mattias Rönnblom <mattias.ronnblom@ericsson.com>
> 
> >> ---
> >>   doc/guides/rel_notes/deprecation.rst | 3 +++
> >>   1 file changed, 3 insertions(+)
> >>
> >> diff --git a/doc/guides/rel_notes/deprecation.rst
> b/doc/guides/rel_notes/deprecation.rst
> >> index 4e5b23c53d..04125db681 100644
> >> --- a/doc/guides/rel_notes/deprecation.rst
> >> +++ b/doc/guides/rel_notes/deprecation.rst
> >> @@ -125,3 +125,6 @@ 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: New fields to represent event queue weight and affinity
> will be
> >> +  added to ``rte_event_queue_conf`` structure in DPDK 22.11.
> >> --
> >> 2.25.1
> >>
  
Thomas Monjalon July 17, 2022, 12:52 p.m. UTC | #5
15/05/2022 12:24, Shijith Thotton:
> Structure rte_event_queue_conf will be extended to include fields to
> support weight and affinity attribute. Once it gets added in DPDK 22.11,
> eventdev internal op, queue_attr_get can be removed.
> 
> Signed-off-by: Shijith Thotton <sthotton@marvell.com>
    Acked-by: Jerin Jacob <jerinj@marvell.com>
    Acked-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
    Acked-by: Mattias Rönnblom <mattias.ronnblom@ericsson.com>

There's not enough non-Marvell interested parties.
But I judge it non-controversial, so
Applied, thanks.
  

Patch

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 4e5b23c53d..04125db681 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -125,3 +125,6 @@  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: New fields to represent event queue weight and affinity will be
+  added to ``rte_event_queue_conf`` structure in DPDK 22.11.