Re: [PATCH v1 1/3] bbdev: new queue stat for available enqueue depth

[Maxime Coquelin]

Hi Nicolas,

On 8/12/24 19:27, Chautru, Nicolas wrote:

Hi Maxime,

The branch origin/next-baseband-for-main doesn’t have yet the updates from 
main, such as the ./doc/rel_notes for 24.11.
Can you refresh this? Or let me know how best to proceed.

I rebased the branches to latest main, you can proceed.

Thanks,
Maxime

Thanks,
Nic

-----Original Message-----
From: Maxime Coquelin <maxime.coque...@redhat.com>
Sent: Monday, August 12, 2024 2:29 AM
To: Chautru, Nicolas <nicolas.chau...@intel.com>; dev@dpdk.org
Cc: hemant.agra...@nxp.com; Marchand, David
<david.march...@redhat.com>; Vargas, Hernan
<hernan.var...@intel.com>
Subject: Re: [PATCH v1 1/3] bbdev: new queue stat for available enqueue
depth

Hi Nicolas,

On 4/4/24 23:04, Nicolas Chautru wrote:

Capturing additional queue stats counter for the depth of enqueue
batch still available on the given queue. This can help application to
monitor that depth at run time.

Signed-off-by: Nicolas Chautru <nicolas.chau...@intel.com>
---
   lib/bbdev/rte_bbdev.h | 2 ++
   1 file changed, 2 insertions(+)

diff --git a/lib/bbdev/rte_bbdev.h b/lib/bbdev/rte_bbdev.h index
0cbfdd1c95..25514c58ac 100644
--- a/lib/bbdev/rte_bbdev.h
+++ b/lib/bbdev/rte_bbdev.h
@@ -283,6 +283,8 @@ struct rte_bbdev_stats {
         *     bbdev operation
         */
        uint64_t acc_offload_cycles;
+       /** Available number of enqueue batch on that queue. */
+       uint16_t enqueue_depth_avail;
   };

   /**

I think it needs to be documented in the ABI change section.

With that done, feel free to add:

Maxime Coquelin <maxime.coque...@redhat.com>

Thanks,
Maxime