Re: [OpenZFS Developer] [openzfs] 6370 ZFS send fails to transmit some holes (#37)

2016-01-28 Thread Paul Dagnelie
@stevenburgess It's still in review internally (mostly regarding changes to zfs_send_007_pos.ksh et al), but when we push here, it'll be that version. Thanks for confirming it works on your end! --- Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/37#is

Re: [OpenZFS Developer] [openzfs] 6541 dedup=[cksum], verify defeated spa feature check (#51)

2016-01-28 Thread Richard Laager
> @@ -210,8 +212,8 @@ zpool_feature_init(void) > ZFEATURE_FLAG_READONLY_COMPAT, bookmarks_deps); > > static const spa_feature_t filesystem_limits_deps[] = { > - SPA_FEATURE_EXTENSIBLE_DATASET, > - SPA_FEATURE_NONE > + SPA_FEATURE_EXTENSIBLE_DATASET, So

Re: [OpenZFS Developer] [openzfs] 6541 dedup=[cksum], verify defeated spa feature check (#51)

2016-01-28 Thread ilovezfs
``` ztest: '/usr/sbin/i86/zdb -bccsv -d -U /var/tmp/jenkins/workspace/openzfs-run-ztest/zpool.cache ztest' exit code 2 running: echo ::stack mdb: failed to open core: No such file or directory Build step 'Execute shell' marked build as failure Finished: FAILURE ``` Not sure what, if anything, to m

[OpenZFS Developer] Jsonify outputs done #3938

2016-01-28 Thread Yacine Kheddache
Hi, Been a while I did not reply to this thread but lots of progress has been done in the mean time. We understand it is a pretty disruptive append to the code base but at least it is not IMHO introduce disruptive changes. We are closed to the final step now, all commands have been implemented u

Re: [OpenZFS Developer] [openzfs] 6513 partially filled holes lose birth time (#46)

2016-01-28 Thread Boris Protopopov
Hi, @pcd1193182 , @ahrens, I have been working on this problem concurrent with your work presented in this pull request. I wanted to summarize my comments and ask for your review of how they were addressed in https://github.com/zfsonlinux/zfs/pull/4024. One somewhat cosmetic issue is proper lo

Re: [OpenZFS Developer] [openzfs] 6541 dedup=[cksum], verify defeated spa feature check (#51)

2016-01-28 Thread ilovezfs
> @@ -210,8 +212,8 @@ zpool_feature_init(void) > ZFEATURE_FLAG_READONLY_COMPAT, bookmarks_deps); > > static const spa_feature_t filesystem_limits_deps[] = { > - SPA_FEATURE_EXTENSIBLE_DATASET, > - SPA_FEATURE_NONE > + SPA_FEATURE_EXTENSIBLE_DATASET, Do

Re: [OpenZFS Developer] [openzfs] 6562 Refquota on receive doesn't account for overage. (#64)

2016-01-28 Thread Yuri Pankov
@danmcd: Tests with results other than PASS that are unexpected: FAIL zvol/zvol_swap/zvol_swap_006_pos (expected PASS) Don't think it's related to your changes, but anyway... --- Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/64#issuecomment-176434

Re: [OpenZFS Developer] [openzfs] 6562 Refquota on receive doesn't account for overage. (#64)

2016-01-28 Thread Dan McDonald
Hmmm... I saw this: ``` failed: /usr/bin/python /usr/local/zfstest-report.py results.txt Build step 'Execute shell' marked build as failure Finished: FAILURE ``` What happened? --- Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/64#issuecomment-17635060

Re: [OpenZFS Developer] [openzfs] 6541 dedup=[cksum], verify defeated spa feature check (#51)

2016-01-28 Thread Richard Laager
> @@ -210,8 +212,8 @@ zpool_feature_init(void) > ZFEATURE_FLAG_READONLY_COMPAT, bookmarks_deps); > > static const spa_feature_t filesystem_limits_deps[] = { > - SPA_FEATURE_EXTENSIBLE_DATASET, > - SPA_FEATURE_NONE > + SPA_FEATURE_EXTENSIBLE_DATASET, Wh

Re: [OpenZFS Developer] [openzfs] 6541 dedup=[cksum], verify defeated spa feature check (#51)

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

Re: [OpenZFS Developer] [openzfs] Improve speculative indirection tables prefetch. (#65)

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

Re: [OpenZFS Developer] [openzfs] 6541 dedup=[cksum], verify defeated spa feature check (#51)

2016-01-28 Thread ilovezfs
@rlaager The fixes for the man page and whitespace problems you flagged, as well as for a couple other whitespace anomalies I noticed, are now in there, if you'd like to look it over. @mahrens Any thoughts on those questions above, and is this looking ready for Mr. Zetta Bot's water tortures?

Re: [OpenZFS Developer] [openzfs] Improve speculative indirection tables prefetch. (#65)

2016-01-28 Thread Steven Hartland
> @@ -305,6 +305,11 @@ int dnode_next_offset(dnode_t *dn, int flags, uint64_t > *off, > void dnode_evict_dbufs(dnode_t *dn); > void dnode_evict_bonus(dnode_t *dn); > > +#define DNODE_IS_CACHEABLE(_dn, _ind) > \ Typo missing space? --- Reply to this ema

Re: [OpenZFS Developer] [openzfs] Improve speculative indirection tables prefetch. (#65)

2016-01-28 Thread zettabot
Can one of the admins verify this patch? --- Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/65#issuecomment-176074350___ developer mailing list developer@open-zfs.org http://lists.open-zfs.org/mailman/listinfo/

[OpenZFS Developer] [openzfs] Improve speculative indirection tables prefetch. (#65)

2016-01-28 Thread Alexander Motin
Scalability of many operations on wide ZFS pool can be limited by requirement to prefetch indirection tables first. Recently added asynchronous indirection tables read partially helped, but did not solve the problem completely. This patch extends existing prefetcher functionality to explicitly wo