From: Trond Myklebust <trondmy@primarydata.com>
To: "aweits@rit.edu" <aweits@rit.edu>
Cc: bcodding redhat <bcodding@redhat.com>,
"anna.schumaker@netapp.com" <anna.schumaker@netapp.com>,
"linux-nfs@vger.kernel.org" <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH v8 00/11] Fix OPEN/CLOSE races
Date: Tue, 7 Nov 2017 18:35:56 +0000 [thread overview]
Message-ID: <1510079753.7834.7.camel@primarydata.com> (raw)
In-Reply-To: <m2y3ninhxs.fsf@discipline.rit.edu>
T24gVHVlLCAyMDE3LTExLTA3IGF0IDExOjU5IC0wNTAwLCBBbmRyZXcgVyBFbGJsZSB3cm90ZToN
Cj4gVHJvbmQgTXlrbGVidXN0IDx0cm9uZG15QHByaW1hcnlkYXRhLmNvbT4gd3JpdGVzOg0KPiAN
Cj4gPiBPbiBNb24sIDIwMTctMTEtMDYgYXQgMTc6NDYgLTA1MDAsIEFuZHJldyBXIEVsYmxlIHdy
b3RlOg0KPiA+ID4gVHJvbmQgTXlrbGVidXN0IDx0cm9uZC5teWtsZWJ1c3RAcHJpbWFyeWRhdGEu
Y29tPiB3cml0ZXM6DQo+ID4gPiANCj4gPiA+ID4gdjg6DQo+ID4gPiA+IC0gUmVhbGx5IGZpeCBj
b21waWxlIGlzc3VlIHdoZW4gQ09ORklHX05GU19WNF8xPW4NCj4gPiA+ID4gLSBuZnNfaW5vZGVf
ZmluZF9zdGF0ZV9hbmRfcmVjb3ZlcigpIHNob3VsZCBhbHNvIHRyeSB0byBtYXRjaA0KPiA+ID4g
PiB0aGUNCj4gPiA+ID4gICBvcGVuX3N0YXRlaWQuDQo+ID4gPiANCj4gPiA+IFNlZWluZyBhIGxv
dCBvZiBURVNUX1NUQVRFSUQncyBmb3IgdGhlIGludmFsaWQgc3RhdGVpZCBnbyBvdmVyDQo+ID4g
PiB0aGUNCj4gPiA+IHdpcmUgLSB0aGlzIGFsc28gbWFrZXMgdGhlIHNlcnZlciBoaXQgdGhpcyBx
dWl0ZSBhIGJpdDoNCj4gPiA+IA0KPiA+ID4gcHJfd2Fybl9yYXRlbGltaXRlZCgiTkZTRDogY2xp
ZW50ICVzIHRlc3Rpbmcgc3RhdGUgSUQgIg0KPiA+ID4gICAgICAgICAgICAgICAgICAgICAgICAg
IndpdGggaW5jb3JyZWN0IGNsaWVudCBJRFxuIiwgYWRkcl9zdHIpOw0KPiA+ID4gDQo+ID4gDQo+
ID4gSSdtIG5vdCBzZWVpbmcgdGhhdCBhdCBhbGwuIENhbiB5b3UgcGxlYXNlIGVsYWJvcmF0ZSBv
biB3aGljaA0KPiA+IHNlcnZlcg0KPiA+IGVycm9ycyBhcmUgdHJpZ2dlcmluZyB0aGlzPyBJJ2Qg
bm90IGV4cGVjdCB0byBldmVyIHNlZSBURVNUX1NUQVRFSUQNCj4gPiBvbg0KPiA+IGEgbm9ybWFs
IHJ1bi4NCj4gDQo+IEhvcGVmdWxseSB0aGlzIGhlbHBzLg0KPiANCj4gTm90IGNvbXBsZXRlbHkg
c3VyZSB0aGlzIGlzIGNvcnJlY3QsIGJ1dCB0aGlzIG1ha2VzIHRoZSBjbGllbnQgbm8NCj4gbG9u
Z2VyDQo+IHRlc3QgdGhlIF9pbnZhbGlkXyBzdGF0ZWlkOg0KPiANCj4gZGlmZiAtLWdpdCBhL2Zz
L25mcy9uZnM0cHJvYy5jIGIvZnMvbmZzL25mczRwcm9jLmMNCj4gaW5kZXggNmVjY2JkMWI5OGNj
Li43ZGM5MTRhNjM1NmYgMTAwNjQ0DQo+IC0tLSBhL2ZzL25mcy9uZnM0cHJvYy5jDQo+ICsrKyBi
L2ZzL25mcy9uZnM0cHJvYy5jDQo+IEBAIC0xNDM2LDYgKzE0MzYsOCBAQCBzdGF0aWMgdm9pZA0K
PiBuZnNfY2xlYXJfb3Blbl9zdGF0ZWlkX2xvY2tlZChzdHJ1Y3QgbmZzNF9zdGF0ZSAqc3RhdGUs
DQo+ICAgICAgICAgICAgICAgICBuZnNfcmVzeW5jX29wZW5fc3RhdGVpZF9sb2NrZWQoc3RhdGUp
Ow0KPiAgICAgICAgICAgICAgICAgZ290byBvdXQ7DQo+ICAgICAgICAgfQ0KPiArICAgICAgIGlm
ICh0ZXN0X2JpdChORlNfT1BFTl9TVEFURSwgJnN0YXRlLT5mbGFncykgPT0gMCkNCj4gKyAgICAg
ICAgICAgICAgIHN0YXRlaWQtPnR5cGUgPSBORlM0X0lOVkFMSURfU1RBVEVJRF9UWVBFOw0KPiAg
ICAgICAgIGlmICh0ZXN0X2JpdChORlNfREVMRUdBVEVEX1NUQVRFLCAmc3RhdGUtPmZsYWdzKSA9
PSAwKQ0KPiAgICAgICAgICAgICAgICAgbmZzNF9zdGF0ZWlkX2NvcHkoJnN0YXRlLT5zdGF0ZWlk
LCBzdGF0ZWlkKTsNCj4gICAgICAgICBuZnM0X3N0YXRlaWRfY29weSgmc3RhdGUtPm9wZW5fc3Rh
dGVpZCwgc3RhdGVpZCk7DQo+IA0KPiANCj4gVGhlIFRFU1RfU1RBVEVJRCdzIGFyZSBiZWluZyBp
c3N1ZWQgZnJvbSB0aGUgImZyZWVtZSIgcGF0aCBvZg0KPiB1cGRhdGVfb3Blbl9zdGF0ZWlkKCkg
dmlhIG5mczRfb3BlbmRhdGFfdG9fbmZzNF9zdGF0ZSgpLiBJIHB1dCBhDQo+IHRyYWNlcG9pbnQN
Cj4gdGhlcmU6DQoNCkFoLCBJIHNlZS4gQ2FuIHdlIGluIHRoYXQgY2FzZSBwbGVhc2UgcmF0aGVy
IGRvIHRoZSBmb2xsb3dpbmc/DQoNCjg8LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t
LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0NCkZyb20gZTNlZDIxYzE5NGY0MjA3Y2YzYTAwYjYyNGZl
ODZiOWE5YjE4OGMxMyBNb24gU2VwIDE3IDAwOjAwOjAwIDIwMDENCkZyb206IFRyb25kIE15a2xl
YnVzdCA8dHJvbmQubXlrbGVidXN0QHByaW1hcnlkYXRhLmNvbT4NCkRhdGU6IFR1ZSwgNyBOb3Yg
MjAxNyAxMzoxMDo0NiAtMDUwMA0KU3ViamVjdDogW1BBVENIIDEvMl0gTkZTdjQ6IG5mc19zZXRf
b3Blbl9zdGF0ZWlkIG11c3Qgbm90IHRyaWdnZXIgc3RhdGUNCiByZWNvdmVyeSBmb3IgY2xvc2Vk
IHN0YXRlDQoNCkluIG5mc19zZXRfb3Blbl9zdGF0ZWlkX2xvY2tlZCwgd2UgbXVzdCBpZ25vcmUg
c3RhdGVpZHMgZnJvbSBjbG9zZWQgc3RhdGUuDQoNClJlcG9ydGVkLWJ5OiBBbmRyZXcgVyBFbGJs
ZSA8YXdlaXRzQHJpdC5lZHU+DQpTaWduZWQtb2ZmLWJ5OiBUcm9uZCBNeWtsZWJ1c3QgPHRyb25k
Lm15a2xlYnVzdEBwcmltYXJ5ZGF0YS5jb20+DQotLS0NCiBmcy9uZnMvbmZzNHByb2MuYyB8IDMg
KystDQogMSBmaWxlIGNoYW5nZWQsIDIgaW5zZXJ0aW9ucygrKSwgMSBkZWxldGlvbigtKQ0KDQpk
aWZmIC0tZ2l0IGEvZnMvbmZzL25mczRwcm9jLmMgYi9mcy9uZnMvbmZzNHByb2MuYw0KaW5kZXgg
ZDUzNmQ0ZWQ5OTBmLi4wYWE4NDYxZTczNDEgMTAwNjQ0DQotLS0gYS9mcy9uZnMvbmZzNHByb2Mu
Yw0KKysrIGIvZnMvbmZzL25mczRwcm9jLmMNCkBAIC0xNTQxLDcgKzE1NDEsOCBAQCBzdGF0aWMg
dm9pZCBuZnNfc2V0X29wZW5fc3RhdGVpZF9sb2NrZWQoc3RydWN0IG5mczRfc3RhdGUgKnN0YXRl
LA0KIAkJd3JpdGVfc2VxbG9jaygmc3RhdGUtPnNlcWxvY2spOw0KIAl9DQogDQotCWlmICghbmZz
NF9zdGF0ZWlkX21hdGNoX290aGVyKHN0YXRlaWQsICZzdGF0ZS0+b3Blbl9zdGF0ZWlkKSkgew0K
KwlpZiAodGVzdF9iaXQoTkZTX09QRU5fU1RBVEUsICZzdGF0ZS0+ZmxhZ3MpICYmDQorCSAgICAh
bmZzNF9zdGF0ZWlkX21hdGNoX290aGVyKHN0YXRlaWQsICZzdGF0ZS0+b3Blbl9zdGF0ZWlkKSkg
ew0KIAkJbmZzNF9zdGF0ZWlkX2NvcHkoZnJlZW1lLCAmc3RhdGUtPm9wZW5fc3RhdGVpZCk7DQog
CQluZnNfdGVzdF9hbmRfY2xlYXJfYWxsX29wZW5fc3RhdGVpZChzdGF0ZSk7DQogCX0NCi0tIA0K
Mi4xMy42DQoNCi0tIA0KVHJvbmQgTXlrbGVidXN0DQpMaW51eCBORlMgY2xpZW50IG1haW50YWlu
ZXIsIFByaW1hcnlEYXRhDQp0cm9uZC5teWtsZWJ1c3RAcHJpbWFyeWRhdGEuY29tDQo=
next prev parent reply other threads:[~2017-11-07 18:35 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-06 20:28 [PATCH v8 00/11] Fix OPEN/CLOSE races Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 01/11] NFSv4: Fix OPEN / CLOSE race Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 02/11] NFSv4: Add a tracepoint to document open stateid updates Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 03/11] NFSv4: Fix open create exclusive when the server reboots Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 04/11] NFS: Fix a typo in nfs_rename() Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 05/11] NFSv4: Retry CLOSE and DELEGRETURN on NFS4ERR_OLD_STATEID Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 06/11] NFSv4: Don't try to CLOSE if the stateid 'other' field has changed Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 07/11] pNFS: Retry NFS4ERR_OLD_STATEID errors in layoutreturn-on-close Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 08/11] NFSv4: Retry NFS4ERR_OLD_STATEID errors in layoutreturn Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 09/11] NFSv4: cleanup nfs4_close_done Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 10/11] NFSv4: Clean up nfs4_delegreturn_done Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 11/11] NFSv4: Check the open stateid when searching for expired state Trond Myklebust
2017-11-06 22:46 ` [PATCH v8 00/11] Fix OPEN/CLOSE races Andrew W Elble
2017-11-06 22:50 ` Trond Myklebust
2017-11-06 23:13 ` Andrew W Elble
2017-11-07 16:59 ` Andrew W Elble
2017-11-07 18:35 ` Trond Myklebust [this message]
2017-11-07 19:00 ` Trond Myklebust
2017-11-07 23:03 ` Andrew W Elble
2017-11-07 23:44 ` Trond Myklebust
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=1510079753.7834.7.camel@primarydata.com \
--to=trondmy@primarydata.com \
--cc=anna.schumaker@netapp.com \
--cc=aweits@rit.edu \
--cc=bcodding@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).