On Wed, Oct 28, 2020 at 9:53 AM Jerin Jacob <jerinjac...@gmail.com> wrote: > > On Wed, Oct 28, 2020 at 1:13 AM David Marchand > <david.march...@redhat.com> wrote: > > > > On Fri, Oct 23, 2020 at 10:01 AM David Marchand > > <david.march...@redhat.com> wrote: > > > > > > Following recent increase of an internal array that was limiting CTF event > > > descriptions, I had a second look at the code. > > > All of this is slow path, so I see no reason in keeping this limitation > > > and we can go with dynamic allocations. > > > > > > While at it, I tweaked the metadata file output. > > > > > > I consider this -rc2 material. > > > > Comments? > > Thanks. > > The Generated metadata has issues. Please check > > Reproducer: > > echo "trace_autotest" | ./build/app/test/dpdk-test -c 0x3 --trace=.* > --no-huge --trace=.*
Err, indeed, thanks for catching. I did some diff on metadata files, but did not notice this trailing character. This is an issue with the metadata string manipulations, that appears with the last patch... I ended up rewriting most of _ctf.c (removing intermediate buffers allocations) and it works but I'll see if I can pinpoint the issue. -- David Marchand
