A brief line about why "zpool" and "tank" could be useful as they are used throughout the documentation without any explitaion of why or how those labels were used. I think zpool was used because it was a zfs-pool to get away from the "tank" name which I always thought was an odd name but it persists in documentation so I would expect it to be mentioned.

This appears like it will get suck in a infinite loop when it finds #PROBLEMS!==== in /etc/services.