On Tue, Oct 1, 2024 at 7:44 PM Morten Brørup <m...@smartsharesystems.com> wrote: > > > From: Jerin Jacob [mailto:jerinjac...@gmail.com] > > Sent: Tuesday, 1 October 2024 16.05 > > > > On Tue, Oct 1, 2024 at 7:19 PM Morten Brørup <m...@smartsharesystems.com> > > wrote: > > > > > > Jerin, > > > > > > If you have no further comments, please add review/ack tag, to help > > Thomas see that the patch has been accepted by the maintainer, and can > > be merged. > > > > There was a comment to make the function as rte_trace_is_enabled() and > > remove internal. The rest looks good to me. I will Ack in the next > > version. > > Perhaps my reply to that comment was unclear... such a public function > already exists in the previous API:
I see. It was not clear. > https://elixir.bootlin.com/dpdk/v24.07/source/lib/eal/include/rte_trace.h#L36 > > That function tells if trace enabled at both build time and runtime, and > returns false if not. > > A separate public function to tell if trace is enabled at build time seems > like overkill to me. Is that what you are asking for? No. Just use rte_trace_is_enabled() in app/test instead of __rte_trace_point_generic_is_enabled() as it is internal.