parlib: Mark const bitops
authorBarret Rhoden <brho@cs.berkeley.edu>
Fri, 8 Sep 2017 14:50:41 +0000 (10:50 -0400)
committerBarret Rhoden <brho@cs.berkeley.edu>
Fri, 8 Sep 2017 17:49:51 +0000 (13:49 -0400)
This will make them more usable for perf.

If someone wants to spatch-rename these functions, go for it.

Signed-off-by: Barret Rhoden <brho@cs.berkeley.edu>
user/parlib/include/parlib/bitmask.h
user/parlib/include/parlib/riscv/bitmask.h
user/parlib/include/parlib/x86/bitmask.h

index d883ced..eafe28c 100644 (file)
@@ -4,7 +4,8 @@
 
 __BEGIN_DECLS
 
-static inline bool BITMASK_IS_SET_IN_RANGE(uint8_t* m, size_t beg, size_t end)
+static inline bool BITMASK_IS_SET_IN_RANGE(const uint8_t *m, size_t beg,
+                                           size_t end)
 {
        for(size_t i=beg; i<end; i++) {
                if(!GET_BITMASK_BIT(m, i))
@@ -13,7 +14,8 @@ static inline bool BITMASK_IS_SET_IN_RANGE(uint8_t* m, size_t beg, size_t end)
        return TRUE;
 }
 
-static inline bool BITMASK_IS_CLR_IN_RANGE(uint8_t* m, size_t beg, size_t end)
+static inline bool BITMASK_IS_CLR_IN_RANGE(const uint8_t *m, size_t beg,
+                                           size_t end)
 {
        for(size_t i=beg; i<end; i++) {
                if(GET_BITMASK_BIT(m, i))
index 14bf590..aa4c39c 100644 (file)
@@ -17,7 +17,7 @@ __BEGIN_DECLS
 #define BYTES_FOR_BITMASK_WITH_CHECK(size) \
        ((size) ? ((size) - (1)) / (8) + (1) : (0))
 
-static bool GET_BITMASK_BIT(uint8_t* name, size_t bit) 
+static bool GET_BITMASK_BIT(const uint8_t *name, size_t bit)
 {
        return (((name)[(bit)/8] & (1 << ((bit) % 8))) ? 1 : 0);
 }
@@ -78,7 +78,7 @@ static void SET_BITMASK_BIT_ATOMIC(uint8_t* name, size_t bit)
        } \
        clear; })
 
-static inline bool BITMASK_IS_FULL(uint8_t* map, size_t size)
+static inline bool BITMASK_IS_FULL(const uint8_t *map, size_t size)
 {
        int _size = size;
        for (int i = 0; i < BYTES_FOR_BITMASK(size); i++) {
index 5e34c53..1c9f24b 100644 (file)
@@ -14,7 +14,7 @@ __BEGIN_DECLS
 #define BYTES_FOR_BITMASK(size) \
        (((size) - 1) / 8 + 1)
 
-static inline bool GET_BITMASK_BIT(uint8_t *name, size_t bit)
+static inline bool GET_BITMASK_BIT(const uint8_t *name, size_t bit)
 {
        return name[bit / 8] & (1 << (bit % 8)) ? TRUE : FALSE;
 }
@@ -50,13 +50,14 @@ static inline void FILL_BITMASK(uint8_t *name, size_t size)
        name[BYTES_FOR_BITMASK(size) - 1] >>= ((size % 8) ? (8 - (size % 8)) : 0);
 }
 
-static inline void COPY_BITMASK(uint8_t *newmask, uint8_t *oldmask, size_t size)
+static inline void COPY_BITMASK(uint8_t *newmask, const uint8_t *oldmask,
+                                size_t size)
 {
        memcpy(newmask, oldmask, BYTES_FOR_BITMASK(size));
 }
 
 /* this checks the entire last byte, so keep it 0 in the other functions */
-static bool BITMASK_IS_CLEAR(uint8_t *name, size_t size)
+static bool BITMASK_IS_CLEAR(const uint8_t *name, size_t size)
 {
        uint32_t __n = BYTES_FOR_BITMASK(size);
        bool clear = TRUE;
@@ -70,7 +71,7 @@ static bool BITMASK_IS_CLEAR(uint8_t *name, size_t size)
        return clear;
 }
 
-static inline bool BITMASK_IS_FULL(uint8_t *map, size_t size)
+static inline bool BITMASK_IS_FULL(const uint8_t *map, size_t size)
 {
        size_t nr_bytes = BYTES_FOR_BITMASK(size);
 
@@ -84,7 +85,7 @@ static inline bool BITMASK_IS_FULL(uint8_t *map, size_t size)
        return TRUE;
 }
 
-static inline void PRINT_BITMASK(uint8_t *name, size_t size)
+static inline void PRINT_BITMASK(const uint8_t *name, size_t size)
 {
        size_t nr_bytes = BYTES_FOR_BITMASK(size);