APPLIED[Unstable]: [SRU v2 B/C/D/U 0/1] build and install libperf-jvmti.so
Seth Forshee
seth.forshee at canonical.com
Thu May 30 20:17:48 UTC 2019
On Tue, May 21, 2019 at 10:52:01AM -0300, Thadeu Lima de Souza Cascardo wrote:
> BugLink: https://bugs.launchpad.net/bugs/1761379
>
> Note that this introduces new build dependencies, and some builders might not
> have them installed. This may cause test builds to fail, but that's because
> some build scripts don't check for build dependencies. Please, make sure these
> new build dependencies are installed when build testing.
>
> Note that this has also been tested on derivatives. The same patch has been
> applied, as if a rebase has been done, and it was verified that the build has
> not failed. We may not want this new library on some derivatives, so the patch
> has been written in a way that derivatives will need to opt-in. On the other
> hand, they will not fail to build after a rebase that includes this patch.
>
> [Impact]
> File libperf-jvmti.so is missing in linux-tools-common deb making it
> impossible to use perf for the JVM JITed methods.
>
> [Test case]
> $ sudo perf record -k 1 -e instructions:u ./java -agentpath:/usr/lib/linux-tools-5.0.0-8/libperf-jvmti.so crc32
> $ sudo perf inject -i ./perf.data -j -o ./perf.data.jitted
> $ sudo perf report -f -i ./perf.data.jitted
>
> [Fix]
> Include java build dependencies and install the library into linux-tools
> package.
>
> [Regression potential]
> Small regression potential, an extra file is distributed and is not
> automatically linked to anything. It could impact the build, which was
> tested.
Applied to unstable/master, thanks!
More information about the kernel-team
mailing list