Illumos #1644, #1645, #1646, #1647, #1708

[mmatuska]

1644 add ZFS "clones" property
1645 add ZFS "written" and "written@..." properties
1646 "zfs send" should estimate size of stream
1647 "zfs destroy" should determine space reclaimed by destroying multiple snapshots
1708 adjust size of zpool history data
Reviewed by: Richard Lowe richlowe@richlowe.net
Reviewed by: George Wilson gwilson@zfsmail.com
Reviewed by: Albert Lee trisk@opensolaris.org
Approved by: Garrett D'Amore garret@nexenta.com

Ported by: Martin Matuska martin@matuska.org

I have also fixed the 2 issues you reported.
What about moving reguid and the other IOCTL's to the end? This way you keep the backwards compatibility with old ioctls (but may still break with delsnaps, but there is no real choice).

[behlendorf]

Thanks for the refreshed patch, I'll give it another try.

As for the ioctl reordering, I don't see a great way to deal with it, so I think we'll just have to live with it. I'd rather keep the ioctl's in the same order as the other implementations anyway. We'll just have to let the various Linux distribution maintains know so they can leverage their packaging systems to strongly encourage a reboot.

[behlendorf]

While this patch builds cleanly, and I did a review of the code, when it's applied I'm observing at least one strange issue. The command zpool list no longer shows me all my imported pools instead it displays dashes. Until we run this to ground and get some more testing of this change I can't merge it.

[mmatuska]

The problem was in the ioctl order in sys/fs/zfs.h - that file wasn't updated.
As of my current patch, users updating from rc9 should have all ioctl's working (only exception: destroy_snaps, here the arguments have changed)

[behlendorf]

Thanks for running this to ground. I'll queue it up again for further testing with your fix.