eal: move CPU operations to OS specific headers
authorPallavi Kadam <pallavi.kadam@intel.com>
Mon, 30 Sep 2019 17:09:34 +0000 (10:09 -0700)
committerDavid Marchand <david.marchand@redhat.com>
Sat, 26 Oct 2019 15:06:41 +0000 (17:06 +0200)
Moving RTE_CPU* definitions from the common code to the Linux and
FreeBSD rte_os.h file to avoid #ifdef clutter.

Signed-off-by: Pallavi Kadam <pallavi.kadam@intel.com>
Signed-off-by: Antara Ganesh Kolar <antara.ganesh.kolar@intel.com>
Reviewed-by: Ranjit Menon <ranjit.menon@intel.com>
Reviewed-by: Jerin Jacob <jerinj@marvell.com>
Signed-off-by: David Marchand <david.marchand@redhat.com>
lib/librte_eal/common/include/rte_lcore.h
lib/librte_eal/freebsd/eal/include/rte_os.h
lib/librte_eal/linux/eal/include/rte_os.h

index c86f72e..63ad4af 100644 (file)
@@ -22,50 +22,6 @@ extern "C" {
 
 #define LCORE_ID_ANY     UINT32_MAX       /**< Any lcore. */
 
-#if defined(__linux__)
-typedef        cpu_set_t rte_cpuset_t;
-#define RTE_CPU_AND(dst, src1, src2) CPU_AND(dst, src1, src2)
-#define RTE_CPU_OR(dst, src1, src2) CPU_OR(dst, src1, src2)
-#define RTE_CPU_FILL(set) do \
-{ \
-       unsigned int i; \
-       CPU_ZERO(set); \
-       for (i = 0; i < CPU_SETSIZE; i++) \
-               CPU_SET(i, set); \
-} while (0)
-#define RTE_CPU_NOT(dst, src) do \
-{ \
-       cpu_set_t tmp; \
-       RTE_CPU_FILL(&tmp); \
-       CPU_XOR(dst, &tmp, src); \
-} while (0)
-#elif defined(__FreeBSD__)
-#include <pthread_np.h>
-typedef cpuset_t rte_cpuset_t;
-#define RTE_CPU_AND(dst, src1, src2) do \
-{ \
-       cpuset_t tmp; \
-       CPU_COPY(src1, &tmp); \
-       CPU_AND(&tmp, src2); \
-       CPU_COPY(&tmp, dst); \
-} while (0)
-#define RTE_CPU_OR(dst, src1, src2) do \
-{ \
-       cpuset_t tmp; \
-       CPU_COPY(src1, &tmp); \
-       CPU_OR(&tmp, src2); \
-       CPU_COPY(&tmp, dst); \
-} while (0)
-#define RTE_CPU_FILL(set) CPU_FILL(set)
-#define RTE_CPU_NOT(dst, src) do \
-{ \
-       cpuset_t tmp; \
-       CPU_FILL(&tmp); \
-       CPU_NAND(&tmp, src); \
-       CPU_COPY(&tmp, dst); \
-} while (0)
-#endif
-
 /**
  * Structure storing internal configuration (per-lcore)
  */
index 49cd4d4..a5efe61 100644 (file)
  * freebsd OS. Functions will be added in future releases.
  */
 
+#include <pthread_np.h>
+
+typedef cpuset_t rte_cpuset_t;
+#define RTE_CPU_AND(dst, src1, src2) do \
+{ \
+       cpuset_t tmp; \
+       CPU_COPY(src1, &tmp); \
+       CPU_AND(&tmp, src2); \
+       CPU_COPY(&tmp, dst); \
+} while (0)
+#define RTE_CPU_OR(dst, src1, src2) do \
+{ \
+       cpuset_t tmp; \
+       CPU_COPY(src1, &tmp); \
+       CPU_OR(&tmp, src2); \
+       CPU_COPY(&tmp, dst); \
+} while (0)
+#define RTE_CPU_FILL(set) CPU_FILL(set)
+#define RTE_CPU_NOT(dst, src) do \
+{ \
+       cpuset_t tmp; \
+       CPU_FILL(&tmp); \
+       CPU_NAND(&tmp, src); \
+       CPU_COPY(&tmp, dst); \
+} while (0)
+
 #endif /* _RTE_OS_H_ */
index bc6ad14..218d4fa 100644 (file)
  * linux OS. Functions will be added in future releases.
  */
 
+#include <sched.h>
+
+typedef cpu_set_t rte_cpuset_t;
+#define RTE_CPU_AND(dst, src1, src2) CPU_AND(dst, src1, src2)
+#define RTE_CPU_OR(dst, src1, src2) CPU_OR(dst, src1, src2)
+#define RTE_CPU_FILL(set) do \
+{ \
+       unsigned int i; \
+       CPU_ZERO(set); \
+       for (i = 0; i < CPU_SETSIZE; i++) \
+               CPU_SET(i, set); \
+} while (0)
+#define RTE_CPU_NOT(dst, src) do \
+{ \
+       cpu_set_t tmp; \
+       RTE_CPU_FILL(&tmp); \
+       CPU_XOR(dst, &tmp, src); \
+} while (0)
+
 #endif /* _RTE_OS_H_ */