I see a consistent coding pattern in the various zfs object cache constructors. These constructors always bzero the new structure.
Given this, all subsequent initializations, that merely set various fields to 0 or NULL are unnecessary. No big deal, but fewer CPU cycles, and fewer lines of code to look at are both good. Also, someone may think that those init's are necessary, and chase a non-existent bug when one of them is missing. :-) -- This message posted from opensolaris.org
