On 1/19/2024 2:26 PM, Rushil Gupta wrote: > Read from shared region to retrieve imissed statistics for GQ from device. > > Signed-off-by: Rushil Gupta <rush...@google.com> > Reviewed-by: Joshua Washington <joshw...@google.com> >
<...> > +static int > +gve_alloc_stats_report(struct gve_priv *priv, > + uint16_t nb_tx_queues, uint16_t nb_rx_queues) > +{ > + char z_name[RTE_MEMZONE_NAMESIZE]; > + int tx_stats_cnt; > + int rx_stats_cnt; > + > + tx_stats_cnt = (GVE_TX_STATS_REPORT_NUM + NIC_TX_STATS_REPORT_NUM) * > + nb_tx_queues; > + rx_stats_cnt = (GVE_RX_STATS_REPORT_NUM + NIC_RX_STATS_REPORT_NUM) * > + nb_rx_queues; > + priv->stats_report_len = sizeof(struct gve_stats_report) + > + sizeof(struct stats) * (tx_stats_cnt + rx_stats_cnt); > + > + snprintf(z_name, sizeof(z_name), "stats_report_%s", > priv->pci_dev->device.name); > + priv->gve_stats_report_mem = rte_memzone_reserve_aligned(z_name, > + priv->stats_report_len, > + rte_socket_id(), > + RTE_MEMZONE_IOVA_CONTIG, PAGE_SIZE); > Adding 'gve_' prefix to memzone name comment seems missed.