Hi,
As part of the effort to enable perf+coresight on fedora+RHEL, I am packaging up opencsd on fedora. This will make me the package maintainer there. As part of that work there were a couple minor tweaks to the install stanza in the makefile, as well as a couple gcc10 related warning cleanups I will forward here RSN too.
If anyone is running fedora the early test packages are here:
https://copr.fedorainfracloud.org/coprs/jlinton/opencsd/build/1519164/ or https://koji.fedoraproject.org/koji/taskinfo?taskID=46715425 and the in progress install tweak https://github.com/Linaro/OpenCSD/pull/26
Thanks,
On 2020-07-08 12:33 -0500, Jeremy Linton wrote:
and the in progress install tweak https://github.com/Linaro/OpenCSD/pull/26
Hmm. weird rpm conventions :-)
If there is a rune that will habitually DTRT for both debian and fedora builds I'm happy to use/recommend it in future.
We went through a lot of changes to the makefiles to make them distro-friendly so if this is all you had to change, that's reasonably encouraging :-)
Have you discovered that you also need a kernel-tools build rune to enable libopencsd in the PERF build? CORESIGHT=1
Hi,
On 7/8/20 1:22 PM, Wookey wrote:
On 2020-07-08 12:33 -0500, Jeremy Linton wrote:
and the in progress install tweak https://github.com/Linaro/OpenCSD/pull/26
Hmm. weird rpm conventions :-)
If there is a rune that will habitually DTRT for both debian and fedora builds I'm happy to use/recommend it in future.
We went through a lot of changes to the makefiles to make them distro-friendly so if this is all you had to change, that's reasonably encouraging :-)
Thanks! yes, it was pretty easy. I also ended up removing the static libraries because they are forbidden without a good reason. So, that probably needs an upstream conditional as well.
Have you discovered that you also need a kernel-tools build rune to enable libopencsd in the PERF build? CORESIGHT=1
Yes, I ended up digging into the perf code because I wasn't initially sure it even needed the library anymore.. Either way, just using the CORESIGHT env is the plan once the package makes its way through the process enough to be usable by the kernel tools pkg.
BTW: Are you shipping x86/etc with CORESIGHT=1 enabled as well?
Thanks,
On 2020-07-08 18:02 -0500, Jeremy Linton wrote:
BTW: Are you shipping x86/etc with CORESIGHT=1 enabled as well?
yes. It should be enabled on all arches.
Wookey
-
Jeremy Linton -
Wookey