03/07/2023 12:56, Volodymyr Fialko: > Since it's header-only library, there is issue with using __rte_intenal > (appeared in v4).
What is the issue? > Even if the function itself is not used directly, it get's included to the > other public files. > It explains why other functions in this library does not have the > rte_internal prefix, but the double underscores. > So, should I simply remove __rte_internal from v4, or there's another > approach to resolve this issue(beside creating .c file)? > > /Volodymyr > > > -----Original Message----- > > From: Dumitrescu, Cristian <[email protected]> > > Sent: Friday, June 23, 2023 2:41 PM > > To: Thomas Monjalon <[email protected]>; Volodymyr Fialko > > <[email protected]> > > Cc: [email protected]; Jerin Jacob Kollanukkaran <[email protected]>; Anoob > > Joseph > > <[email protected]> > > Subject: [EXT] RE: [PATCH v3] bitmap: add scan from offset function > > > > External Email > > > > ---------------------------------------------------------------------- > > > > > > > -----Original Message----- > > > From: Thomas Monjalon <[email protected]> > > > Sent: Thursday, June 22, 2023 6:45 PM > > > To: Volodymyr Fialko <[email protected]> > > > Cc: [email protected]; Dumitrescu, Cristian > > > <[email protected]>; [email protected]; > > > [email protected] > > > Subject: Re: [PATCH v3] bitmap: add scan from offset function > > > > > > 21/06/2023 12:01, Volodymyr Fialko: > > > > Currently, in the case when we search for a bit set after a > > > > particular value, the bitmap has to be scanned from the beginning > > > > and > > > > rte_bitmap_scan() has to be called multiple times until we hit the > > > > value. > > > > > > > > Add a new rte_bitmap_scan_from_offset() function to initialize scan > > > > state at the given offset and perform scan, this will allow getting > > > > the next set bit after certain offset within one scan call. > > > > > > > > Signed-off-by: Volodymyr Fialko <[email protected]> > > > > --- > > > > v2: > > > > - added rte_bitmap_scan_from_offset > > > > v3 > > > > - added note for internal use only for init_at function > > > [...] > > > > +/** > > > > + * @warning > > > > + * @b EXPERIMENTAL: this API may change without prior notice. > > > > + * > > > > + * Bitmap initialize internal scan pointers at the given position > > > > +for the scan > > > function. > > > > + * > > > > + * Note: for private/internal use, for public: > > > > + * @see rte_bitmap_scan_from_offset() > > > > + * > > > > + * @param bmp > > > > + * Handle to bitmap instance > > > > + * @param pos > > > > + * Bit position to start scan > > > > + */ > > > > +__rte_experimental > > > > +static inline void > > > > +__rte_bitmap_scan_init_at(struct rte_bitmap *bmp, uint32_t pos) > > > > > > I think it should marked with __rte_internal instead of experimental. > > > > > > > > > > > > +1 >
