issue: 1557652 Introduce compiler.m4

Use no optimization for asm functions

Signed-off-by: Igor Ivanov <[email protected]>

Author: igor-ivanov

config/m4/compiler.m4

@@ -0,0 +1,53 @@
+# compiler.m4 - Parsing compiler capabilities
+#
+# Copyright (C) Mellanox Technologies Ltd. 2001-2018.  ALL RIGHTS RESERVED.
+# See file LICENSE for terms.
+#
+
+
+# Check compiler specific attributes
+# Usage: CHECK_COMPILER_ATTRIBUTE([attribute], [program], [definition])
+# Note:
+# - [definition] can be omitted if it is equal to attribute
+#
+AC_DEFUN([CHECK_COMPILER_ATTRIBUTE], [
+    AC_CACHE_VAL(vma_cv_attribute_[$1], [
+        #
+        # Try to compile using the C compiler
+        #
+        AC_TRY_COMPILE([$2],[],
+                       [vma_cv_attribute_$1=yes],
+                       [vma_cv_attribute_$1=no])
+        AS_IF([test "x$vma_cv_attribute_$1" = "xyes"], [
+            AC_LANG_PUSH(C++)
+            AC_TRY_COMPILE([extern "C" {
+                           $2
+                           }],[],
+                           [vma_cv_attribute_$1=yes],
+                           [vma_cv_attribute_$1=no])
+            AC_LANG_POP(C++)
+        ])
+    ])
+
+    AC_MSG_CHECKING([for attribute $1])
+    AC_MSG_RESULT([$vma_cv_attribute_$1])
+    AS_IF([test "x$vma_cv_attribute_$1" = "xyes"], [
+        AS_IF([test "x$3" = "x"],
+            [AC_DEFINE_UNQUOTED([DEFINED_$1], [1], [Define to 1 if attribute $1 is supported])],
+            [AC_DEFINE_UNQUOTED([DEFINED_$3], [1], [Define to 1 if attribute $1 is supported])]
+        )
+    ])
+])
+
+
+
+##########################
+# Set compiler capabilities
+#
+AC_DEFUN([COMPILER_CAPABILITY_SETUP],
+[
+
+CHECK_COMPILER_ATTRIBUTE([optimize],
+                         [int foo (int arg) __attribute__ ((optimize("O0")));],
+                         [ATTRIBUTE_OPTIMIZE])
+])

configure.ac

@@ -88,6 +88,7 @@ m4_include([config/m4/opt.m4])
 m4_include([config/m4/verbs.m4])
 m4_include([config/m4/nl.m4])
 m4_include([config/m4/prof.m4])
+m4_include([config/m4/compiler.m4])
 
 FUNC_CONFIGURE_INIT()
 
@@ -163,6 +164,7 @@ case $CC in
         ;;
 esac
 
+COMPILER_CAPABILITY_SETUP()
 
 dnl===-----------------------------------------------------------------------===
 dnl===
@@ -387,22 +389,6 @@ dnl===
 dnl===-----------------------------------------------------------------------===
 show_section_title "Check for functions, types and structures"
 
-# Does this compiler have built-in functions for atomic memory access?
-AC_MSG_CHECKING([for atomic memory access (__sync_bool_compare_and_swap) support])
-AC_TRY_LINK(,
-[
-  int variable = 1;
-  return (__sync_bool_compare_and_swap(&variable, 1, 2)
-         && __sync_add_and_fetch(&variable, 1)) ? 1 : 0;
-],
-[
-  AC_MSG_RESULT([yes])
-  AC_DEFINE(HAVE_BUILTIN_ATOMIC, 1, [Define to 1 if gcc supports __sync_bool_compare_and_swap() a.o.])
-],
-[
-  AC_MSG_RESULT([no])
-])
-
 AC_MSG_CHECKING([for SOF_TIMESTAMPING_SOFTWARE support])
 AC_TRY_LINK(
 #include <linux/net_tstamp.h>

src/utils/Makefile.am

@@ -13,7 +13,8 @@ libutils_la_SOURCES = \
 	clock.h \
 	lock_wrapper.h \
 	rdtsc.h \
-	types.h
+	types.h \
+	compiler.h
 
 noinst_PROGRAMS = timetest
 timetest_LDADD = -lrt libutils.la

src/utils/compiler.h

@@ -0,0 +1,43 @@
+/*
+ * Copyright (c) 2001-2018 Mellanox Technologies, Ltd. All rights reserved.
+ *
+ * This software is available to you under a choice of one of two
+ * licenses.  You may choose to be licensed under the terms of the GNU
+ * General Public License (GPL) Version 2, available from the file
+ * COPYING in the main directory of this source tree, or the
+ * BSD license below:
+ *
+ *     Redistribution and use in source and binary forms, with or
+ *     without modification, are permitted provided that the following
+ *     conditions are met:
+ *
+ *      - Redistributions of source code must retain the above
+ *        copyright notice, this list of conditions and the following
+ *        disclaimer.
+ *
+ *      - Redistributions in binary form must reproduce the above
+ *        copyright notice, this list of conditions and the following
+ *        disclaimer in the documentation and/or other materials
+ *        provided with the distribution.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+ * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+ * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+ * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS
+ * BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN
+ * ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
+ * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ */
+
+
+#ifndef SRC_UTILS_COMPILER_H_
+#define SRC_UTILS_COMPILER_H_
+
+#if defined(DEFINED_ATTRIBUTE_OPTIMIZE)
+  #define VMA_ATTRIBUTE_OPTIMIZE_NONE __attribute__((optimize("O0")))
+#else
+  #define VMA_ATTRIBUTE_OPTIMIZE_NONE
+#endif
+
+#endif /* SRC_UTILS_COMPILER_H_ */

src/vma/util/sys_vars.h

@@ -42,6 +42,7 @@
 #include "vtypes.h"
 #include "config.h"
 
+#include "utils/compiler.h"
 #include "vma/ib/base/verbs_extra.h"
 #include "vma/util/sysctl_reader.h"
 #include "vma/vma_extra.h"
@@ -428,8 +429,8 @@ struct mce_sys_var {
 	int env_to_cpuset(char *orig_start, cpu_set_t *cpu_set);
 	void read_env_variable_with_pid(char* mce_sys_name, size_t mce_sys_max_size, char* env_ptr);
 	bool check_cpuinfo_flag(const char* flag);
-	bool cpuid_hv();
-	const char* cpuid_hv_vendor();
+	bool cpuid_hv() VMA_ATTRIBUTE_OPTIMIZE_NONE;
+	const char* cpuid_hv_vendor() VMA_ATTRIBUTE_OPTIMIZE_NONE;
 	void read_hv();
 
 	// prevent unautothrized creation of objects