bits.h -> bitops.h, start bitops refactor: target all implem. visible

2024-01-03 19:13:44 +01:00
parent b63a5cceda
commit 0dcca10559
13 changed files with 55 additions and 41 deletions
--- a/include/bitops.h
+++ b/include/bitops.h
@@ -1,6 +1,6 @@
-/* bits.h - bits functions.
+/* bitops.h - bits functions.
 *
- * Copyright (C) 2021-2022 Bruno Raoult ("br")
+ * Copyright (C) 2021-2024 Bruno Raoult ("br")
 * Licensed under the GNU General Public License v3.0 or later.
 * Some rights reserved. See COPYING.
 *
@@ -44,6 +44,26 @@ void print_bitops_impl(void);
 /* count set bits:  10101000 -> 3
 *                  ^ ^ ^
 */
+#if __has_builtin(__builtin_popcountll)
+#define ___popcount64_native(n)  __builtin_popcountll(n)
+#endif
+#if __has_builtin(__builtin_popcount)
+#define ___popcount32_native(n)  __builtin_popcount(n)
+#endif
+#define ___popcount_emulated(n)  ({     \
+    int ___count = 0;                   \
+    while (n) {                         \
+        ___count++;                     \
+        n &= (n - 1);                   \
+    }                                   \
+    ___count; })
+
+#ifdef ___popcount64_native
+#define ppcount64(n) ___popcount64_native(n)
+#else
+#define ppcount64(n) ___popcount_emulated(n)
+#endif
+
 static __always_inline int popcount64(u64 n)
 {
 #   if __has_builtin(__builtin_popcountll)
--- a/include/br.h
+++ b/include/br.h
@@ -1,6 +1,6 @@
 /* br.h - misc macros.
 *
- * Copyright (C) 2021-2022 Bruno Raoult ("br")
+ * Copyright (C) 2021-2024 Bruno Raoult ("br")
 * Licensed under the GNU General Public License v3.0 or later.
 * Some rights reserved. See COPYING.
 *
--- a/include/bug.h
+++ b/include/bug.h
@@ -20,9 +20,6 @@
 * can somehow continue operating, perhaps with reduced functionality,
 * it's probably not BUG-worthy.
 *
- * If you're tempted to BUG(), think again:  is completely giving up
- * really the *only* solution?  There are usually better options, where
- * users don't need to reboot ASAP and can mostly shut down cleanly.
 */
 #define BUG() do {                                                                      \
        fprintf(stderr, "BUG: failure at %s:%d/%s()!\n", __FILE__, __LINE__, __func__); \
@@ -32,9 +29,8 @@
 #define BUG_ON(condition) do { if (unlikely(condition)) BUG(); } while (0)

 /*
- * WARN(), WARN_ON(), WARN_ON_ONCE, and so on can be used to report
- * significant kernel issues that need prompt attention if they should ever
- * appear at runtime.
+ * WARN(), WARN_ON(), WARN_ON_ONCE, and so on can be used to report significant
+ * issues that need prompt attention if they should ever appear at runtime.
 *
 * Do not use these macros when checking for invalid external inputs
 * (e.g. invalid system call arguments, or invalid data coming from
@@ -50,6 +46,7 @@
 #define __WARN() do {                                                      \
        fprintf(stderr, "WARNING: failure at %s:%d/%s()!\n", __FILE__, __LINE__, __func__); \
    } while (0)
+
 #define __WARN_printf(arg...) do {              \
        vfprintf(stderr, arg);                  \
    } while (0)
--- a/include/hash.h
+++ b/include/hash.h
@@ -9,8 +9,9 @@
   (C) 2002 Nadia Yvette Chambers, IBM */

 #include <asm/bitsperlong.h>
-#include "bits.h"
+
 #include "br.h"
+#include "bitops.h"

 /*
 * The "GOLDEN_RATIO_PRIME" is used in ifs/btrfs/brtfs_inode.h and
--- a/include/list_sort.h
+++ b/include/list_sort.h
@@ -7,8 +7,6 @@
 #ifndef _BR_LIST_SORT_H
 #define _BR_LIST_SORT_H

-//#include <linux/types.h>
-
 struct list_head;

 typedef int __attribute__((nonnull(2,3))) (*list_cmp_func_t)(void *,
--- a/include/pjwhash-inline.h
+++ b/include/pjwhash-inline.h
@@ -1,6 +1,6 @@
 /* pjwhash-inline.h - PJW hash function, inline version.
 *
- * Copyright (C) 2021-2022 Bruno Raoult ("br")
+ * Copyright (C) 2021-2024 Bruno Raoult ("br")
 * Licensed under the GNU General Public License v3.0 or later.
 * Some rights reserved. See COPYING.
 *
@@ -14,7 +14,7 @@
 #ifndef _PJWHASH_INLINE_H
 #define _PJWHASH_INLINE_H

-#include "bits.h"
+#include "br.h"

 #define THREE_QUARTERS ((int) ((BITS_PER_INT * 3) / 4))
 #define ONE_EIGHTH     ((int) (BITS_PER_INT / 8))
--- a/include/pjwhash.h
+++ b/include/pjwhash.h
@@ -1,6 +1,6 @@
 /* pjwhash.h - PJW hash function, extern version.
 *
- * Copyright (C) 2021-2022 Bruno Raoult ("br")
+ * Copyright (C) 2021-2024 Bruno Raoult ("br")
 * Licensed under the GNU General Public License v3.0 or later.
 * Some rights reserved. See COPYING.
 *
@@ -13,7 +13,7 @@
 #ifndef _PJWHASH_H
 #define _PJWHASH_H

-#include "bits.h"
+#include "br.h"

 /**
 * unsigned int pjwhash - PJW hash function
--- a/include/pool.h
+++ b/include/pool.h
@@ -1,6 +1,6 @@
 /* pool.h - A simple memory pool manager.
 *
- * Copyright (C) 2021-2022 Bruno Raoult ("br")
+ * Copyright (C) 2021-2024 Bruno Raoult ("br")
 * Licensed under the GNU General Public License v3.0 or later.
 * Some rights reserved. See COPYING.
 *
@@ -16,8 +16,9 @@

 #include <stdint.h>
 #include <stddef.h>
+
+#include "br.h"
 #include "list.h"
-#include "bits.h"

 #define POOL_NAME_LENGTH (16)                     /* max name length including trailing \0 */

--- a/include/struct-group.h
+++ b/include/struct-group.h
@@ -1,6 +1,6 @@
 /* struct-group.h - mirrored structure macros.
 *
- * Copyright (C) 2021-2022 Bruno Raoult ("br")
+ * Copyright (C) 2021-2024 Bruno Raoult ("br")
 * Licensed under the GNU General Public License v3.0 or later.
 * Some rights reserved. See COPYING.
 *
@@ -33,10 +33,10 @@
 * as both having struct attributes appended.
 */
 #define __struct_group(TAG, NAME, ATTRS, MEMBERS...) \
-	union { \
-		struct { MEMBERS } ATTRS; \
-		struct TAG { MEMBERS } ATTRS NAME; \
-	}
+    union {                                          \
+        struct { MEMBERS } ATTRS;                    \
+        struct TAG { MEMBERS } ATTRS NAME;           \
+    }

 /**
 * DECLARE_FLEX_ARRAY() - Declare a flexible array usable in a union
@@ -48,11 +48,11 @@
 * struct, it needs to be wrapped in an anonymous struct with at least 1
 * named member, but that member can be empty.
 */
-#define DECLARE_FLEX_ARRAY(TYPE, NAME)	\
-	struct { \
-		struct { } __empty_ ## NAME; \
-		TYPE NAME[]; \
-	}
+#define DECLARE_FLEX_ARRAY(TYPE, NAME)	     \
+    struct {                                 \
+        struct { } __empty_ ## NAME;         \
+        TYPE NAME[];                         \
+    }

 /**
 * struct_group() - Wrap a set of declarations in a mirrored struct
@@ -67,7 +67,7 @@
 * struct members.
 */
 #define struct_group(NAME, MEMBERS...)	\
-	__struct_group(/* no tag */, NAME, /* no attrs */, MEMBERS)
+    __struct_group(/* no tag */, NAME, /* no attrs */, MEMBERS)

 /**
 * struct_group_attr() - Create a struct_group() with trailing attributes
@@ -83,7 +83,7 @@
 * struct members. Includes structure attributes argument.
 */
 #define struct_group_attr(NAME, ATTRS, MEMBERS...) \
-	__struct_group(/* no tag */, NAME, ATTRS, MEMBERS)
+    __struct_group(/* no tag */, NAME, ATTRS, MEMBERS)

 /**
 * struct_group_tagged() - Create a struct_group with a reusable tag
@@ -100,6 +100,6 @@
 * so the specified layout can be reused later.
 */
 #define struct_group_tagged(TAG, NAME, MEMBERS...) \
-	__struct_group(TAG, NAME, /* no attrs */, MEMBERS)
+    __struct_group(TAG, NAME, /* no attrs */, MEMBERS)

 #endif  /* _STRUCT_GROUP_H */