* [PATCH v1 0/1] bbdev: extend range for alloc function @ 2023-06-02 2:04 Nicolas Chautru 2023-06-02 2:04 ` [PATCH v1 1/1] bbdev: extend range of allocation function Nicolas Chautru 0 siblings, 1 reply; 7+ messages in thread From: Nicolas Chautru @ 2023-06-02 2:04 UTC (permalink / raw) To: dev, maxime.coquelin; +Cc: hemant.agrawal, hernan.vargas, Nicolas Chautru Minor change so that these companions functions are aligned with the size of the range for the underlying rte_mempool_bulk_alloc function. Backward compatible but may allow notably to allocation larger number of operations. Nicolas Chautru (1): bbdev: extend range of allocation function lib/bbdev/rte_bbdev_op.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) -- 2.34.1 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v1 1/1] bbdev: extend range of allocation function 2023-06-02 2:04 [PATCH v1 0/1] bbdev: extend range for alloc function Nicolas Chautru @ 2023-06-02 2:04 ` Nicolas Chautru 2023-06-02 4:23 ` Hemant Agrawal ` (2 more replies) 0 siblings, 3 replies; 7+ messages in thread From: Nicolas Chautru @ 2023-06-02 2:04 UTC (permalink / raw) To: dev, maxime.coquelin; +Cc: hemant.agrawal, hernan.vargas, Nicolas Chautru Realigning the argument to unsigned int to align with number support by underlying rte_mempool_get_bulk function. Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com> --- lib/bbdev/rte_bbdev_op.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/bbdev/rte_bbdev_op.h b/lib/bbdev/rte_bbdev_op.h index 96a390cd9b..9353fd588b 100644 --- a/lib/bbdev/rte_bbdev_op.h +++ b/lib/bbdev/rte_bbdev_op.h @@ -982,7 +982,7 @@ rte_bbdev_op_pool_create(const char *name, enum rte_bbdev_op_type type, */ static inline int rte_bbdev_enc_op_alloc_bulk(struct rte_mempool *mempool, - struct rte_bbdev_enc_op **ops, uint16_t num_ops) + struct rte_bbdev_enc_op **ops, unsigned int num_ops) { struct rte_bbdev_op_pool_private *priv; @@ -1013,7 +1013,7 @@ rte_bbdev_enc_op_alloc_bulk(struct rte_mempool *mempool, */ static inline int rte_bbdev_dec_op_alloc_bulk(struct rte_mempool *mempool, - struct rte_bbdev_dec_op **ops, uint16_t num_ops) + struct rte_bbdev_dec_op **ops, unsigned int num_ops) { struct rte_bbdev_op_pool_private *priv; @@ -1045,7 +1045,7 @@ rte_bbdev_dec_op_alloc_bulk(struct rte_mempool *mempool, __rte_experimental static inline int rte_bbdev_fft_op_alloc_bulk(struct rte_mempool *mempool, - struct rte_bbdev_fft_op **ops, uint16_t num_ops) + struct rte_bbdev_fft_op **ops, unsigned int num_ops) { struct rte_bbdev_op_pool_private *priv; -- 2.34.1 ^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [PATCH v1 1/1] bbdev: extend range of allocation function 2023-06-02 2:04 ` [PATCH v1 1/1] bbdev: extend range of allocation function Nicolas Chautru @ 2023-06-02 4:23 ` Hemant Agrawal 2023-06-02 7:56 ` Maxime Coquelin 2023-07-06 9:12 ` Maxime Coquelin 2 siblings, 0 replies; 7+ messages in thread From: Hemant Agrawal @ 2023-06-02 4:23 UTC (permalink / raw) To: Nicolas Chautru, dev, maxime.coquelin; +Cc: hernan.vargas Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com> ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v1 1/1] bbdev: extend range of allocation function 2023-06-02 2:04 ` [PATCH v1 1/1] bbdev: extend range of allocation function Nicolas Chautru 2023-06-02 4:23 ` Hemant Agrawal @ 2023-06-02 7:56 ` Maxime Coquelin 2023-06-02 14:17 ` Chautru, Nicolas 2023-07-06 9:12 ` Maxime Coquelin 2 siblings, 1 reply; 7+ messages in thread From: Maxime Coquelin @ 2023-06-02 7:56 UTC (permalink / raw) To: Nicolas Chautru, dev; +Cc: hemant.agrawal, hernan.vargas On 6/2/23 04:04, Nicolas Chautru wrote: > Realigning the argument to unsigned int to > align with number support by underlying > rte_mempool_get_bulk function. > > Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com> > --- > lib/bbdev/rte_bbdev_op.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/lib/bbdev/rte_bbdev_op.h b/lib/bbdev/rte_bbdev_op.h > index 96a390cd9b..9353fd588b 100644 > --- a/lib/bbdev/rte_bbdev_op.h > +++ b/lib/bbdev/rte_bbdev_op.h > @@ -982,7 +982,7 @@ rte_bbdev_op_pool_create(const char *name, enum rte_bbdev_op_type type, > */ > static inline int > rte_bbdev_enc_op_alloc_bulk(struct rte_mempool *mempool, > - struct rte_bbdev_enc_op **ops, uint16_t num_ops) > + struct rte_bbdev_enc_op **ops, unsigned int num_ops) > { > struct rte_bbdev_op_pool_private *priv; > > @@ -1013,7 +1013,7 @@ rte_bbdev_enc_op_alloc_bulk(struct rte_mempool *mempool, > */ > static inline int > rte_bbdev_dec_op_alloc_bulk(struct rte_mempool *mempool, > - struct rte_bbdev_dec_op **ops, uint16_t num_ops) > + struct rte_bbdev_dec_op **ops, unsigned int num_ops) > { > struct rte_bbdev_op_pool_private *priv; Isn't it breaking the ABI? > @@ -1045,7 +1045,7 @@ rte_bbdev_dec_op_alloc_bulk(struct rte_mempool *mempool, > __rte_experimental > static inline int > rte_bbdev_fft_op_alloc_bulk(struct rte_mempool *mempool, > - struct rte_bbdev_fft_op **ops, uint16_t num_ops) > + struct rte_bbdev_fft_op **ops, unsigned int num_ops) > { > struct rte_bbdev_op_pool_private *priv; > ^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [PATCH v1 1/1] bbdev: extend range of allocation function 2023-06-02 7:56 ` Maxime Coquelin @ 2023-06-02 14:17 ` Chautru, Nicolas 2023-06-05 19:08 ` Maxime Coquelin 0 siblings, 1 reply; 7+ messages in thread From: Chautru, Nicolas @ 2023-06-02 14:17 UTC (permalink / raw) To: Maxime Coquelin, dev; +Cc: hemant.agrawal, Vargas, Hernan Hi Maxime, I don't think it does since no offset position change for the symbol. Also this only extends the type, so still fine if using uin16_t from application. I did not receive an email from CICD related to ABI change when pushing this (unlike the other serie for the MLD/FFT changes pushed earlier this week). Still let me know if you would like this added as well into deprecation notice, but it doesn't look required. Thanks Nic > -----Original Message----- > From: Maxime Coquelin <maxime.coquelin@redhat.com> > Sent: Friday, June 2, 2023 12:56 AM > To: Chautru, Nicolas <nicolas.chautru@intel.com>; dev@dpdk.org > Cc: hemant.agrawal@nxp.com; Vargas, Hernan <hernan.vargas@intel.com> > Subject: Re: [PATCH v1 1/1] bbdev: extend range of allocation function > > > > On 6/2/23 04:04, Nicolas Chautru wrote: > > Realigning the argument to unsigned int to align with number support > > by underlying rte_mempool_get_bulk function. > > > > Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com> > > --- > > lib/bbdev/rte_bbdev_op.h | 6 +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/lib/bbdev/rte_bbdev_op.h b/lib/bbdev/rte_bbdev_op.h index > > 96a390cd9b..9353fd588b 100644 > > --- a/lib/bbdev/rte_bbdev_op.h > > +++ b/lib/bbdev/rte_bbdev_op.h > > @@ -982,7 +982,7 @@ rte_bbdev_op_pool_create(const char *name, > enum rte_bbdev_op_type type, > > */ > > static inline int > > rte_bbdev_enc_op_alloc_bulk(struct rte_mempool *mempool, > > - struct rte_bbdev_enc_op **ops, uint16_t num_ops) > > + struct rte_bbdev_enc_op **ops, unsigned int num_ops) > > { > > struct rte_bbdev_op_pool_private *priv; > > > > @@ -1013,7 +1013,7 @@ rte_bbdev_enc_op_alloc_bulk(struct > rte_mempool *mempool, > > */ > > static inline int > > rte_bbdev_dec_op_alloc_bulk(struct rte_mempool *mempool, > > - struct rte_bbdev_dec_op **ops, uint16_t num_ops) > > + struct rte_bbdev_dec_op **ops, unsigned int num_ops) > > { > > struct rte_bbdev_op_pool_private *priv; > > Isn't it breaking the ABI? > > > @@ -1045,7 +1045,7 @@ rte_bbdev_dec_op_alloc_bulk(struct > rte_mempool *mempool, > > __rte_experimental > > static inline int > > rte_bbdev_fft_op_alloc_bulk(struct rte_mempool *mempool, > > - struct rte_bbdev_fft_op **ops, uint16_t num_ops) > > + struct rte_bbdev_fft_op **ops, unsigned int num_ops) > > { > > struct rte_bbdev_op_pool_private *priv; > > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v1 1/1] bbdev: extend range of allocation function 2023-06-02 14:17 ` Chautru, Nicolas @ 2023-06-05 19:08 ` Maxime Coquelin 0 siblings, 0 replies; 7+ messages in thread From: Maxime Coquelin @ 2023-06-05 19:08 UTC (permalink / raw) To: Chautru, Nicolas, dev; +Cc: hemant.agrawal, Vargas, Hernan On 6/2/23 16:17, Chautru, Nicolas wrote: > Hi Maxime, > I don't think it does since no offset position change for the symbol. Also this only extends the type, so still fine if using uin16_t from application. > I did not receive an email from CICD related to ABI change when pushing this (unlike the other serie for the MLD/FFT changes pushed earlier this week). > Still let me know if you would like this added as well into deprecation notice, but it doesn't look required. If ABI checks are OK, then this is good to me. Thanks, Maxime > Thanks > Nic > >> -----Original Message----- >> From: Maxime Coquelin <maxime.coquelin@redhat.com> >> Sent: Friday, June 2, 2023 12:56 AM >> To: Chautru, Nicolas <nicolas.chautru@intel.com>; dev@dpdk.org >> Cc: hemant.agrawal@nxp.com; Vargas, Hernan <hernan.vargas@intel.com> >> Subject: Re: [PATCH v1 1/1] bbdev: extend range of allocation function >> >> >> >> On 6/2/23 04:04, Nicolas Chautru wrote: >>> Realigning the argument to unsigned int to align with number support >>> by underlying rte_mempool_get_bulk function. >>> >>> Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com> >>> --- >>> lib/bbdev/rte_bbdev_op.h | 6 +++--- >>> 1 file changed, 3 insertions(+), 3 deletions(-) >>> >>> diff --git a/lib/bbdev/rte_bbdev_op.h b/lib/bbdev/rte_bbdev_op.h index >>> 96a390cd9b..9353fd588b 100644 >>> --- a/lib/bbdev/rte_bbdev_op.h >>> +++ b/lib/bbdev/rte_bbdev_op.h >>> @@ -982,7 +982,7 @@ rte_bbdev_op_pool_create(const char *name, >> enum rte_bbdev_op_type type, >>> */ >>> static inline int >>> rte_bbdev_enc_op_alloc_bulk(struct rte_mempool *mempool, >>> - struct rte_bbdev_enc_op **ops, uint16_t num_ops) >>> + struct rte_bbdev_enc_op **ops, unsigned int num_ops) >>> { >>> struct rte_bbdev_op_pool_private *priv; >>> >>> @@ -1013,7 +1013,7 @@ rte_bbdev_enc_op_alloc_bulk(struct >> rte_mempool *mempool, >>> */ >>> static inline int >>> rte_bbdev_dec_op_alloc_bulk(struct rte_mempool *mempool, >>> - struct rte_bbdev_dec_op **ops, uint16_t num_ops) >>> + struct rte_bbdev_dec_op **ops, unsigned int num_ops) >>> { >>> struct rte_bbdev_op_pool_private *priv; >> >> Isn't it breaking the ABI? >> >>> @@ -1045,7 +1045,7 @@ rte_bbdev_dec_op_alloc_bulk(struct >> rte_mempool *mempool, >>> __rte_experimental >>> static inline int >>> rte_bbdev_fft_op_alloc_bulk(struct rte_mempool *mempool, >>> - struct rte_bbdev_fft_op **ops, uint16_t num_ops) >>> + struct rte_bbdev_fft_op **ops, unsigned int num_ops) >>> { >>> struct rte_bbdev_op_pool_private *priv; >>> > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v1 1/1] bbdev: extend range of allocation function 2023-06-02 2:04 ` [PATCH v1 1/1] bbdev: extend range of allocation function Nicolas Chautru 2023-06-02 4:23 ` Hemant Agrawal 2023-06-02 7:56 ` Maxime Coquelin @ 2023-07-06 9:12 ` Maxime Coquelin 2 siblings, 0 replies; 7+ messages in thread From: Maxime Coquelin @ 2023-07-06 9:12 UTC (permalink / raw) To: Nicolas Chautru, dev; +Cc: hemant.agrawal, hernan.vargas On 6/2/23 04:04, Nicolas Chautru wrote: > Realigning the argument to unsigned int to > align with number support by underlying > rte_mempool_get_bulk function. > > Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com> > --- > lib/bbdev/rte_bbdev_op.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/lib/bbdev/rte_bbdev_op.h b/lib/bbdev/rte_bbdev_op.h > index 96a390cd9b..9353fd588b 100644 > --- a/lib/bbdev/rte_bbdev_op.h > +++ b/lib/bbdev/rte_bbdev_op.h > @@ -982,7 +982,7 @@ rte_bbdev_op_pool_create(const char *name, enum rte_bbdev_op_type type, > */ > static inline int > rte_bbdev_enc_op_alloc_bulk(struct rte_mempool *mempool, > - struct rte_bbdev_enc_op **ops, uint16_t num_ops) > + struct rte_bbdev_enc_op **ops, unsigned int num_ops) > { > struct rte_bbdev_op_pool_private *priv; > > @@ -1013,7 +1013,7 @@ rte_bbdev_enc_op_alloc_bulk(struct rte_mempool *mempool, > */ > static inline int > rte_bbdev_dec_op_alloc_bulk(struct rte_mempool *mempool, > - struct rte_bbdev_dec_op **ops, uint16_t num_ops) > + struct rte_bbdev_dec_op **ops, unsigned int num_ops) > { > struct rte_bbdev_op_pool_private *priv; > > @@ -1045,7 +1045,7 @@ rte_bbdev_dec_op_alloc_bulk(struct rte_mempool *mempool, > __rte_experimental > static inline int > rte_bbdev_fft_op_alloc_bulk(struct rte_mempool *mempool, > - struct rte_bbdev_fft_op **ops, uint16_t num_ops) > + struct rte_bbdev_fft_op **ops, unsigned int num_ops) > { > struct rte_bbdev_op_pool_private *priv; > Applied to dpdk-next-baseband/for-main. Thanks, Maxime ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2023-07-06 9:12 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2023-06-02 2:04 [PATCH v1 0/1] bbdev: extend range for alloc function Nicolas Chautru 2023-06-02 2:04 ` [PATCH v1 1/1] bbdev: extend range of allocation function Nicolas Chautru 2023-06-02 4:23 ` Hemant Agrawal 2023-06-02 7:56 ` Maxime Coquelin 2023-06-02 14:17 ` Chautru, Nicolas 2023-06-05 19:08 ` Maxime Coquelin 2023-07-06 9:12 ` Maxime Coquelin
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).