[v1,1/1] eventdev/eth_rx: add event buf size and count in telemetry

Message ID 20230323085828.2709694-1-ganapati.kundapura@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Jerin Jacob
Headers
Series [v1,1/1] eventdev/eth_rx: add event buf size and count in telemetry |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/loongarch-compilation success Compilation OK
ci/loongarch-unit-testing success Unit Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/github-robot: build success github build: passed
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/intel-Functional success Functional PASS
ci/iol-abi-testing success Testing PASS
ci/iol-aarch64-unit-testing success Testing PASS
ci/iol-unit-testing success Testing PASS

Commit Message

Ganapati Kundapura March 23, 2023, 8:58 a.m. UTC
  rx_event_buf_count and rx_event_buf_size is added to structure
rte_event_eth_rx_adapter_stats but they are not displayed in
rxa_stats telemetry handler.

Updated handle_rxa_stats() to display rx_event_buf_count and
rx_event_buf_size.

Signed-off-by: Ganapati Kundapura <ganapati.kundapura@intel.com>
  

Comments

Jerin Jacob April 4, 2023, 10:49 a.m. UTC | #1
On Thu, Mar 23, 2023 at 2:28 PM Ganapati Kundapura
<ganapati.kundapura@intel.com> wrote:
>
> rx_event_buf_count and rx_event_buf_size is added to structure
> rte_event_eth_rx_adapter_stats but they are not displayed in
> rxa_stats telemetry handler.
>
> Updated handle_rxa_stats() to display rx_event_buf_count and
> rx_event_buf_size.
>
> Signed-off-by: Ganapati Kundapura <ganapati.kundapura@intel.com>


Acked-by: Jerin Jacob <jerinj@marvell.com>


>
> diff --git a/lib/eventdev/rte_event_eth_rx_adapter.c b/lib/eventdev/rte_event_eth_rx_adapter.c
> index c1cd124..07d8235 100644
> --- a/lib/eventdev/rte_event_eth_rx_adapter.c
> +++ b/lib/eventdev/rte_event_eth_rx_adapter.c
> @@ -3597,6 +3597,8 @@ handle_rxa_stats(const char *cmd __rte_unused,
>         RXA_ADD_DICT(rx_adptr_stats, rx_enq_block_cycles);
>         RXA_ADD_DICT(rx_adptr_stats, rx_enq_end_ts);
>         RXA_ADD_DICT(rx_adptr_stats, rx_intr_packets);
> +       RXA_ADD_DICT(rx_adptr_stats, rx_event_buf_count);
> +       RXA_ADD_DICT(rx_adptr_stats, rx_event_buf_size);
>
>         return 0;
>  }
> --
> 2.6.4
>
  
Jerin Jacob April 4, 2023, 10:49 a.m. UTC | #2
On Tue, Apr 4, 2023 at 4:19 PM Jerin Jacob <jerinjacobk@gmail.com> wrote:
>
> On Thu, Mar 23, 2023 at 2:28 PM Ganapati Kundapura
> <ganapati.kundapura@intel.com> wrote:
> >
> > rx_event_buf_count and rx_event_buf_size is added to structure
> > rte_event_eth_rx_adapter_stats but they are not displayed in
> > rxa_stats telemetry handler.
> >
> > Updated handle_rxa_stats() to display rx_event_buf_count and
> > rx_event_buf_size.
> >
> > Signed-off-by: Ganapati Kundapura <ganapati.kundapura@intel.com>
>
>
> Acked-by: Jerin Jacob <jerinj@marvell.com>

Applied to dpdk-next-net-eventdev/for-main. Thanks

>
>
> >
> > diff --git a/lib/eventdev/rte_event_eth_rx_adapter.c b/lib/eventdev/rte_event_eth_rx_adapter.c
> > index c1cd124..07d8235 100644
> > --- a/lib/eventdev/rte_event_eth_rx_adapter.c
> > +++ b/lib/eventdev/rte_event_eth_rx_adapter.c
> > @@ -3597,6 +3597,8 @@ handle_rxa_stats(const char *cmd __rte_unused,
> >         RXA_ADD_DICT(rx_adptr_stats, rx_enq_block_cycles);
> >         RXA_ADD_DICT(rx_adptr_stats, rx_enq_end_ts);
> >         RXA_ADD_DICT(rx_adptr_stats, rx_intr_packets);
> > +       RXA_ADD_DICT(rx_adptr_stats, rx_event_buf_count);
> > +       RXA_ADD_DICT(rx_adptr_stats, rx_event_buf_size);
> >
> >         return 0;
> >  }
> > --
> > 2.6.4
> >
  

Patch

diff --git a/lib/eventdev/rte_event_eth_rx_adapter.c b/lib/eventdev/rte_event_eth_rx_adapter.c
index c1cd124..07d8235 100644
--- a/lib/eventdev/rte_event_eth_rx_adapter.c
+++ b/lib/eventdev/rte_event_eth_rx_adapter.c
@@ -3597,6 +3597,8 @@  handle_rxa_stats(const char *cmd __rte_unused,
 	RXA_ADD_DICT(rx_adptr_stats, rx_enq_block_cycles);
 	RXA_ADD_DICT(rx_adptr_stats, rx_enq_end_ts);
 	RXA_ADD_DICT(rx_adptr_stats, rx_intr_packets);
+	RXA_ADD_DICT(rx_adptr_stats, rx_event_buf_count);
+	RXA_ADD_DICT(rx_adptr_stats, rx_event_buf_size);
 
 	return 0;
 }