Hi David, On 12/18/2018 4:07 PM, David Marchand wrote: > Hello Akhil, > > On Fri, Dec 14, 2018 at 1:52 PM Mokhtar, Amr <[email protected]> wrote: > >>> -----Original Message----- >>> From: Neil Horman [mailto:[email protected]] >>> Sent: Friday 14 December 2018 12:36 >>> To: David Marchand <[email protected]> >>> Cc: Akhil Goyal <[email protected]>; [email protected]; [email protected]; >>> [email protected]; Yigit, Ferruh <[email protected]>; Mokhtar, >>> Amr <[email protected]> >>> Subject: Re: [dpdk-dev] [PATCH] bbdev: add missing experimental tags >>> >>> On Fri, Dec 14, 2018 at 11:00:17AM +0100, David Marchand wrote: >>>> On Fri, Dec 14, 2018 at 10:54 AM Akhil Goyal <[email protected]> >>> wrote: >>>>> Hi David, >>>>> >>>>> I can see that there are other APIs as well which are not marked as >>>>> experimental like rte_bbdev_dec_op_alloc_bulk >>>>> rte_bbdev_dec_op_free_bulk, rte_bbdev_enqueue_enc_ops etc. >>>>> >> It seems that all APIs defined in rte_bbdev_op.h are missing their >> experimental tags. >> >>>> Well, that's the problem with inlines... >>>> I don't think we can detect these easily. >>>> >>> We can, if the symbols get added to the version map as they should. But >>> (as you >>> note), because these functions are inlines, theres no error thrown for >> not >>> following that rule. >>> >> Right, there are some APIs missing in the map file. >> I am submitting a patch of those functions missing. >> > Sorry, did not follow up on this. > My patch has been marked as "Changes Requested" in pw but Amr volunteered > to fix those issues. > > How do we proceed, do you want a single fix ? or you can already pick mine. > > I believe a single patch should be enough to fix this, it can be either you or Amr. As of now, none of the patch is complete.
Thanks, Akhil
