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

2018-02-06 Thread Jorgen Lundman
@lundman pushed 1 commit. 114b91e Handle compressed buffers in __dbuf_hold_impl() -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/openzfs/openzfs/pull/489/files/d6cda08bf1d56c7f960d626687818f071ab27250..114b91e4d42657e243d0eb670de58579

[developer] Re: [openzfs/openzfs] 8115 parallel zfs mount (v2) (#536)

2018-02-06 Thread Robert Mustacchi
When trying to review this, I found the use of the fake kernel here really confusing. Why are we using it for a normal user land program? Doesn't it strike others as odd when you have a userland program use a `kmutex_t`, but then also referring to the userland bits like a normal USYNC_THREAD, wh

[developer] Re: [openzfs/openzfs] XXX Attempt to fix remove_mirror (#534)

2018-02-06 Thread Matthew Ahrens
Superseded by https://github.com/openzfs/openzfs/pull/537 -- 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/534#issuecomment-363607351 -- openzfs-d

[developer] Re: [openzfs/openzfs] XXX Attempt to fix remove_mirror (#534)

2018-02-06 Thread Matthew Ahrens
Closed #534. -- 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/534#event-1461325772 -- openzfs-developer Archives: https://openzfs.topicbox.com/gr

[developer] Re: [openzfs/openzfs] 9064 ZFS test remove_mirror should wait for device removal to complete (#537)

2018-02-06 Thread Matthew Ahrens
ahrens 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/537#pullrequestreview-94549984 -- openzfs-developer Archiv

[developer] Re: [openzfs/openzfs] XXX Attempt to fix remove_mirror (#534)

2018-02-06 Thread Prashanth Sreenivasa
Ok thanks @loli10K and @pzakha Here's the PR to address this: https://github.com/openzfs/openzfs/pull/537 -- 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/534#issuecomment-363606177 --

[developer] Re: [openzfs/openzfs] 8115 parallel zfs mount (v2) (#536)

2018-02-06 Thread Prakash Surya
@andy-js I'd appreciate your review on this, if you have the time. I've rebased this onto the latest master changes, and the `libfakekernel` changes you made in #494. I ran into some build issues where some files would `#include ` and this would conflict with symbols/macros from `libfakekernel`

[developer] [openzfs/openzfs] 9064 ZFS test remove_mirror should wait for device removal to complete (#537)

2018-02-06 Thread Prashanth Sreenivasa
Fixes the transient failure of `remove_mirror` zfs-test run(s). You can view, comment on, or merge this pull request online at: https://github.com/openzfs/openzfs/pull/537 -- Commit Summary -- * 9064 ZFS test remove_mirror should wait for device removal to complete -- File Changes --

[developer] Re: [openzfs/openzfs] 8115 parallel zfs mount (#451)

2018-02-06 Thread Prakash Surya
Closed #451. -- 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/451#event-1461296436 -- openzfs-developer Archives: https://openzfs.topicbox.com/gr

[developer] Re: [openzfs/openzfs] 8115 parallel zfs mount (#451)

2018-02-06 Thread Prakash Surya
superseded by https://github.com/openzfs/openzfs/pull/536 -- 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/451#issuecomment-363602620 -- openzfs-d

[developer] [openzfs/openzfs] 8115 parallel zfs mount (v2) (#536)

2018-02-06 Thread Prakash Surya
Reviewed by: Matthew Ahrens mahr...@delphix.com Reviewed by: Pavel Zakharov pavel.zakha...@delphix.com Reviewed by: Brad Lewis Reviewed by: George Wilson Reviewed by: Paul Dagnelie Reviewed by: Prashanth Sreenivasa Overview In analyzing the time it takes for a Delphix Engine to come up follow

[developer] Re: [openzfs/openzfs] XXX Attempt to fix remove_mirror (#534)

2018-02-06 Thread Pavel Zakharov
Note that I've opened https://www.illumos.org/issues/9064 to track this -- 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/534#issuecomment-363539137 --

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

2018-02-06 Thread zettabot
You can view, comment on, or merge this pull request online at: https://github.com/openzfs/openzfs/pull/535 -- Commit Summary -- * 8792 mac_tx_srs_quiesce() can kill a maxbw link * 8652 Tautological comparisons with ZPROP_INVAL (followup) * 8760 troff: variable 'type' set but not used

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

2018-02-06 Thread Prakash Surya
Closed #533. -- 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/533#event-1460686728 -- openzfs-developer Archives: https://openzfs.topicbox.com/gr

[developer] Re: [openzfs/openzfs] 8962 zdb should work on non-idle pools (#520)

2018-02-06 Thread Prakash Surya
Closed #520 via 180ad7925030b21d95932228f1023090ec7d6c3b. -- 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/520#event-1460666478 -- openzfs-develop

[developer] Re: [openzfs/openzfs] 8961 SPA load/import should tell us why it failed (#519)

2018-02-06 Thread Prakash Surya
Closed #519 via 418079e020fb86f40d6d8397026bd2d0a3101bab. -- 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/519#event-1460665717 -- openzfs-develop

[developer] Re: [openzfs/openzfs] 7638 Refactor spa_load_impl into several functions (#518)

2018-02-06 Thread Prakash Surya
Closed #518 via 9fd6bc8362992ecb68cf9e891c0b53440c53a2d5. -- 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/518#event-1460665020 -- openzfs-develop

[developer] Re: [openzfs/openzfs] 8966 Source file zfs_acl.c, function zfs_aclset_common contains a use after end of the lifetime of a local variable (#524)

2018-02-06 Thread Prakash Surya
Closed #524 via c95549fcdc1972cec19d6ca5adee4551b07e4cd6. -- 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/524#event-1460663767 -- openzfs-develop

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

2018-02-06 Thread GernotS
This change did not cure my issue yet. I managed to write 15GB using dd until it crashed again. -- 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/489#issuecomment-363355614 --