mbox series

[0/3] cleanup bsf and fls inline function return types

Message ID 1659993692-17479-1-git-send-email-roretzla@linux.microsoft.com (mailing list archive)
Headers show
Series cleanup bsf and fls inline function return types | expand

Message

Tyler Retzlaff Aug. 8, 2022, 9:21 p.m. UTC
The cleanup resulted from request to review [1] the following
functions where there appeared to be inconsistency in return type
or parameter type selections for the following inline functions.

Since the original request some instances have been fixed so this
series completes the outstanding return value type updates.

[1] http://mails.dpdk.org/archives/dev/2021-March/201590.html

Tyler Retzlaff (3):
  doc: announce cleanup of rte_{bsf, fls} inline functions type use
  eal: change rte_fls and rte_bsf to return uint32_t
  test: fix sign compare warning for rte_bsf64 return type change

 app/test/test_mbuf.c                 | 2 +-
 doc/guides/rel_notes/deprecation.rst | 6 ++++++
 lib/eal/include/rte_common.h         | 6 +++---
 3 files changed, 10 insertions(+), 4 deletions(-)

Comments

Stephen Hemminger Aug. 8, 2022, 9:42 p.m. UTC | #1
On Mon,  8 Aug 2022 14:21:29 -0700
Tyler Retzlaff <roretzla@linux.microsoft.com> wrote:

> The cleanup resulted from request to review [1] the following
> functions where there appeared to be inconsistency in return type
> or parameter type selections for the following inline functions.
> 
> Since the original request some instances have been fixed so this
> series completes the outstanding return value type updates.
> 
> [1] http://mails.dpdk.org/archives/dev/2021-March/201590.html
> 
> Tyler Retzlaff (3):
>   doc: announce cleanup of rte_{bsf, fls} inline functions type use
>   eal: change rte_fls and rte_bsf to return uint32_t
>   test: fix sign compare warning for rte_bsf64 return type change
> 
>  app/test/test_mbuf.c                 | 2 +-
>  doc/guides/rel_notes/deprecation.rst | 6 ++++++
>  lib/eal/include/rte_common.h         | 6 +++---
>  3 files changed, 10 insertions(+), 4 deletions(-)
> 


Series-Acked-by: Stephen Hemminger <stephen@networkplumber.org>
Morten Brørup Aug. 9, 2022, 8:26 a.m. UTC | #2
> From: Tyler Retzlaff [mailto:roretzla@linux.microsoft.com]
> Sent: Monday, 8 August 2022 23.21
> 
> The cleanup resulted from request to review [1] the following
> functions where there appeared to be inconsistency in return type
> or parameter type selections for the following inline functions.
> 
> Since the original request some instances have been fixed so this
> series completes the outstanding return value type updates.
> 
> [1] http://mails.dpdk.org/archives/dev/2021-March/201590.html
> 
> Tyler Retzlaff (3):
>   doc: announce cleanup of rte_{bsf, fls} inline functions type use
>   eal: change rte_fls and rte_bsf to return uint32_t
>   test: fix sign compare warning for rte_bsf64 return type change
> 
>  app/test/test_mbuf.c                 | 2 +-
>  doc/guides/rel_notes/deprecation.rst | 6 ++++++
>  lib/eal/include/rte_common.h         | 6 +++---
>  3 files changed, 10 insertions(+), 4 deletions(-)
> 
> --
> 1.8.3.1
> 

Series-Acked-by: Morten Brørup <mb@smartsharesystems.com>
Thomas Monjalon Oct. 5, 2022, 10:11 a.m. UTC | #3
09/08/2022 10:26, Morten Brørup:
> > From: Tyler Retzlaff [mailto:roretzla@linux.microsoft.com]
> > Sent: Monday, 8 August 2022 23.21
> > 
> > The cleanup resulted from request to review [1] the following
> > functions where there appeared to be inconsistency in return type
> > or parameter type selections for the following inline functions.
> > 
> > Since the original request some instances have been fixed so this
> > series completes the outstanding return value type updates.
> > 
> > [1] http://mails.dpdk.org/archives/dev/2021-March/201590.html
> > 
> > Tyler Retzlaff (3):
> >   doc: announce cleanup of rte_{bsf, fls} inline functions type use
> >   eal: change rte_fls and rte_bsf to return uint32_t
> >   test: fix sign compare warning for rte_bsf64 return type change
> > 
> >  app/test/test_mbuf.c                 | 2 +-
> >  doc/guides/rel_notes/deprecation.rst | 6 ++++++
> >  lib/eal/include/rte_common.h         | 6 +++---
> >  3 files changed, 10 insertions(+), 4 deletions(-)
> 
> Series-Acked-by: Morten Brørup <mb@smartsharesystems.com>

Applied and completed.
Release notes added.
All squashed.