From: Trond Myklebust <trondmy@primarydata.com>
To: "chuck.lever@oracle.com" <chuck.lever@oracle.com>
Cc: "linux-nfs@vger.kernel.org" <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH v2 00/28] Reducing inode->i_lock contention in writebacks
Date: Thu, 3 Aug 2017 15:39:47 +0000 [thread overview]
Message-ID: <1501774784.3680.1.camel@primarydata.com> (raw)
In-Reply-To: <26E54933-A2AB-4F99-9317-0BF300B36A5D@oracle.com>
T24gVGh1LCAyMDE3LTA4LTAzIGF0IDExOjA0IC0wNDAwLCBDaHVjayBMZXZlciB3cm90ZToNCj4g
PiBPbiBBdWcgMywgMjAxNywgYXQgOTo0NCBBTSwgVHJvbmQgTXlrbGVidXN0IDx0cm9uZC5teWts
ZWJ1c3RAcHJpbWFyDQo+ID4geWRhdGEuY29tPiB3cm90ZToNCj4gPiANCj4gPiBDaHVjayBMZXZl
ciBoYXMgcHJlc2VudGVkIG1lYXN1cmVtZW50cywgdGhhdCB3b3VsZCBhcHBlYXIgdG8NCj4gPiBp
bmRpY2F0ZSB0aGF0DQo+ID4gdGhlIGlub2RlLT5pX2xvY2sgY2FuIGJlIGEgc291cmNlIG9mIGNv
bnRlbnRpb24gd2hlbiBkb2luZw0KPiA+IHdyaXRlYmFjayB1c2luZw0KPiA+IGhpZ2ggcGVyZm9y
bWFuY2UgbmV0d29ya3MuIFRoaXMgcGF0Y2ggc2VyaWVzIGlzIHRoZXJlZm9yZSBhbg0KPiA+IGF0
dGVtcHQgdG8NCj4gPiBhZGRyZXNzIHRoZSBzb3VyY2VzIG9mIGNvbnRlbnRpb24gdGhhdCBoaXMg
bWVhc3VyZW1lbnRzIHBvaW50ZWQgdG8uDQo+ID4gDQo+ID4gQWNjb3JkaW5nIHRvIENodWNrJ3Mg
dGFibGUsIHRoZSBtYWluIHNvdXJjZXMgb2YgY29udGVudGlvbiBhcHBlYXINCj4gPiB0bw0KPiA+
IGJlIGluIG5mc19mbHVzaF9pbmNvbXBhdGlibGUoKSwgbmZzX2xvY2tfYW5kX2pvaW5fcmVxdWVz
dHMoKSBhbmQNCj4gPiBuZnNfdXBkYXRlcGFnZSgpLiBNb3N0IG9mIHRoaXMgcGF0Y2ggc2VyaWVz
IGZvY3VzZXMgb24gZml4aW5nDQo+ID4gbmZzX2xvY2tfYW5kX2pvaW5fcmVxdWVzdHMoKSwgc2lu
Y2UgaXQgaG9sZHMgdGhlIGlub2RlIGxvY2ssIHdoaWxlDQo+ID4gdGFraW5nIGxvY2tzIG9uIHRo
ZSBwYWdlIGdyb3VwIGFuZCB0aGUgc3ViLXJlcXVlc3RzIHRoZW1zZWx2ZXMsDQo+ID4gcm9sbGlu
ZyB0aGVtIGFsbCBiYWNrIGlmIGV2ZXIgdGhlcmUgaXMgY29udGVudGlvbi4NCj4gPiBCeSBub3Rp
bmcgYSBmZXcgc2ltcGxlIHJ1bGVzIHRoYXQgYXJlIG1vc3RseSBhbHJlYWR5IGluIHBsYWNlLCB3
ZQ0KPiA+IGNhbiBzaW1wbGlmeSB0aGF0IGxvY2tpbmcgdG8gZW5zdXJlIHRoYXQgd2Ugb25seSBo
YXZlIHRvIGtlZXAgdGhlDQo+ID4gc3BpbiBsb2NrIHdoaWxlIHdlJ3JlIGRlcmVmZXJlbmNpbmcg
YW5kIGxvY2tpbmcgdGhlIGhlYWQgcGFnZQ0KPiA+IHBvaW50ZXINCj4gPiB0aGF0IGlzIHN0b3Jl
ZCBpbiBwYWdlX3ByaXZhdGUocGFnZSkuDQo+ID4gDQo+ID4gQWxvbmcgdGhlIHdheSwgdGhlIHBh
dGNoZXMgYWxzbyBzaW1wbGlmeSB0aGUgY29kZSBhIGxpdHRsZSwgYW5kIGZpeA0KPiA+IGEgbnVt
YmVyIG9mIHN1YnRsZSByYWNlcyB3aGljaCBtYWlubHkgb2NjdXIgd2hlbiB5b3Ugc2V0IHdzaXpl
IHRvDQo+ID4gc29tZSB2YWx1ZSBzbWFsbGVyIHRoYW4gdGhlIHBhZ2Ugc2l6ZS4gVGhlIG1vc3Qg
bm90YWJsZSBzdWNoIHJhY2UNCj4gPiBvY2N1cnMgYmV0d2VlbiBuZnNfbG9ja19hbmRfam9pbl9y
ZXF1ZXN0cygpIGFuZA0KPiA+IG5mc19wYWdlX2dyb3VwX2Rlc3Ryb3koKSwNCj4gPiBhbmQgY291
bGQgbGVhZCB0byBhIGRvdWJsZSBmcmVlKCkgb2Ygc29tZSBvZiB0aGUgc3ViLXJlcXVlc3RzLg0K
PiA+IA0KPiA+IEZpbmFsbHksIHRoZXJlIGFyZSAyIHBhdGNoZXMgdGFja2VkIG9udG8gdGhlIGVu
ZCBvZiB0aGUgc2VyaWVzIHRoYXQNCj4gPiBhdHRlbXB0IHRvIGltcHJvdmUgdGhlIHRocm90dGxp
bmcgb2Ygd3JpdGVzIHdoZW4gdGhlIFJQQyBsYXllciBpcw0KPiA+IGNvbmdlc3RlZC4gVGhleSBk
byBzbyBieSBmb3JjaW5nIGVhY2ggY2FsbGVyIG9mIG5mc19pbml0aWF0ZV9wZ2lvKCkNCj4gPiB0
byB3YWl0IHVudGlsIHRoZSByZXF1ZXN0IGlzIGJlaW5nIHRyYW5zbWl0dGVkLiBUaGUgZXhwZWN0
YXRpb24gaXMNCj4gPiB0aGF0IHRoaXMgbWlnaHQgaGVscCBpbXByb3ZlIGxhdGVuY2llcyB3aGVu
IHRoZXJlIGFyZSBzZXZlcmFsDQo+ID4gcHJvY2Vzc2VzDQo+ID4gY29tcGV0aW5nIGZvciBhY2Nl
c3MgdG8gdGhlIFJQQyB0cmFuc3BvcnQuDQo+ID4gDQo+ID4gdjI6IEZ1cnRoZXIgcGF0Y2hlcyB0
byBlbGltaW5hdGUgdXNlIG9mIGlub2RlLT5pX2xvY2sgdG8gcHJvdGVjdA0KPiA+IHRoZQ0KPiA+
ICAgIHBhZ2UtPnByaXZhdGUgY29udGVudHMuDQo+ID4gICAgUHJvdGVjdCB0aGUgKGxhcmdlISkg
bGlzdCBvZiB1bnN0YWJsZSBwYWdlcyB1c2luZyBhIGRlZGljYXRlZA0KPiA+ICAgIE5GU19JKGlu
b2RlKS0+Y29tbWl0X211dGV4IGluc3RlYWQgb2YgdGhlIGlub2RlLT5pX2xvY2suDQo+ID4gDQo+
ID4gVHJvbmQgTXlrbGVidXN0ICgyOCk6DQo+ID4gIE5GUzogU2ltcGxpZnkgcGFnZSB3cml0ZWJh
Y2sNCj4gPiAgTkZTOiBSZWR1Y2UgbG9jayBjb250ZW50aW9uIGluIG5mc19wYWdlX2ZpbmRfaGVh
ZF9yZXF1ZXN0KCkNCj4gPiAgTkZTOiBSZWR1Y2UgbG9jayBjb250ZW50aW9uIGluIG5mc190cnlf
dG9fdXBkYXRlX3JlcXVlc3QoKQ0KPiA+ICBORlM6IEVuc3VyZSB3ZSBhbHdheXMgZGVyZWZlcmVu
Y2UgdGhlIHBhZ2UgaGVhZCBsYXN0DQo+ID4gIE5GUzogRml4IGEgcmVmZXJlbmNlIGFuZCBsb2Nr
IGxlYWsgaW4gbmZzX2xvY2tfYW5kX2pvaW5fcmVxdWVzdHMoKQ0KPiA+ICBORlM6IEZpeCBhbiBB
QkJBIGlzc3VlIGluIG5mc19sb2NrX2FuZF9qb2luX3JlcXVlc3RzKCkNCj4gPiAgTkZTOiBEb24n
dCBjaGVjayByZXF1ZXN0IG9mZnNldCBhbmQgc2l6ZSB3aXRob3V0IGhvbGRpbmcgYSBsb2NrDQo+
ID4gIE5GUzogRG9uJ3QgdW5sb2NrIHdyaXRlYmFja3MgYmVmb3JlIGRlY2xhcmluZyBQR19XQl9F
TkQNCj4gPiAgTkZTOiBGaXggdGhlIGlub2RlIHJlcXVlc3QgYWNjb3VudGluZyB3aGVuIHBhZ2Vz
IGhhdmUgc3VicmVxdWVzdHMNCj4gPiAgTkZTOiBUZWFjaCBuZnNfdHJ5X3RvX3VwZGF0ZV9yZXF1
ZXN0KCkgdG8gZGVhbCB3aXRoIHJlcXVlc3QNCj4gPiAgICBwYWdlX2dyb3Vwcw0KPiA+ICBORlM6
IFJlbW92ZSBwYWdlIGdyb3VwIGxpbWl0IGluIG5mc19mbHVzaF9pbmNvbXBhdGlibGUoKQ0KPiA+
ICBORlM6IFJlZHVjZSBpbm9kZS0+aV9sb2NrIGNvbnRlbnRpb24gaW4NCj4gPiBuZnNfbG9ja19h
bmRfam9pbl9yZXF1ZXN0cygpDQo+ID4gIE5GUzogRnVydGhlciBvcHRpbWlzZSBuZnNfbG9ja19h
bmRfam9pbl9yZXF1ZXN0cygpDQo+ID4gIE5GUzogRml4IG5mc19wYWdlX2dyb3VwX2Rlc3Ryb3ko
KSBhbmQgbmZzX2xvY2tfYW5kX2pvaW5fcmVxdWVzdHMoKQ0KPiA+ICAgIHJhY2UgY2FzZXMNCj4g
PiAgTkZTOiBSZW1vdmUgbmZzX3BhZ2VfZ3JvdXBfY2xlYXJfYml0cygpDQo+ID4gIE5GUzogUmVt
b3ZlIHVudXNlIGZ1bmN0aW9uIG5mc19wYWdlX2dyb3VwX2xvY2tfd2FpdCgpDQo+ID4gIE5GUzog
UmVtb3ZlIHVudXNlZCBwYXJhbWV0ZXIgZnJvbSBuZnNfcGFnZV9ncm91cF9sb2NrKCkNCj4gPiAg
TkZTOiBGaXggdXAgbmZzX3BhZ2VfZ3JvdXBfY292ZXJzX3BhZ2UoKQ0KPiA+ICBORlN2NDogQ29u
dmVydCBuZnNfbG9ja19hbmRfam9pbl9yZXF1ZXN0cygpIHRvIHVzZQ0KPiA+ICAgIG5mc19wYWdl
X2ZpbmRfaGVhZF9yZXF1ZXN0KCkNCj4gPiAgTkZTOiBSZWZhY3RvciBuZnNfcGFnZV9maW5kX2hl
YWRfcmVxdWVzdCgpDQo+ID4gIE5GU3Y0OiBVc2UgYSBtdXRleCB0byBwcm90ZWN0IHRoZSBwZXIt
aW5vZGUgY29tbWl0IGxpc3RzDQo+ID4gIE5GUzogVXNlIGFuIGF0b21pY19sb25nX3QgdG8gY291
bnQgdGhlIG51bWJlciBvZiByZXF1ZXN0cw0KPiA+ICBORlM6IFVzZSBhbiBhdG9taWNfbG9uZ190
IHRvIGNvdW50IHRoZSBudW1iZXIgb2YgY29tbWl0cw0KPiA+ICBORlM6IFN3aXRjaCB0byB1c2lu
ZyBtYXBwaW5nLT5wcml2YXRlX2xvY2sgZm9yIHBhZ2Ugd3JpdGViYWNrDQo+ID4gbG9va3Vwcy4N
Cj4gPiAgTkZTdjQvcG5mczogUmVwbGFjZSBwbmZzX3B1dF9sc2VnX2xvY2tlZCgpIHdpdGggcG5m
c19wdXRfbHNlZygpDQo+ID4gIE5GUzogV2FpdCBmb3IgcmVxdWVzdHMgdGhhdCBhcmUgbG9ja2Vk
IG9uIHRoZSBjb21taXQgbGlzdA0KPiA+ICBTVU5SUEM6IEFkZCBhIGZ1bmN0aW9uIHRvIGFsbG93
IHdhaXRpbmcgZm9yIFJQQyB0cmFuc21pc3Npb24NCj4gPiAgTkZTOiBUaHJvdHRsZSBJL08gdG8g
dGhlIE5GUyBzZXJ2ZXINCj4gPiANCj4gPiBmcy9uZnMvY2FsbGJhY2tfcHJvYy5jICAgICAgIHwg
ICAyICstDQo+ID4gZnMvbmZzL2RlbGVnYXRpb24uYyAgICAgICAgICB8ICAgMiArLQ0KPiA+IGZz
L25mcy9kaXJlY3QuYyAgICAgICAgICAgICAgfCAgIDQgKy0NCj4gPiBmcy9uZnMvaW5vZGUuYyAg
ICAgICAgICAgICAgIHwgIDEwICstDQo+ID4gZnMvbmZzL3BhZ2VsaXN0LmMgICAgICAgICAgICB8
ICA3MCArKystLS0tDQo+ID4gZnMvbmZzL3BuZnMuYyAgICAgICAgICAgICAgICB8ICA0MSAtLS0t
DQo+ID4gZnMvbmZzL3BuZnMuaCAgICAgICAgICAgICAgICB8ICAgMiAtDQo+ID4gZnMvbmZzL3Bu
ZnNfbmZzLmMgICAgICAgICAgICB8ICAzNyArKy0tDQo+ID4gZnMvbmZzL3dyaXRlLmMgICAgICAg
ICAgICAgICB8IDQ0MCArKysrKysrKysrKysrKysrKysrKy0tLS0tLS0tLS0NCj4gPiAtLS0tLS0t
LS0tLS0tDQo+ID4gaW5jbHVkZS9saW51eC9uZnNfZnMuaCAgICAgICB8ICAgNSArLQ0KPiA+IGlu
Y2x1ZGUvbGludXgvbmZzX3BhZ2UuaCAgICAgfCAgIDMgKy0NCj4gPiBpbmNsdWRlL2xpbnV4L25m
c194ZHIuaCAgICAgIHwgICAyICstDQo+ID4gaW5jbHVkZS9saW51eC9zdW5ycGMvc2NoZWQuaCB8
ICAgMyArDQo+ID4gbmV0L3N1bnJwYy9zY2hlZC5jICAgICAgICAgICB8ICAyMiArKysNCj4gPiBu
ZXQvc3VucnBjL3hwcnQuYyAgICAgICAgICAgIHwgICA2ICsNCj4gPiAxNSBmaWxlcyBjaGFuZ2Vk
LCAyOTUgaW5zZXJ0aW9ucygrKSwgMzU0IGRlbGV0aW9ucygtKQ0KPiANCj4gSGV5IFRyb25kLCBk
byB5b3UgaGF2ZSBhIHRvcGljIGJyYW5jaCBzb21ld2hlcmUgSSBjYW4gcHVsbCB0aGlzPw0KPiAN
Cg0KWWVwLiBJJ3ZlIHB1c2hlZCB0aGUgIndyaXRlYmFjayIgdG9waWMgYnJhbmNoIHRvIGdpdC5s
aW51eC5vcmcuIEl0IGhhcw0KMSBleHRyYSBkZWJ1Z2dpbmcgY29tbWl0IGF0dGFjaGVkIHRvIHRo
ZSBlbmQsIGJ1dCBJIGRvbid0IGV4cGVjdCB0aGF0DQp0byBib3RoZXIgeW91Lg0KDQotLSANClRy
b25kIE15a2xlYnVzdA0KTGludXggTkZTIGNsaWVudCBtYWludGFpbmVyLCBQcmltYXJ5RGF0YQ0K
dHJvbmQubXlrbGVidXN0QHByaW1hcnlkYXRhLmNvbQ0K
prev parent reply other threads:[~2017-08-03 15:39 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-03 13:44 [PATCH v2 00/28] Reducing inode->i_lock contention in writebacks Trond Myklebust
2017-08-03 13:44 ` [PATCH v2 01/28] NFS: Simplify page writeback Trond Myklebust
2017-08-03 13:44 ` [PATCH v2 02/28] NFS: Reduce lock contention in nfs_page_find_head_request() Trond Myklebust
2017-08-03 13:44 ` [PATCH v2 03/28] NFS: Reduce lock contention in nfs_try_to_update_request() Trond Myklebust
2017-08-03 13:44 ` [PATCH v2 04/28] NFS: Ensure we always dereference the page head last Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 05/28] NFS: Fix a reference and lock leak in nfs_lock_and_join_requests() Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 06/28] NFS: Fix an ABBA issue " Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 07/28] NFS: Don't check request offset and size without holding a lock Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 08/28] NFS: Don't unlock writebacks before declaring PG_WB_END Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 09/28] NFS: Fix the inode request accounting when pages have subrequests Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 10/28] NFS: Teach nfs_try_to_update_request() to deal with request page_groups Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 11/28] NFS: Remove page group limit in nfs_flush_incompatible() Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 12/28] NFS: Reduce inode->i_lock contention in nfs_lock_and_join_requests() Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 13/28] NFS: Further optimise nfs_lock_and_join_requests() Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 14/28] NFS: Fix nfs_page_group_destroy() and nfs_lock_and_join_requests() race cases Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 15/28] NFS: Remove nfs_page_group_clear_bits() Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 16/28] NFS: Remove unuse function nfs_page_group_lock_wait() Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 17/28] NFS: Remove unused parameter from nfs_page_group_lock() Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 18/28] NFS: Fix up nfs_page_group_covers_page() Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 19/28] NFSv4: Convert nfs_lock_and_join_requests() to use nfs_page_find_head_request() Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 20/28] NFS: Refactor nfs_page_find_head_request() Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 21/28] NFSv4: Use a mutex to protect the per-inode commit lists Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 22/28] NFS: Use an atomic_long_t to count the number of requests Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 23/28] NFS: Use an atomic_long_t to count the number of commits Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 24/28] NFS: Switch to using mapping->private_lock for page writeback lookups Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 25/28] NFSv4/pnfs: Replace pnfs_put_lseg_locked() with pnfs_put_lseg() Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 26/28] NFS: Wait for requests that are locked on the commit list Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 27/28] SUNRPC: Add a function to allow waiting for RPC transmission Trond Myklebust
2017-08-03 13:45 ` [PATCH v2 28/28] NFS: Throttle I/O to the NFS server Trond Myklebust
2017-08-03 15:04 ` [PATCH v2 00/28] Reducing inode->i_lock contention in writebacks Chuck Lever
2017-08-03 15:39 ` Trond Myklebust [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1501774784.3680.1.camel@primarydata.com \
--to=trondmy@primarydata.com \
--cc=chuck.lever@oracle.com \
--cc=linux-nfs@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.