Re: [OpenZFS Developer] Review Request 147: 5438 zfs_blkptr_verify should continue after zfs_panic_recover

2014-12-16 Thread George Wilson
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/147/#review407 --- Ship it! Ship It! - George Wilson On Dec. 16, 2014, 6:39 a.m., Xin

Re: [OpenZFS Developer] Review Request 128: 5313 Allow I/Os to be aggregated across ZIO priority classes

2014-12-16 Thread George Wilson
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/128/#review408 --- Ship it! Ship It! - George Wilson On Dec. 2, 2014, 12:22 p.m.,

Re: [OpenZFS Developer] Review Request 131: [5056] ZFS deadlock on db_mtx and dn_holds/Various improvements the dmu buf user API.

2014-12-16 Thread Justin Gibbs
On Dec. 15, 2014, 4:18 p.m., George Wilson wrote: usr/src/uts/common/fs/zfs/dmu_objset.c, lines 663-668 https://reviews.csiden.org/r/131/diff/5/?file=13206#file13206line663 Was there a problem with calling dsl_dataset_is_snapshot() and being more explicity about the check? It

Re: [OpenZFS Developer] Review Request 131: [5056] ZFS deadlock on db_mtx and dn_holds/Various improvements the dmu buf user API.

2014-12-16 Thread Justin Gibbs
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/131/ --- (Updated Dec. 16, 2014, 7:32 p.m.) Review request for OpenZFS Developer Mailing

Re: [OpenZFS Developer] Review Request 112: account for ashift when choosing buffers to be written to l2arc device

2014-12-16 Thread Prakash Surya
On Nov. 30, 2014, 9:02 p.m., Matthew Ahrens wrote: You will need to test on illumos too, at least some minimal sanity checking. Let me know if you need help with that. Andriy Gapon wrote: I am still not set up for illumos testing :-( So I must ask for your help again. Thank

Re: [OpenZFS Developer] Review Request 143: 5393 spurious failures from dsl_dataset_hold_obj()

2014-12-16 Thread Prakash Surya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/143/#review411 --- I've kicked this off to our internal regression test infrastructure: