Discussion:
4347 ZPL can use dmu_tx_assign(TXG_WAIT)
Christopher Siden
2013-11-22 22:48:24 UTC
Permalink
http://cr.illumos.org/~webrev/csiden/illumos-4347/

Changes by Matt Ahrens.

This fixes a lock contention problem found by Ned Bass when porting
the new write throttle to Linux.

Chris
Dan McDonald
2013-11-23 18:11:29 UTC
Permalink
Post by Christopher Siden
http://cr.illumos.org/~webrev/csiden/illumos-4347/
Changes by Matt Ahrens.
This fixes a lock contention problem found by Ned Bass when porting
the new write throttle to Linux.
I looked through vnops especially, and it looks like you hit all the cases you need covered.

I hope you get two or more other reviewers, as I'd like this in illumos-gate sooner rather than later (and with two others I feel better approving).

Dan
Boris Protopopov
2013-11-25 18:31:31 UTC
Permalink
LGTM


On Fri, Nov 22, 2013 at 5:48 PM, Christopher Siden <
Post by Christopher Siden
http://cr.illumos.org/~webrev/csiden/illumos-4347/
Changes by Matt Ahrens.
This fixes a lock contention problem found by Ned Bass when porting
the new write throttle to Linux.
Chris
-------------------------------------------
illumos-zfs
Archives: https://www.listbox.com/member/archive/182191/=now
https://www.listbox.com/member/archive/rss/182191/23052084-8e2408bc
https://www.listbox.com/member/?&
Powered by Listbox: http://www.listbox.com
--
Best regards,

Boris Protopopov
Nexenta Systems

455 El Camino Real, Santa Clara, CA 95050

[d] 408.791.3366 | [c] 978.621.6901
Skype: bprotopopov



-------------------------------------------
illumos-zfs
Archives: https://www.listbox.com/member/archive/182191/=now
RSS Feed: https://www.listbox.com/member/archive/rss/182191/23047029-187a0c8d
Modify Your Subscription: https://www.listbox.com/member/?member_id=23047029&id_secret=23047029-2e85923f
Powered by Listbox: http://www.listbox.com
Loading...