[developer] Re: zfs: data vs metadata mismatch during force receive

2017-05-17 Thread Andriy Gapon
On 17/05/2017 11:59, Andriy Gapon wrote: > > A FreeBSD user reported a problem with doing a force receive of a full stream > (non-incremental) while using a debug build: > > panic: solaris assert: dbuf_is_metadata(db) == arc_is_metadata(buf) (0x0 == > 0x1) > assfail3() > dbuf_assign_arcbuf() >

[developer] Re: [openzfs/openzfs] Remove "zfs_acl_chmod_xattr_001_pos" from known zfstest failures (#380)

2017-05-17 Thread Prakash Surya
Closed #380 via 26f08bc1bde4cee2788863a3c15cb591e5b23ec0. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/380#event-1086328011 --

[developer] [openzfs/openzfs] 7432 Reserve send stream flag for large dnode feature (#381)

2017-05-17 Thread Brian Behlendorf
Reserve bit 23 in the zfs send stream flags for the large dnode feature which has been implemented for Linux. https://github.com/zfsonlinux/zfs/pull/3542 You can view, comment on, or merge this pull request online at: https://github.com/openzfs/openzfs/pull/381 -- Commit Summary -- * 7432

[developer] Re: [openzfs/openzfs] 7432 Reserve send stream flag for large dnode feature (#381)

2017-05-17 Thread Tom Caputi
doesn't the large dnodes send feature use bit 21? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/381#issuecomment-302202756 --

[developer] Re: [openzfs/openzfs] 6939 add sysevents to zfs core for commands (#101)

2017-05-17 Thread Matthew Ahrens
Looks like everyone is happy with these changes. @bahamas10 could you rebase on master to resolve the conflicts? Then we'll get this integrated. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[developer] Re: [openzfs/openzfs] 8166 zpool scrub thinks it repaired offline device (#372)

2017-05-17 Thread brad-lewis
brad-lewis approved this pull request. Looks good. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/372#pullrequestreview-38781210 --

[developer] Re: [openzfs/openzfs] 7432 Reserve send stream flag for large dnode feature (#381)

2017-05-17 Thread Tom Caputi
OK. I will adjust the encryption patch to use bit 24 for raw sends then. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/381#issuecomment-302234139

[developer] Re: [openzfs/openzfs] 7432 Reserve send stream flag for large dnode feature (#381)

2017-05-17 Thread Brian Behlendorf
@tcaputi it does, I've opened https://github.com/zfsonlinux/zfs/pull/6139 to switch it to avoid any future compatibility problems. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[developer] Re: [openzfs/openzfs] 8005 poor performance of 1MB writes on certain RAID-Z configurations (#321)

2017-05-17 Thread Matthew Ahrens
Also reviewed internally by @grwilson and @pzakha -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/321#issuecomment-302221128 --

[developer] [openzfs/openzfs] Add "zpool_create_016_pos" to known zfstest failures (#379)

2017-05-17 Thread Prakash Surya
The "zpool_create_016_pos" test will sporadically fail with the message: Test: /opt/zfs-tests/tests/functional/cli_root/zpool_create/zpool_create_016_pos (run as root) [00:00] [FAIL] 01:53:51.76 ASSERTION: 'zpool create' should success with no device in swap. 01:53:51.76 NOTE:

[developer] Re: [openzfs/openzfs] 6363 Add UNMAP/TRIM functionality to ZFS and Illumos (#172)

2017-05-17 Thread Matthew Ahrens
ahrens commented on this pull request. > @@ -3861,6 +3892,52 @@ zpool_do_scrub(int argc, char **argv) return (for_each_pool(argc, argv, B_TRUE, NULL, scrub_callback, )); } +/* + * zpool trim [-s|-r ] ... + * + * -s Stop. Stops any in-progress trim. + * -r

[developer] Re: [openzfs/openzfs] Merge remote-tracking branch 'illumos/master' into illumos-sync (#378)

2017-05-17 Thread Prakash Surya
Merged #378. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/378#event-1085373125 -- openzfs-developer Archives:

[developer] Re: [openzfs/openzfs] Merge remote-tracking branch 'illumos/master' into illumos-sync (#378)

2017-05-17 Thread Prakash Surya
prakashsurya approved this pull request. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/378#pullrequestreview-38577073 -- openzfs-developer

[developer] Re: [openzfs/openzfs] Merge remote-tracking branch 'illumos/master' into illumos-sync (#378)

2017-05-17 Thread Prakash Surya
this failed zfstest with the following: ``` Tests with results other than PASS that are unexpected: FAIL cli_root/zpool_create/zpool_create_016_pos (expected PASS) ``` this failure isn't due to the code being tested, so I'll approve and merge this. additionally, I've opened

[developer] [openzfs/openzfs] Remove "zfs_acl_chmod_xattr_001_pos" from known zfstest failures (#380)

2017-05-17 Thread Prakash Surya
You can view, comment on, or merge this pull request online at: https://github.com/openzfs/openzfs/pull/380 -- Commit Summary -- * Remove "zfs_acl_chmod_xattr_001_pos" from known zfstest failures -- File Changes -- M jenkins/sh/run-zfs-tests/zfstest-report.py (1) -- Patch Links --

[developer] Re: [openzfs/openzfs] Native data and metadata encryption for zfs (#124)

2017-05-17 Thread Jorgen Lundman
The PR is just 3 weeks away from its first birthday, yaaay \o/ -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/124#issuecomment-302028603 --

[developer] zfs: data vs metadata mismatch during force receive

2017-05-17 Thread Andriy Gapon
A FreeBSD user reported a problem with doing a force receive of a full stream (non-incremental) while using a debug build: panic: solaris assert: dbuf_is_metadata(db) == arc_is_metadata(buf) (0x0 == 0x1) assfail3() dbuf_assign_arcbuf() dmu_assign_arcbuf() receive_writer_thread() Here is a