[dpdk-dev] [PATCH 0/3] Rework CTF event description storage
David Marchand
david.marchand at redhat.com
Wed Oct 28 14:09:57 CET 2020
On Wed, Oct 28, 2020 at 9:53 AM Jerin Jacob <jerinjacobk at gmail.com> wrote:
>
> On Wed, Oct 28, 2020 at 1:13 AM David Marchand
> <david.marchand at redhat.com> wrote:
> >
> > On Fri, Oct 23, 2020 at 10:01 AM David Marchand
> > <david.marchand at 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
More information about the dev
mailing list