Move compat macros in 'lttng/align.h' to a private header

No need to expose those compat macros in a public header, only keep the
actual align macros public.

Change-Id: Iee0ac69cfb98400afd9ee83a8a8e218af48971bf
Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

diff --git a/include/Makefile.am b/include/Makefile.am
index 101e8ac72b47236f1c8dc0ae6efb6e23beff85c3..2c78459efc2813b9bf760834e256bd7e9018d22e 100644 (file)
--- a/include/Makefile.am
+++ b/include/Makefile.am
@@ -55,6 +55,7 @@ noinst_HEADERS = \
        ust-snprintf.h \
        ust-bitmap.h \
        ust-comm.h \
+       ust-compat.h \
        ust-fd.h \
        ust-tid.h \
        ust-bitfield.h \

diff --git a/include/lttng/align.h b/include/lttng/align.h
index e46c03015b39aa712024d45111ef93d44d8278b8..6384a00a7c4d86acaf37b9a1e5e92aa4f54cec8e 100644 (file)
--- a/include/lttng/align.h
+++ b/include/lttng/align.h
@@ -4,30 +4,10 @@
  * Copyright (C) 2010-2011 Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
  */
 
-#ifndef _UST_ALIGN_H
-#define _UST_ALIGN_H
+#ifndef _LTTNG_ALIGN_H
+#define _LTTNG_ALIGN_H
 
 #include <lttng/bug.h>
-#include <unistd.h>
-#include <limits.h>
-
-#ifdef __FreeBSD__
-#include <machine/param.h>
-#endif
-
-#ifdef _SC_PAGE_SIZE
-#define LTTNG_UST_PAGE_SIZE    sysconf(_SC_PAGE_SIZE)
-#elif defined(PAGE_SIZE)
-#define LTTNG_UST_PAGE_SIZE    PAGE_SIZE
-#else
-#error "Please add page size detection for your OS."
-#endif
-
-#define LTTNG_UST_PAGE_MASK    (~(LTTNG_UST_PAGE_SIZE - 1))
-
-#define __LTTNG_UST_ALIGN_MASK(v, mask)        (((v) + (mask)) & ~(mask))
-#define LTTNG_UST_ALIGN(v, align)      __LTTNG_UST_ALIGN_MASK(v, (__typeof__(v)) (align) - 1)
-#define LTTNG_UST_PAGE_ALIGN(addr)     LTTNG_UST_ALIGN(addr, LTTNG_UST_PAGE_SIZE)
 
 /**
  * lttng_ust_offset_align - Calculate the offset needed to align an object on
@@ -60,36 +40,4 @@
                (((align_drift) - (alignment)) & ((alignment) - 1));           \
        })
 
-/*
- * Non-namespaced defines for backwards compatibility,
- * introduced in 2.13, should be removed in the future.
- */
-
-/* Cygwin limits.h defines its own PAGE_SIZE */
-#ifndef PAGE_SIZE
-#define PAGE_SIZE      LTTNG_UST_PAGE_SIZE
-#endif
-
-/* FreeBSD and macOS defines their own PAGE_MASK. */
-#ifndef PAGE_MASK
-#define PAGE_MASK      LTTNG_UST_PAGE_MASK
-#endif
-
-/* FreeBSD machine/param.h defines its own ALIGN */
-#ifndef ALIGN
-#define ALIGN          LTTNG_UST_ALIGN
-#endif
-
-#ifndef PAGE_ALIGN
-#define PAGE_ALIGN     LTTNG_UST_PAGE_ALIGN
-#endif
-
-#ifndef offset_align
-#define offset_align lttng_ust_offset_align
-#endif
-
-#ifndef offset_align_floor
-#define offset_align_floor lttng_ust_offset_align_floor
-#endif
-
-#endif /* _UST_ALIGN_H */
+#endif /* _LTTNG_ALIGN_H */

diff --git a/include/ust-compat.h b/include/ust-compat.h
new file mode 100644 (file)
index 0000000..c148e1b
--- /dev/null
+++ b/include/ust-compat.h
@@ -0,0 +1,31 @@
+/*
+ * SPDX-License-Identifier: MIT
+ *
+ * Copyright (C) 2010-2011 Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
+ */
+
+#ifndef _LTTNG_UST_COMPAT_H
+#define _LTTNG_UST_COMPAT_H
+
+#include <unistd.h>
+#include <limits.h>
+
+#ifdef __FreeBSD__
+#include <machine/param.h>
+#endif
+
+#ifdef _SC_PAGE_SIZE
+#define LTTNG_UST_PAGE_SIZE    sysconf(_SC_PAGE_SIZE)
+#elif defined(PAGE_SIZE)
+#define LTTNG_UST_PAGE_SIZE    PAGE_SIZE
+#else
+#error "Please add page size detection for your OS."
+#endif
+
+#define LTTNG_UST_PAGE_MASK    (~(LTTNG_UST_PAGE_SIZE - 1))
+
+#define __LTTNG_UST_ALIGN_MASK(v, mask)        (((v) + (mask)) & ~(mask))
+#define LTTNG_UST_ALIGN(v, align)      __LTTNG_UST_ALIGN_MASK(v, (__typeof__(v)) (align) - 1)
+#define LTTNG_UST_PAGE_ALIGN(addr)     LTTNG_UST_ALIGN(addr, LTTNG_UST_PAGE_SIZE)
+
+#endif

diff --git a/libcounter/counter.c b/libcounter/counter.c
index c98589db8329c7b16145a73e11db9dbdd57def7a..d2b7032af2ada6a6e623a8a755a4e0180e3e98c7 100644 (file)
--- a/libcounter/counter.c
+++ b/libcounter/counter.c
@@ -13,9 +13,9 @@
 #include <urcu/compiler.h>
 #include <stdbool.h>
 #include <ust-helper.h>
-#include <lttng/align.h>
 #include "smp.h"
 #include "shm.h"
+#include "ust-compat.h"
 
 #include "ust-bitmap.h"
 

diff --git a/libcounter/shm.c b/libcounter/shm.c
index 13a9f642d15e10600b67c2317cfff5a6dd48b547..1689ad45bbf490713a36ac75776705e4531d87a6 100644 (file)
--- a/libcounter/shm.c
+++ b/libcounter/shm.c
@@ -368,6 +368,6 @@ struct lttng_counter_shm_ref lttng_counter_zalloc_shm(struct lttng_counter_shm_o
 
 void lttng_counter_align_shm(struct lttng_counter_shm_object *obj, size_t align)
 {
-       size_t offset_len = offset_align(obj->allocated_len, align);
+       size_t offset_len = lttng_ust_offset_align(obj->allocated_len, align);
        obj->allocated_len += offset_len;
 }

diff --git a/liblttng-ust-ctl/ustctl.c b/liblttng-ust-ctl/ustctl.c
index 38076928fbc68164f760c8161a717ae5695538e2..057895d838614a444e434a404b7bbb4a8d69cf69 100644 (file)
--- a/liblttng-ust-ctl/ustctl.c
+++ b/liblttng-ust-ctl/ustctl.c
@@ -20,6 +20,7 @@
 #include <usterr-signal-safe.h>
 #include <ust-comm.h>
 #include <ust-helper.h>
+#include "ust-compat.h"
 
 #include "../libringbuffer/backend.h"
 #include "../libringbuffer/frontend.h"

diff --git a/liblttng-ust-libc-wrapper/lttng-ust-malloc.c b/liblttng-ust-libc-wrapper/lttng-ust-malloc.c
index 694eed172fdedc87fb713198cd0b56019f1ba3e5..be13f214480bfb09402b25d3328fd36c69d3e2b6 100644 (file)
--- a/liblttng-ust-libc-wrapper/lttng-ust-malloc.c
+++ b/liblttng-ust-libc-wrapper/lttng-ust-malloc.c
@@ -19,8 +19,8 @@
 #include <urcu/compiler.h>
 #include <urcu/tls-compat.h>
 #include <urcu/arch.h>
-#include <lttng/align.h>
 #include <ust-helper.h>
+#include "ust-compat.h"
 
 #define TRACEPOINT_DEFINE
 #define TRACEPOINT_CREATE_PROBES

diff --git a/liblttng-ust/lttng-ring-buffer-client.h b/liblttng-ust/lttng-ring-buffer-client.h
index 42d68f663f5296d6e089d30cf06f5387a18b3c17..79786af255444f9b1da6d3f8b38e22b023e1a9a1 100644 (file)
--- a/liblttng-ust/lttng-ring-buffer-client.h
+++ b/liblttng-ust/lttng-ring-buffer-client.h
@@ -10,6 +10,7 @@
 #include <stdint.h>
 #include <lttng/ust-events.h>
 #include "ust-bitfield.h"
+#include "ust-compat.h"
 #include "clock.h"
 #include "lttng-tracer.h"
 #include "../libringbuffer/frontend_types.h"

diff --git a/liblttng-ust/lttng-ring-buffer-metadata-client.h b/liblttng-ust/lttng-ring-buffer-metadata-client.h
index 0b0a65fc7dd1776385561580cda276662a0e0a0b..98af899391650d2dc485e9bfdbf886ec624e4154 100644 (file)
--- a/liblttng-ust/lttng-ring-buffer-metadata-client.h
+++ b/liblttng-ust/lttng-ring-buffer-metadata-client.h
@@ -10,6 +10,7 @@
 #include <stdint.h>
 #include <lttng/ust-events.h>
 #include "ust-bitfield.h"
+#include "ust-compat.h"
 #include "lttng-tracer.h"
 #include "../libringbuffer/frontend_types.h"
 

diff --git a/liblttng-ust/lttng-ust-comm.c b/liblttng-ust/lttng-ust-comm.c
index 891eb16eef2ec71eb81cbf7d0c108a8d04d0f0be..18701936feacd241c505e7f134092ec2079c6956 100644 (file)
--- a/liblttng-ust/lttng-ust-comm.c
+++ b/liblttng-ust/lttng-ust-comm.c
@@ -29,7 +29,6 @@
 #include <urcu/compiler.h>
 #include <lttng/urcu/urcu-ust.h>
 
-#include <lttng/align.h>
 #include <lttng/ust-events.h>
 #include <lttng/ust-abi.h>
 #include <lttng/ust.h>
@@ -50,6 +49,7 @@
 #include "getenv.h"
 #include "ust-events-internal.h"
 #include "context-internal.h"
+#include "ust-compat.h"
 
 /* Concatenate lttng ust shared library name with its major version number. */
 #define LTTNG_UST_LIB_SO_NAME "liblttng-ust.so." __ust_stringify(CONFIG_LTTNG_UST_LIBRARY_VERSION_MAJOR)

diff --git a/libringbuffer/ring_buffer_backend.c b/libringbuffer/ring_buffer_backend.c
index b50c2f3c406405000d8a830af1d19bfe48622e7d..0d4afb4efcceca77de3e990449ec4476f559a932 100644 (file)
--- a/libringbuffer/ring_buffer_backend.c
+++ b/libringbuffer/ring_buffer_backend.c
@@ -18,6 +18,7 @@
 #include "frontend.h"
 #include "smp.h"
 #include "shm.h"
+#include "ust-compat.h"
 
 /**
  * lib_ring_buffer_backend_allocate - allocate a channel buffer

diff --git a/libringbuffer/ring_buffer_frontend.c b/libringbuffer/ring_buffer_frontend.c
index 736c598ae0cd1a3794d263665b7aa36d5a5954b7..3b338fc33b98041bf4ef39bf0c1f0293e1fd03e8 100644 (file)
--- a/libringbuffer/ring_buffer_frontend.c
+++ b/libringbuffer/ring_buffer_frontend.c
@@ -52,6 +52,7 @@
 #include <poll.h>
 #include <ust-helper.h>
 
+#include <lttng/align.h>
 #include "smp.h"
 #include <lttng/ringbuffer-config.h>
 #include "vatomic.h"

diff --git a/tests/unit/libringbuffer/shm.c b/tests/unit/libringbuffer/shm.c
index 4e6ac0826988f4252035df5537b261be8211a272..fe3cb6c0055f70b1d58c41ecb6291cbaef3b56bd 100644 (file)
--- a/tests/unit/libringbuffer/shm.c
+++ b/tests/unit/libringbuffer/shm.c
@@ -12,8 +12,8 @@
 #include <sys/mman.h>
 #include <fcntl.h>
 
-#include <lttng/align.h>
 #include "libringbuffer/shm.h"
+#include "ust-compat.h"
 
 #include "tap.h"