Hi Aman, On 6/14/21 3:42 PM, Singh, Aman Deep wrote: > Hi Ivan, > > As a suggestion, can we add a check for debug log_level in > "rte_flow_snprint" itself.
I see the reason, but I think it a bad idea to put the check inside the function. If calling function cares, it should do the check. > So we can avoid CPU time, in cases where we don't want these logs. [snip]