@dankimmel pushed 1 commit.
6763852 address @yuripv's feedback
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/openzfs/openzfs/pull/397/files/6ff952e46f19a83e0ea94e4862270bff3e068b7b..6763852f08aabca25ba90a34099de091082a4ac1
@prakashsurya Thanks for that. I think I see why it failed there. That `lwb =
zil_lwb_write_start(zilw, lwb);` line after `/* write the last block out */`
was not supposed to be there. I manually merged a rather large *.rej file with
the intention of leaning on the buildbot to help me catch
yuripv commented on this pull request.
> +For full documentation of the ZFS channel program interface, see the manual
+page for
+.Xr zfs-program(1m).
+.Bl -tag -width ""
+.It Fl t Ar timeout
+Execution time limit, in milliseconds. If a channel program executes for longer
+than the provided
This continues what was started in 7801 add more by-dnode routines
by fully converting zvols to avoid unnecessary dnode_hold() calls. This saves a
small amount of CPU time and slightly improves latencies of operations on
zvols.
Ported from:
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/396#pullrequestreview-43791014
--
openzfs-developer
Merged #396.
--
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/396#event-1121797119
--
openzfs-developer
Archives:
Thanks; I'll open the RTI for this later today.
--
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/392#issuecomment-308161200
--
openzfs-developer
this fails to build; it looks like it has references to the "mooch byteswap"
feature, which we haven't upstreamed.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
jwk404 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/402#pullrequestreview-43766525
--
openzfs-developer
Looks good to me.
--
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/402#issuecomment-308139239
--
openzfs-developer
Archives:
gwr 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/402#pullrequestreview-43745502
--
openzfs-developer
Yeah, I've run into this too, i.e. when switching branches where one branch has
some new test and the other doesn't. LGTM.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
Do a simple test for Makefile existence, same as done in usr/src/lib/Makefile.
You can view, comment on, or merge this pull request online at:
https://github.com/openzfs/openzfs/pull/402
-- Commit Summary --
* 8204 Makefile changes in zfstest cannot cope with empty directories
-- File
Looks like it was unrelated after all, everything passed this time.
--
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/392#issuecomment-308071058
@lundman pushed 2 commits.
bbf9bd3 Fixes and improvements after 5th round of review
439cd9c Fixes after rebase and more review
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
15 matches
Mail list logo