[glib/rc-align: 2/2] Test the alignment of the refcounted box allocations
- From: Emmanuele Bassi <ebassi src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [glib/rc-align: 2/2] Test the alignment of the refcounted box allocations
- Date: Mon, 26 Nov 2018 17:35:20 +0000 (UTC)
commit 7a70dc6484d0e9d686b5e3b03920f0ca4049b6a4
Author: Emmanuele Bassi <ebassi gnome org>
Date: Wed Nov 14 17:49:26 2018 +0000
Test the alignment of the refcounted box allocations
Check that the allocations are aligned regardless of the block size.
glib/tests/rcbox.c | 60 ++++++++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 60 insertions(+)
---
diff --git a/glib/tests/rcbox.c b/glib/tests/rcbox.c
index b1a1342bb..73126c7ea 100644
--- a/glib/tests/rcbox.c
+++ b/glib/tests/rcbox.c
@@ -220,6 +220,64 @@ test_atomic_rcbox_dup (void)
g_assert_null (global_point_b);
}
+/* The expected alignment of the refcounted data, absent any other
+ * alignment requirement, is `2 * sizeof(void*)`; GLib only really
+ * supports void* sized 8 or 4 (see the comment in gatomic.h)
+ */
+#if GLIB_SIZEOF_VOID_P == 8
+static const gsize rcbox_alignment = 16;
+#else
+static const gsize rcbox_alignment = 8;
+#endif
+
+/* verify that the refcounted allocation is properly aligned */
+static void
+test_rcbox_alignment (void)
+{
+ const gsize block_sizes[] = {
+ 1,
+ 2,
+ 4,
+ sizeof (gint32) * 3,
+ };
+
+ int i;
+
+ for (i = 0; i < G_N_ELEMENTS (block_sizes); i++)
+ {
+ gpointer p = g_rc_box_alloc0 (block_sizes[i]);
+
+ g_assert_nonnull (p);
+ g_assert_true (((guintptr) p & (rcbox_alignment - 1)) == 0);
+
+ g_rc_box_release (p);
+ }
+}
+
+/* verify that the atomically refcounted allocation is properly aligned */
+static void
+test_atomic_rcbox_alignment (void)
+{
+ const gsize block_sizes[] = {
+ 1,
+ 2,
+ 4,
+ sizeof (gint32) * 3,
+ };
+
+ int i;
+
+ for (i = 0; i < G_N_ELEMENTS (block_sizes); i++)
+ {
+ gpointer p = g_atomic_rc_box_alloc0 (block_sizes[i]);
+
+ g_assert_nonnull (p);
+ g_assert_true (((guintptr) p & (rcbox_alignment - 1)) == 0);
+
+ g_atomic_rc_box_release (p);
+ }
+}
+
int
main (int argc,
char *argv[])
@@ -229,10 +287,12 @@ main (int argc,
g_test_add_func ("/rcbox/new", test_rcbox_new);
g_test_add_func ("/rcbox/release-full", test_rcbox_release_full);
g_test_add_func ("/rcbox/dup", test_rcbox_dup);
+ g_test_add_func ("/rcbox/alignment", test_rcbox_alignment);
g_test_add_func ("/atomic-rcbox/new", test_atomic_rcbox_new);
g_test_add_func ("/atomic-rcbox/release-full", test_atomic_rcbox_release_full);
g_test_add_func ("/atomic-rcbox/dup", test_atomic_rcbox_dup);
+ g_test_add_func ("/atomic-rcbox/alignment", test_atomic_rcbox_alignment);
return g_test_run ();
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]