Page MenuHomeFreeBSD

Import libipt
ClosedPublic

Authored by br on Oct 18 2017, 1:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 18 2024, 4:34 PM
Unknown Object (File)
Nov 23 2023, 11:10 AM
Unknown Object (File)
Nov 22 2023, 2:40 PM
Unknown Object (File)
Nov 22 2023, 9:15 AM
Unknown Object (File)
Nov 12 2023, 9:37 AM
Unknown Object (File)
Nov 9 2023, 7:20 PM
Unknown Object (File)
Nov 3 2023, 12:23 AM
Unknown Object (File)
Oct 22 2023, 5:50 PM
Subscribers

Details

Reviewers
emaste
jhb
br
Summary

Import Intel Processor Tracing library

Test Plan

world build run on amd64 host

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

lib/libipt/Makefile
14

are there c files in these include paths?

${LIBIPTSRC}/include/posix is not required in PATH

lib/libipt/Makefile
14

nope, but these are required for header files too

lib/libipt/Makefile
14

But make's .PATH doesn't affect compiler search paths for includes, they're handled only via CFLAGS+=-I....

lib/libipt/Makefile
14

This is for INCS=intel_pt.h ...
Else we get "make: don't know how to make intel-pt.h. Stop"

For example lib/libucl does the same.

lib/libipt/Makefile
14

Indeed, sorry I missed the INCS=

Add IPT entries to src.libnames.mk, bsd.libnames.mk

wrap lines in lib/libipt/Makefile

I agree with @kib that this should be a private lib

lib/libipt/Makefile
73

out of curiosity what warnings do we have at higher levels?

Oops, pasted in the wrong review.

Where is the upstream ? I only found Intel® Processor Trace Decoder Library.
I would appreciate if the review would be split into vendor import and contrib merge, and then the second review with the FreeBSD changes over contrib + build glue.

In D12717#265981, @kib wrote:

Where is the upstream ? I only found Intel® Processor Trace Decoder Library.
I would appreciate if the review would be split into vendor import and contrib merge, and then the second review with the FreeBSD changes over contrib + build glue.

OK. Upstream is here https://github.com/01org/processor-trace

lib/libipt/Makefile
73

At least cast-align warnings (pt_ild.c has a cast from const uint8_t * to const int16_t *)

This revision is now accepted and ready to land.Apr 16 2018, 12:58 PM