Re: [OpenZFS Developer] [openzfs] 6418 zpool should have a label clearing command (#32)

2015-11-01 Thread Justin T. Gibbs
@ilovezfs Regardless of the labeling information you use (MBR, GPT, GPT with protective MBR, BSD disk label, UFS labels, ZFS labels, GELI encryption labels, software RAID metadata, etc, etc) there is a possibility for conflict. The only way to deal with this is to prioritize the order in which

Re: [OpenZFS Developer] [openzfs] 6392 zdb: Introduce -V for verbatim import (#22)

2015-11-01 Thread Matthew Ahrens
@ryao It looks like you will need to update zdb_001_neg (see the test results; even without the full log I think the problem is clear). --- Reply to this email directly or view it on GitHub:

Re: [OpenZFS Developer] [openzfs] 6418 zpool should have a label clearing command (#32)

2015-11-01 Thread ilovezfs
@scsiguy >The only way to deal with this is to prioritize the order in which handlers >for these formats attach. OS X Disk Arbitration has always had FSProbeOrder, so, no, probe order alone is not a panacea for the described problem, though, yes, like partitioning, probe order can and does

Re: [OpenZFS Developer] [openzfs] 6392 zdb: Introduce -V for verbatim import (#22)

2015-11-01 Thread George Wilson
LGTM --- Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/22#issuecomment-152831278___ developer mailing list developer@open-zfs.org http://lists.open-zfs.org/mailman/listinfo/developer

Re: [OpenZFS Developer] [openzfs] 6418 zpool should have a label clearing command (#32)

2015-11-01 Thread Justin T. Gibbs
@grwilson I think labeling is quite complicated. ZFS has subsumed many other, traditionally separate, storage layer components (e.g. volume management, and software RAID), but doesn't offer a complete replacement for partitioning and labeling tools. The question is, should ZFS take this over

Re: [OpenZFS Developer] [openzfs] 6414 vdev_config_sync could be simpler (#31)

2015-11-01 Thread George Wilson
Looks good to me --- Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/31#issuecomment-152831553___ developer mailing list developer@open-zfs.org http://lists.open-zfs.org/mailman/listinfo/developer

Re: [OpenZFS Developer] Review Request 266: 6400 ZFS smart compression

2015-11-01 Thread Saso Kiselkov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/266/ --- (Updated Nov. 2, 2015, 2:48 a.m.) Review request for OpenZFS Developer Mailing

Re: [OpenZFS Developer] [openzfs] IllumOS 6183 upgrade soft panic (#17)

2015-11-01 Thread Matthew Ahrens
@zettabot go --- Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/17#issuecomment-152888637___ developer mailing list developer@open-zfs.org http://lists.open-zfs.org/mailman/listinfo/developer

Re: [OpenZFS Developer] [openzfs] 6418 zpool should have a label clearing command (#32)

2015-11-01 Thread Matthew Ahrens
@zettabot go --- Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/32#issuecomment-152912185___ developer mailing list developer@open-zfs.org http://lists.open-zfs.org/mailman/listinfo/developer

Re: [OpenZFS Developer] Review Request 263: 6363 Add UNMAP/TRIM functionality to ZFS and Illumos

2015-11-01 Thread Saso Kiselkov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/263/ --- (Updated Nov. 2, 2015, 3:07 a.m.) Review request for OpenZFS Developer Mailing

Re: [OpenZFS Developer] [developer] Review: 6333 ZFS should let the user specify or modify the fsid_guid of a dataset

2015-11-01 Thread Matthew Ahrens
On Sun, Nov 1, 2015 at 4:36 PM, Alexandre Lécuyer < alexandre.lecu...@corp.ovh.com> wrote: > Thank you for reviewing the patch. Sorry about the style and indentation > issues, I have corrected the ones you mentionned and ran cstyle on all > changed files. > > The webrev is updated, and I have

Re: [OpenZFS Developer] Review Request 266: 6400 ZFS smart compression

2015-11-01 Thread Matthew Ahrens
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/266/#review865 --- usr/src/man/man1m/zfs.1m (line 1034)

Re: [OpenZFS Developer] [openzfs] 6391 Override default SPA config location via environment (#21)

2015-11-01 Thread George Wilson
LGTM --- Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/21#issuecomment-152831323___ developer mailing list developer@open-zfs.org http://lists.open-zfs.org/mailman/listinfo/developer