sys/vfs/hammer2: Move duplicated macros H2FM*() to hammer2_disk.h
authorTomohiro Kusumi <kusumi.tomohiro@gmail.com>
Mon, 12 Aug 2019 16:13:47 +0000 (01:13 +0900)
committerTomohiro Kusumi <kusumi.tomohiro@gmail.com>
Mon, 12 Aug 2019 16:21:11 +0000 (01:21 +0900)
These macros exist in both freemap and bulkfree code.
Should be part of ondisk layout spec.

sys/vfs/hammer2/hammer2_bulkfree.c
sys/vfs/hammer2/hammer2_disk.h
sys/vfs/hammer2/hammer2_freemap.c

index 062a7b2..f77a762 100644 (file)
 
 #include "hammer2.h"
 
-/*
- * XXX I made a mistake and made the reserved area begin at each LEVEL1 zone,
- *     which is on a 1GB demark.  This will eat a little more space but for
- *     now we retain compatibility and make FMZONEBASE every 1GB
- */
-#define H2FMZONEBASE(key)      ((key) & ~HAMMER2_FREEMAP_LEVEL1_MASK)
-#define H2FMBASE(key, radix)   ((key) & ~(((hammer2_off_t)1 << (radix)) - 1))
-#define H2FMSHIFT(radix)       ((hammer2_off_t)1 << (radix))
-
 /*
  * breadth-first search
  */
index 0e037e8..1e1fcff 100644 (file)
 #define HAMMER2_FREEMAP_COUNT          (int)(HAMMER2_FREEMAP_LEVELN_PSIZE / \
                                         sizeof(hammer2_bmap_data_t))
 
+/*
+ * XXX I made a mistake and made the reserved area begin at each LEVEL1 zone,
+ *     which is on a 1GB demark.  This will eat a little more space but for
+ *     now we retain compatibility and make FMZONEBASE every 1GB
+ */
+#define H2FMZONEBASE(key)      ((key) & ~HAMMER2_FREEMAP_LEVEL1_MASK)
+#define H2FMBASE(key, radix)   ((key) & ~(((hammer2_off_t)1 << (radix)) - 1))
+#define H2FMSHIFT(radix)       ((hammer2_off_t)1 << (radix))
+
 /*
  * 16KB bitmap granularity (x2 bits per entry).
  */
index 1a97e35..7a2a1b6 100644 (file)
@@ -74,21 +74,10 @@ static int hammer2_freemap_iterate(hammer2_chain_t **parentp,
  * chains use fixed storage offsets in the 4MB reserved area at the
  * beginning of each 2GB zone
  *
- * XXX I made a mistake and made the reserved area begin at each LEVEL1 zone,
- *     which is on a 1GB demark.  This will eat a little more space but for
- *     now we retain compatibility and make FMZONEBASE every 1GB
- *
- *     (see same thing in hammer2_bulkfree.c near the top, as well as in
- *     newfs_hammer2).
- *
  * Rotate between four possibilities.  Theoretically this means we have three
  * good freemaps in case of a crash which we can use as a base for the fixup
  * scan at mount-time.
  */
-#define H2FMZONEBASE(key)      ((key) & ~HAMMER2_FREEMAP_LEVEL1_MASK)
-#define H2FMBASE(key, radix)   ((key) & ~(((hammer2_off_t)1 << (radix)) - 1))
-#define H2FMSHIFT(radix)       ((hammer2_off_t)1 << (radix))
-
 static
 int
 hammer2_freemap_reserve(hammer2_chain_t *chain, int radix)