Re: [developer] Fix for "illumos#3821 Race in rollback, zil close, and zil flush" is incomplete

2017-08-21 Thread Prakash Surya
Hi Jerry, Would it be possible for you to try out the patch contained here: https://github.com/openzfs/openzfs/pull/421 ? A direct link to the diff (as opposed the review/comments) is here: https://patch-diff.githubusercontent.com/raw/openzfs/openzfs/pull/421.diff When developing that change, I r

Re: [developer] Fix for "illumos#3821 Race in rollback, zil close, and zil flush" is incomplete

2017-08-21 Thread Jerry Jelinek
Hi Prakash, We have only seen this panic one time since the original fix was integrated into illumos. I have no way to properly test your new change or reproduce the problem from this bug. I have been working with George Wilson to try to get him access to the dump for debugging. Maybe the best pat

Re: [developer] Fix for "illumos#3821 Race in rollback, zil close, and zil flush" is incomplete

2017-08-21 Thread Prakash Surya
Sounds good. I'll sync up with George. Thanks! On Mon, Aug 21, 2017 at 8:02 AM, Jerry Jelinek wrote: > Hi Prakash, > > We have only seen this panic one time since the original fix was > integrated into illumos. I have no way to properly test your new change or > reproduce the problem from this b

[developer] Re: [openzfs/openzfs] 8414 Implemented zpool scrub pause/resume (#407)

2017-08-21 Thread Prakash Surya
Closed #407 via c296160769c460efd21c8619cb23f91525e1e9d5. -- 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/407#event-1215143893 -- openzfs-develop

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

2017-08-21 Thread Prakash Surya
Closed #445. -- 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/445#event-1215144547 -- openzfs-developer Archives: https://openzfs.topicbox.com/gr

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

2017-08-21 Thread zettabot
You can view, comment on, or merge this pull request online at: https://github.com/openzfs/openzfs/pull/446 -- Commit Summary -- * 8562 xencons: 'instance' may be used uninitialized in this function * 8557 remove trusted desktop remnants * 8541 pfiles does not properly identify PF_KEY o