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>

@@ -240,25 +240,24 @@
  * Log record for creates with optional ACL
  * This log record is also used for recording any FUID
  * information needed for replaying the create.  If the
  * file doesn't have any actual ACEs then the lr_aclcnt
  * would be zero.
+ *
+ * After lr_acl_flags, there are a lr_acl_bytes number of variable sized ace's.
+ * If create is also setting xvattr's, then acl data follows xvattr.
+ * If ACE FUIDs are needed then they will follow the xvattr_t.  Following
+ * the FUIDs will be the domain table information.  The FUIDs for the owner
+ * and group will be in lr_create.  Name follows ACL data.
  */
 typedef struct {
         lr_create_t     lr_create;      /* common create portion */
         uint64_t        lr_aclcnt;      /* number of ACEs in ACL */
         uint64_t        lr_domcnt;      /* number of unique domains */
         uint64_t        lr_fuidcnt;     /* number of real fuids */
         uint64_t        lr_acl_bytes;   /* number of bytes in ACL */
         uint64_t        lr_acl_flags;   /* ACL flags */
-        /* lr_acl_bytes number of variable sized ace's follows */
-        /* if create is also setting xvattr's, then acl data follows xvattr */
-        /* if ACE FUIDs are needed then they will follow the xvattr_t */
-        /* Following the FUIDs will be the domain table information. */
-        /* The FUIDs for the owner and group will be in the lr_create */
-        /* portion of the record. */
-        /* name follows ACL data */
 } lr_acl_create_t;
 
 typedef struct {
         lr_t            lr_common;      /* common portion of log record */
         uint64_t        lr_doid;        /* obj id of directory */