On 19/12/17 15:15, Chris Wilson wrote:
Quoting Lionel Landwerlin (2017-12-19 14:26:11)
With changes going to drm-tip, the tracepoints field locations are
going to change. This change introduces a tracepoint parser (using a
peg parser) which lets us figure out field positions on the fly.
v2: Fix automake build (Lionel)
v3: Make overlay build conditional on peg (Petri)
Make wait_end callback more readable (Chris)
Drop tracepoint_id(), instead parsing from format file (Lionel)
v4: Fix existing configure.ac issue with overlay build (Petri)
v5: Silence unused function (Lionel)
Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@xxxxxxxxx>
For the build system changes:
Acked-by: Petri Latvala <petri.latvala@xxxxxxxxx>
If it works, it works.
Acked-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
One of the next steps would be failing if the tracepoint doesn't contain
the desired data.
-Chris
It will fail if the parsing fails. Now if someone removes a field from
the kernel, yeah, we're a bit screwed...
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
