On 1/25/22 12:41, Kevin Traynor wrote:
> On 25/01/2022 09:41, Wan Junjie wrote:
>> The ```rte_mempool_avail_count``` and ```rte_mempool_in_use_count```
>> can tell us the usage of the mempool. It could be helpful for debug
>> on any memleak in the mempool.
>> Add a line in the cmd's output.
>> - Count: avail (118988), in use (12084)
>>
> 
> Hi Wan Junjie, Not important, but just fyi, for consistency with other commit 
> messages and to match the v2 output, the commit message could be like below. 
> If you agree, then probably the maintainers can update it when applying.
> 
> Either way, thanks for the improvement.
> 
> Acked-by: Kevin Traynor <ktray...@redhat.com>
> 
> -- 
> netdev-dpdk: Add mempool count in cmd get-mempool-info.
> 
> The rte_mempool_avail_count() and rte_mempool_in_use_count() DPDK API
> can tell us the usage of the mempool. It could be helpful for debug
> on any memleak in the mempool.
> 
> Add a line in the cmd's output:
>     count: avail (118988), in use (12084)

Thanks!  Applied.

Best regards, Ilya Maximets.

> 
> 
>> Acked-by: Aaron Conole <acon...@redhat.com>
>> Signed-off-by: Wan Junjie <wanjun...@bytedance.com>
>> ---
>> v2: remove prefix '-' from output
>> ---
>>   lib/netdev-dpdk.c | 3 +++
>>   1 file changed, 3 insertions(+)
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to