Print this page
3742 zfs comments need cleaner, more consistent style
Submitted by:   Will Andrews <willa@spectralogic.com>
Submitted by:   Alan Somers <alans@spectralogic.com>
Reviewed by:    Matthew Ahrens <mahrens@delphix.com>
Reviewed by:    George Wilson <george.wilson@delphix.com>
Reviewed by:    Eric Schrock <eric.schrock@delphix.com>

Split Close
Expand all
Collapse all
          --- old/usr/src/uts/common/fs/zfs/bptree.c
          +++ new/usr/src/uts/common/fs/zfs/bptree.c
↓ open down ↓ 35 lines elided ↑ open up ↑
  36   36  #include <sys/refcount.h>
  37   37  #include <sys/spa.h>
  38   38  
  39   39  /*
  40   40   * A bptree is a queue of root block pointers from destroyed datasets. When a
  41   41   * dataset is destroyed its root block pointer is put on the end of the pool's
  42   42   * bptree queue so the dataset's blocks can be freed asynchronously by
  43   43   * dsl_scan_sync. This allows the delete operation to finish without traversing
  44   44   * all the dataset's blocks.
  45   45   *
  46      - * Note that while bt_begin and bt_end are only ever incremented in this code
       46 + * Note that while bt_begin and bt_end are only ever incremented in this code,
  47   47   * they are effectively reset to 0 every time the entire bptree is freed because
  48   48   * the bptree's object is destroyed and re-created.
  49   49   */
  50   50  
  51   51  struct bptree_args {
  52   52          bptree_phys_t *ba_phys; /* data in bonus buffer, dirtied if freeing */
  53   53          boolean_t ba_free;      /* true if freeing during traversal */
  54   54  
  55   55          bptree_itor_t *ba_func; /* function to call for each blockpointer */
  56   56          void *ba_arg;           /* caller supplied argument to ba_func */
↓ open down ↓ 169 lines elided ↑ open up ↑
XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX