telemetry: fix shared link with make
authorKevin Laatz <kevin.laatz@intel.com>
Wed, 7 Nov 2018 18:10:18 +0000 (18:10 +0000)
committerThomas Monjalon <thomas@monjalon.net>
Mon, 12 Nov 2018 00:21:58 +0000 (01:21 +0100)
commitfe1570a73e77565e1018dfc77f386c09e8f172fa
tree47c429c3084a123280fe7c566aabd4f278d5f629
parent6bdf14455386f39c3cb33d1046a419c235605da9
telemetry: fix shared link with make

Currently, telemetry is not working for shared builds in make.

The --as-needed flag is preventing telemetry from being linked as there are
no direct API calls from the app to telemetry. This is causing the
--telemetry option to not be recognized by EAL.
Telemetry registers it's EAL option using the RTE_INIT constructor. Since
EAL's option parsing is done before the plugins init, the --telemetry
option isn't registered at the time of parsing, and as a result, the
--telemetry option is not being recognized.

This patch fixes this issue by explicitly linking telemetry to the
application by setting the "--no-as-needed" flag for the library in
mk/rte.app.mk.

Fixes: 8877ac688b52 ("telemetry: introduce infrastructure")

Reported-by: Yanjie Xu <yanjie.xu@intel.com>
Signed-off-by: Kevin Laatz <kevin.laatz@intel.com>
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
mk/rte.app.mk