On Mon, Dec 16, 2019 at 09:37:02PM +0530, Naresh Kamboju wrote:
This patch merged into stable-rc tree and perf build failed on OE for Linaro builds for 5.3, 4.19, 4.14 and 4.9. Please find build error logs here,
tests/backward-ring-buffer.c: In function 'test__backward_ring_buffer': tests/backward-ring-buffer.c:147:2: warning: implicit declaration of function 'evlist__close'; did you mean 'perf_evlist__close'? [-Wimplicit-function-declaration] evlist__close(evlist); ^~~~~~~~~~~~~ perf_evlist__close tests/backward-ring-buffer.c:147:2: warning: nested extern declaration of 'evlist__close' [-Wnested-externs] tests/backward-ring-buffer.c:149:8: warning: implicit declaration of function 'evlist__open'; did you mean 'perf_evlist__open'? [-Wimplicit-function-declaration] err = evlist__open(evlist); ^~~~~~~~~~~~ perf_evlist__open tests/backward-ring-buffer.c:149:8: warning: nested extern declaration of 'evlist__open' [-Wnested-externs] perf/1.0-r9/recipe-sysroot/usr/lib/python2.7/config/libpython2.7.a(posixmodule.o): In function `posix_tmpnam': /usr/src/debug/python/2.7.15-r1/Python-2.7.15/Modules/posixmodule.c:7648: warning: the use of `tmpnam_r' is dangerous, better use `mkstemp' perf/1.0-r9/recipe-sysroot/usr/lib/python2.7/config/libpython2.7.a(posixmodule.o): In function `posix_tempnam': /usr/src/debug/python/2.7.15-r1/Python-2.7.15/Modules/posixmodule.c:7595: warning: the use of `tempnam' is dangerous, better use `mkstemp' perf/1.0-r9/perf-1.0/perf-in.o: In function `test__backward_ring_buffer': perf/1.0-r9/perf-1.0/tools/perf/tests/backward-ring-buffer.c:147: undefined reference to `evlist__close' perf/1.0-r9/perf-1.0/tools/perf/tests/backward-ring-buffer.c:149: undefined reference to `evlist__open'
Full log can be found at, https://ci.linaro.org/view/lkft/job/openembedded-lkft-linux-stable-rc-5.3/DI... https://ci.linaro.org/view/lkft/job/openembedded-lkft-linux-stable-rc-4.19/D... https://ci.linaro.org/view/lkft/job/openembedded-lkft-linux-stable-rc-4.14/D... https://ci.linaro.org/view/lkft/job/openembedded-lkft-linux-stable-rc-4.9/DI...
Good catch, thanks, I'll go drop it from all of these queues.
greg k-h