telemetry: fix build for armv7
authorDavid Marchand <david.marchand@redhat.com>
Mon, 11 May 2020 13:32:12 +0000 (15:32 +0200)
committerDavid Marchand <david.marchand@redhat.com>
Mon, 11 May 2020 15:44:13 +0000 (17:44 +0200)
telemetry can not depend on EAL anymore but it still wants to get arch
headers.
We directly point at the right source directories by using the same logic
than EAL. However the special case of armv7 has been missed.

Fix this by defaulting ARCH_DIR to RTE_ARCH.

Caught on OBS:
[  162s]   SYMLINK-FILE include/rte_telemetry.h
[  162s]   CC telemetry.o
[  162s]   CC telemetry_data.o
[  162s]   CC telemetry_legacy.o
[  162s] .../lib/librte_telemetry/telemetry.c:15:10: fatal error:
 rte_spinlock.h: No such file or directory
[  162s]  #include <rte_spinlock.h>
[  162s]           ^~~~~~~~~~~~~~~~
[  162s] compilation terminated.

Fixes: 6dd571fd07c3 ("telemetry: introduce new functionality")

Signed-off-by: David Marchand <david.marchand@redhat.com>
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
lib/librte_telemetry/Makefile

index 29115ba..c62cbd8 100644 (file)
@@ -6,6 +6,8 @@ include $(RTE_SDK)/mk/rte.vars.mk
 # library name
 LIB = librte_telemetry.a
 
+ARCH_DIR ?= $(RTE_ARCH)
+
 CFLAGS += -O3
 CFLAGS += $(WERROR_FLAGS) -I$(SRCDIR)
 CFLAGS += -I$(RTE_SDK)/lib/librte_metrics/