From 67d3049b02b014ed205b44437d2377d138d3ab0d Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Tue, 6 Jun 2017 21:40:19 +0800 Subject: [PATCH] crc32c: include acconfig.h to fix ceph_crc32c_aarch64() HAVE_ARMV8_CRC is defined by "acconfig.h". so we need to include it for the definition. it's a regresssion introduced by 2a3382f. in that change, crc32c_intel_baseline.h was removed from crc32.cc, and that header includes accconfig.h indirectly. that's why we will have a dummy ceph_crc32c_aarch64() which always returns 0. the dummy implementation was intended to please the compiler. but we can probably just drop it in future as a cleanup. Signed-off-by: Kefu Chai --- src/common/crc32c_aarch64.h | 1 + 1 file changed, 1 insertion(+) diff --git a/src/common/crc32c_aarch64.h b/src/common/crc32c_aarch64.h index 3727f545cecd3..51f0542fe302f 100644 --- a/src/common/crc32c_aarch64.h +++ b/src/common/crc32c_aarch64.h @@ -1,6 +1,7 @@ #ifndef CEPH_COMMON_CRC32C_AARCH64_H #define CEPH_COMMON_CRC32C_AARCH64_H +#include "acconfig.h" #include "arch/arm.h" #ifdef __cplusplus -- 2.39.5