[developer] Re: [openzfs/openzfs] 8484 Implement aggregate sum and use for arc counters (#423)

2017-09-02 Thread C Fraire
Hi, @pcd1193182. I was inquiring about the inverse—whether including more information in hash will cause hashing to diverge where it was previously equal for similar values of \`os' or \`spa'. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or

[developer] Re: [openzfs/openzfs] 8585 improve batching done in zil_commit() (5d95a3a)

2017-09-02 Thread Sam Zaydel
I believe this should read `safe for it _to_ return`. -- 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/commit/5d95a3a1d59b9c1f59faeef7886c82eaf831f956#commitcomment-24046149

[developer] Re: [openzfs/openzfs] 8585 improve batching done in zil_commit() (5d95a3a)

2017-09-02 Thread Sam Zaydel
Sounds more correct to say: `contained within each` as opposed to `contained by each`. -- 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] 8604 Avoid unnecessary search in VFS when unmounting snapshots (#459)

2017-09-02 Thread Andrew Stormont
andy-js approved this pull request. LGTM. -- 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/459#pullrequestreview-60271942 -- openzfs-developer

[developer] Re: [openzfs/openzfs] 8484 Implement aggregate sum and use for arc counters (#423)

2017-09-02 Thread Paul Dagnelie
Hi @idodeclare, including the lower 8 bits of the spa and many more bits of the os (because we're dropping the lower 8 and then & with 0xFF, so we only get bits 8-15) does not decrease the quality of the hashing. It will not cause any new collisions which did not already occur, and we perform a

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

2017-09-02 Thread zettabot
You can view, comment on, or merge this pull request online at: https://github.com/openzfs/openzfs/pull/461 -- Commit Summary -- * 8139 loader: efi multiboot2 update * 8572 ccompile.h: rename __GNU_UNUSED to __unused * 8558 lwp_create() returns EAGAIN on system with more than 80K ZFS