RE: [PATCH v17] app/procinfo: display eventdev xstats

2023-07-08 Thread Sevincer, Abdullah
>+Has new coverity issue. >+The reason is the boolean is set every time because it gets every time. >+Looks like code goes over eventdev_var[] even if no eventdevs are present. >+Should only look for the number of eventdevs Thanks Stephen, I will add a condition at the top of the function like:

Re: [PATCH v17] app/procinfo: display eventdev xstats

2023-07-08 Thread Stephen Hemminger
On Sat, 8 Jul 2023 15:11:45 + "Sevincer, Abdullah" wrote: > >+Has new coverity issue. > >+The reason is the boolean is set every time because it gets every time. > > >+Looks like code goes over eventdev_var[] even if no eventdevs are present. > >+Should only look for the number of eventdev

[PATCH v1] app/procinfo: revise display eventdev xstats

2023-07-08 Thread Abdullah Sevincer
process_eventdev_xstats() function was iterating over eventdev_var[] structure even if there is no eventdev present. Revised the code to check to iterate and only look for the number of eventdevs present in the system. Also, shortened function name to eventdev_xstats(). Coverity issue: 395458 Fixe

Re: [PATCH v3] dts: replace pexpect with fabric

2023-07-08 Thread Patrick Robb
Tested-by: Patrick Robb On Fri, Jun 9, 2023 at 5:46 AM Juraj Linkeš wrote: > Pexpect is not a dedicated SSH connection library while Fabric is. With > Fabric, all SSH-related logic is provided and we can just focus on > what's DTS specific. > > Signed-off-by: Juraj Linkeš > --- > > Notes: >

Re: [PATCH v4] dmadev: add tracepoints

2023-07-08 Thread fengchengwen
Hi Thomas, On 2023/7/7 18:40, Thomas Monjalon wrote: > 26/05/2023 10:42, Chengwen Feng: >> Add tracepoints at important APIs for tracing support. >> >> Signed-off-by: Chengwen Feng >> Acked-by: Morten Brørup >> >> --- >> v4: Fix asan smoke fail. >> v3: Address Morten's comment: >> Move stats

Re: [PATCH v5 00/11] use rte_pktmbuf_mto_offset

2023-07-08 Thread fengchengwen
Series-acked-by: Chengwen Feng On 2023/7/8 9:57, Stephen Hemminger wrote: > v5 - fix checkpatch warnings about long lines > split up complex expression in test/crypto_dev > > Stephen Hemminger (11): > gro: use rte_pktmbuf_mtod_offset > gso: use rte_pktmbuf_mtod_offset > test/crypto_de