sys/vfs/hammer: Fix terminology for undo fifo
authorTomohiro Kusumi <kusumi.tomohiro@gmail.com>
Thu, 3 Nov 2016 05:51:32 +0000 (14:51 +0900)
committerTomohiro Kusumi <kusumi.tomohiro@gmail.com>
Thu, 3 Nov 2016 07:04:07 +0000 (16:04 +0900)
Always use "UNDO/REDO FIFO" whenever printf/manpage/etc refer to
undo in terms of HAMMER's zone-3 data structure, because that's
how it's been used originally.

But leave comments as they are now. There are tons of comments
that refer to UNDO, but not REDO when they could be UNDO/REDO,
but those mostly come from the fact that REDO was added later.

sbin/hammer/hammer.8
sbin/newfs_hammer/newfs_hammer.8
sys/vfs/hammer/hammer_disk.h
sys/vfs/hammer/hammer_redo.c
sys/vfs/hammer/hammer_undo.c

index 9a4096f..089dbb1 100644 (file)
@@ -388,7 +388,7 @@ option.
 .It Cm checkmap
 Check the blockmap allocation count.
 .Nm
-will scan the freemap, B-Tree, UNDO FIFO, then collect allocation information,
+will scan the freemap, B-Tree, UNDO/REDO FIFO, then collect allocation information,
 and construct a blockmap in-memory.
 It will then check that blockmap against the on-disk blockmap.
 .Pp
@@ -798,7 +798,7 @@ as it contains
 .Ar filesystem
 meta data such as
 .Nm HAMMER Ns 's
-layer1 blockmap and UNDO FIFO.
+layer1 blockmap and UNDO/REDO FIFO.
 .Pp
 This command may
 .Cm reblock
index 4299cac..25ac29d 100644 (file)
@@ -89,7 +89,7 @@ file systems are designed for large storage systems, up to 1 Exabyte, and
 will not operate efficiently on small storage systems.
 The minimum recommended file system size is 50GB.
 .Nm HAMMER
-must reserve 512MB to 1GB of its storage for reblocking and UNDO/REDO.
+must reserve 512MB to 1GB of its storage for reblocking and UNDO/REDO FIFO.
 In addition,
 .Nm HAMMER
 file systems operating normally, with full history
index 80f7192..e0fef76 100644 (file)
@@ -133,7 +133,7 @@ typedef uint32_t hammer_crc_t;
  * zone 0:             available, a big-block that contains the offset is unused
  * zone 1 (z,v,o):     raw volume relative (offset 0 is the volume header)
  * zone 2 (z,v,o):     raw buffer relative (offset 0 is the first buffer)
- * zone 3 (z,o):       undo fifo       - actually zone-2 address, fixed phys array in vol hdr
+ * zone 3 (z,o):       undo/redo fifo  - fixed zone-2 offset array in volume header
  * zone 4 (z,v,o):     freemap         - only real blockmap
  * zone 8 (z,v,o):     B-Tree          - actually zone-2 address
  * zone 9 (z,v,o):     meta            - actually zone-2 address
index 016d2fc..e036638 100644 (file)
@@ -88,7 +88,7 @@ hammer_generate_redo(hammer_transaction_t trans, hammer_inode_t ip,
 
        /* undo had better not roll over (loose test) */
        if (hammer_undo_space(trans) < len + HAMMER_BUFSIZE*3)
-               hpanic("insufficient undo FIFO space for redo!");
+               hpanic("insufficient UNDO/REDO FIFO space for redo!");
 
        /*
         * Loop until the undo for the entire range has been laid down.
index ea54934..ed66081 100644 (file)
@@ -142,7 +142,7 @@ hammer_generate_undo(hammer_transaction_t trans,
 
        /* undo had better not roll over (loose test) */
        if (hammer_undo_space(trans) < len + HAMMER_BUFSIZE*3)
-               hpanic("insufficient undo FIFO space!");
+               hpanic("insufficient UNDO/REDO FIFO space for undo!");
 
        /*
         * Loop until the undo for the entire range has been laid down.