Discussion:
Review Request 101: 5174 add sdt probe for blocked read in dbuf_read()
Matthew Ahrens via illumos-zfs
2014-09-17 02:46:54 UTC
Permalink
---------- Forwarded message ----------
From: Matthew Ahrens <***@csiden.org>
Date: Tue, Sep 16, 2014 at 7:45 PM
Subject: Review Request 101: 5174 add sdt probe for blocked read in
dbuf_read()
To: Christopher Siden <***@delphix.com>
Cc: OpenZFS Developer Mailing List <***@open-zfs.org>, Matthew Ahrens
<***@delphix.com>


This is an automatically generated e-mail. To reply, visit:
https://reviews.csiden.org/r/101/
Review request for OpenZFS Developer Mailing List and Christopher Siden.
By Matthew Ahrens.
*Bugs: * 5174 <https://www.illumos.org/projects/illumos-gate//issues/5174>
*Repository: * illumos-gate
Description

5174 add sdt probe for blocked read in dbuf_read()

Reviewed by: Matthew Ahrens <***@delphix.com>

Reviewed by: Basil Crow <***@delphix.com>

Original author: Adam H. Leventhal

Testing

manual testing that the probe fires

Diffs

- usr/src/uts/common/fs/zfs/dbuf.c
(4b0ec01a02cb49bee849fa7e038c26b245fc5e23)

View Diff <https://reviews.csiden.org/r/101/diff/>



-------------------------------------------
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
Garrett D'Amore via illumos-zfs
2014-09-17 16:12:48 UTC
Permalink
Looks good to me.

Sent from my iPhone
Post by Matthew Ahrens via illumos-zfs
---------- Forwarded message ----------
Date: Tue, Sep 16, 2014 at 7:45 PM
Subject: Review Request 101: 5174 add sdt probe for blocked read in dbuf_read()
This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/101/
Review request for OpenZFS Developer Mailing List and Christopher Siden.
By Matthew Ahrens.
Bugs: 5174
Repository: illumos-gate
Description
5174 add sdt probe for blocked read in dbuf_read()
Original author: Adam H. Leventhal
Testing
manual testing that the probe fires
Diffs
usr/src/uts/common/fs/zfs/dbuf.c (4b0ec01a02cb49bee849fa7e038c26b245fc5e23)
View Diff
illumos-zfs | Archives | Modify Your Subscription
-------------------------------------------
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...